*** markvoelker has quit IRC | 00:07 | |
*** markvoelker has joined #openstack-sdks | 00:07 | |
*** markvoelker has quit IRC | 00:12 | |
*** bobh has quit IRC | 00:59 | |
*** slaweq has quit IRC | 01:07 | |
*** bobh has joined #openstack-sdks | 01:26 | |
*** dave-mccowan has joined #openstack-sdks | 01:32 | |
*** bobh has quit IRC | 01:48 | |
*** whoami-rajat has joined #openstack-sdks | 02:07 | |
*** markvoelker has joined #openstack-sdks | 02:08 | |
*** markvoelker has quit IRC | 02:42 | |
*** dave-mccowan has quit IRC | 03:50 | |
*** dtantsur|afk is now known as dtantsur | 05:49 | |
*** markvoelker has joined #openstack-sdks | 05:58 | |
*** markvoelker has quit IRC | 06:02 | |
*** slaweq has joined #openstack-sdks | 06:14 | |
*** e0ne has joined #openstack-sdks | 06:19 | |
*** holser_ has joined #openstack-sdks | 06:25 | |
*** holser_ has quit IRC | 06:49 | |
*** holser_ has joined #openstack-sdks | 06:50 | |
*** gtema has joined #openstack-sdks | 06:58 | |
*** e0ne has quit IRC | 07:05 | |
*** e0ne has joined #openstack-sdks | 07:06 | |
*** e0ne has quit IRC | 07:26 | |
*** jpena|off is now known as jpena | 07:38 | |
*** ttsiouts has joined #openstack-sdks | 07:43 | |
*** ITD27M01_ has joined #openstack-sdks | 07:44 | |
*** ralonsoh has joined #openstack-sdks | 07:58 | |
*** ttsiouts has quit IRC | 07:58 | |
*** markvoelker has joined #openstack-sdks | 07:58 | |
*** ttsiouts has joined #openstack-sdks | 07:59 | |
*** ttsiouts has quit IRC | 08:03 | |
*** ttsiouts has joined #openstack-sdks | 08:07 | |
*** ttsiouts has quit IRC | 08:22 | |
*** ttsiouts has joined #openstack-sdks | 08:23 | |
*** ttsiouts has quit IRC | 08:27 | |
*** markvoelker has quit IRC | 08:32 | |
*** ttsiouts has joined #openstack-sdks | 08:42 | |
*** e0ne has joined #openstack-sdks | 08:49 | |
openstackgerrit | zhangboye proposed openstack/shade master: Cap sphinx for py2 to match global requirements https://review.opendev.org/663545 | 08:52 |
---|---|---|
*** markvoelker has joined #openstack-sdks | 09:29 | |
*** markvoelker has quit IRC | 10:03 | |
*** ttsiouts has quit IRC | 10:14 | |
*** ttsiouts has joined #openstack-sdks | 10:14 | |
*** ttsiouts has quit IRC | 10:19 | |
*** jpena is now known as jpena|away | 10:36 | |
*** gtema_ has joined #openstack-sdks | 10:42 | |
*** gtema has quit IRC | 10:42 | |
*** holser_ is now known as holser|lunch | 10:51 | |
*** markvoelker has joined #openstack-sdks | 10:59 | |
*** tssurya has joined #openstack-sdks | 11:15 | |
tssurya | dtroyer: 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 IRC | 11:32 | |
*** ttsiouts has joined #openstack-sdks | 11:47 | |
*** gtema_ has quit IRC | 12:05 | |
*** gtema_ has joined #openstack-sdks | 12:27 | |
*** dave-mccowan has joined #openstack-sdks | 12:28 | |
*** markvoelker has joined #openstack-sdks | 12:29 | |
*** markvoelker has quit IRC | 13:03 | |
*** holser|lunch is now known as holser_ | 13:12 | |
efried | dtantsur: 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 |
dtantsur | efried: will have a short snack and check it | 13:14 |
efried | thank you sir | 13:14 |
dtantsur | having the CI (on the way to being) green is appreciated :) | 13:15 |
efried | of course | 13:15 |
*** shubham_potale has joined #openstack-sdks | 13:28 | |
*** gtema_ has quit IRC | 13:32 | |
*** ttsiouts has quit IRC | 13:35 | |
openstackgerrit | Eric Fried proposed openstack/openstacksdk master: Handle oslo.config exceptions in from_conf https://review.opendev.org/663439 | 13:35 |
efried | dtantsur: That should do it ^ | 13:35 |
efried | mordred: ^ | 13:35 |
*** ttsiouts has joined #openstack-sdks | 13:36 | |
*** ttsiouts has quit IRC | 13:40 | |
mordred | efried: oh good - I was just looking at that but hadn't yet figured out what was up with the add_service calls :) | 13:43 |
mordred | Shrews: 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 on | 13:45 |
mordred | Shrews: 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-sdks | 13:47 | |
*** whoami-rajat has quit IRC | 13:47 | |
efried | mordred, dtantsur: I'll add consolidation of the log message to the fixups catch-all I'm working on now. | 13:48 |
mordred | coolio | 13:48 |
mordred | dtantsur: 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 stack | 13:49 |
dtantsur | lgtm | 13:52 |
*** jpena|away is now known as jpena | 13:53 | |
*** whoami-rajat has joined #openstack-sdks | 13:57 | |
*** ricolin has joined #openstack-sdks | 13:57 | |
mordred | woot! | 13:59 |
*** markvoelker has joined #openstack-sdks | 13:59 | |
*** ttsiouts has quit IRC | 14:05 | |
*** ttsiouts has joined #openstack-sdks | 14:06 | |
*** ttsiouts has quit IRC | 14:11 | |
*** gtema_ has joined #openstack-sdks | 14:16 | |
*** tssurya has quit IRC | 14:20 | |
*** ttsiouts has joined #openstack-sdks | 14:23 | |
openstackgerrit | Eric Fried proposed openstack/openstacksdk master: Minor fixups from from_conf changes https://review.opendev.org/663685 | 14:33 |
*** markvoelker has quit IRC | 14:33 | |
efried | dtantsur, mordred: fixups ^ | 14:33 |
efried | mordred: You can abandon https://review.opendev.org/#/c/663435/ now, yah? | 14:34 |
mordred | yup! done | 14:35 |
mordred | efried: oh - piddle. we didn't add a release note for the service disabling. let me add one real quick | 14:35 |
efried | mordred: In terms of "here's a new feature you can use"? Cause otherwise, who would notice? | 14:36 |
mordred | yeah- well, there's a new behavior if you set has_foo in your config file | 14: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 |
efried | heh | 14:40 |
openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Add release note for new disable service functionality https://review.opendev.org/663690 | 14:40 |
efried | mordred: comment inline... | 14:41 |
mordred | yup. good call | 14:41 |
openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Add release note for new disable service functionality https://review.opendev.org/663690 | 14:44 |
efried | whapow, big ol +1 | 14:45 |
efried | https://www.youtube.com/watch?v=ggXbzjnffAo | 14:46 |
mordred | this couch is cut in half | 15:07 |
openstackgerrit | Merged openstack/openstacksdk master: Support skipping unknown QP https://review.opendev.org/662698 | 15:08 |
*** markvoelker has joined #openstack-sdks | 15:29 | |
*** e0ne has quit IRC | 15:45 | |
elmiko | API SIG office hour is now open! | 16:00 |
edleafe | Damn, you beat me again | 16:00 |
elmiko | i even waited that time XD | 16:00 |
edleafe | Was just going to press Enter | 16:00 |
elmiko | i figured you would beat me to it lol | 16:00 |
*** markvoelker has quit IRC | 16:02 | |
elmiko | just 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 formatting | 16:04 |
elmiko | that patch has been merged into the master | 16:04 |
elmiko | there was a question about backporting from the folks who ran into this originally, but i think that is better left to the keystoneauth team | 16:05 |
edleafe | Yeah, that patch looked sane | 16:05 |
elmiko | =) | 16:06 |
*** whoami-rajat has quit IRC | 16:07 | |
*** ttsiouts has quit IRC | 16:07 | |
*** ttsiouts has joined #openstack-sdks | 16:08 | |
* dtantsur is too busy today, sorry | 16:08 | |
edleafe | We'll try to handle the load ourselves :) | 16:08 |
elmiko | hehe | 16:12 |
*** ttsiouts has quit IRC | 16:12 | |
*** whoami-rajat has joined #openstack-sdks | 16:13 | |
*** holser_ has quit IRC | 16:20 | |
dtroyer | stephenfin: 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 |
dtroyer | mordred: ^^^ if you feel like looking at Something Completely Different(TM) | 16:35 |
stephenfin | dtroyer: 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, heh | 16:36 |
edleafe | elmiko: Gotta run out a little early. Try to hold the fort while I'm gone. | 16:44 |
dtroyer | stephenfin: ok, thanks. I just wanted to make sure there were no outstanding concerns | 16:44 |
elmiko | edleafe: ack, take care o/ | 16:44 |
stephenfin | dtroyer: 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 argparse | 16:45 |
stephenfin | which is a hole I went far too deep down | 16:45 |
dtroyer | stephenfin: 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 again | 16:47 |
*** ricolin has quit IRC | 16:54 | |
mordred | dtroyer: lgtm! | 16:58 |
dtroyer | mordred: Shit it! Thanks | 16:58 |
dtroyer | ooooooo!!!!!!! | 16:59 |
dtroyer | SHIP | 16:59 |
dtroyer | SHIP | 16:59 |
dtroyer | SHIP | 16:59 |
* dtroyer heads off to remedial typing class | 16:59 | |
*** markvoelker has joined #openstack-sdks | 16:59 | |
mordred | hahahaha | 16:59 |
*** mrhillsman is now known as openlab | 17:02 | |
*** openlab is now known as codebauss | 17:05 | |
*** dtantsur is now known as dtantsur|afk | 17:06 | |
*** holser_ has joined #openstack-sdks | 17:06 | |
*** holser_ has quit IRC | 17:08 | |
*** holser_ has joined #openstack-sdks | 17:09 | |
*** codebauss is now known as openlab | 17:13 | |
*** openlab is now known as codebauss | 17:14 | |
*** codebauss is now known as openlab | 17:15 | |
*** bobh has joined #openstack-sdks | 17:15 | |
*** openlab is now known as codebauss | 17:16 | |
openstackgerrit | Dean Troyer proposed openstack/osc-lib master: Add FakeModule from OSC https://review.opendev.org/660230 | 17:18 |
efried | mordred: lower constraints job is not happy, but I can't fathom why. | 17:18 |
*** yolanda__ has joined #openstack-sdks | 17:20 | |
efried | os-service-types, I'll betcha. | 17:20 |
mordred | efried: WEIRD | 17:21 |
efried | mordred: 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 |
efried | mordred: Is it okay to bump the ost l-c to match the u-c? | 17:22 |
*** bobh has quit IRC | 17:22 | |
mordred | absolutely | 17:22 |
mordred | are we required to have a lower-constraints entry for every library? | 17:22 |
efried | ight, suitable for a ninja approve if I patch that in right quick? | 17:22 |
*** yolanda has quit IRC | 17:23 | |
mordred | yup | 17:23 |
efried | I 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|off | 17:23 | |
mordred | nod. well - makes sense I guess - be sure to bump the requirements lower bound too, obvs | 17:23 |
efried | I know we're keeping placement up to date with latest, forcefully. | 17:23 |
*** codebauss is now known as mrhillsman | 17:24 | |
openstackgerrit | Eric Fried proposed openstack/openstacksdk master: Handle oslo.config exceptions in from_conf https://review.opendev.org/663439 | 17:25 |
openstackgerrit | Eric Fried proposed openstack/openstacksdk master: Minor fixups from from_conf changes https://review.opendev.org/663685 | 17:25 |
efried | mordred: ^, top rebased, will require only +W. Not sure if your reno patch will need rebase+W | 17:26 |
efried | btw, ost 1.7.0 is out, not sure if we should keep sdk in sync | 17:27 |
efried | fwiw 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 |
efried | e.g. https://review.opendev.org/#/c/661132/ | 17:28 |
openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Add release note for new disable service functionality https://review.opendev.org/663690 | 17:30 |
mordred | efried: all three re-+A'd | 17:30 |
mordred | efried: we probably shoudl do the same - I like that as an idea | 17:30 |
mordred | sdk should always have latest ost | 17:30 |
*** markvoelker has quit IRC | 17:32 | |
efried | mordred: okay, I'll propose a canary patch | 17:33 |
openstackgerrit | Merged openstack/python-openstackclient master: Add changes-before attribute to server list https://review.opendev.org/657373 | 17:40 |
openstackgerrit | Eric Fried proposed openstack/openstacksdk master: Pin to latest os-service-types https://review.opendev.org/663733 | 17:47 |
efried | mordred: ^ | 17:47 |
efried | I didn't know where to put it | 17:47 |
efried | I didn't want to make a whole new test module for it | 17:48 |
efried | If you want me to do so, or have a better suggestion for where to put it, lmk. | 17:48 |
*** gtema_ has quit IRC | 17:50 | |
*** holser_ has quit IRC | 17:54 | |
*** ralonsoh has quit IRC | 18:04 | |
mordred | efried: I think that looks great | 18:14 |
*** e0ne has joined #openstack-sdks | 18:26 | |
*** markvoelker has joined #openstack-sdks | 18:29 | |
*** e0ne has quit IRC | 18:31 | |
*** e0ne has joined #openstack-sdks | 18:32 | |
*** e0ne has quit IRC | 18:38 | |
openstackgerrit | Merged openstack/shade master: Cap sphinx for py2 to match global requirements https://review.opendev.org/663545 | 18:44 |
efried | I'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 fingers | 18:44 | |
mordred | efried: hopefully we'll get you an 0.31 soon | 18:48 |
efried | mordred: That'll be neat... but I'm not sure I actually need it. | 18:48 |
efried | oh, never mind | 18:48 |
efried | was gonna say, nova will be well-behaved and only use Connection for things it's supposed to. | 18:49 |
efried | But that doesn't help the case where conf is misconfigured | 18:49 |
efried | though that's a pretty cornery corner. | 18:49 |
efried | Because 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 |
mordred | nod | 18:54 |
*** markvoelker has quit IRC | 19:02 | |
openstackgerrit | Merged openstack/python-openstackclient master: Deprecate openstack server migrate --host option https://review.opendev.org/659382 | 19:05 |
*** e0ne has joined #openstack-sdks | 19:17 | |
*** zbr has quit IRC | 19:25 | |
*** ITD27M01_ has quit IRC | 19:50 | |
*** markvoelker has joined #openstack-sdks | 19:59 | |
*** e0ne has quit IRC | 20:05 | |
*** markvoelker has quit IRC | 20:32 | |
openstackgerrit | Merged openstack/openstacksdk master: Handle oslo.config exceptions in from_conf https://review.opendev.org/663439 | 20:50 |
openstackgerrit | Merged openstack/openstacksdk master: Add release note for new disable service functionality https://review.opendev.org/663690 | 20:50 |
openstackgerrit | Merged openstack/openstacksdk master: Minor fixups from from_conf changes https://review.opendev.org/663685 | 20:50 |
*** holser_ has joined #openstack-sdks | 20:55 | |
openstackgerrit | Eric Fried proposed openstack/openstacksdk master: Pin to latest os-service-types https://review.opendev.org/663733 | 20:57 |
efried | mordred: resolved merge conflict ^ | 20:57 |
mordred | efried: lookie there though - the other patches merged! | 20:58 |
efried | yup! | 20:58 |
efried | could get a new release any time now | 20:58 |
efried | though I'm still waiting for test results on the nova side - we might want to include fixes if those fail | 20:58 |
*** whoami-rajat has quit IRC | 21:27 | |
*** markvoelker has joined #openstack-sdks | 21:29 | |
*** markvoelker has quit IRC | 22:03 | |
*** slaweq has quit IRC | 22:27 | |
mordred | efried: ++ | 22:28 |
*** slaweq has joined #openstack-sdks | 22:55 | |
*** markvoelker has joined #openstack-sdks | 22:59 | |
*** slaweq has quit IRC | 23:08 | |
*** slaweq has joined #openstack-sdks | 23:11 | |
openstackgerrit | Merged openstack/python-openstackclient master: Batch up minor cleanups for release https://review.opendev.org/659982 | 23:19 |
*** holser_ has quit IRC | 23:20 | |
*** slaweq has quit IRC | 23:24 | |
*** markvoelker has quit IRC | 23:31 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!