| 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:20 |
|---|---|---|
| frickler | gtema: ^^ this should hopefully fix the test errors on stable branches, I'll try to verify locally | 11:22 |
| gtema | Yes, seen that. Thanks | 11:22 |
| gtema | I think we need to establish devstack jobs for all stable branches (keeping latest SDK) | 11:23 |
| gtema | That might reveal more of those | 11:23 |
| opendevreview | Jiri Podivin proposed openstack/cliff stable/victoria: Remove lower-constraints https://review.opendev.org/c/openstack/cliff/+/826874 | 11:31 |
| opendevreview | Jiri Podivin proposed openstack/cliff stable/ussuri: Remove lower-constraints https://review.opendev.org/c/openstack/cliff/+/826875 | 11:33 |
| opendevreview | Jiri Podivin proposed openstack/cliff stable/train: [Train-Only] Remove lower-constraints https://review.opendev.org/c/openstack/cliff/+/826876 | 11:37 |
| frickler | slaweq: fyi, all those sdk tests are also spectacularly failing with NEUTRON_ENABLE_SCOPE=true ^^ | 11:37 |
| opendevreview | Dr. Jens Harbott proposed openstack/openstacksdk master: WIP: Test new legacy job https://review.opendev.org/c/openstack/openstacksdk/+/826877 | 11:57 |
| frickler | gtema: ^^ let's see how this goes, maybe it will be enough to run against the oldest supported branch | 11:58 |
| gtema | hopefully | 11:58 |
| frickler | I tested 826873 and it seems to work like I intended, care to fast-approve? | 11:59 |
| gtema | done, hopefully tests on zuul are also happy | 11:59 |
| * frickler has the finger on the recheck button ;) | 12:04 | |
| frickler | but of course I'll also check for genuine failures | 12:04 |
| gtema | :) | 12:05 |
| ralonsoh | stephenfin, hi. Is this enough? --> https://review.opendev.org/c/openstack/python-openstackclient/+/806016/7/openstackclient/tests/functional/common/test_quota.py | 12:10 |
| stephenfin | It sure does :) I missed that. Apologies! | 12:14 |
| ralonsoh | stephenfin, thanks a lot | 12:22 |
| stephenfin | np, sorry for the delay | 12:25 |
| 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:31 |
| isabek | 2) https://review.opendev.org/c/openstack/python-openstackclient/+/805618 | 12:32 |
| isabek | Thanks in advance! | 12:32 |
| stephenfin | dtantsur: Likely dumb question, but I assume ironic w/o nova still needs glance? | 12:54 |
| dtantsur | stephenfin: ironic can work in any combination: with nova, without nova with glance, with neither nova nor glance, etc. | 12:55 |
| stephenfin | dtantsur: Ack. Will it be present in the openstacksdk-functional-devstack-ironic job? | 12:56 |
| * stephenfin is reading the .zuul.yaml at the moment but isn't sure where to look | 12:57 | |
| dtantsur | stephenfin: I'm looking at the definition, and it seems like we don't disable glance there | 12:57 |
| dtantsur | ideally, we should not rely on knowing the job definitions: people may run these tests locally | 12:57 |
| stephenfin | fair point | 12:59 |
| opendevreview | Stephen Finucane proposed openstack/openstacksdk master: tests: Centralize configuration of default flavor, image https://review.opendev.org/c/openstack/openstacksdk/+/822814 | 12:59 |
| stephenfin | dtantsur: ^ hopefully that will do the trick | 12:59 |
| dtantsur | thx, let's see what the CI says | 12:59 |
| stephenfin | (y) | 13:00 |
| frickler | focal is not supported for ussuri, great | 13:07 |
| opendevreview | Dr. Jens Harbott proposed openstack/openstacksdk master: WIP: Test new legacy job https://review.opendev.org/c/openstack/openstacksdk/+/826877 | 13:13 |
| 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 |
| gtema | oh great | 14:20 |
| gtema | is looking really not so bad | 14:21 |
| opendevreview | Mamatisa Nurmatov proposed openstack/python-openstackclient master: Support Neutron Local IP CRUD https://review.opendev.org/c/openstack/python-openstackclient/+/805618 | 14:24 |
| 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:28 |
| isabek | stephenfin: Thank you for your review! Updated OSC patch [1] . 1) https://review.opendev.org/c/openstack/python-openstackclient/+/805618 | 14:31 |
| opendevreview | Merged openstack/openstacksdk master: Skip qos-pps-minimum tests if extension is missing https://review.opendev.org/c/openstack/openstacksdk/+/826873 | 15:13 |
| opendevreview | Dr. Jens Harbott proposed openstack/openstacksdk master: WIP: Test new legacy job https://review.opendev.org/c/openstack/openstacksdk/+/826877 | 15:16 |
| opendevreview | Dr. Jens Harbott proposed openstack/openstacksdk master: Skip tests when needed extensions are disabled https://review.opendev.org/c/openstack/openstacksdk/+/826899 | 15:16 |
| frickler | slaweq: ralonsoh: ^^ for some of those I'm not sure if I selected the correct extension, please check | 16:43 |
| 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:44 |
| ralonsoh | frickler, let me chck | 16:45 |
| gtema | let's go with ussuri, we can later go further. I think it is really great to start with that already | 16:45 |
| ralonsoh | frickler, I think the extensions are correct | 16:49 |
| frickler | ralonsoh: cool, thx for checking | 16:50 |
| opendevreview | Artem Goncharov proposed openstack/openstacksdk master: Restore get_compute_limits backward compatibility https://review.opendev.org/c/openstack/openstacksdk/+/826920 | 16:57 |
| opendevreview | Artem Goncharov proposed openstack/openstacksdk master: Improve compute flavor handling https://review.opendev.org/c/openstack/openstacksdk/+/826924 | 17:35 |
| 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:40 |
| opendevreview | Artem Goncharov proposed openstack/openstacksdk master: Improve compute flavor handling https://review.opendev.org/c/openstack/openstacksdk/+/826924 | 17:41 |
| 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 | 18:50 |
| 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 |
| opendevreview | Dr. Jens Harbott proposed openstack/openstacksdk master: Revive legacy job https://review.opendev.org/c/openstack/openstacksdk/+/826877 | 19:37 |
| opendevreview | Dr. Jens Harbott proposed openstack/openstacksdk master: Restore get_compute_limits backward compatibility https://review.opendev.org/c/openstack/openstacksdk/+/826920 | 20:05 |
| opendevreview | Mamatisa Nurmatov proposed openstack/python-openstackclient master: (wip) dnm test gate https://review.opendev.org/c/openstack/python-openstackclient/+/826936 | 20:17 |
| *** efried1 is now known as efried | 21:24 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!