Earlier  
Posted Nick Remark
#openstack-sdks - 2020-03-13
10:49:11 noonedeadpunk also there are the way more usages _find_identity_resource....
10:49:46 noonedeadpunk actually I'm really wondering why osc thtows the same error for so different exceptions...
11:50:42 openstackgerrit Mark Chappell proposed openstack/ansible-collections-openstack master: os_nova_host_aggregate: Fixup sanity test failures https://review.opendev.org/712917
13:53:35 amoralej hi, wrt https://review.opendev.org/#/c/712904/, is osc-lib gate broken?
13:54:09 amoralej ERROR: Package 'neutron' requires a different Python: 2.7.17 not in '>=3.6'
14:06:22 mordred noonedeadpunk: yeah - that's the larger mess I was talking about - the same exception multiple causes thing
14:06:57 mordred BakaKuna: \o/
14:09:54 openstackgerrit Monty Taylor proposed openstack/python-openstackclient master: Don't look up project by id if given id https://review.opendev.org/712810
14:10:01 mordred noonedeadpunk: try that one
14:11:18 mordred amoralej|lunch: might be - let me look
14:14:05 openstackgerrit Monty Taylor proposed openstack/osc-lib master: Stop using lib-forward-testing https://review.opendev.org/712961
14:14:15 mordred amoralej|lunch: I think that should do it
14:16:40 noonedeadpunk mordred: smae thing for me
14:16:44 noonedeadpunk *same
14:18:35 mordred noonedeadpunk: it's still not working?
14:20:12 noonedeadpunk nope. I don't have even nothing common to `_is_uuid_like` in dir(openstack.connect())
14:20:40 noonedeadpunk openstacksdk 0.43.0
14:21:42 mordred yeah - sorry - bad patch - one sec
14:29:17 openstackgerrit Monty Taylor proposed openstack/python-openstackclient master: Don't look up project by id if given id https://review.opendev.org/712810
14:29:29 mordred noonedeadpunk: ok. one more try - hopefullyt his time with more coffee
14:31:57 noonedeadpunk yeah, if only I've found _is_uuid_like method...
14:33:16 mordred noonedeadpunk: I think we need to apply that pattern to many of the places in osc where identity resources are given
14:33:51 mordred for _most_ things treating input as "name_or_id" works well - but it's actually quite common that users don't have access rights to look up identity resources (like project) by name unless they are an admin
14:33:56 noonedeadpunk mordred: btw, I'd say that search works as well, it just has completely different URL base (so actually client_manager, which is probably do not exist)
14:34:04 mordred so the pattern _completely_ falls apart for things like project
14:34:36 noonedeadpunk they actually can, with https://keystone:5000/v3/users/USER_UUID/projects?name=vexxhost
14:34:50 noonedeadpunk the same should work for groups, etc
14:34:57 mordred awesome. good to know - we'll fix that too then
14:35:15 mordred noonedeadpunk: I think what I'll do is a pass to replace keystoneclient with sdk (similar to the glanceclient patch)
14:35:47 noonedeadpunk yeah, sdk usage is the way cleaner I think
14:35:48 mordred and as part of that we'll add in _is_uuid_like calls and then hopefully fix the project/groups/etc calls
14:35:55 mordred if they aren't already fixed
14:36:26 noonedeadpunk I think it's more workaround rather than fix
14:36:36 mordred is_uuid_like ?
14:36:53 mordred yeah - it's a workaround - although it does also save a roundtrip
14:37:07 noonedeadpunk as api allows regular users to search in projects they have access to by it's name
14:37:16 mordred ++
14:38:03 mordred yeah. I think that's the right call to use here
14:54:05 mordred Merged openstack/governance master: Merge SDK and OSC teams https://review.opendev.org/711265
14:54:10 mordred FYI ^^
14:55:30 openstackgerrit Alfredo Moralejo proposed openstack/osc-lib master: Replace assertItemsEqual with assertCountEqual https://review.opendev.org/712904
14:55:44 amoralej thanks mordred
15:02:12 smcginnis https://gph.is/2ax5JZu
15:15:32 openstackgerrit Monty Taylor proposed openstack/ansible-collections-openstack master: Remove MANIFEST.json before building https://review.opendev.org/712970
15:33:32 openstackgerrit Dmitriy Rabotyagov (noonedeadpunk) proposed openstack/python-openstackclient master: Optimize adding project to image slightly more https://review.opendev.org/712813
15:37:07 openstackgerrit Mark Chappell proposed openstack/ansible-collections-openstack master: os_nova_host_aggregate: Add support for not 'purging' missing hosts https://review.opendev.org/712979
15:49:58 openstackgerrit Mark Chappell proposed openstack/ansible-collections-openstack master: os_nova_host_aggregate: Fixup sanity test failures https://review.opendev.org/712917
15:49:59 openstackgerrit Mark Chappell proposed openstack/ansible-collections-openstack master: os_nova_host_aggregate: Add support for not 'purging' missing hosts https://review.opendev.org/712979
16:20:00 openstackgerrit Merged openstack/ansible-collections-openstack master: Add option for modules_default groups https://review.opendev.org/712057
16:54:25 openstackgerrit Sean McGinnis proposed openstack/os-client-config master: Use unittest.mock instead of third party mock https://review.opendev.org/713029
17:14:34 openstackgerrit Merged openstack/ansible-collections-openstack master: Remove MANIFEST.json before building https://review.opendev.org/712970
17:37:21 openstackgerrit Merged openstack/ansible-collections-openstack master: os_nova_host_aggregate: Fixup sanity test failures https://review.opendev.org/712917
17:43:07 openstackgerrit Merged openstack/ansible-collections-openstack master: os_nova_host_aggregate: Add support for not 'purging' missing hosts https://review.opendev.org/712979
18:46:13 openstackgerrit Merged openstack/openstacksdk master: baremetal: fail-less mode for wait_for_nodes_provision_state https://review.opendev.org/712637
18:51:29 openstackgerrit Sean McGinnis proposed openstack/os-client-config master: Uncap jsonschema https://review.opendev.org/713049
18:51:29 openstackgerrit Sean McGinnis proposed openstack/os-client-config master: Use unittest.mock instead of third party mock https://review.opendev.org/713029
19:51:40 openstackgerrit Monty Taylor proposed openstack/ansible-collections-openstack master: Remove all of the os_ prefixes https://review.opendev.org/713056
21:19:43 mnaser mordred: what do you think of https://review.opendev.org/#/c/708277/3 ? i think having it merge as is with a future of adding it to 'set' and stuff seems reasonable?
21:20:57 mordred mnaser: I agree
21:21:07 mordred mnaser: I think we should not block on perfectin
21:21:28 mnaser this is miles better than having to switch to swift client to do this operation anyways
21:23:44 openstackgerrit Merged openstack/python-openstackclient master: Incorrect title for service provider https://review.opendev.org/703600
21:23:45 openstackgerrit Merged openstack/python-openstackclient master: Update http links in docs https://review.opendev.org/567729
22:02:38 openstackgerrit Merged openstack/os-client-config master: Uncap jsonschema https://review.opendev.org/713049
22:05:46 openstackgerrit Monty Taylor proposed openstack/ansible-collections-openstack master: Remove all of the os_ prefixes https://review.opendev.org/713056
22:12:32 openstackgerrit Merged openstack/os-client-config master: Use unittest.mock instead of third party mock https://review.opendev.org/713029
22:34:37 openstackgerrit Merged openstack/python-openstackclient master: Add qos_network_policy_id to network port tests https://review.opendev.org/701786
22:51:53 openstackgerrit Merged openstack/openstacksdk master: Fixes get_user when identity responses are paged https://review.opendev.org/688704
22:53:48 openstackgerrit Merged openstack/python-openstackclient master: Add storage policy option to create container command https://review.opendev.org/708277
22:57:03 openstackgerrit Merged openstack/openstacksdk master: Switch to futurist for concurrency https://review.opendev.org/711301
#openstack-sdks - 2020-03-14
22:16:01 openstackgerrit Monty Taylor proposed openstack/python-openstackclient master: Build utility image for using osc https://review.opendev.org/711246
22:46:30 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Set max_microversion to 2.53 for hypervisors https://review.opendev.org/711294
23:26:41 openstackgerrit Merged openstack/osc-lib master: Stop using lib-forward-testing https://review.opendev.org/712961
#openstack-sdks - 2020-03-15
01:37:39 openstackgerrit Merged openstack/openstacksdk master: Set max_microversion to 2.53 for hypervisors https://review.opendev.org/711294
#openstack-sdks - 2020-03-16
06:37:15 openstackgerrit Vishakha Agarwal proposed openstack/python-openstackclient master: Adding options to user cli https://review.opendev.org/697444
06:40:19 openstackgerrit Vishakha Agarwal proposed openstack/python-openstackclient master: Adding options to user cli https://review.opendev.org/697444
06:45:45 openstackgerrit Vishakha Agarwal proposed openstack/python-openstackclient master: Adding options to user cli https://review.opendev.org/697444
07:00:56 openstackgerrit Daniel Bengtsson proposed openstack/python-openstackclient master: Stop configuring install_command in tox and stop use pip. https://review.opendev.org/694484
09:08:04 openstackgerrit Vishakha Agarwal proposed openstack/python-openstackclient master: Adding options to user cli https://review.opendev.org/697444
10:20:15 openstackgerrit Riccardo Pittau proposed openstack/openstacksdk master: Import generate_temp_url from swiftclient https://review.opendev.org/711640
10:39:37 openstackgerrit Vishakha Agarwal proposed openstack/python-openstackclient master: Adding options to user cli https://review.opendev.org/697444
14:33:33 openstackgerrit Merged openstack/python-openstackclient master: Stop configuring install_command in tox and stop use pip. https://review.opendev.org/694484
15:05:22 openstackgerrit Mark Chappell proposed openstack/ansible-collections-openstack master: Move default availability_zone docs into doc_fragment https://review.opendev.org/713254
15:05:23 openstackgerrit Mark Chappell proposed openstack/ansible-collections-openstack master: Fix doc requirements, and consistently require python 3.6+ https://review.opendev.org/713255
15:05:24 openstackgerrit Mark Chappell proposed openstack/ansible-collections-openstack master: Add missing type information to docs https://review.opendev.org/713256
15:05:25 openstackgerrit Mark Chappell proposed openstack/ansible-collections-openstack master: Clean up doc-required-mismatch https://review.opendev.org/713257
15:05:26 openstackgerrit Mark Chappell proposed openstack/ansible-collections-openstack master: Ensure defaults match spec https://review.opendev.org/713258
15:05:27 openstackgerrit Mark Chappell proposed openstack/ansible-collections-openstack master: Use dict rather than complex for os_quota return type. https://review.opendev.org/713259
15:05:28 openstackgerrit Mark Chappell proposed openstack/ansible-collections-openstack master: Ensure documented choices match spec https://review.opendev.org/713260
15:05:29 openstackgerrit Mark Chappell proposed openstack/ansible-collections-openstack master: Fixup elements related sanity failures https://review.opendev.org/713261
15:18:17 dtantsur mordred: hey! would be good to release openstacksdk from train to pick up the bare metal fixes.
15:20:00 mordred dtantsur: kk
15:21:13 mordred dtantsur: remote: https://review.opendev.org/713267 Release 0.36.2 of openstacksdk for train
15:21:25 dtantsur thanks!
15:22:15 mordred dtantsur: fwiw - you're listed as a release liason for openstacksdk - so feel free to propose releases if you need them
15:22:28 dtantsur if you did not respond, I would :)
15:22:35 mordred \o/
15:22:38 dtantsur there is always the aspect of laziness, right?
15:22:57 BakaKuna Hi all, is the floating IP of an instance available through the openstacksdk? I am now looping through all networks an instance is connected to, but that feels like a detour
15:23:56 BakaKuna I'm thinking of something like: `instance.floating_ip`, but maybe that is not possible to generalize for all deployments.

Earlier   Later