| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2022-02-16 | |||
| 13:17:21 | opendevreview | Anton Sidelnikov proposed openstack/openstacksdk master: load_balancer proxy _find _delete now accept attrs https://review.opendev.org/c/openstack/openstacksdk/+/829540 | |
| 14:47:52 | opendevreview | Anton Sidelnikov proposed openstack/openstacksdk master: Change-Id: I6d156bf56e7b5618a1e84724480062d188f12c89 https://review.opendev.org/c/openstack/openstacksdk/+/829551 | |
| 15:03:39 | opendevreview | Anton Sidelnikov proposed openstack/openstacksdk master: load_balancer proxy _find _delete now accept attrs https://review.opendev.org/c/openstack/openstacksdk/+/829514 | |
| 15:08:50 | opendevreview | Merged openstack/ansible-collections-openstack master: Fix os_quota docs https://review.opendev.org/c/openstack/ansible-collections-openstack/+/829273 | |
| 16:06:27 | stephenfin | fuhrmannb[m]: It's not something that anyone has scoped. It's doable, but we'd need people to work on it. Would be a nice effort | |
| 16:10:11 | fuhrmannb[m] | Ok, that can be a topic I'm interested to work. I never contribute that deep in OpenStack but I can start to have a look. | |
| 16:13:30 | gtema | I fear amount of work is not going to be small, since you need to enable it everywhere, and SDK already depend on at least keystoneauth (as basic requests handling layer) where it would also require work | |
| 16:56:18 | fuhrmannb[m] | Clearly, all network related dependencies, like keystoneauth1, will require to support async too | |
| #openstack-sdks - 2022-02-17 | |||
| 03:32:39 | headphoneJames | gtema: if anyone has time for a review, please have a look at: https://review.opendev.org/c/openstack/openstacksdk/+/826193 Thanks! | |
| 10:00:09 | opendevreview | Vadym Markov proposed openstack/openstacksdk master: Make "find resource" case insensitive by default https://review.opendev.org/c/openstack/openstacksdk/+/829666 | |
| 10:04:29 | opendevreview | Vadym Markov proposed openstack/openstacksdk master: Make "find resource" case sensitive by default https://review.opendev.org/c/openstack/openstacksdk/+/829666 | |
| 10:08:16 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: Run jobs on stable branch https://review.opendev.org/c/openstack/ansible-collections-openstack/+/829530 | |
| 10:08:35 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack stable/1.0.0: Run jobs on stable branch https://review.opendev.org/c/openstack/ansible-collections-openstack/+/829638 | |
| 10:10:55 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack stable/1.0.0: DNM test branch jobs for collection https://review.opendev.org/c/openstack/ansible-collections-openstack/+/829668 | |
| 11:46:24 | frickler | why does "openstack volume list" need a compute endpoint? if I deploy cinder before nova and try to verify it by creating a volume, I get an error because the compute endpoint is missing | |
| 13:05:48 | opendevreview | Merged openstack/openstacksdk master: network: Fix update of network provider https://review.opendev.org/c/openstack/openstacksdk/+/761933 | |
| 13:22:54 | opendevreview | Thobias Trevisan proposed openstack/python-openstackclient master: compute: Add 'Security Groups' for 'server list' https://review.opendev.org/c/openstack/python-openstackclient/+/819368 | |
| 14:54:51 | stephenfin | frickler: I suspect that's a hangover from the days when volumes were a nova thing | |
| 15:04:16 | frickler | stephenfin: actually no, I just looked a bit, it's trying to pre-cache the server list in order to be able to show attachments more efficient. the generation of the cache is wrapped in a try, but not the creation of the client, I think that's an easy patch | |
| 15:09:20 | opendevreview | Dr. Jens Harbott proposed openstack/python-openstackclient master: volume list: don't fail when there's no compute service https://review.opendev.org/c/openstack/python-openstackclient/+/829725 | |
| 15:21:37 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack stable/1.0.0: Run jobs on stable branch https://review.opendev.org/c/openstack/ansible-collections-openstack/+/829638 | |
| 15:21:58 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack stable/1.0.0: DNM test branch jobs for collection https://review.opendev.org/c/openstack/ansible-collections-openstack/+/829668 | |
| 15:22:13 | isabek_ | stephenfin: Hi! Can I refactor to sdk properties network_fakes in OSC (example [1]) . Is there any task for this? 1) https://github.com/openstack/python-openstackclient/blob/master/openstackclient/tests/unit/network/v2/fakes.py#L116 | |
| 15:43:47 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack stable/1.0.0: Run jobs on stable branch https://review.opendev.org/c/openstack/ansible-collections-openstack/+/829638 | |
| 15:43:56 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack stable/1.0.0: DNM test branch jobs for collection https://review.opendev.org/c/openstack/ansible-collections-openstack/+/829668 | |
| 17:41:57 | stephenfin | isabek_: Go for it | |
| 17:42:25 | stephenfin | though of course, if you update the fakes you need to update the code that the fakes is testing to use the SDK :) | |
| #openstack-sdks - 2022-02-18 | |||
| 08:45:38 | thomasb06 | gtema: Hello. The file openstack/proxy.py is ready. To push it I need to `checkout master`, `pull origin master`, and then copy my file and commit it? | |
| 08:46:08 | gtema | yes, should be | |
| 08:46:18 | thomasb06 | let's go | |
| 09:14:44 | opendevreview | Thomas Bucaioni proposed openstack/openstacksdk master: Indentation of the docstrings https://review.opendev.org/c/openstack/openstacksdk/+/829862 | |
| 09:15:01 | thomasb06 | ah, it worked... | |
| 09:15:28 | thomasb06 | now I check in the UI, sometimes I miss some details | |
| 09:18:48 | thomasb06 | gtema: the other file is ressources? | |
| 09:19:36 | thomasb06 | more precisely, openstack/ressource.py | |
| 09:22:52 | gtema | there should be more. check proactively others | |
| 09:30:57 | thomasb06 | it's not the root only then? Understood | |
| 09:53:16 | opendevreview | Artem Goncharov proposed openstack/openstacksdk master: Merge 'origin/feature/r1' into master https://review.opendev.org/c/openstack/openstacksdk/+/829871 | |
| 10:48:25 | frickler | I haven't seen this in gerrit before, nice summary of what is happening https://review.opendev.org/c/openstack/openstacksdk/+/829871/-1..1//MERGE_LIST | |
| 12:04:02 | opendevreview | Mamatisa Nurmatov proposed openstack/python-openstackclient master: Refactor network fakes to sdk properties PART 1 https://review.opendev.org/c/openstack/python-openstackclient/+/829881 | |
| 13:22:10 | opendevreview | Mamatisa Nurmatov proposed openstack/python-openstackclient master: Refactor network fakes to sdk properties PART 1 https://review.opendev.org/c/openstack/python-openstackclient/+/829881 | |
| 13:33:49 | thomasb06 | gtema: Zuul returns an error, but I guess it's not me: https://review.opendev.org/c/openstack/openstacksdk/+/829862 | |
| 13:34:00 | thomasb06 | After a couple of days, I recheck? | |
| 13:37:53 | isabek_ | thomasb06: Hi! I think you can recheck it now, there some timeout | |
| 13:38:16 | thomasb06 | isabek_: great, thank you very much | |
| 13:39:45 | thomasb06 | since it's the main branch, I'm a bit cautious this time | |
| 13:47:52 | isabek_ | thomasb06: Welcome! Now job is success : https://zuul.openstack.org/status/change/829862,1 | |
| 13:48:18 | gtema | thomasb06 - all fine. This was a "normal" sporadic failure | |
| 13:49:33 | thomasb06 | isabek_: gtema: alright then, nothing is broken | |
| 14:49:30 | isabek_ | gtema: Hi! Can we remove tenant_id from network resources? example: https://github.com/openstack/openstacksdk/blob/master/openstack/network/v2/address_scope.py#L43 | |
| 14:50:12 | gtema | isabek_ for now we can not drop it completely | |
| 14:50:30 | gtema | we have old clients potentially still trying to pass it | |
| 14:51:01 | gtema | sdk is used by osc/ansible which depending on their version may also result in very old things still being passed | |
| 14:51:58 | isabek_ | At this moment I'm working on network_fakes to use sdk properties. https://review.opendev.org/c/openstack/python-openstackclient/+/829881 | |
| 14:52:38 | isabek_ | Because of tenant_id property is failing https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_72d/829881/2/check/osc-tox-py36-tips/72dc317/testr_results.html | |
| 14:53:26 | isabek_ | gtema: What do you recommend for it? Just fix unit tests? | |
| 14:56:56 | gtema | yes, tests need to be fixed. I actually always wanted that SDK also maintains "fakes" that can be then consumed by osc | |
| 14:57:44 | gtema | I really dislike current approach, especially that we will move more and more OSC stuff to sdk | |
| 15:01:14 | isabek_ | gtema: Can I simply hide with hidden_columns? | |
| 15:02:08 | gtema | honestly I feel like I was already fixing this | |
| 15:02:26 | gtema | https://review.opendev.org/c/openstack/python-openstackclient/+/824746 | |
| 15:02:33 | gtema | I am surprised you have it now | |
| 15:03:47 | isabek_ | Because now I'm using sdk properties where is tenant_id | |
| 15:04:15 | isabek_ | such as https://github.com/openstack/openstacksdk/blob/master/openstack/network/v2/address_scope.py#L43 | |
| 15:07:24 | gtema | and what was before? | |
| 15:08:27 | gtema | ah, before it was simply FakeResource | |
| 15:10:33 | isabek_ | yes | |
| 15:10:34 | gtema | also this function approach "create_availability_zone" is not how I think it should be in the long run | |
| 15:10:38 | gtema | I image | |
| 15:10:53 | gtema | from openstack.fakes.network.v2 import availability_zone | |
| 15:11:26 | gtema | but before those appear in sdk we can prepare things in osc. Afterwards we simply relocate them to sdk | |
| 15:12:29 | gtema | I would even start implementing it in SDK, but I have no feeling we can release that in next days/weeks. Rather few months still to finalise r1 changes | |
| 19:01:29 | opendevreview | James Kirsch proposed openstack/openstacksdk master: Identity: Add support for system role assignment https://review.opendev.org/c/openstack/openstacksdk/+/826193 | |
| 22:23:56 | 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-02-20 | |||
| 17:40:24 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack stable/1.0.0: Run jobs on stable branch https://review.opendev.org/c/openstack/ansible-collections-openstack/+/829638 | |
| 17:40:33 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack stable/1.0.0: DNM test branch jobs for collection https://review.opendev.org/c/openstack/ansible-collections-openstack/+/829668 | |
| 21:17:48 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack stable/1.0.0: Run jobs on stable branch https://review.opendev.org/c/openstack/ansible-collections-openstack/+/829638 | |
| 21:17:57 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack stable/1.0.0: DNM test branch jobs for collection https://review.opendev.org/c/openstack/ansible-collections-openstack/+/829668 | |
| #openstack-sdks - 2022-02-21 | |||
| 11:00:04 | opendevreview | Dr. Jens Harbott proposed openstack/osc-lib master: Fixed test for interface key https://review.opendev.org/c/openstack/osc-lib/+/828876 | |
| 11:16:17 | opendevreview | Dr. Jens Harbott proposed openstack/osc-lib master: Don't fail on extlinks warnings for reno build https://review.opendev.org/c/openstack/osc-lib/+/830147 | |
| 11:17:36 | frickler | gtema: ^^ same issue as for osc, noticed during publishing tag 2.5.0, see the discussion in #-release | |
| 11:19:07 | frickler | if we are going to do another release, it would be nice to get https://review.opendev.org/c/openstack/osc-lib/+/784559/2 and https://review.opendev.org/c/openstack/osc-lib/+/822764 in, too | |
| 11:24:54 | opendevreview | Dr. Jens Harbott proposed openstack/osc-lib master: Don't fail on extlinks warnings for reno build https://review.opendev.org/c/openstack/osc-lib/+/830147 | |
| 12:20:27 | opendevreview | Merged openstack/osc-lib master: Respect 'interface' key from clouds.yaml https://review.opendev.org/c/openstack/osc-lib/+/784559 | |
| 12:20:34 | opendevreview | Merged openstack/osc-lib master: Fix TestTagHelps for python3.10 https://review.opendev.org/c/openstack/osc-lib/+/822764 | |
| 12:20:37 | opendevreview | Merged openstack/osc-lib master: Don't fail on extlinks warnings for reno build https://review.opendev.org/c/openstack/osc-lib/+/830147 | |
| 12:21:59 | opendevreview | Mamatisa Nurmatov proposed openstack/python-openstackclient master: Refactor network fakes to sdk properties PART 1 https://review.opendev.org/c/openstack/python-openstackclient/+/829881 | |
| 12:31:34 | jm1 | frickler gtema: wuuuhuuu finally the interface keyword works again. thank you both for merging my patches :) | |
| 15:13:29 | opendevreview | Grzegorz Grasza proposed openstack/keystoneauth master: Fix bindep for current rpm based distributions https://review.opendev.org/c/openstack/keystoneauth/+/830211 | |
| 17:16:38 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack stable/1.0.0: DNM test override master https://review.opendev.org/c/openstack/ansible-collections-openstack/+/830262 | |
| 17:16:39 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack stable/1.0.0: DNM test jobs https://review.opendev.org/c/openstack/ansible-collections-openstack/+/830263 | |
| 21:52:43 | opendevreview | Slawek Kaplonski proposed openstack/python-openstackclient master: Add support for setting extra DHCP options on existing ports https://review.opendev.org/c/openstack/python-openstackclient/+/830289 | |
| #openstack-sdks - 2022-02-22 | |||
| 09:41:12 | opendevreview | Arx Cruz proposed openstack/ansible-collections-openstack master: Update compute service info module to use proxy layer https://review.opendev.org/c/openstack/ansible-collections-openstack/+/828698 | |
| 09:53:18 | opendevreview | Mamatisa Nurmatov proposed openstack/python-openstackclient master: Fix metavars and typos in local_ip https://review.opendev.org/c/openstack/python-openstackclient/+/830342 | |
| 11:38:38 | opendevreview | Mamatisa Nurmatov proposed openstack/openstacksdk master: Add network address_group proxy doc and unit tests https://review.opendev.org/c/openstack/openstacksdk/+/830380 | |
| 12:49:48 | opendevreview | Mamatisa Nurmatov proposed openstack/openstacksdk master: Add network address_group proxy doc and unit tests https://review.opendev.org/c/openstack/openstacksdk/+/830380 | |
| 13:00:46 | opendevreview | Florian Haas proposed openstack/openstacksdk master: Add get_*_by_id() methods for Magnum clusters and templates https://review.opendev.org/c/openstack/openstacksdk/+/828791 | |
| 17:23:59 | Guest7 | So I'm actually in Mexico ATM so I can't make it to the meeting today stephenfin and gtema | |
| 17:25:04 | stephenfin | Should be fine | |
| 17:25:21 | gtema | yupp | |