| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2017-11-30 | |||
| 21:18:21 | openstackgerrit | Andreas Jaeger proposed openstack/os-api-ref master: Add os-api-ref-nova-src job https://review.openstack.org/524353 | |
| 21:18:59 | briancurtin | thingee: i ran by Sizzle Pie the other day. just thought you should know | |
| 21:19:38 | thingee | briancurtin: and you didn't go in? | |
| 21:19:48 | openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Update the shade-merge document https://review.openstack.org/524391 | |
| 21:20:33 | mordred | thingee: ^^ ok - I updated that doc a bit- it should probably be fundamentally rewritten and moved somewhere more discoverable | |
| 21:20:45 | briancurtin | thingee: unfortunately not. a mid-run pizza didn’t sound great at the time, but actually sounds great now | |
| 21:22:09 | mordred | briancurtin: the concept of a mid-run pizza might almost be enough to convince me to run | |
| 21:22:39 | thingee | mordred: thanks | |
| 21:22:40 | mordred | although I'd probably just end up skipping the first part of the run, going straight for pizza, then skipping the second part because I'm full | |
| 21:23:03 | mordred | thingee: oh - it occurs to me there's another thing to put in there ... | |
| 21:26:23 | adriant | mordred: I honestly don't mind working on bugs in the SDK, so potentially I'll get distracted and submit some patches for the Swift issues I've found. | |
| 21:26:52 | mordred | adriant: woot! | |
| 21:27:01 | adriant | mordred: and yes a function of flag for "a connection with just these args thanks" would be amazing. | |
| 21:27:10 | adriant | or* flag | |
| 21:28:04 | openstackgerrit | Julia Kreger proposed openstack-infra/shade master: Remove version arg from updated ironic calls https://review.openstack.org/524393 | |
| 21:28:11 | adriant | mordred: I submitted one patch already for swift related nonsense: https://review.openstack.org/#/c/524066/ | |
| 21:30:19 | openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Update the shade-merge document https://review.openstack.org/524391 | |
| 21:35:04 | openstackgerrit | Krzysztof Klimonda proposed openstack-infra/shade master: Prioritize external networks for assigning FIP to instance https://review.openstack.org/524396 | |
| 21:40:27 | openstackgerrit | Dean Troyer proposed openstack/python-openstackclient master: Remove a bunch of things we promised to remove in 2H 2017 https://review.openstack.org/524399 | |
| 21:45:52 | openstackgerrit | Monty Taylor proposed openstack/os-api-ref master: Avoid tox_install.sh for constraints support https://review.openstack.org/524344 | |
| 21:45:52 | openstackgerrit | Monty Taylor proposed openstack/os-api-ref master: Add os-api-ref-nova-src job https://review.openstack.org/524353 | |
| 22:00:14 | TheJulia | mordred: so hit a minor issue. python-ironicclient has retry logic baked in for handling nodes locked by the conductor. Any thoughts? I'm thinking the least invasive would be just to double check for the locks before operations that could be blocked by them and wait for the lock to clear, which would clear until ironic likely needs to go re-check the power state. | |
| 22:04:38 | mordred | TheJulia: ah- interesting ... and yes, that seems like a good plan ... altough I suppose there's a race condition there and that something could still lock between the check and the operation - so we still probably want to handle geting a node-locked error | |
| 22:05:36 | TheJulia | Yeah... | |
| 22:09:25 | adriant | mordred: omfg. my notfound 404 issue. it's simple. no one stuck `exceptions.raise_from_response(resp)` in the _head function | |
| 22:09:35 | adriant | although I'm curious if that breaks anything else | |
| 22:09:47 | adriant | will submit a patch and we can test/review | |
| 22:10:09 | mordred | adriant: woot! | |
| 22:12:05 | TheJulia | mordred: I guess it only really matters on actions that are actually trying to make a change that requires a lock in the first place | |
| 22:16:25 | dtroyer | that sounds like it involved neutron…. where I think the only atomic operation is "halt" | |
| 22:19:22 | TheJulia | I guess there could be code someplace that requires the task object to be re-read, which could make things... fun if someone deleted ports while a task was running in ironic | |
| 22:33:08 | openstackgerrit | Merged openstack-infra/shade master: De-clientify baremetal create/delete https://review.openstack.org/500119 | |
| 22:33:09 | openstackgerrit | Merged openstack-infra/shade master: De-client-ify baremetal machine port list https://review.openstack.org/500120 | |
| 22:33:10 | openstackgerrit | Merged openstack-infra/shade master: De-client-ify machine patch operations https://review.openstack.org/500121 | |
| 22:45:30 | openstackgerrit | Lance Bragstad proposed openstack/python-openstackclient master: Add system role functionality https://review.openstack.org/524416 | |
| 22:51:28 | openstackgerrit | Adrian Turjak proposed openstack/python-openstacksdk master: Swift 404s to raise NotFound correctly https://review.openstack.org/524418 | |
| 22:52:04 | adriant | mordred: this fixes my current swift issues with 404: https://review.openstack.org/#/c/524418/ | |
| 22:52:16 | adriant | potentially may break other APIs that use HEAD differently, but I doubt it | |
| 22:52:40 | adriant | I think someone just forgot to add that function all to resource.head | |
| 22:52:51 | adriant | function call* | |
| 22:57:04 | openstackgerrit | Merged openstack-infra/shade master: Remove version arg from updated ironic calls https://review.openstack.org/524393 | |
| 22:57:29 | openstackgerrit | Brian Haley proposed openstack/python-openstackclient master: Display IPv6 addresses better https://review.openstack.org/524420 | |
| 23:36:04 | thingee | adriant: curious why the key would be in the values? | |
| 23:36:29 | thingee | for https://review.openstack.org/#/c/524066/1 | |
| 23:37:11 | thingee | adriant: nevermind, found the storyboard ticket with the info | |
| 23:38:30 | mordred | Shrews, briancurtin, dtroyer, SamYaple, thingee, TheJulia: I have discovered that pypy unittests are basically unpossible in the gate atm (the version of cryptography in global-requirements is not compatible with the version of pypy in xenial) | |
| 23:39:04 | TheJulia | ugh | |
| 23:39:14 | mordred | there is a ppa that has newer pypy, but that'll take a decent amount of consideration to deal with broadly - so I leaning toward just disabling the pypy tox env for now | |
| 23:39:34 | mordred | until we can get a job set up that will *actually* test it | |
| 23:40:08 | mordred | (it's been green so far because of a different bug that has caused it to not actually use pypy - fixing that causes the job to start (correctly) failing) | |
| 23:42:22 | dtroyer | mordred: ++ on disabling for now… I'd say make it experimental but we know it's never going to actually work without tasks elsewhere first | |
| 23:42:37 | mordred | yah, that was my thinking | |
| 23:42:54 | dtroyer | mordred: fwiw, I'm testing a fix for OSC's -tips job… I finally duplicated it here | |
| 23:42:57 | mordred | HOWEVER - I just had another idea I'm going to try out REAL quick ... we could tell it to run on fedora26 nodes instead of xenial nodes ... | |
| 23:43:02 | mordred | dtroyer: woot! | |
| 23:43:19 | dtroyer | I'd support the fedora alternative | |
| 23:44:21 | mordred | yah. fedora-26 has new enough ... | |
| 23:45:41 | dtroyer | you're going to love what the problem was :) if it succeeds here I'll push it up and see if you have a thought on what the correct fix actually is | |
| 23:46:23 | openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Fix py35 and pypy tox env https://review.openstack.org/523383 | |
| 23:46:33 | mordred | dtroyer: I can't wait :) | |
| 23:47:04 | mordred | dtroyer: that *should* work for pypy ^^ - and is also a nice example of using a pre-existing job but overriding the node type :) | |
| 23:47:06 | openstackgerrit | Dean Troyer proposed openstack/python-openstackclient master: DNM: Test -tips variations https://review.openstack.org/523938 | |
| 23:47:20 | dtroyer | I think it's past the failure point ^^^^^ | |
| 23:47:40 | mordred | dtroyer: oh. yah. whoops. | |
| 23:48:05 | dtroyer | the question is what _is_ the correct thing to check for in that case, since it's SDK/Shade not o-c-c | |
| 23:48:09 | dtroyer | this works though | |
| 23:48:35 | dtroyer | damn… sopke too soon | |
| 23:48:36 | mordred | yup. I think that's the right thing to check for for now | |
| 23:49:22 | mordred | but I'm clearly not correct | |
| 23:49:57 | dtroyer | it still took the 'old' branch, I think I'm not setting this up right…oh wait, … nevermind, let's let it run in the gate, it should work in spite of my local pilot error | |
| 23:50:11 | dtroyer | dinnertime, will check up later | |
| 23:56:25 | openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Fix py35 and pypy tox env https://review.openstack.org/523383 | |
| 23:58:48 | mordred | dtroyer: I have the log stream open in a tab and am waiting to see it be happy | |
| #openstack-sdks - 2017-12-01 | |||
| 00:12:03 | mordred | woot! the f26 pypy seems to be working, as does the https://review.openstack.org/523938 tips fix | |
| 00:12:18 | mordred | (and by seems to be working, I mean it passed) | |
| 00:19:20 | adriant | thingee: I'm trying to shift some code from swift client to the SDK, and that was one issue I found. | |
| 00:33:30 | openstackgerrit | Adrian Turjak proposed openstack/python-openstacksdk master: Swift 404s to raise NotFound correctly https://review.openstack.org/524418 | |
| 00:34:20 | adriant | mordred: ^ there, that should fix swift 404s being broken | |
| 00:35:24 | adriant | and it is unlikely to break anything since only swift appears to actually use HEAD | |
| 00:36:01 | adriant | hmmm, or at least called directly | |
| 00:37:14 | adriant | yeah, allow_head seems to only be applicable to swift :/ | |
| 00:37:27 | adriant | so I can see why no one found this bug before | |
| 00:38:25 | mordred | adriant: to my knowledge, swift is the only service that uses head at all | |
| 00:39:05 | adriant | yep, that would be why this was broken :P | |
| 00:40:09 | mordred | heh | |
| 00:49:22 | adriant | mordred: and it appears part of this issue is actually fixed in resource2 | |
| 00:50:04 | mordred | woot! so the part of the plan which is "finish migrating to resource2" is a good one | |
| 00:51:28 | adriant | are we entirely dropping resource and moving to resource2 or is it being kept around for some form of backwards compat? | |
| 01:00:44 | openstackgerrit | Adrian Turjak proposed openstack/python-openstacksdk master: Swift 404s to raise NotFound correctly https://review.openstack.org/524418 | |
| 02:09:16 | dtroyer | adriant: IIRC finishing that was one of the things holding up a 1.0 release. OSC's network code has a good bit of compat in it for that changeover that we're not removing until 1.0 is released | |
| 02:14:32 | openstackgerrit | Dean Troyer proposed openstack/python-openstackclient master: DNM: Test -tips variations https://review.openstack.org/523938 | |
| 03:45:49 | openstackgerrit | Andreas Jaeger proposed openstack/os-api-ref master: Fix Zuul jobs https://review.openstack.org/524353 | |
| 03:48:31 | dayou | Anyone know about the correct parameter to feed os-client-config get_one_cloud for token authentication tyep? | |
| 03:48:47 | dayou | cloud_config = occ.OpenStackConfig(load_yaml_config=False).get_one_cloud( | |
| 03:48:50 | dayou | verify=not insecure, | |
| 03:48:52 | dayou | cacert=cacert, | |
| 03:48:54 | dayou | region_name=request.user.services_region, | |
| 03:48:56 | dayou | auth_plugin='token', | |
| 03:48:58 | dayou | auth=dict( | |
| 03:49:00 | dayou | project_id=request.user.project_id, | |