Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2019-08-02
20:11:53 mordred you know - how about I go read the docs and then ask you follow up questions
20:12:15 efried mordred: And especially if you discover holes in the docs that don't answer these questions, let us know (or open stories or whatever)
20:12:50 efried mordred: cdent has been in charge of this kind of stuff, so I'm pretty sure we're following all the rules wrt discovery documents and endpoint construction and whatnot.
20:13:03 mordred cool
20:13:19 efried like, I'm 95% sure you won't find snowflake behavior
20:15:18 efried {"versions": [{"status": "CURRENT", "min_version": "1.0", "max_version": "1.36", "id": "v1.0", "links": [{"href": "", "rel": "self"}]}]}
20:15:18 efried efried@canuc:~$ curl http://192.168.218.27/placement
20:15:31 efried that href="" is suspicious
20:22:11 efried mordred: Poking at my devstack, it is looking like we do not have versioned endpoints for placement.
20:22:26 efried So forget what I said about cdent, and let's burn him at the stake.
21:54:25 mordred efried: that's fine - I've already got sdk provisions for this sort of thing :)
#openstack-sdks - 2019-08-03
07:09:00 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Enable ansible module test for keypair to check return data https://review.opendev.org/674333
18:59:53 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Enable ansible module test for keypair to check return data https://review.opendev.org/674333
19:18:40 openstackgerrit Merged openstack/openstacksdk master: Add CloudRegion helper method for arbitrary endpoints https://review.opendev.org/674130
19:20:05 openstackgerrit Merged openstack/openstacksdk master: Fix README example for cloud layer https://review.opendev.org/674041
19:20:06 openstackgerrit Merged openstack/openstacksdk master: Move the history lesson into the docs https://review.opendev.org/674045
20:22:38 openstackgerrit Merged openstack/os-client-config master: Add Python 3 Train unit tests https://review.opendev.org/669450
#openstack-sdks - 2019-08-04
12:56:44 openstackgerrit norman shen proposed openstack/python-openstackclient master: support create domain project https://review.opendev.org/674487
#openstack-sdks - 2019-08-05
07:57:13 openstackgerrit Merged openstack/openstacksdk master: Fix DeprecationWarning for using logger.warn https://review.opendev.org/674401
08:58:08 openstackgerrit Bence Romsics proposed openstack/python-openstackclient master: Add command: router add/remove route --route https://review.opendev.org/674325
10:18:34 openstackgerrit Vishakha Agarwal proposed openstack/openstacksdk master: Add application credential CRUD support https://review.opendev.org/669331
12:10:25 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:44 gtema dtantsur, can you please have a look, whether ironic failure in the tests is related or not
12:11:20 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
13:21:40 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:22:01 mordred being broken for ansible on python3 is a bad thing
13:22:17 mordred should we add a second run of the ansible tests to cover python2?
13:26:59 efried mordred: should I nack https://review.opendev.org/#/c/674515/ on the theory that that placement placeholder will be in place soon?
14:09:08 gtema mordred: yeah, adding ansible py2 would be likely good
14:15:26 mordred efried: yeah - I'll get you one today
14:22:17 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Add empty placement proxy https://review.opendev.org/674620
14:22:18 mordred efried: there's the first stab at it
14:23:36 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Add empty placement proxy https://review.opendev.org/674620
14:31:45 efried \o/ Thanks mordred
15:38:12 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:39:34 Shrews mordred: do you remember who added magnum support to sdk?
15:40:51 mordred Shrews: I do not
15:40:58 Shrews oh, maybe yolanda
15:41:03 mordred oh - yeah - I think so
15:55:35 openstackgerrit David Shrewsbury proposed openstack/openstacksdk master: Fix bm tests: sort lists being compared https://review.opendev.org/674640
15:55:43 Shrews dtantsur|afk: ^^
16:05:35 mordred Shrews: ++
16:14:04 corvus when i use the interactive form of the cli, i get this exception: http://paste.openstack.org/show/755535/
16:14:34 corvus same command works fine non-interactively
16:27:10 mordred dtroyer: ^^?
16:30:33 frickler seems related to cmd2>=0.9.0, if I install cmd==0.8.9 things work fine
16:30:44 mordred frickler: "neat"
16:33:20 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:38:50 dtroyer yeah, we've not kept up with cmd2… I thought we capped it somewhere?
16:38:53 mordred frickler: so should we be pinning cmd2 for now?
16:38:56 mordred dtroyer: jinx
16:39:21 dtroyer maybe not in stable branches, or far enough back in them?
16:40:07 mordred corvus: what version of osc are you using there?
16:41:12 corvus openstack 3.19.0
16:43:27 mordred corvus: and, if you don't mind, what version of cliff is installed in that venv?
16:44:48 corvus cliff==2.15.0
16:45:44 mordred thanks
16:47:57 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:52:14 mordred frickler: I don't know why it's not capped
16:52:58 mordred OH
16:53:00 mordred it's because I can't read
16:54:06 openstackgerrit Monty Taylor proposed openstack/cliff master: Pin cmd2 on python3 as well https://review.opendev.org/674652
16:54:10 mordred frickler, dtroyer: ^&
17:01:05 efried [2] https://opendev.org/openstack/nova/src/branch/master/nova/tests/fixtures.py#L2125
17:01:05 efried [1] https://logs.opendev.org/25/674625/1/check/nova-tox-functional-py36/27b16b6/testr_results.html.gz
17:01:05 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:13:26 mordred efried: hrm. you're making my brain do work
17:13:44 mordred efried: I agree - I don't think you should need to chase fixtures for internals lke that
17:14:11 efried I did open a story for sdk to publish fixtures for this purpose, but people are busy
17:14:22 mordred yeah. and I really want to publish those
17:14:46 efried perhaps initially it's as simple as moving the one from nova into sdk, and it can get better refined later.
17:14:51 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:15:19 efried okay, I'll try that on the nova test patch and see what shakes out.
17:15:55 mordred efried: cool. and I'll see if I can sort out something WRT workable long-term exported fixture status
17:16:45 efried thanks mordred
17:24:54 openstackgerrit Merged openstack/openstacksdk master: Enable ansible module test for keypair to check return data https://review.opendev.org/674333
17:28:05 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:58:24 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:44 mordred and then start using sdk methods instead of python-cinderclient methods for cinder operations - it's not going to be a QUICK process
18:01:40 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:50 tosky or so I understood
18:02:10 tosky and maybe other people could help with the porting effort
18:02:26 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:09:49 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:10:18 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)
19:01:14 openstackgerrit Eric Fried proposed openstack/openstacksdk master: Allow limiting Connection service_types from oslo.config https://review.opendev.org/674675
19:01:24 efried mordred: howza ^
21:14:33 mordred efried: awesome
21:43:21 openstackgerrit Merged openstack/openstacksdk master: Fix bm tests: sort lists being compared https://review.opendev.org/674640
23:53:45 openstackgerrit Christian Berendt proposed openstack/openstacksdk master: Update betacloud vendor entry https://review.opendev.org/674715
#openstack-sdks - 2019-08-06
08:24:43 dtantsur Shrews, mordred: there was an existing patch for the traits problem, but thanks for fixing it anyway
10:22:17 openstackgerrit Merged openstack/openstacksdk master: Update betacloud vendor entry https://review.opendev.org/674715
10:22:18 openstackgerrit Merged openstack/openstacksdk master: Add Python 3 Train unit tests https://review.opendev.org/667494
10:32:13 dtantsur mordred: we don't support global_request_id in SDK, do we?
10:36:55 dtantsur also it seems that the Volume support it very basic, right?
12:10:11 mordred dtantsur: I'm not sure what global_request_id is? so probably not - but maybe we do via keystoneauth?
12:11:09 mordred dtantsur: and yeah - the openstack.block_storage layer could use more love for sure.
12:25:46 dtantsur oh, yeah, global_request_id is a keystoneauth thing. I just need to know how to pass it to a Connectin..
12:44:33 mordred dtantsur: ok. I just read the global_request_id stuff in ksa
12:45:08 mordred this is going to be a fun one ... because it's supposed to be set per-request, but the individual requests aren't partcularly exposed

Earlier   Later