Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2019-05-30
16:01:41 edleafe So we both win!
16:01:52 elmiko hahaha, amazing
16:15:23 cdent I lost track, did the errors guidelines discussion go anywhere?
16:15:56 edleafe There was a little async back-and-forth in this channel, but it didn't really go anywhere
16:16:09 elmiko ++
16:16:12 cmurphy o/
16:16:39 cmurphy i don't think there's much more to discuss, it is what it is
16:16:56 edleafe cdent: Starting here: https://leafe.com/timeline-middle/%23openstack-sdks/2019-05-23T20:34:53
16:18:53 cdent is there a next step?
16:19:17 elmiko unless we are going to rewrite the guideline, i don't think so
16:19:32 cdent potentially a patch to keystoneauth?
16:19:38 elmiko that might be nice
16:19:58 elmiko i have some time, i could take a look at patching it
16:20:17 cdent you have time? I guess that's what happens when you work on something oher than openstack ;)
16:20:34 elmiko yeah... plus when weird things happen to your group internally ;)
16:27:19 nwilburn I found an issue with the add_ip_list function in opestack/cloud/openstackcloud.py. Basically nat_destination is not being passed to the _attach_ip_to_server function which causes issues with Ansible's os_floating_ip module (and probably other things). The immediate "fix" that I can see is to simply add a parameter "nat_destination" for add_ip_list, but this probably isn't the right way of doing
16:27:25 nwilburn things.
16:27:27 nwilburn Where is the best place to submit a bug report for this?
16:30:35 nwilburn Apologies, it now lives in openstack/cloud/_floating_ip.py
16:33:52 elmiko nwilburn: is that part of the sdk package?
16:35:03 nwilburn It is
16:35:20 elmiko hmm, maybe mordred knows
16:35:50 elmiko at a guess, i would think here https://opendev.org/openstack/openstacksdk
16:35:54 elmiko but i'm not 100% sure on that
16:36:11 nwilburn Ok thanks
16:36:21 elmiko don't quote me on that though ;)
16:40:58 elmiko edleafe: it's been awhile since i've made pr's to openstack projects, do i want to be cloning from opendev and the using the standard `git review` mechanics?
16:46:54 edleafe elmiko: I clone from github, and then use `git review`
16:46:54 Shrews nwilburn: are you setting nat_destination in your cloud.yaml file?
16:47:04 elmiko ack, thanks!
16:47:55 nwilburn Shrews: nat_destination is being passed in via the os_floating_ip ansible module. It does make it to the function which calls add_ip_list, but add_ip_list doesn't take a parameter of nat_destination
16:48:38 Shrews nwilburn: ah, well that does sound like a bug
16:49:08 nwilburn So openstacksdk tries to be smart about discovering what the nat_destination should be and it goes through a list of openstack networks looking for the last one which is attached to the public network. When the ansible module is being ran with admin credentials this results in a network being found which isn't even in the project that the server is in. Super strange
16:49:42 Shrews indeed. floating ips and the surrounding logic is ... hard
16:50:18 nwilburn So I added nat_destination to add_ip_list and changes up the function args for the calls. Things worked but this isn't the right way of doing things since the list of floating ips you want to attach may need differenation nat_destinations. Maybe the ip_list should actually be a list of dicts with ip:nat_destination?
17:01:19 elmiko have a good weekend edleafe, i'm gonna take a look into patching keystoneauth o/
17:04:23 edleafe elmiko: \o
18:19:14 openstackgerrit Michael McCune proposed openstack/keystoneauth master: add handling for multiple error returns https://review.opendev.org/662281
18:26:55 openstackgerrit Michael McCune proposed openstack/keystoneauth master: add handling for multiple error returns https://review.opendev.org/662281
18:29:04 elmiko cmurphy: just a heads up ^^
18:30:17 cmurphy thanks elmiko
19:25:55 dtroyer elmiko: This doesn't affect what you are doing now, but cloning from Github has some dark corner-cases now that not everything is mirrored there from the current opendev repos. That will affect everything that moved out of the official openstack/ namespace but has not been moved in Github yet..
19:26:32 dtroyer All official project repos are unaffected by that AFAIK
19:44:06 elmiko dtroyer: ack, tyvm for the explanation. i defaulted to cloning from opendev =)
20:04:15 nwilburn So I added nat_destination to add_ip_list and changes up the function args for the calls. Things worked but this isn't the right way of doing things since the list of floating ips you want to attach may need differenation nat_destinations. Maybe the ip_list should actually be a list of dicts with ip:nat_destination?
20:04:23 nwilburn Oops sorry. Wrong window
20:15:38 elmiko we'll forgive you /this time/ nwilburn XD
21:36:18 openstackgerrit Michael McCune proposed openstack/keystoneauth master: add handling for multiple error returns https://review.opendev.org/662281
23:03:07 openstackgerrit Michael McCune proposed openstack/keystoneauth master: add handling for multiple error returns https://review.opendev.org/662281
23:03:39 openstackgerrit Michael McCune proposed openstack/keystoneauth master: add handling for multiple error returns https://review.opendev.org/662281
#openstack-sdks - 2019-05-31
05:39:29 openstackgerrit Brin Zhang proposed openstack/python-openstackclient master: Add changes-before attribute to server list https://review.opendev.org/657373
06:11:55 dtantsur mordred: hi! could you release the sdk in the near future?
07:35:14 ITD27M01 gtema: Hello! Are you online? Have you a few seconds for me?
07:35:36 gtema Hi, I am online, but pretty busy in next hour
07:36:32 ITD27M01 gtema: Well, just take a note to see the PR: https://github.com/ansible/ansible/pull/51401
07:36:42 gtema ok
07:36:47 ITD27M01 gtema: I really need your and mordred attention
07:36:53 gtema ok
10:18:42 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Rework statistics reporting https://review.opendev.org/659841
12:46:06 openstackgerrit Michael McCune proposed openstack/keystoneauth master: add handling for multiple error returns https://review.opendev.org/662281
13:05:30 ITD27M01 mordred: Hello! Are you online? Have you a few seconds for me?
13:19:12 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Rework statistics reporting https://review.opendev.org/659841
13:20:25 gtema ITD27M01: I will start reworking cloud layer of compute to migrate it to "proxy" layer and will also address proper filtering, so that it is passing as much as possible to server
13:23:22 ITD27M01 gtema: Thank you very much! This will be very helpful. So, we wait for mordred's approval for this PR.
13:24:33 gtema cool. I wish it was possible to filter servers by metadata on server side, but that's life
14:03:12 ITD27M01 Shrews: You also marked as reviewer for PR: https://github.com/ansible/ansible/pull/51401
14:06:22 Shrews i'll put it on my list
14:08:30 ITD27M01 Shrews: I will be very grateful to you. This is a very useful feature.
19:21:32 openstackgerrit Brian Haley proposed openstack/python-openstackclient master: Support IPv6 addresses better https://review.opendev.org/524420
22:13:24 efried mordred: I'm looking into the unit tests for https://review.opendev.org/#/c/643601/
22:13:32 efried not sure how we're going to do this without importing oslo.config
22:13:42 efried Is it okay to put that in test-requirements.txt?
#openstack-sdks - 2019-06-01
00:29:10 openstackgerrit Eric Fried proposed openstack/openstacksdk master: WIP Make factory for a CloudRegion from CONF objects https://review.opendev.org/643601
00:31:29 efried mordred: Made tests ^ but not behaving as I think they should, please advise.
00:31:45 efried (next week)
01:31:34 mordred efried: cool. I'm out through monday, but will dig in tuesday. also - yes, adding oslo.config to test-requirements is perfectly fine
13:02:40 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: WIP Make factory for a CloudRegion from CONF objects https://review.opendev.org/643601
13:05:23 mordred ITD27M01: I'm out through monday for some family things (not really here right now) - but will review tuesday
13:25:12 mordred dtantsur|afk: remote: https://review.opendev.org/662606 Release 0.29.0 of openstacksdk
13:25:45 mordred efried: I responded, and also pushed up a patch which I think might fix the issue (Also thanks for getting that started)
19:01:52 openstackgerrit Eric Fried proposed openstack/openstacksdk master: Make factory for a CloudRegion from CONF objects https://review.opendev.org/643601
19:06:48 efried mordred, dtantsur|afk, gtema: This just might be ready to go ^
#openstack-sdks - 2019-06-02
13:26:35 openstackgerrit LIU Yulong proposed openstack/python-openstackclient master: Add floating IP Port Forwarding commands https://review.opendev.org/650062
#openstack-sdks - 2019-06-03
07:11:36 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Use resource layer for compute flavors https://review.opendev.org/650903
07:47:01 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Support skipping unknown QP https://review.opendev.org/662698
09:34:45 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Use Resource layer for network SecurityGroups https://review.opendev.org/662724
09:48:52 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Use resource layer for compute flavors https://review.opendev.org/650903
10:55:11 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Use Resource layer for compute KeyPairs https://review.opendev.org/662729
10:58:39 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Use Resource layer for network SecurityGroups https://review.opendev.org/662724
11:07:48 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Use Resource layer for compute KeyPairs https://review.opendev.org/662729
11:42:28 openstackgerrit Surya Seetharaman proposed openstack/python-openstackclient master: Microversion 2.73: Support adding the reason behind a server lock https://review.opendev.org/659124
11:44:58 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Use Resource layer for compute AZ and Aggregates https://review.opendev.org/662738
11:47:38 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Use resource layer for compute flavors https://review.opendev.org/650903
12:04:11 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Use Resource layer for network SecurityGroups https://review.opendev.org/662724
12:31:27 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Use Resource layer for the compute Hypervsors and Images https://review.opendev.org/662743
12:34:30 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Use Resource layer for compute KeyPairs https://review.opendev.org/662729
14:06:46 Shrews mordred: when you return, I left comments on that PR ITD27M01 was asking about, but you should also have a look I think
15:09:41 openstackgerrit Michael McCune proposed openstack/keystoneauth master: add handling for multiple error returns https://review.opendev.org/662281
17:23:42 mordred Shrews: nod, thanks
19:16:10 openstackgerrit Eric Fried proposed openstack/openstacksdk master: Make factory for a CloudRegion from CONF objects https://review.opendev.org/643601

Earlier   Later