| *** gtema has joined #openstack-sdks | 01:24 | |
| *** gtema has quit IRC | 01:28 | |
| *** whoami-rajat has joined #openstack-sdks | 02:38 | |
| *** ricolin has joined #openstack-sdks | 02:51 | |
| *** ricolin_ has joined #openstack-sdks | 03:35 | |
| *** ricolin has quit IRC | 03:38 | |
| *** Luzi has joined #openstack-sdks | 05:06 | |
| *** ricolin__ has joined #openstack-sdks | 06:41 | |
| *** ricolin__ is now known as ricolin | 06:41 | |
| *** ricolin_ has quit IRC | 06:45 | |
| *** ralonsoh has joined #openstack-sdks | 06:54 | |
| *** slaweq has joined #openstack-sdks | 07:04 | |
| *** tosky has joined #openstack-sdks | 07:19 | |
| *** gkadam has joined #openstack-sdks | 07:26 | |
| *** jangutter has joined #openstack-sdks | 07:31 | |
| openstackgerrit | Merged openstack/openstacksdk master: Fix DeprecationWarning for using logger.warn https://review.opendev.org/674401 | 07:57 |
|---|---|---|
| *** e0ne has joined #openstack-sdks | 08:48 | |
| openstackgerrit | Bence Romsics proposed openstack/python-openstackclient master: Add command: router add/remove route --route https://review.opendev.org/674325 | 08:58 |
| *** Luzi has quit IRC | 09:43 | |
| *** dasp has quit IRC | 09:49 | |
| *** dasp has joined #openstack-sdks | 09:49 | |
| *** cdent has joined #openstack-sdks | 09:49 | |
| *** Luzi has joined #openstack-sdks | 09:56 | |
| openstackgerrit | Vishakha Agarwal proposed openstack/openstacksdk master: Add application credential CRUD support https://review.opendev.org/669331 | 10:18 |
| *** gkadam has quit IRC | 11:32 | |
| *** gkadam has joined #openstack-sdks | 11:49 | |
| *** gtema has joined #openstack-sdks | 12:05 | |
| gtema | dtantsur, mordred, Shrews: https://review.opendev.org/#/c/674333/ is really important (event as a hotfix), since without it Ansible can't get/create keypairs | 12:10 |
| gtema | dtantsur, can you please have a look, whether ironic failure in the tests is related or not | 12:10 |
| gtema | this change also includes switching Ansible tests to Py3, since the problem is visible only there and more and more Platforms are delivering py3 only ansible | 12:11 |
| *** gtema has quit IRC | 12:16 | |
| *** gtema has joined #openstack-sdks | 12:18 | |
| *** gtema has quit IRC | 12:23 | |
| *** gtema has joined #openstack-sdks | 12:25 | |
| *** gtema has quit IRC | 12:29 | |
| *** jroll has quit IRC | 12:39 | |
| *** jroll has joined #openstack-sdks | 12:39 | |
| *** gtema has joined #openstack-sdks | 12:53 | |
| *** gtema has quit IRC | 12:55 | |
| *** goldyfruit has joined #openstack-sdks | 13:06 | |
| *** goldyfruit has quit IRC | 13:11 | |
| *** mriedem has joined #openstack-sdks | 13:14 | |
| mordred | dtantsur|afk, Shrews: yeah - I recommend looking at that patch from gtema - and when it lands let's cut a bugfix release with it | 13:21 |
| mordred | being broken for ansible on python3 is a bad thing | 13:22 |
| mordred | should we add a second run of the ansible tests to cover python2? | 13:22 |
| efried | mordred: should I nack https://review.opendev.org/#/c/674515/ on the theory that that placement placeholder will be in place soon? | 13:26 |
| *** Luzi has quit IRC | 13:46 | |
| *** gtema has joined #openstack-sdks | 14:08 | |
| gtema | mordred: yeah, adding ansible py2 would be likely good | 14:09 |
| mordred | efried: yeah - I'll get you one today | 14:15 |
| *** gtema has quit IRC | 14:20 | |
| openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Add empty placement proxy https://review.opendev.org/674620 | 14:22 |
| mordred | efried: there's the first stab at it | 14:22 |
| openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Add empty placement proxy https://review.opendev.org/674620 | 14:23 |
| *** gtema has joined #openstack-sdks | 14:26 | |
| *** gtema has quit IRC | 14:28 | |
| *** mriedem has quit IRC | 14:29 | |
| *** mriedem has joined #openstack-sdks | 14:29 | |
| efried | \o/ Thanks mordred | 14:31 |
| *** goldyfruit has joined #openstack-sdks | 15:26 | |
| *** ralonsoh has quit IRC | 15:29 | |
| Shrews | dtantsur|afk: FYI, looks like the asserts in openstack.tests.functional.baremetal.test_baremetal_node compare lists (node.traits) in unsorted order and randomly fail (see https://logs.opendev.org/33/674333/3/check/openstacksdk-functional-devstack-ironic/0e8be8d/testr_results.html.gz) | 15:38 |
| Shrews | mordred: do you remember who added magnum support to sdk? | 15:39 |
| mordred | Shrews: I do not | 15:40 |
| Shrews | oh, maybe yolanda | 15:40 |
| mordred | oh - yeah - I think so | 15:41 |
| openstackgerrit | David Shrewsbury proposed openstack/openstacksdk master: Fix bm tests: sort lists being compared https://review.opendev.org/674640 | 15:55 |
| Shrews | dtantsur|afk: ^^ | 15:55 |
| mordred | Shrews: ++ | 16:05 |
| corvus | when i use the interactive form of the cli, i get this exception: http://paste.openstack.org/show/755535/ | 16:14 |
| corvus | same command works fine non-interactively | 16:14 |
| *** e0ne has quit IRC | 16:19 | |
| mordred | dtroyer: ^^? | 16:27 |
| *** gtema has joined #openstack-sdks | 16:29 | |
| frickler | seems related to cmd2>=0.9.0, if I install cmd==0.8.9 things work fine | 16:30 |
| mordred | frickler: "neat" | 16:30 |
| frickler | the newer ones also give errors when doing interactive mode with options, like "openstack --os-cloud mycloud", which also works fine with older cmd2 | 16:33 |
| *** gtema has quit IRC | 16:33 | |
| dtroyer | yeah, we've not kept up with cmd2… I thought we capped it somewhere? | 16:38 |
| mordred | frickler: so should we be pinning cmd2 for now? | 16:38 |
| mordred | dtroyer: jinx | 16:38 |
| dtroyer | maybe not in stable branches, or far enough back in them? | 16:39 |
| mordred | corvus: what version of osc are you using there? | 16:40 |
| corvus | openstack 3.19.0 | 16:41 |
| mordred | corvus: and, if you don't mind, what version of cliff is installed in that venv? | 16:43 |
| corvus | cliff==2.15.0 | 16:44 |
| mordred | thanks | 16:45 |
| frickler | I tested some older versions of cliff with no effect, versions 0.9.x of cmd2 give varying errors, so based on that short test list, I'd advise to cap cmd2<0.9 for now | 16:47 |
| mordred | frickler: I don't know why it's not capped | 16:52 |
| mordred | OH | 16:52 |
| mordred | it's because I can't read | 16:53 |
| openstackgerrit | Monty Taylor proposed openstack/cliff master: Pin cmd2 on python3 as well https://review.opendev.org/674652 | 16:54 |
| mordred | frickler, dtroyer: ^& | 16:54 |
| *** ricolin has quit IRC | 16:57 | |
| efried | mordred: This [1] is likely because of the way we're (not) fixturing [2], which was pretty weak to begin with. I suspect I can get moving by adding a mock for get_endpoint_from_catalog, and then possibly also get_session_client... but I'm kinda not sure I should keep chasing fixtures for sdk internals like that. Thoughts? | 17:01 |
| efried | [1] https://logs.opendev.org/25/674625/1/check/nova-tox-functional-py36/27b16b6/testr_results.html.gz | 17:01 |
| efried | [2] https://opendev.org/openstack/nova/src/branch/master/nova/tests/fixtures.py#L2125 | 17:01 |
| mordred | efried: hrm. you're making my brain do work | 17:13 |
| mordred | efried: I agree - I don't think you should need to chase fixtures for internals lke that | 17:13 |
| efried | I did open a story for sdk to publish fixtures for this purpose, but people are busy | 17:14 |
| mordred | yeah. and I really want to publish those | 17:14 |
| efried | perhaps initially it's as simple as moving the one from nova into sdk, and it can get better refined later. | 17:14 |
| mordred | efried: maybe for today just adding in a get_endpoint_from_catalog mock - but let's elevate the priority of getting you a real fixture | 17:14 |
| efried | okay, I'll try that on the nova test patch and see what shakes out. | 17:15 |
| mordred | efried: cool. and I'll see if I can sort out something WRT workable long-term exported fixture status | 17:15 |
| efried | thanks mordred | 17:16 |
| *** goldyfruit has quit IRC | 17:18 | |
| *** goldyfruit_ has joined #openstack-sdks | 17:18 | |
| openstackgerrit | Merged openstack/openstacksdk master: Enable ansible module test for keypair to check return data https://review.opendev.org/674333 | 17:24 |
| tosky | mordred: hi, is there any update on version handling in SDK (and then microversioning in osc)? I happened to discuss about https://review.opendev.org/#/c/590807/ a few days ago with some cinder developers | 17:28 |
| mordred | tosky: well - sdk handles versions and microversions - but I think to take advantage of that in osc we need to make progress on osc4 | 17:58 |
| mordred | and then start using sdk methods instead of python-cinderclient methods for cinder operations - it's not going to be a QUICK process | 17:58 |
| tosky | mordred: so it does not match that "a couple of weeks" from that comment, I guess? Can you share there (or some public place) what is missing? Because this was one of the blockers for cinder to even start considering OSC as proper replacement | 18:01 |
| tosky | or so I understood | 18:01 |
| tosky | and maybe other people could help with the porting effort | 18:02 |
| mordred | tosky: I just replied on the change - I think we need to circle with dtroyer on plans here - but yes, I totally agree it would be best if we get people to help with the porting effort | 18:02 |
| *** e0ne has joined #openstack-sdks | 18:05 | |
| tosky | mordred: slightly different question - I noticed that the openstackclient functional tests implements their own base classes; have you considered using CLIClient from tempest.lib? | 18:09 |
| tosky | It can be used without the rest of tempest; that's what the python-cinderclient functional tests do (stestr run and that's it) | 18:10 |
| *** tosky has quit IRC | 18:32 | |
| *** e0ne has quit IRC | 18:45 | |
| *** cdent has quit IRC | 18:59 | |
| openstackgerrit | Eric Fried proposed openstack/openstacksdk master: Allow limiting Connection service_types from oslo.config https://review.opendev.org/674675 | 19:01 |
| efried | mordred: howza ^ | 19:01 |
| *** e0ne has joined #openstack-sdks | 19:16 | |
| *** e0ne has quit IRC | 19:31 | |
| mordred | efried: awesome | 21:14 |
| *** mriedem has quit IRC | 21:19 | |
| *** mriedem has joined #openstack-sdks | 21:21 | |
| openstackgerrit | Merged openstack/openstacksdk master: Fix bm tests: sort lists being compared https://review.opendev.org/674640 | 21:43 |
| *** whoami-rajat has quit IRC | 21:58 | |
| *** tosky has joined #openstack-sdks | 22:26 | |
| *** tosky has quit IRC | 22:47 | |
| openstackgerrit | Christian Berendt proposed openstack/openstacksdk master: Update betacloud vendor entry https://review.opendev.org/674715 | 23:53 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!