Earlier  
Posted Nick Remark
#openstack-sdks - 2020-03-19
14:31:50 openstackgerrit Mark Chappell proposed openstack/ansible-collections-openstack master: New module: os_keystone_mapping_info https://review.opendev.org/713686
15:02:01 openstackgerrit Merged openstack/openstacksdk master: Add support for federation mappings https://review.opendev.org/713633
15:25:00 openstackgerrit Monty Taylor proposed openstack/python-openstackclient master: Complete switch from glanceclient to SDK for image service https://review.opendev.org/699416
15:25:00 openstackgerrit Monty Taylor proposed openstack/python-openstackclient master: Switch image to use SDK https://review.opendev.org/650374
15:25:01 openstackgerrit Monty Taylor proposed openstack/python-openstackclient master: Optimize adding project to image slightly more https://review.opendev.org/712813
15:25:01 openstackgerrit Monty Taylor proposed openstack/python-openstackclient master: Don't look up project by id if given id https://review.opendev.org/712810
15:26:31 mordred slaweq: https://review.opendev.org/#/c/713698/ look ok to you?
15:31:39 slaweq mordred: looking
15:32:53 slaweq mordred: approved
15:34:53 mordred slaweq: woot - thanks
16:00:28 elmiko SIG API office hour now open
16:58:32 elmiko have a good weekend all, stay safe and healthy out there o/
17:05:47 openstackgerrit Merged openstack/python-openstackclient stable/train: Fix router create/show if extraroute not supported https://review.opendev.org/713607
17:34:14 openstackgerrit Merged openstack/python-openstackclient master: Allow setting floating IP description https://review.opendev.org/713698
21:18:10 openstackgerrit Monty Taylor proposed openstack/python-openstackclient master: Update image building jobs https://review.opendev.org/713983
21:18:29 openstackgerrit Monty Taylor proposed openstack/python-openstackclient master: Update image building jobs https://review.opendev.org/713983
#openstack-sdks - 2020-03-20
01:15:26 openstackgerrit Merged openstack/python-openstackclient master: Update image building jobs https://review.opendev.org/713983
09:46:52 openstackgerrit jayaditya gupta proposed openstack/python-openstackclient master: FIX for openstack quota set --force https://review.opendev.org/714062
10:42:51 openstackgerrit Sagi Shnaidman proposed openstack/ansible-collections-openstack master: Add dependencies for long jobs on linters https://review.opendev.org/714070
10:48:48 openstackgerrit Sagi Shnaidman proposed openstack/ansible-collections-openstack master: Add rocky, stein jobs https://review.opendev.org/714073
10:58:14 openstackgerrit Sagi Shnaidman proposed openstack/openstacksdk stable/train: Add ansible collection job to train branch https://review.opendev.org/714077
10:58:49 openstackgerrit Sagi Shnaidman proposed openstack/openstacksdk stable/stein: Add ansible collection job to stein branch https://review.opendev.org/714078
10:59:43 openstackgerrit Sagi Shnaidman proposed openstack/openstacksdk stable/rocky: Add ansible collection job to rocky branch https://review.opendev.org/714079
11:12:09 openstackgerrit Sagi Shnaidman proposed openstack/openstacksdk stable/rocky: Add tests for Ansible built-in modules in 2.8,2.9 https://review.opendev.org/714086
11:13:33 openstackgerrit Sagi Shnaidman proposed openstack/openstacksdk stable/stein: Add tests for Ansible built-in modules in 2.8,2.9 https://review.opendev.org/714087
11:14:46 openstackgerrit Sagi Shnaidman proposed openstack/openstacksdk stable/train: Add tests for Ansible built-in modules in 2.8,2.9 https://review.opendev.org/714089
11:15:30 openstackgerrit Bence Romsics proposed openstack/python-openstackclient master: Bump lower constraint of MarkupSafe https://review.opendev.org/714090
11:15:39 openstackgerrit Bence Romsics proposed openstack/python-openstackclient master: Add command: router add/remove route --route https://review.opendev.org/674325
12:19:16 openstackgerrit Brin Zhang proposed openstack/python-openstackclient master: WIP: Add function test for server migrate list https://review.opendev.org/681473
12:28:45 dtantsur elmiko: hey! please check https://review.opendev.org/713832 when you have a minute
12:34:38 elmiko dtantsur: done
12:42:25 openstackgerrit Roman Dobosz proposed openstack/openstacksdk master: Add bulk creation of rules for Security Group. https://review.opendev.org/714103
12:45:20 openstackgerrit Merged openstack/api-sig master: Verify JSON files in the CI https://review.opendev.org/713832
12:48:43 openstackgerrit Roman Dobosz proposed openstack/openstacksdk master: Add bulk creation of rules for Security Group. https://review.opendev.org/714103
13:48:44 openstackgerrit jayaditya gupta proposed openstack/python-openstackclient master: added tests code for --force, also added check for neutron and cinder https://review.opendev.org/714114
14:00:26 openstackgerrit Mark Chappell proposed openstack/openstacksdk master: Add support for not including the ID in creation requests https://review.opendev.org/714120
14:02:54 openstackgerrit Mark Chappell proposed openstack/openstacksdk master: Add support for Identity Providers https://review.opendev.org/714122
14:45:03 openstackgerrit jayaditya gupta proposed openstack/python-openstackclient master: added tests code for --force, also added check for neutron and cinder https://review.opendev.org/714114
14:58:53 openstackgerrit jayaditya gupta proposed openstack/python-openstackclient master: pep8 fix on test_quota.py https://review.opendev.org/714135
15:10:25 openstackgerrit jayaditya gupta proposed openstack/python-openstackclient master: pep8 fix on test_quota.py https://review.opendev.org/714135
15:12:32 nightmare_unreal hey can someone help me with rebase
15:12:45 nightmare_unreal i have 2 git review.
15:12:53 nightmare_unreal but it's for same bug
15:13:23 smcginnis nightmare_unreal: It's not a rebase issue. I've added comments on that second one and abandon it. If I've misunderstood you can restore it, but the original patch needs the pep8 fixes.
15:13:41 nightmare_unreal okay :) let me read that first :)
15:14:03 smcginnis nightmare_unreal: Just let me know if anything there doesn't make sense.
15:14:13 nightmare_unreal sure
15:23:01 nightmare_unreal blank line error by tox
15:23:21 smcginnis nightmare_unreal: If you want to push up an update to that review, I can take a look at it and see if I can figure out what's going on.
15:23:52 smcginnis Based on https://www.irccloud.com/pastebin/g2wfhXlK/error it looks like one file doesn't have a new line at the end, and the other might have two (so one extra?)
15:24:09 nightmare_unreal in a minute I think i figured out :)
15:24:10 nightmare_unreal thanks
15:24:20 smcginnis Great! :)
15:51:03 openstackgerrit jayaditya gupta proposed openstack/python-openstackclient master: added fix for pep8, releasenote, test case for --force flag https://review.opendev.org/714153
16:01:07 openstackgerrit jayaditya gupta proposed openstack/python-openstackclient master: FIX for openstack quota set --force https://review.opendev.org/713881
17:07:25 openstackgerrit Merged openstack/python-openstackclient master: Update the content about Import Format https://review.opendev.org/569931
17:07:26 openstackgerrit Merged openstack/python-openstackclient master: Replace port 35357 with 5000 for "auth_url" https://review.opendev.org/583514
17:09:40 openstackgerrit jayaditya gupta proposed openstack/python-openstackclient master: FIX for openstack quota set --force https://review.opendev.org/713881
18:59:44 openstackgerrit Merged openstack/python-openstackclient master: Now we can add description for role creation in OSC https://review.opendev.org/484355
#openstack-sdks - 2020-03-21
17:10:16 openstackgerrit jayaditya gupta proposed openstack/python-openstackclient master: FIX for openstack quota set --force https://review.opendev.org/713881
#openstack-sdks - 2020-03-22
14:08:23 openstackgerrit Sagi Shnaidman proposed openstack/openstacksdk master: Run fetch-subunit-output role conditionally https://review.opendev.org/714302
14:13:29 openstackgerrit Sagi Shnaidman proposed openstack/openstacksdk master: Add ansible stable-2.9 job and run 2.8 and 2.9 https://review.opendev.org/711471
14:14:35 sshnaidm folks, please review these two patches ^^ dtantsur|afk mordred odyssey4me
19:13:48 openstackgerrit Alexey Stupnikov proposed openstack/python-openstackclient stable/queens: Fix subnet host_routes error https://review.opendev.org/659545
21:36:47 openstackgerrit Merged openstack/ansible-collections-openstack master: Add dependencies for long jobs on linters https://review.opendev.org/714070
21:40:54 openstackgerrit Merged openstack/ansible-collections-openstack master: Add rocky, stein jobs https://review.opendev.org/714073
#openstack-sdks - 2020-03-23
01:28:34 openstackgerrit Sagi Shnaidman proposed openstack/openstacksdk master: Add ansible stable-2.9 job and run 2.8 and 2.9 https://review.opendev.org/711471
01:33:50 openstackgerrit Sagi Shnaidman proposed openstack/ansible-collections-openstack master: Don't fetch subunit in jobs https://review.opendev.org/714342
01:37:48 openstackgerrit Sagi Shnaidman proposed openstack/openstacksdk master: DNM test fetch_subunit https://review.opendev.org/714343
02:26:35 openstackgerrit Sam Morrison proposed openstack/openstacksdk master: Add availability zone and availability zone profile resources to load balancer https://review.opendev.org/714345
04:19:58 openstackgerrit Sam Morrison proposed openstack/openstacksdk master: Add availability zone and availability zone profile resources to load balancer https://review.opendev.org/714345
08:34:13 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Lay a foundation for the project cleanup https://review.opendev.org/700219
08:55:36 openstackgerrit Mark Chappell proposed openstack/openstacksdk master: Add support for not including the ID in creation requests https://review.opendev.org/714120
09:01:26 openstackgerrit Daniel Bengtsson proposed openstack/osc-lib master: Stop configuring install_command in tox and stop use pip. https://review.opendev.org/694510
09:17:02 openstackgerrit Mark Chappell proposed openstack/openstacksdk master: Add support for Identity Providers https://review.opendev.org/714122
09:29:29 rpittau good morning everyone, just a doubt looking at the code, in the service_description module, shouldn't we set skip_discovery to True for the proxy object if we have both endpoint_override and version_string? Precisely inside this "if" https://opendev.org/openstack/openstacksdk/src/branch/master/openstack/service_description.py#L174
10:38:10 dtantsur I'll look after the mandatory tea
10:59:00 dtantsur rpittau: why so? there is still information to discover, e.g. the precise location of a major version or available microversions
11:09:45 rpittau dtantsur: this is related to the migration of ironic-inspector to use openstacksdk for the ironic module, when running the functional test it fails with the current conf, I tracked down the issue to the discovery part between openstacksdk and keystone, when creating the proxy object
11:12:22 dtantsur rpittau: if you have problems with discovery, it usually means you've got a wrong URL
11:12:31 dtantsur and I do agree that the error message may be quite confusing
11:14:44 rpittau dtantsur: yeah, the current config has endpoint_override=http://url , I tried with different ones with no luck
11:15:00 dtantsur rpittau: unless you mock out keystoneauth, it will try accessing that
11:15:35 rpittau dtantsur: I also tried that, maybe I'm mocking it in the wrong place :/
11:16:02 dtantsur rpittau: just mock the hell out of get_client or how it's called now
11:16:11 dtantsur I don't think it's useful to go deeper
11:22:38 rpittau dtantsur: tried that too :/ again maybe I'm doing something wrong while mocking
11:23:04 dtantsur rpittau: do you have a patch with get_client mocked? if it's mocked properly, it cannot end up in keystoneauth
11:24:13 rpittau dtantsur: no, I didn't upload it as it was failing
11:24:44 ouroboros8 I have an open review ( https://review.opendev.org/#/c/713698/ ) for a change to the openstack cli, but one of the later Zuul build steps is failing (osc-promote-image). I've had a look at it but the logs don't really make much sense to me, and I'm not sure what the step is trying to achieve. Anyone able to give me some pointers?
11:26:50 dtantsur ouroboros8: your patch has merged. I'm not sure what that job does, but you may talk to #openstack-infra about it.
11:27:34 ouroboros8 thanks!
11:30:16 dtantsur rpittau: I suggest you upload it and we walk from there?
11:30:27 rpittau dtantsur: alright, will do
12:48:53 openstackgerrit Mark Chappell proposed openstack/openstacksdk master: Add support for Federation Protocols https://review.opendev.org/714431
13:02:20 openstackgerrit jayaditya gupta proposed openstack/python-openstackclient master: FIX for openstack quota set --force https://review.opendev.org/713881
13:09:18 openstackgerrit Monty Taylor proposed openstack/python-openstackclient master: Change dockerhub password https://review.opendev.org/714436
13:42:49 openstackgerrit Mark Chappell proposed openstack/openstacksdk master: Add support for Federation Protocols https://review.opendev.org/714431
13:55:58 mordred dtantsur, rpittau: another option past normal mocking is to use requests-mock ... although that's probably not a _awesome_ choice in the short-term if this isn't in sdk itself. I started work on building a fixture in sdk that can be used in other places: https://review.opendev.org/#/c/675187/ - but am not yet done
13:56:41 mordred not that it helps today - but the idea there would be to be able to make a FakeOpenStackCloud object and tell it what you want it to do and it'll do all the underlying requests-mock magic that we do in the sdk unit tests

Earlier   Later