| *** shu-mutou has joined #openstack-sdks | 00:22 | |
| *** dave-mccowan has quit IRC | 00:47 | |
| *** chenyb4 has joined #openstack-sdks | 01:08 | |
| *** annp has joined #openstack-sdks | 01:22 | |
| *** dave-mccowan has joined #openstack-sdks | 01:46 | |
| *** edmondsw has quit IRC | 01:51 | |
| *** dave-mccowan has quit IRC | 01:52 | |
| *** thrash is now known as thrash|g0ne | 01:58 | |
| *** dave-mccowan has joined #openstack-sdks | 02:00 | |
| openstackgerrit | Chen proposed openstack/python-openstackclient master: Compute: Add description support for server https://review.openstack.org/568549 | 02:06 |
|---|---|---|
| openstackgerrit | Chen proposed openstack/python-openstackclient master: Compute: Add description support for server https://review.openstack.org/568549 | 02:12 |
| openstackgerrit | Chen proposed openstack/python-openstackclient master: Compute: Add description support for server Part 1 https://review.openstack.org/568549 | 02:31 |
| *** dave-mccowan has quit IRC | 02:37 | |
| *** gildub has joined #openstack-sdks | 02:38 | |
| *** edmondsw has joined #openstack-sdks | 02:56 | |
| *** edmondsw has quit IRC | 03:01 | |
| *** pooja-jadhav is now known as pooja_jadhav | 03:36 | |
| *** Jeffrey4l has quit IRC | 03:48 | |
| *** Jeffrey4l has joined #openstack-sdks | 03:55 | |
| *** bobh has joined #openstack-sdks | 04:25 | |
| *** edmondsw has joined #openstack-sdks | 04:45 | |
| *** bobh has quit IRC | 04:48 | |
| *** bobh has joined #openstack-sdks | 04:48 | |
| *** edmondsw has quit IRC | 04:50 | |
| *** bobh has quit IRC | 04:53 | |
| *** annp has quit IRC | 05:37 | |
| *** annp has joined #openstack-sdks | 05:38 | |
| *** edmondsw has joined #openstack-sdks | 06:33 | |
| *** edmondsw has quit IRC | 06:38 | |
| *** e0ne has joined #openstack-sdks | 07:09 | |
| *** gtema has joined #openstack-sdks | 07:28 | |
| *** markvoelker has quit IRC | 07:55 | |
| *** jpena|off is now known as jpena | 07:58 | |
| *** gtema has quit IRC | 07:59 | |
| *** gtema has joined #openstack-sdks | 08:00 | |
| *** gkadam has joined #openstack-sdks | 08:18 | |
| *** markvoelker has joined #openstack-sdks | 08:56 | |
| dtantsur | mordred: well, I'd argue with the "no additional effort" bit.. at least for me as rust-openstack developer supporting major versions is a bit of a pain | 09:21 |
| *** markvoelker has quit IRC | 09:30 | |
| *** chenyb4 has quit IRC | 10:07 | |
| *** edmondsw has joined #openstack-sdks | 10:10 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/openstacksdk master: baremetal: refuse to inspect associated machines https://review.openstack.org/569375 | 10:12 |
| *** edmondsw has quit IRC | 10:14 | |
| *** lbragstad has quit IRC | 10:15 | |
| *** lbragstad has joined #openstack-sdks | 10:19 | |
| *** shu-mutou has quit IRC | 10:23 | |
| *** markvoelker has joined #openstack-sdks | 10:28 | |
| *** thrash|g0ne is now known as thrash | 10:42 | |
| *** annp has quit IRC | 10:51 | |
| *** markvoelker has quit IRC | 11:00 | |
| openstackgerrit | Chen proposed openstack/python-openstackclient master: Compute: Add description support for server Part 2 https://review.openstack.org/569385 | 11:00 |
| openstackgerrit | tianhui proposed openstack/python-openstackclient master: Compute: Add tags support for server https://review.openstack.org/569386 | 11:00 |
| frickler | dmsimard|off: do you use the default contrib/inventory/openstack.py from the ansible repo or do you have a modified version? I had a similar issue with the former that was resolved in https://github.com/openstack/os-client-config/commit/e65033910714d6f7d198d8eff6997fdaec102474 | 11:15 |
| *** markvoelker has joined #openstack-sdks | 11:15 | |
| *** edmondsw has joined #openstack-sdks | 11:26 | |
| *** jpena is now known as jpena|lunch | 11:49 | |
| mordred | dtantsur: that's fair. I guess what I should have said is that since major versions already exist, clients that want to support the existing openstack installations have to have major version discovery / handling code today anyway - and we have documentation about how to handle all of the cases of it | 11:57 |
| mordred | dmsimard|off, frickler: I think in this case it's an issue with executing the openstack dynamic inventory from a directory containing the inventory with the inventory script still named 'openstack.py' | 11:58 |
| mordred | since it causes os-client-config's import of 'openstack.config' to fail by finding the inventory script as "openstack" instead | 11:58 |
| mordred | dmsimard|off, frickler the solutions for now are either to rename the inventory script before running it, or to switch to the new inventory plugin | 11:59 |
| dmsimard|off | mordred: oh yeah that makes sense | 12:11 |
| dmsimard|off | mordred: I've had something similar happen with a json.py file at some point.. | 12:11 |
| frickler | mordred: hah, good catch, that explains why I couldn't reproduce this. mine is called 10-openstack.py for $reasons | 12:11 |
| openstackgerrit | Monty Taylor proposed openstack/python-openstackclient master: Update command test for volume.v3 https://review.openstack.org/569398 | 12:12 |
| 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:12 |
| 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:14 |
| 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 |
| amotoki | mordred: looking | 12:15 |
| mordred | amotoki: thanks | 12:16 |
| *** EmilienM is now known as EvilienM | 12:21 | |
| dtantsur | jroll: good catch re unit testing! I suspect that for some reason the test is not running at all... | 12:23 |
| jroll | dtantsur: yeah, it's strange :) | 12:23 |
| dtantsur | SKIPPED: Ironic operations not supported yet | 12:24 |
| dtantsur | yay, mordred is it expected for all ironic unit tests to be skipped with ^^^? | 12:24 |
| jroll | O_o | 12:24 |
| *** gildub has quit IRC | 12:25 | |
| * dtantsur suspects an epic fail somewhere | 12:25 | |
| 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:26 |
| 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:29 |
| jroll | mordred: yeah, it just feels heavy | 12:30 |
| jroll | most of this is about getting rid of bad apis or tech debt, right | 12:30 |
| mordred | well - except that getting rid of bad apis is an illusion | 12:30 |
| 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 |
| jroll | ok, ignore "bad apis" for now, we can discuss that bit later :) | 12:31 |
| 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:31 |
| openstackgerrit | Dmitry Tantsur proposed openstack/openstacksdk master: baremetal: refuse to inspect associated machines https://review.openstack.org/569375 | 12:32 |
| 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:32 |
| 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:33 |
| 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 |
| 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:34 |
| mordred | but if we did new majors too often, it would also cause undue burden on people | 12:35 |
| 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 |
| jroll | why are you making so much sense, I want to argue | 12:35 |
| jroll | :) | 12:35 |
| * mordred hands jroll some meth so that he can argue no matter what | 12:35 | |
| dtantsur | :D | 12:36 |
| jroll | lol | 12:36 |
| jroll | I think I'm good with that proposal, though probably needs more thought | 12:36 |
| mordred | jroll: yes. much more thought - and discussion and whatnot | 12:37 |
| openstackgerrit | Dmitry Tantsur proposed openstack/openstacksdk master: Enable bare metal unit tests https://review.openstack.org/569401 | 12:38 |
| dtantsur | mordred: this is quite urgent IMO ^^^ | 12:38 |
| mordred | dtantsur: I agree - +A | 12:39 |
| openstackgerrit | Dmitry Tantsur proposed openstack/openstacksdk master: baremetal: refuse to inspect associated machines https://review.openstack.org/569375 | 12:40 |
| *** jpena|lunch is now known as jpena | 12:44 | |
| 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 | 12:51 |
| *** edleafe is now known as figleaf | 12:59 | |
| dtantsur | I see | 13:00 |
| * dtantsur feels like https://docs.openstack.org/openstacksdk/latest/user/guides/baremetal.html could use more text :D | 13:01 | |
| mordred | dtantsur: much text is needed in many places :) | 13:08 |
| dtantsur | yep :) since I'm doing some work with openstacksdk now, I may be able to write a few paragraphs there | 13:09 |
| dtantsur | (probably not today though) | 13:09 |
| *** chenyb4 has joined #openstack-sdks | 13:09 | |
| *** dtantsur is now known as dtantsur|brb | 13:11 | |
| *** dave-mcc_ has joined #openstack-sdks | 13:18 | |
| *** chenyb4 has quit IRC | 13:27 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/python-openstackclient master: Add --name-lookup option to server list https://review.openstack.org/568345 | 13:46 |
| *** bobh has joined #openstack-sdks | 13:50 | |
| 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 |
| gtema | mordred: Nice comment in the code " | 14:02 |
| gtema | Sending it can cause a policy violation error | 14:02 |
| gtema | # on some clouds. | 14:02 |
| gtema | mordred: " makes me wonder how to fix the situation | 14:02 |
| *** dtantsur|brb is now known as dtantsur | 14:14 | |
| mordred | gtema: it looks to me like it defaults to None... ah | 14:14 |
| openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Allow explicitly setting enable_snat to either value https://review.openstack.org/569422 | 14:16 |
| mordred | gtema: how about that ^^ ? | 14:16 |
| 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:18 |
| gtema | mordred: what is the plan to switch ansible from shade to sdk? | 14:19 |
| mordred | gtema: yes - we'll need the same patch for shade | 14:20 |
| 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:22 |
| openstackgerrit | Monty Taylor proposed openstack-infra/shade master: Allow explicitly setting enable_snat to either value https://review.openstack.org/569425 | 14:23 |
| mordred | gtema: there's the cherry-pick to shade | 14:23 |
| gtema | mordred: thanks | 14:24 |
| gtema | mordred: do you which me to start some work on transition? | 14:24 |
| gtema | mordred: ah those typos: do you wish, that I start? | 14:26 |
| 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:28 |
| 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:29 |
| gtema | mordred: no problem. Kurt will ping you if necessary ;-) | 14:30 |
| 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 |
| mordred | gtema: awesome :) | 14:30 |
| *** dave-mcc_ has quit IRC | 15:15 | |
| *** dave-mccowan has joined #openstack-sdks | 15:18 | |
| *** e0ne has quit IRC | 15:19 | |
| *** knikolla has quit IRC | 15:21 | |
| *** gkadam_ has joined #openstack-sdks | 15:21 | |
| *** knikolla has joined #openstack-sdks | 15:22 | |
| *** jroll is now known as jrollinhatin | 15:23 | |
| *** gkadam has quit IRC | 15:24 | |
| *** thrash is now known as thrash|biab | 15:40 | |
| *** dave-mcc_ has joined #openstack-sdks | 15:42 | |
| *** dave-mccowan has quit IRC | 15:43 | |
| *** melwitt is now known as jgwentworth | 15:49 | |
| *** lbragstad is now known as elbragstad | 15:50 | |
| *** dave-mcc_ has quit IRC | 16:17 | |
| *** e0ne has joined #openstack-sdks | 16:26 | |
| *** e0ne has quit IRC | 16:32 | |
| *** e0ne has joined #openstack-sdks | 16:33 | |
| openstackgerrit | Artem Goncharov proposed openstack/openstacksdk master: Allow explicitly setting enable_snat to either value https://review.openstack.org/569422 | 16:39 |
| *** dtantsur is now known as dtantsur|afk | 16:40 | |
| *** gkadam_ has quit IRC | 16:44 | |
| *** gtema has quit IRC | 17:07 | |
| *** jpena is now known as jpena|off | 17:07 | |
| *** thrash|biab is now known as thrash | 17:13 | |
| *** e0ne has quit IRC | 18:10 | |
| *** dave-mccowan has joined #openstack-sdks | 18:20 | |
| *** bobh has quit IRC | 18:21 | |
| *** bobh has joined #openstack-sdks | 18:22 | |
| *** bobh has quit IRC | 18:26 | |
| openstackgerrit | Joshua Harlow proposed openstack-infra/shade master: Add 'on_chunk_cb' param to list_servers https://review.openstack.org/569491 | 18:42 |
| openstackgerrit | Monty Taylor proposed openstack/os-client-config master: Turn osc-functional-devstack-tips to nonvoting https://review.openstack.org/569492 | 18:42 |
| openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Turn osc-functional-devstack-tips to nonvoting https://review.openstack.org/569493 | 18:45 |
| mordred | dhellmann: https://review.openstack.org/#/c/569398/ is green after your recheck | 18:46 |
| mordred | dhellmann: well, it's green on voting jobs | 18:46 |
| 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: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 | 19:25 |
| *** kuzko has quit IRC | 19:36 | |
| *** kuzko has joined #openstack-sdks | 19:38 | |
| *** e0ne has joined #openstack-sdks | 19:55 | |
| 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:08 |
| 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:17 |
| *** e0ne has quit IRC | 20:30 | |
| openstackgerrit | Merged openstack/python-openstackclient master: Update command test for volume.v3 https://review.openstack.org/569398 | 20:43 |
| 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:17 |
| *** slaweq has quit IRC | 21:20 | |
| *** slaweq has joined #openstack-sdks | 21:20 | |
| *** slaweq has quit IRC | 21:20 | |
| *** slaweq has joined #openstack-sdks | 21:21 | |
| *** dave-mccowan has quit IRC | 21:25 | |
| *** slaweq has quit IRC | 21:26 | |
| *** dave-mccowan has joined #openstack-sdks | 21:35 | |
| *** figleaf is now known as edleafe | 21:40 | |
| *** dtroyer has quit IRC | 21:47 | |
| *** dtroyer has joined #openstack-sdks | 21:47 | |
| *** dave-mccowan has quit IRC | 21:52 | |
| *** slaweq has joined #openstack-sdks | 22:11 | |
| *** slaweq has quit IRC | 22:16 | |
| *** edmondsw has quit IRC | 22:54 | |
| *** d0ugal_ has joined #openstack-sdks | 23:12 | |
| *** d0ugal has quit IRC | 23:13 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!