| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2018-04-05 | |||
| 17:21:29 | elmiko | mordred: sweeet! i'm all for a good nerd-sniping ;) | |
| 17:21:41 | mrhillsman | https://github.com/theopenlab/cicd-howto | |
| 17:21:44 | elmiko | that sounded so wrong... | |
| 17:21:51 | mrhillsman | all we have to do is get the github app installed | |
| 17:21:53 | mnaser | mordred: is that a bug that things like load-balancer or object-store or placement are returning an empty list? | |
| 17:21:59 | mrhillsman | ready to make it happen right now? | |
| 17:22:03 | mordred | mnaser: it's not a bug in the script :) | |
| 17:22:10 | mrhillsman | i can work with you now to get it setup | |
| 17:22:17 | mordred | mnaser: neither load-balancer or object-store have version discovery documents | |
| 17:22:20 | mnaser | oh okay | |
| 17:22:20 | mnaser | so is our service registry wrong | |
| 17:22:37 | mnaser | http://load-balancer-ca-ymq-1.vexxhost.net/ octavia does :> | |
| 17:22:38 | mordred | mnaser: I believe johnsom may have already fixed octavia in master? | |
| 17:22:49 | mnaser | except you cant access v1, because its meant to be accessed only by neutron | |
| 17:22:52 | mordred | mnaser: it does? awesome - lemme figure out what went wrong with it then | |
| 17:22:52 | mnaser | so technically its v2 only | |
| 17:22:58 | mnaser | also wait | |
| 17:23:08 | mnaser | is it returning json wrapped in xml? | |
| 17:24:21 | mordred | mnaser: looks fine when I do it manually: | |
| 17:24:23 | mordred | get('http://load-balancer-ca-ymq-1.vexxhost.net/').content | |
| 17:24:25 | mordred | b'{"versions": [{"status": "SUPPORTED", "updated": "2014-12-11T00:00:00Z", "id": "v1"}, {"status": "CURRENT", "updated": "2017-06-22T00:00:00Z", "id": "v2.0"}]}' | |
| 17:24:33 | mnaser | mordred: ok no thats just because the browser is probably has a weird accept: clause | |
| 17:24:53 | mnaser | using accept: application/xml, it retuns <result>{json}</result> | |
| 17:24:59 | dtantsur | mrhillsman: thanks! what's your time zone? it's a bit late here | |
| 17:25:10 | johnsom | I haven't tackled the version discovery document issue yet. It dropped off my radar. | |
| 17:25:15 | mrhillsman | US Central | |
| 17:26:19 | johnsom | mordred I think you were going to open a bug with what we did wrong and/or pointer to an example done right. | |
| 17:26:35 | mordred | OH | |
| 17:26:42 | mordred | I remember what's wrong now | |
| 17:26:51 | mordred | mnaser: if you look, you will see there are no urls in that document returned | |
| 17:27:03 | mordred | johnsom: ++ I can do - I guess I should file an actual bug | |
| 17:27:32 | mrhillsman | dtantsur: i created issue in the SDK repo as well as under the OpenLab repo for tracking | |
| 17:27:35 | mnaser | https://github.com/openstack/octavia/blob/master/octavia/api/root_controller.py | |
| 17:27:38 | mnaser | looks like an easy fix :> | |
| 17:27:42 | johnsom | https://storyboard.openstack.org | |
| 17:28:01 | johnsom | Yeah, probably is, just wasn't sure what we needed. | |
| 17:28:29 | dtantsur | mrhillsman: cool! I'll read the docs and ping you agai | |
| 17:28:35 | mrhillsman | ++ | |
| 17:30:10 | mordred | mnaser, cdent: also, I do get this: keystoneauth1.exceptions.connection.ConnectFailure: Unable to establish connection to https://placement-ca-ymq-1.vexxhost.net: HTTPSConnectionPool(host='placement-ca-ymq-1.vexxhost.net', port=443): | |
| 17:30:30 | mnaser | i think its because we don't expose the placement api over our public load balancer | |
| 17:30:34 | mnaser | ..should we? | |
| 17:31:00 | cdent | it depends? | |
| 17:31:03 | mordred | well - if you have registered a public endpoint in your catalog I would - or otherwise just don't put placement into your catalog with a ublic interface | |
| 17:31:13 | cdent | yeah, what mordred says | |
| 17:31:27 | mnaser | oh actually | |
| 17:31:29 | mnaser | its exposed | |
| 17:31:33 | cdent | it does have to be in the service catalog, otherwise nova won't work, but it doesn't need to be public | |
| 17:31:39 | mnaser | someone didn't configure dns | |
| 17:31:42 | mordred | mnaser: \o/ | |
| 17:31:44 | mnaser | curl -H 'Host: placement-ca-ymq-1.vexxhost.net' http://199.204.44.10 | |
| 17:32:05 | mnaser | but it looks like / is protected by keystone, let me fix dns | |
| 17:32:11 | mordred | mnaser: well, when I get this report done - it might wind up being a helpful report for operators - is your cloud actually exposing what you think it is | |
| 17:32:42 | mnaser | mordred: cool if we can get this integrated to the openstack marketplace too | |
| 17:32:42 | mordred | mnaser: yah - I think we chatted about that in the nova channel a little while back - and I made the request that / of placement not be protected | |
| 17:34:11 | mnaser | mordred: ok dns fixed, we have a low 300s ttl so should be available soon | |
| 17:36:34 | mordred | woot | |
| 17:40:51 | dtantsur | mnaser: thanks a lot again! the list-images example in rust-openstack worked, that's already something :) | |
| 17:42:06 | mnaser | woo | |
| 17:50:45 | mnaser | mordred: placement resolving for me now | |
| 17:56:52 | dtantsur | aaand the first bug: listing networks fail :) | |
| 18:01:06 | elmiko | cdent dtantsur, ready to print, https://etherpad.openstack.org/p/api-sig-newsletter | |
| 18:01:26 | dtantsur | mordred: see, the first fruit of multicloud testing: RDO cloud returns created_at in network listing, vexxhost does not :) | |
| 18:01:38 | mordred | dtantsur: \o/ | |
| 18:02:01 | dtantsur | I wonder why, it's not marked as optional in the docs... | |
| 18:02:33 | cdent | elmiko: link to the "mailing sent out by edleafe"? | |
| 18:02:57 | elmiko | cdent: should be there, [8] | |
| 18:03:19 | cdent | oh, I see, maybe ref it again? | |
| 18:03:23 | elmiko | added another reference to it =) | |
| 18:03:25 | cdent | jinx | |
| 18:03:33 | elmiko | haha, next round's on me ;) | |
| 18:04:12 | cdent | looks good to me, but I admit to have brain fail at the moment | |
| 18:04:31 | elmiko | fair, i'll wait for dtantsur to give the second ack | |
| 18:04:41 | elmiko | \o/ | |
| 18:05:27 | dtantsur | elmiko: should we mention that we plan on landing the http update soon(ish)? | |
| 18:05:35 | dtantsur | everything else LGTM | |
| 18:06:00 | elmiko | edleafe is updating now =) | |
| 18:06:17 | edleafe | dtantsur: done | |
| 18:06:21 | dtantsur | ok, then see you all tomorrow :) | |
| 18:06:31 | elmiko | thanks edleafe ! | |
| 18:06:41 | elmiko | we gtg? | |
| 18:07:20 | edleafe | elmiko: off it goes! | |
| 18:08:16 | elmiko | done and done, thanks again =) | |
| 18:15:32 | openstackgerrit | Colleen Murphy proposed openstack/keystoneauth master: add lower-constraints job https://review.openstack.org/555625 | |
| 19:18:53 | openstackgerrit | Monty Taylor proposed openstack/keystoneauth master: Add methods to get all of the version data https://review.openstack.org/559154 | |
| 19:19:20 | mordred | mnaser, elmiko, dtantsur|afk, edleafe, cdent, cmurphy: ^^ with that patch, the report consumption code is now: | |
| 19:19:31 | mordred | c = openstack.connect(cloud='vexxhost') | |
| 19:19:33 | mordred | report = c.config.get_session().get_all_version_data() | |
| 19:19:35 | mordred | print(yaml.dump(report, default_flow_style=False)) | |
| 19:20:01 | mordred | I've tested running it against vexxhost- but have not yet written unittests | |
| 19:20:09 | elmiko | wow, very cool | |
| 19:20:50 | mordred | incidentally, I updated it to include all regions and all interfaces with optional region and interface filters ... | |
| 19:21:11 | mordred | so a cloud admin should be able to run it against their cloud and get a single document that summarizes everything on the cloud | |
| 19:22:14 | mordred | http://paste.openstack.org/show/718522 <-- is updated output including regions and interfaces | |
| 19:31:21 | elmiko | mordred ++ | |
| 19:56:48 | mnaser | mordred: it would be nice to come to terms on deciding about endpoints for cinder | |
| 19:57:01 | mnaser | volume/volumev2/volumev3 which all report all 3 versions | |
| 20:06:57 | mordred | mnaser: yup - agree - that'll be fixed by the next patch I'm working on | |
| 20:22:56 | openstackgerrit | Slawek Kaplonski proposed openstack-infra/shade master: add lower-constraints job https://review.openstack.org/555299 | |
| 20:34:59 | openstackgerrit | Duc Truong proposed openstack/openstacksdk master: Add function examples test for clustering https://review.openstack.org/555762 | |
| 20:51:55 | openstackgerrit | Duc Truong proposed openstack/openstacksdk master: Add function examples test for clustering https://review.openstack.org/555762 | |