Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2018-05-18
12:12:21 openstackgerrit Monty Taylor proposed openstack/python-openstackclient master: Update command test for volume.v3 https://review.openstack.org/569398
12:12:58 mordred dmsimard|off, frickler: I think it just goes to show that the new inventory plugin feature is a much better idea than the old inventory script model :)
12:14:36 mordred dtroyer, dhellmann, amotoki: ^^ the above osc patch is needed to fix osc gate jobs now that the cinder-v3-by-default patch landed
12:15:38 mordred dtroyer, dhellmann, amotoki: I'd love it if y'all could land it - that job not working also breaks occ's gate, and we just got a patch from a saltstack person fixing a regression they're having in production that I'd love to get landed and out the door
12:15:54 amotoki mordred: looking
12:16:00 mordred amotoki: thanks
12:23:25 dtantsur jroll: good catch re unit testing! I suspect that for some reason the test is not running at all...
12:23:42 jroll dtantsur: yeah, it's strange :)
12:24:08 dtantsur SKIPPED: Ironic operations not supported yet
12:24:21 dtantsur yay, mordred is it expected for all ironic unit tests to be skipped with ^^^?
12:24:24 jroll O_o
12:26:59 jroll mordred: totally agree that bumping the major version is the right thing to do - though seems like lots of effort to drop e.g. 1.11 and below, when the max is 1.48. 1.12 would become 2.1, 1.13, would become 2.2, etc, besides the code work it feels like lots of mental/docs overhead
12:29:45 mordred jroll: yes, I agree - I was actually thinking cut the new major at the latest max of the current - and keep the old major around for at least the normal deprecation cycle to give people a chance to catch up - so instead of cutting a v2 at 1.11 - just go ahead and cut a new major at 1.48 - but it's possible that's a bad idea for other reasons I haven't realized yet
12:30:23 jroll mordred: yeah, it just feels heavy
12:30:31 jroll most of this is about getting rid of bad apis or tech debt, right
12:30:50 mordred well - except that getting rid of bad apis is an illusion
12:31:02 jroll so either you do that in small pieces (and have many major versions in the end), or you have to do it all at once to keep it in a single major version bump
12:31:40 jroll ok, ignore "bad apis" for now, we can discuss that bit later :)
12:31:47 mordred I think it should be more about making progress towards a place where the 'default' api people see is nicer than it is about removing things
12:32:26 openstackgerrit Dmitry Tantsur proposed openstack/openstacksdk master: baremetal: refuse to inspect associated machines https://review.openstack.org/569375
12:32:38 mordred the reason I've come to like microversions so much is that I can completely and totally ignore them - and for the most part that's what I do
12:33:23 mordred the problem I have with bumping a min is that it makes caring about microversions cease to be optional - and now microversions become the new major versions and there are 48 of them
12:34:15 mordred I'd rather us add things / change things over a few cycles using microversions until we like where we've got ... and then do a legit new major version based on where we got by iterating over time with the microversions
12:34:51 mordred that way consumers can continue to ignore microversions if they don't care about a specific feature and just program to the current major api version - and when a new major api version comes out then they can start caring about it
12:35:09 mordred but if we did new majors too often, it would also cause undue burden on people
12:35:31 mordred so it's certainly a balancing act that would take thought and likely can't be codified in a policy about 'when' a major is cut
12:35:40 jroll why are you making so much sense, I want to argue
12:35:42 jroll :)
12:36:02 dtantsur :D
12:36:05 jroll lol
12:36:14 jroll I think I'm good with that proposal, though probably needs more thought
12:37:16 mordred jroll: yes. much more thought - and discussion and whatnot
12:38:15 openstackgerrit Dmitry Tantsur proposed openstack/openstacksdk master: Enable bare metal unit tests https://review.openstack.org/569401
12:38:16 dtantsur mordred: this is quite urgent IMO ^^^
12:39:19 mordred dtantsur: I agree - +A
12:40:21 openstackgerrit Dmitry Tantsur proposed openstack/openstacksdk master: baremetal: refuse to inspect associated machines https://review.openstack.org/569375
12:51:34 mordred dtantsur: there's gonna me a small rabbit hole here - we need to land an osc patch that I pushed up this morning to fix the osc gate so that those patches will be green. but I have confidence we can get it done soon
13:00:16 dtantsur I see
13:08:40 mordred dtantsur: much text is needed in many places :)
13:09:05 dtantsur yep :) since I'm doing some work with openstacksdk now, I may be able to write a few paragraphs there
13:09:25 dtantsur (probably not today though)
13:46:42 openstackgerrit Pavlo Shchelokovskyy proposed openstack/python-openstackclient master: Add --name-lookup option to server list https://review.openstack.org/568345
14:02:11 gtema mordred: in my cloud router enable_snat is False by default. But in both openstacksdk and shade it is assumed to be True and therefore can be only switched off.
14:02:23 gtema mordred: Nice comment in the code "
14:02:24 gtema Sending it can cause a policy violation error
14:02:26 gtema # on some clouds.
14:02:54 gtema mordred: " makes me wonder how to fix the situation
14:14:19 mordred gtema: it looks to me like it defaults to None... ah
14:16:28 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Allow explicitly setting enable_snat to either value https://review.openstack.org/569422
14:16:34 mordred gtema: how about that ^^ ?
14:18:01 gtema great. Was also thinking about that, but was confused by this comment. shade will also need the same, since ansible is having same problem
14:19:38 gtema mordred: what is the plan to switch ansible from shade to sdk?
14:20:03 mordred gtema: yes - we'll need the same patch for shade
14:22:09 mordred gtema: there's two parts to the plane. one is to make shade a thin wrapper around openstacksdk like we just did with os-client-config. the other is to update the ansible modules... I think we could start that transition in the modules before the thin-shim layer is done for ansible 2.6 - as shade for ansible 2.5 already has a dependency on openstacksdk, so people should already have it installed
14:23:34 openstackgerrit Monty Taylor proposed openstack-infra/shade master: Allow explicitly setting enable_snat to either value https://review.openstack.org/569425
14:23:41 mordred gtema: there's the cherry-pick to shade
14:24:00 gtema mordred: thanks
14:24:58 gtema mordred: do you which me to start some work on transition?
14:26:07 gtema mordred: ah those typos: do you wish, that I start?
14:28:49 mordred gtema: :) yes - that would be great - although my brain is totally distracted by the summit next week so I may not be able to provide much help until it's over
14:29:16 mordred gtema: if you wnated to start thinking about the modules though, https://github.com/ansible/ansible/blob/devel/lib/ansible/module_utils/openstack.py#L111 is probably a good place to start
14:30:02 gtema mordred: no problem. Kurt will ping you if necessary ;-)
14:30:13 mordred maybe a PR that changes that method to just import openstack and create a connection through sdk - and then update any of the modules that are declaring a minimum shade version or are using the 'shade' return value (usually for exception handling)
14:30:17 mordred gtema: awesome :)
16:39:50 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Allow explicitly setting enable_snat to either value https://review.openstack.org/569422
18:42:24 openstackgerrit Joshua Harlow proposed openstack-infra/shade master: Add 'on_chunk_cb' param to list_servers https://review.openstack.org/569491
18:42:26 openstackgerrit Monty Taylor proposed openstack/os-client-config master: Turn osc-functional-devstack-tips to nonvoting https://review.openstack.org/569492
18:45:31 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Turn osc-functional-devstack-tips to nonvoting https://review.openstack.org/569493
18:46:41 mordred dhellmann: https://review.openstack.org/#/c/569398/ is green after your recheck
18:46:55 mordred dhellmann: well, it's green on voting jobs
19:24:45 openstackgerrit Joshua Harlow proposed openstack-infra/shade master: Switch to iterable version of server listing (and expose iterable method) https://review.openstack.org/569503
19:25:24 openstackgerrit Joshua Harlow proposed openstack-infra/shade master: Switch to iterable version of server listing (and expose iterable method) https://review.openstack.org/569503
20:08:44 openstackgerrit Slawek Kaplonski proposed openstack/python-openstackclient master: Don't display router's is_ha and is_distributed attributes always https://review.openstack.org/567620
20:17:23 openstackgerrit Slawek Kaplonski proposed openstack/python-openstackclient master: Make max_burst_kbps option as optional for bw limit QoS rule https://review.openstack.org/567828
20:43:36 openstackgerrit Merged openstack/python-openstackclient master: Update command test for volume.v3 https://review.openstack.org/569398
21:17:37 openstackgerrit Joshua Harlow proposed openstack-infra/shade master: Switch to iterable version of server listing (and expose iterable method) https://review.openstack.org/569503
#openstack-sdks - 2018-05-19
13:34:18 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Rename service_key to service_type https://review.openstack.org/558049
13:34:18 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Pass default_microversion to adapter constructor https://review.openstack.org/557935
15:48:30 openstackgerrit Merged openstack/openstacksdk master: Enable bare metal unit tests https://review.openstack.org/569401
#openstack-sdks - 2018-05-20
19:30:33 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Plumb additional_user_agent support throughout https://review.openstack.org/569666
21:30:55 openstackgerrit Monty Taylor proposed openstack/os-client-config master: Remove app_name config tests https://review.openstack.org/569673
21:31:13 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Plumb additional_user_agent support throughout https://review.openstack.org/569666
21:38:03 openstackgerrit Monty Taylor proposed openstack/os-client-config master: Add release note for the vendor data regression https://review.openstack.org/569675
22:54:04 openstackgerrit Merged openstack/openstacksdk master: baremetal: refuse to inspect associated machines https://review.openstack.org/569375
22:58:29 openstackgerrit Merged openstack/os-client-config master: fix saltstack breakage introduced in v1.31.0 https://review.openstack.org/569291
23:04:54 openstackgerrit Merged openstack/os-client-config master: Add release note for the vendor data regression https://review.openstack.org/569675
#openstack-sdks - 2018-05-21
03:36:03 openstackgerrit Trent Lloyd proposed openstack/python-openstackclient master: openstack image list: Support for community images https://review.openstack.org/569692
06:25:47 openstackgerrit tianhui proposed openstack/python-openstackclient master: Compute: Add tags support for server https://review.openstack.org/569386
07:48:31 openstackgerrit tianhui proposed openstack/python-openstackclient master: Compute: Add tags support for server https://review.openstack.org/569386
19:50:33 openstackgerrit Matt Smith (_alastor_) proposed openstack/openstacksdk master: Adding block_storage v3 API support https://review.openstack.org/566355
20:27:44 openstackgerrit Joshua Harlow proposed openstack-infra/shade master: Switch to iterable version of server listing (and expose iterable method) https://review.openstack.org/569503
21:49:25 Guest83262 mordred: you around? I tried to catch you after your sdk talk, but you move too fast :)
#openstack-sdks - 2018-05-22
05:55:29 openstackgerrit sonu proposed openstack/openstacksdk master: Fix filter style consistency for keystone assignment API https://review.openstack.org/569924
07:45:01 openstackgerrit Chason Chan proposed openstack/python-openstackclient master: Update the content about Import Format https://review.openstack.org/569931
08:53:37 frickler mordred: another hit on the "import openstack.py" ansible issue in #openstack. maybe a rename of that script in the ansible repo would be helpful?
08:54:17 gtema fricker: I work on PR for 2.6 to switch from shade to openstacksdk
08:54:50 gtema import importlib
08:54:55 gtema sdk = importlib.import_module('openstack')
08:55:44 gtema frickler: ^ ^
11:27:53 openstackgerrit tianhui proposed openstack/python-openstackclient master: Compute: Add tags support for server https://review.openstack.org/569386

Earlier   Later