| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2022-09-13 | |||
| 07:14:24 | frickler | I thinking whether this should better move under TC umbrella nowadays | |
| 07:14:25 | gtema | tja, let's wait wat dtantsur responds and eventually consider adoption. I was part of the team, but by that time there was already no reason for cores to exist | |
| 07:15:23 | gtema | since it relates to os-service-types repo I would rather think to merge those 2 and leave them under sdk, since we are the main consumer of this type of data | |
| 07:16:33 | frickler | not sure about that, it is currently blocking api-ref publishing for venus, so at least some wider impact. | |
| 07:17:00 | gtema | hmm, maybe right | |
| 09:14:18 | dtantsur | frickler: the API SIG is no longer active, but I do still have core rights | |
| 09:15:09 | dtantsur | somebody needs to fix the CI there.. | |
| 10:35:51 | opendevreview | Polina Gubina proposed openstack/openstacksdk master: add missing function to identity resource https://review.opendev.org/c/openstack/openstacksdk/+/857045 | |
| 11:45:59 | opendevreview | Dr. Jens Harbott proposed openstack/service-types-authority master: Set py_modules to an empty list in setup.py https://review.opendev.org/c/openstack/service-types-authority/+/857349 | |
| 11:48:30 | opendevreview | Dr. Jens Harbott proposed openstack/service-types-authority master: Add Venus as log-management service https://review.opendev.org/c/openstack/service-types-authority/+/857080 | |
| 12:05:06 | frickler | dtantsur: ^^ seems like that was an easy fix, stolen from other projects which had the same issue some time ago. | |
| 12:05:15 | frickler | brinzhang: added a comment in the venus patch | |
| 12:06:13 | gtema | yeah, this "fix" is slowly spreading across other projects | |
| 12:07:03 | opendevreview | Ananya proposed openstack/ansible-collections-openstack master: Makes stack module compatible with new sdk version https://review.opendev.org/c/openstack/ansible-collections-openstack/+/848266 | |
| 12:07:04 | frickler | gtema: that sounds as if you don't like it, I'm open to better proposals | |
| 12:07:21 | gtema | nono, don't mean that. | |
| 12:08:09 | gtema | I mean it feels rather hack to an artificial problem. I clearly admit there is no other way and I just observe it being applied quite often | |
| 12:25:04 | opendevreview | Dr. Jens Harbott proposed openstack/service-types-authority master: Update api-ref for cyborg https://review.opendev.org/c/openstack/service-types-authority/+/857367 | |
| 12:26:27 | frickler | ^^ that should be the fixup for https://review.opendev.org/c/openstack/os-service-types/+/856373 | |
| 12:30:21 | dtantsur | frickler: approving; the venus patch has an outstanding comment | |
| 12:31:23 | opendevreview | Merged openstack/openstacksdk master: added api requirements for new "openstack image metadefs namespace list" command https://review.opendev.org/c/openstack/openstacksdk/+/854612 | |
| 12:31:26 | opendevreview | Merged openstack/openstacksdk master: image: Trivial grouping of image proxy methods https://review.opendev.org/c/openstack/openstacksdk/+/856270 | |
| 12:33:09 | frickler | dtantsur: yes, that comment is mine, too. if you agree with it I can fixup the patch since I think brinzhang may have eod'd by now | |
| 12:33:51 | frickler | or I can do a follow-up, then we can see the publishing actually succeed in between | |
| 12:34:15 | dtantsur | frickler: yeah, a follow-up is probably fine (and I wonder how many wrong links we have there already - ironic-inspector is one) | |
| 12:34:45 | frickler | I just wanted to avoid generating too many revisions, but seeing what happened in 2019 that's maybe moot | |
| 12:35:03 | gtema | dtantsur, maybe you remember where it is consumed and whether we can merge it with os-service-types? | |
| 12:35:39 | dtantsur | gtema: I would assume it's consumed everywhere where machine-friendly information is needed. os-service-type is python, we need to be careful around it. | |
| 12:36:42 | gtema | well, json (inside) is machine friendly, so we could just eventually add additional interface for that | |
| 12:37:06 | gtema | cause I really hate having 2 projects with same data but serving 2 interfaces | |
| 12:39:57 | opendevreview | Merged openstack/service-types-authority master: Set py_modules to an empty list in setup.py https://review.opendev.org/c/openstack/service-types-authority/+/857349 | |
| 12:51:39 | opendevreview | Kristian Kucerak proposed openstack/ansible-collections-openstack master: Change subnets_info module to work with new SDK https://review.opendev.org/c/openstack/ansible-collections-openstack/+/857041 | |
| 13:17:22 | opendevreview | Dr. Jens Harbott proposed openstack/service-types-authority master: Drop override for venus api-ref https://review.opendev.org/c/openstack/service-types-authority/+/857430 | |
| 13:25:03 | opendevreview | Merged openstack/service-types-authority master: Update api-ref for cyborg https://review.opendev.org/c/openstack/service-types-authority/+/857367 | |
| 13:33:17 | frickler | another job that needs fixing, I almost expected that to fail https://zuul.opendev.org/t/openstack/build/acfdf5109f4044b2992c1389eade3045 | |
| 13:34:32 | frickler | it even failed the last time it ran before today in 2019, not a good sign https://zuul.opendev.org/t/openstack/builds?job_name=propose-updates&project=openstack/service-types-authority | |
| 13:35:06 | gtema | tja | |
| 13:39:39 | opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack master: Update quota for 2.0.0 https://review.opendev.org/c/openstack/ansible-collections-openstack/+/851483 | |
| 14:15:52 | opendevreview | Eunyoung Kim proposed openstack/openstacksdk master: Add metadef schema resource to v2 metadefs api https://review.opendev.org/c/openstack/openstacksdk/+/857200 | |
| 14:18:27 | opendevreview | Eunyoung Kim proposed openstack/openstacksdk master: Add metadef schema resource to v2 metadefs api https://review.opendev.org/c/openstack/openstacksdk/+/857200 | |
| 15:30:14 | opendevreview | Eunyoung Kim proposed openstack/openstacksdk master: Add metadef schema resource to v2 metadefs api https://review.opendev.org/c/openstack/openstacksdk/+/857200 | |
| 15:36:40 | opendevreview | Merged openstack/service-types-authority master: Add Venus as log-management service https://review.opendev.org/c/openstack/service-types-authority/+/857080 | |
| 15:40:48 | opendevreview | OpenStack Proposal Bot proposed openstack/os-service-types master: Updated from OpenStack Service Type Authority https://review.opendev.org/c/openstack/os-service-types/+/857466 | |
| 15:42:00 | frickler | gtema: dtantsur: ^^ seems that worked out in the end | |
| 15:42:21 | dtantsur | nice! | |
| 15:43:05 | gtema | jut | |
| 16:01:17 | frickler | it looks like this might need some help with the formatting, but otherwise it finally worked https://docs.openstack.org/api-ref/log-management/ | |
| 16:04:08 | opendevreview | Rajat Dhasmana proposed openstack/python-openstackclient master: compute: Add support for microversion 2.93 https://review.opendev.org/c/openstack/python-openstackclient/+/831014 | |
| 16:06:44 | whoami-rajat | stephenfin, hey, I've updated the patch, please take a look when you get some time https://review.opendev.org/c/openstack/python-openstackclient/+/831014 | |
| 16:08:16 | stephenfin | ack | |
| 17:02:03 | opendevreview | sean mooney proposed openstack/openstacksdk master: Revert "Fix incremental backups handling in project cleanup" https://review.opendev.org/c/openstack/openstacksdk/+/857471 | |
| 17:19:39 | opendevreview | sean mooney proposed openstack/openstacksdk master: Revert "Fix incremental backups handling in project cleanup" https://review.opendev.org/c/openstack/openstacksdk/+/857471 | |
| 17:43:28 | opendevreview | Merged openstack/keystoneauth stable/zed: Update .gitreview for stable/zed https://review.opendev.org/c/openstack/keystoneauth/+/856793 | |
| 17:45:52 | opendevreview | Merged openstack/keystoneauth stable/zed: Update TOX_CONSTRAINTS_FILE for stable/zed https://review.opendev.org/c/openstack/keystoneauth/+/856794 | |
| 19:12:38 | opendevreview | Merged openstack/os-service-types master: Updated from OpenStack Service Type Authority https://review.opendev.org/c/openstack/os-service-types/+/857466 | |
| 21:33:13 | opendevreview | Polina Gubina proposed openstack/openstacksdk master: add missing function to identity resource https://review.opendev.org/c/openstack/openstacksdk/+/857045 | |
| 21:38:36 | opendevreview | Polina Gubina proposed openstack/openstacksdk master: add missing function to identity resource https://review.opendev.org/c/openstack/openstacksdk/+/857045 | |
| 21:39:57 | opendevreview | Polina Gubina proposed openstack/openstacksdk master: add missing function to identity resource https://review.opendev.org/c/openstack/openstacksdk/+/857045 | |
| #openstack-sdks - 2022-09-14 | |||
| 02:37:33 | opendevreview | Eunyoung Kim proposed openstack/openstacksdk master: Add metadef schema resource to v2 metadefs api https://review.opendev.org/c/openstack/openstacksdk/+/857200 | |
| 03:10:41 | opendevreview | Eunyoung Kim proposed openstack/openstacksdk master: Add metadef schema resource to v2 metadefs api https://review.opendev.org/c/openstack/openstacksdk/+/857200 | |
| 03:47:42 | opendevreview | Eunyoung Kim proposed openstack/openstacksdk master: Add metadef schema resource to v2 metadefs api https://review.opendev.org/c/openstack/openstacksdk/+/857200 | |
| 07:01:49 | opendevreview | Polina Gubina proposed openstack/openstacksdk master: add missing function to identity resource https://review.opendev.org/c/openstack/openstacksdk/+/857045 | |
| 07:58:14 | seongsoocho | stephenfin: I found that you already have implemented the image namespace feature in openstacksdk. ( https://review.opendev.org/c/openstack/openstacksdk/+/856271 ) My mentee also doing same work now, So How about adding image namespace cli to openstackclient by my mentee? ( If it's not urgent to add this feature.. ) | |
| 07:59:05 | gtema | seongsoocho - this would be good | |
| 08:00:28 | seongsoocho | gtema: Thanks. Currently, we are working with image namespace, property, object to add in sdk and cli | |
| 09:18:23 | opendevreview | Will Szumski proposed openstack/ansible-collections-openstack master: Allow networks to be updated https://review.opendev.org/c/openstack/ansible-collections-openstack/+/854170 | |
| 09:18:50 | opendevreview | Ghanshyam proposed openstack/keystoneauth master: Switch to 2023.1 Python3 unit tests and generic template name https://review.opendev.org/c/openstack/keystoneauth/+/856796 | |
| 10:25:05 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: compute: Require image when rebuilding a volume-backed server https://review.opendev.org/c/openstack/python-openstackclient/+/857630 | |
| 10:25:05 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: compute: Add support for microversion 2.93 https://review.opendev.org/c/openstack/python-openstackclient/+/831014 | |
| 10:31:12 | stephenfin | seongsoocho: Yes, please do. Someone has already implemented 'openstack metadef namespace list' but the other CRUD operations need to be implemented | |
| 10:31:38 | stephenfin | seongsoocho: You can build on top of this https://review.opendev.org/c/openstack/python-openstackclient/+/854557 | |
| 10:35:16 | stephenfin | *'openstack image metadef namespace list' | |
| 10:35:25 | stephenfin | quite the mouthful :) | |
| 10:36:51 | stephenfin | whoami-rajat: +W on that patch. I split it into two so we can backport the bit that insists on an '--image' parameter if rebuilding a BFV instance since that was a clear bug (as opposed to a new feature). Kept you as author on both though. I hope you don't mind /o\ | |
| 10:37:26 | stephenfin | gtema: Actually, maybe you want to look at it also. Up to you. https://review.opendev.org/c/openstack/python-openstackclient/+/857630/ and https://review.opendev.org/c/openstack/python-openstackclient/+/831014/ | |
| 10:39:06 | whoami-rajat | stephenfin, sure, works for me. thanks! | |
| 10:52:40 | seongsoocho | stephenfin: thank you. My mentee trying to fix a test failing with that review(854557). Thank you for the opportunity . | |
| 12:16:43 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: tests: Improve functional testing for image methods https://review.opendev.org/c/openstack/openstacksdk/+/856272 | |
| 12:16:43 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: image: Add support for other metadef namespace operations https://review.opendev.org/c/openstack/openstacksdk/+/856271 | |
| 12:16:44 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: docs: Add missing docs for proxy helper methods https://review.opendev.org/c/openstack/openstacksdk/+/857655 | |
| 12:17:51 | dtantsur | stephenfin: hi! would you be able to check the 3 patches starting with https://review.opendev.org/c/openstack/openstacksdk/+/849564 please? Artem has already given his +2. | |
| 12:21:26 | stephenfin | dtantsur: sure | |
| 12:29:53 | opendevreview | Merged openstack/python-openstackclient master: compute: Require image when rebuilding a volume-backed server https://review.opendev.org/c/openstack/python-openstackclient/+/857630 | |
| 12:44:13 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: volume: Volume names are optional https://review.opendev.org/c/openstack/python-openstackclient/+/857670 | |
| 12:57:40 | jm1 | gtema: hello :) didn't you say that new python-openstackclient works fine with older openstacksdk releases? it does not XP >0.99.0? | |
| 12:57:54 | jm1 | gtema: hello :) didn't you say that new python-openstackclient works fine with older openstacksdk releases? it does not XP https://logserver.rdoproject.org/62/44962/1/check/DLRN-rpmbuild-centos9/ad3f587/job-output.txt | |
| 13:01:47 | gtema | U, that is a surprise for me | |
| 13:13:06 | jm1 | gtema: now its getting really a headache for us.. | |
| 13:13:41 | jm1 | gtema: aoc/tripleo are not ready for openstacksdk >0.99.0 yet | |
| 13:13:52 | frickler | ndp_proxy is a recent addition. might want to make that conditional | |
| 13:16:40 | jm1 | frickler, gtema: whatever it takes ;) else we have to pin osc in rhos as well. that would not be cool | |
| 13:26:16 | opendevreview | Merged openstack/python-openstackclient master: compute: Add support for microversion 2.93 https://review.opendev.org/c/openstack/python-openstackclient/+/831014 | |
| 13:44:32 | opendevreview | Eunyoung Kim proposed openstack/openstacksdk master: workflow: Trivial fix doc title https://review.opendev.org/c/openstack/openstacksdk/+/857700 | |
| 13:54:36 | opendevreview | Merged openstack/openstacksdk master: Actually use openstacksdk from source in the Bifrost job https://review.opendev.org/c/openstack/openstacksdk/+/849564 | |
| 15:31:43 | opendevreview | jihyun huh proposed openstack/openstacksdk master: Add support for other metadef property operations https://review.opendev.org/c/openstack/openstacksdk/+/857727 | |
| 15:31:54 | opendevreview | KIM SOJUNG proposed openstack/python-openstackclient master: Adding missing command mapping in docs https://review.opendev.org/c/openstack/python-openstackclient/+/857728 | |
| 16:48:22 | opendevreview | EunYoung Kim proposed openstack/openstacksdk master: Add metadef schema resource to v2 metadefs api https://review.opendev.org/c/openstack/openstacksdk/+/857200 | |
| 16:50:36 | opendevreview | Merged openstack/openstacksdk master: workflow: Trivial fix doc title https://review.opendev.org/c/openstack/openstacksdk/+/857700 | |
| 16:50:39 | opendevreview | Merged openstack/openstacksdk master: docs: Add missing docs for proxy helper methods https://review.opendev.org/c/openstack/openstacksdk/+/857655 | |
| 17:40:46 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: tests: Improve functional testing for image methods https://review.opendev.org/c/openstack/openstacksdk/+/856272 | |