| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2020-01-06 | |||
| 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 ;-) | |
| 22:43:54 | timburke | well, unless you want to talk bulk operations... | |
| 22:51:00 | mordred | timburke: let's just assume for now that I don't :) | |
| 23:45:00 | openstackgerrit | Merged openstack/openstacksdk master: Fix bug in object storage container creation setting metadata https://review.opendev.org/699904 | |
| #openstack-sdks - 2020-01-08 | |||
| 00:01:35 | adriant | lol, timburke, mordred, that's probably one of my next SDK patches :P | |
| 00:01:54 | adriant | Once I can convince our ops to actually enable bulk operations | |
| 00:54:06 | timburke | adriant, out of curiosity, what's driving their hesitation? | |
| 01:10:00 | adriant | timburke: at first just worries about performance, but we'll be watching that, now it's just issues with our deployment tooling | |
| 01:43:41 | openstackgerrit | Merged openstack/keystoneauth master: fix bandit complains https://review.opendev.org/701461 | |