Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2018-03-08
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
22:59:20 openstackgerrit Merged openstack/cliff master: Align parsed() call with cmd2 versions >= 0.7.3 https://review.openstack.org/548625
#openstack-sdks - 2018-03-10
00:09:19 openstackgerrit Merged openstack/cliff master: Fix cmd2 doc URL https://review.openstack.org/548063
08:25:38 frickler dtroyer: while testing https://review.openstack.org/550105 I was able to reproduce the SSL issue mentioned earlier. your patch introduces it for the "add fip" command
08:26:27 frickler dtroyer: for "openstack floating ip list" it is present in earlier versions and git bisect points to https://review.openstack.org/523938
08:28:09 frickler dtroyer: it might also be relevant that when I test this on a local devstack, running any OSC command from .tox/py27 only works if I add the "--insecure" option, which I don't need when running the devstack-installed global OSC version
08:29:12 frickler so the quintessence seems to be that 523938 seems to break the network client from being passed the insecure option properly
08:39:08 frickler it seems that there are in fact two issues with that, I need this double patch in order to resolve the issue for me http://paste.openstack.org/show/697635/
13:05:53 openstackgerrit OpenStack Proposal Bot proposed openstack-infra/shade master: Updated from global requirements https://review.openstack.org/551449
13:09:06 openstackgerrit OpenStack Proposal Bot proposed openstack/cliff master: Updated from global requirements https://review.openstack.org/548672
13:17:05 openstackgerrit OpenStack Proposal Bot proposed openstack/keystoneauth master: Updated from global requirements https://review.openstack.org/549536
13:39:34 openstackgerrit OpenStack Proposal Bot proposed openstack/openstackclient master: Updated from global requirements https://review.openstack.org/521902
13:39:52 openstackgerrit OpenStack Proposal Bot proposed openstack/os-api-ref master: Updated from global requirements https://review.openstack.org/551592
13:40:17 openstackgerrit OpenStack Proposal Bot proposed openstack/os-client-config master: Updated from global requirements https://review.openstack.org/551595
13:40:41 openstackgerrit OpenStack Proposal Bot proposed openstack/os-service-types master: Updated from global requirements https://review.openstack.org/551598
13:41:37 openstackgerrit OpenStack Proposal Bot proposed openstack/osc-lib master: Updated from global requirements https://review.openstack.org/551601
13:50:01 openstackgerrit OpenStack Proposal Bot proposed openstack/python-openstackclient master: Updated from global requirements https://review.openstack.org/544222
13:50:13 openstackgerrit OpenStack Proposal Bot proposed openstack/python-openstacksdk master: Updated from global requirements https://review.openstack.org/551631
14:06:59 openstackgerrit Merged openstack-infra/shade master: Updated from global requirements https://review.openstack.org/551449
14:11:21 openstackgerrit Monty Taylor proposed openstack/os-client-config master: Replace guts with openstack.config https://review.openstack.org/549307
14:43:44 openstackgerrit Merged openstack-infra/shade master: Adds toggle port security on network create https://review.openstack.org/550646
15:01:17 openstackgerrit Merged openstack/python-openstacksdk master: Redo role assignment list query filters https://review.openstack.org/550700

Earlier   Later