Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2019-06-06
16:06:30 elmiko =)
16:08:49 edleafe We'll try to handle the load ourselves :)
16:12:20 elmiko hehe
16:35:19 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:52 dtroyer mordred: ^^^ if you feel like looking at Something Completely Different(TM)
16:36:51 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:44:01 edleafe elmiko: Gotta run out a little early. Try to hold the fort while I'm gone.
16:44:21 dtroyer stephenfin: ok, thanks. I just wanted to make sure there were no outstanding concerns
16:44:31 elmiko edleafe: ack, take care o/
16:45: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:57 stephenfin which is a hole I went far too deep down
16:47:16 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:58:35 mordred dtroyer: lgtm!
16:58:55 dtroyer mordred: Shit it! Thanks
16:59:01 dtroyer ooooooo!!!!!!!
16:59:03 dtroyer SHIP
16:59:04 dtroyer SHIP
16:59:05 dtroyer SHIP
16:59:45 mordred hahahaha
17:18:09 openstackgerrit Dean Troyer proposed openstack/osc-lib master: Add FakeModule from OSC https://review.opendev.org/660230
17:18:36 efried mordred: lower constraints job is not happy, but I can't fathom why.
17:20:52 efried os-service-types, I'll betcha.
17:21:33 mordred efried: WEIRD
17:22:12 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:28 efried mordred: Is it okay to bump the ost l-c to match the u-c?
17:22:40 mordred absolutely
17:22:57 mordred are we required to have a lower-constraints entry for every library?
17:22:58 efried ight, suitable for a ninja approve if I patch that in right quick?
17:23:03 mordred yup
17:23:17 efried I think you're required to have l-c for anything listed in requirements or test-requirements, yeah.
17:23:43 mordred nod. well - makes sense I guess - be sure to bump the requirements lower bound too, obvs
17:23:44 efried I know we're keeping placement up to date with latest, forcefully.
17:25:24 openstackgerrit Eric Fried proposed openstack/openstacksdk master: Minor fixups from from_conf changes https://review.opendev.org/663685
17:25:24 openstackgerrit Eric Fried proposed openstack/openstacksdk master: Handle oslo.config exceptions in from_conf https://review.opendev.org/663439
17:26:40 efried mordred: ^, top rebased, will require only +W. Not sure if your reno patch will need rebase+W
17:27:28 efried btw, ost 1.7.0 is out, not sure if we should keep sdk in sync
17:28:06 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:29 efried e.g. https://review.opendev.org/#/c/661132/
17:30:07 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Add release note for new disable service functionality https://review.opendev.org/663690
17:30:15 mordred efried: all three re-+A'd
17:30:36 mordred efried: we probably shoudl do the same - I like that as an idea
17:30:44 mordred sdk should always have latest ost
17:33:41 efried mordred: okay, I'll propose a canary patch
17:40:49 openstackgerrit Merged openstack/python-openstackclient master: Add changes-before attribute to server list https://review.opendev.org/657373
17:47:44 openstackgerrit Eric Fried proposed openstack/openstacksdk master: Pin to latest os-service-types https://review.opendev.org/663733
17:47:49 efried mordred: ^
17:47:56 efried I didn't know where to put it
17:48:04 efried I didn't want to make a whole new test module for it
17:48:16 efried If you want me to do so, or have a better suggestion for where to put it, lmk.
18:14:38 mordred efried: I think that looks great
18:44:22 openstackgerrit Merged openstack/shade master: Cap sphinx for py2 to match global requirements https://review.opendev.org/663545
18:44:34 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:48:14 mordred efried: hopefully we'll get you an 0.31 soon
18:48:31 efried mordred: That'll be neat... but I'm not sure I actually need it.
18:48:50 efried oh, never mind
18:49:03 efried was gonna say, nova will be well-behaved and only use Connection for things it's supposed to.
18:49:10 efried But that doesn't help the case where conf is misconfigured
18:49:25 efried though that's a pretty cornery corner.
18:50:05 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:54:43 mordred nod
19:05:46 openstackgerrit Merged openstack/python-openstackclient master: Deprecate openstack server migrate --host option https://review.opendev.org/659382
20:50:20 openstackgerrit Merged openstack/openstacksdk master: Handle oslo.config exceptions in from_conf https://review.opendev.org/663439
20:50:22 openstackgerrit Merged openstack/openstacksdk master: Add release note for new disable service functionality https://review.opendev.org/663690
20:50:23 openstackgerrit Merged openstack/openstacksdk master: Minor fixups from from_conf changes https://review.opendev.org/663685
20:57:34 openstackgerrit Eric Fried proposed openstack/openstacksdk master: Pin to latest os-service-types https://review.opendev.org/663733
20:57:43 efried mordred: resolved merge conflict ^
20:58:22 mordred efried: lookie there though - the other patches merged!
20:58:27 efried yup!
20:58:38 efried could get a new release any time now
20:58:58 efried though I'm still waiting for test results on the nova side - we might want to include fixes if those fail
22:28:07 mordred efried: ++
23:19:13 openstackgerrit Merged openstack/python-openstackclient master: Batch up minor cleanups for release https://review.opendev.org/659982
#openstack-sdks - 2019-06-07
00:02:52 openstackgerrit Merged openstack/openstacksdk master: Pin to latest os-service-types https://review.opendev.org/663733
03:49:29 shubham_potale Hi all
03:49:32 shubham_potale I have one question regarding the url formation for the APIs like tacker, neutron apis for example openstack --debug network list the url for the above command is http://10.232.48.200:9696/v2.0/networks in above url where that v2.0 is appended is not clear to me. If we see openstack endpoint list we get network base url as http://10.232.48.200:9696/ for this part (/v2.0/networks) i found that /networks is app
03:50:41 shubham_potale if anyone knows about it, please let me know
04:40:28 efried mordred: Nova side looks good (except I needed to bump the ost l-c there too, which seems weird) so for my money we can cut a release any time. LMK if you want me to do the paperwork.
06:44:50 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Add access alias (aka) for the resource attributes https://review.opendev.org/663846
06:48:42 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Use Resource layer for next compute methods https://review.opendev.org/663064
06:50:16 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Use resource layer for compute flavors https://review.opendev.org/650903
09:27:53 openstackgerrit Dmitry Tantsur proposed openstack/openstacksdk master: baremetal: raise more specific ResourceFailure in wait_for_* methods https://review.opendev.org/663867
11:03:41 dtantsur folks, a trivial fix ^^^ when you have some time
11:10:07 mordred gtema: ^^ have a sec to look at that patch from dtantsur ? also - nice aka patch
11:10:41 gtema yeah - it found actual problem in the image checksum validation - hehe
11:13:17 mordred \o/
11:37:00 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Add access alias (aka) for the resource attributes https://review.opendev.org/663846
11:55:32 mordred gtema: if that passes tests, that looks pretty nice!
11:55:53 gtema thanks
11:56:13 gtema some time later probably rename "alias" to "alternate_attribute"
11:57:44 gtema it actually proved, that in the case of image the checksum validation would have never worked really
11:58:07 gtema since after fetch the checksum was in reality empty
11:59:44 gtema what's happening - already second change fails due to timeout in nodepool job
12:18:19 mordred uhoh
12:35:05 openstackgerrit Yves-Gwenael Bourhis proposed openstack/shade master: URL encode swift objects endpoints https://review.opendev.org/663922
12:39:34 openstackgerrit Yves-Gwenael Bourhis proposed openstack/openstacksdk master: URL encode swift objects endpoints https://review.opendev.org/663925
12:56:11 openstackgerrit Merged openstack/openstacksdk master: baremetal: raise more specific ResourceFailure in wait_for_* methods https://review.opendev.org/663867
13:14:30 efried mordred: We green for a release of sdk?
13:16:14 mordred efried: yup!
13:16:21 efried mordred: I'm proposing, stand by.
13:17:22 mordred remote: https://review.opendev.org/663939 Release 0.31.0 of openstacksdk

Earlier   Later