| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2018-08-22 | |||
| 20:59:51 | TheJulia | dtantsur|afk: I mean, we can always take your change, and run bifrost, we have magical CI things :) | |
| 23:59:15 | mordred | dtantsur|afk, TheJulia: the version discovery document that is in that unit test is not a valid version discovery document | |
| 23:59:44 | mordred | I have a patch coming that fixes it - although I would like to confirm what a single-version version discovery document from ironic looks like | |
| #openstack-sdks - 2018-08-23 | |||
| 00:00:24 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: baremetal: support newer microversions in {get,update,patch}_machine https://review.openstack.org/594242 | |
| 00:00:40 | mordred | TheJulia, dtantsur|afk : ^^ | |
| 00:01:51 | mordred | that is A form of a valid single-version version discovery cdocument ... if ironic's is different we should update the tests to match - and then if keystoneauth doesn't process it we whould update keystoneauth and add a test for it | |
| 00:02:19 | mordred | but I REALLY hope ironic doesn't have a weird single-version version discovery document- the multi-version one is so nice | |
| 00:58:35 | openstackgerrit | Merged openstack/openstackclient master: Update homepage url https://review.openstack.org/536732 | |
| 01:00:26 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Test _alternate_id logic https://review.openstack.org/379594 | |
| 01:05:59 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Add capability to compute.limits object to poll other projects https://review.openstack.org/508968 | |
| 01:08:13 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Base devstack sanity check on os_service_types https://review.openstack.org/533325 | |
| 01:13:53 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Adding block_storage v3 API support https://review.openstack.org/566355 | |
| 04:56:03 | openstackgerrit | Deepak Mourya proposed openstack/python-openstackclient master: Now we can add description for role creation in OSC https://review.openstack.org/484355 | |
| 05:45:05 | openstackgerrit | LIU Yulong proposed openstack/openstacksdk master: Support firewall service for SDK https://review.openstack.org/592303 | |
| 05:54:39 | openstackgerrit | Federico Ressi proposed openstack/os-service-types master: Close descriptor after reading file. https://review.openstack.org/595491 | |
| 07:51:57 | openstackgerrit | lei zhang proposed openstack-infra/shade master: Disable the badge from openstackdocstheme 1.23 https://review.openstack.org/595521 | |
| 08:34:50 | dtantsur | mordred: well, that's what ironic *actually* returns on a /v1 endpoint... so you're now testing something that is not real | |
| 08:37:41 | dtantsur | mordred: an example from my system: http://paste.openstack.org/show/728661/ | |
| 08:39:07 | dtantsur | yes, this kind of bad.. our only clue is 'id': 'v1' | |
| 08:41:18 | openstackgerrit | Dmitry Tantsur proposed openstack/openstacksdk master: Run bifrost integration test jobs https://review.openstack.org/586564 | |
| 09:56:54 | openstackgerrit | Deepak Mourya proposed openstack/python-openstackclient master: Now we can add description for role creation in OSC https://review.openstack.org/484355 | |
| 10:04:03 | openstackgerrit | Dmitry Tantsur proposed openstack/openstacksdk master: [DNM] Testing bifrost CI https://review.openstack.org/595572 | |
| 12:32:07 | dtantsur | TheJulia: it looks like bifrost CI does not pick openstacksdk from source.. I tried ^^^ and it still passes | |
| 12:32:33 | TheJulia | yeah, bifrost is not pulling in latest ansible yet either to leverage it | |
| 12:33:09 | TheJulia | I can look at that in a little bit, one of the folks already proposed a change but it lacked a reno | |
| 12:33:15 | dtantsur | cool | |
| 12:33:25 | TheJulia | After coffee :) | |
| 12:35:58 | Shrews | all things after coffee | |
| 12:37:58 | TheJulia | It is like a rocket, the coffee must be loaded for it to work | |
| 12:38:34 | dtantsur | fair enough :) | |
| 12:56:32 | samueldmq | does anybody know if there's any specific reason we don't support volume resize (extend) in -sdk/shade part? | |
| 13:06:11 | Shrews | samueldmq: i cannot recall any reason. features were added to shade as needed | |
| 13:11:51 | samueldmq | Shrews: cool, I will consider taking a look at that bit, seems an interesting functionality to have | |
| 14:20:37 | openstackgerrit | Merged openstack/service-types-authority master: import zuul job settings from project-config https://review.openstack.org/593705 | |
| 14:24:18 | openstackgerrit | Doug Hellmann proposed openstack/service-types-authority master: fix whitespace from scripted edit https://review.openstack.org/595801 | |
| 14:45:07 | openstackgerrit | Merged openstack/service-types-authority master: fix whitespace from scripted edit https://review.openstack.org/595801 | |
| 14:50:40 | TheJulia | so for clarification since I've been out of the loop, current shade just wraps openstacksdk in case there is still intermediate code? | |
| 15:01:58 | Shrews | TheJulia: not quite yet | |
| 15:02:11 | Shrews | i think that's the next thing tbd | |
| 15:02:25 | TheJulia | joy | |
| 15:02:28 | TheJulia | okay | |
| 15:06:58 | samueldmq | Shrews: why does container get return a dict with headers rather than an object? | |
| 15:06:59 | samueldmq | http://paste.openstack.org/show/728691/ | |
| 15:07:14 | samueldmq | #link https://github.com/openstack/openstacksdk/blob/master/openstack/cloud/openstackcloud.py#L7280 | |
| 15:07:42 | samueldmq | Shrews: this is odd. I can't even do container.name because it's a dict, differently from all other objects in openstackcloud | |
| 15:07:44 | samueldmq | is that a bug? | |
| 15:09:34 | Shrews | samueldmq: likely a bug | |
| 15:09:59 | samueldmq | yeah, it does self._container_cache[name] = container.headers | |
| 15:10:08 | samueldmq | and then the container object is just headers lol | |
| 15:10:38 | samueldmq | Shrews: I'll open a bug against that... | |
| 15:47:05 | openstackgerrit | Dmitry Tantsur proposed openstack/openstacksdk master: Run bifrost integration test jobs https://review.openstack.org/586564 | |
| 15:47:32 | openstackgerrit | Dmitry Tantsur proposed openstack/openstacksdk master: [DNM] Testing bifrost CI https://review.openstack.org/595572 | |
| 15:51:59 | openstackgerrit | Merged openstack/openstackclient master: Add python-qinlingclient to zuul job https://review.openstack.org/562949 | |
| 16:20:19 | openstackgerrit | Merged openstack/api-sig master: import zuul job settings from project-config https://review.openstack.org/593943 | |
| 16:45:29 | elmiko | dtantsur edleafe i need a clarification, that change we merged is to fix the api-wg reference in the tc repo? | |
| 16:47:23 | dtantsur | elmiko: I approved a change to move CI jobs in-tree | |
| 16:47:42 | dtantsur | this is a purely technical thing, related to a reorganization of project-config due to approaching Python 3 transition | |
| 16:48:21 | elmiko | dtantsur: ok, so not related to the tc stuff | |
| 16:50:19 | dtantsur | nope | |
| 16:50:30 | elmiko | thanks! | |
| 16:51:48 | mordred | dtantsur: oh wow. that is a CRAZY thing to return from a /v1 endpoint | |
| 16:52:17 | dtantsur | mordred: probably :) | |
| 16:52:22 | mordred | but also - yeah - we're going to need to update keystoneauth - as that does not match any of the known or supported payloads | |
| 16:52:43 | mordred | and is especially bad because it contains the key 'version' which is one of the things that keystoneauth looks for to figure out what sort of payload it has found | |
| 16:52:58 | mordred | so it expects the contents of 'version' at the top level to be a dict :) | |
| 16:54:19 | elmiko | cdent dtantsur edleafe https://etherpad.openstack.org/p/api-sig-newsletter ready for review | |
| 16:54:24 | mordred | dtantsur: I must now get lunch (crazypants travel day) - I'll see if I can come up with some options in a bit | |
| 16:54:54 | dtantsur | mordred: cool! I'll be out already, but I'll read your pings in the morning hopefully | |
| 16:54:59 | elmiko | get a good belt for those crazy pants XD | |
| 16:55:49 | edleafe | elmiko: wow, no pedantic grammar corrections needed. Huzzah! | |
| 16:55:50 | cdent | elmiko: I don't know, all that stuff about working group stuff is like "we've got no news, so I'm going to relay this really booooring thing" | |
| 16:56:13 | elmiko | edleafe: whoa... this is a first! | |
| 16:56:27 | elmiko | cdent: i can drop that last para | |
| 16:56:35 | edleafe | But I agree with cdent - that last paragraph should be scrapped | |
| 16:56:43 | elmiko | ++ | |
| 16:56:51 | elmiko | done | |
| 16:57:01 | edleafe | Not like anyone reads these things | |
| 16:57:05 | elmiko | haha | |
| 16:57:14 | cdent | often more than once | |
| 16:57:15 | elmiko | i did read the ones for the weeks i missed, that's at least one person | |
| 16:57:27 | dtantsur | the same ^^ | |
| 16:57:36 | edleafe | We should probably put in a bit like "if you are reading this, ping cdent and he will send you $10" | |
| 16:57:37 | elmiko | ok, i'm gonna ship this. thanks again! | |
| 16:57:42 | elmiko | LOL | |
| 16:57:49 | edleafe | I meant *outside* of our group | |
| 16:57:57 | elmiko | also, no surprise, i will be out for next week's meeting =( | |
| 19:37:01 | openstackgerrit | Mathieu Gagné proposed openstack/openstacksdk master: Add the ability to extend a volume size https://review.openstack.org/595873 | |
| #openstack-sdks - 2018-08-24 | |||
| 01:24:18 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: baremetal: support newer microversions in {get,update,patch}_machine https://review.openstack.org/594242 | |
| 02:06:20 | mordred | dtantsur|afk: ok. SO - I dug all the way down the the rabbit hole in to keystoneauth and I don't think we can support versioned endpoints for ironic with the noauth / standalone without some updates to the version response from ironic | |
| 02:06:25 | mordred | we can update to fail more better | |
| 02:06:34 | mordred | but the version response has no microversion data | |
| 02:06:40 | mordred | so we have to assume v1 | |
| 02:07:01 | mordred | that said - I think it wouldn't be awful to add the microversion info to the versioned response | |
| 02:07:18 | mordred | and we can update keystoneauth slightly to make it upwards compatible when ironic grows doing that | |
| 02:16:44 | openstackgerrit | Monty Taylor proposed openstack/keystoneauth master: Add support for ironic single-version responses https://review.openstack.org/595980 | |
| 02:16:51 | mordred | dtantsur|afk, TheJulia: ^^ | |
| 02:20:48 | mordred | dtantsur|afk: for now, in sdk, I think just putting in the try/except around the discovery failure seems to 'work' for endpoint things ... although given the goal is supporting newer microversions I'm not sure it fully achieves all of the goals | |
| 02:22:06 | mordred | dtantsur|afk: we COULD try to add a warning if someone tries to use ironic through sdk with an endpoint_override that ends in /v1 - saying that microversions canot be detected | |
| 02:22:08 | mordred | hrm | |
| 02:23:02 | mordred | the discovery path might do the wrong thing is given an unversioned endpoint in endpoint_override | |