| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2019-12-19 | |||
| 17:09:32 | Shrews | no worries :) | |
| 17:11:00 | gtema | ok, hacking "project-cleanup" into SDK, so brain is in a quite weird state | |
| 18:20:48 | openstackgerrit | Merged openstack/openstacksdk master: Handle HTTP errors in add/remove router interface calls https://review.opendev.org/687304 | |
| 18:59:05 | gundalow | sshnaidm If you ask in #ansible-devel there should be others that can help. | |
| 18:59:35 | sshnaidm | gundalow, thanks, I think we're good, no rush | |
| 19:02:07 | gundalow | Ok | |
| #openstack-sdks - 2019-12-20 | |||
| 05:15:38 | rm_work | I'd like to add "--loadbalancer" to the `openstack availability zone list` client function ... but ... that'll mean I need to actually add octaviaclient as a dep? MAYBE? I guess I could try to weave it in to detect whether it's present or not... anyway, thoughts on the whole concept? | |
| 05:15:42 | rm_work | mordred maybe? ^^ | |
| 13:25:59 | rm_work | so much part/join spam, lol | |
| 13:26:22 | rm_work | restating: I'd like to add "--loadbalancer" to the `openstack availability zone list` client function ... but ... that'll mean I need to actually add octaviaclient as a dep? MAYBE? I guess I could try to weave it in to detect whether it's present or not... anyway, thoughts on the whole concept? would this be accepted? | |
| 13:26:53 | rm_work | (also, is this the right channel for the client?) | |
| 13:32:38 | frickler | rm_work: this is the right channel, that question I can answer easily | |
| 13:32:54 | rm_work | :D | |
| 13:33:17 | frickler | for the real question, mordred and/or dtroyer may be able to be more helpful | |
| 13:34:29 | frickler | and now that I summoned them, I'm also asking for reviews on https://review.opendev.org/679834 , that feature just merged in neutron and the OSC patch is used in it's docs https://review.opendev.org/679834 | |
| 14:00:06 | mordred | rm_work: how about use sdk for that instead of octaviaclient | |
| 14:00:47 | mordred | there's already an sdk object in osc - you can use it to do things. if the feature you need isn't in sdk - adding it is likely a good idea :) | |
| 14:01:24 | mordred | (we're wanting to migrate osc away from *client completely anyway - so adding new *client deps would be ++sad) | |
| 14:08:11 | rm_work | mordred: ah you know, i think i might already be doing that (had started working on the code) | |
| 14:08:17 | rm_work | yeah i think I am, lol | |
| 14:08:18 | rm_work | whelp | |
| 14:08:32 | rm_work | anyway, question is also about whether folks are cool with adding that to the main call there | |
| 15:00:03 | dtroyer | rm_work: that —loadbalancer option is fine as long as it doesn't add a hard dependency | |
| 15:00:58 | dtroyer | frickler: I think I can make some time for reviews later this morning… it is waaaaay overdue | |
| 15:01:31 | gtema | yey, dtroyer is back at reviewing patches | |
| 15:03:14 | gtema | one of those is https://review.opendev.org/#/c/698874/ - I would reuse this then for network and image | |
| 15:24:30 | rm_work | Cool thanks dtroyer 👍 | |
| 18:31:06 | openstackgerrit | Artem Goncharov proposed openstack/openstacksdk master: WIP Lay a foundation for the project cleanup https://review.opendev.org/700219 | |
| 20:18:32 | openstackgerrit | Artem Goncharov proposed openstack/openstacksdk master: WIP Lay a foundation for the project cleanup https://review.opendev.org/700219 | |
| #openstack-sdks - 2019-12-22 | |||
| 19:59:37 | openstackgerrit | Merged openstack/openstacksdk master: Add return cluster_id when query actions list https://review.opendev.org/699650 | |
| #openstack-sdks - 2019-12-23 | |||
| 09:33:59 | openstackgerrit | Artem Goncharov proposed openstack/openstacksdk master: WIP Lay a foundation for the project cleanup https://review.opendev.org/700219 | |
| 14:34:20 | openstackgerrit | Artem Goncharov proposed openstack/openstacksdk master: WIP Lay a foundation for the project cleanup https://review.opendev.org/700219 | |
| 16:29:31 | openstackgerrit | Artem Goncharov proposed openstack/openstacksdk master: WIP Lay a foundation for the project cleanup https://review.opendev.org/700219 | |
| 20:53:43 | openstackgerrit | Merged openstack/os-service-types master: Bump the openstackdocstheme extension to 1.20 https://review.opendev.org/688250 | |
| 20:53:44 | openstackgerrit | Merged openstack/os-service-types master: Blacklist sphinx 2.1.0 (autodoc bug) https://review.opendev.org/687795 | |
| 21:03:34 | openstackgerrit | Merged openstack/os-service-types master: Update the constraints url https://review.opendev.org/685556 | |
| 21:14:38 | openstackgerrit | Merged openstack/os-service-types master: Updated from OpenStack Service Type Authority https://review.opendev.org/672295 | |
| #openstack-sdks - 2019-12-24 | |||
| 06:46:11 | openstackgerrit | Merged openstack/os-service-types master: Switch to Ussuri jobs https://review.opendev.org/690859 | |
| 10:40:56 | openstackgerrit | Vishakha Agarwal proposed openstack/python-openstackclient master: Adding options to user cli https://review.opendev.org/697444 | |
| 10:46:09 | openstackgerrit | Vishakha Agarwal proposed openstack/python-openstackclient master: Adding options to user cli https://review.opendev.org/697444 | |
| 10:47:59 | openstackgerrit | Vishakha Agarwal proposed openstack/python-openstackclient master: Adding options to user cli https://review.opendev.org/697444 | |
| 14:21:13 | openstackgerrit | Artem Goncharov proposed openstack/openstacksdk master: WIP Lay a foundation for the project cleanup https://review.opendev.org/700219 | |
| #openstack-sdks - 2019-12-27 | |||
| 08:06:15 | openstackgerrit | pengyuesheng proposed openstack/osc-lib master: Drop python 2.7 support and testing https://review.opendev.org/700686 | |
| 13:29:43 | openstackgerrit | Pedro Henrique Pereira Martins proposed openstack/python-openstackclient master: Add documentation about login with federation https://review.opendev.org/693232 | |
| 13:50:04 | openstackgerrit | Pedro Henrique Pereira Martins proposed openstack/keystoneauth master: Add OTP to v3OIDCpassword plugin https://review.opendev.org/697348 | |
| 15:30:04 | openstackgerrit | Pedro Henrique Pereira Martins proposed openstack/keystoneauth master: Add OTP to v3OIDCpassword plugin https://review.opendev.org/697348 | |
| #openstack-sdks - 2020-01-02 | |||
| 13:19:43 | mordred | dtroyer: whence you are back online, https://review.opendev.org/#/c/695427/ https://review.opendev.org/#/c/700686/ are both easy post-new-years patches | |
| 13:21:45 | mordred | also - happy new year! | |
| 13:58:43 | mordred | eandersson: I think we need https://review.opendev.org/700897 Add support for global_request_id for your backport of connection backrefs to work | |
| 13:58:46 | mordred | in stein | |
| 14:00:21 | mordred | I rebased it to stack them together - let's see if it works now | |
| 16:26:14 | mordred | eandersson: nope! I think it's a different problem | |
| 19:16:11 | eandersson | mordred it looks close thou maybe two compounding issues? | |
| 20:06:15 | openstackgerrit | Keith Berger proposed openstack/python-openstackclient master: Fix openstack server list --deleted --marker option https://review.opendev.org/691045 | |
| #openstack-sdks - 2020-01-03 | |||
| 18:28:01 | mordred | eandersson: pushed up a new try | |
| 18:28:14 | eandersson | sweet | |
| 18:28:32 | mordred | it's a smidge of a hack - but if it works, I think it's an acceptable one in this case | |
| 18:38:23 | eandersson | Do you know why we ended up here? | |
| 18:54:39 | KeithMnemonic | dtroyer: can you please help get this patch merged? https://review.opendev.org/#/c/691045/ | |
| 19:14:39 | KeithMnemonic | of course anyone else able to review is greatly appreciated | |
| 19:15:37 | mordred | eandersson: no, I still haven't been able to sort out a root cause | |
| #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. | |