| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2022-01-26 | |||
| 14:20:53 | opendevreview | Pierre Riteau proposed openstack/ansible-collections-openstack master: Support for setting parent when creating projects https://review.opendev.org/c/openstack/ansible-collections-openstack/+/825108 | |
| 14:35:31 | opendevreview | Pierre Riteau proposed openstack/ansible-collections-openstack master: Support setting project tags https://review.opendev.org/c/openstack/ansible-collections-openstack/+/826475 | |
| 15:21:31 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: Write tests log to a separate file https://review.opendev.org/c/openstack/ansible-collections-openstack/+/826454 | |
| 23:02:28 | opendevreview | Ian Wienand proposed openstack/openstacksdk master: Revert "Temporarily disable nodepool job" https://review.opendev.org/c/openstack/openstacksdk/+/826543 | |
| #openstack-sdks - 2022-01-27 | |||
| 11:19:59 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: Write tests log to a separate file https://review.opendev.org/c/openstack/ansible-collections-openstack/+/826454 | |
| 13:53:48 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: Write tests log to a separate file https://review.opendev.org/c/openstack/ansible-collections-openstack/+/826454 | |
| 16:58:06 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: doc: Update testing documentation https://review.opendev.org/c/openstack/openstacksdk/+/822813 | |
| 16:58:06 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: doc: Remove references to 'examples' test env https://review.opendev.org/c/openstack/openstacksdk/+/822812 | |
| 16:58:07 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: tests: Centralize configuration of default flavor, image https://review.opendev.org/c/openstack/openstacksdk/+/822814 | |
| 17:40:53 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: Add zuul artifact https://review.opendev.org/c/openstack/ansible-collections-openstack/+/826746 | |
| 18:01:23 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: Add zuul artifact https://review.opendev.org/c/openstack/ansible-collections-openstack/+/826746 | |
| 19:01:43 | opendevreview | Merged openstack/ansible-collections-openstack master: Write tests log to a separate file https://review.opendev.org/c/openstack/ansible-collections-openstack/+/826454 | |
| 23:51:20 | headphoneJames | https://review.opendev.org/c/openstack/openstacksdk/+/826193 is ready for review. Thanks! | |
| #openstack-sdks - 2022-01-28 | |||
| 11:20:30 | opendevreview | Dr. Jens Harbott proposed openstack/openstacksdk master: Skip qos-pps-minimum tests if extension is missing https://review.opendev.org/c/openstack/openstacksdk/+/826873 | |
| 11:22:02 | frickler | gtema: ^^ this should hopefully fix the test errors on stable branches, I'll try to verify locally | |
| 11:22:34 | gtema | Yes, seen that. Thanks | |
| 11:23:22 | gtema | I think we need to establish devstack jobs for all stable branches (keeping latest SDK) | |
| 11:23:34 | gtema | That might reveal more of those | |
| 11:31:54 | opendevreview | Jiri Podivin proposed openstack/cliff stable/victoria: Remove lower-constraints https://review.opendev.org/c/openstack/cliff/+/826874 | |
| 11:33:50 | opendevreview | Jiri Podivin proposed openstack/cliff stable/ussuri: Remove lower-constraints https://review.opendev.org/c/openstack/cliff/+/826875 | |
| 11:37:06 | opendevreview | Jiri Podivin proposed openstack/cliff stable/train: [Train-Only] Remove lower-constraints https://review.opendev.org/c/openstack/cliff/+/826876 | |
| 11:37:07 | frickler | slaweq: fyi, all those sdk tests are also spectacularly failing with NEUTRON_ENABLE_SCOPE=true ^^ | |
| 11:57:47 | opendevreview | Dr. Jens Harbott proposed openstack/openstacksdk master: WIP: Test new legacy job https://review.opendev.org/c/openstack/openstacksdk/+/826877 | |
| 11:58:20 | frickler | gtema: ^^ let's see how this goes, maybe it will be enough to run against the oldest supported branch | |
| 11:58:42 | gtema | hopefully | |
| 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 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: tests: Centralize configuration of default flavor, image https://review.opendev.org/c/openstack/openstacksdk/+/822814 | |
| 12:59:32 | stephenfin | fair point | |
| 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 | |