| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2022-08-18 | |||
| 16:31:51 | gtema | I added it to agenda anyway. The question is that we have added some of the vpnaas functionality, but we can't test it since I can't understand how to activate it in devstack | |
| 16:32:07 | gtema | I did everything like vpnaas code does but tests are still skipped | |
| 16:32:17 | diablo_rojo_phone | Ahh that is indeed an issue. | |
| 16:32:32 | diablo_rojo_phone | I don't know how to set that stuff up in devstack either | |
| 16:33:02 | gtema | yeah, that is the point. I was asking slaweq about that, he promised to have a look, but most likely no time | |
| 16:33:35 | diablo_rojo_phone | That happens. | |
| 16:33:47 | gtema | right, this is life | |
| 16:34:21 | diablo_rojo_phone | Indeed | |
| 16:34:27 | gtema | that's all folks (c) | |
| 16:34:50 | gtema | or is it (r) - no clue | |
| 16:35:21 | diablo_rojo_phone | Yep that's it | |
| 16:36:16 | diablo_rojo_phone | #endmeeting | |
| 16:36:31 | diablo_rojo_phone | Except I have no authority. | |
| 16:36:36 | gtema | #agreed :) | |
| 16:36:38 | diablo_rojo_phone | On this nick apparently | |
| 16:36:43 | seongsoocho | diablo_rojo: gtema I have an one question to you. There seem to be a lot of old and already resolved issues registered in the storyboard. We want to find these and close the issue, but rather than closing it at will, we want to have you review it. How to do it efficiently? | |
| 16:36:44 | gtema | #endmeeting | |
| 16:37:14 | frickler | wait until the hour is up, then anyone can end the meeting | |
| 16:37:26 | gtema | hmm. My problem is that I can't use storyboard effectively, I just become mad after 10 seconds of starting with it | |
| 16:37:30 | diablo_rojo_phone | frickler: ahh got it. Thank you. | |
| 16:38:03 | seongsoocho | gtema: Yes.. The storyboard is really slow. | |
| 16:38:39 | gtema | seongsoocho, maybe you comment inside the issues whether you think it can be closed. At least I will receive email (if spamfilter will not catch it first) | |
| 16:39:01 | gtema | and with that at least I will have a chance to comment or really close it | |
| 16:39:27 | seongsoocho | Okay. I'll comment if I think I can close it. | |
| 16:39:40 | gtema | thanks | |
| 16:39:44 | seongsoocho | thanks | |
| 16:40:16 | slaweq | gtema: hi, sorry, I forgot about it | |
| 16:40:32 | slaweq | Do you have patch woth that job so I can check it? | |
| 16:40:37 | gtema | slaweq: no worries, it will not run away | |
| 16:40:49 | gtema | https://review.opendev.org/c/openstack/openstacksdk/+/846618 | |
| 16:40:55 | slaweq | I'm not at my desk now but I will take a look later tonight or tomorrow morning | |
| 16:41:13 | diablo_rojo | I'm back! | |
| 16:41:16 | opendevmeet | Log: https://meetings.opendev.org/meetings/sdk_osc/2022/sdk_osc.2022-08-18-16.02.log.html | |
| 16:41:16 | opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/sdk_osc/2022/sdk_osc.2022-08-18-16.02.txt | |
| 16:41:16 | opendevmeet | Minutes: https://meetings.opendev.org/meetings/sdk_osc/2022/sdk_osc.2022-08-18-16.02.html | |
| 16:41:16 | opendevmeet | Meeting ended Thu Aug 18 16:41:16 2022 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | |
| 16:41:16 | diablo_rojo | #endmeeting | |
| 16:41:47 | diablo_rojo | I guess I could have kept it going since slaweq is here now | |
| 16:41:50 | gtema | awesome, thanks. In the change I just repeated how vpnaas code sets up jobs, but tests are still skipped, so I do not know whether the it is not activated, or whether tests are having wrong skip | |
| 16:42:14 | gtema | he is not on desk, therefore no problems of not logging it | |
| 16:42:27 | gtema | s/on/at/ | |
| 20:36:22 | opendevreview | hai wu proposed openstack/openstacksdk master: Add support for fault object per Server API https://review.opendev.org/c/openstack/openstacksdk/+/853703 | |
| 20:56:57 | opendevreview | hai wu proposed openstack/openstacksdk master: Add support for fault object per Server API https://review.opendev.org/c/openstack/openstacksdk/+/853703 | |
| 20:57:42 | opendevreview | hai wu proposed openstack/openstacksdk master: Add support for fault object per Server API https://review.opendev.org/c/openstack/openstacksdk/+/853703 | |
| #openstack-sdks - 2022-08-19 | |||
| 06:09:12 | opendevreview | Rafael Castillo proposed openstack/ansible-collections-openstack master: Updates volume for 2.0.0 https://review.opendev.org/c/openstack/ansible-collections-openstack/+/853749 | |
| 11:53:55 | opendevreview | Areg Grigoryan proposed openstack/openstacksdk master: Bugfix for paginating for complex paths https://review.opendev.org/c/openstack/openstacksdk/+/853656 | |
| 13:17:56 | opendevreview | Areg Grigoryan proposed openstack/openstacksdk master: Bugfix for paginating for complex paths https://review.opendev.org/c/openstack/openstacksdk/+/853656 | |
| 18:29:05 | opendevreview | hai wu proposed openstack/openstacksdk master: Add support for fault object per Server API https://review.opendev.org/c/openstack/openstacksdk/+/853703 | |
| 18:31:22 | opendevreview | hai wu proposed openstack/openstacksdk master: Add support for fault object per Server API https://review.opendev.org/c/openstack/openstacksdk/+/853703 | |
| #openstack-sdks - 2022-08-20 | |||
| 01:21:41 | opendevreview | Takashi Natsume proposed openstack/python-openstackclient master: Replace assertItemsEqual with assertCountEqual https://review.opendev.org/c/openstack/python-openstackclient/+/853842 | |
| 01:24:13 | opendevreview | hai wu proposed openstack/openstacksdk master: Add support for fault object per Server API https://review.opendev.org/c/openstack/openstacksdk/+/853703 | |
| 12:50:34 | opendevreview | hai wu proposed openstack/openstacksdk master: Add support for fault object per Server API https://review.opendev.org/c/openstack/openstacksdk/+/853703 | |
| 14:53:06 | opendevreview | hai wu proposed openstack/openstacksdk master: Add support for fault object per Server API https://review.opendev.org/c/openstack/openstacksdk/+/853703 | |
| 19:02:29 | opendevreview | hai wu proposed openstack/openstacksdk master: Add support for fault object per Server API https://review.opendev.org/c/openstack/openstacksdk/+/853703 | |
| #openstack-sdks - 2022-08-22 | |||
| 08:13:53 | opendevreview | Arx Cruz proposed openstack/ansible-collections-openstack master: Update federation_mapping_info module to use proxy https://review.opendev.org/c/openstack/ansible-collections-openstack/+/849283 | |
| 08:58:48 | opendevreview | Frikin Evgenii proposed openstack/ansible-collections-openstack master: Fix processing stack_status when make create stack https://review.opendev.org/c/openstack/ansible-collections-openstack/+/853768 | |
| 09:23:19 | opendevreview | Frikin Evgenii proposed openstack/ansible-collections-openstack master: Fix processing stack_status when make create stack https://review.opendev.org/c/openstack/ansible-collections-openstack/+/853768 | |
| 09:57:46 | opendevreview | Arx Cruz proposed openstack/ansible-collections-openstack master: Update federation_mapping_info module to use proxy https://review.opendev.org/c/openstack/ansible-collections-openstack/+/849283 | |
| 10:12:14 | opendevreview | huang liujie proposed openstack/openstacksdk master: Replace base64.encodestring with encodebytes https://review.opendev.org/c/openstack/openstacksdk/+/853940 | |
| 10:30:43 | opendevreview | Jan Hartkopf proposed openstack/python-openstackclient master: add support for microversion 2.93 https://review.opendev.org/c/openstack/python-openstackclient/+/847792 | |
| 10:48:53 | opendevreview | Jan Hartkopf proposed openstack/python-openstackclient master: add support for microversion 2.93 https://review.opendev.org/c/openstack/python-openstackclient/+/847792 | |
| 10:55:29 | opendevreview | ribaudr proposed openstack/openstacksdk master: Microversion 2.91: Support specifying destination host to unshelve https://review.opendev.org/c/openstack/openstacksdk/+/853949 | |
| 11:14:03 | opendevreview | Arx Cruz proposed openstack/ansible-collections-openstack master: Update federation_mapping_info module to use proxy https://review.opendev.org/c/openstack/ansible-collections-openstack/+/849283 | |
| 12:10:33 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Deprecate special value 'auto' for attribute 'id' of compute_flavor https://review.opendev.org/c/openstack/ansible-collections-openstack/+/852936 | |
| 12:12:58 | jm1 | gtema: hello :) maybe you could help me with this patch? https://review.opendev.org/c/openstack/ansible-collections-openstack/+/852119/ | |
| 12:13:24 | gtema | ok, will have a look later today | |
| 12:13:54 | jm1 | gtema: updating the server fails because no floating ip gets attached when auto_ip is set to true | |
| 12:14:33 | gtema | uhg, that is one of the most tricky cases: how IP is getting assinged. There are 2000000000 different ways | |
| 12:14:42 | gtema | will have a deeper look | |
| 12:17:56 | anskiy | gtema: could you, please, take a look at https://review.opendev.org/c/openstack/openstacksdk/+/848095/ again? | |
| 12:18:20 | gtema | ok | |
| 12:19:18 | anskiy | thank you! :) | |
| 12:19:27 | jm1 | gtema: it uses add_ips_to_server and passes a server object (with server details but without add_server_interfaces being called before) and auto_ips=True. https://review.opendev.org/c/openstack/ansible-collections-openstack/+/852119/8/plugins/modules/server.py#1093 | |
| 12:21:31 | jm1 | gtema: the issue seems to be that add_ips_to_server returns None instead of a server object | |
| 12:35:59 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Replaced expensive get_server() and fixed issues in server module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/852119 | |
| 12:36:58 | jm1 | gtema: oh... maybe the issue is obscured by the until keyword in ci test. i am trying something else and will ping you when i need more help | |
| 12:40:15 | gtema | hmm, ok. Such things we would need to be able to reproduce on bare devstack as well, cause only this will be useful | |
| 12:48:19 | jm1 | gtema: i have another issue with this add_ips_to_server function. this is what i want to accomplish: https://review.opendev.org/c/openstack/ansible-collections-openstack/+/828613/11/ci/roles/floating_ip/tasks/main.yml#271 | |
| 12:48:58 | jm1 | gtema: here it is called: https://review.opendev.org/c/openstack/ansible-collections-openstack/+/828613/11/plugins/modules/floating_ip.py#311 | |
| 12:49:38 | jm1 | gtema: this call returns None as one can see at timestamp "2022-08-10 09:47:26.103680" at https://e045d1163dd923a48c48-5f55b49caeaffd6ebf7a83fc93de4834.ssl.cf2.rackcdn.com/828613/11/check/ansible-collections-openstack-functional-devstack-ansible-2.12/218c3fb/job-output.txt | |
| 12:50:54 | jm1 | gtema: sorry, not the call to the function. the module returns null for floating_ip which means our code cannot find a floating ip | |
| 12:51:35 | jm1 | ..when searching for all floating ips attached to that server | |
| 12:52:23 | jm1 | gtema: i think add_ips_to_server somehow decides that it does not have to do anything. | |
| 12:53:11 | gtema | ;-( this is so hard to analyse. Best would be to have a server and to be able to try out function manually. Cause add_ips_to_server is an entry to the hell | |
| 12:53:38 | jm1 | gtema: sometimes it takes a while until a floating ip is listed for a server. but i handle that by waiting (with until) for the floating ip to be attached to the server... which never happens | |
| 12:54:02 | jm1 | gtema: actually this works in my local devstack but it fails in our ci.. | |
| 12:54:46 | gtema | he, and this is exactly what I told you yesterday: no single cloud (instead of devstack) is openstack cloud (tm) | |
| 12:55:11 | gtema | do we have here ip_pool or ips set? | |
| 12:55:29 | gtema | I guess only ip_pool | |
| 12:56:43 | gtema | madness is in reality in this case due to pool and attempt to reuse eventually existing fips | |
| 12:59:02 | gtema | for the sake of logs and reprodibility: can you add debug of the full server info and list all existing fips before this task is executed? | |
| 12:59:34 | gtema | maybe actually we can also try to start using sdk_logging here, but that would be simply huge | |
| 13:02:37 | opendevreview | ribaudr proposed openstack/python-openstackclient master: Microversion 2.91: Support specifying destination host to unshelve https://review.opendev.org/c/openstack/python-openstackclient/+/831902 | |
| 13:04:22 | opendevreview | ribaudr proposed openstack/python-openstackclient master: Microversion 2.91: Support specifying destination host to unshelve https://review.opendev.org/c/openstack/python-openstackclient/+/831902 | |
| 13:04:38 | jm1 | gtema: ack, will add debug statements | |
| 13:06:32 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Refactored floating_ip and floating_ip_info modules https://review.opendev.org/c/openstack/ansible-collections-openstack/+/828613 | |
| 13:11:37 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Refactored floating_ip and floating_ip_info modules https://review.opendev.org/c/openstack/ansible-collections-openstack/+/828613 | |
| 13:14:36 | jm1 | gtema: we could replace add_ips_to_server in aoc modules, but that would replicate code. this belongs to sdk i would think | |
| 13:15:15 | gtema | definitely. I want to gather few examples (conditions) when it fails and extend functional tests | |
| 14:07:15 | opendevreview | Arx Cruz proposed openstack/ansible-collections-openstack master: Update federation_mapping_info module to use proxy https://review.opendev.org/c/openstack/ansible-collections-openstack/+/849283 | |