Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2018-08-23
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
14:26:38 mnaser mordred: does shade have a way of listing stack resources, i don't see one right off the bat..
14:29:13 Shrews mnaser: i don't see one either
14:29:56 mnaser Shrews: is there a way to get a client via shade or not anymore? would need to go to openstacksdk for that?
14:31:23 Shrews mnaser: there should be a orchestration_client property
14:32:18 mnaser Shrews: that's wonderful, thank you
14:54:53 openstackgerrit Samuel de Medeiros Queiroz proposed openstack/openstacksdk master: WIP: Functional tests for core functionality https://review.openstack.org/596359
14:57:00 pooja_jadhav_ hello all, I am wrirring functional_tests in openstacksdk for masakari. In that, for test_host module. for get_host_test, trying assert host objects. (expected, actual). But its failing because of timestamp in created_at field.
14:58:48 pooja_jadhav_ while creating host its giving correct datetime with timestamp. but while saving host record in DB, its saves only date nd time (not timestamp). so while getting that host, created_at field will append timestamp as 0000. its mismatching reference and actual object.
14:58:58 pooja_jadhav_ please anyone can help me in this?
15:01:51 openstackgerrit Samuel de Medeiros Queiroz proposed openstack/openstacksdk master: WIP: Functional tests for core functionality https://review.openstack.org/596359
15:03:01 cdent Shrews: you have any insight on pooja_jadhav_ 's issue?
15:04:13 Shrews pooja_jadhav_: cdent: my initial thought is to maybe compare individual fields (as test_create() does), or to mock the time function used
15:04:48 Shrews (if i understand correctly)
15:04:58 Shrews sorry, have to AFK for a bit now
15:05:35 samueldmq mordred: cdent: Shrews: would be awesome to have your eyes on https://review.openstack.org/596359

Earlier   Later