| -@gerrit:opendev.org- Michael Kelly proposed: | 04:39 | |
| - [zuul/zuul-jobs] 871539: roles: Add git-submodule-init role https://review.opendev.org/c/zuul/zuul-jobs/+/871539 | ||
| - [zuul/zuul-jobs] 871679: roles: Add ensure-git-lfs https://review.opendev.org/c/zuul/zuul-jobs/+/871679 | ||
| - [zuul/zuul-jobs] 871680: roles: Add git-lfs-init https://review.opendev.org/c/zuul/zuul-jobs/+/871680 | ||
| -@gerrit:opendev.org- Michael Kelly proposed: | 04:42 | |
| - [zuul/zuul-jobs] 871539: roles: Add git-submodule-init role https://review.opendev.org/c/zuul/zuul-jobs/+/871539 | ||
| - [zuul/zuul-jobs] 871679: roles: Add ensure-git-lfs https://review.opendev.org/c/zuul/zuul-jobs/+/871679 | ||
| - [zuul/zuul-jobs] 871680: roles: Add git-lfs-init https://review.opendev.org/c/zuul/zuul-jobs/+/871680 | ||
| -@gerrit:opendev.org- Zuul merged on behalf of Ade Lee: [zuul/zuul-jobs] 874907: Add conditional for UA registration role https://review.opendev.org/c/zuul/zuul-jobs/+/874907 | 04:46 | |
| -@gerrit:opendev.org- Michael Kelly proposed: | 04:49 | |
| - [zuul/zuul-jobs] 871679: roles: Add ensure-git-lfs https://review.opendev.org/c/zuul/zuul-jobs/+/871679 | ||
| - [zuul/zuul-jobs] 871680: roles: Add git-lfs-init https://review.opendev.org/c/zuul/zuul-jobs/+/871680 | ||
| -@gerrit:opendev.org- Michael Kelly proposed: | 04:56 | |
| - [zuul/zuul-jobs] 871539: roles: Add git-submodule-init role https://review.opendev.org/c/zuul/zuul-jobs/+/871539 | ||
| - [zuul/zuul-jobs] 871679: roles: Add ensure-git-lfs https://review.opendev.org/c/zuul/zuul-jobs/+/871679 | ||
| - [zuul/zuul-jobs] 871680: roles: Add git-lfs-init https://review.opendev.org/c/zuul/zuul-jobs/+/871680 | ||
| -@gerrit:opendev.org- Michael Kelly proposed: | 04:59 | |
| - [zuul/zuul-jobs] 871679: roles: Add ensure-git-lfs https://review.opendev.org/c/zuul/zuul-jobs/+/871679 | ||
| - [zuul/zuul-jobs] 871680: roles: Add git-lfs-init https://review.opendev.org/c/zuul/zuul-jobs/+/871680 | ||
| -@gerrit:opendev.org- Zuul merged on behalf of Ade Lee: [zuul/zuul-jobs] 873893: Changes to make fips work on ubuntu https://review.opendev.org/c/zuul/zuul-jobs/+/873893 | 05:02 | |
| -@gerrit:opendev.org- Michael Kelly proposed: | 05:08 | |
| - [zuul/zuul-jobs] 871679: roles: Add ensure-git-lfs https://review.opendev.org/c/zuul/zuul-jobs/+/871679 | ||
| - [zuul/zuul-jobs] 871680: roles: Add git-lfs-init https://review.opendev.org/c/zuul/zuul-jobs/+/871680 | ||
| -@gerrit:opendev.org- Michael Kelly proposed: | 05:21 | |
| - [zuul/zuul-jobs] 871539: roles: Add git-submodule-init role https://review.opendev.org/c/zuul/zuul-jobs/+/871539 | ||
| - [zuul/zuul-jobs] 871679: roles: Add ensure-git-lfs https://review.opendev.org/c/zuul/zuul-jobs/+/871679 | ||
| - [zuul/zuul-jobs] 871680: roles: Add git-lfs-init https://review.opendev.org/c/zuul/zuul-jobs/+/871680 | ||
| -@gerrit:opendev.org- Michael Kelly proposed: | 05:42 | |
| - [zuul/zuul-jobs] 871539: roles: Add git-submodule-init role https://review.opendev.org/c/zuul/zuul-jobs/+/871539 | ||
| - [zuul/zuul-jobs] 871679: roles: Add ensure-git-lfs https://review.opendev.org/c/zuul/zuul-jobs/+/871679 | ||
| - [zuul/zuul-jobs] 871680: roles: Add git-lfs-init https://review.opendev.org/c/zuul/zuul-jobs/+/871680 | ||
| -@gerrit:opendev.org- Zuul merged on behalf of Simon Westphahl: [zuul/zuul] 874544: Only store trigger event info on queue item https://review.opendev.org/c/zuul/zuul/+/874544 | 15:18 | |
| -@gerrit:opendev.org- Benjamin Schanzel proposed on behalf of Tobias Henkel: [zuul/nodepool] 775797: Log openstack requests https://review.opendev.org/c/zuul/nodepool/+/775797 | 15:28 | |
| @scrungusss:matrix.org | hi guys. i'm struggling to understand inheritance in zuul. I have a patch here for magnum https://review.opendev.org/c/openstack/magnum/+/872755 that depends on my changes to magnum tempest tests here https://review.opendev.org/c/openstack/magnum-tempest-plugin/+/872759, and it seems to pull in my new tests i've written, and I've also made changes to the parent of one of these tests. however i suspect the errors that I'm getting are due to the parent from my patch not getting used, rather the parent from the master branch is. is that possible? | 16:08 |
|---|---|---|
| @fungicide:matrix.org | scrungus: make sure magnum-tempest-plugin is listed as a required-project for that job and is being installed from the on-disk checkout rather than from pypi | 16:57 |
| @fungicide:matrix.org | cross-project dependencies rely on having a list of which repositories should be included so zuul knows which ones to push a predicted future git state for, otherwise you're ending up with whatever git state was cached on the test node | 17:00 |
| @scrungusss:matrix.org | so this would be magnum-tempest-plugin as a required-project for magnum? how would this be done when the job is not defined in that .zuul file? I know that you can do | 17:04 |
| ``` | ||
| - job: | ||
| name: my-job | ||
| required-projects: | ||
| - stuff | ||
| ``` | ||
| but can you do | ||
| ``` | ||
| - project: | ||
| queue: magnum | ||
| check: | ||
| jobs: | ||
| - magnum-tempest-plugin-tests-api | ||
| required-projects: | ||
| - stuff | ||
| ``` | ||
| ? | ||
| @scrungusss:matrix.org | ahh this makes more sense. thanks! | 17:05 |
| @scrungusss:matrix.org | or would you define something like: | 17:08 |
| ``` | ||
| - job: | ||
| name: magnum-tempest-plugins-tests-api | ||
| parent: magnum-tempest-plugins-tests-api | ||
| required-projects: | ||
| - openstack/magnum-tempest-plugin | ||
| ``` | ||
| @fungicide:matrix.org | scrungus: oh, never mind. you're running the magnum-tempest-plugin-tests-api job which is defined in the magnum-tempest-plugin repository, so that repository will be an implicit required-project in this case. the problem likely lies elsewhere. i'll see if i can spot a smoking gun in the build log | 17:16 |
| @fungicide:matrix.org | yeah, the build log definitely shows zuul pushing the calculated git state for the magnum-tempest-plugin repo onto the test node | 17:25 |
| @fungicide:matrix.org | interesting that the job runs tox but doesn't collect tox's logs, so it's hard to see exactly what tox installed into the venv it ran from | 17:34 |
| @fungicide:matrix.org | the log also shows `venv-tempest installed: [...] magnum-tempest-plugin @ file:///opt/stack/magnum-tempest-plugin` which i think is where you're consuming it for the tests you're concerned about | 17:42 |
| @fungicide:matrix.org | the base pre-run also clearly says `opendev.org/openstack/magnum-tempest-plugin checked out to: 88c8ca650a36565531a5ccc70c1c6650a6d95e71 DNM: WIP Get tests passing on cluster-api` | 17:45 |
| @fungicide:matrix.org | and in the devstack pre-run you can see it rsync'ing `src/opendev.org/openstack/magnum-tempest-plugin` into `/opt/stack` | 17:47 |
| @fungicide:matrix.org | scrungus: as best i can tell, your dnm change's state was installed into the tempest venv, so this is where my limited expertise ends. probably you'll want to hit up the openstack qa folks in the #openstack-qa channel on the oftc irc network (reachable here in matrix via the oftc bridge) | 17:48 |
| @scrungusss:matrix.org | ok nice one, thanks for your help anyway! | 18:17 |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul] 873470: Match events to pipelines based on topic deps https://review.opendev.org/c/zuul/zuul/+/873470 | 21:49 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: | 22:26 | |
| - [zuul/zuul] 875574: Set layout state event ltime in delete-pipeline-state https://review.opendev.org/c/zuul/zuul/+/875574 | ||
| - [zuul/zuul] 875575: Avoid layout updates after delete-pipeline-state https://review.opendev.org/c/zuul/zuul/+/875575 | ||
| @jim:acmegating.com | fungi: Clark : i'm not sure if you recall from a week or two ago, we ran into a data corruption issue and used the `delete-pipeline-state` command. during that process, we found that we had to wait through two reconfigurations before the system recovered (one automatic, and one manual). those two changes remove the necessity of each of those reconfigurations (574 removes the 2nd manual one, 575 removes the 1st automatic one). so if we have to do that again (i hope not!), the process should be much faster. | 22:28 |
| @clarkb:matrix.org | nice! I'll try to review those soon, but I've found myself swamped under Gerrit related fallout | 22:29 |
| @clarkb:matrix.org | basically the first half of my day was spent doing things not already on my todo list :) | 22:29 |
| @jim:acmegating.com | Clark: no rush! just wanted to share :) | 22:30 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!