| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2020-05-04 | |||
| 19:48:59 | mordred | dtroyer: hahaha. "yay" | |
| 19:49:12 | mordred | dtroyer: I *think* it "works" for some value of "it" and "works | |
| 19:49:18 | cmurphy | mordred: dtroyer afaik adriant hasn't gotten around to adding support for MFA receipts (https://specs.openstack.org/openstack/keystone-specs/specs/keystone/stein/mfa-auth-receipt.html) to osc so it is still pretty broken | |
| 19:49:54 | mordred | cmurphy: ah - ok . so I'm probably remembering "someone *wants* to use it" more than "someone is using it" | |
| 19:52:44 | cmurphy | it's possible that osc could use the v3multifactor auth plugin from ksa and it could just work | |
| 19:53:03 | cmurphy | that wouldn't require the receipts workflow | |
| 19:53:31 | dtroyer | cmurphy: Reading through that, is it only the OSC bit that is still undone? Horizon is working on the cloud I am testing on, OSC is not (no well anyway) | |
| 19:53:44 | dtroyer | hmmm… I need to learn about v3multifactor | |
| 19:54:07 | cmurphy | dtroyer: afaik horizon doesn't have it yet either but i could be out of date | |
| 19:54:14 | cmurphy | dtroyer: https://docs.openstack.org/keystoneauth/latest/authentication-plugins.html#multi-factor-with-v3-identity-plugins | |
| 19:55:24 | dtroyer | I do not know exactly how this cloud is configured, they are using Duo if that means anything specific | |
| 19:58:17 | mordred | dtroyer: if only you knew someone who could add multifactor support to osc | |
| 20:00:53 | dtroyer | i know, right? | |
| 20:03:32 | dtroyer | actually, some of what I am seeing so far may be around the session/token not getting re-used properly. and if we only had some insecure method of caching that between commands | |
| 20:19:15 | mordred | dtroyer: if only | |
| 20:19:26 | mordred | dtroyer: I feel like someone should have thought about this before | |
| 20:28:18 | openstackgerrit | Noah Mickus proposed openstack/openstacksdk master: Add TLS protocol support for Octavia https://review.opendev.org/724789 | |
| 22:02:22 | adriant | cmurphy, mordred: I was working on Horizon and got side tracked/stuck with that, but never touched OSC | |
| 22:02:57 | adriant | the support is in KSA, just doesn't exist in the cli tools properly unless you get/set a token somehow yourself and then use the tools | |
| 22:03:34 | adriant | I don't have the time for it, but will gladly review/comment on the design of the support in OSC etc | |
| 22:19:21 | rm_work | ah mordred: https://review.opendev.org/#/c/650374/22/openstackclient/compute/v2/server.py this changed the behavior of the image lookup in the server commands :( | |
| 22:19:31 | rm_work | just caught it internally today | |
| 22:19:46 | rm_work | now lookups by image *name* fail, where they worked before | |
| 22:20:14 | rm_work | because it seems the switch to the SDK used the id lookup function directly, rather than emulating that "id, then name" lookup feature the old code had | |
| 22:20:27 | rm_work | (I believe) | |
| 23:05:53 | rm_work | yeah, patch incoming | |
| 23:11:53 | openstackgerrit | Adam Harwell proposed openstack/python-openstackclient master: Correct image lookup during server rebuild https://review.opendev.org/725427 | |
| 23:12:00 | rm_work | mordred: ^^ should resolve it | |
| 23:12:07 | rm_work | will be testing locally today/tomorrow | |
| 23:20:01 | rm_work | oh right, this is client, tested locally just now and it resolved the issue :) | |
| #openstack-sdks - 2020-05-05 | |||
| 02:10:03 | mordred | rm_work: thanks for the patch - +2 from me | |
| 14:11:31 | mordred | slaweq: comment on https://review.opendev.org/#/c/717881 | |
| 14:13:34 | slaweq | mordred: thx | |
| 14:13:45 | slaweq | I will take a look later today | |
| #openstack-sdks - 2020-05-06 | |||
| 15:13:36 | rpittau | hi all, if anyone got a minute, those should be fairly quick to review https://review.opendev.org/724904 https://review.opendev.org/724932 https://review.opendev.org/724968 thanks! | |
| 15:39:50 | dtantsur | dunno if I should +2 stuff on osc-lib, but I just did | |
| 15:55:16 | mordred | dtantsur: you totally should :) | |
| 16:06:25 | rpittau | thanks! :) | |
| 16:10:16 | dtantsur | :) | |
| 17:43:44 | openstackgerrit | Merged openstack/osc-lib master: Use mock from unittest https://review.opendev.org/724904 | |
| 17:43:45 | openstackgerrit | Merged openstack/osc-lib master: Add pep8 import order check https://review.opendev.org/724932 | |
| 18:00:03 | openstackgerrit | Dean Troyer proposed openstack/osc-lib master: Save the results from the early authentication https://review.opendev.org/725953 | |
| 18:03:23 | dtroyer | I have no idea how long ago that ^^^^ changed but at a minimum keystoneclient was affected…and no I'm not doing a git blame as I'm probably the one who did it… | |
| 19:22:15 | mordred | dtroyer: you angered the gods of pep8 | |
| 19:24:46 | openstackgerrit | Monty Taylor proposed openstack/osc-lib master: Save the results from the early authentication https://review.opendev.org/725953 | |
| 19:26:52 | dtroyer | mordred: hrm… why did it pass locally???? | |
| 19:27:19 | mordred | dtroyer: maybe stale venv? | |
| 19:27:23 | dtroyer | ah, I see, my local bit-rotted dev environment... | |
| 19:27:38 | mordred | yeah - the python3 push caused a bump in several of the linters | |
| 19:27:57 | mordred | in other news though - no more python2.7! | |
| 19:28:13 | mordred | (and, in fact, for osc, no more 3.5 - so you can use f-strings if you want to!) | |
| 19:28:26 | dtroyer | \o/ except everything in my workspace was still set for that… | |
| 19:29:00 | dtroyer | a good reason to start over and ignore about 27 stale branches that had at least one good idea in them | |
| 19:29:24 | mordred | ignoring things is the best | |
| 19:30:33 | dtroyer | except I am likely to resurrect the devstack technique I tried for running osc in the background and piping comamnds to it… we have a rather annoying auth situation here, which is why cutting the /auth/token round trips in half is worthwhile | |
| 19:31:07 | dtroyer | and for my next trick, learning to do the same for fog… | |
| 19:32:01 | mordred | dtroyer: I don't suppose getting the cloud to update and add appcred support is likely then | |
| 19:32:41 | dtroyer | It's lucky to be on queens (vmware-backed) and they don't even know what that is. | |
| 19:33:59 | mordred | dtroyer: well - I guess our insistence on supporting old openstacks has paid off | |
| 19:35:05 | dtroyer | absofrickinlutely…. never give up that fight :) | |
| 19:36:05 | dtroyer | mordred: while I've got you, what did you guys switch to rather than cloud-init? | |
| 19:36:21 | mordred | we use glean | |
| 19:36:30 | mordred | https://opendev.org/opendev/glean | |
| 19:36:44 | mordred | if you build images with dib, there is a "simple-init" element that sets it up | |
| 19:37:20 | mordred | https://opendev.org/openstack/diskimage-builder/src/branch/master/diskimage_builder/elements/simple-init | |
| 19:37:24 | dtroyer | ah, right. thanks. this is an existing chef/packer/rhel stack that can't configure networking properly without help | |
| 19:37:41 | mordred | nod. you should be able to install it with packer | |
| 19:37:53 | mordred | (but you might want to go crib systemd units from that element) | |
| 19:38:12 | mordred | it's limited in scope - it can do networking and ssh keys and only reads data from config-drive | |
| 19:38:51 | mordred | but - it can handle static network config in config-drive | |
| 19:39:30 | dtroyer | apparently cloud-init on rhel doesn't do that well at all | |
| 19:48:26 | mordred | yeah - static networking from cloud-init is why we wrote glean | |
| 19:49:29 | dtroyer | and I love that dependency list! | |
| 19:49:29 | mordred | rax only does static networking (no dhcp) and only does config-drive - and for quite a long time cloud-init dev was frozen because they were working on a v2 so we coudlnt' get the support for it added | |
| 19:49:37 | mordred | (there was a rax-specific fork of cloud-init - but we didn't really feel like that was the right life decision) | |
| 19:50:26 | mordred | and yeah - also - cloud-init pulls in a bunch of python, and we happen to, you know, be in the business of testing a lot of python | |
| #openstack-sdks - 2020-05-07 | |||
| 07:27:00 | openstackgerrit | Johannes Kulik proposed openstack/openstacksdk master: Add "id" to Port's query parameters https://review.opendev.org/726050 | |
| 07:29:11 | openstackgerrit | Johannes Kulik proposed openstack/openstacksdk master: Add "id" to Port's query parameters https://review.opendev.org/726050 | |
| 09:11:17 | openstackgerrit | Riccardo Pittau proposed openstack/osc-lib master: Remove six library from dependencies https://review.opendev.org/724968 | |
| 13:10:58 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Use floating ip for interface ip if it exists https://review.opendev.org/726115 | |
| 13:11:30 | openstackgerrit | Monty Taylor proposed openstack/openstacksdk master: Use floating ip for interface ip if it exists https://review.opendev.org/726115 | |
| 13:41:06 | nightmare_unreal | how does OSC CLI determine which api version to use ? | |
| 13:41:28 | nightmare_unreal | for e.g. use 2.15 or something or 2.60 etc etc. | |
| 14:17:02 | openstackgerrit | Merged openstack/shade stable/ussuri: Break out py27 job templates https://review.opendev.org/723353 | |
| 14:23:02 | openstackgerrit | Merged openstack/shade stable/ussuri: Update .gitreview for stable/ussuri https://review.opendev.org/719019 | |
| 14:23:03 | openstackgerrit | Merged openstack/shade stable/ussuri: Update TOX/UPPER_CONSTRAINTS_FILE for stable/ussuri https://review.opendev.org/719020 | |
| 14:40:06 | openstackgerrit | Merged openstack/osc-lib master: Remove six library from dependencies https://review.opendev.org/724968 | |
| 14:47:03 | openstackgerrit | Artem Goncharov proposed openstack/openstacksdk master: Fix metric names in the object_store https://review.opendev.org/726167 | |
| 14:54:55 | openstackgerrit | Artem Goncharov proposed openstack/openstacksdk master: Fix metric names in the object_store https://review.opendev.org/726167 | |
| 16:01:12 | elmiko | API SIG office hour open | |
| 16:10:46 | brtknr | hmm why doesnt pip2 understand not to install openstacksdk version greater than 0.40.0 by default | |
| 16:40:44 | mordred | brtknr: 0.45 is the last py2 version | |
| 16:42:02 | mordred | brtknr: we dropped 2.7 from the gate in 0.40 but didn't get the python-requires line in. we accidentally broke 2.7 in 0.44, reverted the patch and released 0.45, then added the metadata in 0.45 to tell pip2 to do the right thing | |
| 16:42:37 | mordred | brtknr: are you have having issues with 0.45 and python 2.7? if so, we should figure them out and see if we can't figure out how to release an 0.45.1 point release | |
| 16:42:49 | mordred | but otherwise, you should get 0.45 and it should work | |
| 16:59:23 | elmiko | catch y'all later, stay safe out there o/ | |
| 17:43:35 | dtroyer | Are there any known issues with uploading images to vmware-backed clouds since the image code was switched to SDK? I am getting persistent checksum failures with osc 5+ but not 4.0.0 | |
| 18:40:02 | mordred | dtroyer: no, not to my knowledge. that said - I do wanna say we found $something related to checksums uploading to $somewhere - but I thought it was fixed | |
| 18:42:38 | mordred | I'm not finding anything ... | |
| 18:43:21 | mordred | gtema: ^^ | |