Thursday, 2019-06-06

*** markvoelker has quit IRC00:07
*** markvoelker has joined #openstack-sdks00:07
*** markvoelker has quit IRC00:12
*** bobh has quit IRC00:59
*** slaweq has quit IRC01:07
*** bobh has joined #openstack-sdks01:26
*** dave-mccowan has joined #openstack-sdks01:32
*** bobh has quit IRC01:48
*** whoami-rajat has joined #openstack-sdks02:07
*** markvoelker has joined #openstack-sdks02:08
*** markvoelker has quit IRC02:42
*** dave-mccowan has quit IRC03:50
*** dtantsur|afk is now known as dtantsur05:49
*** markvoelker has joined #openstack-sdks05:58
*** markvoelker has quit IRC06:02
*** slaweq has joined #openstack-sdks06:14
*** e0ne has joined #openstack-sdks06:19
*** holser_ has joined #openstack-sdks06:25
*** holser_ has quit IRC06:49
*** holser_ has joined #openstack-sdks06:50
*** gtema has joined #openstack-sdks06:58
*** e0ne has quit IRC07:05
*** e0ne has joined #openstack-sdks07:06
*** e0ne has quit IRC07:26
*** jpena|off is now known as jpena07:38
*** ttsiouts has joined #openstack-sdks07:43
*** ITD27M01_ has joined #openstack-sdks07:44
*** ralonsoh has joined #openstack-sdks07:58
*** ttsiouts has quit IRC07:58
*** markvoelker has joined #openstack-sdks07:58
*** ttsiouts has joined #openstack-sdks07:59
*** ttsiouts has quit IRC08:03
*** ttsiouts has joined #openstack-sdks08:07
*** ttsiouts has quit IRC08:22
*** ttsiouts has joined #openstack-sdks08:23
*** ttsiouts has quit IRC08:27
*** markvoelker has quit IRC08:32
*** ttsiouts has joined #openstack-sdks08:42
*** e0ne has joined #openstack-sdks08:49
openstackgerritzhangboye proposed openstack/shade master: Cap sphinx for py2 to match global requirements  https://review.opendev.org/66354508:52
*** markvoelker has joined #openstack-sdks09:29
*** markvoelker has quit IRC10:03
*** ttsiouts has quit IRC10:14
*** ttsiouts has joined #openstack-sdks10:14
*** ttsiouts has quit IRC10:19
*** jpena is now known as jpena|away10:36
*** gtema_ has joined #openstack-sdks10:42
*** gtema has quit IRC10:42
*** holser_ is now known as holser|lunch10:51
*** markvoelker has joined #openstack-sdks10:59
*** tssurya has joined #openstack-sdks11:15
tssuryadtroyer: hi! could you have a look at https://review.opendev.org/#/c/659124/ when you have time ? thanks in advance!11:16
ITD27M01_mordred: Hello! Are you online? Did you have time to see the https://github.com/ansible/ansible/pull/51401 ?11:26
*** markvoelker has quit IRC11:32
*** ttsiouts has joined #openstack-sdks11:47
*** gtema_ has quit IRC12:05
*** gtema_ has joined #openstack-sdks12:27
*** dave-mccowan has joined #openstack-sdks12:28
*** markvoelker has joined #openstack-sdks12:29
*** markvoelker has quit IRC13:03
*** holser|lunch is now known as holser_13:12
efrieddtantsur: You around to have a look at https://review.opendev.org/663439 ? I'm fixing up the test failures now, but wanted to catch you before you left.13:14
dtantsurefried: will have a short snack and check it13:14
efriedthank you sir13:14
dtantsurhaving the CI (on the way to being) green is appreciated :)13:15
efriedof course13:15
*** shubham_potale has joined #openstack-sdks13:28
*** gtema_ has quit IRC13:32
*** ttsiouts has quit IRC13:35
openstackgerritEric Fried proposed openstack/openstacksdk master: Handle oslo.config exceptions in from_conf  https://review.opendev.org/66343913:35
efrieddtantsur: That should do it ^13:35
efriedmordred: ^13:35
*** ttsiouts has joined #openstack-sdks13:36
*** ttsiouts has quit IRC13:40
mordredefried: oh good - I was just looking at that but hadn't yet figured out what was up with the add_service calls :)13:43
mordredShrews: if you're bored this morning, ^^ that has 2x+2 but one is from me and I wrote some of the patch, so might be worth having another set of eyes on13:45
mordredShrews: otoh - efried wrote most of it, so you don't have to worry about much terrible code from me :)13:45
*** ttsiouts has joined #openstack-sdks13:47
*** whoami-rajat has quit IRC13:47
efriedmordred, dtantsur: I'll add consolidation of the log message to the fixups catch-all I'm working on now.13:48
mordredcoolio13:48
mordreddtantsur: if you have another second, https://review.opendev.org/#/c/662698 by gtema could use another review - it's the underlying base for the rest of his stack13:49
dtantsurlgtm13:52
*** jpena|away is now known as jpena13:53
*** whoami-rajat has joined #openstack-sdks13:57
*** ricolin has joined #openstack-sdks13:57
mordredwoot!13:59
*** markvoelker has joined #openstack-sdks13:59
*** ttsiouts has quit IRC14:05
*** ttsiouts has joined #openstack-sdks14:06
*** ttsiouts has quit IRC14:11
*** gtema_ has joined #openstack-sdks14:16
*** tssurya has quit IRC14:20
*** ttsiouts has joined #openstack-sdks14:23
openstackgerritEric Fried proposed openstack/openstacksdk master: Minor fixups from from_conf changes  https://review.opendev.org/66368514:33
*** markvoelker has quit IRC14:33
efrieddtantsur, mordred: fixups ^14:33
efriedmordred: You can abandon https://review.opendev.org/#/c/663435/ now, yah?14:34
mordredyup! done14:35
mordredefried: oh - piddle. we didn't add a release note for the service disabling. let me add one real quick14:35
efriedmordred: In terms of "here's a new feature you can use"? Cause otherwise, who would notice?14:36
mordredyeah- well, there's a new behavior if you set has_foo in your config file14:38
mordred(also, I think we should cut a release as soon as these changes land, and it would be a release without any release notes to indicate what's in it otherwise :) )14:38
efriedheh14:40
openstackgerritMonty Taylor proposed openstack/openstacksdk master: Add release note for new disable service functionality  https://review.opendev.org/66369014:40
efriedmordred: comment inline...14:41
mordredyup. good call14:41
openstackgerritMonty Taylor proposed openstack/openstacksdk master: Add release note for new disable service functionality  https://review.opendev.org/66369014:44
efriedwhapow, big ol +114:45
efriedhttps://www.youtube.com/watch?v=ggXbzjnffAo14:46
mordredthis couch is cut in half15:07
openstackgerritMerged openstack/openstacksdk master: Support skipping unknown QP  https://review.opendev.org/66269815:08
*** markvoelker has joined #openstack-sdks15:29
*** e0ne has quit IRC15:45
elmikoAPI SIG office hour is now open!16:00
edleafeDamn, you beat me again16:00
elmikoi even waited that time XD16:00
edleafeWas just going to press Enter16:00
elmikoi figured you would beat me to it lol16:00
*** markvoelker has quit IRC16:02
elmikojust as a followup to the discussion that happened last week, i proposed a patch to keystoneauth that fixes the issue with api-sig suggested error formatting16:04
elmikothat patch has been merged into the master16:04
elmikothere was a question about backporting from the folks who ran into this originally, but i think that is better left to the keystoneauth team16:05
edleafeYeah, that patch looked sane16:05
elmiko=)16:06
*** whoami-rajat has quit IRC16:07
*** ttsiouts has quit IRC16:07
*** ttsiouts has joined #openstack-sdks16:08
* dtantsur is too busy today, sorry16:08
edleafeWe'll try to handle the load ourselves :)16:08
elmikohehe16:12
*** ttsiouts has quit IRC16:12
*** whoami-rajat has joined #openstack-sdks16:13
*** holser_ has quit IRC16:20
dtroyerstephenfin: if you're around I would love to have you take one last look at https://review.opendev.org/#/c/659382/ (—live-migration) before merging it.  This is the last thing before cutting a 3.19 release for OSC.16:35
dtroyermordred: ^^^ if you feel like looking at Something Completely Different(TM)16:35
stephenfindtroyer: Won't have time for a full review but I've made my peace with that approach. I've already fought with argparse too much, heh16:36
edleafeelmiko: Gotta run out a little early. Try to hold the fort while I'm gone.16:44
dtroyerstephenfin: ok, thanks.  I just wanted to make sure there were no outstanding concerns16:44
elmikoedleafe: ack, take care o/16:44
stephenfindtroyer: Not from me. My concern was with the duplication in 'migration --live-migration' but I haven't found a way to avoid that that doesn't involve rewriting argparse16:45
stephenfinwhich is a hole I went far too deep down16:45
dtroyerstephenfin: maybe in a year or so we can bring —live back as an alias :)  it would be a forced break for anyone attempting to give it an argument from the old syntax.  thanks again16:47
*** ricolin has quit IRC16:54
mordreddtroyer: lgtm!16:58
dtroyermordred: Shit it!    Thanks16:58
dtroyerooooooo!!!!!!!16:59
dtroyerSHIP16:59
dtroyerSHIP16:59
dtroyerSHIP16:59
* dtroyer heads off to remedial typing class16:59
*** markvoelker has joined #openstack-sdks16:59
mordredhahahaha16:59
*** mrhillsman is now known as openlab17:02
*** openlab is now known as codebauss17:05
*** dtantsur is now known as dtantsur|afk17:06
*** holser_ has joined #openstack-sdks17:06
*** holser_ has quit IRC17:08
*** holser_ has joined #openstack-sdks17:09
*** codebauss is now known as openlab17:13
*** openlab is now known as codebauss17:14
*** codebauss is now known as openlab17:15
*** bobh has joined #openstack-sdks17:15
*** openlab is now known as codebauss17:16
openstackgerritDean Troyer proposed openstack/osc-lib master: Add FakeModule from OSC  https://review.opendev.org/66023017:18
efriedmordred: lower constraints job is not happy, but I can't fathom why.17:18
*** yolanda__ has joined #openstack-sdks17:20
efriedos-service-types, I'll betcha.17:20
mordredefried: WEIRD17:21
efriedmordred: nah, we changed Connection's has_service to use CloudConfig's has_service, which is different in that it defaults to looking up the service type in ost.17:22
efriedmordred: Is it okay to bump the ost l-c to match the u-c?17:22
*** bobh has quit IRC17:22
mordredabsolutely17:22
mordredare we required to have a lower-constraints entry for every library?17:22
efriedight, suitable for a ninja approve if I patch that in right quick?17:22
*** yolanda has quit IRC17:23
mordredyup17:23
efriedI think you're required to have l-c for anything listed in requirements or test-requirements, yeah.17:23
*** jpena is now known as jpena|off17:23
mordrednod. well - makes sense I guess - be sure to bump the requirements lower bound too, obvs17:23
efriedI know we're keeping placement up to date with latest, forcefully.17:23
*** codebauss is now known as mrhillsman17:24
openstackgerritEric Fried proposed openstack/openstacksdk master: Handle oslo.config exceptions in from_conf  https://review.opendev.org/66343917:25
openstackgerritEric Fried proposed openstack/openstacksdk master: Minor fixups from from_conf changes  https://review.opendev.org/66368517:25
efriedmordred: ^, top rebased, will require only +W. Not sure if your reno patch will need rebase+W17:26
efriedbtw, ost 1.7.0 is out, not sure if we should keep sdk in sync17:27
efriedfwiw philosophy taken in placement is "we should always be pinning to the latest os-traits and os-resource-classes". So we actually put in tests that fail as soon as a new release hits upper-constraints.17:28
efriede.g. https://review.opendev.org/#/c/661132/17:28
openstackgerritMonty Taylor proposed openstack/openstacksdk master: Add release note for new disable service functionality  https://review.opendev.org/66369017:30
mordredefried: all three re-+A'd17:30
mordredefried: we probably shoudl do the same - I like that as an idea17:30
mordredsdk should always have latest ost17:30
*** markvoelker has quit IRC17:32
efriedmordred: okay, I'll propose a canary patch17:33
openstackgerritMerged openstack/python-openstackclient master: Add changes-before attribute to server list  https://review.opendev.org/65737317:40
openstackgerritEric Fried proposed openstack/openstacksdk master: Pin to latest os-service-types  https://review.opendev.org/66373317:47
efriedmordred: ^17:47
efriedI didn't know where to put it17:47
efriedI didn't want to make a whole new test module for it17:48
efriedIf you want me to do so, or have a better suggestion for where to put it, lmk.17:48
*** gtema_ has quit IRC17:50
*** holser_ has quit IRC17:54
*** ralonsoh has quit IRC18:04
mordredefried: I think that looks great18:14
*** e0ne has joined #openstack-sdks18:26
*** markvoelker has joined #openstack-sdks18:29
*** e0ne has quit IRC18:31
*** e0ne has joined #openstack-sdks18:32
*** e0ne has quit IRC18:38
openstackgerritMerged openstack/shade master: Cap sphinx for py2 to match global requirements  https://review.opendev.org/66354518:44
efriedI've bumped the nova side to use 0.30.0, which *should* make the bottom patch and the placement swapout go green.18:44
* efried crosses fingers18:44
mordredefried: hopefully we'll get you an 0.31 soon18:48
efriedmordred: That'll be neat... but I'm not sure I actually need it.18:48
efriedoh, never mind18:48
efriedwas gonna say, nova will be well-behaved and only use Connection for things it's supposed to.18:49
efriedBut that doesn't help the case where conf is misconfigured18:49
efriedthough that's a pretty cornery corner.18:49
efriedBecause we still have to pass in the session, which only exists if we had ksa session opts, which we pull in at the same time as adapter opts. So the only busted path is interface+valid_interfaces I think.18:50
mordrednod18:54
*** markvoelker has quit IRC19:02
openstackgerritMerged openstack/python-openstackclient master: Deprecate openstack server migrate --host option  https://review.opendev.org/65938219:05
*** e0ne has joined #openstack-sdks19:17
*** zbr has quit IRC19:25
*** ITD27M01_ has quit IRC19:50
*** markvoelker has joined #openstack-sdks19:59
*** e0ne has quit IRC20:05
*** markvoelker has quit IRC20:32
openstackgerritMerged openstack/openstacksdk master: Handle oslo.config exceptions in from_conf  https://review.opendev.org/66343920:50
openstackgerritMerged openstack/openstacksdk master: Add release note for new disable service functionality  https://review.opendev.org/66369020:50
openstackgerritMerged openstack/openstacksdk master: Minor fixups from from_conf changes  https://review.opendev.org/66368520:50
*** holser_ has joined #openstack-sdks20:55
openstackgerritEric Fried proposed openstack/openstacksdk master: Pin to latest os-service-types  https://review.opendev.org/66373320:57
efriedmordred: resolved merge conflict ^20:57
mordredefried: lookie there though - the other patches merged!20:58
efriedyup!20:58
efriedcould get a new release any time now20:58
efriedthough I'm still waiting for test results on the nova side - we might want to include fixes if those fail20:58
*** whoami-rajat has quit IRC21:27
*** markvoelker has joined #openstack-sdks21:29
*** markvoelker has quit IRC22:03
*** slaweq has quit IRC22:27
mordredefried: ++22:28
*** slaweq has joined #openstack-sdks22:55
*** markvoelker has joined #openstack-sdks22:59
*** slaweq has quit IRC23:08
*** slaweq has joined #openstack-sdks23:11
openstackgerritMerged openstack/python-openstackclient master: Batch up minor cleanups for release  https://review.opendev.org/65998223:19
*** holser_ has quit IRC23:20
*** slaweq has quit IRC23:24
*** markvoelker has quit IRC23:31

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