Thursday, 2018-09-27

*** slaweq has joined #openstack-sdks00:11
*** slaweq has quit IRC00:15
*** mriedem_away has quit IRC00:18
*** gildub_ has joined #openstack-sdks00:24
*** slaweq has joined #openstack-sdks01:11
*** slaweq has quit IRC01:15
*** edisonxiang has joined #openstack-sdks01:51
*** chenyb4 has joined #openstack-sdks02:10
*** imacdonn has quit IRC02:50
*** markvoelker has joined #openstack-sdks02:50
*** imacdonn has joined #openstack-sdks02:51
*** openstackgerrit has joined #openstack-sdks03:07
openstackgerritMerged openstack/openstacksdk master: Normalize security groups when using Neutron  https://review.openstack.org/60214703:07
*** slaweq has joined #openstack-sdks03:11
*** slaweq has quit IRC03:15
openstackgerritMerged openstack/openstacksdk master: Allow JMESPath on searching networking resources  https://review.openstack.org/59907803:26
*** dave-mccowan has quit IRC03:46
*** edisonxiang has quit IRC03:53
*** haleyb has quit IRC04:07
*** dave-mccowan has joined #openstack-sdks04:50
*** slaweq has joined #openstack-sdks05:11
*** slaweq has quit IRC05:15
*** slaweq has joined #openstack-sdks06:11
*** slaweq has quit IRC06:15
*** slaweq has joined #openstack-sdks06:23
*** Luzi has joined #openstack-sdks07:06
*** gkadam has joined #openstack-sdks07:36
*** gildub_ has quit IRC07:41
*** tosky has joined #openstack-sdks07:45
*** jpich has joined #openstack-sdks07:47
*** e0ne has joined #openstack-sdks08:12
*** dtantsur|afk is now known as dtantsur08:36
*** ttsiouts has joined #openstack-sdks08:49
*** gkadam is now known as gkadam-afk08:52
*** ttsiouts has quit IRC08:58
*** gkadam-afk is now known as gkadam08:59
*** ttsiouts has joined #openstack-sdks09:00
*** a-pugachev has joined #openstack-sdks09:01
*** cdent has joined #openstack-sdks09:44
*** gildub_ has joined #openstack-sdks10:04
*** purplerbot has joined #openstack-sdks10:08
*** cdent has quit IRC10:08
*** chenyb4 has quit IRC10:12
openstackgerritMerged openstack/openstacksdk master: Change the method of role update  https://review.openstack.org/60067110:38
*** ttsiouts has quit IRC10:41
*** cdent has joined #openstack-sdks11:00
*** tosky has quit IRC11:44
*** tosky has joined #openstack-sdks11:45
*** ttsiouts has joined #openstack-sdks11:45
*** olivierb has joined #openstack-sdks11:47
*** olivierb has quit IRC11:58
*** olivierb has joined #openstack-sdks11:59
*** e0ne has quit IRC12:09
*** e0ne has joined #openstack-sdks12:09
*** mriedem has joined #openstack-sdks12:22
openstackgerritMonty Taylor proposed openstack/openstacksdk master: Remove all the deprecated stuff  https://review.openstack.org/60550812:26
* dtantsur links this ^^^ commit summary12:27
dtantsurs/links/likes/12:27
mordred\o/12:27
mordreddtantsur: I love deleting code12:27
mordreddtroyer: if you have any spare time amongst all of the starlingxing ... https://review.openstack.org/#/c/600292 is the last main big-boy before 1.0 and is ready for review. it's GIANT, so I don't blame you if you don't want to review it - but a lot of the patch is just removing unneeded lines from unit tests12:28
*** mriedem has quit IRC12:34
toskymordred: hi! Do you remember us discussing about https://review.openstack.org/#/c/589465/ ? Apparently openstackclient-check-plugins is still failing on older stable branches12:39
toskyfor example http://logs.openstack.org/36/588336/4/check/openstackclient-check-plugins/348b548/12:39
mordredtosky: oy. that's ugly12:47
mordredtosky: I remember talking about it - I don't remember what we said or if we had come up with a solution that just needed implementing12:48
*** mriedem has joined #openstack-sdks12:49
toskymordred: I think that the last discussion (before the PTG) was more or less "oh, ok, something is wrong"12:53
mordredtosky: darn.12:58
mordredtosky: well, I agree with pre-ptg us12:58
*** bobh has joined #openstack-sdks12:59
toskymordred: could it be that openstackclient runs the tests from its master branch, thus requirement master requirements.txt, which conflicts with the requirements from a stable branch?13:10
toskythat would explain the issue with python-keystonclient 3.15.0 installed, but conflicting with the requirement of python-keystoneclient 3.17.013:10
*** ttsiouts has quit IRC13:19
mordredtosky: it should be stable/queens of osc -- http://logs.openstack.org/36/588336/4/check/openstackclient-check-plugins/348b548/job-output.txt.gz#_2018-09-27_12_18_28_09508513:19
mordredoh - but openstack/openstackclient is master13:20
toskymordred: that's python-openstackclient, not openstackclient13:20
toskyyep13:20
mordredhttp://logs.openstack.org/36/588336/4/check/openstackclient-check-plugins/348b548/job-output.txt.gz#_2018-09-27_12_18_31_88208613:20
mordredyeah13:20
mordredbut requirements is also on stable/queens http://logs.openstack.org/36/588336/4/check/openstackclient-check-plugins/348b548/job-output.txt.gz#_2018-09-27_12_18_34_62201513:21
toskymordred: but this requirements.txt comes from openstackclient, so from master http://git.openstack.org/cgit/openstack/openstackclient/tree/playbooks/openstackclient-check-plugins/run.yaml#n1113:22
*** e0ne has quit IRC13:22
mordredyah13:22
mordredI wonder if we need stable branch versions of that repo13:22
mordreddtroyer, dhellmann: ^^ I'm not 100% sure of how that hangs together13:23
toskyit used to have stable branches - was everything originally under openstack/openstackclient and then the main code was moved to openstack/python-openstackclient?13:23
*** e0ne has joined #openstack-sdks13:26
dhellmanntosky : no, the code was always in openstack/python-openstackclient. The openstackclient repo is a meta-package for pulling in all of the extra dependencies to make it easier to install13:30
dhellmanntosky , mordred ,dtroyer : if we're testing against things that branch, we need to branch13:30
dhellmanneven if we never do anything with the branches, that's just how the test machinery works13:31
toskyI guess it would make sense to keep the job definition in master only13:34
toskyso that the only thing to take care is the check_osc_commands.py13:34
toskythough in openstackclient has code in stable/queens13:35
*** ttsiouts has joined #openstack-sdks13:35
toskyuh, or there is something weird in my local checkout13:38
toskyfor some reason I needed `git remote prune origin` in my local checkout of openstackclient13:40
*** gildub_ has quit IRC13:44
dtroyerit is totally possible that the openstackclient repo is not properly branched…  I had wanted to move the plugin tests there but don't think that was ever completed, so it needs to be handled like the main repo13:46
dtroyermordred: I am release-branching stx today if zuul ever catches up, but that may give me an excuse to slip into a dark corner and refrsh myself with sdk :)13:47
dtroyerI see how you pull the strings behind the scenes :)13:48
mordreddtroyer: I've gotta do _something_ :)13:49
*** Horrorcat has quit IRC14:06
*** Leo_m has joined #openstack-sdks14:09
*** ttsiouts has quit IRC14:15
*** ttsiouts has joined #openstack-sdks14:30
*** ttsiouts has quit IRC14:58
*** ttsiouts has joined #openstack-sdks14:59
*** ttsiouts has quit IRC15:04
*** Luzi has quit IRC15:13
*** ttsiouts has joined #openstack-sdks15:20
*** ttsiouts has quit IRC15:23
*** dave-mccowan has quit IRC15:23
*** dave-mccowan has joined #openstack-sdks15:41
*** e0ne has quit IRC15:51
*** dave-mccowan has quit IRC16:08
*** jpich has quit IRC16:12
*** cdent has quit IRC16:26
*** a-pugachev has quit IRC16:34
*** bobh has quit IRC16:37
elmikoedleafe dtantsur let me know how this strikes you https://etherpad.openstack.org/p/api-sig-newsletter16:41
*** bobh has joined #openstack-sdks16:42
edleafeelmiko: that's not a python docstring :)16:42
elmikohaha ;)16:42
elmikooh you... always making things so /grammatical/16:42
edleafeI removed the "meeting agenda" from the bottom16:43
elmiko++ thanks16:43
edleafeI guess I need to update the meeting in governance16:44
elmikoi took an action for that, but if you want to no worries =)16:44
edleafeoh, go ahead. I didn't know you took that action16:44
elmikoi snuck it in the meeting XD16:45
elmikoi was gonna follow the format for the other office hours entries, i'll make the pr shortly16:45
elmikoam i good to publish on the newsletter?16:45
edleafesend it16:46
elmikothanks!16:46
dtantsurelmiko: LGTM16:49
elmikothanks16:50
*** e0ne has joined #openstack-sdks16:52
*** cdent has joined #openstack-sdks16:53
*** dtantsur is now known as dtantsur|afk16:54
*** e0ne has quit IRC16:54
edleafeelmiko: dtantsur|afk: I updated the meetings page: https://wiki.openstack.org/wiki/Meetings/API-SIG16:56
edleafeelmiko: you're going to take care of the changes in openstack-infra/irc-meetings, right?16:57
elmikoedleafe: https://review.openstack.org/#/c/605808/16:58
elmikothanks ed !16:59
*** e0ne has joined #openstack-sdks17:16
*** cdent has quit IRC17:16
*** gkadam has quit IRC17:20
mnaserhttps://review.openstack.org/#/c/589012 -- can we have eyes on this?17:32
mnaserit's impossible to do live migrations without force specifying a host which isnt ideal17:33
*** dims_ is now known as dims18:05
*** bobh has quit IRC18:12
*** olivierb has quit IRC18:15
*** olivierb has joined #openstack-sdks18:16
*** olivierb has quit IRC18:43
*** olivierb has joined #openstack-sdks18:43
*** bobh has joined #openstack-sdks18:45
mordredmnaser: how about nargs='?' to make the argument to --live be optional19:05
mnasermordred: i like that, it doesn't break the backwards compatibility19:05
mnasermordred: i dunno.. should i respin that patch? push another one and this one can stay as an alternate when we want to further refactor things?19:06
mordredmnaser: might need to also add const='<nonce>' - which will tell argparse to put '<nonce>' in as the argument to --live if none is otherwise given19:06
mnasercould we use default=None?19:06
mordredwell, I don't think the patch in its current state is viable bdcause it's a total breaking change19:06
mnaseri think that is the most preferable thing on the nova api side of things19:06
mordredmnaser: maybe? it's worth a try19:06
mordredmnaser: so - default is the value that will be put into the variable if --live isn't given. const is the value that is put in if --live is given with no argument19:07
mnasermordred: ah okay, so const is what we want then19:08
mordredmnaser: so maybe nargs='?', const=False, default=None19:08
mordredthat way you can easily tell all three cases19:08
mnaserok ill try to look at that, i should be paying attention to the call i'm in.. but zzz19:08
mordredmnaser: bah. calls are boring19:10
*** dayou has quit IRC19:10
mordredI left the feedback on the patch too19:10
*** bobh has quit IRC19:30
*** bobh has joined #openstack-sdks19:30
*** dayou has joined #openstack-sdks19:37
*** e0ne has quit IRC19:42
*** e0ne has joined #openstack-sdks19:47
*** e0ne has quit IRC19:50
*** bobh has quit IRC20:01
*** e0ne has joined #openstack-sdks20:08
openstackgerritMonty Taylor proposed openstack/openstacksdk master: Optimize the floating ip functional test a bit  https://review.openstack.org/60586020:10
*** dayou has quit IRC20:38
openstackgerritMonty Taylor proposed openstack/openstacksdk master: Remove all the deprecated stuff  https://review.openstack.org/60550820:56
*** dayou has joined #openstack-sdks21:06
*** e0ne has quit IRC21:14
*** bobh has joined #openstack-sdks21:18
*** bobh has quit IRC21:18
openstackgerritBrian Haley proposed openstack/python-openstackclient master: Fix some spaces in help messages  https://review.openstack.org/60320621:54
*** mriedem has quit IRC21:55
*** Leo_m has quit IRC22:01
*** mriedem has joined #openstack-sdks22:07
*** tosky has quit IRC22:53
*** olivierb has quit IRC23:08
*** olivierb has joined #openstack-sdks23:11
*** mriedem has quit IRC23:26
*** bobh has joined #openstack-sdks23:28
*** olivierb has quit IRC23:29
*** gildub_ has joined #openstack-sdks23:44

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