| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2024-11-15 | |||
| 00:24:08 | opendevreview | Monty Taylor proposed openstack/openstacksdk master: Replace netifaces with netifaces-2 https://review.opendev.org/c/openstack/openstacksdk/+/935222 | |
| 09:59:50 | frickler | gtema: stephenfin: please check https://review.opendev.org/c/openstack/openstacksdk/+/934141 , while I haven't been able to reproduce it with my test patch so far, I do keep seeing gate failures matching the pattern | |
| 10:01:25 | gtema | ack, perhaps we can try to have only unittest for that case | |
| 12:29:32 | opendevreview | Merged openstack/openstacksdk master: Skip disabling compute service https://review.opendev.org/c/openstack/openstacksdk/+/934141 | |
| 15:38:35 | stephenfin | gtema: antoniagaete: M0weng[m]: I won't make it to the meeting today again, I'm afraid. Feel free to direct any questions for me this way though | |
| 15:43:28 | gtema | ack stephenfin | |
| 16:45:04 | opendevreview | Merged openstack/openstacksdk master: Fix swift metadata setting documentation https://review.opendev.org/c/openstack/openstacksdk/+/935229 | |
| 17:53:02 | opendevreview | Merged openstack/keystoneauth master: typing: Annotate keystoneauth1.access.service_providers https://review.opendev.org/c/openstack/keystoneauth/+/930771 | |
| 21:21:19 | opendevreview | Merged openstack/keystoneauth master: access: Add more helper attributes to AccessInfoV3 https://review.opendev.org/c/openstack/keystoneauth/+/930772 | |
| 21:24:19 | opendevreview | Merged openstack/keystoneauth master: access: Remove use of _missingproperty wrapper https://review.opendev.org/c/openstack/keystoneauth/+/930773 | |
| 21:38:01 | opendevreview | Merged openstack/keystoneauth master: typing: Annotate keystoneauth1.access.access https://review.opendev.org/c/openstack/keystoneauth/+/930774 | |
| #openstack-sdks - 2024-11-16 | |||
| 04:19:14 | opendevreview | Ghanshyam proposed openstack/openstacksdk master: Update gate jobs as per the 2025.1 cycle testing runtime https://review.opendev.org/c/openstack/openstacksdk/+/935415 | |
| 04:19:16 | opendevreview | Ghanshyam proposed openstack/python-openstackclient master: Update gate jobs as per the 2025.1 cycle testing runtime https://review.opendev.org/c/openstack/python-openstackclient/+/935416 | |
| 13:52:17 | opendevreview | Merged openstack/keystoneauth master: typing: Annotate keystoneauth1.identity.base https://review.opendev.org/c/openstack/keystoneauth/+/932982 | |
| 14:09:52 | opendevreview | Merged openstack/keystoneauth master: identity: Remove support for manual OIDC grant type https://review.opendev.org/c/openstack/keystoneauth/+/932983 | |
| 14:16:00 | opendevreview | Merged openstack/keystoneauth master: identity: Remove use of kwargs https://review.opendev.org/c/openstack/keystoneauth/+/932984 | |
| 15:03:36 | opendevreview | Merged openstack/keystoneauth master: typing: Annotate keystoneauth1.identity.access https://review.opendev.org/c/openstack/keystoneauth/+/932985 | |
| 15:03:38 | opendevreview | Merged openstack/keystoneauth master: typing: Annotate keystoneauth1.identity.v2 https://review.opendev.org/c/openstack/keystoneauth/+/932986 | |
| 15:03:40 | opendevreview | Merged openstack/keystoneauth master: identity: Remove kwarg-generation helpers https://review.opendev.org/c/openstack/keystoneauth/+/932987 | |
| #openstack-sdks - 2024-11-18 | |||
| 04:00:20 | opendevreview | Takashi Kajinami proposed openstack/openstacksdk master: Remove reference to removed nova services https://review.opendev.org/c/openstack/openstacksdk/+/935477 | |
| 09:26:20 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: make sort_key and sort_dir in neutron array https://review.opendev.org/c/openstack/codegenerator/+/935489 | |
| 11:40:46 | opendevreview | Josephine Seifert proposed openstack/python-openstackclient master: WIP: Add admin-editable metadata field to volume types https://review.opendev.org/c/openstack/python-openstackclient/+/928132 | |
| 13:42:43 | opendevreview | Merged openstack/openstacksdk master: Remove reference to removed nova services https://review.opendev.org/c/openstack/openstacksdk/+/935477 | |
| 15:47:12 | opendevreview | Merged openstack/keystoneauth master: typing: Annotate keystoneauth1.identity.generic https://review.opendev.org/c/openstack/keystoneauth/+/932988 | |
| 15:49:57 | opendevreview | Merged openstack/keystoneauth master: typing: Annotate keystoneauth1.identity.v3 https://review.opendev.org/c/openstack/keystoneauth/+/932989 | |
| 15:49:58 | opendevreview | Merged openstack/keystoneauth master: typing: Annotate keystoneauth1.extras (1/2) https://review.opendev.org/c/openstack/keystoneauth/+/932990 | |
| 15:49:59 | opendevreview | Merged openstack/keystoneauth master: typing: Annotate keystoneauth1.loading.base https://review.opendev.org/c/openstack/keystoneauth/+/932991 | |
| 15:50:01 | opendevreview | Merged openstack/keystoneauth master: typing: Annotate keystoneauth1.loading.opts https://review.opendev.org/c/openstack/keystoneauth/+/932992 | |
| 16:09:15 | opendevreview | Merged openstack/openstacksdk master: Fix missing 'f' prefix in exception message https://review.opendev.org/c/openstack/openstacksdk/+/933375 | |
| 16:54:48 | opendevreview | Merged openstack/codegenerator master: make sort_key and sort_dir in neutron array https://review.opendev.org/c/openstack/codegenerator/+/935489 | |
| 19:44:42 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Use Array for serialized param instead of Set https://review.opendev.org/c/openstack/codegenerator/+/935559 | |
| 20:54:21 | opendevreview | Merged openstack/keystoneauth master: typing: Annotate keystoneauth1.loading.cli https://review.opendev.org/c/openstack/keystoneauth/+/932993 | |
| 20:54:22 | opendevreview | Merged openstack/keystoneauth master: typing: Annotate keystoneauth1.loading.conf https://review.opendev.org/c/openstack/keystoneauth/+/932994 | |
| 20:54:24 | opendevreview | Merged openstack/keystoneauth master: typing: Annotate keystoneauth1.loading.adapter https://review.opendev.org/c/openstack/keystoneauth/+/932995 | |
| 20:54:25 | opendevreview | Merged openstack/keystoneauth master: typing: Annotate keystoneauth1.loading.session https://review.opendev.org/c/openstack/keystoneauth/+/932996 | |
| 20:54:27 | opendevreview | Merged openstack/keystoneauth master: typing: Annotate keystoneauth1.loading.identity https://review.opendev.org/c/openstack/keystoneauth/+/932997 | |
| 20:54:28 | opendevreview | Merged openstack/keystoneauth master: typing: Annotate keystoneauth1.loading._utils https://review.opendev.org/c/openstack/keystoneauth/+/932998 | |
| 20:54:30 | opendevreview | Merged openstack/keystoneauth master: typing: Annotate keystoneauth1.loading._plugins https://review.opendev.org/c/openstack/keystoneauth/+/932999 | |
| #openstack-sdks - 2024-11-19 | |||
| 08:52:46 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Use Array for serialized param instead of Set https://review.opendev.org/c/openstack/codegenerator/+/935559 | |
| 11:07:42 | opendevreview | Merged openstack/codegenerator master: Use Array for serialized param instead of Set https://review.opendev.org/c/openstack/codegenerator/+/935559 | |
| 12:50:43 | frickler | gtema: please have a look at these when you have a moment https://review.opendev.org/c/openstack/releases/+/934606 https://review.opendev.org/c/openstack/releases/+/934597 | |
| 12:51:11 | gtema | ack frickler, thanks for reminder | |
| 13:32:42 | opendevreview | Merged openstack/openstacksdk stable/2024.2: compute: Add locked filter for Server https://review.opendev.org/c/openstack/openstacksdk/+/932401 | |
| 13:32:44 | opendevreview | Merged openstack/openstacksdk stable/2024.1: compute: Add locked filter for Server https://review.opendev.org/c/openstack/openstacksdk/+/932402 | |
| 13:33:40 | opendevreview | Merged openstack/keystoneauth master: typing: Annotate keystoneauth1.extras (2/2) https://review.opendev.org/c/openstack/keystoneauth/+/933000 | |
| 13:39:02 | opendevreview | Merged openstack/openstacksdk stable/2023.2: compute: Add locked filter for Server https://review.opendev.org/c/openstack/openstacksdk/+/932403 | |
| 17:05:15 | M0weng[m] | stephenfin: test ping | |
| 17:05:31 | stephenfin | M0weng[m]: Now you appear. Matrix weirdness | |
| 17:42:23 | opendevreview | Merged openstack/keystoneauth master: typing: Invert module settings https://review.opendev.org/c/openstack/keystoneauth/+/933001 | |
| 17:42:24 | opendevreview | Merged openstack/keystoneauth master: typing: Set remaining 'disallow_*' opts to 'true' https://review.opendev.org/c/openstack/keystoneauth/+/933002 | |
| 17:57:10 | opendevreview | Merged openstack/keystoneauth master: typing: Move configuration to pyproject.toml https://review.opendev.org/c/openstack/keystoneauth/+/933003 | |
| 17:57:11 | opendevreview | Merged openstack/keystoneauth master: Add py.typed file https://review.opendev.org/c/openstack/keystoneauth/+/933004 | |
| 17:57:14 | opendevreview | Merged openstack/keystoneauth master: typing: Make loaders broadly generic https://review.opendev.org/c/openstack/keystoneauth/+/933005 | |
| 18:21:43 | opendevreview | Antonia Gaete proposed openstack/python-openstackclient master: identity: Migrate 'trust' commands to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/931694 | |
| #openstack-sdks - 2024-11-20 | |||
| 11:34:21 | opendevreview | Stephen Finucane proposed openstack/keystoneauth master: typing: Simplify some types, other TODOs https://review.opendev.org/c/openstack/keystoneauth/+/935764 | |
| 11:34:22 | opendevreview | Stephen Finucane proposed openstack/keystoneauth master: typing: Remove unused AuthMethod.get_auth_data arguments https://review.opendev.org/c/openstack/keystoneauth/+/935765 | |
| 11:34:23 | opendevreview | Stephen Finucane proposed openstack/keystoneauth master: typing: Remove unused BaseAuthPlugin.get_auth_ref kwargs https://review.opendev.org/c/openstack/keystoneauth/+/935766 | |
| 11:34:24 | opendevreview | Stephen Finucane proposed openstack/keystoneauth master: typing: Remove unused _Rescoped.get_unscoped_auth_ref kwargs https://review.opendev.org/c/openstack/keystoneauth/+/935767 | |
| 11:34:26 | opendevreview | Stephen Finucane proposed openstack/keystoneauth master: typing: Remove unused BaseAuthPlugin.get_headers kwargs https://review.opendev.org/c/openstack/keystoneauth/+/935768 | |
| 11:34:28 | opendevreview | Stephen Finucane proposed openstack/keystoneauth master: typing: Remove unused BaseAuthPlugin.get_token kwargs https://review.opendev.org/c/openstack/keystoneauth/+/935769 | |
| 11:34:30 | opendevreview | Stephen Finucane proposed openstack/keystoneauth master: WIP: typing: Deprecate use of AuthConstructor https://review.opendev.org/c/openstack/keystoneauth/+/935770 | |
| 11:34:32 | opendevreview | Stephen Finucane proposed openstack/keystoneauth master: WIP: typing: Removing remaining used kwargs https://review.opendev.org/c/openstack/keystoneauth/+/935771 | |
| 11:38:31 | opendevreview | Stephen Finucane proposed openstack/keystoneauth master: loading.opts: Add 'argparse_envvars' helper https://review.opendev.org/c/openstack/keystoneauth/+/933110 | |
| 11:38:32 | opendevreview | Stephen Finucane proposed openstack/keystoneauth master: doc: Fix indentation https://review.opendev.org/c/openstack/keystoneauth/+/933111 | |
| 11:38:33 | opendevreview | Stephen Finucane proposed openstack/keystoneauth master: docs: Rework plugin-options doc https://review.opendev.org/c/openstack/keystoneauth/+/933112 | |
| 11:38:34 | opendevreview | Stephen Finucane proposed openstack/keystoneauth master: loading: Add blurb for most plugins https://review.opendev.org/c/openstack/keystoneauth/+/933113 | |
| 11:38:36 | opendevreview | Stephen Finucane proposed openstack/keystoneauth master: docs: Document CLI options, envvars https://review.opendev.org/c/openstack/keystoneauth/+/933114 | |
| 12:18:18 | opendevreview | Stephen Finucane proposed openstack/keystoneauth master: Fix misuse of assertTrue https://review.opendev.org/c/openstack/keystoneauth/+/935777 | |
| 14:29:39 | tobias-urdin | stephenfin: can u have a look at https://review.opendev.org/c/openstack/python-openstackclient/+/932618 which for cinder v2 (my fail) is already approved | |
| 14:36:30 | opendevreview | Takashi Kajinami proposed openstack/openstacksdk master: Replace netifaces library https://review.opendev.org/c/openstack/openstacksdk/+/931588 | |
| 15:11:05 | stephenfin | tobias-urdin: +W | |
| 16:38:58 | opendevreview | Merged openstack/python-openstackclient master: Show Created At column for volume backups in v3 https://review.opendev.org/c/openstack/python-openstackclient/+/932618 | |
| 16:40:22 | opendevreview | Takashi Kajinami proposed openstack/openstacksdk master: Replace netifaces library https://review.opendev.org/c/openstack/openstacksdk/+/931588 | |
| 19:51:50 | adamcarthur5 | Hey stephenfin, feel free to close https://review.opendev.org/c/openstack/ironic/+/935697/1, Ironic is pretty good for quick review-turnover so I've just merged this change directly into the prev commit | |
| 19:59:39 | tobias-urdin | stephenfin: thx! | |
| 20:14:14 | opendevreview | Eric Harney proposed openstack/python-openstackclient master: WIP: Fix volume snapshot create on cinder mv >= 3.66 https://review.opendev.org/c/openstack/python-openstackclient/+/935829 | |
| 22:58:00 | opendevreview | Antonia Gaete proposed openstack/python-openstackclient master: identity: Migrate 'endpoint' commands to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/934817 | |
| 22:59:16 | opendevreview | Antonia Gaete proposed openstack/python-openstackclient master: identity: Migrate 'endpoint' commands to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/934817 | |
| 23:20:00 | opendevreview | Antonia Gaete proposed openstack/python-openstackclient master: identity: Migrate 'domain' commands to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/923785 | |
| 23:26:39 | opendevreview | Antonia Gaete proposed openstack/python-openstackclient master: identity: Migrate `region` commands to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/930937 | |
| 23:28:55 | opendevreview | Antonia Gaete proposed openstack/python-openstackclient master: identity: Migrate 'domain' commands to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/923785 | |
| 23:38:45 | opendevreview | Antonia Gaete proposed openstack/python-openstackclient master: identity: Migrate 'service provider' commands to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/929961 | |
| #openstack-sdks - 2024-11-21 | |||
| 01:12:12 | opendevreview | Takashi Kajinami proposed openstack/openstacksdk master: Replace netifaces library https://review.opendev.org/c/openstack/openstacksdk/+/931588 | |
| 12:55:42 | stephenfin | gtema: Can you sanity check something for me? Why is this input https://paste.openstack.org/show/bY1G6R4A0pO0eZzFzmfB/ invalid against this schema? https://paste.openstack.org/show/bBWN4ljq3WoE7gTkSuS2/ | |
| 12:56:15 | gtema | gimme a sec to look | |
| 13:01:18 | stephenfin | gtema: got it: turns out additionalProperties and oneOf do not play nicely together. That's a fairly nasty gotcha https://github.com/python-jsonschema/jsonschema/issues/193 | |
| 13:01:36 | gtema | yeah, I slowly came to the same conclusion | |
| 13:02:17 | gtema | if you put whole impls into the oneOf kinds and drop everything else on the base it goes fine | |
| 13:02:42 | stephenfin | yup, that's what I've ended up doing. Bit of duplication but at least it works | |
| 15:17:02 | frickler | gtema: stephenfin: I've seen two gate failures today in reqs sdk job, both times the two tests after openstack.tests.functional.compute.v2.test_service.TestService.test_update were failing, even though the latter is now skipped, so the skip doesn't seem to have helped much :( | |
| 15:17:29 | gtema | :( | |
| 15:17:34 | frickler | maybe some resource issue still, not sure if one of you can look into it or if I should rather try to check with nova folks | |
| 15:17:48 | frickler | https://zuul.opendev.org/t/openstack/build/2862593e22bd41718da98e504fe67c6d is one of them | |
| 15:18:33 | gtema | today I am completely booked for other stuff | |
| 17:51:37 | opendevreview | Stephen Finucane proposed openstack/keystoneauth master: Allow passing None for username in v2.Password (redux) https://review.opendev.org/c/openstack/keystoneauth/+/935897 | |
| 18:34:54 | opendevreview | OpenStack Release Bot proposed openstack/keystoneauth master: reno: Update master for unmaintained/2023.1 https://review.opendev.org/c/openstack/keystoneauth/+/935961 | |
| #openstack-sdks - 2024-11-22 | |||
| 11:56:59 | opendevreview | Dr. Jens Harbott proposed openstack/openstacksdk master: Skip disabling compute service https://review.opendev.org/c/openstack/openstacksdk/+/936016 | |
| 11:58:51 | frickler | gtema: ^^ seems there is another slightly destructive test ... :-( | |
| 12:13:36 | gtema | Damn | |
| 12:20:07 | frickler | well I'll be glad if it really turns out to be this and not some weird, deep nova issue | |