*** mattgriffin has joined #openstack-trove | 00:18 | |
*** yidclare has quit IRC | 00:28 | |
*** jmontemayor has quit IRC | 00:31 | |
*** IvanZ has quit IRC | 00:48 | |
*** mattgriffin has quit IRC | 00:48 | |
juice | amcrn: around? | 00:51 |
---|---|---|
*** mattgriffin has joined #openstack-trove | 00:56 | |
dougshelley66 | juice: I'm no amcrn but I'm here :) | 01:05 |
juice | ha ha - thanks dougshelley66. I was just responding to an earlier comment he made about the commit. I am not sure what he meant | 01:08 |
dougshelley66 | i 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 |
dougshelley66 | but i could be wrong... | 01:12 |
openstackgerrit | Justin Hopper proposed a change to openstack/trove-integration: Remove the iptables workaround https://review.openstack.org/84555 | 01:18 |
SlickNik | juice: he meant that your rebase is messed up. | 01:20 |
juice | slicknik: I understand that but not why | 01:20 |
juice | git seemed pretty happy with what I did | 01:20 |
*** eguz has quit IRC | 01:22 | |
SlickNik | I 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 |
SlickNik | Maybe he didn't realize that you had modified the content? (i.e. updated the "Using Openstack Clients" section) | 01:23 |
juice | I can remove the section completely - what do you think | 01:25 |
juice | not trying to pit your word against amcrn :) | 01:25 |
SlickNik | I like it; but would move I'd move it to another changeset cause it's unrelated. | 01:26 |
juice | okie doke | 01:26 |
SlickNik | Thanks juice. | 01:27 |
openstackgerrit | Justin Hopper proposed a change to openstack/trove-integration: Remove the iptables workaround https://review.openstack.org/84555 | 01:28 |
*** nosnos has joined #openstack-trove | 01:35 | |
*** ramishra has joined #openstack-trove | 01:38 | |
*** erkules has quit IRC | 01:50 | |
*** amytron has quit IRC | 01:50 | |
*** robertmyers has joined #openstack-trove | 01:55 | |
*** haomaiwa_ has joined #openstack-trove | 01:57 | |
*** haomaiw__ has quit IRC | 02:00 | |
*** ramishra has quit IRC | 02:04 | |
*** robertmyers has quit IRC | 02:04 | |
amrith | mat-lowery, did my comments earlier make sense? | 02:05 |
*** robertmyers has joined #openstack-trove | 02:05 | |
*** ramishra has joined #openstack-trove | 02:05 | |
*** mattgriffin has quit IRC | 02:11 | |
*** haomaiwa_ has quit IRC | 02:13 | |
*** haomaiwang has joined #openstack-trove | 02:13 | |
*** michael-yu has quit IRC | 02:17 | |
*** ramishra has quit IRC | 02:19 | |
*** ramishra has joined #openstack-trove | 02:20 | |
*** harlowja is now known as harlowja_away | 02:27 | |
openstackgerrit | Doug Shelley proposed a change to openstack/trove: Make config attach/detach language consistent https://review.openstack.org/91219 | 02:29 |
openstackgerrit | Doug Shelley proposed a change to openstack/trove: Make config attach/detach language consistent https://review.openstack.org/91219 | 02:30 |
*** coolsvap|afk is now known as coolsvap | 02:30 | |
*** Barker has joined #openstack-trove | 02:31 | |
openstackgerrit | Nikhil Manchanda proposed a change to openstack/trove: Improve help strings https://review.openstack.org/91166 | 02:34 |
*** haomaiw__ has joined #openstack-trove | 02:35 | |
*** haomaiwang has quit IRC | 02:38 | |
*** ramishra has quit IRC | 02:44 | |
openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/python-troveclient: Updated from global requirements https://review.openstack.org/91246 | 02:47 |
*** michael-yu has joined #openstack-trove | 02:47 | |
*** Barker has quit IRC | 02:47 | |
openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/trove: Updated from global requirements https://review.openstack.org/91252 | 02:48 |
*** NehaV has joined #openstack-trove | 02:50 | |
openstackgerrit | Craig Vyvial proposed a change to openstack/trove: Datastore Configuration Parameters stored in db https://review.openstack.org/79850 | 02:53 |
*** jasonb365 has joined #openstack-trove | 02:53 | |
*** sbfox has quit IRC | 02:54 | |
*** matsuhashi has quit IRC | 02:56 | |
*** matsuhashi has joined #openstack-trove | 02:57 | |
*** ramishra has joined #openstack-trove | 02:57 | |
*** sbfox has joined #openstack-trove | 03:05 | |
*** amcrn has joined #openstack-trove | 03:07 | |
*** michael-yu has quit IRC | 03:09 | |
*** mattgriffin has joined #openstack-trove | 03:12 | |
*** robertmyers has quit IRC | 03:22 | |
*** robertmyers has joined #openstack-trove | 03:22 | |
*** robertmyers has quit IRC | 03:28 | |
*** freyes has quit IRC | 03:36 | |
*** coolsvap is now known as coolsvap|afk | 03:42 | |
openstackgerrit | Craig Vyvial proposed a change to openstack/trove: Datastore Configuration Parameters stored in db https://review.openstack.org/79850 | 03:44 |
*** erkules has joined #openstack-trove | 03:46 | |
*** NehaV has quit IRC | 03:47 | |
*** jasonb365 has quit IRC | 03:48 | |
*** eghobo has joined #openstack-trove | 03:49 | |
*** haomaiw__ has quit IRC | 03:51 | |
*** haomaiwang has joined #openstack-trove | 03:52 | |
*** coolsvap|afk is now known as coolsvap | 03:54 | |
*** shakamunyi has quit IRC | 04:05 | |
*** matsuhashi has quit IRC | 04:10 | |
*** michael-yu has joined #openstack-trove | 04:18 | |
*** nosnos has quit IRC | 04:26 | |
*** sbfox has quit IRC | 04:26 | |
*** eghobo has quit IRC | 04:32 | |
*** Barker has joined #openstack-trove | 04:47 | |
*** Barker has quit IRC | 04:49 | |
*** sbfox has joined #openstack-trove | 04:49 | |
*** haomaiw__ has joined #openstack-trove | 04:53 | |
*** haomaiwang has quit IRC | 04:53 | |
*** eghobo has joined #openstack-trove | 04:55 | |
*** shivam_ has left #openstack-trove | 05:02 | |
*** shivam_ has joined #openstack-trove | 05:02 | |
*** matsuhashi has joined #openstack-trove | 05:04 | |
*** nosnos has joined #openstack-trove | 05:16 | |
*** nosnos has quit IRC | 05:22 | |
*** nosnos has joined #openstack-trove | 05:23 | |
*** michael-yu has quit IRC | 05:28 | |
*** matsuhashi has quit IRC | 05:30 | |
*** matsuhashi has joined #openstack-trove | 05:30 | |
*** matsuhashi has quit IRC | 05:36 | |
*** matsuhashi has joined #openstack-trove | 05:36 | |
openstackgerrit | Craig Vyvial proposed a change to openstack/trove: configuration datastore paramters description https://review.openstack.org/91267 | 05:37 |
openstackgerrit | Craig Vyvial proposed a change to openstack/python-troveclient: descriptions on datastore configuration parameters https://review.openstack.org/91268 | 05:37 |
*** yogeshmehra has joined #openstack-trove | 05:38 | |
*** ramishra has quit IRC | 05:44 | |
*** amcrn has quit IRC | 05:54 | |
*** yogeshmehra has quit IRC | 05:57 | |
openstackgerrit | A change was merged to openstack/trove: Improve help strings https://review.openstack.org/91166 | 05:58 |
*** ramishra has joined #openstack-trove | 06:03 | |
*** haomaiw__ has quit IRC | 06:07 | |
*** haomaiwang has joined #openstack-trove | 06:07 | |
*** yogeshmehra has joined #openstack-trove | 06:07 | |
openstackgerrit | OpenStack Proposal Bot proposed a change to openstack/trove: Imported Translations from Transifex https://review.openstack.org/91272 | 06:21 |
*** yogeshmehra has quit IRC | 06:24 | |
*** ramishra has quit IRC | 06:27 | |
*** ramishra has joined #openstack-trove | 06:30 | |
*** mattgriffin has quit IRC | 06:43 | |
*** flaper87|afk is now known as flaper87 | 06:49 | |
*** flaper87 is now known as flaper87|afk | 06:51 | |
*** TomasR_ has quit IRC | 06:53 | |
*** michael-yu has joined #openstack-trove | 06:53 | |
*** flaper87|afk is now known as flaper87 | 07:03 | |
*** matsuhashi has quit IRC | 07:08 | |
*** yogeshmehra has joined #openstack-trove | 07:09 | |
*** matsuhashi has joined #openstack-trove | 07:11 | |
*** matsuhashi has quit IRC | 07:17 | |
*** michael-yu has quit IRC | 07:17 | |
*** matsuhashi has joined #openstack-trove | 07:17 | |
*** michael-yu has joined #openstack-trove | 07:20 | |
*** sbfox has quit IRC | 07:20 | |
*** michael-yu has quit IRC | 07:20 | |
*** sgotliv has joined #openstack-trove | 07:23 | |
*** michael-yu has joined #openstack-trove | 07:26 | |
openstackgerrit | Shashank Gupta proposed a change to openstack/trove: Corrected the assert message in test_dbaas.py https://review.openstack.org/89815 | 07:29 |
*** eghobo has quit IRC | 07:30 | |
*** amcrn has joined #openstack-trove | 07:30 | |
*** sgotliv_ has joined #openstack-trove | 07:42 | |
*** coolsvap is now known as coolsvap|afk | 07:44 | |
*** michael-yu has quit IRC | 07:44 | |
*** sgotliv has quit IRC | 07:44 | |
*** haomai___ has joined #openstack-trove | 07:45 | |
*** haomaiwang has quit IRC | 07:48 | |
*** coolsvap|afk is now known as coolsvap | 07:51 | |
*** zigo_ is now known as zigo | 07:55 | |
*** yogeshmehra has quit IRC | 08:15 | |
*** yogeshmehra has joined #openstack-trove | 08:16 | |
*** ihrachyshka has joined #openstack-trove | 08:17 | |
*** yogeshmehra has quit IRC | 08:20 | |
*** sgotliv_ has quit IRC | 08:21 | |
*** sgotliv_ has joined #openstack-trove | 08:33 | |
*** coolsvap is now known as coolsvap|afk | 08:54 | |
*** haomai___ has quit IRC | 08:58 | |
*** haomaiwa_ has joined #openstack-trove | 08:58 | |
*** haomaiwa_ has quit IRC | 09:49 | |
*** haomaiwang has joined #openstack-trove | 09:49 | |
*** denis_makogon has joined #openstack-trove | 09:50 | |
*** coolsvap|afk is now known as coolsvap | 09:55 | |
*** sgotliv_ has quit IRC | 10:02 | |
*** sgotliv_ has joined #openstack-trove | 10:04 | |
*** sgotliv_ has quit IRC | 10:19 | |
*** nosnos has quit IRC | 10:21 | |
*** sgotliv has joined #openstack-trove | 10:26 | |
*** haomai___ has joined #openstack-trove | 10:28 | |
*** nosnos has joined #openstack-trove | 10:29 | |
*** haomaiwang has quit IRC | 10:31 | |
*** coolsvap is now known as coolsvap|afk | 10:33 | |
*** SushillKM has joined #openstack-trove | 10:47 | |
*** openstackgerrit has quit IRC | 10:51 | |
*** TomasR has joined #openstack-trove | 10:53 | |
TomasR | Hi, 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 |
TomasR | I don't see any error and I do see an attempt to inject info into /etc/guest_info | 10:54 |
TomasR | I added the log here: https://gist.github.com/anonymous/ea8e38c6695e14099f1d | 10:54 |
denis_makogon | TomasR, trove doesn't inject guest_id into trove-guestagent.conf/sample, it goes to guest_info | 11:16 |
TomasR | denis_makagon, I looked at the log file and I found this: | 11:17 |
*** SushillKM has quit IRC | 11:17 | |
TomasR | "personality": [{"path": "/etc/guest_info", "contents": "W0RFRkFVTFRdCmd1ZXN0X2lkPTA2YjdmNDZhLWVmMjQtNDgwOS1hZmFmLTZhMTk1MzhmN2ExYgpkYXRhc3RvcmVfbWFuYWdlcj1teXNxbAp0ZW5hbnRfaWQ9N2QzNTk2MDA4MTUxNDRlZWFjMjY4YTEwZWZhYmE2MmIK"}]}} _process_stack /usr/lib/python2.7/dist-packages/nova/api/openstack/wsgi.py:940 | 11:17 |
TomasR | decoding the info: | 11:17 |
TomasR | [DEFAULT] guest_id=06b7f46a-ef24-4809-afaf-6a19538f7a1b datastore_manager=mysql tenant_id=7d359600815144eeac268a10efaba62b | 11:17 |
denis_makogon | yes | 11:18 |
denis_makogon | correct | 11:19 |
denis_makogon | https://github.com/openstack/trove/blob/master/trove/taskmanager/models.py#L556-L560 | 11:19 |
TomasR | trove 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.conf | 11:20 |
TomasR | Can you have a quick look at my trove agent config file and see if you spot any problem? | 11:20 |
TomasR | https://gist.github.com/anonymous/17ca840d378bda471e2c | 11:21 |
denis_makogon | TomasR, https://github.com/openstack/trove-integration/blob/master/scripts/files/trove-guest.systemd.conf#L15 | 11:21 |
denis_makogon | TomasR, how do you start the guest ? | 11:24 |
TomasR | via 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 |
TomasR | root 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.log | 11:25 |
TomasR | I 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 ping | 11:26 |
TomasR | I'll perhaps create a new empty image and will have a fresh install of trove agent | 11:27 |
denis_makogon | TomasR, wrong execution script | 11:28 |
denis_makogon | TomasR, 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/trove | 11:28 |
*** haomai___ has quit IRC | 11:29 | |
*** haomaiwang has joined #openstack-trove | 11:29 | |
TomasR | let me do these changes and see if it helps | 11:30 |
denis_makogon | TomasR, 100% it'll help | 11:30 |
TomasR | :) | 11:31 |
*** demorris has joined #openstack-trove | 11:36 | |
*** haomaiw__ has joined #openstack-trove | 11:58 | |
*** haomaiwang has quit IRC | 11:59 | |
*** haomaiw__ has quit IRC | 12:01 | |
*** matsuhashi has quit IRC | 12:02 | |
*** sgotliv has quit IRC | 12:02 | |
*** haomaiwang has joined #openstack-trove | 12:02 | |
*** matsuhashi has joined #openstack-trove | 12:02 | |
*** matsuhashi has quit IRC | 12:03 | |
*** pdmars has joined #openstack-trove | 12:06 | |
*** sgotliv has joined #openstack-trove | 12:07 | |
*** demorris has quit IRC | 12:09 | |
*** nosnos has quit IRC | 12:09 | |
*** achampion has quit IRC | 12:17 | |
*** coolsvap|afk is now known as coolsvap | 12:20 | |
dougshelley66 | hi all - I'm seeing a failure on zuul in check-grenade-dsvm..any advice on how to correct? | 12:22 |
*** NehaV has joined #openstack-trove | 12:32 | |
*** coolsvap is now known as coolsvap|afk | 12:34 | |
*** michael-yu has joined #openstack-trove | 12:37 | |
*** NehaV has quit IRC | 12:40 | |
*** michael-yu_ has joined #openstack-trove | 12:43 | |
*** michael-yu has quit IRC | 12:45 | |
*** michael-yu_ is now known as michael-yu | 12:45 | |
*** michael-yu has quit IRC | 12:48 | |
*** BorisBelous_ has quit IRC | 12:55 | |
*** demorris has joined #openstack-trove | 12:57 | |
*** amytron has joined #openstack-trove | 12:58 | |
*** casanch1 has joined #openstack-trove | 13:00 | |
*** radez_g0n3 is now known as radez | 13:12 | |
*** kevinconway has joined #openstack-trove | 13:14 | |
*** kevinconway has quit IRC | 13:15 | |
*** achampion has joined #openstack-trove | 13:18 | |
*** NehaV has joined #openstack-trove | 13:19 | |
*** jcru has joined #openstack-trove | 13:25 | |
*** casanch1 has quit IRC | 13:36 | |
*** kevinconway has joined #openstack-trove | 13:37 | |
*** mattgriffin has joined #openstack-trove | 13:45 | |
*** ramishra has quit IRC | 13:45 | |
*** ramishra has joined #openstack-trove | 13:47 | |
*** ramishra has quit IRC | 13:48 | |
*** ramishra has joined #openstack-trove | 13:50 | |
*** freyes has joined #openstack-trove | 13:54 | |
*** openstackgerrit has joined #openstack-trove | 13:55 | |
*** Barker has joined #openstack-trove | 13:56 | |
openstackgerrit | Denis M. proposed a change to openstack/trove: Add stack_id to base instance model https://review.openstack.org/67873 | 13:59 |
openstackgerrit | Denis M. proposed a change to openstack/trove: Refactor taskmanager. Split heat/native/migration code path https://review.openstack.org/91331 | 13:59 |
*** grapex has joined #openstack-trove | 13:59 | |
*** rueb7363 has joined #openstack-trove | 14:03 | |
*** grapex has quit IRC | 14:05 | |
imsplitbit | hey guys can I get some core eyes on https://review.openstack.org/#/c/82124/ please? | 14:05 |
*** grapex has joined #openstack-trove | 14:05 | |
imsplitbit | grapex: can you please re-review https://review.openstack.org/#/c/82124/ | 14:05 |
imsplitbit | I <3 u | 14:06 |
*** sbfox has joined #openstack-trove | 14:11 | |
*** ramishra_ has joined #openstack-trove | 14:18 | |
*** robertmyers has joined #openstack-trove | 14:20 | |
*** ramishra has quit IRC | 14:20 | |
*** ramishra_ has quit IRC | 14:21 | |
TomasR | denis_makogon, so I recreated an image with modifying trove-agent service to include --config-file=/etc/guest_info | 14:22 |
TomasR | # ps -ef | grep trove | 14:22 |
TomasR | root 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.log | 14:22 |
TomasR | cat /etc/guest_info | 14:22 |
TomasR | cat: /etc/guest_info: No such file or directory | 14:22 |
*** rueb7363_ has joined #openstack-trove | 14:34 | |
*** rueb7363_ has quit IRC | 14:34 | |
*** rueb7363 has quit IRC | 14:35 | |
*** sbfox has quit IRC | 14:37 | |
peterstac | dougshelley66: Looks like the process is to find the bug you hit on the recheck page: status.openstack.org/rechecks/ | 14:38 |
peterstac | dougshelley66: Then add a comment to your review 'recheck bug #####' with the bug number you found | 14:39 |
*** jasonb365 has joined #openstack-trove | 14:40 | |
*** rueb7363 has joined #openstack-trove | 14:42 | |
*** rueb7363 has quit IRC | 14:47 | |
*** michael-yu has joined #openstack-trove | 14:47 | |
*** michael-yu has quit IRC | 14:48 | |
*** jmontemayor has joined #openstack-trove | 15:01 | |
*** jmontemayor has quit IRC | 15:02 | |
grapex | imsplitbit: Done | 15:02 |
*** jmontemayor has joined #openstack-trove | 15:02 | |
*** sriram_tesora has joined #openstack-trove | 15:04 | |
*** Barker has quit IRC | 15:06 | |
*** ramishra has joined #openstack-trove | 15:07 | |
*** Barker has joined #openstack-trove | 15:18 | |
*** thedodd has joined #openstack-trove | 15:19 | |
*** ViswaV has joined #openstack-trove | 15:19 | |
*** ViswaV_ has joined #openstack-trove | 15:20 | |
*** doddstack has joined #openstack-trove | 15:20 | |
*** thedodd has quit IRC | 15:23 | |
*** ViswaV has quit IRC | 15:24 | |
*** Barker has quit IRC | 15:40 | |
*** ViswaV has joined #openstack-trove | 15:41 | |
*** ViswaV__ has joined #openstack-trove | 15:42 | |
*** ViswaV_ has quit IRC | 15:42 | |
*** ViswaV has quit IRC | 15:45 | |
*** ramishra has quit IRC | 15:47 | |
denis_makogon | TomasR, it should be there, try to recreate an instance | 15:48 |
*** serverascode has quit IRC | 15:52 | |
*** demorris has quit IRC | 15:53 | |
*** Barker has joined #openstack-trove | 15:53 | |
*** ihrachyshka has quit IRC | 15:54 | |
*** serverascode has joined #openstack-trove | 15:54 | |
*** eghobo has joined #openstack-trove | 15:55 | |
*** sbfox has joined #openstack-trove | 16:04 | |
vgnbkr | TomasR: did you create the instance by hand, or was it started by trove? | 16:10 |
TomasR | vgnbkr, I created an image uploaded it to glance associated it with mysql and Trove spawned a new instance automatically | 16:11 |
vgnbkr | TomasR: 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 timfreund | 16:14 | |
TomasR | vgnbkr, yes it was an image we took from Ubuntu cloud and uploaded to Glance | 16:15 |
TomasR | I also checked metadata service, I think if you don't have it explicitly configured then deafult configuration is being used | 16:15 |
TomasR | I see that : | 16:15 |
TomasR | netstat -a | grep 8775 | 16:16 |
TomasR | tcp 0 0 *:8775 *:* LISTEN | 16:16 |
TomasR | guys, 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-trove | 16:21 | |
*** Barker has quit IRC | 16:44 | |
*** demorris has quit IRC | 16:47 | |
*** harlowja_away is now known as harlowja | 16:54 | |
*** yidclare has joined #openstack-trove | 16:57 | |
*** sbfox has quit IRC | 17:01 | |
*** sbfox has joined #openstack-trove | 17:01 | |
*** jmontemayor has quit IRC | 17:05 | |
*** ramishra has joined #openstack-trove | 17:10 | |
*** ramishra has quit IRC | 17:11 | |
*** jmontemayor has joined #openstack-trove | 17:14 | |
*** sgotliv has quit IRC | 17:15 | |
*** zuqiang has joined #openstack-trove | 17:17 | |
esp | SlickNik: hey is the reddwarf gate working ok? | 17:19 |
SlickNik | esp: Should be. I had to fix it yesterday after the gerrit upgrade, but I think it's been okay since then. | 17:20 |
SlickNik | esp: For a few hours yesterday, it wasn't reporting on some changes though. | 17:21 |
esp | SlickNik: yeah I think that’s what happened to my patches | 17:21 |
esp | SlickNik: so I tried to retrigger it but jenkins threw an exception in my face | 17:22 |
esp | SlickNik: https://rdjenkins.dyndns.org/job/Trove-Gate/3448/gerrit-trigger-retrigger-this/? | 17:23 |
*** sbfox has quit IRC | 17:23 | |
SlickNik | esp: You can retrigger manually from https://rdjenkins.dyndns.org/gerrit_manual_trigger/? | 17:28 |
SlickNik | put your gerrit patchset number as the "Query String" | 17:28 |
juice | dougshelley66: I get those from time to time too | 17:29 |
esp | k, let me give it a try | 17:29 |
juice | dougshelley66: most of the time it has little if anything to do with the your patch. | 17:29 |
juice | dougshelley66: the grenade test is a specific scenario which I think is an upgrade of an existing openstack env | 17:30 |
*** amcrn has quit IRC | 17:31 | |
dougshelley66 | juice: ah ok thx - i did the recheck thing and it seems to work no | 17:32 |
dougshelley66 | now | 17:32 |
juice | yep :) | 17:32 |
amrith | esmute ... yt? | 17:32 |
esmute | Morning amrith | 17:32 |
esmute | or good afternoon | 17:32 |
amrith | esmute, hi steve ... 91085. you and I may have fallen in the same manhole. | 17:33 |
amrith | if I understand correctly | 17:33 |
amrith | if nic_info.get('net-id') and nic_info.get('port-id'): | 17:33 |
amrith | is false if both of them are not provided | 17:33 |
esp | SlickNik: is there query param or do you just do http://....?29 | 17:33 |
amrith | hence the bool(a) == bool(b) is ~ not xor (a, b) | 17:34 |
esp | SlickNik: maybe something like ?id=29 | 17:34 |
SlickNik | esp: Sorry if I wasn't clear. Just go to that page and there's a text box to put it in. | 17:34 |
amrith | esmute, is my understanding of your comment accurate? | 17:35 |
esp | SlickNik: that page? | 17:35 |
esp | SlickNik: you parameters? | 17:35 |
SlickNik | Yes, the page at the URI: https://rdjenkins.dyndns.org/gerrit_manual_trigger/? | 17:35 |
esp | ^ you mean parameters | 17:35 |
esp | ah, I never got that | 17:36 |
esp | interesting | 17:36 |
esmute | amrith: let me study it closely again. I woke up an hour ago so i may be half asleep | 17:36 |
amrith | esmute, I know the feeling. if you look I did the same thing you did yesterday ;) | 17:36 |
esmute | amrith: correct, if neither value is provided, the statement would be false and the exception would not be raised | 17:38 |
esmute | let me fix my review. | 17:39 |
amrith | esmute, thx | 17:39 |
esmute | Thanks for catching that | 17:39 |
amrith | esmute, np | 17:39 |
*** demorris has joined #openstack-trove | 17:41 | |
esp | SlickNik: seems to be working, thx! | 17:43 |
*** yogeshmehra has joined #openstack-trove | 17:43 | |
SlickNik | esp: Sweet. Anytime! | 17:43 |
SlickNik | Quick reminder that we have our weekly Trove meeting in #openstack-meeting-alt in ~15 minutes. | 17:44 |
SlickNik | Agenda at: https://wiki.openstack.org/wiki/Meetings/TroveMeeting | 17:45 |
*** amcrn has joined #openstack-trove | 17:53 | |
*** yogeshmehra has quit IRC | 17:59 | |
*** doddstack has quit IRC | 18:00 | |
konetzed | woot woot | 18:02 |
*** SnowDust has joined #openstack-trove | 18:07 | |
*** ramishra has joined #openstack-trove | 18:12 | |
*** rueb7363 has joined #openstack-trove | 18:12 | |
*** ramishra has quit IRC | 18:16 | |
*** ranjitha has joined #openstack-trove | 18:27 | |
*** rustlebee has left #openstack-trove | 18:29 | |
*** sbfox has joined #openstack-trove | 18:31 | |
*** ramashri has joined #openstack-trove | 18:57 | |
*** ramashri has quit IRC | 19:00 | |
*** ramashri_ has joined #openstack-trove | 19:01 | |
*** eghobo has quit IRC | 19:01 | |
amrith | SlickNik, we need a recount with paper ballot | 19:05 |
amrith | I'm contacting Scalia, Roberts, ... and three of my other good friends | 19:05 |
amrith | they're the ones who will decide it eventually | 19:05 |
juice | so what patches do not requires bugs? | 19:05 |
SlickNik | juice, TBD | 19:06 |
juice | what are the exceptions | 19:06 |
juice | next meeting? | 19:06 |
juice | to be continued... | 19:06 |
amrith | I can't tell you what the are but you'll know it when you see it. | 19:06 |
SlickNik | But my view would be things like Transifex, merges from requirements, doc patches. | 19:06 |
SnowDust | amrith, especially when you are syncing code from oslo u dont need file bug | 19:06 |
SlickNik | They aren't really bugs, and they aren't really bp's | 19:06 |
SnowDust | amrith, you are just fast forwarding your project to adopt to latest oslo that way | 19:07 |
SlickNik | amrith: lol at the reference. | 19:07 |
*** Barker has joined #openstack-trove | 19:08 | |
cp16net | juice: i dont think its black and white | 19:08 |
cp16net | i think there is some middle ground on needing a bug or not | 19:09 |
amrith | SlickNick, SnowDust sorry, phone call | 19:09 |
amrith | am back | 19:09 |
amrith | what's the reference you are making SlickNik | 19:09 |
cp16net | i think a blueprint adds new features which should be required | 19:09 |
amrith | SnowDust, the issue with the oslo change is that it was a partial change | 19:09 |
cp16net | bugs not alwasy | 19:09 |
cp16net | if they are a small change its not that big a deal | 19:09 |
SnowDust | cp16net, amrith i gave an example when u dont need either of them to catchup ... | 19:09 |
amrith | SnowDust, I just entered a second bug (the guy did have a bug) but now I have a second | 19:09 |
amrith | SnowDust, what's the example? | 19:10 |
SnowDust | spelling errors, comment enhancements ... are the other things for which you dont need either | 19:10 |
SlickNik | amrith: "I'll know it when I see it" | 19:10 |
amcrn | cp16net: 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 |
amrith | SlickNik, OK, sorry. I get it ;( | 19:10 |
amrith | SlickNik, am running underclocking today | 19:10 |
vgnbkr | +1 amrith | 19:10 |
amrith | SlickNik, new diet | 19:10 |
SnowDust | amrith, especially when you are syncing code from oslo u dont need file bug | 19:10 |
vgnbkr | +1 amcrn, not amrith | 19:10 |
amrith | SnowDust, why? | 19:10 |
SnowDust | that was an example scenario | 19:10 |
SnowDust | in the commit you clearly mention .. what is the intention | 19:11 |
amrith | SnowDust, I know you "don't have to". I'm saying that's not the preferred alternative. | 19:11 |
SnowDust | its not bug .. but a sync activity | 19:11 |
amrith | by your argument, bugs are redundant | 19:11 |
amcrn | SnowDust: 99% of the time i see a ton of details in the bug notes, but not in the commit msg | 19:11 |
amrith | blueprints are redundant | 19:11 |
amrith | just put everything in the commit comments | 19:11 |
amrith | the code is self-documenting | 19:11 |
SnowDust | amcrn : agree .. just -1 in that case | 19:11 |
amrith | all simple ;) | 19:11 |
*** ranjitha has quit IRC | 19:11 | |
amrith | no? | 19:12 |
SnowDust | when a commit msg is empty .. and doesnot even have "Closes Bug" or " Implements" | 19:12 |
SnowDust | just -1 | 19:12 |
SnowDust | :) | 19:12 |
SnowDust | its benevolent :D | 19:12 |
amcrn | SnowDust: yeah, you don't understand the ramifications on those who approve the patches | 19:12 |
amrith | SnowDust, I'm not getting your point. sorry ... | 19:12 |
*** ramishra has joined #openstack-trove | 19:12 | |
amrith | so here's my view of things | 19:13 |
SnowDust | amcrn : sry ! but i value ur time .. | 19:13 |
amrith | in order to make a change | 19:13 |
*** ramashri_ has quit IRC | 19:13 | |
amrith | you need some time from core to get it through | 19:13 |
amrith | and that time is valuable, they've got shit to do | 19:13 |
amrith | and you need time to make your change | 19:13 |
amrith | and you need time of people to review the code | 19:13 |
amrith | at the end of the day, all resources are finite | 19:13 |
amrith | and we all want to improve the product | 19:13 |
amrith | it would help in that process if we aligned resources | 19:13 |
amrith | so if there were bugs and bugs were prioritized to sprints | 19:14 |
amrith | that would align the resources | 19:14 |
amrith | if bp's were written, approved coded and then reviewed, it would align resources | 19:14 |
amrith | without these, a hunk of code with no rationale will linger | 19:14 |
amrith | and the project will appear to be unresponsive | 19:14 |
amrith | we have a desire for contribution from a community | 19:14 |
amrith | we want to be responsive | 19:14 |
amrith | yet we also want the project to move forward in a deliberate and methodical way | 19:15 |
amrith | a bug and a blue print, IMHO help that process | 19:15 |
* amrith steps off soap box | 19:15 | |
*** sriram_tesora has quit IRC | 19:15 | |
amrith | thoughts, anyone | 19:15 |
amrith | did i forget to turn on the mic | 19:15 |
amrith | am I on mute? ;) | 19:15 |
*** ranjitha has joined #openstack-trove | 19:15 | |
*** sriram_tesora has joined #openstack-trove | 19:16 | |
dougshelley66 | amrith press the mute button | 19:16 |
amcrn | amrith: i don't think any one would disagree with your sentiments | 19:16 |
*** ranjitha has quit IRC | 19:16 | |
* amrith punches the mute button | 19:16 | |
* glucas trips over somebody's soap box | 19:16 | |
amrith | amcrn, thx. but the devil is in the implementation detail(s) | 19:16 |
SnowDust | amrith : 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 IRC | 19:17 | |
amrith | So why would you resist a bug for a fix with a typo? | 19:18 |
SnowDust | only 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 |
amrith | so I think we're in agreement? | 19:19 |
SlickNik | So really. I didn't mean to get a holy war going with this one. | 19:20 |
* amrith dusts off Emacs Vs. vi - the definitive bible | 19:20 | |
SlickNik | There are cases when we've had some folks from other OS teams come to Trove, and contribute a small change. | 19:20 |
SlickNik | And the first thing they get pelted with, is no bug? −1. | 19:21 |
hub_cap | amrith: they have a new version "Emacs and those other editors, unabridged" | 19:21 |
hub_cap | and we should not operate differently from other projecs | 19:21 |
hub_cap | we should encourage contribution | 19:22 |
hub_cap | if its a bug, lets list it as such | 19:22 |
amrith | SlickNik, I understand what you are saying | 19:22 |
cp16net | amcrn: 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 midcyle | 19:22 |
hub_cap | but we dont have to be _insert phrase here_ about every change | 19:22 |
SlickNik | cp16net: As a quick example of what I mean: https://review.openstack.org/#/c/91166/ | 19:24 |
amrith | can we apply the Stewart test to changes and reasonably say that if it appears like a little change we just collectively let it go | 19:24 |
amrith | and there's going to be some edge cases where we go the wrong way. | 19:24 |
cp16net | SlickNik: i think that fine to not ahve a bug on it | 19:24 |
SlickNik | cp16net: same here. | 19:25 |
cp16net | SlickNik: but... https://review.openstack.org/#/c/88478 i think should have one | 19:25 |
SlickNik | cp16net: Also agreed. :) | 19:25 |
cp16net | hehe :) | 19:26 |
cp16net | mainly because i didnt completely understand when i was reading through the reason for making the changes initially | 19:26 |
* amrith goes to get tea | 19:26 | |
cp16net | i think the main benifit of the bug is to track the code changes that we make for a release | 19:28 |
SlickNik | cp16net: +1 | 19:29 |
cp16net | i should emphsize CODE | 19:29 |
* cp16net steps off soap box | 19:29 | |
cp16net | Bender: Bite my shiny, metal ass! | 19:29 |
* amrith steps on soap box. soap box breaks. | 19:30 | |
hub_cap | cp16net: perfect bender quote | 19:31 |
hub_cap | Bender: In order to get busy at maximum efficiency, I need a girl with a big, 400-ton booty. | 19:31 |
amrith | hub_cap, is that a typo? s/booty/bootd/ | 19:32 |
amrith | do I need a bug for that? | 19:32 |
hub_cap | yes | 19:32 |
hub_cap | yes u do | 19:32 |
cp16net | lol | 19:32 |
SnowDust | lol amrith | 19:33 |
*** SnowDust has quit IRC | 19:34 | |
esp | lol | 19:35 |
*** ihrachyshka has joined #openstack-trove | 19:35 | |
*** sbfox has quit IRC | 19:36 | |
*** sbfox has joined #openstack-trove | 19:37 | |
*** ihrachyshka has quit IRC | 19:38 | |
*** doddstack has joined #openstack-trove | 19:41 | |
*** michael-yu has joined #openstack-trove | 19:43 | |
*** michael-yu_ has joined #openstack-trove | 19:46 | |
*** michael-yu has quit IRC | 19:46 | |
*** michael-yu_ is now known as michael-yu | 19:46 | |
*** sbfox1 has joined #openstack-trove | 20:03 | |
*** sbfox has quit IRC | 20:04 | |
*** eghobo has joined #openstack-trove | 20:07 | |
*** ranjitha has joined #openstack-trove | 20:08 | |
*** mattgriffin has quit IRC | 20:12 | |
*** ranjitha has quit IRC | 20:13 | |
*** ramishra has joined #openstack-trove | 20:13 | |
*** sbfox1 has quit IRC | 20:14 | |
*** freyes_ has joined #openstack-trove | 20:15 | |
*** freyes_ has quit IRC | 20:15 | |
*** ramishra has quit IRC | 20: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 | |
openstackgerrit | Auston McReynolds proposed a change to openstack/trove: Fix prepare call for redis guest agent https://review.openstack.org/84863 | 20:29 |
openstackgerrit | Auston McReynolds proposed a change to openstack/trove: Fix prepare call for redis guest agent https://review.openstack.org/84863 | 20:36 |
cp16net | amcrn: on that review ^^ | 20:36 |
cp16net | its more than just redis though right? | 20:37 |
openstackgerrit | Auston McReynolds proposed a change to openstack/trove: Added support of resize-flavor to Redis datastore https://review.openstack.org/88709 | 20:37 |
*** radez is now known as radez_g0n3 | 20:38 | |
*** sriram_tesora has quit IRC | 20:40 | |
*** sriram_tesora has joined #openstack-trove | 20:41 | |
openstackgerrit | Auston McReynolds proposed a change to openstack/trove: Fix prepare call for redis guest agent https://review.openstack.org/84863 | 20:43 |
grapex | SlickNik amcrn : Here's the doc pull request if one of you want to approve it: https://review.openstack.org/#/c/87807/ | 20:45 |
openstackgerrit | Auston McReynolds proposed a change to openstack/trove: Fix prepare call for redis guest agent https://review.openstack.org/84863 | 20:45 |
*** eghobo has quit IRC | 20:46 | |
SlickNik | grapex: Oh, very cool! Thanks will take a look. | 20:47 |
*** shakayumi has joined #openstack-trove | 20:47 | |
*** shakayumi has quit IRC | 20:47 | |
*** michael-yu_ has joined #openstack-trove | 20:47 | |
*** michael-yu has quit IRC | 20:50 | |
openstackgerrit | Auston McReynolds proposed a change to openstack/trove: Added support of resize-flavor to Redis datastore https://review.openstack.org/88709 | 20:50 |
*** michael-yu has joined #openstack-trove | 20:51 | |
*** mattgriffin has joined #openstack-trove | 20:52 | |
*** michael-yu_ has quit IRC | 20:53 | |
*** eghobo has joined #openstack-trove | 20:54 | |
*** eghobo has quit IRC | 20:54 | |
*** eghobo has joined #openstack-trove | 20:54 | |
*** michael-yu_ has joined #openstack-trove | 20:58 | |
*** ranjitha has joined #openstack-trove | 20:59 | |
esmute | Hi all, can you guys review https://review.openstack.org/#/c/81379/ when you have a chance? It should be short and quick | 21:00 |
*** michael-yu has quit IRC | 21:00 | |
*** michael-yu has joined #openstack-trove | 21:00 | |
*** pdmars has quit IRC | 21:01 | |
*** michael-yu_ has quit IRC | 21:02 | |
*** ranjitha has quit IRC | 21:09 | |
*** ranjitha has joined #openstack-trove | 21:11 | |
*** ramishra has joined #openstack-trove | 21:14 | |
*** shakayumi has joined #openstack-trove | 21:15 | |
*** shakayumi has quit IRC | 21:15 | |
*** ramishra has quit IRC | 21:18 | |
cp16net | esmute i think i've seen _() stuff... | 21:19 |
*** ranjitha has quit IRC | 21:19 | |
*** sbfox has joined #openstack-trove | 21:22 | |
esmute | _()? | 21:23 |
cp16net | i just made 2 comment on that review | 21:23 |
*** sbfox has quit IRC | 21:25 | |
kevinconway | esmute: the _() adds the text to the pile for the translation service we use | 21:25 |
esmute | ahh for localization | 21:25 |
esmute | cool.. will do.. thanks | 21:26 |
cp16net | _("this is a hardcoded english phrase nobody will ever change") | 21:26 |
kevinconway | yeah, we've even had a contrib add a language already | 21:26 |
*** kevinconway has quit IRC | 21:26 | |
cp16net | kevinconway: you see how maybe languages there were? | 21:26 |
hub_cap | sweet! | 21:26 |
hub_cap | https://www.transifex.com/projects/p/trove | 21:26 |
cp16net | like 50 with all english translations for everyone of them | 21:27 |
esmute | cp16net: 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 called | 21:27 |
cp16net | lol | 21:27 |
cp16net | so technically its only 1 lang | 21:27 |
*** achampion has quit IRC | 21:27 | |
cp16net | esmute: oh gotcha | 21:28 |
hub_cap | im not sure what kevin was talkin about hhe | 21:28 |
hub_cap | *heh | 21:29 |
*** chipc has left #openstack-trove | 21:34 | |
*** sbfox has joined #openstack-trove | 21:34 | |
*** sbfox has quit IRC | 21:35 | |
*** sbfox has joined #openstack-trove | 21:38 | |
*** Barker has quit IRC | 21:43 | |
openstackgerrit | Steve Leon proposed a change to openstack/trove: Unmounting ephemeral if it has already been mounted https://review.openstack.org/81379 | 21:54 |
*** robertmyers has quit IRC | 21:57 | |
*** yidclare has quit IRC | 22:05 | |
amcrn | SlickNik: 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 |
esmute | hey amcrn, what kind of concern do you have about https://review.openstack.org/#/c/89972/? | 22:07 |
TomasR | Guys, I'm working on recreating a guest image from scratch. | 22:09 |
TomasR | I followed the instructions from here: http://docs.openstack.org/developer/trove/dev/manual_install.html | 22:10 |
TomasR | and I do see an empty /etc/guest_info file on newly spawn VM. | 22:10 |
TomasR | however the instruction is a bit confusing as it doesn't say anything about installing guest agent into the image | 22:11 |
TomasR | So 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 image | 22:12 |
amcrn | TomasR: this missing /etc/guest_info file looks to be a bug | 22:14 |
*** ramishra has joined #openstack-trove | 22:15 | |
dougshelley66 | amcrn, cp16net: could you take a look at https://review.openstack.org/#/c/91219/ | 22:15 |
esp | amcrn: TomasR: I ran into this last week and I think SlickNik told me to rebase, not sure if it applies here | 22:15 |
amcrn | esp: i just reproduced it on a brand new devstack with absolutely no code changes | 22:15 |
esp | amcrn: k | 22:16 |
esp | I’ll try to give it a spin on my devstack | 22:16 |
esp | actually you know what I do remember trying to build an instance that had no guest_info recently | 22:17 |
esp | probably is a bug somewhere but it didn’t repro for me | 22:17 |
*** jcru has quit IRC | 22:18 | |
amcrn | esp: yeah, it seems to be intermittent | 22:18 |
esp | amcrn: I remember having to craft a guest_info and that seemed allow the guest agent to start up… | 22:19 |
esp | not cool | 22:19 |
*** ramishra has quit IRC | 22:19 | |
*** demorris has quit IRC | 22:25 | |
juice | amcrn: about the bad rebase yesterday...does this look better? https://review.openstack.org/#/c/84555/ | 22:26 |
openstackgerrit | A change was merged to openstack/trove-integration: Remove the iptables workaround https://review.openstack.org/84555 | 22:28 |
*** yidclare has joined #openstack-trove | 22:30 | |
SlickNik | amcrn: That's interesting. I haven't hit that particular issue. | 22:32 |
amcrn | juice: yes :) | 22:32 |
juice | amcrn: great :) | 22:32 |
juice | I knew I could do it | 22:32 |
SlickNik | amcrn: I wonder what triggers the bug, and whether it has affected any of the rdjenkins-gate runs. | 22:34 |
*** achampion has joined #openstack-trove | 22:34 | |
SlickNik | amcrn: Not enough to cause a noticeable blip, I guess. | 22:34 |
SlickNik | esp: The one that called for a rebase was a slightly different issue (It had to do with enforcing global-requirements, IIRC). | 22:35 |
esp | SlickNik: k | 22:36 |
amcrn | i think i have a hunch, but i'm going to try some things about before i say anything and look like a fool | 22:36 |
SlickNik | amcrn: Sure, leave me in suspense while you go "try some things out…" :) | 22:39 |
SlickNik | aka. Okay sounds good, let me know what you find. | 22:39 |
amcrn | :D | 22:39 |
*** doddstack has quit IRC | 22:40 | |
amcrn | esmute: 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 |
esp | amcrn: nice | 22:44 |
esmute | amcrn: the client is reflecting the API. Given that the BP was approved. | 22:44 |
esmute | what i can do is add this to the monday meeting so we can discuss further about it | 22:45 |
amcrn | esmute: that's what the aim was, apologies if that wasn't clear. | 22:45 |
*** ramishra has joined #openstack-trove | 22:47 | |
*** eguz has joined #openstack-trove | 22:48 | |
*** ramishra has quit IRC | 22:48 | |
*** ramishra has joined #openstack-trove | 22:49 | |
*** eghobo has quit IRC | 22:52 | |
*** ramishra has quit IRC | 22:54 | |
*** achampio1 has joined #openstack-trove | 22:59 | |
*** flaper87 is now known as flaper87|afk | 23:00 | |
*** achampion has quit IRC | 23:02 | |
openstackgerrit | Nikhil Manchanda proposed a change to openstack/trove-integration: Trove guestagent should not use sample conf https://review.openstack.org/88478 | 23:03 |
openstackgerrit | Nikhil Manchanda proposed a change to openstack/trove-integration: Add support for a neutron-based install https://review.openstack.org/78123 | 23:03 |
*** jmontemayor has quit IRC | 23:03 | |
*** sriram_tesora has quit IRC | 23:12 | |
openstackgerrit | Saurabh Surana proposed a change to openstack/trove-integration: updating sources list as part of percona elements https://review.openstack.org/90791 | 23:15 |
*** michael-yu has quit IRC | 23:19 | |
TomasR | JFYI - 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.html | 23:23 |
TomasR | and still can't get information automatically injected into /etc/guest_info | 23:24 |
SlickNik | TomasR: Could you verify that file injection/server personality is working correctly for an instance booted through nova? | 23:26 |
SlickNik | TomasR: Trove relies on this nova feature | 23:27 |
TomasR | SlickNik, how to check it? I'm new to Trove and OpenStack, is there a link in the docs? | 23:27 |
esp | maybe try to inject another file? or location? | 23:28 |
esp | trove 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 |
abramley | devstack sets force_config_drive=always to enable file injection | 23:29 |
*** demorris has joined #openstack-trove | 23:30 | |
esp | TomasR: https://github.com/openstack/trove/blob/master/trove/taskmanager/models.py#L553-L560 | 23:30 |
abramley | which you would set in nova.conf on your compute node | 23:31 |
esp | cool abramley :) | 23:31 |
TomasR | esp, I can see this in nova logs | 23:31 |
TomasR | nova-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_on | 23:31 |
TomasR | ... | 23:32 |
TomasR | "personality": [{"path": "/etc/guest_info", "contents": "W0RF... | 23:32 |
*** rueb7363 has quit IRC | 23:32 | |
TomasR | and then I do base64 decode of contents, I see, guest_id, tenant_id | 23:32 |
esp | TomasR: so trove is sending it ... | 23:32 |
esp | TomasR: is that really the contents value “WORF.. | 23:33 |
esp | ? | 23:33 |
TomasR | it's not the full value | 23:33 |
esp | k | 23:33 |
TomasR | I was able to decode that string into this | 23:33 |
abramley | trove 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 instance | 23:34 |
esp | let me check my nova log | 23:34 |
TomasR | [DEFAULT] guest_id=85292f2a-a336-4409-9889-a5bb646859ff datastore_manager=mysql tenant_id=7d359600815144eeac268a10efaba62b | 23:34 |
esp | k | 23:34 |
abramley | in havana libvirt inject partition was most likely enabled - but in icehouse this is now disabled by default and deprecated I believe | 23:34 |
abramley | So using config_drive seems like the right way to do it | 23:34 |
esp | abramley: that makes sense, we heard file injection is going away | 23:34 |
abramley | Today 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 calls | 23:35 |
openstackgerrit | Nikhil Manchanda proposed a change to openstack/trove-integration: Simplify Building Trove Guest Images https://review.openstack.org/91487 | 23:36 |
abramley | I can create a BP to add this in | 23:36 |
abramley | ... assuming people think that is the right way to do this | 23:36 |
esp | thx abramley | 23:38 |
esp | TomasR: my nova logs look sames as yours | 23:38 |
esp | TomasR: I’m checking configs now | 23:39 |
TomasR | I added force_config_drive=true to nova.conf to see if it makes any difference | 23:39 |
abramley | You 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-2 | 23:40 |
esp | [libvirt] | 23:40 |
esp | inject_partition = -2 | 23:40 |
esp | ^ that in my nova.conf on devstack | 23:41 |
esp | *thats | 23:41 |
abramley | esp - right so that is the old deprecated partition injection - and -2 means disabled | 23:41 |
esp | abramley: k, I just built it new 2 days ago | 23:41 |
*** grapex has quit IRC | 23:42 | |
TomasR | esp, my config is here: https://gist.github.com/anonymous/11386531 | 23:44 |
TomasR | maybe you can put your config on gist, so I can check tomorrow morning? | 23:44 |
TomasR | have to go to sleep now as it's 00:45am | 23:44 |
esp | TomasR: k, give me a min | 23:45 |
esp | TomasR: but I the param I showed you was only in the nova.conf .. coming right up | 23:45 |
TomasR | p.s. force_config_drive=true didn't help, I still can't get guest_id injected | 23:46 |
TomasR | 2014-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:40 | 23:46 |
esp | TomasR: abramley: https://gist.github.com/dan1/1d90c1e656ca817a8acb | 23:46 |
esp | nova.conf from devstack ^ | 23:46 |
abramley | I think it is meant to be force_config_drive=always | 23:47 |
abramley | but I have seen conflicting info in various documents about that - I am using 'always' and it works for me | 23:47 |
TomasR | it is "=true" according to this document: http://docs.openstack.org/grizzly/openstack-compute/admin/content/config-drive.html | 23:48 |
abramley | esp - yes that has the force_config_drive = always - that matches what we are doing | 23:48 |
esp | abramley: gotcha | 23:48 |
TomasR | ok,guys, have to go now, I'll resume tomorrow. good luck | 23:49 |
esp | k, see ya later. | 23:49 |
esp | thx for the info TomasR and abramley | 23:49 |
*** ramishra has joined #openstack-trove | 23:50 | |
*** venkater has joined #openstack-trove | 23:52 | |
*** ramishra has quit IRC | 23:54 | |
*** achampion has joined #openstack-trove | 23:54 | |
*** demorris has quit IRC | 23:55 | |
*** demorris has joined #openstack-trove | 23:56 | |
*** achampio1 has quit IRC | 23:56 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!