| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2019-08-13 | |||
| 17:01:55 | mordred | efried: should we also land https://review.opendev.org/#/c/674807/ too? | |
| 17:02:12 | mordred | (that also contains the keystoneauth dep bump) | |
| 17:03:07 | efried | mordred: could do, yes. Is there a reason that wasn't +W'd? | |
| 17:03:24 | efried | mordred: also, that one bumps to 3.15.0, but we need .16. to pick up both retry changes. | |
| 17:03:34 | efried | I could rebase my bump on that one and +W it. | |
| 17:03:47 | mordred | ++ | |
| 17:11:01 | openstackgerrit | Eric Fried proposed openstack/openstacksdk master: Bump keystoneauth1 minimum to 3.16.0 https://review.opendev.org/676247 | |
| 17:11:04 | efried | mordred: ^ | |
| 17:11:12 | efried | Shrews: too | |
| 17:15:33 | efried | dustinc: I'm going to start hacking at that thing I said earlier, about deprecating the existing ironic retry opts. There are other ways to get the right timeouts into the sdk proxy, but they're messier. | |
| 17:15:44 | efried | ...unless you want it | |
| 17:17:37 | dustinc | efried: go for it, I am about to push a small fix to the bottom patch | |
| 17:18:16 | efried | dustinc: are you taking out 'fields' for now? | |
| 17:18:39 | dustinc | I was going to leave it expecting the SDK release to add support | |
| 17:18:52 | dustinc | I can take it out for now if you want.. | |
| 17:19:40 | efried | dustinc: meh | |
| 17:19:50 | efried | I think we're not going to be able to land anything until we've got some of these issues sorted out | |
| 17:19:54 | efried | so probably no hurry | |
| 17:20:05 | efried | but you're going to want to put your series on top of this thing I'm doing | |
| 17:20:11 | efried | which will also require the new sdk release | |
| 17:20:21 | efried | for which I'm going to wait for the ksa bump | |
| 17:20:31 | efried | so we got a little while until all those pieces come together. | |
| 18:35:22 | openstackgerrit | Merged openstack/openstacksdk master: Add support for global_request_id https://review.opendev.org/674807 | |
| 20:35:43 | openstackgerrit | Merged openstack/openstacksdk master: Bump keystoneauth1 minimum to 3.16.0 https://review.opendev.org/676247 | |
| 20:44:33 | efried | openstacksdk release 0.34.0 proposed: https://review.opendev.org/676276 | |
| 22:50:30 | openstackgerrit | Merged openstack/openstacksdk master: Minor refactor cleanup of _make_proxy https://review.opendev.org/676015 | |
| 23:51:47 | openstackgerrit | Adrian Turjak proposed openstack/keystoneauth master: add support for auth_receipts and multi-method auth https://review.opendev.org/675049 | |
| #openstack-sdks - 2019-08-14 | |||
| 14:52:30 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: DNM Test patch for sphinx job https://review.opendev.org/676432 | |
| 15:04:24 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: DNM Test patch for sphinx job https://review.opendev.org/676432 | |
| 15:06:47 | openstackgerrit | Shubham Potale proposed openstack/openstacksdk master: Add support for nfv-orchestration service https://review.opendev.org/675604 | |
| 16:04:59 | Shrews | https://docs.openstack.org/swift/latest/api/object-expiration.html | |
| 16:05:06 | Shrews | that would have been helpful a few days ago | |
| 16:05:15 | Shrews | but yeah for existing | |
| 16:05:38 | Shrews | s/yeah/yay/ | |
| 16:07:51 | mordred | Shrews: you don't have all of the API docs memorized? | |
| 19:01:11 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Make block-storage work with proxies https://review.opendev.org/675382 | |
| 19:05:09 | efried | mordred: I think I need to pre-emptively and backward-compatibly fixture nova for stuff that changed in _make_proxy between 0.33.0 and 0.34.0 https://review.opendev.org/#/c/676457/1 | |
| 19:10:38 | mordred | efried: poop. oh - because of the mocking? | |
| 19:11:13 | mordred | (sorry - the sharable fixture is taking eversoslightly longer than I'd hoped - I may have uncovered a few cans with many worms) | |
| 19:12:39 | efried | if not self.supported_versions: | |
| 19:12:39 | efried | # Skip all the lower logic. | |
| 19:12:39 | efried | # explicitly just using us for a passthrough REST adapter. | |
| 19:12:39 | efried | # We don't know anything about this service, so the user is | |
| 19:12:39 | efried | mordred: I haven't fully nailed it down yet, just have a local repro. It's because of this code path: | |
| 19:12:40 | efried | self.service_type, | |
| 19:12:40 | efried | temp_client = config.get_session_client( | |
| 19:12:41 | efried | ) | |
| 19:12:41 | efried | allow_version_hack=True, | |
| 19:12:42 | efried | temp_client.get_endpoint() | |
| 19:12:42 | efried | # trigger EndpointNotFound exception if this is bogus | |
| 19:12:43 | efried | return temp_client | |
| 19:13:15 | efried | That temp_client.get_endpoint is raising MissingAuthPlugin from ksa | |
| 19:14:54 | efried | I have a fix... | |
| 19:14:58 | mordred | cool | |
| 19:36:03 | openstackgerrit | Sean McGinnis proposed openstack/python-openstackclient master: Default to Cinder v3 API https://review.opendev.org/676505 | |
| 21:55:14 | openstackgerrit | Steve Baker proposed openstack/openstacksdk master: Add 'node' attribute to baremetal Allocation https://review.opendev.org/676521 | |
| #openstack-sdks - 2019-08-15 | |||
| 01:44:00 | openstackgerrit | Merged openstack/keystoneauth master: add support for auth_receipts and multi-method auth https://review.opendev.org/675049 | |
| 02:04:21 | adriant | mordred, dtroyer: I assume the two of you are who I need to bug about MFA in the sdk and osc? | |
| 02:06:03 | adriant | I'm thinking, if I can focus on Horizon, I can probably get that done before Train release (hopefully) | |
| 02:06:18 | adriant | so I don't know if I can do the sdk and osc work, but I can review it | |
| 04:34:22 | adriant | http://paste.openstack.org/show/757171/ | |
| 04:34:22 | adriant | dtroyer, mordred: I kind of am imagining we eventually do something like this: | |
| 04:34:41 | adriant | which maybe even generates and saves (based on an option) a clouds.yaml file with the values you enter | |
| 04:35:09 | adriant | additionally we may want to think about how we want to handle the MFA error from keystoneauth when it gets raised | |
| 04:35:54 | adriant | and... lastly I don't think we have any SDK or CLI ways to enable MFA for a user and manage their auth rules? | |
| 04:36:05 | adriant | but I'm not 100% sure on that | |
| 05:18:38 | adriant | yeah, doesn't look like the SDK supports user options as a value on the user object | |
| 05:18:55 | adriant | but user options lets you patch a dict so i think you can update them | |
| 05:19:06 | adriant | https://github.com/openstack/openstacksdk/blob/master/openstack/identity/v3/_proxy.py#L673 | |
| 05:19:31 | adriant | ^ I think. I'm not 100% sure if that will fail because options doesn't exist as an option on the user object | |
| 08:43:02 | openstackgerrit | Rabi Mishra proposed openstack/keystoneauth master: Allow initializing session with connection retries https://review.opendev.org/676648 | |
| 10:45:18 | mordred | adriant: yeah- I've actually wanted a login command for a while - I think if we do it right it could also be used to add a clouds.yaml entry - like from openrc values - or maybe like kubectl setcontext to set a "default" flag in the clouds.yaml (*waves hands*) | |
| 12:37:50 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Make block-storage work with proxies https://review.opendev.org/675382 | |
| 12:38:38 | mordred | Shrews, dtantsur|afk: ^^ I think that should work now - but we might need to squash it with the previous patch to make it all work happily. I figure reviewing separate before the squash would be friendlier | |
| 12:53:51 | Shrews | mordred: okie dokie. will take a peek shortly | |
| 12:54:22 | mordred | Shrews: you're going to "love" it | |
| 12:55:18 | Shrews | like i "love" getting punched in the face? | |
| 13:01:10 | mordred | Shrews: yeah. pretty much just like that | |
| 14:14:21 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Make block-storage work with proxies https://review.opendev.org/675382 | |
| 14:16:16 | mordred | stupid pep8 | |
| 14:52:32 | Shrews | mordred: i left you a question/comment on that | |
| 15:01:29 | openstackgerrit | David Shrewsbury proposed openstack/openstacksdk master: Add header to auto-delete image upload objects https://review.opendev.org/676714 | |
| 15:30:46 | mordred | corvus: ^^ might be the rare SDK patch you want to review | |
| 15:33:04 | corvus | mordred, Shrews: ++ | |
| 15:33:26 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Make block-storage work with proxies https://review.opendev.org/675382 | |
| 15:33:35 | mordred | corvus: I don't know why it took us so long to think of that :) | |
| 15:59:46 | openstackgerrit | David Shrewsbury proposed openstack/openstacksdk master: Avoid unnecessary object meta prefix in proxy https://review.opendev.org/676726 | |
| 16:00:01 | edleafe | Shout out to all the API-SIG Offcie Hour people! Take a number and queue politely! | |
| 16:06:45 | edleafe | ok, mriedem, back of the line! | |
| 17:11:21 | mordred | Shrews: zomg. https://review.opendev.org/#/c/675382 worked | |
| 17:20:21 | mordred | Shrews: there's also two patches before the fixture patch - https://review.opendev.org/#/c/675178 and https://review.opendev.org/#/c/675130 that are ready whenever | |
| 18:10:07 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Replace catalog-v3.json with keystoneauth fixture https://review.opendev.org/675187 | |
| 18:11:09 | mordred | Shrews: thanks! ^^ merged the block-storage patch into the catalog fixture patch | |
| 18:11:23 | mordred | (which is still a net-reduction in LOC) | |
| 19:14:54 | efried | mordred: Still that first ironic-sdk-in-nova patch succeeds but the second one fails. Do you have a minute to help me debug? | |
| 19:24:35 | openstackgerrit | Merged openstack/openstacksdk master: Avoid unnecessary object meta prefix in proxy https://review.opendev.org/676726 | |
| 19:24:36 | openstackgerrit | Merged openstack/openstacksdk master: Add header to auto-delete image upload objects https://review.opendev.org/676714 | |
| 19:36:59 | openstackgerrit | Merged openstack/openstacksdk master: Validate that connect_as connects as the project https://review.opendev.org/675178 | |
| 20:55:50 | mordred | efried: poop - was AFK - can you point me at the failng patch? | |
| 20:56:13 | efried | https://zuul.opendev.org/t/openstack/build/fcf71dbb7bd743249b69c83c8f2ba72a/log/controller/logs/screen-n-cpu.txt.gz?severity=3 | |