| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2022-01-28 | |||
| 11:59:08 | frickler | I tested 826873 and it seems to work like I intended, care to fast-approve? | |
| 11:59:58 | gtema | done, hopefully tests on zuul are also happy | |
| 12:04:35 | frickler | but of course I'll also check for genuine failures | |
| 12:05:03 | gtema | :) | |
| 12:10:49 | ralonsoh | stephenfin, hi. Is this enough? --> https://review.opendev.org/c/openstack/python-openstackclient/+/806016/7/openstackclient/tests/functional/common/test_quota.py | |
| 12:14:19 | stephenfin | It sure does :) I missed that. Apologies! | |
| 12:22:27 | ralonsoh | stephenfin, thanks a lot | |
| 12:25:06 | stephenfin | np, sorry for the delay | |
| 12:31:46 | isabek | gtema, stephenfin: Hi! Can you please take a look [1] and [2]? I have fixed all comments. 1) https://review.opendev.org/c/openstack/openstacksdk/+/822305 | |
| 12:32:02 | isabek | 2) https://review.opendev.org/c/openstack/python-openstackclient/+/805618 | |
| 12:32:10 | isabek | Thanks in advance! | |
| 12:54:08 | stephenfin | dtantsur: Likely dumb question, but I assume ironic w/o nova still needs glance? | |
| 12:55:24 | dtantsur | stephenfin: ironic can work in any combination: with nova, without nova with glance, with neither nova nor glance, etc. | |
| 12:56:52 | stephenfin | dtantsur: Ack. Will it be present in the openstacksdk-functional-devstack-ironic job? | |
| 12:57:11 | dtantsur | stephenfin: I'm looking at the definition, and it seems like we don't disable glance there | |
| 12:57:29 | dtantsur | ideally, we should not rely on knowing the job definitions: people may run these tests locally | |
| 12:59:32 | stephenfin | fair point | |
| 12:59:32 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: tests: Centralize configuration of default flavor, image https://review.opendev.org/c/openstack/openstacksdk/+/822814 | |
| 12:59:40 | stephenfin | dtantsur: ^ hopefully that will do the trick | |
| 12:59:58 | dtantsur | thx, let's see what the CI says | |
| 13:00:02 | stephenfin | (y) | |
| 13:07:08 | frickler | focal is not supported for ussuri, great | |
| 13:13:53 | opendevreview | Dr. Jens Harbott proposed openstack/openstacksdk master: WIP: Test new legacy job https://review.opendev.org/c/openstack/openstacksdk/+/826877 | |
| 14:20:21 | frickler | gtema: slaweq: ^^ that works quite nicely now, seems only neutron is affected with another list of tests that need to check for extension-availability, I'll add in another patch https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_26e/826877/2/check/openstacksdk-functional-devstack-legacy/26e71bd/testr_results.html | |
| 14:20:43 | gtema | oh great | |
| 14:21:01 | gtema | is looking really not so bad | |
| 14:24:29 | opendevreview | Mamatisa Nurmatov proposed openstack/python-openstackclient master: Support Neutron Local IP CRUD https://review.opendev.org/c/openstack/python-openstackclient/+/805618 | |
| 14:28:02 | opendevreview | Jiri Podivin proposed openstack/cliff stable/train: Adding upper constraint on bandit as it isn't covered by openstack constraint https://review.opendev.org/c/openstack/cliff/+/826894 | |
| 14:31:44 | isabek | stephenfin: Thank you for your review! Updated OSC patch [1] . 1) https://review.opendev.org/c/openstack/python-openstackclient/+/805618 | |
| 15:13:03 | opendevreview | Merged openstack/openstacksdk master: Skip qos-pps-minimum tests if extension is missing https://review.opendev.org/c/openstack/openstacksdk/+/826873 | |
| 15:16:29 | opendevreview | Dr. Jens Harbott proposed openstack/openstacksdk master: WIP: Test new legacy job https://review.opendev.org/c/openstack/openstacksdk/+/826877 | |
| 15:16:30 | opendevreview | Dr. Jens Harbott proposed openstack/openstacksdk master: Skip tests when needed extensions are disabled https://review.opendev.org/c/openstack/openstacksdk/+/826899 | |
| 16:43:12 | frickler | slaweq: ralonsoh: ^^ for some of those I'm not sure if I selected the correct extension, please check | |
| 16:44:23 | frickler | gtema: are you fine with ussuri as current "legacy" version or would you want to test against even older branches? I'd then un-wip the new job patch | |
| 16:45:04 | ralonsoh | frickler, let me chck | |
| 16:45:06 | gtema | let's go with ussuri, we can later go further. I think it is really great to start with that already | |
| 16:49:38 | ralonsoh | frickler, I think the extensions are correct | |
| 16:50:08 | frickler | ralonsoh: cool, thx for checking | |
| 16:57:47 | opendevreview | Artem Goncharov proposed openstack/openstacksdk master: Restore get_compute_limits backward compatibility https://review.opendev.org/c/openstack/openstacksdk/+/826920 | |
| 17:35:24 | opendevreview | Artem Goncharov proposed openstack/openstacksdk master: Improve compute flavor handling https://review.opendev.org/c/openstack/openstacksdk/+/826924 | |
| 17:40:00 | opendevreview | Artem Goncharov proposed openstack/openstacksdk master: The recent changes have broken nodepool's use of openstacksdk. This job brings up a cloud environment with devstack, then a nodepool instance that talks to it. It then builds an image with dib and attempts to have nodepool upload, boot and talk to the instance in the devstack cloud. https://review.opendev.org/c/openstack/openstacksdk/+/826543 | |
| 17:41:01 | opendevreview | Artem Goncharov proposed openstack/openstacksdk master: Improve compute flavor handling https://review.opendev.org/c/openstack/openstacksdk/+/826924 | |
| 18:50:55 | opendevreview | Merged openstack/cliff stable/train: Adding upper constraint on bandit as it isn't covered by openstack constraint https://review.opendev.org/c/openstack/cliff/+/826894 | |
| 19:37:55 | opendevreview | Dr. Jens Harbott proposed openstack/openstacksdk master: Skip tests when needed extensions are disabled https://review.opendev.org/c/openstack/openstacksdk/+/826899 | |
| 19:37:56 | opendevreview | Dr. Jens Harbott proposed openstack/openstacksdk master: Revive legacy job https://review.opendev.org/c/openstack/openstacksdk/+/826877 | |
| 20:05:22 | opendevreview | Dr. Jens Harbott proposed openstack/openstacksdk master: Restore get_compute_limits backward compatibility https://review.opendev.org/c/openstack/openstacksdk/+/826920 | |
| 20:17:26 | opendevreview | Mamatisa Nurmatov proposed openstack/python-openstackclient master: (wip) dnm test gate https://review.opendev.org/c/openstack/python-openstackclient/+/826936 | |
| #openstack-sdks - 2022-01-29 | |||
| 10:55:09 | opendevreview | Artem Goncharov proposed openstack/openstacksdk master: Improve compute flavor handling https://review.opendev.org/c/openstack/openstacksdk/+/826924 | |
| #openstack-sdks - 2022-01-30 | |||
| 18:43:21 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: Add zuul artifact https://review.opendev.org/c/openstack/ansible-collections-openstack/+/826746 | |
| 18:47:39 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: WIP fix collection for new SDK https://review.opendev.org/c/openstack/ansible-collections-openstack/+/825291 | |
| #openstack-sdks - 2022-01-31 | |||
| 08:14:23 | opendevreview | Artem Goncharov proposed openstack/openstacksdk master: Use empty read_acl for swift container https://review.opendev.org/c/openstack/openstacksdk/+/826993 | |
| 08:56:03 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: WIP fix collection for new SDK https://review.opendev.org/c/openstack/ansible-collections-openstack/+/825291 | |
| 09:29:46 | opendevreview | Mamatisa Nurmatov proposed openstack/python-openstackclient master: (wip) dnm test gate https://review.opendev.org/c/openstack/python-openstackclient/+/826936 | |
| 09:44:31 | isabek | gtema, frickler: Hi! OSC job build-openstack-releasenotes is failing now. See: https://paste.opendev.org/show/812446/ | |
| 09:45:23 | isabek | do we need change all notes link to new style? | |
| 09:48:20 | isabek | https://zuul.opendev.org/t/openstack/build/c45ba8704105458ab8b75f8be99d7cac | |
| 09:56:14 | opendevreview | Merged openstack/openstacksdk master: Skip tests when needed extensions are disabled https://review.opendev.org/c/openstack/openstacksdk/+/826899 | |
| 10:30:53 | frickler | isabek: hmm, nice. not. I don't think that we want to touch releasenotes retrospectively. I'll have a look | |
| 10:49:46 | opendevreview | Jiri Podivin proposed openstack/cliff stable/train: [Train-Only] Remove lower-constraints https://review.opendev.org/c/openstack/cliff/+/826876 | |
| 10:50:33 | isabek | frickler: Thanks! | |
| 11:14:05 | frickler | hmm, this feature was introduced in Sphinx 4.4.0 https://github.com/sphinx-doc/sphinx/pull/9800. seeing the number of places where we use the shortcut vs where we don't use it, it might be more convenient to drop it completely. | |
| 11:20:15 | opendevreview | Merged openstack/openstacksdk master: Revive legacy job https://review.opendev.org/c/openstack/openstacksdk/+/826877 | |
| 11:45:31 | opendevreview | Merged openstack/ansible-collections-openstack master: Add zuul artifact https://review.opendev.org/c/openstack/ansible-collections-openstack/+/826746 | |
| 12:36:50 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: Add CentOS 9 tripleo job https://review.opendev.org/c/openstack/ansible-collections-openstack/+/827066 | |
| 13:55:24 | opendevreview | Balazs Gibizer proposed openstack/python-openstackclient master: Add support for CRUD operations for QoS minimum packet rate rule https://review.opendev.org/c/openstack/python-openstackclient/+/810559 | |
| 14:12:50 | opendevreview | Merged openstack/openstacksdk master: Use empty read_acl for swift container https://review.opendev.org/c/openstack/openstacksdk/+/826993 | |
| 14:14:50 | isabek | gtema: Hi! Can you please check https://review.opendev.org/c/openstack/openstacksdk/+/822305 ? All comments were resolved | |
| 14:15:11 | isabek | Thanks in advance! | |
| 23:56:25 | headphoneJames | frickler: I rebuilt the system scope role patch for sdk on the master branch. All comments should have been addressed. Can you please review when you have time? Thanks! https://review.opendev.org/c/openstack/openstacksdk/+/826193 | |
| #openstack-sdks - 2022-02-01 | |||
| 09:22:06 | opendevreview | Lajos Katona proposed openstack/python-openstackclient master: Add support for CRUD operations for QoS minimum packet rate rule https://review.opendev.org/c/openstack/python-openstackclient/+/810559 | |
| 11:26:35 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: WIP fix identity domain 'enabled' erorr https://review.opendev.org/c/openstack/ansible-collections-openstack/+/827267 | |
| 12:14:09 | opendevreview | Ananya proposed openstack/ansible-collections-openstack master: Moves keypair_info from cloud to proxy object https://review.opendev.org/c/openstack/ansible-collections-openstack/+/827274 | |
| 13:03:53 | opendevreview | Ananya proposed openstack/ansible-collections-openstack master: Moves keypair_info from cloud to proxy object https://review.opendev.org/c/openstack/ansible-collections-openstack/+/827274 | |
| 15:53:53 | opendevreview | Jiri Podivin proposed openstack/cliff stable/ussuri: Remove lower-constraints https://review.opendev.org/c/openstack/cliff/+/826875 | |
| 15:54:19 | opendevreview | Jiri Podivin proposed openstack/cliff stable/train: [Train-Only] Remove lower-constraints https://review.opendev.org/c/openstack/cliff/+/826876 | |
| 16:13:07 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: Move identity domain to use proxy layer https://review.opendev.org/c/openstack/ansible-collections-openstack/+/827267 | |
| 16:33:29 | opendevreview | Artem Goncharov proposed openstack/openstacksdk master: Improve compute flavor handling https://review.opendev.org/c/openstack/openstacksdk/+/826924 | |
| 17:31:07 | stephenfin | diablo_rojo_phone, gtema: I'm joining, but I will be 5 mins late | |
| 18:02:49 | opendevreview | Merged openstack/openstacksdk master: Add query parameters to local ip https://review.opendev.org/c/openstack/openstacksdk/+/822305 | |
| 20:45:55 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: Move identity domain to use proxy layer https://review.opendev.org/c/openstack/ansible-collections-openstack/+/827267 | |
| #openstack-sdks - 2022-02-02 | |||
| 00:22:29 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: DNM test identity fix with new SDK https://review.opendev.org/c/openstack/ansible-collections-openstack/+/827379 | |
| 07:10:34 | opendevreview | Merged openstack/keystoneauth master: Fix docs build https://review.opendev.org/c/openstack/keystoneauth/+/825777 | |
| 10:37:19 | opendevreview | Frode Nordahl proposed openstack/python-openstackclient master: Add support for 'smart-nic' vnic-type https://review.opendev.org/c/openstack/python-openstackclient/+/814518 | |
| 13:16:06 | opendevreview | Takashi Kajinami proposed openstack/openstackclient master: Include osc-placement in the plugin check job https://review.opendev.org/c/openstack/openstackclient/+/780601 | |
| 13:46:12 | opendevreview | Ananya proposed openstack/ansible-collections-openstack master: Moves keypair_info from cloud to proxy object https://review.opendev.org/c/openstack/ansible-collections-openstack/+/827274 | |
| 13:51:26 | opendevreview | Ananya proposed openstack/ansible-collections-openstack master: Moves keypair_info from cloud to proxy object https://review.opendev.org/c/openstack/ansible-collections-openstack/+/827274 | |
| 14:10:12 | opendevreview | Ananya proposed openstack/ansible-collections-openstack master: Moves keypair_info from cloud to proxy object https://review.opendev.org/c/openstack/ansible-collections-openstack/+/827274 | |
| 14:12:01 | opendevreview | Merged openstack/cliff stable/victoria: Remove lower-constraints https://review.opendev.org/c/openstack/cliff/+/826874 | |
| 14:14:54 | opendevreview | Dr. Jens Harbott proposed openstack/python-openstackclient master: Don't fail on warnings for reno build https://review.opendev.org/c/openstack/python-openstackclient/+/827504 | |
| 14:15:23 | frickler | gtema: ^^ this isn't the nicest solution but I failed to find anything else that would seem feasible | |
| 14:16:26 | gtema | hm, will have a look. Thanks | |
| 14:19:51 | frickler | the number of releasenotes that would need fixing is above 500. we could drop the :lpbug: link instead and fix the 12 renos that reference it, but I haven't been able to make the renos not be pushed into the latest release when touching them | |
| 14:37:04 | fungi | yeah, reno's a neat idea, but the biggest drawback it has is there's effectively no way to go back and correct a past release note | |
| 14:37:49 | fungi | we did some serious git surgery in zuul at one point to try to add a release note to a prior release, and that was nightmarish | |
| 14:38:42 | fungi | maybe if it grew some sort of configurable override, like the ability to add metadata to a note indicating the release it should appear in, and then have reno prioritize that over its automatic git inspection mechanism | |
| 14:42:12 | fungi | i doubt i'll find time to try to implement that, but i'll stick it in my mental backlog for the next time i get bored | |
| 15:19:58 | opendevreview | Jonathan Rosser proposed openstack/openstacksdk master: Add support for federation service providers https://review.opendev.org/c/openstack/openstacksdk/+/799118 | |
| 16:16:21 | opendevreview | Merged openstack/openstacksdk master: Improve compute flavor handling https://review.opendev.org/c/openstack/openstacksdk/+/826924 | |
| 16:52:18 | opendevreview | Artem Goncharov proposed openstack/python-openstackclient master: Don't fail on warnings for reno build https://review.opendev.org/c/openstack/python-openstackclient/+/827504 | |
| 17:34:27 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: Move identity domain to use proxy layer https://review.opendev.org/c/openstack/ansible-collections-openstack/+/827267 | |