| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2020-01-24 | |||
| 14:10:48 | frickler | actually maybe there still is something wrong with that patch, let me test locally a bit | |
| 16:18:08 | dtantsur | mordred, gtema, I'm considering an openstacksdk release, we need it for ironic to move on with migration. If there are no objects, I'll request it on Monday. | |
| 16:18:35 | gtema | yes, fully support this, since for a long time I depend on my imaging patch | |
| #openstack-sdks - 2020-01-25 | |||
| 09:33:16 | openstackgerrit | Radosław Piliszek proposed openstack/js-openstack-lib master: WIP: New nodejs jobs https://review.opendev.org/702132 | |
| 10:14:27 | openstackgerrit | Radosław Piliszek proposed openstack/js-openstack-lib master: WIP: New nodejs jobs https://review.opendev.org/702132 | |
| 10:24:07 | mordred | dtantsur|afk: ++ | |
| 10:24:17 | mordred | dtantsur|afk: link me to if when you do and I'll ack it | |
| #openstack-sdks - 2020-01-26 | |||
| 09:25:52 | frickler | mordred: this should fix the stable/stein testing https://review.opendev.org/704243 Fix branch for stable job testing | |
| 09:26:29 | frickler | I thinking that this pinning to master might generally be obsolete now that we do seem to have stable branches everywhere | |
| 09:30:47 | frickler | mordred: dtroyer: /me thinks we should also see stable branch patches here, proposed https://review.opendev.org/704244 Adapt branches for sdk projects in gerritbot | |
| 10:04:12 | yoctozepto | frickler, mordred, dtroyer: you might also like the patchbot we have in a few channels (e.g. #openstack-kolla) around (provided by timburke) - it responds to gerrit links like openstack to launchpad's | |
| 10:04:33 | yoctozepto | 11:04:22 <yoctozepto> testing... | |
| 10:04:34 | yoctozepto | 11:04:25 <patchbot> patch 704244 - project-config - Adapt branches for sdk projects in gerritbot - 1 patch set | |
| 10:04:34 | yoctozepto | 11:04:24 |
|
| 10:05:39 | yoctozepto | gmann: while we are at it, qa might also like it ^ | |
| 11:15:10 | frickler | yoctozepto: yes I've seen patchbot but I have a difficult time to decide whether I consider it more useful than noisy. I'd also prefer it to be infra-managed if people decide they want it | |
| 11:53:36 | yoctozepto | frickler: good point, it is a tradeoff but I'm obviously in favor of the extra help so that I don't have to type out descriptions of changes I am linking to | |
| #openstack-sdks - 2020-01-27 | |||
| 01:45:23 | mordred | dtantsur|afk: https://review.opendev.org/704264 Release 0.40.0 of openstacksdk | |
| 02:56:41 | openstackgerrit | Merged openstack/openstacksdk stable/stein: Fix branch for stable job testing https://review.opendev.org/704243 | |
| 03:13:43 | openstackgerrit | Merged openstack/openstacksdk stable/stein: Add fake support for global_request_id https://review.opendev.org/700897 | |
| 07:07:16 | frickler | mordred: dtantsur|afk: can we also have a stable/stein release with this in, please? ^^ | |
| 07:08:17 | frickler | I'm also wondering when would be a good time for a 1.0.0 release. would give some more numbering space for stable branches | |
| 10:23:53 | dtantsur | frickler: will request as well | |
| 10:31:34 | dtantsur | frickler: honestly, https://review.opendev.org/#/c/700897/ is sooo bad.. I'm having hard time signing off on a release with it | |
| 10:33:32 | dtantsur | I will wait for mordred who probably understands why this was approved | |
| 10:58:54 | frickler | dtantsur: the job is running against neutron stable/stein patches and was constantly failing there, although it seems to be fixed with the merge, so maybe we don't actually need the release as I had expected http://zuul.openstack.org/builds?job_name=openstacksdk-functional-devstack-networking&branch=stable%2Fstein | |
| 10:59:57 | dtantsur | frickler: what was the root cause? some project using recent features? cannot it be reverted on the consumer side? | |
| 11:00:10 | frickler | oh, wait, the successes were against openstacksdk, will have to wait for another result from neutron | |
| 11:00:19 | dtantsur | ah | |
| 11:00:22 | dtantsur | anyway | |
| 11:00:35 | dtantsur | we're about to release a new stub feature on a stable branch | |
| 11:00:36 | frickler | dtantsur: I think the issue came from running master tempest which requires the new attribute | |
| 11:00:57 | frickler | dtantsur: but I'm fine with waiting for further feedback from mordred | |
| 11:00:59 | dtantsur | so, the branchless nature of tempest bites us | |
| 11:01:19 | dtantsur | I think tempest must be compatible with every release of the SDK without us stabbing out features it uses incorrectly | |
| 11:01:30 | dtantsur | but it may be only me. I'll wait for mordred. | |
| 11:03:04 | frickler | dtantsur: oh, wait, if you look at e.g. http://zuul.openstack.org/build/44ccbf47065a435d98e54b4c604c66a2 it was an OSC call that was failing, so not tempest related this time | |
| 11:03:13 | dtantsur | hmmm? | |
| 11:04:05 | dtantsur | I see. Is it OSC using something new or the devstack plugin not constraining OSC properly? | |
| 11:06:10 | frickler | dtantsur: looking again, maybe https://review.opendev.org/704243 was the real issue and 700897 wasn't even needed anymore after that | |
| 11:06:43 | dtantsur | frickler: is it possible to try reverting it and have a job run with depends-on? | |
| 11:07:10 | frickler | so that issue was running master openstacksdk against stable/stein keystoneauth1 if I traced it correctly | |
| 11:07:23 | frickler | dtantsur: I guess I could do that, yes, give me a few | |
| 11:07:59 | dtantsur | ++ thx | |
| 11:08:26 | openstackgerrit | Jens Harbott (frickler) proposed openstack/openstacksdk stable/stein: Revert "Add fake support for global_request_id" https://review.opendev.org/704305 | |
| 11:12:20 | frickler | dtantsur: let's see what the CI has to say about this one: https://review.opendev.org/704307 DNM: Test openstacksdk revert | |
| 11:13:05 | dtantsur | frickler: ++ but zuul doesn't like something here | |
| 11:15:30 | frickler | dtantsur: ah yes, thx, I was editing out the other jobs, but not enough of them | |
| 12:11:39 | mordred | dtantsur, frickler: oh good - I didn't like backporting that either - let's fingers crossed that the revert works | |
| 12:19:51 | frickler | lgtm http://zuul.openstack.org/stream/7e749006afc448498fe34e31844e3286?logfile=console.log | |
| 12:23:51 | mordred | frickler: woot! | |
| 12:44:34 | openstackgerrit | Daniel Bengtsson proposed openstack/python-openstackclient master: Stop configuring install_command in tox and stop use pip. https://review.opendev.org/694484 | |
| 12:54:41 | dtantsur | mordred: thx for checking! FYI https://review.opendev.org/#/c/704300/ | |
| 12:58:11 | mordred | dtantsur: woot | |
| 12:59:12 | mordred | dtantsur: btw - I agree on a 1.0 - I've been hoping to get a minute to walk through and make sure we don't have any super gross things still laying around that need excising first. I'll try to do that next week when I'm properly back | |
| 13:06:22 | mordred | oh - wait - it was frickler mentioning a 1.0 ... | |
| 13:12:48 | frickler | crazy ideas I get in the morning ;) | |
| 13:26:35 | sshnaidm | just a reminder, this patch needs some love from openstacksdk cores: https://review.opendev.org/#/c/703342/ | |
| 13:34:40 | mordred | sshnaidm, dtantsur: I left a question. also - I left a +2 but not a +A because I'm only partially here and not 100% caught up and dtantsur had a question.. so I'd like to make sure dtantsur is happy before pulling the trigger | |
| 14:32:07 | dtantsur | ++ | |
| 14:33:02 | frickler | sshnaidm: for the job failure, I think ansible-collections-openstack-functional-devstack is missing openstack/ansible-collections-openstack as required-project, making it fail when not running against itself | |
| 14:33:47 | sshnaidm | frickler, I don't think you need it in required project, since it's defined in this project | |
| 14:33:56 | sshnaidm | the job actually pass on collections repo | |
| 14:34:21 | sshnaidm | seems like it's something weird in devstack post playbook, let's see the last run.. | |
| 14:34:23 | frickler | sshnaidm: yes, when running against the project itself, it is there, when running against other project, like openstacksdk, it is missing | |
| 14:34:44 | frickler | FileNotFoundError: [Errno 2] No such file or directory: '/home/zuul/src/opendev.org/openstack/ansible-collections-openstack' | |
| 14:36:32 | dtantsur | sshnaidm: the project where a job is defined in is not necessary added to required-projects | |
| 14:36:48 | dtantsur | (e.g. for ironic we explicitly add ironic to all its jobs to make them reusable) | |
| 14:38:49 | sshnaidm | hmm.. was sure it's smart enough, will check again | |
| 14:41:15 | openstackgerrit | Sagi Shnaidman proposed openstack/ansible-collections-openstack master: Add colleciton repo to required projects https://review.opendev.org/704332 | |
| 14:42:04 | openstackgerrit | Sagi Shnaidman proposed openstack/openstacksdk master: Fail a job for ansible modules with message https://review.opendev.org/703342 | |
| 14:42:11 | sshnaidm | let's try ^ | |
| 16:15:38 | sshnaidm | ok, the job pass, you're welcome to vote: https://review.opendev.org/#/c/703342/ | |
| 20:49:31 | openstackgerrit | Keith Berger proposed openstack/python-openstackclient stable/rocky: Fix openstack server list --deleted --marker option https://review.opendev.org/704422 | |
| #openstack-sdks - 2020-01-28 | |||
| 01:22:40 | openstackgerrit | Merged openstack/ansible-collections-openstack master: Add colleciton repo to required projects https://review.opendev.org/704332 | |
| 01:22:41 | openstackgerrit | Merged openstack/ansible-collections-openstack master: fix os_user_role for groups in multidomain context https://review.opendev.org/703930 | |
| 01:22:43 | openstackgerrit | Merged openstack/ansible-collections-openstack master: Fix argument spec for os_subnet and os_zone https://review.opendev.org/703554 | |
| 05:53:39 | openstackgerrit | Kendall Nelson proposed openstack/python-openstackclient master: Remove mention of Meetings from docs https://review.opendev.org/704494 | |
| 05:54:24 | openstackgerrit | Kendall Nelson proposed openstack/python-openstackclient master: Remove mention of meetings from docs https://review.opendev.org/704494 | |
| 06:03:15 | mordred | diablo_rojo: ++ | |
| 06:55:33 | openstackgerrit | Monty Taylor proposed openstack/python-openstackclient master: Stop testing python2 https://review.opendev.org/686590 | |
| 07:19:45 | openstackgerrit | Bram Verschueren proposed openstack/ansible-collections-openstack master: Add os_routers_info module https://review.opendev.org/703927 | |
| 07:28:42 | bverschueren | @sshnaidm: regarding https://review.opendev.org/#/c/703927/; shall I clear tests/unit/requirements.txt from unnecessary packages ? | |
| 07:30:36 | bverschueren | those were copied during the migration from the ansible repo but I guess they're not used for openstack testing | |
| 07:34:04 | sshnaidm | bverschueren, of course, if you see something unnecessary you can clean it, but in different patch please so we can manage this easily | |
| 07:35:40 | sshnaidm | bverschueren, would be great if you make some functional test here for new role: https://github.com/openstack/ansible-collections-openstack/blob/master/ci/roles/router/tasks/main.yml | |
| 07:36:57 | bverschueren | yes, I will have a look at it | |
| 07:37:29 | bverschueren | need to figure out how to run locally as I hadn't worked with collections before | |
| 12:30:33 | openstackgerrit | Merged openstack/openstacksdk master: Fail a job for ansible modules with message https://review.opendev.org/703342 | |
| 12:43:10 | openstackgerrit | Merged openstack/openstacksdk stable/stein: Revert "Add fake support for global_request_id" https://review.opendev.org/704305 | |
| 12:48:37 | openstackgerrit | Sagi Shnaidman proposed openstack/ansible-collections-openstack master: WIP: porting Openstack guidelines spec from Ansible https://review.opendev.org/704558 | |
| 13:54:05 | openstackgerrit | Bram Verschueren proposed openstack/ansible-collections-openstack master: Add os_routers_info module https://review.opendev.org/703927 | |
| 14:25:49 | openstackgerrit | Will Tome proposed openstack/ansible-collections-openstack master: fix constructed compose https://review.opendev.org/704582 | |
| 14:58:01 | openstackgerrit | Sagi Shnaidman proposed openstack/ansible-collections-openstack master: WIP: porting Openstack guidelines spec from Ansible https://review.opendev.org/704558 | |
| 15:23:09 | openstackgerrit | Sagi Shnaidman proposed openstack/ansible-collections-openstack master: Add rstcheck for ansible-test to lint RST files https://review.opendev.org/704590 | |
| 15:46:27 | openstackgerrit | Will Tome proposed openstack/ansible-collections-openstack master: fix constructed compose https://review.opendev.org/704582 | |
| 17:05:15 | pabelanger | hello | |
| 17:05:48 | pabelanger | it was pointed out to me in #ansible, that with the migration of ansible 'openstack' content to openstack project, a CLA is now required: https://opendev.org/openstack/project-config/src/branch/master/gerrit/acls/openstack/ansible-collections-openstack.config#L23 | |
| 17:05:55 | pabelanger | Is this correct | |
| 17:06:24 | pabelanger | mordred: sshnaidm ^ | |