Wednesday, 2014-04-30

*** mattgriffin has joined #openstack-trove00:18
*** yidclare has quit IRC00:28
*** jmontemayor has quit IRC00:31
*** IvanZ has quit IRC00:48
*** mattgriffin has quit IRC00:48
juiceamcrn: around?00:51
*** mattgriffin has joined #openstack-trove00:56
dougshelley66juice: I'm no amcrn but I'm here :)01:05
juiceha ha - thanks dougshelley66.  I was just responding to an earlier comment he made about the commit.  I am not sure what he meant01:08
dougshelley66i believe it had something to do with one commit removing the "redstack rd-client" etc from the readme and your commit putting them back?01:11
dougshelley66but i could be wrong...01:12
openstackgerritJustin Hopper proposed a change to openstack/trove-integration: Remove the iptables workaround  https://review.openstack.org/8455501:18
SlickNikjuice: he meant that your rebase is messed up.01:20
juiceslicknik: I understand that but not why01:20
juicegit seemed pretty happy with what I did01:20
*** eguz has quit IRC01:22
SlickNikI think he was surprised that you added a couple of lines in the README.md file, when that section had been removed by the rebase.01:22
SlickNikMaybe he didn't realize that you had modified the content? (i.e. updated the "Using Openstack Clients" section)01:23
juiceI can remove the section completely - what do you think01:25
juicenot trying to pit your word against amcrn :)01:25
SlickNikI like it; but would move I'd move it to another changeset cause it's unrelated.01:26
juiceokie doke01:26
SlickNikThanks juice.01:27
openstackgerritJustin Hopper proposed a change to openstack/trove-integration: Remove the iptables workaround  https://review.openstack.org/8455501:28
*** nosnos has joined #openstack-trove01:35
*** ramishra has joined #openstack-trove01:38
*** erkules has quit IRC01:50
*** amytron has quit IRC01:50
*** robertmyers has joined #openstack-trove01:55
*** haomaiwa_ has joined #openstack-trove01:57
*** haomaiw__ has quit IRC02:00
*** ramishra has quit IRC02:04
*** robertmyers has quit IRC02:04
amrithmat-lowery, did my comments earlier make sense?02:05
*** robertmyers has joined #openstack-trove02:05
*** ramishra has joined #openstack-trove02:05
*** mattgriffin has quit IRC02:11
*** haomaiwa_ has quit IRC02:13
*** haomaiwang has joined #openstack-trove02:13
*** michael-yu has quit IRC02:17
*** ramishra has quit IRC02:19
*** ramishra has joined #openstack-trove02:20
*** harlowja is now known as harlowja_away02:27
openstackgerritDoug Shelley proposed a change to openstack/trove: Make config attach/detach language consistent  https://review.openstack.org/9121902:29
openstackgerritDoug Shelley proposed a change to openstack/trove: Make config attach/detach language consistent  https://review.openstack.org/9121902:30
*** coolsvap|afk is now known as coolsvap02:30
*** Barker has joined #openstack-trove02:31
openstackgerritNikhil Manchanda proposed a change to openstack/trove: Improve help strings  https://review.openstack.org/9116602:34
*** haomaiw__ has joined #openstack-trove02:35
*** haomaiwang has quit IRC02:38
*** ramishra has quit IRC02:44
openstackgerritOpenStack Proposal Bot proposed a change to openstack/python-troveclient: Updated from global requirements  https://review.openstack.org/9124602:47
*** michael-yu has joined #openstack-trove02:47
*** Barker has quit IRC02:47
openstackgerritOpenStack Proposal Bot proposed a change to openstack/trove: Updated from global requirements  https://review.openstack.org/9125202:48
*** NehaV has joined #openstack-trove02:50
openstackgerritCraig Vyvial proposed a change to openstack/trove: Datastore Configuration Parameters stored in db  https://review.openstack.org/7985002:53
*** jasonb365 has joined #openstack-trove02:53
*** sbfox has quit IRC02:54
*** matsuhashi has quit IRC02:56
*** matsuhashi has joined #openstack-trove02:57
*** ramishra has joined #openstack-trove02:57
*** sbfox has joined #openstack-trove03:05
*** amcrn has joined #openstack-trove03:07
*** michael-yu has quit IRC03:09
*** mattgriffin has joined #openstack-trove03:12
*** robertmyers has quit IRC03:22
*** robertmyers has joined #openstack-trove03:22
*** robertmyers has quit IRC03:28
*** freyes has quit IRC03:36
*** coolsvap is now known as coolsvap|afk03:42
openstackgerritCraig Vyvial proposed a change to openstack/trove: Datastore Configuration Parameters stored in db  https://review.openstack.org/7985003:44
*** erkules has joined #openstack-trove03:46
*** NehaV has quit IRC03:47
*** jasonb365 has quit IRC03:48
*** eghobo has joined #openstack-trove03:49
*** haomaiw__ has quit IRC03:51
*** haomaiwang has joined #openstack-trove03:52
*** coolsvap|afk is now known as coolsvap03:54
*** shakamunyi has quit IRC04:05
*** matsuhashi has quit IRC04:10
*** michael-yu has joined #openstack-trove04:18
*** nosnos has quit IRC04:26
*** sbfox has quit IRC04:26
*** eghobo has quit IRC04:32
*** Barker has joined #openstack-trove04:47
*** Barker has quit IRC04:49
*** sbfox has joined #openstack-trove04:49
*** haomaiw__ has joined #openstack-trove04:53
*** haomaiwang has quit IRC04:53
*** eghobo has joined #openstack-trove04:55
*** shivam_ has left #openstack-trove05:02
*** shivam_ has joined #openstack-trove05:02
*** matsuhashi has joined #openstack-trove05:04
*** nosnos has joined #openstack-trove05:16
*** nosnos has quit IRC05:22
*** nosnos has joined #openstack-trove05:23
*** michael-yu has quit IRC05:28
*** matsuhashi has quit IRC05:30
*** matsuhashi has joined #openstack-trove05:30
*** matsuhashi has quit IRC05:36
*** matsuhashi has joined #openstack-trove05:36
openstackgerritCraig Vyvial proposed a change to openstack/trove: configuration datastore paramters description  https://review.openstack.org/9126705:37
openstackgerritCraig Vyvial proposed a change to openstack/python-troveclient: descriptions on datastore configuration parameters  https://review.openstack.org/9126805:37
*** yogeshmehra has joined #openstack-trove05:38
*** ramishra has quit IRC05:44
*** amcrn has quit IRC05:54
*** yogeshmehra has quit IRC05:57
openstackgerritA change was merged to openstack/trove: Improve help strings  https://review.openstack.org/9116605:58
*** ramishra has joined #openstack-trove06:03
*** haomaiw__ has quit IRC06:07
*** haomaiwang has joined #openstack-trove06:07
*** yogeshmehra has joined #openstack-trove06:07
openstackgerritOpenStack Proposal Bot proposed a change to openstack/trove: Imported Translations from Transifex  https://review.openstack.org/9127206:21
*** yogeshmehra has quit IRC06:24
*** ramishra has quit IRC06:27
*** ramishra has joined #openstack-trove06:30
*** mattgriffin has quit IRC06:43
*** flaper87|afk is now known as flaper8706:49
*** flaper87 is now known as flaper87|afk06:51
*** TomasR_ has quit IRC06:53
*** michael-yu has joined #openstack-trove06:53
*** flaper87|afk is now known as flaper8707:03
*** matsuhashi has quit IRC07:08
*** yogeshmehra has joined #openstack-trove07:09
*** matsuhashi has joined #openstack-trove07:11
*** matsuhashi has quit IRC07:17
*** michael-yu has quit IRC07:17
*** matsuhashi has joined #openstack-trove07:17
*** michael-yu has joined #openstack-trove07:20
*** sbfox has quit IRC07:20
*** michael-yu has quit IRC07:20
*** sgotliv has joined #openstack-trove07:23
*** michael-yu has joined #openstack-trove07:26
openstackgerritShashank Gupta proposed a change to openstack/trove: Corrected the assert message in test_dbaas.py  https://review.openstack.org/8981507:29
*** eghobo has quit IRC07:30
*** amcrn has joined #openstack-trove07:30
*** sgotliv_ has joined #openstack-trove07:42
*** coolsvap is now known as coolsvap|afk07:44
*** michael-yu has quit IRC07:44
*** sgotliv has quit IRC07:44
*** haomai___ has joined #openstack-trove07:45
*** haomaiwang has quit IRC07:48
*** coolsvap|afk is now known as coolsvap07:51
*** zigo_ is now known as zigo07:55
*** yogeshmehra has quit IRC08:15
*** yogeshmehra has joined #openstack-trove08:16
*** ihrachyshka has joined #openstack-trove08:17
*** yogeshmehra has quit IRC08:20
*** sgotliv_ has quit IRC08:21
*** sgotliv_ has joined #openstack-trove08:33
*** coolsvap is now known as coolsvap|afk08:54
*** haomai___ has quit IRC08:58
*** haomaiwa_ has joined #openstack-trove08:58
*** haomaiwa_ has quit IRC09:49
*** haomaiwang has joined #openstack-trove09:49
*** denis_makogon has joined #openstack-trove09:50
*** coolsvap|afk is now known as coolsvap09:55
*** sgotliv_ has quit IRC10:02
*** sgotliv_ has joined #openstack-trove10:04
*** sgotliv_ has quit IRC10:19
*** nosnos has quit IRC10:21
*** sgotliv has joined #openstack-trove10:26
*** haomai___ has joined #openstack-trove10:28
*** nosnos has joined #openstack-trove10:29
*** haomaiwang has quit IRC10:31
*** coolsvap is now known as coolsvap|afk10:33
*** SushillKM has joined #openstack-trove10:47
*** openstackgerrit has quit IRC10:51
*** TomasR has joined #openstack-trove10:53
TomasRHi, guys. I got Trove guest agent to go into 'Active' by manually injecting guest_id into /etc/trove/trove-guestagent.conf. Now I want to fix the problem and get information injected automatically into /etc/guest_info. I turned DEBUG mode on in Nova.10:54
TomasRI don't see any error and I do see an attempt to inject info into /etc/guest_info10:54
TomasRI added the log here: https://gist.github.com/anonymous/ea8e38c6695e14099f1d10:54
denis_makogonTomasR, trove doesn't inject guest_id into trove-guestagent.conf/sample, it goes to guest_info11:16
TomasRdenis_makagon, I looked at the log file and I found this:11:17
*** SushillKM has quit IRC11:17
TomasR"personality": [{"path": "/etc/guest_info", "contents": "W0RFRkFVTFRdCmd1ZXN0X2lkPTA2YjdmNDZhLWVmMjQtNDgwOS1hZmFmLTZhMTk1MzhmN2ExYgpkYXRhc3RvcmVfbWFuYWdlcj1teXNxbAp0ZW5hbnRfaWQ9N2QzNTk2MDA4MTUxNDRlZWFjMjY4YTEwZWZhYmE2MmIK"}]}} _process_stack /usr/lib/python2.7/dist-packages/nova/api/openstack/wsgi.py:94011:17
TomasRdecoding the info:11:17
TomasR[DEFAULT] guest_id=06b7f46a-ef24-4809-afaf-6a19538f7a1b datastore_manager=mysql tenant_id=7d359600815144eeac268a10efaba62b11:17
denis_makogonyes11:18
denis_makogoncorrect11:19
denis_makogonhttps://github.com/openstack/trove/blob/master/trove/taskmanager/models.py#L556-L56011:19
TomasRtrove agent also is not able to pick any information if I add into /etc/guest_info, it only picks it up if I add directly into the config file: /etc/trove/trove-guestagent.conf11:20
TomasRCan you have a quick look at my trove agent  config file and see if you spot any problem?11:20
TomasRhttps://gist.github.com/anonymous/17ca840d378bda471e2c11:21
denis_makogonTomasR, https://github.com/openstack/trove-integration/blob/master/scripts/files/trove-guest.systemd.conf#L1511:21
denis_makogonTomasR, how do you start the guest ?11:24
TomasRvia service. Actually we had Ubuntu packages installed first and then we ran into bugs, I installed trove guest agent manually. So my trove agent runs with these parameters:11:25
TomasRroot      1960     1  0 11:15 ?        00:00:00 /usr/bin/python /usr/bin/trove-guestagent --config-file=/etc/trove/trove-guestagent.conf --log-dir=/var/log/trove --logfile=guestagent.log11:25
TomasRI also for now had to ran as root as we had some permission issues for trove user and mysqladmin wasn't able to do the ping11:26
TomasRI'll perhaps create a new empty image and will have a fresh install of trove agent11:27
denis_makogonTomasR, wrong execution script11:28
denis_makogonTomasR, it should be /usr/bin/python /usr/bin/trove-guestagent --config-file=/etc/trove/trove-guestagent.conf --config-file=/etc/guest_info --log-dir=/var/log/trove11:28
*** haomai___ has quit IRC11:29
*** haomaiwang has joined #openstack-trove11:29
TomasRlet me do these changes and see if it helps11:30
denis_makogonTomasR, 100% it'll help11:30
TomasR:)11:31
*** demorris has joined #openstack-trove11:36
*** haomaiw__ has joined #openstack-trove11:58
*** haomaiwang has quit IRC11:59
*** haomaiw__ has quit IRC12:01
*** matsuhashi has quit IRC12:02
*** sgotliv has quit IRC12:02
*** haomaiwang has joined #openstack-trove12:02
*** matsuhashi has joined #openstack-trove12:02
*** matsuhashi has quit IRC12:03
*** pdmars has joined #openstack-trove12:06
*** sgotliv has joined #openstack-trove12:07
*** demorris has quit IRC12:09
*** nosnos has quit IRC12:09
*** achampion has quit IRC12:17
*** coolsvap|afk is now known as coolsvap12:20
dougshelley66hi all - I'm seeing a failure on zuul in check-grenade-dsvm..any advice on how to correct?12:22
*** NehaV has joined #openstack-trove12:32
*** coolsvap is now known as coolsvap|afk12:34
*** michael-yu has joined #openstack-trove12:37
*** NehaV has quit IRC12:40
*** michael-yu_ has joined #openstack-trove12:43
*** michael-yu has quit IRC12:45
*** michael-yu_ is now known as michael-yu12:45
*** michael-yu has quit IRC12:48
*** BorisBelous_ has quit IRC12:55
*** demorris has joined #openstack-trove12:57
*** amytron has joined #openstack-trove12:58
*** casanch1 has joined #openstack-trove13:00
*** radez_g0n3 is now known as radez13:12
*** kevinconway has joined #openstack-trove13:14
*** kevinconway has quit IRC13:15
*** achampion has joined #openstack-trove13:18
*** NehaV has joined #openstack-trove13:19
*** jcru has joined #openstack-trove13:25
*** casanch1 has quit IRC13:36
*** kevinconway has joined #openstack-trove13:37
*** mattgriffin has joined #openstack-trove13:45
*** ramishra has quit IRC13:45
*** ramishra has joined #openstack-trove13:47
*** ramishra has quit IRC13:48
*** ramishra has joined #openstack-trove13:50
*** freyes has joined #openstack-trove13:54
*** openstackgerrit has joined #openstack-trove13:55
*** Barker has joined #openstack-trove13:56
openstackgerritDenis M. proposed a change to openstack/trove: Add stack_id to base instance model  https://review.openstack.org/6787313:59
openstackgerritDenis M. proposed a change to openstack/trove: Refactor taskmanager. Split heat/native/migration code path  https://review.openstack.org/9133113:59
*** grapex has joined #openstack-trove13:59
*** rueb7363 has joined #openstack-trove14:03
*** grapex has quit IRC14:05
imsplitbithey guys can I get some core eyes on https://review.openstack.org/#/c/82124/ please?14:05
*** grapex has joined #openstack-trove14:05
imsplitbitgrapex: can you please re-review https://review.openstack.org/#/c/82124/14:05
imsplitbitI <3 u14:06
*** sbfox has joined #openstack-trove14:11
*** ramishra_ has joined #openstack-trove14:18
*** robertmyers has joined #openstack-trove14:20
*** ramishra has quit IRC14:20
*** ramishra_ has quit IRC14:21
TomasRdenis_makogon, so I recreated an image with modifying trove-agent service to include --config-file=/etc/guest_info14:22
TomasR# ps -ef | grep trove14:22
TomasRroot      1501     1  0 11:39 ?        00:00:03 /usr/bin/python /usr/bin/trove-guestagent --config-file=/etc/trove/trove-guestagent.conf --config-file=/etc/guest_info --log-dir=/var/log/trove --logfile=guestagent.log14:22
TomasRcat /etc/guest_info14:22
TomasRcat: /etc/guest_info: No such file or directory14:22
*** rueb7363_ has joined #openstack-trove14:34
*** rueb7363_ has quit IRC14:34
*** rueb7363 has quit IRC14:35
*** sbfox has quit IRC14:37
peterstacdougshelley66: Looks like the process is to find the bug you hit on the recheck page: status.openstack.org/rechecks/14:38
peterstacdougshelley66: Then add a comment to your review 'recheck bug #####' with the bug number you found14:39
*** jasonb365 has joined #openstack-trove14:40
*** rueb7363 has joined #openstack-trove14:42
*** rueb7363 has quit IRC14:47
*** michael-yu has joined #openstack-trove14:47
*** michael-yu has quit IRC14:48
*** jmontemayor has joined #openstack-trove15:01
*** jmontemayor has quit IRC15:02
grapeximsplitbit: Done15:02
*** jmontemayor has joined #openstack-trove15:02
*** sriram_tesora has joined #openstack-trove15:04
*** Barker has quit IRC15:06
*** ramishra has joined #openstack-trove15:07
*** Barker has joined #openstack-trove15:18
*** thedodd has joined #openstack-trove15:19
*** ViswaV has joined #openstack-trove15:19
*** ViswaV_ has joined #openstack-trove15:20
*** doddstack has joined #openstack-trove15:20
*** thedodd has quit IRC15:23
*** ViswaV has quit IRC15:24
*** Barker has quit IRC15:40
*** ViswaV has joined #openstack-trove15:41
*** ViswaV__ has joined #openstack-trove15:42
*** ViswaV_ has quit IRC15:42
*** ViswaV has quit IRC15:45
*** ramishra has quit IRC15:47
denis_makogonTomasR, it should be there, try to recreate an instance15:48
*** serverascode has quit IRC15:52
*** demorris has quit IRC15:53
*** Barker has joined #openstack-trove15:53
*** ihrachyshka has quit IRC15:54
*** serverascode has joined #openstack-trove15:54
*** eghobo has joined #openstack-trove15:55
*** sbfox has joined #openstack-trove16:04
vgnbkrTomasR: did you create the instance by hand, or was it started by trove?16:10
TomasRvgnbkr, I created an image uploaded it to glance associated it with mysql and Trove spawned a new instance automatically16:11
vgnbkrTomasR: then guest_info should be there.  I believe it is injected into the guest instance from taskmanager=>nova=>cloudinit.  Oh, it was a "cloud image" that you uploaded to glance, right?16:13
*** timfreun1 is now known as timfreund16:14
TomasRvgnbkr, yes it was an image we took from Ubuntu cloud and uploaded to Glance16:15
TomasRI also checked metadata service, I think if you don't have it explicitly configured then deafult configuration is being used16:15
TomasRI see that :16:15
TomasRnetstat -a | grep 877516:16
TomasRtcp        0      0 *:8775                  *:*                     LISTEN16:16
TomasRguys, quick question: do we need to explicitly set up metadata service for nova or is it set automatically and if we run it on the same server as nova api, we don't need to configure metadata_host, etc?16:20
*** demorris has joined #openstack-trove16:21
*** Barker has quit IRC16:44
*** demorris has quit IRC16:47
*** harlowja_away is now known as harlowja16:54
*** yidclare has joined #openstack-trove16:57
*** sbfox has quit IRC17:01
*** sbfox has joined #openstack-trove17:01
*** jmontemayor has quit IRC17:05
*** ramishra has joined #openstack-trove17:10
*** ramishra has quit IRC17:11
*** jmontemayor has joined #openstack-trove17:14
*** sgotliv has quit IRC17:15
*** zuqiang has joined #openstack-trove17:17
espSlickNik: hey is the reddwarf gate working ok?17:19
SlickNikesp: Should be. I had to fix it yesterday after the gerrit upgrade, but I think it's been okay since then.17:20
SlickNikesp: For a few hours yesterday, it wasn't reporting on some changes though.17:21
espSlickNik: yeah I think that’s what happened to my patches17:21
espSlickNik: so I tried to retrigger it but jenkins threw an exception in my face17:22
espSlickNik: https://rdjenkins.dyndns.org/job/Trove-Gate/3448/gerrit-trigger-retrigger-this/?17:23
*** sbfox has quit IRC17:23
SlickNikesp: You can retrigger manually from  https://rdjenkins.dyndns.org/gerrit_manual_trigger/?17:28
SlickNikput your gerrit patchset number as the "Query String"17:28
juicedougshelley66: I get those from time to time too17:29
espk, let me give it a try17:29
juicedougshelley66: most of the time it has little if anything to do with the your patch.17:29
juicedougshelley66: the grenade test is a specific scenario which I think is an upgrade of an existing openstack env17:30
*** amcrn has quit IRC17:31
dougshelley66juice: ah ok thx - i did the recheck thing and it seems to work no17:32
dougshelley66now17:32
juiceyep :)17:32
amrithesmute ... yt?17:32
esmuteMorning amrith17:32
esmuteor good afternoon17:32
amrithesmute, hi steve ... 91085. you and I may have fallen in the same manhole.17:33
amrithif I understand correctly17:33
amrith if nic_info.get('net-id') and nic_info.get('port-id'):17:33
amrithis false if both of them are not provided17:33
espSlickNik: is there query param or do you just do http://....?2917:33
amrithhence the bool(a) == bool(b) is ~ not xor (a, b)17:34
espSlickNik: maybe something like ?id=2917:34
SlickNikesp: Sorry if I wasn't clear. Just go to that page and there's a text box to put it in.17:34
amrithesmute, is my understanding of your comment accurate?17:35
espSlickNik: that page?17:35
espSlickNik: you parameters?17:35
SlickNikYes, the page at the URI: https://rdjenkins.dyndns.org/gerrit_manual_trigger/?17:35
esp^ you mean parameters17:35
espah, I never got that17:36
espinteresting17:36
esmuteamrith: let me study it closely again. I woke up an hour ago so i may be half asleep17:36
amrithesmute, I know the feeling. if you look I did the same thing you did yesterday ;)17:36
esmuteamrith: correct, if neither value is provided, the statement would be false and the exception would not be raised17:38
esmutelet me fix my review.17:39
amrithesmute, thx17:39
esmuteThanks for catching that17:39
amrithesmute, np17:39
*** demorris has joined #openstack-trove17:41
espSlickNik: seems to be working, thx!17:43
*** yogeshmehra has joined #openstack-trove17:43
SlickNikesp: Sweet. Anytime!17:43
SlickNikQuick reminder that we have our weekly Trove meeting in #openstack-meeting-alt in ~15 minutes.17:44
SlickNikAgenda at: https://wiki.openstack.org/wiki/Meetings/TroveMeeting17:45
*** amcrn has joined #openstack-trove17:53
*** yogeshmehra has quit IRC17:59
*** doddstack has quit IRC18:00
konetzedwoot woot18:02
*** SnowDust has joined #openstack-trove18:07
*** ramishra has joined #openstack-trove18:12
*** rueb7363 has joined #openstack-trove18:12
*** ramishra has quit IRC18:16
*** ranjitha has joined #openstack-trove18:27
*** rustlebee has left #openstack-trove18:29
*** sbfox has joined #openstack-trove18:31
*** ramashri has joined #openstack-trove18:57
*** ramashri has quit IRC19:00
*** ramashri_ has joined #openstack-trove19:01
*** eghobo has quit IRC19:01
amrithSlickNik, we need a recount with paper ballot19:05
amrithI'm contacting Scalia, Roberts, ... and three of my other good friends19:05
amriththey're the ones who will decide it eventually19:05
juiceso what patches do not requires bugs?19:05
SlickNikjuice, TBD19:06
juicewhat are the exceptions19:06
juicenext meeting?19:06
juiceto be continued...19:06
amrithI can't tell you what the are but you'll know it when you see it.19:06
SlickNikBut my view would be things like Transifex, merges from requirements, doc patches.19:06
SnowDustamrith, especially when you are syncing code from oslo  u dont need file bug19:06
SlickNikThey aren't really bugs, and they aren't really bp's19:06
SnowDustamrith, you are just fast forwarding your project to adopt to latest oslo that way19:07
SlickNikamrith: lol at the reference.19:07
*** Barker has joined #openstack-trove19:08
cp16netjuice: i dont think its black and white19:08
cp16neti think there is some middle ground on needing a bug or not19:09
amrithSlickNick, SnowDust sorry, phone call19:09
amritham back19:09
amrithwhat's the reference you are making SlickNik19:09
cp16neti think a blueprint adds new features which should be required19:09
amrithSnowDust, the issue with the oslo change is that it was a partial change19:09
cp16netbugs not alwasy19:09
cp16netif they are a small change its not that big a deal19:09
SnowDustcp16net, amrith  i gave an example when u dont need either of them to catchup ...19:09
amrithSnowDust, I just entered a second bug (the guy did have a bug) but now I have a second19:09
amrithSnowDust, what's the example?19:10
SnowDustspelling errors, comment enhancements ... are the other things for which you dont need either19:10
SlickNikamrith: "I'll know it when I see it"19:10
amcrncp16net: which just means that we're going to have unnecessary conversations in the coming weeks/months discussing what should and shouldn't require a bug; by requiring it outright, life was simple. should a grammar change require a bug? how about a comment change? how about a typo in a test? how about docs?19:10
amrithSlickNik, OK, sorry. I get it ;(19:10
amrithSlickNik, am running underclocking today19:10
vgnbkr+1 amrith19:10
amrithSlickNik, new diet19:10
SnowDustamrith, especially when you are syncing code from oslo  u dont need file bug19:10
vgnbkr+1 amcrn, not amrith19:10
amrithSnowDust, why?19:10
SnowDustthat was an example scenario19:10
SnowDustin the commit you clearly mention .. what is the intention19:11
amrithSnowDust, I know you "don't have to". I'm saying that's not the preferred alternative.19:11
SnowDustits not bug .. but a sync activity19:11
amrithby your argument, bugs are redundant19:11
amcrnSnowDust: 99% of the time i see a ton of details in the bug notes, but not in the commit msg19:11
amrithblueprints are redundant19:11
amrithjust put everything in the commit comments19:11
amriththe code is self-documenting19:11
SnowDustamcrn : agree .. just -1 in that case19:11
amrithall simple ;)19:11
*** ranjitha has quit IRC19:11
amrithno?19:12
SnowDustwhen a commit msg is empty .. and doesnot even have "Closes Bug" or " Implements"19:12
SnowDustjust -119:12
SnowDust:)19:12
SnowDustits benevolent :D19:12
amcrnSnowDust: yeah, you don't understand the ramifications on those who approve the patches19:12
amrithSnowDust, I'm not getting your point. sorry ...19:12
*** ramishra has joined #openstack-trove19:12
amrithso here's my view of things19:13
SnowDustamcrn : sry ! but i value ur time ..19:13
amrithin order to make a change19:13
*** ramashri_ has quit IRC19:13
amrithyou need some time from core to get it through19:13
amrithand that time is valuable, they've got shit to do19:13
amrithand you need time to make your change19:13
amrithand you need time of people to review the code19:13
amrithat the end of the day, all resources are finite19:13
amrithand we all want to improve the product19:13
amrithit would help in that process if we aligned resources19:13
amrithso if there were bugs and bugs were prioritized to sprints19:14
amriththat would align the resources19:14
amrithif bp's were written, approved coded and then reviewed, it would align resources19:14
amrithwithout these, a hunk of code with no rationale will linger19:14
amrithand the project will appear to be unresponsive19:14
amrithwe have a desire for contribution from a community19:14
amrithwe want to be responsive19:14
amrithyet we also want the project to move forward in a deliberate and methodical way19:15
amritha bug and a blue print, IMHO help that process19:15
* amrith steps off soap box19:15
*** sriram_tesora has quit IRC19:15
amriththoughts, anyone19:15
amrithdid i forget to turn on the mic19:15
amritham I on mute? ;)19:15
*** ranjitha has joined #openstack-trove19:15
*** sriram_tesora has joined #openstack-trove19:16
dougshelley66amrith press the mute button19:16
amcrnamrith: i don't think any one would disagree with your sentiments19:16
*** ranjitha has quit IRC19:16
* amrith punches the mute button19:16
* glucas trips over somebody's soap box19:16
amrithamcrn, thx. but the devil is in the implementation detail(s)19:16
SnowDustamrith : thats usual process .. a commit either has one of these " implements = in case of supporting a BP"  or "Closes Bug = in support of closing a bug"19:16
*** ramishra has quit IRC19:17
amrithSo why would you resist a bug for a fix with a typo?19:18
SnowDustonly  maintenance jobs .. may skip the definition .. and is only done in agreement with cores and PTL (who have prior information of the commit)19:18
amrithso I think we're in agreement?19:19
SlickNikSo really. I didn't mean to get a holy war going with this one.19:20
* amrith dusts off Emacs Vs. vi - the definitive bible19:20
SlickNikThere are cases when we've had some folks from other OS teams come to Trove, and contribute a small change.19:20
SlickNikAnd the first thing they get pelted with, is no bug? −1.19:21
hub_capamrith: they have a new version "Emacs and those other editors, unabridged"19:21
hub_capand we should not operate differently from other projecs19:21
hub_capwe should encourage contribution19:22
hub_capif its a bug, lets list it as such19:22
amrithSlickNik, I understand what you are saying19:22
cp16netamcrn: i'd say if its docs that should be a bug because anne said we should add something to commit msg to make sure a doc bug gets created at the midcyle19:22
hub_capbut we dont have to be _insert phrase here_ about every change19:22
SlickNikcp16net: As a quick example of what I mean: https://review.openstack.org/#/c/91166/19:24
amrithcan we apply the Stewart test to changes and reasonably say that if it appears like a little change we just collectively let it go19:24
amrithand there's going to be some edge cases where we go the wrong way.19:24
cp16netSlickNik: i think that fine to not ahve a bug on it19:24
SlickNikcp16net: same here.19:25
cp16netSlickNik: but... https://review.openstack.org/#/c/88478 i think should have one19:25
SlickNikcp16net: Also agreed. :)19:25
cp16nethehe :)19:26
cp16netmainly because i didnt completely understand when i was reading through the reason for making the changes initially19:26
* amrith goes to get tea19:26
cp16neti think the main benifit of the bug is to track the code changes that we make for a release19:28
SlickNikcp16net: +119:29
cp16neti should emphsize CODE19:29
* cp16net steps off soap box19:29
cp16netBender: Bite my shiny, metal ass!19:29
* amrith steps on soap box. soap box breaks. 19:30
hub_capcp16net: perfect bender quote19:31
hub_capBender: In order to get busy at maximum efficiency, I need a girl with a big, 400-ton booty.19:31
amrithhub_cap, is that a typo? s/booty/bootd/19:32
amrithdo I need a bug for that?19:32
hub_capyes19:32
hub_capyes u do19:32
cp16netlol19:32
SnowDustlol amrith19:33
*** SnowDust has quit IRC19:34
esplol19:35
*** ihrachyshka has joined #openstack-trove19:35
*** sbfox has quit IRC19:36
*** sbfox has joined #openstack-trove19:37
*** ihrachyshka has quit IRC19:38
*** doddstack has joined #openstack-trove19:41
*** michael-yu has joined #openstack-trove19:43
*** michael-yu_ has joined #openstack-trove19:46
*** michael-yu has quit IRC19:46
*** michael-yu_ is now known as michael-yu19:46
*** sbfox1 has joined #openstack-trove20:03
*** sbfox has quit IRC20:04
*** eghobo has joined #openstack-trove20:07
*** ranjitha has joined #openstack-trove20:08
*** mattgriffin has quit IRC20:12
*** ranjitha has quit IRC20:13
*** ramishra has joined #openstack-trove20:13
*** sbfox1 has quit IRC20:14
*** freyes_ has joined #openstack-trove20:15
*** freyes_ has quit IRC20:15
*** ramishra has quit IRC20:17
-openstackstatus- NOTICE: the gate is backed up due to broken nodepool images, fix in progress (eta 22:00 utc)20:26
*** ChanServ changes topic to "the gate is backed up due to broken nodepool images, fix in progress (eta 22:00 utc)"20:26
openstackgerritAuston McReynolds proposed a change to openstack/trove: Fix prepare call for redis guest agent  https://review.openstack.org/8486320:29
openstackgerritAuston McReynolds proposed a change to openstack/trove: Fix prepare call for redis guest agent  https://review.openstack.org/8486320:36
cp16netamcrn: on that review ^^20:36
cp16netits more than just redis though right?20:37
openstackgerritAuston McReynolds proposed a change to openstack/trove: Added support of resize-flavor to Redis datastore  https://review.openstack.org/8870920:37
*** radez is now known as radez_g0n320:38
*** sriram_tesora has quit IRC20:40
*** sriram_tesora has joined #openstack-trove20:41
openstackgerritAuston McReynolds proposed a change to openstack/trove: Fix prepare call for redis guest agent  https://review.openstack.org/8486320:43
grapexSlickNik amcrn : Here's the doc pull request if one of you want to approve it: https://review.openstack.org/#/c/87807/20:45
openstackgerritAuston McReynolds proposed a change to openstack/trove: Fix prepare call for redis guest agent  https://review.openstack.org/8486320:45
*** eghobo has quit IRC20:46
SlickNikgrapex: Oh, very cool! Thanks will take a look.20:47
*** shakayumi has joined #openstack-trove20:47
*** shakayumi has quit IRC20:47
*** michael-yu_ has joined #openstack-trove20:47
*** michael-yu has quit IRC20:50
openstackgerritAuston McReynolds proposed a change to openstack/trove: Added support of resize-flavor to Redis datastore  https://review.openstack.org/8870920:50
*** michael-yu has joined #openstack-trove20:51
*** mattgriffin has joined #openstack-trove20:52
*** michael-yu_ has quit IRC20:53
*** eghobo has joined #openstack-trove20:54
*** eghobo has quit IRC20:54
*** eghobo has joined #openstack-trove20:54
*** michael-yu_ has joined #openstack-trove20:58
*** ranjitha has joined #openstack-trove20:59
esmuteHi all, can you guys review https://review.openstack.org/#/c/81379/ when you have a chance? It should be short and quick21:00
*** michael-yu has quit IRC21:00
*** michael-yu has joined #openstack-trove21:00
*** pdmars has quit IRC21:01
*** michael-yu_ has quit IRC21:02
*** ranjitha has quit IRC21:09
*** ranjitha has joined #openstack-trove21:11
*** ramishra has joined #openstack-trove21:14
*** shakayumi has joined #openstack-trove21:15
*** shakayumi has quit IRC21:15
*** ramishra has quit IRC21:18
cp16netesmute i think i've seen _() stuff...21:19
*** ranjitha has quit IRC21:19
*** sbfox has joined #openstack-trove21:22
esmute_()?21:23
cp16neti just made 2 comment on that review21:23
*** sbfox has quit IRC21:25
kevinconwayesmute: the _() adds the text to the pile for the translation service we use21:25
esmuteahh for localization21:25
esmutecool.. will do.. thanks21:26
cp16net_("this is a hardcoded english phrase nobody will ever change")21:26
kevinconwayyeah, we've even had a contrib add a language already21:26
*** kevinconway has quit IRC21:26
cp16netkevinconway: you see how maybe languages there were?21:26
hub_capsweet!21:26
hub_caphttps://www.transifex.com/projects/p/trove21:26
cp16netlike 50 with all english translations for everyone of them21:27
esmutecp16net: to your other question, mount_points() gives you the mounting for a device path. If the device is already unmounted, it will not give you any result... and thus the unmount wont be called21:27
cp16netlol21:27
cp16netso technically its only 1 lang21:27
*** achampion has quit IRC21:27
cp16netesmute: oh gotcha21:28
hub_capim not sure what kevin was talkin about hhe21:28
hub_cap*heh21:29
*** chipc has left #openstack-trove21:34
*** sbfox has joined #openstack-trove21:34
*** sbfox has quit IRC21:35
*** sbfox has joined #openstack-trove21:38
*** Barker has quit IRC21:43
openstackgerritSteve Leon proposed a change to openstack/trove: Unmounting ephemeral if it has already been mounted  https://review.openstack.org/8137921:54
*** robertmyers has quit IRC21:57
*** yidclare has quit IRC22:05
amcrnSlickNik: well that's interesting. i'm now seeing the same issue TomasR was last night, with /etc/guest_info not showing up. just reproduced this on devstack.22:07
esmutehey amcrn, what kind of concern do you have about https://review.openstack.org/#/c/89972/?22:07
TomasRGuys, I'm working on recreating a guest image from scratch.22:09
TomasRI followed the instructions from here: http://docs.openstack.org/developer/trove/dev/manual_install.html22:10
TomasRand I do see an empty /etc/guest_info file on newly spawn VM.22:10
TomasRhowever the instruction is a bit confusing as it doesn't say anything about installing guest agent into the image22:11
TomasRSo I guess, I'll have to start VM then install guest agent, then create a snapshot out of it and then update mysql to use a new image22:12
amcrnTomasR: this missing /etc/guest_info file looks to be a bug22:14
*** ramishra has joined #openstack-trove22:15
dougshelley66amcrn, cp16net: could you take a look at https://review.openstack.org/#/c/91219/22:15
espamcrn: TomasR: I ran into this last week and I think SlickNik told me to rebase, not sure if it applies here22:15
amcrnesp: i just reproduced it on a brand new devstack with absolutely no code changes22:15
espamcrn: k22:16
espI’ll try to give it a spin on my devstack22:16
espactually you know what I do remember trying to build an instance that had no guest_info recently22:17
espprobably is a bug somewhere but it didn’t repro for me22:17
*** jcru has quit IRC22:18
amcrnesp: yeah, it seems to be intermittent22:18
espamcrn: I remember having to craft a guest_info and that seemed allow the guest agent to start up…22:19
espnot cool22:19
*** ramishra has quit IRC22:19
*** demorris has quit IRC22:25
juiceamcrn: about the bad rebase yesterday...does this look better? https://review.openstack.org/#/c/84555/22:26
openstackgerritA change was merged to openstack/trove-integration: Remove the iptables workaround  https://review.openstack.org/8455522:28
*** yidclare has joined #openstack-trove22:30
SlickNikamcrn: That's interesting. I haven't hit that particular issue.22:32
amcrnjuice: yes :)22:32
juiceamcrn: great :)22:32
juiceI knew I could do it22:32
SlickNikamcrn: I wonder what triggers the bug, and whether it has affected any of the rdjenkins-gate runs.22:34
*** achampion has joined #openstack-trove22:34
SlickNikamcrn: Not enough to cause a noticeable blip, I guess.22:34
SlickNikesp: The one that called for a rebase was a slightly different issue (It had to do with enforcing global-requirements, IIRC).22:35
espSlickNik: k22:36
amcrni think i have a hunch, but i'm going to try some things about before i say anything and look like a fool22:36
SlickNikamcrn: Sure, leave me in suspense while you go "try some things out…" :)22:39
SlickNikaka. Okay sounds good, let me know what you find.22:39
amcrn:D22:39
*** doddstack has quit IRC22:40
amcrnesmute: that's a long conversation, but in short: such a command makes the assumption that the deployment has federated identity across regions (i.e. the token in one region works in another, and that the tenants align as well)22:42
espamcrn: nice22:44
esmuteamcrn: the client is reflecting the API. Given that the BP was approved.22:44
esmutewhat i can do is add this to the monday meeting so we can discuss further about it22:45
amcrnesmute: that's what the aim was, apologies if that wasn't clear.22:45
*** ramishra has joined #openstack-trove22:47
*** eguz has joined #openstack-trove22:48
*** ramishra has quit IRC22:48
*** ramishra has joined #openstack-trove22:49
*** eghobo has quit IRC22:52
*** ramishra has quit IRC22:54
*** achampio1 has joined #openstack-trove22:59
*** flaper87 is now known as flaper87|afk23:00
*** achampion has quit IRC23:02
openstackgerritNikhil Manchanda proposed a change to openstack/trove-integration: Trove guestagent should not use sample conf  https://review.openstack.org/8847823:03
openstackgerritNikhil Manchanda proposed a change to openstack/trove-integration: Add support for a neutron-based install  https://review.openstack.org/7812323:03
*** jmontemayor has quit IRC23:03
*** sriram_tesora has quit IRC23:12
openstackgerritSaurabh Surana proposed a change to openstack/trove-integration: updating sources list as part of percona elements  https://review.openstack.org/9079123:15
*** michael-yu has quit IRC23:19
TomasRJFYI - i recreated a guest image from scratch, using an image from ubunti cloud, installed trove using instructions from here: http://docs.openstack.org/developer/trove/dev/manual_install.html23:23
TomasRand still can't get information automatically injected into /etc/guest_info23:24
SlickNikTomasR: Could you verify that file injection/server personality is working correctly for an instance booted through nova?23:26
SlickNikTomasR: Trove relies on this nova feature23:27
TomasRSlickNik, how to check it? I'm new to Trove and OpenStack, is there a link in the docs?23:27
espmaybe try to inject another file? or location?23:28
esptrove does this in create:  files = {"/etc/guest_info": ("[DEFAULT]\nguest_id=%s\n"23:28
esp                                     "datastore_manager=%s\n"23:28
esp                                     "tenant_id=%s\n" %23:28
esp                                     (self.id, datastore_manager,23:28
esp                                      self.tenant_id))}23:28
abramleydevstack sets force_config_drive=always to enable file injection23:29
*** demorris has joined #openstack-trove23:30
espTomasR: https://github.com/openstack/trove/blob/master/trove/taskmanager/models.py#L553-L56023:30
abramleywhich you would set in nova.conf on your compute node23:31
espcool abramley :)23:31
TomasResp, I can see this in nova logs23:31
TomasRnova-api.log:2014-05-01 01:16:55.027 20431 DEBUG nova.api.openstack.wsgi [req-ffe6cc1d-3295-4ebf-b16f-aaa67f51c7c2 222c54b44c4746aa8072537e9f05e8cb 7d359600815144eeac268a10efaba62b] Action: 'create', body: {"server": {"name": "85292f2a-a336-4409-9889-a5bb646859ff-lay", "imageRef": "322c9b7f-6997-4544-81c3-50cedeb1087c", "block_device_mapping": [{"volume_size": "2", "volume_id": "4d02a488-0748-438d-a9d5-e0d4931f2c36", "delete_on23:31
TomasR...23:32
TomasR "personality": [{"path": "/etc/guest_info", "contents": "W0RF...23:32
*** rueb7363 has quit IRC23:32
TomasRand then I do base64 decode of contents, I see, guest_id, tenant_id23:32
espTomasR: so trove is sending it ...23:32
espTomasR: is that really the contents value “WORF..23:33
esp?23:33
TomasRit's not the full value23:33
espk23:33
TomasRI was able to decode that string into this23:33
abramleytrove is passing it to nova - but you need to use either config_drive or libvirt inject partition in order to have nova pass the file into the nova instance23:34
esplet me check my nova log23:34
TomasR[DEFAULT] guest_id=85292f2a-a336-4409-9889-a5bb646859ff datastore_manager=mysql tenant_id=7d359600815144eeac268a10efaba62b23:34
espk23:34
abramleyin havana libvirt inject partition was most likely enabled - but in icehouse this is now disabled by default and deprecated I believe23:34
abramleySo using config_drive seems like the right way to do it23:34
espabramley: that makes sense, we heard file injection is going away23:34
abramleyToday for trove the way to enable config_drive is to use the force_config_drive option in nova - however, in future I think we should be doing this programatically in the trove task manager on nova boot calls23:35
openstackgerritNikhil Manchanda proposed a change to openstack/trove-integration: Simplify Building Trove Guest Images  https://review.openstack.org/9148723:36
abramleyI can create a BP to add this in23:36
abramley... assuming people think that is the right way to do this23:36
espthx abramley23:38
espTomasR: my nova logs look sames as yours23:38
espTomasR: I’m checking configs now23:39
TomasRI added force_config_drive=true to nova.conf to see if it makes any difference23:39
abramleyYou can also check the nova guest instance once it boots - you should see an extra volume under /dev/disk/by-label - I think it is called something like config-223:40
esp[libvirt]23:40
espinject_partition = -223:40
esp^ that in my nova.conf on devstack23:41
esp*thats23:41
abramleyesp - right so that is the old deprecated partition injection - and -2 means disabled23:41
espabramley: k, I just built it new 2 days ago23:41
*** grapex has quit IRC23:42
TomasResp, my config is here: https://gist.github.com/anonymous/1138653123:44
TomasRmaybe you can put your config on gist, so I can check tomorrow morning?23:44
TomasRhave to go to sleep now as it's 00:45am23:44
espTomasR: k, give me a min23:45
espTomasR: but I the param I showed you was only in the nova.conf .. coming right up23:45
TomasRp.s. force_config_drive=true didn't help, I still can't get guest_id injected23:46
TomasR2014-04-30 23:45:05.678 1217 DEBUG trove.conductor.api [-] Making async call to cast heartbeat for instance: None heartbeat /root/pip-1.4.1/trove/trove/conductor/api.py:4023:46
espTomasR: abramley: https://gist.github.com/dan1/1d90c1e656ca817a8acb23:46
espnova.conf from devstack ^23:46
abramleyI think it is meant to be force_config_drive=always23:47
abramleybut I have seen conflicting info in various documents about that - I am using 'always' and it works for me23:47
TomasRit is "=true" according to this document: http://docs.openstack.org/grizzly/openstack-compute/admin/content/config-drive.html23:48
abramleyesp - yes that has the force_config_drive = always - that matches what we are doing23:48
espabramley: gotcha23:48
TomasRok,guys, have to go now, I'll resume tomorrow. good luck23:49
espk, see ya later.23:49
espthx for the info TomasR and abramley23:49
*** ramishra has joined #openstack-trove23:50
*** venkater has joined #openstack-trove23:52
*** ramishra has quit IRC23:54
*** achampion has joined #openstack-trove23:54
*** demorris has quit IRC23:55
*** demorris has joined #openstack-trove23:56
*** achampio1 has quit IRC23:56

Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!