| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2018-03-08 | |||
| 13:02:27 | elmiko | cdent dtantsur edleafe , i will most likely miss the meeting today, i've got a conflict internally here | |
| 13:31:39 | edleafe | cdent: I'll run it | |
| 13:46:00 | openstackgerrit | Monty Taylor proposed openstack/keystoneauth master: Remove tox_install.sh and align with constraints consumption https://review.openstack.org/550837 | |
| 14:01:54 | openstackgerrit | Monty Taylor proposed openstack/keystoneauth master: Remove tox_install.sh and align with constraints consumption https://review.openstack.org/550837 | |
| 16:48:03 | edleafe | dtantsur: cdent: redy for review https://etherpad.openstack.org/p/api-sig-newsletter | |
| 16:48:10 | edleafe | geez | |
| 16:48:12 | edleafe | *ready | |
| 16:48:47 | dtantsur | short and nice, lgtm :) | |
| 16:51:08 | cdent | edleafe: you wanna add the cache-control review? | |
| 17:01:04 | edleafe | cdent: ah, good catch | |
| 17:03:18 | edleafe | added and shipped | |
| 17:08:46 | cdent | woot | |
| 17:57:38 | lbragstad | o/ curious if anyone from the api sig/wg would be interesting in sharing their thoughts on http://lists.openstack.org/pipermail/openstack-dev/2018-March/128027.html ? | |
| 17:57:44 | lbragstad | interested* | |
| 17:58:20 | lbragstad | the team is curious about where we should meet usability with the consistency of the rest of the API | |
| 17:58:43 | cdent | lbragstad: will put that in my queue for tomorrow. edleafe, dtantsur|afk ^ | |
| 17:59:15 | lbragstad | thanks cdent | |
| 17:59:46 | cdent | would today but I got melty brain | |
| 17:59:56 | lbragstad | that's understandable | |
| 18:00:25 | lbragstad | s/coffee/beer/ if symptoms persist :) | |
| 18:02:40 | cdent | :) | |
| 18:28:47 | edleafe | wait - brains aren't supposed to be melty? | |
| 19:42:48 | mordred | edleafe: oy. I've been doing it all wrong | |
| 21:41:13 | adriant | mordred: I found a bug, although this one is easy: https://review.openstack.org/#/c/550700/ | |
| 21:41:38 | adriant | I also took the chance to check all our resources for identity and make sure the query filters matched. They did, just this one was wrong. | |
| 21:42:34 | adriant | lbragstad: ^ I you fellows are evil for "user.id" as the filter rather than "user_id" :P | |
| 21:42:57 | adriant | and I think it's the only keystone API to do that. :( | |
| 21:43:43 | lbragstad | hmmmmmm | |
| 21:43:56 | lbragstad | i might have to dig into why that is | |
| 21:45:47 | adriant | lbragstad: https://developer.openstack.org/api-ref/identity/v3/#id595 this one | |
| 21:46:25 | adriant | I always used the keystoneclient so never realised until now when trying to switch to the sdk that the API was weird :P | |
| 21:47:25 | adriant | the keystoneclient handles it here: https://github.com/openstack/python-keystoneclient/blob/master/keystoneclient/v3/role_assignments.py#L101-L122 | |
| 22:18:47 | mordred | adriant: that's ... that's just amazing | |
| 22:19:18 | lbragstad | "you had *one* job!" | |
| 22:19:25 | mordred | :) | |
| 22:19:36 | mordred | adriant: btw - I'm thrilled that your patch works | |
| 22:19:54 | mordred | adriant: it's almost like that system working as designed! | |
| 22:20:32 | adriant | lbragstad: at least adding the '_' variants to keystone should be easy | |
| 22:20:43 | adriant | then we'll have both! :P | |
| 22:20:53 | lbragstad | yeah - that's about the best we could do | |
| 22:21:09 | lbragstad | adriant: would you mind opening a bug? we should be able to target something like that to office hours | |
| 22:21:21 | adriant | will do | |
| 22:21:28 | lbragstad | ty sir | |
| 22:21:52 | adriant | lbragstad: also, I swear I'll find time next week to write up that blasted token/mfa spec... | |
| 22:21:54 | adriant | I swear... | |
| 22:22:31 | openstackgerrit | Monty Taylor proposed openstack/python-openstacksdk master: Allow not resolving outputs on get stacks https://review.openstack.org/548595 | |
| 22:23:36 | adriant | and mordred, if briancurtin was responsible for those query mapping things, then he deserves thanks :P | |
| 22:23:52 | adriant | It's nice being able to make this stuff just work | |
| 22:23:58 | mordred | ++ | |
| 22:26:59 | adriant | now back to writing audit scripts... We rolled out MFA to our cloud this week, and now as a follow up action I'm writing monitoring tools for: "complain if anyone other than this list has the 'admin' role", "complain if those with admin and not on the exempt list don't have MFA enabled" | |
| 23:08:00 | openstackgerrit | Merged openstack-infra/shade master: Add extra failure codes to bad request exception https://review.openstack.org/550107 | |
| 23:09:15 | openstackgerrit | Merged openstack-infra/shade master: Add proper return value for validate_node https://review.openstack.org/549981 | |
| 23:11:22 | olivierbourdon38 | thx to all involved for the reviews and merges of the shade patchsets | |
| 23:28:04 | openstackgerrit | Merged openstack/python-openstacksdk master: Run os-client-config tests on sdk changes https://review.openstack.org/549474 | |
| 23:34:11 | mordred | olivierbourdon38: thanks for thepatches! | |
| #openstack-sdks - 2018-03-09 | |||
| 05:43:26 | openstackgerrit | James Denton proposed openstack/python-openstacksdk master: Adds prefixlen to the request body when creating subnets https://review.openstack.org/550558 | |
| 07:50:01 | openstackgerrit | rarm nagalingam proposed openstack-infra/shade master: Adds toggle port security on network create https://review.openstack.org/550646 | |
| 09:55:13 | openstackgerrit | rarm nagalingam proposed openstack-infra/shade master: Adds toggle port security on network create https://review.openstack.org/550646 | |
| 09:59:06 | openstackgerrit | rarm nagalingam proposed openstack-infra/shade master: Adds toggle port security on network create https://review.openstack.org/550646 | |
| 10:05:22 | openstackgerrit | rarm nagalingam proposed openstack-infra/shade master: Adds toggle port security on network create https://review.openstack.org/550646 | |
| 11:35:09 | amotoki | mordred: I see you in the etherpad of ops meetup tokyo. I added some note based on conversations after the session. It looks like a wrong approach which tries to extend/inherit SDK class. I suggest him to propose masakari API to OpenStackSDK :) | |
| 12:25:48 | mordred | amotoki: yes please! | |
| 12:55:28 | openstackgerrit | James Denton proposed openstack/python-openstacksdk master: Adds prefixlen to the request body when creating subnets https://review.openstack.org/550558 | |
| 13:04:31 | frickler | when running any command without authentication I'm seeing "AttributeError: 'module' object has no attribute 'OpenStackConfigException'" instead of a proper error, is this a known issue? | |
| 13:31:19 | openstackgerrit | Merged openstack/python-openstacksdk master: Allow not resolving outputs on get stacks https://review.openstack.org/548595 | |
| 17:06:46 | openstackgerrit | Merged openstack/python-openstacksdk master: Update the invalid url in pages https://review.openstack.org/548149 | |
| 17:25:22 | openstackgerrit | Merged openstack/python-openstackclient master: Update links in README https://review.openstack.org/549159 | |
| 17:49:32 | cdent | lbragstad: I'm trying to parse the email and irc log you point api-sig folk at (about unified limits) and I think I'm missing a critical piece: What is the actual question? | |
| 17:50:17 | cdent | Or if not question, then problem? | |
| 17:50:21 | cdent | I'm lacking context | |
| 17:50:24 | lbragstad | cdent: my question specifically | |
| 17:50:51 | lbragstad | is does it make sense to support a PUT API in that way when it's not consistent with any other parts of keystone's API | |
| 17:50:53 | cdent | the PUT of multple limits, if getting rid of that is okay? | |
| 17:51:13 | lbragstad | well - it's a PUT, with a list of limits, and an ID in each limit | |
| 17:51:17 | lbragstad | which seems weird | |
| 17:51:28 | cdent | yes, that is weird | |
| 17:51:40 | lbragstad | i can see the batch update case | |
| 17:51:45 | cdent | and the reason it got in was because of the desire to do batch changes? | |
| 17:51:49 | lbragstad | but the batch create seems more critical | |
| 17:51:55 | lbragstad | right | |
| 17:51:57 | cdent | It seems to me that batch creates are much more common than batch updates, yeah? | |
| 17:52:05 | lbragstad | i would agree | |
| 17:52:06 | cdent | or rather not common | |
| 17:52:12 | cdent | but voluminous | |
| 17:52:20 | cdent | and upatess more granular | |
| 17:52:36 | lbragstad | right - if we associate an id to the thing, you can do a PATCH | |
| 17:52:42 | lbragstad | and that feels a little better | |
| 17:52:54 | cdent | also, in order for that PUT to be legit it would have to replace or update all of the limits | |
| 17:53:02 | lbragstad | true | |
| 17:53:41 | lbragstad | hmmm - i can see that being a big list if you just wanted to update a single limit | |
| 17:53:44 | cdent | So I would think removing PUT would be right, except for backward compat issues or whatever... | |
| 17:53:49 | cdent | yup | |
| 17:54:16 | lbragstad | ok - that makes sense | |
| 17:54:28 | lbragstad | the api is still marked as experimental and discoverable | |
| 17:55:09 | cdent | ah, good | |
| 17:55:33 | cdent | do you want me to respond to the message, or was above enough/useful? | |
| 17:56:18 | lbragstad | cdent: i can send a note linking to this conversation if that's easier, but certainly wouldn't want you to *not* weigh in on the conversation | |
| 17:56:46 | cdent | I'll go ahead and write something, for sake of completeness | |
| 17:57:01 | lbragstad | cdent: thanks for the input, we appreciate it | |
| 18:57:08 | openstackgerrit | eden nuriel proposed openstack-infra/shade master: added filters to inventory class and filters args to cmd/inventory.py - usefull for getting smaller set of cloud resources as the inventory when sharing a big project. https://review.openstack.org/530278 | |