Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2017-11-30
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,
03:49:02 dayou project_domain_id=request.user.domain_id,
03:49:04 dayou user_domain_id=request.user.domain_id,
03:49:06 dayou token=request.user.token.unscoped_token,
03:49:08 dayou auth_url=request.user.endpoint),
03:49:10 dayou app_name='octavia-dashboard',
03:49:12 dayou app_version=octavia_dashboard.__version__)
03:49:14 dayou With the following I got "__init__() got an unexpected keyword argument 'token'" error
04:12:37 SamYaple mordred: meh. its only pypy
04:45:09 openstackgerrit Gilles Dubreuil proposed openstack/api-wg master: Adds API-Schema guide https://review.openstack.org/524467
06:32:14 adriant dayou: the issue is that it isn't 'auth_plugin' but 'auth_type'
06:32:42 adriant I ran into that issue using mordred's example as well :P
06:33:04 adriant it's assuming the password plugin, and getting confused
06:50:15 dayou adriant, thanks, that works, also I found user_domain_id can't be used also!

Earlier   Later