| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2025-01-18 | |||
| 08:25:14 | opendevreview | Pierre Riteau proposed openstack/openstacksdk master: Fix TestKeypairAdmin tests https://review.opendev.org/c/openstack/openstacksdk/+/939564 | |
| 09:31:20 | opendevreview | Pierre Riteau proposed openstack/openstacksdk master: Fix TestKeypairAdmin tests https://review.opendev.org/c/openstack/openstacksdk/+/939564 | |
| 09:32:50 | gtema | rpittau: there is small issue in the sdk itself that it doesn't properly pass query parameter in the "get" case. It only handles is in the list | |
| 09:33:15 | gtema | I am looking at it now | |
| 09:37:35 | gtema | priteau: there is small issue in the sdk itself that it doesn't properly pass query parameter in the "get" case. It only handles is in the list | |
| 09:37:35 | gtema | I am looking at it now (sorry, mityped your nick before) | |
| 10:03:40 | opendevreview | Artem Goncharov proposed openstack/openstacksdk master: Fix TestKeypairAdmin tests and compute.get_keypair https://review.opendev.org/c/openstack/openstacksdk/+/939564 | |
| 10:16:10 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Generate TUI response structure https://review.opendev.org/c/openstack/codegenerator/+/938997 | |
| 11:20:04 | opendevreview | Artem Goncharov proposed openstack/openstacksdk master: Fix TestKeypairAdmin tests and compute.*_keypair https://review.opendev.org/c/openstack/openstacksdk/+/939564 | |
| 12:01:39 | opendevreview | Merged openstack/python-openstackclient master: Add the trunk subports information to the port list command https://review.opendev.org/c/openstack/python-openstackclient/+/926611 | |
| 12:13:04 | opendevreview | Merged openstack/ansible-collections-openstack master: Update tags when changing server https://review.opendev.org/c/openstack/ansible-collections-openstack/+/939396 | |
| 13:16:12 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Drop overloaded federation service provider schema https://review.opendev.org/c/openstack/codegenerator/+/939584 | |
| 13:21:17 | opendevreview | Artem Goncharov proposed openstack/openstacksdk master: Fix TestKeypairAdmin tests and compute.*_keypair https://review.opendev.org/c/openstack/openstacksdk/+/939564 | |
| 19:04:57 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: CI: add retries for trait test https://review.opendev.org/c/openstack/ansible-collections-openstack/+/939591 | |
| 20:37:18 | opendevreview | Merged openstack/ansible-collections-openstack master: Fix missed client_cert in OpenStackModule https://review.opendev.org/c/openstack/ansible-collections-openstack/+/939525 | |
| #openstack-sdks - 2025-01-20 | |||
| 08:41:15 | opendevreview | Pierre Riteau proposed openstack/openstacksdk master: Fix TestKeypairAdmin tests and compute.*_keypair https://review.opendev.org/c/openstack/openstacksdk/+/939564 | |
| 08:54:05 | opendevreview | Pierre Riteau proposed openstack/openstacksdk master: Fix TestKeypairAdmin tests and compute.*_keypair https://review.opendev.org/c/openstack/openstacksdk/+/939564 | |
| 09:55:57 | opendevreview | Mridula Joshi proposed openstack/openstacksdk master: Modified create_image https://review.opendev.org/c/openstack/openstacksdk/+/910218 | |
| 13:29:25 | opendevreview | Merged openstack/openstacksdk master: Fix TestKeypairAdmin tests and compute.*_keypair https://review.opendev.org/c/openstack/openstacksdk/+/939564 | |
| 15:58:56 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: Release 2.4.1 version https://review.opendev.org/c/openstack/ansible-collections-openstack/+/939651 | |
| 16:49:30 | opendevreview | Merged openstack/ansible-collections-openstack master: Release 2.4.1 version https://review.opendev.org/c/openstack/ansible-collections-openstack/+/939651 | |
| 19:41:02 | opendevreview | Merged openstack/ansible-collections-openstack master: CI: add retries for trait test https://review.opendev.org/c/openstack/ansible-collections-openstack/+/939591 | |
| #openstack-sdks - 2025-01-21 | |||
| 10:50:58 | opendevreview | Slawek Kaplonski proposed openstack/openstacksdk master: Add "vlan_qinq" attribute to the "network" resource https://review.opendev.org/c/openstack/openstacksdk/+/939703 | |
| 11:27:48 | opendevreview | Slawek Kaplonski proposed openstack/python-openstackclient master: Add "qinq-vlan" and "no-qinq-vlan" params to the "network create" cmd https://review.opendev.org/c/openstack/python-openstackclient/+/939706 | |
| 13:44:57 | opendevreview | Fernando Royo proposed openstack/python-openstackclient master: Get "security_groups" when port list https://review.opendev.org/c/openstack/python-openstackclient/+/939715 | |
| 18:51:32 | opendevreview | Fernando Royo proposed openstack/python-openstackclient master: Get "security_groups" when port list https://review.opendev.org/c/openstack/python-openstackclient/+/939715 | |
| #openstack-sdks - 2025-01-22 | |||
| 08:17:43 | opendevreview | Ivan Anfimov proposed openstack/python-openstackclient master: Update README for use python3 https://review.opendev.org/c/openstack/python-openstackclient/+/939769 | |
| 08:22:56 | opendevreview | Ivan Anfimov proposed openstack/python-openstackclient master: Update README for use python3 https://review.opendev.org/c/openstack/python-openstackclient/+/939769 | |
| 09:32:16 | opendevreview | Ivan Anfimov proposed openstack/python-openstackclient master: Update README for use python3 https://review.opendev.org/c/openstack/python-openstackclient/+/939769 | |
| 09:54:13 | opendevreview | Ivan Anfimov proposed openstack/python-openstackclient master: Update README for use python3 https://review.opendev.org/c/openstack/python-openstackclient/+/939769 | |
| 14:33:54 | opendevreview | Ivan Anfimov proposed openstack/python-openstackclient master: Update README for use python3 https://review.opendev.org/c/openstack/python-openstackclient/+/939769 | |
| 14:51:11 | opendevreview | Ivan Anfimov proposed openstack/python-openstackclient master: Update README for use python3 https://review.opendev.org/c/openstack/python-openstackclient/+/939769 | |
| 15:12:09 | opendevreview | Ivan Anfimov proposed openstack/python-openstackclient master: Update README for use python3 https://review.opendev.org/c/openstack/python-openstackclient/+/939769 | |
| 15:24:26 | opendevreview | Ivan Anfimov proposed openstack/python-openstackclient master: Update README for use python3 https://review.opendev.org/c/openstack/python-openstackclient/+/939769 | |
| 16:46:50 | opendevreview | Dmitriy Chubinidze proposed openstack/python-openstackclient master: Update README for use python3 https://review.opendev.org/c/openstack/python-openstackclient/+/939769 | |
| 18:57:06 | opendevreview | Ivan Anfimov proposed openstack/python-openstackclient master: Update README for use python3 https://review.opendev.org/c/openstack/python-openstackclient/+/939769 | |
| 21:24:30 | opendevreview | Ivan Anfimov proposed openstack/python-openstackclient master: Update README for use python3 https://review.opendev.org/c/openstack/python-openstackclient/+/939769 | |
| 22:25:28 | opendevreview | Ivan Anfimov proposed openstack/python-openstackclient master: Update README for use python3 https://review.opendev.org/c/openstack/python-openstackclient/+/939769 | |
| #openstack-sdks - 2025-01-23 | |||
| 04:35:19 | opendevreview | Callum Dickinson proposed openstack/ansible-collections-openstack master: Add the object_containers_info module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/923334 | |
| 04:36:33 | opendevreview | Callum Dickinson proposed openstack/ansible-collections-openstack master: Add the object_containers_info module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/923334 | |
| 05:55:57 | opendevreview | Callum Dickinson proposed openstack/ansible-collections-openstack master: Add the object_containers_info module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/923334 | |
| 06:01:05 | opendevreview | Callum Dickinson proposed openstack/ansible-collections-openstack master: Add the object_containers_info module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/923334 | |
| 06:16:31 | opendevreview | Callum Dickinson proposed openstack/ansible-collections-openstack master: Add the object_containers_info module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/923334 | |
| 09:39:54 | opendevreview | Callum Dickinson proposed openstack/ansible-collections-openstack master: Add the object_containers_info module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/923334 | |
| 12:24:49 | opendevreview | Callum Dickinson proposed openstack/ansible-collections-openstack master: Add the object_containers_info module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/923334 | |
| 14:45:47 | opendevreview | Ivan Anfimov proposed openstack/python-openstackclient master: Update README for use python3 https://review.opendev.org/c/openstack/python-openstackclient/+/939769 | |
| 15:47:03 | opendevreview | Merged openstack/python-openstackclient master: Get "security_groups" when port list https://review.opendev.org/c/openstack/python-openstackclient/+/939715 | |
| 19:40:06 | opendevreview | Callum Dickinson proposed openstack/ansible-collections-openstack master: Add the object_containers_info module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/923334 | |
| 21:43:28 | opendevreview | Callum Dickinson proposed openstack/ansible-collections-openstack master: Add the object_containers_info module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/923334 | |
| #openstack-sdks - 2025-01-24 | |||
| 01:54:21 | opendevreview | Lars Kellogg-Stedman proposed openstack/openstacksdk master: Ignore AuthPluginException in get_all() https://review.opendev.org/c/openstack/openstacksdk/+/940087 | |
| 08:26:31 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Drop overloaded federation service provider schema https://review.opendev.org/c/openstack/codegenerator/+/939584 | |
| 09:39:03 | slaweq | gtema hi, how are you? Can you help me with https://review.opendev.org/c/openstack/openstacksdk/+/939703 maybe? I have no idea why osc-tox-py{39,312}-tips jobs are failing on this patch really, the same tests are passing for me locally :/ | |
| 09:39:17 | slaweq | maybe I am missing something obvious there and you will spot it quickly | |
| 09:39:20 | slaweq | thx in advance | |
| 09:40:00 | gtema | hey hey. It's a curse. I am vice-versa trying to understand why certain patch doesn't work locally while it works remotely :) | |
| 09:40:07 | gtema | looking | |
| 09:40:21 | slaweq | haha | |
| 09:40:24 | slaweq | thx a lot | |
| 09:41:42 | gtema | I see order of columns in the response differs | |
| 09:42:14 | slaweq | there is one "not expected" column there and it is "is_vlan_qinq" | |
| 09:42:22 | gtema | actually this new attr is_vlan_qinq is not defined in the unittest while it is now returned | |
| 09:42:24 | slaweq | this is new thing added by the patch on which it is running | |
| 09:42:42 | slaweq | but why it is passing for me locally? | |
| 09:42:59 | slaweq | I don't understand it :/ | |
| 09:43:15 | gtema | and you see the attr returned when you invoke osc? | |
| 09:48:17 | gtema | slaweq: the test fails for me locally. Can it be that you changed the test locally but didn't pushed it? | |
| 09:48:22 | slaweq | when I run this test locally this new column is there too | |
| 09:50:02 | slaweq | all changes are pushed | |
| 09:50:33 | gtema | I think the problem is (and we had it already) osc respects depends-on, sdk can not install osc patch and it uses the master branch of it | |
| 09:51:07 | slaweq | but then isn't it chicken-egg problem? | |
| 09:51:30 | gtema | it is a circular dependency and you should first prepare osc test (disable it or extend with dummy) | |
| 09:51:37 | gtema | we had this clearly some time ago | |
| 09:51:54 | slaweq | ok, I will do it that way | |
| 09:51:55 | slaweq | thx | |
| 09:51:56 | gtema | once this osc change merges you can proceed with your changes | |
| 09:52:04 | gtema | you are welcome | |
| 09:56:35 | gtema | I see historically there were your/Rodolfo changes that used also depends-on on the sdk change | |
| 09:56:53 | gtema | and Zuul was itself resolving circular dependency - try it this way, slaweq | |
| 09:57:18 | gtema | i.e. https://review.opendev.org/c/openstack/python-openstackclient/+/892792 | |
| 10:09:00 | opendevreview | Slawek Kaplonski proposed openstack/python-openstackclient master: Temporary ignore "is_vlan_qinq" column in the output https://review.opendev.org/c/openstack/python-openstackclient/+/940111 | |
| 10:12:28 | opendevreview | Slawek Kaplonski proposed openstack/python-openstackclient master: Temporary ignore "is_vlan_qinq" column in the output https://review.opendev.org/c/openstack/python-openstackclient/+/940111 | |
| 10:12:28 | opendevreview | Slawek Kaplonski proposed openstack/python-openstackclient master: Add "qinq-vlan" and "no-qinq-vlan" params to the "network create" cmd https://review.opendev.org/c/openstack/python-openstackclient/+/939706 | |
| 10:13:42 | opendevreview | Slawek Kaplonski proposed openstack/openstacksdk master: Add "vlan_qinq" attribute to the "network" resource https://review.opendev.org/c/openstack/openstacksdk/+/939703 | |
| 10:15:50 | slaweq | gtema ok, lets see how ^^ will work | |
| 14:39:06 | opendevreview | Stephen Finucane proposed openstack/keystoneauth master: loading.opts: Add 'argparse_envvars' helper https://review.opendev.org/c/openstack/keystoneauth/+/933110 | |
| 14:39:07 | opendevreview | Stephen Finucane proposed openstack/keystoneauth master: doc: Fix indentation https://review.opendev.org/c/openstack/keystoneauth/+/933111 | |
| 14:39:08 | opendevreview | Stephen Finucane proposed openstack/keystoneauth master: docs: Rework plugin-options doc https://review.opendev.org/c/openstack/keystoneauth/+/933112 | |
| 14:39:09 | opendevreview | Stephen Finucane proposed openstack/keystoneauth master: loading: Add blurb for most plugins https://review.opendev.org/c/openstack/keystoneauth/+/933113 | |
| 14:39:11 | opendevreview | Stephen Finucane proposed openstack/keystoneauth master: docs: Document CLI options, envvars https://review.opendev.org/c/openstack/keystoneauth/+/933114 | |
| 14:49:40 | stephenfin | gtema: If you've time, would you be able to sweep through those? ^ | |
| 14:49:53 | gtema | ack | |
| 14:53:17 | stephenfin | thanks | |
| 15:10:41 | opendevreview | Matthias Rabe proposed openstack/ansible-collections-openstack master: Let clouds_yaml_path behave as documented (Override path to clouds.yaml file) https://review.opendev.org/c/openstack/ansible-collections-openstack/+/940137 | |
| 15:18:48 | opendevreview | Stephen Finucane proposed openstack/keystoneauth master: typing: Simplify some types, other TODOs https://review.opendev.org/c/openstack/keystoneauth/+/935764 | |
| 15:18:49 | opendevreview | Stephen Finucane proposed openstack/keystoneauth master: typing: Remove unused AuthMethod.get_auth_data arguments https://review.opendev.org/c/openstack/keystoneauth/+/935765 | |
| 15:18:50 | opendevreview | Stephen Finucane proposed openstack/keystoneauth master: typing: Remove unused BaseAuthPlugin.get_auth_ref kwargs https://review.opendev.org/c/openstack/keystoneauth/+/935766 | |
| 15:18:51 | opendevreview | Stephen Finucane proposed openstack/keystoneauth master: typing: Remove unused _Rescoped.get_unscoped_auth_ref kwargs https://review.opendev.org/c/openstack/keystoneauth/+/935767 | |
| 15:18:52 | opendevreview | Stephen Finucane proposed openstack/keystoneauth master: typing: Remove unused BaseAuthPlugin.get_headers kwargs https://review.opendev.org/c/openstack/keystoneauth/+/935768 | |
| 15:18:53 | opendevreview | Stephen Finucane proposed openstack/keystoneauth master: typing: Remove unused BaseAuthPlugin.get_token kwargs https://review.opendev.org/c/openstack/keystoneauth/+/935769 | |
| 15:18:55 | opendevreview | Stephen Finucane proposed openstack/keystoneauth master: WIP: typing: Deprecate use of AuthConstructor https://review.opendev.org/c/openstack/keystoneauth/+/935770 | |