Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2020-03-24
13:02:08 openstackgerrit Mark Chappell proposed openstack/ansible-collections-openstack master: New Module: os_keystone_identity_provider_info https://review.opendev.org/714633
13:05:38 smcginnis mordred: They publish v1 but support v2? *smh*
13:05:54 mordred smcginnis: yeah man. don't even ...
13:06:14 smcginnis It's anarchy... anarchy I say!
13:06:24 mordred smcginnis: however - they publish v1 on the "volume" service type and just never went down the volumev2 service type road
13:06:47 mordred so - in some ways, it's a very old issue that stems from their refusal to do a thing we never should have asked anyone to do as a project in the first place ;)
13:07:08 smcginnis ;)
13:07:20 mordred smcginnis: why can't we have nice things?
13:08:30 gtema because life is not interesting enough then
13:09:39 mordred good point
13:11:24 mordred gtema, dtantsur, smcginnis: this week is freeze for sdk - so we should probably get stuff landed
13:11:36 gtema oh yeah
13:12:14 mordred specifically, the gtema project cleanup patch https://review.opendev.org/#/c/700219/ - the glanceectomy - https://review.opendev.org/#/c/650374/
13:12:25 mordred and the patches that tremble is pushing above related to federation
13:13:07 openstackgerrit Monty Taylor proposed openstack/python-openstackclient master: Complete switch from glanceclient to SDK for image service https://review.opendev.org/699416
13:13:13 openstackgerrit Monty Taylor proposed openstack/python-openstackclient master: Don't look up project by id if given id https://review.opendev.org/712810
13:13:18 openstackgerrit Monty Taylor proposed openstack/python-openstackclient master: Optimize adding project to image slightly more https://review.opendev.org/712813
13:13:35 mordred gtema: looks like the tempest failure on your first glance patch is just a cinder race
13:13:37 mordred rechecked
13:13:50 gtema ah ok, I was already wondering
13:13:53 gtema thks
13:14:20 tremble mordred, Does that mean a version bump (freeing up the Ansible modules) is due soon?
13:14:21 mordred gtema: also - I added two patches on top of it that came from an issue reported by noonedeadpunk that are worth looking at real quick
13:14:28 mordred tremble: yup
13:15:02 gtema oki
13:15:50 tremble mordred, For what it's worth I have local Ansible modules + tests which seem to imply that the Protocols and IDP SDK pieces are working.
13:18:56 dtantsur mordred: looking
13:22:14 gtema wow, I can have a landing party today??
13:25:17 dtantsur +A for cleanup, will check the glancifction after lunch
13:25:54 gtema a bit late for lunch
13:25:57 gtema thks
13:26:33 dtantsur my lunch time can be used as an entropy source
13:31:41 mordred tremble: woot!
13:32:02 mordred dtantsur: mmm. tasty entropy
13:32:26 gtema cool, so we don't need any random generators anymore, just ask dtantsur when he has lunch today
13:44:06 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Add ansible stable-2.9 job and run 2.8 and 2.9 https://review.opendev.org/711471
13:45:51 tremble 2020-03-24 13:26:43.908035 | ubuntu-bionic | ERROR: could not install deps [-r/home/zuul/src/opendev.org/openstack/ansible-collections-openstack/test-requirements.txt]; v = InvocationError('/home/zuul/src/opendev.org/openstack/ansible-collections-openstack/.tox/pep8/bin/python -m pip install -r/home/zuul/src/opendev.org/openstack/ansible-collections-openstack/test-requirements.txt', 1)
13:45:57 tremble Is that a known flake?
13:52:10 dtantsur tremble: cannot tell without full logs, but likely
13:53:36 tremble dtantsur, I've got 4 zuul failures all complaining about not being able to install the tox dependencies.
13:54:07 dtantsur it may be a mirror glitch
13:54:37 tremble That would make sense
13:54:51 dtantsur e.g. this is a networking problem: https://zuul.opendev.org/t/openstack/build/61b24d96e0ec4fc7b1fe67841c64dbd6/log/job-output.txt#459
13:56:31 tremble ok, I'll give 'recheck' a shot.
14:07:03 openstackgerrit Merged openstack/python-openstackclient master: Add "fields" parameter to ListSecurityGroup query https://review.opendev.org/710825
14:52:02 openstackgerrit Merged openstack/openstacksdk master: Lay a foundation for the project cleanup https://review.opendev.org/700219
14:58:53 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Update Rackspace vendor profile for cinder v2 https://review.opendev.org/714630
15:01:58 openstackgerrit Merged openstack/openstacksdk master: Run fetch-subunit-output role conditionally https://review.opendev.org/714302
16:09:56 tremble dtantsur, Speaking of Ironic... https://903cda9ebd0e1994db88-a122e892f22a38ddc5f951fb4c0c5ba5.ssl.cf5.rackcdn.com/714120/2/gate/openstack-tox-lower-constraints/9e8f4d6/testr_results.html I don't see how I could have caused an additional failure there, especially since it was working...
16:10:21 dtantsur something is mocked what wasn't before..
16:10:30 dtantsur sorry, I have a headache and cannot reason well
16:16:54 openstackgerrit Merged openstack/ansible-collections-openstack master: Don't fetch subunit in jobs https://review.opendev.org/714342
16:16:59 gtema_ dtantsur, if you can please have a look also at https://review.opendev.org/#/c/699416/ - this is a follow-up of the de-glanceclientification for v1
16:17:31 dtantsur I will, when the headache is over
16:17:38 dtantsur (quite probably tomorrow..)
16:18:01 gtema_ oki, get well (if you still find Ibuprofen somewhere :)
16:25:18 dtantsur found already, assuming it helps (it doesn't always)
16:54:10 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Update Rackspace vendor profile for cinder v2 https://review.opendev.org/714630
17:19:09 openstackgerrit Merged openstack/python-openstackclient master: Bump lower constraint of MarkupSafe https://review.opendev.org/714090
17:54:10 openstackgerrit Merged openstack/cliff master: Fix nested argument groups with ignore conflict handler https://review.opendev.org/708882
18:36:00 openstackgerrit Merged openstack/python-openstackclient master: Honor endpoint override from config for volume https://review.opendev.org/714624
19:14:43 openstackgerrit Merged openstack/openstacksdk master: Add support for not including the ID in creation requests https://review.opendev.org/714120
20:36:48 openstackgerrit Sagi Shnaidman proposed openstack/openstacksdk stable/train: Run fetch-subunit-output role conditionally https://review.opendev.org/714770
20:36:59 openstackgerrit Sagi Shnaidman proposed openstack/openstacksdk stable/stein: Run fetch-subunit-output role conditionally https://review.opendev.org/714771
20:40:27 openstackgerrit Sagi Shnaidman proposed openstack/openstacksdk stable/rocky: Run fetch-subunit-output role conditionally https://review.opendev.org/714774
21:24:52 openstackgerrit Merged openstack/python-openstackclient master: Add unit tests and release note for dns_publish_fixed_ip https://review.opendev.org/702266
21:34:44 openstackgerrit Merged openstack/python-openstackclient master: Fix faulthy state argument choice https://review.opendev.org/699931
22:18:58 openstackgerrit Monty Taylor proposed openstack/ansible-collections-openstack master: Add ansible-collections-openstack to the required-projects https://review.opendev.org/714784
23:32:30 openstackgerrit Monty Taylor proposed openstack/ansible-collections-openstack master: Update the badge shown in galaxy https://review.opendev.org/714795
#openstack-sdks - 2020-03-25
00:26:13 openstackgerrit Merged openstack/openstacksdk master: Add bulk creation of rules for Security Group. https://review.opendev.org/714103
06:40:46 openstackgerrit Merged openstack/ansible-collections-openstack master: Add ansible-collections-openstack to the required-projects https://review.opendev.org/714784
06:42:35 openstackgerrit Merged openstack/ansible-collections-openstack master: Update the badge shown in galaxy https://review.opendev.org/714795
07:08:30 openstackgerrit Merged openstack/openstacksdk master: Add support for Identity Providers https://review.opendev.org/714122
09:13:00 openstackgerrit Sagi Shnaidman proposed openstack/openstacksdk stable/rocky: Fix pep8 issues in project https://review.opendev.org/714899
09:36:13 openstackgerrit Sagi Shnaidman proposed openstack/openstacksdk stable/rocky: Fix pep8 issues in project https://review.opendev.org/714899
10:02:40 openstackgerrit Mark Chappell proposed openstack/openstacksdk master: Add support for Federation Protocols https://review.opendev.org/714431
10:07:59 sshnaidm dtantsur, mordred, cores, can you please take a look at https://review.opendev.org/#/c/714899 ? I don't really understand what Jens Harbott means, maybe I don't have a right context. But something should be done there to restore the gates. I don't know why fixing pep8 is such a problem, tbh.
10:19:58 openstackgerrit Jens Harbott (frickler) proposed openstack/openstacksdk stable/rocky: Cap hacking to avoid pep8 failures https://review.opendev.org/714911
10:20:36 frickler sshnaidm: that's me, btw, and ^^ is my proposed alternative. not sure test-reqs.txt is the correct location, but I hope so
10:21:02 sshnaidm frickler, great, thanks!
10:21:07 sshnaidm let's see if it helps
10:21:39 sshnaidm but there are a lot of pep8 issues, including non-4 indentation, etc
10:27:54 dtantsur thanks frickler, I guess we should do it on all branches?
10:28:15 sshnaidm frickler, it fails, on pep8 and requirements-check now
10:33:48 rpittau frickler: I left a comment in the review
11:09:45 openstackgerrit Jens Harbott (frickler) proposed openstack/openstacksdk stable/rocky: Cap hacking to avoid pep8 failures https://review.opendev.org/714911
11:10:38 frickler rpittau: seems you were right, not sure why we do not pull in our local (test-)reqs.txt in the pep8 env, but this should fix the errors
11:11:46 rpittau frickler: yeah, that should work
11:12:06 frickler dtantsur: I didn't see pep8 failures for newer branches
11:21:59 dtantsur frickler: they're probably fixed, but I think at some point we (as a community) came to a resolution to cap hacking
11:37:37 frickler dtantsur: indeed, with https://review.opendev.org/640597 mordred fixed it the way I would have expected for train and beyond
11:37:50 frickler not sure about stein, though
11:38:07 dtantsur frickler: should we backport that instead of your patch maybe?
11:39:59 frickler dtantsur: maybe, let's wait what mordred thinks about it, did a -W on my patch in the meantime
11:41:51 dtantsur frickler: I personally find it quite non-controversial. but I'm fine with waiting as well.
11:44:18 tremble dtantsur, Can I interest you in https://review.opendev.org/#/c/714431/ and get the last of my SDK changes in?
11:44:46 dtantsur you can try :)
11:50:54 dtantsur +2, a couple of minor things for your optional consideration
11:52:46 tremble dtantsur, The brackets nit is an old habit from switching between programming languages I'm never 100% exactly what the evaluation order is and brackets make it obvious
11:53:12 gtema tremble - will you do a followup or change it here?

Earlier   Later