| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2023-09-14 | |||
| 22:13:30 | opendevreview | Julia Kreger proposed openstack/openstacksdk master: [baremetal] Add support for service steps https://review.opendev.org/c/openstack/openstacksdk/+/895224 | |
| 22:13:30 | opendevreview | Julia Kreger proposed openstack/openstacksdk master: [baremetal] Add firmware_interface support https://review.opendev.org/c/openstack/openstacksdk/+/895223 | |
| 22:13:31 | opendevreview | Julia Kreger proposed openstack/openstacksdk master: [baremetal] Add some missing fields to node object https://review.opendev.org/c/openstack/openstacksdk/+/895225 | |
| 22:13:54 | TheJulia | JayF: fyi ^ | |
| 22:19:52 | JayF | O_O | |
| 22:26:18 | JayF | +1, with an actionable nit comment in "Add shard listing test" (looks like you added some noop whitespace to openstack/resource.py) | |
| 22:29:28 | TheJulia | heh, those first two should have been combined | |
| 22:29:30 | TheJulia | c'est la vie | |
| 22:30:25 | opendevreview | Julia Kreger proposed openstack/openstacksdk master: Add shard listing test https://review.opendev.org/c/openstack/openstacksdk/+/895219 | |
| #openstack-sdks - 2023-09-15 | |||
| 00:12:53 | opendevreview | Julia Kreger proposed openstack/openstacksdk master: [baremetal] Ensure baremetal shard parameter gets passed https://review.opendev.org/c/openstack/openstacksdk/+/895220 | |
| 00:13:16 | opendevreview | Julia Kreger proposed openstack/openstacksdk master: [baremetal] Add support for parent_nodes https://review.opendev.org/c/openstack/openstacksdk/+/895221 | |
| 00:13:24 | opendevreview | Julia Kreger proposed openstack/openstacksdk master: [baremetal] Add unhold provision state verb support https://review.opendev.org/c/openstack/openstacksdk/+/895222 | |
| 00:13:35 | opendevreview | Julia Kreger proposed openstack/openstacksdk master: [baremetal] Add firmware_interface support https://review.opendev.org/c/openstack/openstacksdk/+/895223 | |
| 00:13:41 | opendevreview | Julia Kreger proposed openstack/openstacksdk master: [baremetal] Add support for service steps https://review.opendev.org/c/openstack/openstacksdk/+/895224 | |
| 00:13:46 | opendevreview | Julia Kreger proposed openstack/openstacksdk master: [baremetal] Add some missing fields to node object https://review.opendev.org/c/openstack/openstacksdk/+/895225 | |
| 11:54:10 | opendevreview | Baek Seung Ju proposed openstack/openstacksdk master: Implemented heat 'stack suspend' and 'stack resume' function https://review.opendev.org/c/openstack/openstacksdk/+/892156 | |
| 13:03:23 | andrewbogott_ | stephenfin: one of my coworkers just tripped over the issue so I'm once again motivated to get https://review.opendev.org/c/openstack/openstacksdk/+/893283 merged. Can you have a look at the latest version? | |
| 14:14:57 | frickler | gtema: there's a pending comment on https://review.opendev.org/c/openstack/releases/+/893418 regarding the osc release | |
| 15:49:09 | gtema | frickler - discussed directly in #openstack-release | |
| 15:50:24 | frickler | gtema: yes, I saw, but when I pinged you there you were shown offline ... matrix bridge at work once again | |
| 15:52:44 | frickler | gtema: also fyi the prometheus client is getting imported into openstack now https://review.opendev.org/c/openstack/governance/+/894915 | |
| 15:53:04 | gtema | ack | |
| 16:03:08 | opendevreview | OpenStack Release Bot proposed openstack/python-openstackclient stable/2023.2: Update .gitreview for stable/2023.2 https://review.opendev.org/c/openstack/python-openstackclient/+/895487 | |
| 16:03:09 | opendevreview | OpenStack Release Bot proposed openstack/python-openstackclient stable/2023.2: Update TOX_CONSTRAINTS_FILE for stable/2023.2 https://review.opendev.org/c/openstack/python-openstackclient/+/895488 | |
| 16:03:11 | opendevreview | OpenStack Release Bot proposed openstack/python-openstackclient master: Update master for stable/2023.2 https://review.opendev.org/c/openstack/python-openstackclient/+/895489 | |
| #openstack-sdks - 2023-09-16 | |||
| 07:35:12 | opendevreview | anguoming proposed openstack/openstacksdk master: Add client api for enable and disable device https://review.opendev.org/c/openstack/openstacksdk/+/894888 | |
| 07:53:23 | opendevreview | anguoming proposed openstack/openstacksdk master: Add client api for enable and disable device https://review.opendev.org/c/openstack/openstacksdk/+/894888 | |
| 08:11:02 | opendevreview | anguoming proposed openstack/openstacksdk master: Add client api for enable and disable device https://review.opendev.org/c/openstack/openstacksdk/+/894888 | |
| 08:13:44 | opendevreview | anguoming proposed openstack/openstacksdk master: Add client api for enable and disable device https://review.opendev.org/c/openstack/openstacksdk/+/894888 | |
| 12:36:52 | opendevreview | Merged openstack/openstacksdk master: Update master for stable/2023.2 https://review.opendev.org/c/openstack/openstacksdk/+/894151 | |
| 13:55:19 | opendevreview | Merged openstack/openstacksdk master: Treat server as a dict in add_server_interfaces https://review.opendev.org/c/openstack/openstacksdk/+/887906 | |
| 13:58:05 | opendevreview | Merged openstack/openstacksdk master: Support passing a subnetpool for create_subnet https://review.opendev.org/c/openstack/openstacksdk/+/868104 | |
| 13:58:07 | opendevreview | Merged openstack/openstacksdk master: Add volume attachment support https://review.opendev.org/c/openstack/openstacksdk/+/884167 | |
| 13:58:08 | opendevreview | Merged openstack/openstacksdk master: Fix bulk_delete support determination https://review.opendev.org/c/openstack/openstacksdk/+/893071 | |
| 14:05:04 | opendevreview | Merged openstack/openstacksdk master: Fix Swift endpoint conversion to determine info/caps url https://review.opendev.org/c/openstack/openstacksdk/+/893062 | |
| 14:31:59 | opendevreview | Merged openstack/python-openstackclient stable/2023.2: Update .gitreview for stable/2023.2 https://review.opendev.org/c/openstack/python-openstackclient/+/895487 | |
| 14:36:56 | opendevreview | Merged openstack/python-openstackclient stable/2023.2: Update TOX_CONSTRAINTS_FILE for stable/2023.2 https://review.opendev.org/c/openstack/python-openstackclient/+/895488 | |
| 14:36:57 | opendevreview | Merged openstack/python-openstackclient master: Update master for stable/2023.2 https://review.opendev.org/c/openstack/python-openstackclient/+/895489 | |
| #openstack-sdks - 2023-09-17 | |||
| 12:13:05 | opendevreview | BAESUKJU proposed openstack/python-openstackclient master: Add new glance cache command https://review.opendev.org/c/openstack/python-openstackclient/+/893905 | |
| #openstack-sdks - 2023-09-18 | |||
| 00:45:18 | opendevreview | anguoming proposed openstack/openstacksdk master: Add client api for enable and disable device https://review.opendev.org/c/openstack/openstacksdk/+/894888 | |
| 02:56:30 | opendevreview | anguoming proposed openstack/openstacksdk master: Add client api for enable and disable device https://review.opendev.org/c/openstack/openstacksdk/+/894888 | |
| 03:07:14 | opendevreview | anguoming proposed openstack/openstacksdk master: Add client api for enable and disable device https://review.opendev.org/c/openstack/openstacksdk/+/894888 | |
| 07:22:43 | opendevreview | jihyun huh proposed openstack/openstacksdk master: Add support for other metadef property operations https://review.opendev.org/c/openstack/openstacksdk/+/857727 | |
| 09:33:43 | opendevreview | Mridula Joshi proposed openstack/openstacksdk master: Adding support for image upload https://review.opendev.org/c/openstack/openstacksdk/+/881939 | |
| 10:07:23 | opendevreview | Mridula Joshi proposed openstack/openstacksdk master: Adding support for image upload https://review.opendev.org/c/openstack/openstacksdk/+/881939 | |
| 13:07:52 | opendevreview | Rodolfo Alonso proposed openstack/python-openstackclient master: DNM test default role list problem https://review.opendev.org/c/openstack/python-openstackclient/+/895657 | |
| 13:09:57 | ralonsoh | hi folks, what is the bug site for openstackclient? storyboard? launchpad? | |
| 13:09:59 | ralonsoh | stephenfin, ^ | |
| 13:11:33 | frickler | ralonsoh: currently still storyboard but planning to move back to lp real soon now | |
| 13:11:41 | ralonsoh | frickler, thanks! | |
| 13:11:55 | frickler | just waiting to get access for the python-openstacksdk project there | |
| 13:13:38 | frickler | gtema: did you actually contact anyone there? or should we just simply register launchpad.net/openstacksdk and ignore the duplicate as historic coincidence? | |
| 13:14:13 | gtema | frickler - I was not contacting anybody. I would suggest the later - ignore and start from scratch | |
| 14:29:15 | opendevreview | Rodolfo Alonso proposed openstack/python-openstackclient master: Check the default role list before adding a new one https://review.opendev.org/c/openstack/python-openstackclient/+/895657 | |
| 14:29:38 | ralonsoh | folks, please check ^. This is breaking all functional tests right now in OSC (in master) | |
| 14:35:40 | stephenfin | done | |
| 15:18:54 | opendevreview | Rodolfo Alonso proposed openstack/python-openstackclient master: Check the default role list before adding a new one https://review.opendev.org/c/openstack/python-openstackclient/+/895657 | |
| 15:19:02 | ralonsoh | sorry, silly pep8 error in previous PS ^^^ | |
| 15:19:14 | ralonsoh | (but the FT test is passing now) | |
| 15:19:20 | ralonsoh | 2023-09-18 15:09:27.695125 | controller | {0} openstackclient.tests.functional.identity.v3.test_role.RoleTests.test_implied_role_list [11.400274s] ... ok | |
| 15:50:12 | opendevreview | Julia Kreger proposed openstack/openstacksdk master: [baremetal] Add unhold provision state verb support https://review.opendev.org/c/openstack/openstacksdk/+/895222 | |
| 15:50:13 | opendevreview | Julia Kreger proposed openstack/openstacksdk master: [baremetal] Add support for service steps https://review.opendev.org/c/openstack/openstacksdk/+/895224 | |
| 15:50:13 | opendevreview | Julia Kreger proposed openstack/openstacksdk master: [baremetal] Add firmware_interface support https://review.opendev.org/c/openstack/openstacksdk/+/895223 | |
| 15:50:14 | opendevreview | Julia Kreger proposed openstack/openstacksdk master: [baremetal] Add some missing fields to node object https://review.opendev.org/c/openstack/openstacksdk/+/895225 | |
| #openstack-sdks - 2023-09-19 | |||
| 06:34:52 | frickler | gtema: the osc-functional-devstack-tips job has a branch filter "^(?!stable)", which is deprecated syntax. do you foresee future feature branches or can we transform this to a simple "master" match instead? https://opendev.org/openstack/python-openstackclient/src/branch/master/.zuul.yaml#L224-L227 | |
| 06:35:57 | frickler | the other option would be a negated "^stable" match | |
| 06:37:28 | gtema | I would say let it be just master for now. We have rarely branches | |
| 06:37:41 | gtema | And if we need we can then extend this regex here | |
| 06:38:20 | frickler | ack, patch incoming, thx | |
| 06:38:31 | gtema | Thank you | |
| 06:42:01 | frickler | one day openstack really should start renaming the default branch to main everywhere. that'll be a fun project | |
| 06:42:26 | opendevreview | Dr. Jens Harbott proposed openstack/python-openstackclient master: Update branch regex for osc-functional-devstack-tips https://review.opendev.org/c/openstack/python-openstackclient/+/895765 | |
| 06:43:09 | frickler | I'll make stable branch patches that completely drop the -tips job there, no need to keep the job definition if it isn't run | |
| 06:49:34 | gtema | Right, makes sense as well | |
| 07:00:43 | opendevreview | Dr. Jens Harbott proposed openstack/python-openstackclient stable/2023.2: [stable-only] CI: Drop all tips jobs https://review.opendev.org/c/openstack/python-openstackclient/+/895767 | |
| 09:46:36 | ralonsoh | stephenfin, hi! if you have 1 minute: https://review.opendev.org/c/openstack/python-openstackclient/+/895657 | |
| 09:46:37 | ralonsoh | thanks! | |
| 09:53:02 | stephenfin | done | |
| 10:59:49 | opendevreview | Merged openstack/python-openstackclient master: Check the default role list before adding a new one https://review.opendev.org/c/openstack/python-openstackclient/+/895657 | |
| 13:11:31 | opendevreview | Merged openstack/python-openstackclient master: tests: Explicitly specify port fields for output https://review.opendev.org/c/openstack/python-openstackclient/+/894510 | |
| 13:11:34 | opendevreview | Merged openstack/python-openstackclient master: volume: Deprecate '--detailed' options https://review.opendev.org/c/openstack/python-openstackclient/+/894527 | |
| 13:14:07 | opendevreview | Merged openstack/python-openstackclient master: Add a warning for resizing servers booted from volumes https://review.opendev.org/c/openstack/python-openstackclient/+/890184 | |
| 13:14:12 | opendevreview | Merged openstack/python-openstackclient master: Remove use of oslo.utils https://review.opendev.org/c/openstack/python-openstackclient/+/894528 | |
| 13:59:38 | priteau | Hi stephenfin. A colleague of mine has just discovered that openstacksdk 1.5.0 is broken on Python 3.6 because of the use of ty.Literal. Not a problem as such, but setup.cfg hasn't been updated so it still has: python_requires = >=3.6 | |
| 14:00:54 | priteau | And if it is changed in the next release, it will stay broken unless we use upper constraints because pip3.6 will pick 1.5.0 as latest compatible release. | |
| 14:01:20 | stephenfin | damn, okay, i guess we need to drop that, release, and re-add it (or yank 1.5.0) | |
| 14:01:31 | stephenfin | and then bump python_requires | |
| 14:01:34 | priteau | I see there is a typing-extensions library that brings compatiblity with earlier Python | |
| 14:02:10 | priteau | If you would prefer to go that route when re-releasing | |
| 14:04:58 | priteau | However changelog suggests that we might need typing-extensions 4.6.0, that's not even in master u-c yet | |
| 15:04:41 | andrewbogott_ | stephenfin: here's another nudge for https://review.opendev.org/c/openstack/openstacksdk/+/893283 :) | |
| 18:48:32 | JayF | I wanted to ensure the SDK team was aware of this: https://review.opendev.org/c/openstack/governance/+/894915/2#message-575de1c1a31190684835fbbf671104c416ec6d6a -- please have a look | |
| 18:49:16 | JayF | Basically a prometheus client plugin for OSC is asking to be included in the telemetry project. I wanted to make sure you all were aware it was happening (and I'm generally curious if it'd be the first OSC plugin to talk to not-openstack) | |
| 18:51:08 | frickler | JayF: thanks for the ping, I mentioned this to gtema already earlier before they wanted to become an openstack project. I myself consider this also to be a bit weird but not enough to really object to it | |
| 18:51:30 | JayF | I have zero objections; just trying to make sure communication lines are flowing :D | |
| 18:51:48 | JayF | TBH I find it kinda cool, as PTL of a project that crosses CNCF/OpenStack lines in many places already and emits metrics to prometheus :D | |
| 18:52:25 | frickler | I think there's gnocchiclient, which would be similar since gnocchi moved out of openstack governance | |
| 18:52:46 | JayF | you mind putting those sentiments on the gerrit change, so they remain beyond this conversation? | |
| 18:52:53 | JayF | including the info about gnocchi | |
| 18:54:46 | frickler | I can do that, yes | |