| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2023-02-22 | |||
| 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 | |
| 15:10:35 | opendevreview | OpenStack Release Bot proposed openstack/keystoneauth stable/2023.1: Update TOX_CONSTRAINTS_FILE for stable/2023.1 https://review.opendev.org/c/openstack/keystoneauth/+/875105 | |
| 15:10:36 | opendevreview | OpenStack Release Bot proposed openstack/keystoneauth master: Update master for stable/2023.1 https://review.opendev.org/c/openstack/keystoneauth/+/875106 | |
| 20:01:17 | ozzzo_work | I'm trying to use the SDK to stop a server with: lambda serverID: cloud.compute.update_server(serverID,'status=SHUTOFF'), and then output = argdict[resource]['stop'](key) | |
| 20:01:46 | ozzzo_work | but I get an error TypeError: update_server() takes 2 positional arguments but 3 were given | |
| 20:01:56 | ozzzo_work | is that not the right way to stop a server? | |
| 20:09:44 | ozzzo_work | This is my code: https://bpa.st/2HVQY | |
| 20:12:09 | ozzzo_work | er... bad paste: it's https://bpa.st/MD3K6 | |
| 20:12:43 | ozzzo_work | The relevant lines are 32 and 119 | |
| #openstack-sdks - 2023-02-25 | |||
| 04:35:05 | opendevreview | Reynaldo proposed openstack/openstacksdk master: Add share snapshot instance resource https://review.opendev.org/c/openstack/openstacksdk/+/798464 | |
| #openstack-sdks - 2023-02-27 | |||
| 12:20:09 | opendevreview | Merged openstack/os-api-ref master: remove unicode prefix from code https://review.opendev.org/c/openstack/os-api-ref/+/852970 | |
| 13:48:32 | opendevreview | Mridula Joshi proposed openstack/openstacksdk master: Adds Support for ``glance cache-clear`` https://review.opendev.org/c/openstack/openstacksdk/+/874940 | |
| 14:05:04 | opendevreview | Mridula Joshi proposed openstack/openstacksdk master: Adding support for glance cache-queue Command https://review.opendev.org/c/openstack/openstacksdk/+/874372 | |
| 14:58:10 | opendevreview | Merged openstack/keystoneauth stable/2023.1: Update .gitreview for stable/2023.1 https://review.opendev.org/c/openstack/keystoneauth/+/875104 | |
| 15:10:33 | opendevreview | Merged openstack/keystoneauth stable/2023.1: Update TOX_CONSTRAINTS_FILE for stable/2023.1 https://review.opendev.org/c/openstack/keystoneauth/+/875105 | |
| 15:37:33 | ozzzo_work | What am I doing wrong here? https://bpa.st/MD3K6 | |
| 15:38:11 | ozzzo_work | I'm providing two arguments to update_server but the error says 3 were given | |
| 15:39:29 | ozzzo_work | The 'delete' part where I only pass a variable seems to work - it looks like I may not be passing the text argument correctly | |
| 15:40:46 | gtema | ozzzo_work: you should be using function stop_server(serverID) [https://opendev.org/openstack/openstacksdk/src/branch/master/openstack/compute/v2/_proxy.py#L1092] instead | |
| 15:41:39 | gtema | also - you cannot simply pass parameters as a string into the update. Only passing them as regular function parameters is supported | |
| 15:54:18 | opendevreview | Lajos Katona proposed openstack/openstacksdk master: Add BGPVPN to SDK https://review.opendev.org/c/openstack/openstacksdk/+/875530 | |
| 15:58:49 | ozzzo_work | gtema: ty! I was looking at https://docs.openstack.org/openstacksdk/latest/user/proxies/compute.html but I see that it is missing some things | |
| 16:22:02 | ozzzo_work | how can I change a text argument into a regular function parameter? | |
| 16:44:32 | gtema | update method accepts same parameters as create call (not all of them are allowed to be changed though) | |