Monday, 2017-06-19

*** gkadam has joined #openstack-shade03:02
*** gouthamr has quit IRC03:17
*** gouthamr has joined #openstack-shade03:58
*** gkadam has quit IRC04:01
*** gouthamr has quit IRC04:02
*** gkadam has joined #openstack-shade04:03
*** gkadam has quit IRC04:06
*** gkadam has joined #openstack-shade04:06
*** gkadam has quit IRC04:09
*** gkadam has joined #openstack-shade04:10
*** gkadam_ has joined #openstack-shade04:23
*** gkadam has quit IRC04:26
*** yfried has joined #openstack-shade05:57
*** gkadam_ is now known as gkadam06:44
*** openstackgerrit has quit IRC07:48
*** nokes has joined #openstack-shade09:29
*** noshankus has quit IRC09:29
*** nokes is now known as noshankus09:29
*** ioggstream has joined #openstack-shade09:33
*** gkadam has quit IRC10:28
*** frickler has quit IRC10:30
*** frickler has joined #openstack-shade10:30
*** gkadam has joined #openstack-shade10:30
*** noshankus has quit IRC11:17
lastmikoimordred: hello ! I'm still running on the development branch of shade pending release, and I would like to report that cloudinit user_data format on server creation is broken since I04c3b08d80cf496dabec8ce7000ca5fb679fe3b9 "RESTify create_server"11:33
lastmikoiIndeed, it's no longer base64-encoded, hence the virtual machine becomes unable to parse that data11:34
lastmikoiI've used git bisect to single out the commit, so there's a good chance the bug was introduced then11:34
lastmikoimake that 36b659eed403aea3ecf8666a68524b8e0f8ef494, I don't know what I pasted earlier but that's no git commit hash...12:02
*** gkadam has quit IRC12:31
*** gouthamr has joined #openstack-shade12:53
mordredlastmikoi: oh - thank you!12:57
*** openstackgerrit has joined #openstack-shade13:18
openstackgerritMonty Taylor proposed openstack-infra/shade master: base64 encode user_data sent to create server  https://review.openstack.org/47538513:18
mordredlastmikoi: ^^ I think that should fix you - and should also be the last patch we need before releasing your other fix (sorry for the delay on that, we had some gate issues last week)13:21
openstackgerritMonty Taylor proposed openstack-infra/shade master: base64 encode user_data sent to create server  https://review.openstack.org/47538513:44
mordredlastmikoi: this time with a test :)13:44
*** yfried has quit IRC14:00
lastmikoimordred: great to hear :) No worries on my end about the delay, I still have a fair bit of work before the playbook I'm working on would expect a released shade to be installed14:05
mordredlastmikoi: ok good - and thankfully you found a good bug :)14:05
*** yfried has joined #openstack-shade14:07
rodsmordred just wondering if there's a mistake or something here https://github.com/openstack-infra/shade/blob/master/shade/openstackcloud.py#L280214:12
rodsseems a little weird to have a `get_server_by_id` method that calls `meta.add_server_interfaces`14:12
lastmikoimordred: cherry-picked the commit on top of the current master branch and the issue seems indeed fixed, thanks again !14:14
rodsmordred nm, I see the reason here https://github.com/openstack-infra/shade/commit/060ce2e055cfceabd22a97801f4cf33c227aa6a214:21
*** gouthamr has quit IRC14:26
*** gouthamr has joined #openstack-shade14:28
openstackgerritMonty Taylor proposed openstack-infra/shade master: base64 encode user_data sent to create server  https://review.openstack.org/47538514:48
samueldmqmorning shade14:58
mordredmorning samueldmq !14:58
samueldmqmordred: o/14:58
samueldmqmordred: is all the code in https://review.openstack.org/#/c/475385 copied over from novaclient15:00
samueldmq?15:00
mordredsamueldmq: well - half from novaclient and half from oslo.utils15:08
openstackgerritMonty Taylor proposed openstack-infra/shade master: base64 encode user_data sent to create server  https://review.openstack.org/47538515:13
openstackgerritMonty Taylor proposed openstack-infra/shade master: Try all the detected encodings, not just one  https://review.openstack.org/47542415:13
mordredsamueldmq: renamed the method - good point15:14
*** yfried has quit IRC15:18
*** slaweq has joined #openstack-shade15:37
*** slaweq has quit IRC16:03
*** slaweq has joined #openstack-shade16:52
*** slaweq has quit IRC16:54
*** slaweq has joined #openstack-shade16:56
*** slaweq has quit IRC17:00
*** slaweq has joined #openstack-shade17:01
*** slaweq has quit IRC17:07
openstackgerritMerged openstack-infra/shade master: Convert remaining nova tests to requests_mock  https://review.openstack.org/47519017:08
openstackgerritMerged openstack-infra/shade master: Translate final nova calls to REST  https://review.openstack.org/47519117:09
openstackgerritMerged openstack-infra/shade master: Remove novaclient from shade's dependencies  https://review.openstack.org/47519217:09
*** slaweq has joined #openstack-shade17:55
*** zigo has quit IRC18:05
*** zigo has joined #openstack-shade18:09
*** slaweq has quit IRC18:16
*** jamielennox has quit IRC18:19
*** jamielennox has joined #openstack-shade18:21
*** slaweq has joined #openstack-shade18:27
*** purplerbot has quit IRC18:27
*** slaweq has quit IRC18:31
openstackgerritMonty Taylor proposed openstack-infra/shade master: Try all the detected encodings, not just one  https://review.openstack.org/47542418:43
openstackgerritMonty Taylor proposed openstack-infra/shade master: base64 encode user_data sent to create server  https://review.openstack.org/47538518:43
*** ioggstream has quit IRC18:50
openstackgerritMonty Taylor proposed openstack-infra/shade master: Try all the detected encodings, not just one  https://review.openstack.org/47542418:53
openstackgerritMonty Taylor proposed openstack-infra/shade master: base64 encode user_data sent to create server  https://review.openstack.org/47538518:53
openstackgerritMonty Taylor proposed openstack-infra/shade master: Try all the detected encodings, not just one  https://review.openstack.org/47542419:01
openstackgerritMonty Taylor proposed openstack-infra/shade master: base64 encode user_data sent to create server  https://review.openstack.org/47538519:21
mordredShrews: so - did you see I 86'd novaclient over the weekend?19:29
Shrewsmordred: i did not, buy \o/19:31
Shrewss/buy/but/19:32
*** slaweq has joined #openstack-shade19:35
openstackgerritRosario Di Somma proposed openstack-infra/shade master: Don't remove top-container element for server REST API calls  https://review.openstack.org/47550119:39
mordredShrews: if you hav ea sec, https://review.openstack.org/#/c/475385/ could use another bump19:41
*** gouthamr has quit IRC20:15
*** gouthamr has joined #openstack-shade20:43
openstackgerritMerged openstack-infra/shade master: base64 encode user_data sent to create server  https://review.openstack.org/47538520:55
*** mgagne has quit IRC21:21
*** mgagne has joined #openstack-shade21:22
*** slaweq has quit IRC22:31
*** slaweq has joined #openstack-shade22:34
*** slaweq has quit IRC22:35
openstackgerritMerged openstack-infra/shade master: Don't remove top-container element for server REST API calls  https://review.openstack.org/47550122:43

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