| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2018-06-12 | |||
| 17:25:04 | openstackgerrit | David Moreau Simard proposed openstack/openstacksdk master: Add a description field for port_create, port_update https://review.openstack.org/574831 | |
| 17:25:08 | dmsimard | mordred: ^ | |
| 17:25:15 | mordred | dmsimard: woot! thanks | |
| 17:31:44 | vadim | hi, any ideas https://storyboard.openstack.org/#!/story/2002142 ? | |
| 18:00:37 | mordred | vadim: I believe this is fixed in master but not yet released -- looking | |
| 18:02:04 | mordred | vadim: nope. I was wrong | |
| 18:28:55 | openstackgerrit | Nicola Soranzo proposed openstack/cliff master: Build universal wheels https://review.openstack.org/574846 | |
| 19:45:56 | openstackgerrit | Merged openstack/keystoneauth master: raise_exc default in Adapter https://review.openstack.org/574784 | |
| 23:02:20 | mnaser | could we get an openstacksdk release anytime soon? :x | |
| 23:19:29 | openstackgerrit | Sami Makki proposed openstack/python-openstackclient master: Fix the `role implies list` command. https://review.openstack.org/557359 | |
| 23:30:17 | mnaser | mordred: i'm a bit stuck. i'm trying to use os_client_config (openstacksdk seems to have not gotten me very far) to talk to placement api | |
| 23:30:33 | mnaser | `placement = os_client_config.make_rest_client('placement')` and then making any requests | |
| 23:30:47 | mnaser | spits out a traceback | |
| 23:31:03 | mnaser | http://paste.openstack.org/show/723359/ | |
| 23:31:15 | mnaser | if you have *any* few seconds to have a look that'd be great, but i've been stuck for the past hour on this | |
| #openstack-sdks - 2018-06-13 | |||
| 07:55:53 | umbSublime | when creating a connection with openstack.connect, how can I specify an alternate path to the default clouds.yaml file ? | |
| 08:20:43 | frickler | mnaser: mordred: dtroyer: IIUC it would make sense to release OSC and sdk together, after having this merged https://review.openstack.org/533809 | |
| 08:39:20 | openstackgerrit | tianhui proposed openstack/python-openstackclient master: Compute: Add tags support for server https://review.openstack.org/569386 | |
| 08:41:52 | openstackgerrit | tianhui proposed openstack/python-openstackclient master: Compute: Add tags support for server https://review.openstack.org/569386 | |
| 09:39:16 | openstackgerrit | yanpuqing proposed openstack/python-openstackclient master: Modify the argument when setting floating ip other properties https://review.openstack.org/575057 | |
| 10:50:53 | openstackgerrit | yanpuqing proposed openstack/python-openstackclient master: Do not require port argument when updating floating IP https://review.openstack.org/575057 | |
| 12:17:43 | umbSublime | How can I specify an arbitrary cloud config file when using the openstacksdk ? | |
| 12:18:16 | umbSublime | rather "arbitrary path for the cloud config file" | |
| 12:57:28 | mordred | umbSublime: you can either set the env var OS_CLIENT_CONFIG_FILE or you can pass a list of paths to the config_files parameter of openstack.config.OpenStackConfig- so like: config = openstack.config.OpenStackConfig(config_files=['/path/to/file']) ; cloud_region = config.get_one(cloud='my_cloud') ; conn = openstack.connection.Connection(config=cloud_region) | |
| 12:58:27 | mordred | frickler: I'm getting failing osc-tips jobs on https://review.openstack.org/#/c/557935/ but it doesn't make immediate sense to me why - does it make sense to you? | |
| 12:58:58 | mordred | mnaser: ah - interesting. "Skipping over invalid version data. No stability status in version." | |
| 12:59:12 | mordred | cdent: ^^ does placement's version discovery doc not have a status field? | |
| 13:00:03 | mordred | cdent: http://paste.openstack.org/show/723359/ is the paste in question | |
| 13:00:05 | cdent | mordred: apparently not. I can fix that | |
| 13:00:37 | cdent | what is it supposed to say. or rather, which of your tomes tells me what to do? | |
| 13:01:07 | cdent | nm, this seems good enough: http://specs.openstack.org/openstack/api-wg/guidelines/microversion_specification.html#version-discovery | |
| 13:01:41 | mordred | cdent: yes - that would be ideal | |
| 13:02:06 | mordred | mnaser: in the mean time, try setting placement_endpoint_override to your placement endpoint | |
| 13:02:19 | cdent | hmmm. we've always had a id, {min,max}_version, but never a status | |
| 13:02:55 | cdent | what's different from what mnaser is doing and what what the scheduler report client (which uses ksa) is doing? | |
| 13:03:09 | mordred | cdent: when keystoneauth does version discovery it filters out statuses that are not CURRENT or SUPPORTED unless you provide a flag | |
| 13:03:28 | mordred | cdent: I'm not 100% sure yet why it's working in nova | |
| 13:04:26 | mordred | cdent: where can I look at the scheduler report client? | |
| 13:04:40 | cdent | nova/scheduler/client/report.py | |
| 13:06:34 | mordred | cdent: weird. I'll have to dig a bit more to figure it out | |
| 13:06:38 | umbSublime | awesome thanks mordred | |
| 13:07:49 | cdent | mordred: it may be in the way nova.utils.get_ksa_adapter is happening | |
| 13:08:01 | cdent | but in any case, I'll add a status | |
| 13:08:41 | mordred | cdent: something something API-SIG recommendations something something | |
| 13:08:44 | mordred | :) | |
| 13:09:20 | cdent | indeed, but also, something something someone ate my clones something something | |
| 13:10:01 | mordred | darned clone eaters | |
| 13:10:07 | mordred | I made you such nice ones just the other day | |
| 13:12:31 | cdent | such brief candles | |
| 13:14:56 | mordred | out. out | |
| 13:21:18 | openstackgerrit | Pavlo Shchelokovskyy proposed openstack/python-openstackclient master: Skip calls to glance and nova when got no servers https://review.openstack.org/568344 | |
| 13:21:19 | openstackgerrit | Pavlo Shchelokovskyy proposed openstack/python-openstackclient master: Add --name-lookup option to server list https://review.openstack.org/568345 | |
| 13:24:21 | openstackgerrit | Graham Hayes proposed openstack/os-api-ref master: Use 'sphinx.util.logging' https://review.openstack.org/563179 | |
| 13:37:09 | cdent | mordred: help me believe that the version discovery doc is not itself microversioned. If it is I'm going to gouge my eyes out | |
| 13:37:25 | cdent | and collapse into a logical black hole | |
| 13:40:12 | mordred | cdent: it is, in fact, not microversioned | |
| 13:40:20 | cdent | bless you | |
| 13:40:54 | mordred | cdent: and I contend that any changes to it that align it more with the API-SIG guidelines are not breaking API changes | |
| 13:41:43 | cdent | as would I | |
| 13:56:46 | mnaser | mordred: thanks. `placement = os_client_config.make_rest_client('placement', placement_endpoint_override='https://placement-ca-ymq-1.vexxhost.net')` doesn't seem to do the trick however? | |
| 13:58:41 | mordred | mnaser: well that's annoying | |
| 13:59:12 | mnaser | mordred: my wip is http://paste.openstack.org/show/723387/ | |
| 13:59:33 | mordred | mnaser: I'm unfortunately in a morning of meetings - but I'm very interested in helping to solve this for real -so if I go dark for chunks of time I haven't forgotten about you | |
| 13:59:44 | mnaser | i tried openstacksdk and i was running into the same issues too, alongside is_public not being something changable (that has since been fixed but not released) | |
| 13:59:58 | mnaser | mordred: oh don't worry about it, as async as you want, im not in a total rush about this | |
| 14:02:33 | mordred | mnaser: fwiw, I can reproduce it locally: | |
| 14:02:35 | mordred | >>> import openstack | |
| 14:02:37 | mordred | >>> c=openstack.connect(cloud='vexxhost') | |
| 14:02:39 | mordred | >>> c.placement.get('/allocation_candidates') | |
| 14:02:55 | mordred | (give me same traceback) | |
| 14:03:41 | mnaser | mordred: at least it's not me doing something wrong, yay. if there's anything on our side to do, i can look into it but this is a queens deployment | |
| 14:04:10 | mordred | mnaser: how hard would it be for you to cherry-pick https://review.openstack.org/575117 onto your nova? | |
| 14:05:07 | mnaser | mordred: not very, but i'd be much happier doing it if that type of thing gets backported | |
| 14:05:33 | mnaser | like: if you want a cherry pick to see how it looks like with no guarantee that it will stay there (because we will redeploy from upstream stable and the cherry pick will disappear) | |
| 14:06:05 | mordred | cdent: what do you think the chances are we convince anyone to cherry-pick your patch back to stable/queens? | |
| 14:06:05 | cdent | mnaser: that _will_ get back ported | |
| 14:06:09 | cdent | jinx | |
| 14:06:12 | mordred | _awesome_ | |
| 14:06:14 | mnaser | cool | |
| 14:06:23 | mnaser | this seems to be relatively low touch | |
| 14:06:25 | mnaser | lets break stuff now | |
| 14:07:54 | mnaser | i guess there will be a merge conflict because it looks like placement exists in `nova/api/openstack/placement/` in queens but we can take care of that easily in the backport | |
| 14:11:18 | mnaser | mordred, cdent: https://review.openstack.org/#/c/575117/1 cherry-picked and i can see it in effect http://placement-ca-ymq-1.vexxhost.net/ | |
| 14:11:32 | mnaser | i'm still getting a traceback in my code but who knows | |
| 14:11:40 | cdent | same or different? | |
| 14:11:46 | cdent | (traceback and code) | |
| 14:12:27 | mnaser | same code http://paste.openstack.org/show/723387/ giving same traceback | |
| 14:12:38 | cdent | drop the override? | |
| 14:12:51 | mnaser | oh right | |
| 14:14:31 | cdent | I'm concered confused by why the override didn't work, but I'm struggling to speculate because of lack of familiarity with the tools | |
| 14:15:48 | mnaser | i cant even get my `clouds.yaml` running properly | |
| 14:17:00 | mnaser | ok no my clouds.yaml is fine | |
| 14:17:30 | mnaser | http://paste.openstack.org/show/723390/ gives me http://paste.openstack.org/show/723391/ | |
| 14:17:43 | mnaser | (the nova print flavors parts works fine) | |
| 14:17:55 | openstackgerrit | Graham Hayes proposed openstack/os-api-ref master: General overhaul of testing setup https://review.openstack.org/575124 | |
| 14:23:26 | cdent | mnaser: i guess something is having trouble understanding how to define auth handling durig make_rest_client, but again, I've got no insight into that code. | |
| 14:24:00 | mnaser | cdent: yeah i tried digging into it but it is way beyond me :( it's not a fun user experience but then again not many people interact with the placement api directly i assume | |
| 14:24:44 | cdent | as far as I can tell the problems you're experience aren't because of placement itself, it is something about how the libraries are trying to contact it | |
| 14:24:51 | cdent | placement itself is way simple | |
| 14:25:09 | cdent | if curl to placement with a valid keystone token, it will just work | |