| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2018-04-27 | |||
| 13:14:38 | mordred | Shrews: me either - I kind of failed at paying attention to freeze dates, thus why we woundup with queens releases :) | |
| 13:35:10 | amotoki | melwitt: sorry for late. I will take a look at the backport patch during this weekend at the latest. I need to refresh my memory on compute/network hybrid commands. | |
| 13:44:50 | openstackgerrit | Monty Taylor proposed openstack/os-client-config master: Replace guts with openstack.config https://review.openstack.org/549307 | |
| 13:53:44 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Add Limestone Networks vendor info https://review.openstack.org/564728 | |
| 13:56:49 | melwitt | thanks amotoki | |
| 13:57:10 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: pypy is not checked at gate https://review.openstack.org/564729 | |
| 15:07:04 | openstackgerrit | Merged openstack/openstacksdk master: Remove DataCentred from list of vendors https://review.openstack.org/564727 | |
| 15:57:54 | openstackgerrit | Daniel Speichert proposed openstack/openstacksdk master: Flavor: added is_public query parameter and description property https://review.openstack.org/564817 | |
| 18:37:57 | openstackgerrit | Monty Taylor proposed openstack-infra/shade master: Make name setting in connect_as more resilient https://review.openstack.org/564861 | |
| 20:15:25 | openstackgerrit | Hongbin Lu proposed openstack/openstacksdk master: Add 'port_details' to Floating IP https://review.openstack.org/533811 | |
| 20:29:36 | openstackgerrit | Hongbin Lu proposed openstack/python-openstackclient master: Format port_details field of Floating IP https://review.openstack.org/533809 | |
| 20:50:41 | openstackgerrit | Hongbin Lu proposed openstack/openstacksdk master: Add 'port_details' to Floating IP https://review.openstack.org/533811 | |
| 22:35:34 | openstackgerrit | Mark Kirkwood proposed openstack/python-openstackclient master: Add an option to container create for storage policy. Also display some policy data for container show and object store account show commands. https://review.openstack.org/564665 | |
| 22:55:56 | openstackgerrit | Hongbin Lu proposed openstack/openstacksdk master: Add 'port_details' to Floating IP https://review.openstack.org/533811 | |
| #openstack-sdks - 2018-04-28 | |||
| 16:04:21 | openstackgerrit | Merged openstack/openstacksdk master: Flavor: added is_public query parameter and description property https://review.openstack.org/564817 | |
| #openstack-sdks - 2018-04-29 | |||
| 17:59:47 | openstackgerrit | Theodoros Tsioutsias proposed openstack/python-openstackclient master: Add project info to server list from all projects https://review.openstack.org/562300 | |
| 18:00:34 | openstackgerrit | Theodoros Tsioutsias proposed openstack/python-openstackclient master: Add force parameter for quota set https://review.openstack.org/551811 | |
| #openstack-sdks - 2018-04-30 | |||
| 05:37:34 | openstackgerrit | Nobuto Murata proposed openstack/python-openstackclient master: Support --community in openstack image list https://review.openstack.org/565152 | |
| 10:39:09 | rcarrillocruz | looking at openlab zuul dashboard, i see jobs for a terraform open-telekom specific provider. Does it mean open-telekom is a fork from openstack APIs ? | |
| 11:47:29 | mordred | rcarrillocruz: last I checked it was possible to use open-telekom with normal apis | |
| 11:49:28 | openstackgerrit | Monty Taylor proposed openstack-infra/shade master: Adds Senlin support to shade https://review.openstack.org/542537 | |
| 12:27:02 | frickler | slaweq: can you take a look at http://git.openstack.org/cgit/openstack/openstacksdk/tree/openstack/network/v2/port.py#n108 please? to me it seems like these options are bogus ("option_name" and "option_value") | |
| 12:27:40 | frickler | looking at the api-ref, those seem to be used only within extra_dhcp_opt, not as top-level values https://developer.openstack.org/api-ref/network/v2/#extra-dhcp-option-extra-dhcp-opt-extension | |
| 12:29:28 | frickler | slaweq: actually there are even more strange parameters, e.g. "ip_address" | |
| 12:29:51 | slaweq | frickler: looking | |
| 12:31:22 | frickler | all introduced in this refactoring commit quite some time ago https://review.openstack.org/#/c/364052/4/openstack/network/v2/port.py | |
| 12:35:40 | slaweq | frickler: you're right, looking into https://github.com/openstack/neutron-lib/blob/master/neutron_lib/api/definitions/extra_dhcp_opt.py those are only options for extra_dhcp_opt | |
| 12:36:28 | slaweq | and according to ip_address comment, it's for https://github.com/openstack/neutron-lib/blob/master/neutron_lib/api/definitions/allowedaddresspairs.py | |
| 12:39:00 | frickler | slaweq: o.k., thanks for confirming, I'll do a cleanup patch | |
| 12:43:05 | slaweq | frickler: thx | |
| 12:59:52 | openstackgerrit | Doug Hellmann proposed openstack-infra/shade master: remove redundant information from release notes build https://review.openstack.org/565214 | |
| 13:08:35 | openstackgerrit | Jens Harbott (frickler) proposed openstack/openstacksdk master: Drop bogus attributes from network port resource https://review.openstack.org/565217 | |
| 13:08:45 | frickler | slaweq: FYI ^^ | |
| 13:29:36 | openstackgerrit | Merged openstack/openstacksdk master: Add Limestone Networks vendor info https://review.openstack.org/564728 | |
| 13:33:51 | openstackgerrit | Merged openstack-infra/shade master: Make name setting in connect_as more resilient https://review.openstack.org/564861 | |
| 14:12:53 | openstackgerrit | Merged openstack-infra/shade master: remove redundant information from release notes build https://review.openstack.org/565214 | |
| 20:05:03 | gtmanfred | mordred: you around to talk about https://storyboard.openstack.org/#!/story/2001794 ? | |
| #openstack-sdks - 2018-05-01 | |||
| 00:37:52 | gtmanfred | ok, maybe tomorrow | |
| 01:13:02 | openstackgerrit | Mark Kirkwood proposed openstack/python-openstackclient master: Add an option to container create for storage policy https://review.openstack.org/564665 | |
| 03:05:19 | mnaser | it seems like providing 'key_manager_endpoint_override' doesn't seem to properly work in a connection? | |
| 04:59:46 | openstackgerrit | XiaojueGuan proposed openstack/keystoneauth master: Trivial: Update pypi url to new url https://review.openstack.org/565418 | |
| 10:36:17 | mordred | mnaser: no? | |
| 10:36:54 | lifeless | mordred: ETIMEZONE | |
| 10:38:52 | mordred | lifeless: so much E | |
| 10:38:58 | mordred | wait - that implies the wrong thing :) | |
| 10:39:15 | mordred | lifeless: also, o hai | |
| 11:00:14 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Defer all endpoint discovery to keystoneauth https://review.openstack.org/564494 | |
| 11:45:00 | mnaser | mordred: I’ll paste what I have that’s not working for me. By looking over the code it seems that cloud region get_endpoint_session() uses the override however the normal adapter doesn’t seem to use it | |
| 11:45:35 | mnaser | mordred: use case here is using sensu-plugins-python + openstacksdk for small integration monitoring tests | |
| 12:19:39 | mordred | mnaser: seems like a good use case - I'm looking through the code to see if I can see where endpoint_override isn't getting passed ... | |
| 12:20:11 | mordred | mnaser: ah. there it is. it's in the part where it's not getting passed | |
| 12:22:17 | mnaser | mordred: currently doing this.. but it also feels a little dirty, i could be doing it the wrong way | |
| 12:22:26 | mnaser | i didnt find a way to just connect to a region with overrides | |
| 12:22:34 | mordred | yah - there'sa bug - patch coming | |
| 12:23:01 | mnaser | (the reason i'm overriding is because i want to test the *local* instance using that monitoring agent rather than going through load balancer, etc) | |
| 12:23:03 | mnaser | and cool thank you! | |
| 12:24:30 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Honor endpoint_override for get_session_client https://review.openstack.org/565489 | |
| 12:24:38 | mordred | mnaser: ^^ | |
| 12:24:55 | mnaser | yay | |
| 12:24:59 | mnaser | let me try and test that locally | |
| 12:29:27 | mnaser | mordred: your patch is missing a , after 'endpoint_override=self.get_endpoint(service_key)' | |
| 12:29:37 | mnaser | (before the **kwargs) | |
| 12:30:05 | mnaser | and it also fixes the issue-- CheckBarbicanIntegration CRITICAL: Unable to establish connection to http://foobar/secrets: HTTPConnectionPool(host='foobar', port=80): Max retries exceeded with url: /secrets (Caused by NewConnectionError(' |
|
| 12:32:22 | mordred | mnaser: whoops | |
| 12:32:51 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Honor endpoint_override for get_session_client https://review.openstack.org/565489 | |
| 12:35:20 | mordred | Shrews: ^^ if you get a sec | |
| 14:15:18 | mordred | Shrews: you may also enjoy https://review.openstack.org/549307 | |
| 14:15:40 | mordred | slaweq: ^^ you may also enjoy both of those patches (or you might not :) ) | |
| 14:15:54 | slaweq | mordred: looking | |
| 14:15:55 | slaweq | :) | |
| 14:57:35 | gtmanfred | mordred: ping | |
| 14:59:09 | mordred | gtmanfred: hi! I'm about to be on a phone call for an hour - but then I'll be here | |
| 14:59:52 | gtmanfred | awesome, thanks | |
| 15:00:49 | mordred | gtmanfred: however, that race condition you mention should not exist on a reasonably modern openstack - we use the create-floaing-ip-with-server-port approach rather than the 'create-floating-ip-then-attach-to-server' approach which suffers from the race condition mentioned | |
| 15:01:37 | mordred | if the cloud in quetsion does not expose the neutron endpoints needed to safely create floating ips in a multi-process or multi-threaded manner, we will fall back to create-and-attach | |
| 15:02:03 | mordred | (this is all just backgound info) | |
| 15:04:00 | gtmanfred | kk | |
| 15:04:55 | gtmanfred | lemme check what they are running, it is older | |
| 15:06:07 | mordred | yah- the pure nova api version of this is ... not great .. for the reasons listed in the story | |
| 15:10:07 | gtmanfred | mordred: he is using mitaka | |
| 15:12:06 | mordred | hrm. that should be new enough | |
| 15:17:44 | gtmanfred | I also think I tested with pike, and was running into the same problem. | |
| 15:18:05 | gtmanfred | what it seems like the problem is, is that we are trying to use floating ips that have already been allocated | |
| 15:18:30 | gtmanfred | and since multiple processes list all the available ips, and then pick one, multiple processes try to assign the same ip | |
| 15:19:38 | gtmanfred | this is why we went with `create and assign by default. if there are no free ips, log a warning that we are going to be not multiprocess safe, because we need to list available ones and pick one from a list` | |
| 15:23:08 | openstackgerrit | Hongbin Lu proposed openstack/openstacksdk master: Add 'port_details' to Floating IP https://review.openstack.org/533811 | |
| 15:23:39 | mordred | yah. multi-process re-use of pre-existing floating ips is basically unpossible - without some sort of shared locking/brokering system | |
| 15:24:32 | mordred | in nodepool (which does this process in many many parallel threads) - we make sure we delete floating ips when we delete servers - and also have a cleanup thread that deletes any unattached floating ips it finds | |
| 15:25:13 | mordred | since if the create-with-server-port method is being used, there is no such thing as a validly unattached floating ip | |
| 15:25:27 | gtmanfred | ok, i think what I am just going to do is change the default `reuse_ips` to False, and allow them to set it if they really really really really really really really really are dumb and want to use it | |
| 15:26:25 | openstackgerrit | XiaojueGuan proposed openstack/keystoneauth master: Trivial: Update pypi url to new url https://review.openstack.org/565418 | |
| 15:26:57 | mordred | gtmanfred: yah | |
| 15:27:05 | mordred | gtmanfred: so - that said - there is an option we can do | |
| 15:27:36 | mordred | (it's on my list already = it's just deep down there and there's a few big ticket items we need to likely get done first) | |
| 15:28:46 | mordred | task number 1 is to finish making the caching tier use dogpile.cache everywhere (servers, ports and floating ips are currently done manually if caching is enabled - and there are some tricky interactions as it relates to batched operations and thundering herd that we hvae to be careful about when fixing that) | |
| 15:29:27 | mordred | but once we've got that, then we should be able to add in a locking/multi-process aware TaskManager (similar to the multithreaded TaskManager that's in nodepool right now) | |
| 15:29:43 | mordred | and leverge dogpile for the shared locking | |
| 15:29:47 | gtmanfred | :+1: | |
| 15:30:06 | mordred | so if someone then wants to do multi-process with reuse_ips there will be a story for it | |
| 15:30:06 | gtmanfred | Lemme know what I can help with, i need to figure out what to contribute to so I can go to berlin. | |