Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2020-03-25
21:11:53 mordred so - this is something we've fixed before and forgotten about :)
21:23:39 sshnaidm but why need overrides..
21:29:35 mordred sshnaidm: probably don't
21:29:44 mordred sshnaidm: s/probably//
22:10:25 openstackgerrit Merged openstack/python-openstackclient master: Don't look up project by id if given id https://review.opendev.org/712810
22:46:56 openstackgerrit Merged openstack/openstacksdk master: Update Rackspace vendor profile for cinder v2 https://review.opendev.org/714630
23:55:56 openstackgerrit Merged openstack/openstacksdk stable/train: Remove master override for openstacksdk https://review.opendev.org/715065
#openstack-sdks - 2020-03-26
07:17:25 openstackgerrit Merged openstack/openstacksdk master: [tests] Improve devstack/post playbook efficiency https://review.opendev.org/715052
09:24:58 openstackgerrit Sagi Shnaidman proposed openstack/openstacksdk stable/train: Add ansible collection job to train branch https://review.opendev.org/714077
10:05:49 openstackgerrit Bence Romsics proposed openstack/python-openstackclient master: Add command: router add/remove route --route https://review.opendev.org/674325
10:40:45 openstackgerrit Sean McGinnis proposed openstack/openstacksdk master: Add cap for hacking https://review.opendev.org/715155
12:16:58 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Change default image type in the OTC vendor profile https://review.opendev.org/715183
12:27:52 mordred smcginnis: your hacking patch hit a set of errors that were apparently getting missed before - you wanna fix them or I can?
12:28:37 smcginnis mordred: I can. Confused on why adding a cap introduced failures though.
12:29:10 smcginnis mordred: Looks like something else going on there - fixtures._fixtures.timeout.TimeoutException
12:29:42 smcginnis But I can fix up the pep8 issues at least.
12:29:48 mordred smcginnis: yeah - that seems super unlikely to be caused by flake8 ... but oy
12:30:28 smcginnis "F821 undefined name 'Optional'" Wasn't that just added?
12:31:49 smcginnis Hmm, and doesn't fail locally for me.
12:32:12 tremble smcginnis, I kept hitting those timeout exceptions too, I think there's a flakey test somewhere.
12:32:14 mordred smcginnis: I LOVE when flake8 is weird like that
12:32:41 smcginnis At least a few of the things are legitimate. Might need to add some ignores in there for others.
12:32:46 mordred smcginnis: maybe it has something to do with the python version?
12:33:01 smcginnis And hopefully another pass through the check queue will get around those UT failures.
12:33:08 mordred the Optional error would be solved by importing Optional from typing
12:33:39 smcginnis Ohhhh, interesting...
12:33:59 smcginnis Current master that states hacking>=2.0.0 results in "hacking==1.1.0" getting installed.
12:34:09 smcginnis And with the patch just adding a cap, 2.0.0 gets installed.
12:34:21 mordred aroo?
12:34:36 mordred can we agree that doesn't make sense?
12:34:37 smcginnis Glad I just pulled a shot.
12:34:44 smcginnis Completely.
12:35:01 mordred tremble: new release of sdk is winding its way through the machinery
12:36:03 smcginnis OK, I need more coffee.
12:36:14 smcginnis I am NOT just adding a cap.
12:36:19 sshnaidm odyssey4me, hi, are you going to backport https://review.opendev.org/#/c/715052/ to stable branches?
12:36:33 smcginnis I actually am bumping hacking to 2.0, which now makes total sense why it would trigger new errors.
12:36:39 odyssey4me sshnaidm: yep, happy to once the preceding patches merge
12:36:50 openstackgerrit Bence Romsics proposed openstack/python-openstackclient master: Add command: router add/remove route --route https://review.opendev.org/674325
12:36:52 smcginnis And obviously needed more coffee when I did that in the first place, because I should have expected new things to pop up.
12:38:12 sshnaidm odyssey4me, preceding patches..?
12:38:20 mordred smcginnis: haha. well, that at least makes more sense?
12:38:34 odyssey4me sshnaidm: https://review.opendev.org/#/q/I50b1020c896d7d8a7d58e62de778bb8c2b6e970e - I'll pick the patch on top of those
12:39:05 openstackgerrit Jesse Pretorius (odyssey4me) proposed openstack/openstacksdk stable/train: [tests] Improve devstack/post playbook efficiency https://review.opendev.org/715192
12:39:08 sshnaidm odyssey4me, I can just abandon mines, you have a full solution in yours
12:40:00 openstackgerrit Sagi Shnaidman proposed openstack/openstacksdk stable/train: Run fetch-subunit-output role conditionally https://review.opendev.org/714770
12:40:16 openstackgerrit Igor Malinovskiy proposed openstack/python-openstackclient master: Add 'address_scope' type support to network rbac commands https://review.opendev.org/709124
12:40:37 openstackgerrit Jesse Pretorius (odyssey4me) proposed openstack/openstacksdk stable/stein: [tests] Improve devstack/post playbook efficiency https://review.opendev.org/715193
12:40:40 sshnaidm odyssey4me, train and stein should be fine, rocky is waiting for pep8 fix
12:41:15 openstackgerrit Jesse Pretorius (odyssey4me) proposed openstack/openstacksdk stable/train: [tests] Improve devstack/post playbook efficiency https://review.opendev.org/715192
12:41:16 openstackgerrit Igor Malinovskiy proposed openstack/python-openstackclient master: Add 'subnetpool' type support to rbac commands https://review.opendev.org/712705
12:41:31 tremble mordred, zo/
12:41:32 odyssey4me sshnaidm: either way works - I've picked them already
12:41:34 tremble mordred, \o/
12:42:11 tremble mordred, Anything I can watch to see it go through?
12:42:56 mordred tremble: https://review.opendev.org/#/c/715090/
12:43:03 mordred although it looks like it just went through
12:44:03 mordred tremble: there's release jobs in https://zuul.opendev.org/t/openstack/status for the openstack/releases project that will actually publish to pypi
12:45:44 tremble Will the test jobs pick it up already?
12:47:26 openstackgerrit Sean McGinnis proposed openstack/openstacksdk master: Raise hacking to 2.x https://review.opendev.org/715155
12:48:06 smcginnis tremble: If you filter on "release", there is a job that just started in teh release-post queue.
12:48:19 smcginnis That will tag the repo, among other things.
12:48:37 smcginnis Once that tagging happens, that triggers yet another job to run that will package it up and publish to pypi.
12:48:55 smcginnis So probably 15-20 minutes or so until it's published and available.
12:49:57 tremble Jobs triggering jobs triggering jobs. That looks like the kind of crazy I'd set up.
12:51:08 smcginnis ;)
12:58:48 sshnaidm mordred, can we merge? https://review.opendev.org/#/c/714771/ odyssey4me set his backport on top of that
13:00:08 mordred sshnaidm: done
13:00:29 sshnaidm mordred, thanks, odyssey4me ^^
13:00:41 odyssey4me np :)
13:05:13 openstackgerrit Pedro Henrique Pereira Martins proposed openstack/python-openstackclient master: Add description field to portforwarding NAT rules https://review.opendev.org/714512
13:15:01 openstackgerrit Mark Chappell proposed openstack/ansible-collections-openstack master: New Module: os_keystone_mapping https://review.opendev.org/713643
13:15:02 openstackgerrit Mark Chappell proposed openstack/ansible-collections-openstack master: New module: os_keystone_mapping_info https://review.opendev.org/713686
13:15:03 openstackgerrit Mark Chappell proposed openstack/ansible-collections-openstack master: New Module: os_keystone_identity_provider https://review.opendev.org/714632
13:15:04 openstackgerrit Mark Chappell proposed openstack/ansible-collections-openstack master: New Module: os_keystone_identity_provider_info https://review.opendev.org/714633
13:15:05 openstackgerrit Mark Chappell proposed openstack/ansible-collections-openstack master: New Module: os_keystone_federation_protocol https://review.opendev.org/714991
13:15:06 openstackgerrit Mark Chappell proposed openstack/ansible-collections-openstack master: New Module: os_keystone_federation_protocol_info https://review.opendev.org/714992
13:28:37 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Add link to image sharing api docs https://review.opendev.org/677266
13:47:18 openstackgerrit Monty Taylor proposed openstack/python-openstackclient master: Add libc6-dev to bindep https://review.opendev.org/715210
13:50:14 openstackgerrit Monty Taylor proposed openstack/python-openstackclient master: Be explicit about python version in image https://review.opendev.org/715211
13:50:50 mordred amoralej|lunch: ^^ the libc6-dev patch should fix the https://review.opendev.org/#/c/697444/ patch
13:51:06 mordred gah
13:51:13 mordred amotoki: ^^
13:51:24 mordred amoralej|lunch: sorry - bad tab-complete
13:51:41 openstackgerrit Monty Taylor proposed openstack/python-openstackclient master: Adding options to user cli https://review.opendev.org/697444
14:28:51 gtema nodepool builds seem to be failing everywhere in SDK
14:29:01 gtema I mean nodepool jobs
14:30:17 mordred there's an issue in setuptools
14:30:22 mordred that has been fixed
14:30:25 gtema ah, ok
14:30:42 mordred but it's gonna be a minute before it winds its way through the layers of things to dib
14:30:56 gtema oki, keep waiting ;)
14:30:59 mordred (the bug caused non-entrypoint scripts to lose their executable bit when they were installed)
14:31:12 gtema cool
14:31:17 mordred yeah. so awesome
14:31:51 smcginnis Hah, oops.
14:31:53 gtema but from our logs you don't see this really - just that it failed
14:55:38 mordred gtema_: have a sec for 2 easy patches: https://review.opendev.org/#/c/715210/ https://review.opendev.org/#/c/715211/ ?
14:58:00 gtema yupp, looking now
14:59:30 mordred gtema: thanks!

Earlier   Later