Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2022-08-18
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
14:46:56 opendevreview Merged openstack/openstacksdk master: Replace base64.encodestring with encodebytes https://review.opendev.org/c/openstack/openstacksdk/+/853940
14:53:11 jm1 gtema: new log for first issue https://e3e2390f493f14093eaf-5e1de375231dae3d22db48edd8691108.ssl.cf5.rackcdn.com/852119/9/check/ansible-collections-openstack-functional-devstack-ansible-2.9/b2d8736/job-output.txt
14:54:37 jm1 gtema: add_ips_to_server with auto_ip=True should simply add a floating ip to server but fails with "No port on server XXX was found matching your NAT destination network XXX"
14:57:51 jm1 gtema: i guess you want some extra output. will add debug statements
14:59:30 gtema ok, thanks
14:59:37 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
15:32:11 jm1 gtema: for the second issue with add_ips_to_server we have some debug output now https://dbb517edecb69c839e64-a1eedb38eefc88bab97a9b5aab34511e.ssl.cf1.rackcdn.com/828613/13/check/ansible-collections-openstack-functional-devstack-ansible-2.12/2bd2d7b/job-output.txt
15:32:42 jm1 gtema: this is the patch https://review.opendev.org/c/openstack/ansible-collections-openstack/+/828613
15:33:33 gtema cool, thks
15:34:40 gtema okay, this means clearly we are in the "reuse existing unbound fip"
15:34:57 gtema this is what I was suspecting.
15:42:21 gtema @jm1: do you see that port_id in the query is of a fixed ip and not floating?
15:44:37 jm1 gtema:?
15:46:51 gtema "Get floating ip attached to server 2" task in the log uses "port": "1e08abff-72cd-465c-b92d-18806df814d6" as filter
15:47:09 gtema and this port is of a fixed address and not floating
15:48:16 gtema I am wondered that in the test you simply use server2_ports.ports[1].id
16:04:37 opendevreview mbu proposed openstack/osc-lib stable/train: Do not output log on bash autocomplete generation https://review.opendev.org/c/openstack/osc-lib/+/854016
16:07:06 matbu Hey folks, is it possible to merge this patch, which is a cherry pick from master: https://review.opendev.org/c/openstack/osc-lib/+/854016
16:07:10 matbu thank you by advance

Earlier   Later