| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2022-02-17 | |||
| 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 | |
| 17:27:38 | Guest229 | Also seriously struggling with nickserv today | |
| 17:28:24 | gtema | so many names in one day? | |
| 17:29:19 | Guest229 | Yeah.. | |
| 17:31:07 | gtema | are we able to join meeting without you? | |
| 17:35:59 | Guest229 | Uhh idk | |
| 17:36:02 | Guest229 | Can you? | |
| 17:36:18 | Guest229 | I can try to hop in to let you if you're not able to? | |
| 17:39:22 | gtema | we can't, but stephenfin already sent email and Ryan anyway wrote he resigns | |
| 17:43:18 | Guest229 | Oh. Shit. Okay. | |
| 17:44:50 | 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 | |
| #openstack-sdks - 2022-02-23 | |||
| 07:55:29 | opendevreview | Merged openstack/python-openstackclient master: Add support for 'remote-managed' vnic type https://review.opendev.org/c/openstack/python-openstackclient/+/814518 | |
| 10:05:47 | opendevreview | Arx Cruz proposed openstack/ansible-collections-openstack master: Move compute flavor info to use proxy layer https://review.opendev.org/c/openstack/ansible-collections-openstack/+/828108 | |
| 10:09:33 | 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 | |
| 18:11:59 | 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 | |
| 18:12:51 | 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 | |
| 18:13:26 | 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 | |