| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2023-02-21 | |||
| 13:28:38 | opendevreview | Merged openstack/python-openstackclient master: Switch server volume update to sdk https://review.opendev.org/c/openstack/python-openstackclient/+/821734 | |
| 13:28:40 | opendevreview | Merged openstack/python-openstackclient master: compute: 'server volume update' -> 'server volume set' https://review.opendev.org/c/openstack/python-openstackclient/+/867845 | |
| 13:28:45 | opendevreview | Merged openstack/python-openstackclient master: tests: Use fake SDK Migration object https://review.opendev.org/c/openstack/python-openstackclient/+/869211 | |
| 13:28:48 | opendevreview | Merged openstack/python-openstackclient master: Finish switching server migration to sdk https://review.opendev.org/c/openstack/python-openstackclient/+/867334 | |
| 13:39:20 | opendevreview | Gaël THEROND proposed openstack/ansible-collections-openstack master: Add module to filter available volume services. https://review.opendev.org/c/openstack/ansible-collections-openstack/+/868480 | |
| 13:46:11 | opendevreview | Mridula Joshi proposed openstack/openstacksdk master: Adds SDK support for ``glance cache-delete`` https://review.opendev.org/c/openstack/openstacksdk/+/874584 | |
| 17:47:30 | opendevreview | Gaël THEROND proposed openstack/ansible-collections-openstack master: Add module to filter available volume services. https://review.opendev.org/c/openstack/ansible-collections-openstack/+/868480 | |
| 18:37:19 | opendevreview | Christopher Parks proposed openstack/openstacksdk master: Add share types to shared file systems. https://review.opendev.org/c/openstack/openstacksdk/+/783123 | |
| 19:12:17 | opendevreview | Reynaldo proposed openstack/openstacksdk master: Add share instances to shared file systems https://review.opendev.org/c/openstack/openstacksdk/+/873109 | |
| 19:27:33 | opendevreview | Goutham Pacha Ravi proposed openstack/openstacksdk master: Add share types to shared file systems. https://review.opendev.org/c/openstack/openstacksdk/+/783123 | |
| 19:44:17 | opendevreview | Merged openstack/ansible-collections-openstack master: Do not hardcode image name and properly delete mock images in CI tests https://review.opendev.org/c/openstack/ansible-collections-openstack/+/874545 | |
| 19:51:16 | opendevreview | Christopher Parks proposed openstack/openstacksdk master: Add share types to shared file systems. https://review.opendev.org/c/openstack/openstacksdk/+/783123 | |
| 20:10:01 | opendevreview | Christopher Parks proposed openstack/openstacksdk master: Add share types to shared file systems. https://review.opendev.org/c/openstack/openstacksdk/+/783123 | |
| 20:14:12 | opendevreview | Michael Johnson proposed openstack/openstacksdk master: Add Designate (DNS) zone share API https://review.opendev.org/c/openstack/openstacksdk/+/874695 | |
| 22:20:41 | opendevreview | Christopher Parks proposed openstack/openstacksdk master: Add share types to shared file systems. https://review.opendev.org/c/openstack/openstacksdk/+/783123 | |
| #openstack-sdks - 2023-02-22 | |||
| 09:15:06 | opendevreview | Artem Goncharov proposed openstack/openstacksdk master: Implement acceptance test job https://review.opendev.org/c/openstack/openstacksdk/+/860399 | |
| 09:55:16 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: Wait for volume being available to set bootable or readonly https://review.opendev.org/c/openstack/python-openstackclient/+/845991 | |
| 10:03:02 | opendevreview | Merged openstack/python-openstackclient master: doc: Update nova command mapping https://review.opendev.org/c/openstack/python-openstackclient/+/858952 | |
| 10:04:30 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: zuul: Merge osc-functional-devstack-base into only child https://review.opendev.org/c/openstack/python-openstackclient/+/863898 | |
| 10:05:35 | opendevreview | Artem Goncharov proposed openstack/openstacksdk master: Implement acceptance test job https://review.opendev.org/c/openstack/openstacksdk/+/860399 | |
| 10:07:47 | opendevreview | Mark Goddard proposed openstack/ansible-collections-openstack master: Add baremetal_deploy_template module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/874755 | |
| 10:15:47 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: Add missing documentation for state options for "image set" https://review.opendev.org/c/openstack/python-openstackclient/+/813393 | |
| 11:03:08 | opendevreview | Gaël THEROND proposed openstack/ansible-collections-openstack master: Add module to filter available volume services. https://review.opendev.org/c/openstack/ansible-collections-openstack/+/868480 | |
| 11:03:08 | opendevreview | Gaël THEROND proposed openstack/ansible-collections-openstack master: Add module to filter available volume services. https://review.opendev.org/c/openstack/ansible-collections-openstack/+/868480 | |
| 12:32:10 | opendevreview | Merged openstack/python-openstackclient master: Deprecate positional args for 'volume group create' https://review.opendev.org/c/openstack/python-openstackclient/+/874151 | |
| 12:32:12 | opendevreview | Merged openstack/python-openstackclient master: volume: Remove duplication from 'consistency group create' opts https://review.opendev.org/c/openstack/python-openstackclient/+/873730 | |
| 12:32:14 | opendevreview | Merged openstack/python-openstackclient master: Add port ranges on floating ip portforwardings cli https://review.opendev.org/c/openstack/python-openstackclient/+/836870 | |
| 12:32:17 | opendevreview | Merged openstack/python-openstackclient master: Fix really long help strings https://review.opendev.org/c/openstack/python-openstackclient/+/867272 | |
| 12:32:19 | opendevreview | Merged openstack/python-openstackclient master: Update 'host list' and 'host show' command to use sdk https://review.opendev.org/c/openstack/python-openstackclient/+/866818 | |
| 12:32:21 | opendevreview | Merged openstack/python-openstackclient master: Wait for volume being available to set bootable or readonly https://review.opendev.org/c/openstack/python-openstackclient/+/845991 | |
| 12:32:23 | opendevreview | Merged openstack/python-openstackclient master: Drop default from ask_user_yesno question https://review.opendev.org/c/openstack/python-openstackclient/+/868674 | |
| 12:53:01 | opendevreview | Merged openstack/python-openstackclient master: Add missing documentation for state options for "image set" https://review.opendev.org/c/openstack/python-openstackclient/+/813393 | |
| 13:37:23 | opendevreview | Artem Goncharov proposed openstack/openstacksdk master: Implement acceptance test job https://review.opendev.org/c/openstack/openstacksdk/+/860399 | |
| 13:38:38 | opendevreview | Gaël THEROND proposed openstack/ansible-collections-openstack master: Add module to filter available volume services. https://review.opendev.org/c/openstack/ansible-collections-openstack/+/868480 | |
| 14:07:17 | ozzzo_work | I'm trying to use the SDK to pull a list of security groups with "cloud.network.security_groups()" | |
| 14:07:41 | ozzzo_work | That works fine for small clusters but on large clusters it times out with: 504 Gateway Time-out: The server didn't respond in time. | |
| 14:08:07 | ozzzo_work | I tried adding --timeout=1000 but it seems to be ignored; the call times out after 60 seconds | |
| 14:08:14 | ozzzo_work | https://paste.openstack.org/show/b96XXB9Us8utAlk57DCt/ | |
| 14:08:19 | ozzzo_work | What am I missing? | |
| 14:08:37 | gtema | then you either implement some reasonable filter or you analyze your SGs in a loop, since "security_groups()" return generator | |
| 14:09:06 | ozzzo_work | how do I loop through the generator? | |
| 14:09:30 | gtema | for sg in cloud.network.security_groups(): | |
| 14:09:39 | ozzzo_work | ok I'll try that | |
| 14:10:48 | gtema | if even that doesnot help try adding "limit=100" parameter into the listing call. That should instruct server to only return 100 entries per "page" | |
| 14:16:40 | ozzzo_work | Does the timeout not work? I see it here: https://github.com/openstack/openstacksdk/blob/master/openstack/config/loader.py#L755 | |
| 14:17:03 | gtema | how do you pass it? | |
| 14:27:27 | ozzzo_work | I'm passing it when I open the connection at line 57 | |
| 14:29:34 | gtema | ah, I overseen paste | |
| 14:30:24 | gtema | generally it uses it, but it is not able to instruct server (proxy) to wait longer. In your case since you get 504 it means that proxy server of your cloud is not capable to get response in defined time | |
| 14:30:40 | gtema | timeout will not help you further. You need to start using "limit" option to the list | |
| 14:31:13 | ozzzo_work | ok | |
| 14:31:41 | ozzzo_work | is there a document on paging? | |
| 14:35:36 | gtema | https://docs.openstack.org/api-ref/network/v2/index.html?expanded=list-security-groups-detail#pagination | |
| 14:36:17 | gtema | but you need to only use "limit" which limits page size, pagination as such is managed by sdk itself | |
| 14:50:08 | ozzzo_work | that works, ty! | |
| 15:12:40 | opendevreview | Gaël THEROND proposed openstack/ansible-collections-openstack master: Add module to filter available volume services. https://review.opendev.org/c/openstack/ansible-collections-openstack/+/868480 | |
| 17:19:09 | lutimura | Hi guys. I've a question about OpenStackClient (OSC) plugins discovery order. | |
| 17:19:23 | lutimura | I have a custom plugin that introduces some commands to the OSC's CLI and also overrides some of the entry points defined by other plugins (e.g., python-glanceclient). Due to these entry points overrides, the order in which my plugin is loaded in OSC is extremely important to determine whether the CLI will be able to redirect the Glance-related commands to my entry points or not (for example, to python-glanceclient's entry points instead). | |
| 17:19:44 | lutimura | So my question is, currently, is there a way to set the plugin load order in OSC? To ensure all custom plugins are loaded last? | |
| 18:48:37 | frickler | lutimura: I don't think that we (want to) support that use case. IMO the best solution for your issue would be to also patch the existing plugins to drop the commands you want to override. or even better, submit your changes upstream for those plugins | |
| 19:04:15 | opendevreview | Merged openstack/os-api-ref master: Fix the 'No such file or directory' error https://review.opendev.org/c/openstack/os-api-ref/+/874332 | |
| 19:04:22 | gtema | +1 to fricler's statement | |
| 20:15:23 | lutimura | frickler: Thanks for answering. I agree with you. It really makes more sense to keep the plugins focused on extending the CLI instead of overwriting existing entry points (of other plugins). Anyways, I'll think about some alternatives and I'll definitely consider the possibility of going upstream. Again, thank you. | |
| 21:21:50 | opendevreview | Christopher Parks proposed openstack/openstacksdk master: Add share types to shared file systems. https://review.opendev.org/c/openstack/openstacksdk/+/783123 | |
| 21:34:17 | opendevreview | Christopher Parks proposed openstack/openstacksdk master: Add share types to shared file systems. https://review.opendev.org/c/openstack/openstacksdk/+/783123 | |
| 21:47:59 | opendevreview | Christopher Parks proposed openstack/openstacksdk master: Add share types to shared file systems. https://review.opendev.org/c/openstack/openstacksdk/+/783123 | |
| 22:28:40 | opendevreview | Arvid Requate proposed openstack/keystoneauth master: New auth plugin v3oidcdeviceauthz https://review.opendev.org/c/openstack/keystoneauth/+/869876 | |
| #openstack-sdks - 2023-02-23 | |||
| 01:18:14 | opendevreview | Christopher Parks proposed openstack/openstacksdk master: Add share types to shared file systems. https://review.opendev.org/c/openstack/openstacksdk/+/783123 | |
| 04:28:58 | opendevreview | Christopher Parks proposed openstack/openstacksdk master: Add share types to shared file systems. https://review.opendev.org/c/openstack/openstacksdk/+/783123 | |
| 09:09:19 | opendevreview | Arvid Requate proposed openstack/keystoneauth master: New auth plugin v3oidcdeviceauthz https://review.opendev.org/c/openstack/keystoneauth/+/869876 | |
| 09:15:21 | opendevreview | Ke Niu proposed openstack/os-api-ref master: remove unicode prefix from code https://review.opendev.org/c/openstack/os-api-ref/+/852970 | |
| 10:54:43 | opendevreview | Mridula Joshi proposed openstack/openstacksdk master: Adds SDK support for ``glance cache-delete`` https://review.opendev.org/c/openstack/openstacksdk/+/874584 | |
| 11:04:54 | noonedeadpunk | Fun thing about released ansible-collections 2.0 - they can't be used as of today as now SDK in U-C lower version then needed: https://opendev.org/openstack/requirements/src/branch/master/upper-constraints.txt#L453 | |
| 11:05:22 | noonedeadpunk | I bet it was discussed, but is sdk 1.0 will end up in UC for 2023.1? | |
| 11:14:29 | frickler | noonedeadpunk: there are two blockers still if you look at the reqs patch. octaviaclient should get fixed today, not sure about kuryr. also not sure whether it would need and get yet another ffe | |
| 11:15:34 | frickler | we are having the same issue in kolla fwiw | |
| 11:16:35 | noonedeadpunk | Yeah, I truly believe I'm not alone struggling with mathing sdk with collections hehe | |
| 11:17:26 | noonedeadpunk | but it's all fair, I wasn't following closely lately so decided to ask | |
| 11:26:45 | frickler | gtema: did you have any discussion regarding this with reqs or release teams yet? would you want to push that still? | |
| 11:35:01 | gtema | fricler: I wasn't discussing this so far | |
| 11:37:50 | frickler | gtema: so do you think we should keep sdk<1.0 for A? or still bump it despite being in client freeze? https://review.opendev.org/c/openstack/releases/+/874864 should fix the osc docs build issue fwiw | |
| 11:39:17 | gtema | No, we should make it >=1.0 | |
| 11:39:43 | gtema | We had release for quite a while already | |
| 11:40:15 | gtema | Just that UC patch never landed | |
| 11:42:28 | frickler | yes, because of the above mentioned blockers. I would support ignoring the kuryr issue, as there is a patch for them, just their CI seems broken anyway | |
| 11:43:11 | frickler | do you want to pick this up with the release team? I can try to do it myself, but ptl-authority might work better | |
| 12:07:33 | gtema | So what you think I should discuss with them, approval on finally merging UC patch? | |
| 12:13:42 | frickler | gtema: yes, because formally requirements are frozen now until the release | |
| 12:14:45 | frickler | next step would then be discussing the kuryr situation, not sure if that is also release-affecting | |
| 12:16:21 | frickler | oh, seems that stephenfin has that mostly covered, too. so would need just another ffe likely https://review.opendev.org/c/openstack/kuryr-kubernetes/+/874571/4 | |
| 12:20:18 | frickler | I just mentioned the situation in #-release, feel free to chime in as you see fit | |
| 16:20:05 | opendevreview | Mridula Joshi proposed openstack/openstacksdk master: Adding support for glance cache-queue Command https://review.opendev.org/c/openstack/openstacksdk/+/874372 | |
| 16:49:35 | opendevreview | Mridula Joshi proposed openstack/openstacksdk master: Adds Support for ``glance cache-clear`` https://review.opendev.org/c/openstack/openstacksdk/+/874940 | |
| 20:21:57 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Respect description option and delete security group rules first https://review.opendev.org/c/openstack/ansible-collections-openstack/+/874960 | |
| 21:40:11 | opendevreview | Merged openstack/python-openstackclient master: zuul: Merge osc-functional-devstack-base into only child https://review.opendev.org/c/openstack/python-openstackclient/+/863898 | |
| 22:00:25 | opendevreview | Merged openstack/ansible-collections-openstack master: Use true and false instead of yes and no for boolean values https://review.opendev.org/c/openstack/ansible-collections-openstack/+/874538 | |
| 23:16:22 | opendevreview | Christopher Parks proposed openstack/openstacksdk master: Add share types to shared file systems. https://review.opendev.org/c/openstack/openstacksdk/+/783123 | |
| #openstack-sdks - 2023-02-24 | |||
| 00:14:32 | opendevreview | Christopher Parks proposed openstack/openstacksdk master: Add share types to shared file systems. https://review.opendev.org/c/openstack/openstacksdk/+/783123 | |
| 09:50:16 | opendevreview | Christian Kueppers proposed openstack/ansible-collections-openstack master: Fix for AttributeError: 'dict' object has no attribute 'status'. https://review.opendev.org/c/openstack/ansible-collections-openstack/+/874996 | |
| 15:10:34 | opendevreview | OpenStack Release Bot proposed openstack/keystoneauth stable/2023.1: Update .gitreview for stable/2023.1 https://review.opendev.org/c/openstack/keystoneauth/+/875104 | |