Posted	Nick	Remark
#openstack-sdks - 2020-01-05
12:14:18	openstackgerrit	Alex Katz proposed openstack/python-openstackclient master: Show correct name for resource with quota set to zero https://review.opendev.org/701110
12:29:25	openstackgerrit	Alex Katz proposed openstack/python-openstackclient master: Show correct name for resource with quota set to zero https://review.opendev.org/701110
#openstack-sdks - 2020-01-06
01:40:39	openstackgerrit	Adrian Turjak proposed openstack/openstacksdk master: Fix bug in object storage container creation setting metadata https://review.opendev.org/699904
01:40:58	openstackgerrit	Adrian Turjak proposed openstack/openstacksdk master: Add support for object storage container policies https://review.opendev.org/699905
09:05:11	openstackgerrit	zhufl proposed openstack/openstacksdk master: Fix duplicated words issue like "was not not found" https://review.opendev.org/701174
10:21:30	openstackgerrit	Riccardo Pittau proposed openstack/openstacksdk master: Add reset_interfaces argument to patch_node https://review.opendev.org/689414
14:59:08	KeithMnemonic	can anyone help me figure out this zuul failure. https://zuul.opendev.org/t/openstack/build/e3e57f71fea4489ab614e6b174ea5c26/log/job-output.txt#1958
15:09:39	efried	KeithMnemonic: I can't get to the results (rackcdn takes forever to populate for me, for some reason) but: is it running under py3 by chance?
15:10:30	efried	oh, nm, got to the results. Looks like a requirements conflict?
15:12:50	KeithMnemonic	yeah, something with osc-lib 1.11.1 vs 1.40.0
15:12:55	KeithMnemonic	yeah, something with osc-lib 1.11.1 vs 1.14.0
15:13:55	KeithMnemonic	I would not have thought that cherry pick wolud have caused that.
15:15:13	efried	no, I agree
15:16:06	efried	possibly worth throwing out a no-op patch to see if it's just stable/rocky being busted.
15:16:32	efried	and... this sounds like a job for promethianfire/#openstack-requirements
15:17:16	efried	The last merge to stable/rocky was end of Sept. Entirely possible things have changed breakingly since then.
15:19:20	KeithMnemonic	thanks efried:
15:20:28	efried	upper constraints for rocky show 1.11.1, and both python-openstackclient and osc-lib show they're using stable/rocky's upper constraints. Neither has a requirements.txt/lower-constraint higher than that.
15:20:41	efried	probably a chained dep from somewhere mysterious.
15:24:12	efried	KeithMnemonic: ah, here's a thing: neither p-osc nor osc-lib show a constraint for the openstackclient project, so you'd be getting the version of that guy from u-c
15:24:32	efried	...but that's not listed in u-c.
15:26:07	efried	but also, openstackclient isn't using u-c, so... that might be how you're getting the wrong osc-lib.
15:26:35	efried	Though I would have expected the test to honor the u-c restriction from python-openstackclient (the project it's actually testing) <scratches head>
15:27:07	efried	anyway, mr. fire ought to be able to synthesize those data points into something useful. Good luck.
16:06:27	openstackgerrit	Merged openstack/openstacksdk master: Add reset_interfaces argument to patch_node https://review.opendev.org/689414
16:40:00	KeithMnemonic	is there anyone who can review this patch https://review.opendev.org/691045 and possibly score CR+2/WF+1 ?
17:14:00	openstackgerrit	Merged openstack/openstacksdk master: Fix duplicated words issue like "was not not found" https://review.opendev.org/701174
18:35:04	efried	KeithMnemonic: I think dtroyer and mordred are the only active cores on python-openstackclient.
21:20:40	mordred	efried: mriedem is also a core
21:20:56	efried	mordred: mriedem is no longer with us
21:21:26	efried	I don't mean he died or anything. But he has left OpenStack.
21:22:09	mordred	efried: oh. darn. that happened already?
21:22:40	efried	mordred: Theoretically his last day was 12/20. Though he's still been sweeping up a few bits and pieces since then.
21:22:48	mordred	efried: BLECH
21:22:56	mordred	efried: we tried to -2 that right?
21:23:25	efried	Yeah, unfortunately mriedem-core only has one member, and it ain't me.
21:26:51	mordred	efried: that is a shame
21:27:39	mordred	efried: amotoki is also osc core - but yeah, that core team is ... small now
21:27:43	efried	The only glimmer of hope is, since he's now working on quantum, he should eventually be able to go back in time and be in both places at once.
21:28:07	efried	which may in fact be why he's still been somewhat active for the past couple of weeks.
21:28:15	mordred	it's me, dtroyer, dhellman, amotoki and mriedem - 2 of whom are non-openstack and one of whom is frequently distracted
21:28:28	efried	yup
21:28:52	efried	I'm responding to Neil's email on the list, on the same topic.
21:30:45	mordred	efried: it's also worth noting that it's been the holdays ...
21:30:58	KeithMnemonic	thanks efried, let me see if amotoki can help
21:31:35	efried	mordred: yeah, I noted that. The month since that fix was posted has effectively been less than a week of work time.
21:31:36	mordred	efried: I just +A'd neil's Edouard's patch - looking at KeithMnemonic's now
21:31:40	mordred	yah
21:31:58	efried	thanks mordred
21:32:02	mordred	I mean - he's not wrong about the overall situation - but in this specific instance I'd say that patch isn't too long in the tooth
21:32:48	efried	I don't know that greasing the squeaky wheel is going to help push the message that we need more corporate investment, but it's good community stewardship I guess :P
21:33:33	mordred	yeah
21:34:00	mordred	I'm tempted to just approve the KeithMnemonic patch - it had a +2 from mriedem already, looks fine, and since that +2 just had a comment updated to fix a nit
21:37:20	efried	seems reasonable to me.
21:39:40	KeithMnemonic	thanks all
22:52:45	openstackgerrit	Sean McGinnis proposed openstack/openstackclient master: Constrain openstackclient-check-plugins run on master https://review.opendev.org/701293
22:54:05	openstackgerrit	Sean McGinnis proposed openstack/openstackclient master: Constrain openstackclient-check-plugins run on master https://review.opendev.org/701293
23:15:06	openstackgerrit	Eric Fried proposed openstack/openstackclient master: Update doc index links (for storyboard etc) https://review.opendev.org/691728
23:53:46	openstackgerrit	Merged openstack/openstackclient master: Update doc index links (for storyboard etc) https://review.opendev.org/691728
#openstack-sdks - 2020-01-07
00:18:45	openstackgerrit	Merged openstack/python-openstackclient master: Fix openstack server list --deleted --marker option https://review.opendev.org/691045
00:57:31	openstackgerrit	Brin Zhang proposed openstack/python-openstackclient master: Add server migrate list CLI https://review.opendev.org/675304
03:16:27	openstackgerrit	wayen proposed openstack/keystoneauth master: Make tests pass in 2020 https://review.opendev.org/701320
13:18:39	mordred	amotoki: are you still around? if so, could you hit https://review.opendev.org/#/c/695427 ?
13:39:04	openstackgerrit	Monty Taylor proposed openstack/openstacksdk master: Fix bug in object storage container creation setting metadata https://review.opendev.org/699904
13:43:15	openstackgerrit	Merged openstack/python-openstackclient master: Fix router create/show if extraroute not supported https://review.opendev.org/697240
13:49:34	openstackgerrit	Monty Taylor proposed openstack/openstacksdk master: Fix bug in object storage container creation setting metadata https://review.opendev.org/699904
13:49:47	mordred	adriant: how's that as a fix for your fix? ^^
13:50:19	mordred	(also, I think that test case was flat wrong)
14:10:21	amotoki	mordred: looking at it now
14:23:09	openstackgerrit	Monty Taylor proposed openstack/python-openstackclient master: Bump tox minversion https://review.opendev.org/701369
14:23:14	mordred	amotoki: awesome, thanks! there's the tox bump
14:24:13	amotoki	mordred: thanks for the follow-up so far :)
14:27:14	KeithMnemonic	looking for some reviews to get this patch merged that solves issues with getting stable python-openstackclient patches to verify. https://review.opendev.org/#/c/701293/
14:57:17	KeithMnemonic	this one as well please https://review.opendev.org/#/c/695645/
15:03:54	openstackgerrit	Merged openstack/keystoneauth master: Switch to Ussuri jobs https://review.opendev.org/691233
15:06:06	openstackgerrit	Merged openstack/osc-lib master: Drop python 2.7 support and testing https://review.opendev.org/700686
15:11:58	openstackgerrit	Eric Fried proposed openstack/openstackclient master: Use python3 for openstackclient-check-plugins job https://review.opendev.org/701378
15:39:57	openstackgerrit	Merged openstack/python-openstackclient master: Stop testing python 2 in tox and zuul. https://review.opendev.org/695427
15:56:43	openstackgerrit	Eric Fried proposed openstack/openstackclient master: Use python3 for openstackclient-check-plugins job https://review.opendev.org/701378
16:32:08	stephenfin	smcginnis: About? I've just looked at that cliff/openstacksdk docs issue
17:31:14	openstackgerrit	Monty Taylor proposed openstack/keystoneauth master: [ussuri][goal] Drop python 2.7 support and testing https://review.opendev.org/699121
19:01:40	openstackgerrit	Eric Fried proposed openstack/openstackclient master: Use python3 for openstackclient-check-plugins job https://review.opendev.org/701378
20:57:35	openstackgerrit	Merged openstack/keystoneauth master: [ussuri][goal] Drop python 2.7 support and testing https://review.opendev.org/699121
21:52:50	openstackgerrit	guang-yee proposed openstack/keystoneauth master: fix bandit complains https://review.opendev.org/701461
22:01:04	adriant	mordred: that looks good to me
22:01:16	mordred	adriant: woot!
22:01:36	adriant	and I see you removed json body from the create entirely, which for containers makes perfect sense
22:02:14	mordred	yeah - since we're already overriding that method anyway - seemed like the better thing to do
22:02:40	adriant	mordred: I guess I shouldn't feel bad, because they need to be fixed, but I keep finding all the swift bugs in the SDK :P
22:02:57	mordred	adriant: it's great! :)
22:03:08	mordred	Shrews: if you have a sec, feel like a review of https://review.opendev.org/#/c/699904/ ?
22:03:25	mordred	Shrews: adriant wrote the first version, then I updated it - so we're in murky core land
22:09:54	adriant	I've +2'd it, but yeah someone else should look and merge as well
22:25:46	openstackgerrit	Monty Taylor proposed openstack/openstacksdk master: Add support for object storage container policies https://review.opendev.org/699905
22:25:56	mordred	rebased that one
22:29:36	Shrews	mordred: done. trusting you that there's nothing in the body to send
22:38:14	mordred	Shrews: yah - there is no body in either direction with swift container interactions
22:38:33	mordred	Shrews: I'm also sure that just by saying the word swift, timburke would appear magically and tell us if we were wrong
22:43:29	timburke	o/
22:43:35	timburke	not wrong, on either count ;-)

Earlier   Later