| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2023-02-02 | |||
| 17:09:22 | umbSublime | For reference using those 2 envvars work as expected with using the openstackclient | |
| 17:11:17 | gtema | umbSublime: it is always recommended to be explicit. afair once sdk finds entry in clouds.yaml it does not merge env vars into it | |
| 17:12:52 | umbSublime | gtema: good to know, since the envvar OS_CLOUD would need to read the config, I was expecting OS_REGION_NAME to be merged after. This appears to be how the openstackclient works. Would there be additional logic in the osc to handle this ? | |
| 17:14:15 | umbSublime | gtema: I agree explicit is always better, but having openstack.connect() work as I was expecting saves me from adding logic to handle those from argparse (or other). | |
| 17:14:58 | gtema | yes, OSC is doing things bit differently | |
| 17:22:52 | umbSublime | gtema: thanks) | |
| 17:23:18 | umbSublime | Simple work around for my specific usecase: openstack.connect(region_name=os.environ.get('OS_REGION_NAME', None)); c.current_location.region_name | |
| 17:25:10 | gtema | or duplicating entry in clouds.yaml - afaik there are some clouds not really sharing identity between regions so you anyway need to use different user | |
| 17:25:52 | umbSublime | agreed, that's another solution. In my case however there are 25+ regions all with same auth. | |
| 17:26:06 | gtema | ok, convinced | |
| #openstack-sdks - 2023-02-03 | |||
| 05:50:56 | opendevreview | Mridula Joshi proposed openstack/openstacksdk master: Add support for glance cache list https://review.opendev.org/c/openstack/openstacksdk/+/869710 | |
| 19:27:57 | opendevreview | Rajat Dhasmana proposed openstack/python-openstackclient master: Add block storage manageable list commands https://review.opendev.org/c/openstack/python-openstackclient/+/872692 | |
| #openstack-sdks - 2023-02-06 | |||
| 17:40:48 | opendevreview | Rajat Dhasmana proposed openstack/python-openstackclient master: Add options to create volume group from source https://review.opendev.org/c/openstack/python-openstackclient/+/872798 | |
| #openstack-sdks - 2023-02-07 | |||
| 20:40:46 | opendevreview | Goutham Pacha Ravi proposed openstack/openstacksdk master: Add share access rules to shared file system https://review.opendev.org/c/openstack/openstacksdk/+/784162 | |
| 20:57:46 | opendevreview | Goutham Pacha Ravi proposed openstack/openstacksdk master: Add share types to shared file systems. https://review.opendev.org/c/openstack/openstacksdk/+/783123 | |
| #openstack-sdks - 2023-02-08 | |||
| 12:42:48 | opendevreview | Carlos Eduardo proposed openstack/openstacksdk master: Add share instances to shared file systems https://review.opendev.org/c/openstack/openstacksdk/+/873109 | |
| #openstack-sdks - 2023-02-09 | |||
| 12:01:12 | opendevreview | Johannes Beisiegel proposed openstack/openstacksdk master: fix docstrings refering to volume attachments instead of server migrations https://review.opendev.org/c/openstack/openstacksdk/+/873245 | |
| 16:40:42 | opendevreview | Artom Lifshitz proposed openstack/python-openstackclient master: Support compute microversion 2.94 https://review.opendev.org/c/openstack/python-openstackclient/+/872420 | |
| 16:51:20 | opendevreview | Artom Lifshitz proposed openstack/python-openstackclient master: Support compute microversion 2.94 https://review.opendev.org/c/openstack/python-openstackclient/+/872420 | |
| 20:03:19 | opendevreview | Ashley Rodriguez proposed openstack/openstacksdk master: [WIP] add share replica export location to shared file system https://review.opendev.org/c/openstack/openstacksdk/+/873323 | |
| 20:06:31 | opendevreview | Ashley Rodriguez proposed openstack/openstacksdk master: [WIP] add share replica export location to shared file system https://review.opendev.org/c/openstack/openstacksdk/+/873323 | |
| 20:10:36 | opendevreview | Ashley Rodriguez proposed openstack/openstacksdk master: [WIP] Add share replica resource to shared file system https://review.opendev.org/c/openstack/openstacksdk/+/873324 | |
| 20:20:01 | opendevreview | Ashley Rodriguez proposed openstack/openstacksdk master: [WIP] Add share instance export location resource to shared file system https://review.opendev.org/c/openstack/openstacksdk/+/873327 | |
| 20:22:38 | opendevreview | Ashley Rodriguez proposed openstack/openstacksdk master: [WIP] Adds service resource to shared file system https://review.opendev.org/c/openstack/openstacksdk/+/873328 | |
| 20:24:01 | opendevreview | Ashley Rodriguez proposed openstack/openstacksdk master: [WIP] Adds Security Service to shared file system https://review.opendev.org/c/openstack/openstacksdk/+/873329 | |
| #openstack-sdks - 2023-02-10 | |||
| 09:44:37 | opendevreview | Hiromu Asahina proposed openstack/keystoneauth master: OAuth 2.0 Mutual-TLS Support https://review.opendev.org/c/openstack/keystoneauth/+/860614 | |
| 11:43:11 | opendevreview | Carlos Eduardo proposed openstack/openstacksdk master: Add share network resource to shared file system https://review.opendev.org/c/openstack/openstacksdk/+/873363 | |
| 11:52:14 | opendevreview | Carlos Eduardo proposed openstack/openstacksdk master: Add share snapshot instance resource https://review.opendev.org/c/openstack/openstacksdk/+/873364 | |
| 11:53:14 | opendevreview | Carlos Eduardo proposed openstack/openstacksdk master: Add share network subnet resource to shared file system https://review.opendev.org/c/openstack/openstacksdk/+/873365 | |
| 11:56:55 | opendevreview | Hiromu Asahina proposed openstack/keystoneauth master: OAuth 2.0 Mutual-TLS Support https://review.opendev.org/c/openstack/keystoneauth/+/860614 | |
| 14:01:12 | opendevreview | Carlos Eduardo proposed openstack/openstacksdk master: Add export location resource to shared file system https://review.opendev.org/c/openstack/openstacksdk/+/780990 | |
| 14:21:20 | opendevreview | Carlos Eduardo proposed openstack/openstacksdk master: Add share snapshot instance resource https://review.opendev.org/c/openstack/openstacksdk/+/873364 | |
| 18:46:31 | opendevreview | Ashley Rodriguez proposed openstack/openstacksdk master: Add share instances to shared file systems https://review.opendev.org/c/openstack/openstacksdk/+/873109 | |
| 18:49:07 | opendevreview | Ashley Rodriguez proposed openstack/openstacksdk master: Add share network resource to shared file system https://review.opendev.org/c/openstack/openstacksdk/+/873363 | |
| #openstack-sdks - 2023-02-13 | |||
| 10:03:06 | opendevreview | Marvin Vogt proposed openstack/openstacksdk master: Remove usage of deprecated `sre_constants` module https://review.opendev.org/c/openstack/openstacksdk/+/873521 | |
| 10:07:29 | opendevreview | Marvin Vogt proposed openstack/openstacksdk master: Remove usage of deprecated `sre_constants` module https://review.opendev.org/c/openstack/openstacksdk/+/873521 | |
| 16:02:37 | whoami-rajat | stephenfin, hey, around ? | |
| 16:03:34 | whoami-rajat | stephenfin, wanted to mention that I've proposed patches for all missing cinder commands, would be good if we can get it in this cycle's OSC release (this week) https://review.opendev.org/q/topic:add-missing-cinderclient-commands | |
| 16:04:02 | stephenfin | yup, back today. I'll take a look (y) | |
| 20:12:38 | opendevreview | Ashley Rodriguez proposed openstack/openstacksdk master: Add share snapshot instance resource https://review.opendev.org/c/openstack/openstacksdk/+/798464 | |
| 20:15:23 | opendevreview | Ashley Rodriguez proposed openstack/openstacksdk master: Add share network subnet resource to shared file system https://review.opendev.org/c/openstack/openstacksdk/+/783582 | |
| #openstack-sdks - 2023-02-14 | |||
| 02:35:58 | gmann | gtema: just in case you missed, next cycle PTL nomination are closing on Feb 15 | |
| 12:42:12 | opendevreview | Mridula Joshi proposed openstack/openstacksdk master: Add support for glance cache https://review.opendev.org/c/openstack/openstacksdk/+/869710 | |
| 14:22:05 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: DNM: check CI run https://review.opendev.org/c/openstack/ansible-collections-openstack/+/873713 | |
| 15:46:05 | stephenfin | whoami-rajat: Done. You'll need to ask gtema if you want a second +2/+W on the ones I've approved | |
| 15:46:27 | stephenfin | I think he also accepts 🍺 as part payment | |
| 15:46:32 | stephenfin | :) | |
| 16:42:35 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: volume: Remove duplication from 'consistency group create' opts https://review.opendev.org/c/openstack/python-openstackclient/+/873730 | |
| 19:36:57 | whoami-rajat | stephenfin, sure, thanks for all your reviews :) | |
| 19:53:58 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: Convert recordset module to use StateMachine https://review.opendev.org/c/openstack/ansible-collections-openstack/+/873747 | |
| 20:55:20 | opendevreview | Merged openstack/openstacksdk master: Add support for glance cache https://review.opendev.org/c/openstack/openstacksdk/+/869710 | |
| #openstack-sdks - 2023-02-15 | |||
| 01:11:28 | opendevreview | Reynaldo proposed openstack/openstacksdk master: Add share snapshot instance resource https://review.opendev.org/c/openstack/openstacksdk/+/798464 | |
| 08:19:14 | frickler | gtema: could you add https://review.opendev.org/c/openstack/python-openstackclient/+/864497 to your review list please? | |
| 08:19:28 | gtema | ack | |
| 08:20:32 | gtema | frickler - we were talking with you about naming: "force" is a bad idea from my pov | |
| 08:24:13 | frickler | gtema: so what would you prefer instead? "--no-confirm"? | |
| 08:26:16 | gtema | yes, maybe, or --assume-yes or --auto-approve (like in TF) | |
| 08:27:20 | frickler | the other option would be to make the whole thing non-interactive and just have a "--dry-run" option for the timid | |
| 08:32:54 | gtema | hmm, this is too dangerous in my eyes | |
| 08:34:58 | frickler | o.k., so I'll amend to use --auto-approve | |
| 08:35:13 | gtema | cool, thanks | |
| 08:43:15 | opendevreview | Dr. Jens Harbott proposed openstack/python-openstackclient master: Add auto-approve option to project cleanup https://review.opendev.org/c/openstack/python-openstackclient/+/864497 | |
| 09:10:57 | frickler | seems I totally missed 1.0.0, was there a good party? | |
| 10:27:10 | gtema | frickler - a party was sadly in mind only | |
| 10:56:19 | lajoskatona | gtema, frickler: just a note, sdk 1.0.0 is still not in use, as u-c bump is not merged: https://review.opendev.org/c/openstack/requirements/+/872282 | |
| 11:00:42 | frickler | lajoskatona: yes, though we actually have 1.0.1 now. but requirements CI is broken as well as kuryr like you mentioned | |
| 11:01:48 | frickler | you could try to depend your neutronclient patch on the u-c bump, though | |
| 11:03:59 | lajoskatona | frickler: yeah, but anyway we can't merge (and release) it without having correct u-c.txt | |
| 16:46:35 | opendevreview | Rodolfo Alonso proposed openstack/openstacksdk master: Include "security_groups" to "Port" query parameters https://review.opendev.org/c/openstack/openstacksdk/+/873946 | |
| 16:51:22 | opendevreview | Rodolfo Alonso proposed openstack/openstacksdk master: Include "security_groups" to "Port" query parameters https://review.opendev.org/c/openstack/openstacksdk/+/873946 | |
| 19:27:12 | opendevreview | Elod Illes proposed openstack/cliff master: DNM: dummy change to test gate health https://review.opendev.org/c/openstack/cliff/+/873979 | |
| 19:30:04 | opendevreview | Elod Illes proposed openstack/osc-lib master: DNM: dummy change to test gate health https://review.opendev.org/c/openstack/osc-lib/+/873986 | |
| #openstack-sdks - 2023-02-16 | |||
| 00:58:59 | opendevreview | Goutham Pacha Ravi proposed openstack/openstacksdk master: Remove "feature/r1" override from manila job https://review.opendev.org/c/openstack/openstacksdk/+/874046 | |
| 06:23:50 | opendevreview | Reynaldo proposed openstack/openstacksdk master: Add share snapshot instance resource https://review.opendev.org/c/openstack/openstacksdk/+/798464 | |
| 09:01:52 | opendevreview | Rajat Dhasmana proposed openstack/python-openstackclient master: Add volume revert command https://review.opendev.org/c/openstack/python-openstackclient/+/869654 | |
| 09:01:53 | opendevreview | Rajat Dhasmana proposed openstack/python-openstackclient master: Add block storage log level {list, set} commands https://review.opendev.org/c/openstack/python-openstackclient/+/869811 | |
| 09:01:53 | opendevreview | Rajat Dhasmana proposed openstack/python-openstackclient master: Add block storage cleanup command https://review.opendev.org/c/openstack/python-openstackclient/+/872390 | |
| 09:01:54 | opendevreview | Rajat Dhasmana proposed openstack/python-openstackclient master: Add block storage manageable list commands https://review.opendev.org/c/openstack/python-openstackclient/+/872692 | |
| 09:01:54 | opendevreview | Rajat Dhasmana proposed openstack/python-openstackclient master: Add options to create volume group from source https://review.opendev.org/c/openstack/python-openstackclient/+/872798 | |
| 09:12:20 | whoami-rajat | stephenfin, gtema updated all the patches ^ | |
| 10:27:12 | opendevreview | Stephen Finucane proposed openstack/osc-lib master: tox 4 compatibility https://review.opendev.org/c/openstack/osc-lib/+/874075 | |
| 11:00:45 | frickler | gtema: stephenfin: would one of you have time to look into https://zuul.opendev.org/t/openstack/build/284499a2edc041f78d6b94dbee1e1e07 ? this is blocking getting sdk 1.0.1 into u-c (not the only blocker, but kuryr at least has a fix proposed) | |
| 11:01:54 | gtema | frickler, ack, looking | |
| 11:02:50 | gtema | but I am not sure whether this is OSC issue or octaviaclient | |
| 11:30:41 | gtema | frickler - I think I found a direction. python-openstackclient is not installing python-octaviaclient through requirements.txt. It is done in doc/requirements.txt | |
| 11:31:31 | gtema | but I am not sure this is respected in the requirements job, cause according to the log python-octaviaclient is not visible in the packages installed into the tox venv | |
| 11:33:20 | gtema | at least it does not look like it is there and if I remove it locally in the docs venv of OSC I get the same error | |
| 12:05:58 | frickler | gtema: ah, so I'll check the job definition in the reqs repo, thx for looking | |
| 12:09:02 | gtema | frickler, since this was running before I assume this might be something new in toxv4 as well. Logs show octaviaclient is installed (into some env), but during the build it is just not there where expected | |
| 12:46:22 | frickler | yes, this is weird. the job is only failing for the sdk bump, not for other updates, so there must be some connection. let me try with a dep on osc | |
| 12:57:09 | frickler | I can reproduce the error locally if I manually update openstacksdk in the venv or point to the updated u-c file. but I have no idea why this is happening. also not related to tox4, can repro with tox3, too | |
| 12:58:51 | opendevreview | Dr. Jens Harbott proposed openstack/python-openstackclient master: DNM: Test sdk update https://review.opendev.org/c/openstack/python-openstackclient/+/874095 | |
| 13:01:22 | gtema | Weird | |
| 13:02:51 | gtema | What we had figured out for Octaviaclient is that in SDK 1.0 we dropped munch from dependencies | |
| 13:02:55 | gtema | I added it i to requirements there and it was merged | |
| 13:03:50 | frickler | ah, maybe the bump for octaviaclient needs to be merged first, then | |
| 13:04:23 | gtema | Maybe. But that is weird that in this case it causes such a hidden failure | |
| 13:05:13 | gtema | Can you try where you reporoduced the issue to manually install munch and check whether it works then or not | |
| 13:05:14 | gtema | It may be the pip issue and not tox | |
| 13:07:48 | frickler | I just had the same idea, with much installed into the docs venv, the issue goes away | |