Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2018-08-21
20:22:07 guilhermesp so let me know if there's something missing, if it is ok, I think can be merged :)
20:22:52 Shrews guilhermesp: i'll look in a bit. fwiw, i think there is a merge captain for all backports (so i don't have control over those)
20:23:26 Shrews i keep getting behind on how ansible handles PRs
20:23:47 guilhermesp is there something that I can do for now?
20:24:09 Shrews i don't think so
20:24:21 guilhermesp cool, thanks for your help
20:25:21 Shrews mordred can add another shipit for 44478. we can add shipits on the others, but i'm not positive they'll merge until the backport captian does it
22:49:30 openstackgerrit Merged openstack/python-openstackclient master: import zuul job settings from project-config https://review.openstack.org/594288
#openstack-sdks - 2018-08-22
01:48:07 openstackgerrit LIU Yulong proposed openstack/openstacksdk master: Support firewall service for SDK https://review.openstack.org/592303
12:14:05 cdent elmiko: are you still on pto, going to twenty conferences, etc?
12:14:22 dtantsur heh
12:17:14 elmiko cdent: i am back at the desk =)
12:17:31 elmiko that said, i will be out of town next week...
12:17:33 cdent a) hah, b) awwww
12:18:02 cdent edleafe and I said that you would finish https://review.openstack.org/#/c/592003/
12:20:22 cdent elmiko: actually the real thing was whether you had some input on how to balance that out. I was trying to present a specific heuristic for thinking that works in some cases, whereas ed was hoping to make it generally useful
12:20:47 elmiko cdent: ack, was just reading through it
12:21:00 elmiko i will add it to my list for today =)
12:21:25 cdent I don't think there's any real rush. I just needed a good excuse to say "hey"
12:21:26 cdent hey
12:21:45 elmiko hahaha, hey
12:25:22 cdent huzzah!
13:01:45 edleafe Huzzah to you, elmiko!
13:02:02 elmiko edleafe: huzzah!
14:13:39 openstackgerrit Merged openstack/openstacksdk master: import zuul job settings from project-config https://review.openstack.org/594373
15:19:41 dtantsur TheJulia: I have problems with these tests: https://review.openstack.org/#/c/594242/2/openstack/tests/unit/cloud/test_operator_noauth.py
15:20:00 dtantsur do we really use versioned endpoints in the standalone case? because they're not compatible with version discovery..
15:20:07 dtantsur cc mordred ^^
15:31:20 openstackgerrit Merged openstack/osc-lib master: import zuul job settings from project-config https://review.openstack.org/594282
16:34:28 openstackgerrit Merged openstack/os-client-config master: import zuul job settings from project-config https://review.openstack.org/594377
17:44:02 TheJulia dtantsur: so I think it was a carry over pattern from python-ironicclient because it injected a v1
17:44:12 TheJulia dtantsur: that being said, I think we should be able to remove it
17:50:58 dtantsur okay, I can try it
17:51:28 dtantsur or we should convince keystoneauth to accept versioned endpoints for discovery /cc mordred again
19:08:06 mordred dtantsur|afk: keysteonauth _should_ handle versioned endpoints
19:08:57 mordred dtantsur|afk: sorry, I'm in the middle of a large day of travel so am fairly useless - but if discovery isn't working with versioned endpoints that's a bug
19:09:00 mordred TheJulia: ^^
20:59:17 TheJulia mordred: there was some edge case when I was conveting the tests that I believe was fixed
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 :)

Earlier   Later