| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2022-01-21 | |||
| 13:56:34 | gtema | I do not actually even know why it is reported here, but it doesn't harm, cause we depend on that heavily | |
| 18:16:01 | opendevreview | hamza proposed openstack/ansible-collections-openstack master: Add dns_[name,domain] to the port module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/825674 | |
| 21:49:33 | opendevreview | hamza proposed openstack/ansible-collections-openstack master: Add dns_[name,domain] to the port module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/825674 | |
| #openstack-sdks - 2022-01-23 | |||
| 18:38:58 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: DNM test with SDK https://review.opendev.org/c/openstack/ansible-collections-openstack/+/825291 | |
| #openstack-sdks - 2022-01-24 | |||
| 13:26:41 | isabek | gtema: Hi! On this patch set you said that no need 'project_id' [1]. But according API def we need list by project_id. When I'm removing project_id from query_paramateres, there is key error | |
| 13:26:46 | isabek | 1) https://review.opendev.org/c/openstack/openstacksdk/+/814765/2/openstack/network/v2/local_ip.py#38 | |
| 13:27:47 | gtema | isabek: you should leave simply `, project_id, ...`, but not `project_id='project_id'` | |
| 13:30:00 | gtema | however I was not able to find in the neutron whether this extension will at all support querying by project_id | |
| 13:31:06 | isabek | ohh, ok got it! Thank you! | |
| 13:31:15 | isabek | I will update | |
| 13:36:08 | opendevreview | Mamatisa Nurmatov proposed openstack/openstacksdk master: Add query parameters to local ip https://review.opendev.org/c/openstack/openstacksdk/+/822305 | |
| 13:40:56 | dtantsur | frickler, gtema, I wonder if keystoneauth should move under the SDK or maybe have a shared ownership | |
| 13:41:09 | dtantsur | it seems to be that its main usage is in various clients | |
| 14:09:12 | opendevreview | Merged openstack/openstacksdk master: Merge feature/r1 branch into master https://review.opendev.org/c/openstack/openstacksdk/+/823961 | |
| 14:10:07 | gtema | dtantsur: we may open the discussion on the mailinglist about that | |
| 14:47:08 | thomasb06 | gtema: Hello. This should be it for the Cloud services. All the docstrings seem to pass the tests | |
| 15:01:15 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: DNM test CI https://review.opendev.org/c/openstack/ansible-collections-openstack/+/820480 | |
| 17:36:17 | opendevreview | hamza proposed openstack/ansible-collections-openstack master: Add dns_[name,domain] to the port module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/825674 | |
| 20:22:11 | opendevreview | hamza proposed openstack/ansible-collections-openstack master: Add dns_[name,domain] to the port module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/825674 | |
| 20:29:53 | opendevreview | hamza proposed openstack/ansible-collections-openstack master: Add dns_[name,domain] to the port module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/825674 | |
| 23:14:21 | opendevreview | James Kirsch proposed openstack/openstacksdk master: Identity: Add support for system role assignment https://review.opendev.org/c/openstack/openstacksdk/+/826193 | |
| #openstack-sdks - 2022-01-25 | |||
| 02:35:15 | opendevreview | James Kirsch proposed openstack/openstacksdk master: Identity: Add support for system role assignment https://review.opendev.org/c/openstack/openstacksdk/+/826193 | |
| 06:13:07 | opendevreview | Ales Musil proposed openstack/openstacksdk master: network: Fix update of network provider https://review.opendev.org/c/openstack/openstacksdk/+/761933 | |
| 10:39:38 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: Fix CI for new openstack SDK https://review.opendev.org/c/openstack/ansible-collections-openstack/+/826264 | |
| 12:02:06 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: DNM test with SDK https://review.opendev.org/c/openstack/ansible-collections-openstack/+/825291 | |
| 12:55:27 | ralonsoh | artom, hi, can you please check https://review.opendev.org/c/openstack/python-openstackclient/+/806016? | |
| 12:55:36 | ralonsoh | thank you in advance | |
| 12:57:04 | ralonsoh | stephenfin, ^^ | |
| 13:08:17 | stephenfin | ralonsoh: I was waiting on a new revision https://review.opendev.org/c/openstack/python-openstackclient/+/806016/7/openstackclient/tests/functional/common/test_quota.py#174 | |
| 13:08:32 | ralonsoh | ok | |
| 13:16:29 | artom | ralonsoh, I'm assuming you mean gtema? :) | |
| 13:16:40 | artom | I'm the *other* Art(e|o)m | |
| 13:19:20 | ralonsoh | artom, sorry!! | |
| 13:52:44 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: DNM test with SDK https://review.opendev.org/c/openstack/ansible-collections-openstack/+/825291 | |
| 14:06:32 | artom | ralonsoh, no worries! It's rare that there are multiples of my first name in anglophone spheres, for once I get to enjoy the chaos that it causes ;) | |
| 14:06:48 | ralonsoh | hahaha | |
| 14:24:04 | isabek | gtema, hi! Can you please take a look to this patches [1], [2]? Thanks in advance! | |
| 14:24:05 | isabek | 1) https://review.opendev.org/c/openstack/python-openstackclient/+/805618 2) https://review.opendev.org/c/openstack/openstacksdk/+/822305 | |
| 14:43:05 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: DNM test ooo with SDK https://review.opendev.org/c/openstack/ansible-collections-openstack/+/826296 | |
| 15:00:58 | opendevreview | Merged openstack/ansible-collections-openstack master: Fix CI for new openstack SDK https://review.opendev.org/c/openstack/ansible-collections-openstack/+/826264 | |
| 15:59:43 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: Add dns_[name,domain] to the port module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/825674 | |
| 17:32:03 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: Add new SDK job non-voting https://review.opendev.org/c/openstack/ansible-collections-openstack/+/826339 | |
| 18:02:21 | opendevreview | hamza proposed openstack/ansible-collections-openstack master: Add dns_[name,domain] to the port module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/825674 | |
| 18:22:47 | opendevreview | hamza proposed openstack/ansible-collections-openstack master: Add dns_[name,domain] to the port module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/825674 | |
| 19:00:59 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: Add new SDK job non-voting https://review.opendev.org/c/openstack/ansible-collections-openstack/+/826339 | |
| 19:45:27 | opendevreview | hamza proposed openstack/ansible-collections-openstack master: Add dns_[name,domain] to the port module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/825674 | |
| 20:52:45 | opendevreview | hamza proposed openstack/ansible-collections-openstack master: Add dns_[name,domain] to the port module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/825674 | |
| 21:50:59 | opendevreview | Merged openstack/ansible-collections-openstack master: Add new SDK job non-voting https://review.opendev.org/c/openstack/ansible-collections-openstack/+/826339 | |
| 21:56:06 | opendevreview | hamza proposed openstack/ansible-collections-openstack master: Add dns_[name,domain] to the port module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/825674 | |
| 22:06:50 | opendevreview | hamza proposed openstack/ansible-collections-openstack master: Add dns_[name,domain] to the port module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/825674 | |
| 22:18:56 | opendevreview | James Kirsch proposed openstack/ansible-collections-openstack master: Add support for system role in role assignment https://review.opendev.org/c/openstack/ansible-collections-openstack/+/826395 | |
| 22:51:33 | opendevreview | hamza proposed openstack/ansible-collections-openstack master: Add dns_[name,domain] to the port module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/825674 | |
| #openstack-sdks - 2022-01-26 | |||
| 01:57:48 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: DNM test with SDK https://review.opendev.org/c/openstack/ansible-collections-openstack/+/825291 | |
| 06:43:23 | opendevreview | Ales Musil proposed openstack/openstacksdk master: network: Fix update of network provider https://review.opendev.org/c/openstack/openstacksdk/+/761933 | |
| 07:52:33 | opendevreview | Mamatisa Nurmatov proposed openstack/openstacksdk master: Add query parameters to local ip https://review.opendev.org/c/openstack/openstacksdk/+/822305 | |
| 08:41:28 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: DNM test with SDK https://review.opendev.org/c/openstack/ansible-collections-openstack/+/825291 | |
| 12:01:42 | opendevreview | Frode Nordahl proposed openstack/python-openstackclient master: Add support for 'smart-nic' vnic-type https://review.opendev.org/c/openstack/python-openstackclient/+/814518 | |
| 12:20:40 | 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 | |
| 12:24:28 | 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 | |
| 12:56:11 | opendevreview | Merged openstack/ansible-collections-openstack master: Add dns_[name,domain] to the port module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/825674 | |
| 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 | |