| *** diablo_rojo is now known as Guest2326 | 00:47 | |
| *** marios is now known as marios|ruck | 05:04 | |
| *** rpittau|afk is now known as rpittau | 07:15 | |
| *** elodilles is now known as elodilles_afk | 08:10 | |
| *** gthiemon1e is now known as gthiemonge | 08:23 | |
| *** ykarel is now known as ykarel|lunch | 08:31 | |
| *** ysandeep|out is now known as ysandeep | 08:45 | |
| ttx | looking | 08:53 |
|---|---|---|
| ttx | yeah it's weird, can't find the source for the error. Maybe some temporary sync issue, rechecking | 09:03 |
| ttx | hmm, looks like we'll need fungi's help on this one. Summary being: puppet-panko temporarily had its puppet-release-jobs template removed, but now it's back to the same as the other puppet-* things, and yet it triggers a validation error that the others do not trigger at https://zuul.opendev.org/t/openstack/build/5bd631f96753468c9a74c3ce35db8c88/log/job-output.txt#22394 | 09:38 |
| *** ykarel|lunch is now known as ykarel | 09:38 | |
| opendevreview | Hervé Beraud proposed openstack/reno master: Moving to OFTC https://review.opendev.org/c/openstack/reno/+/796622 | 10:05 |
| hberaud | ttx: thanks | 10:11 |
| *** jbadiapa is now known as jbadiapa|afk | 10:48 | |
| *** ykarel_ is now known as ykarel | 10:54 | |
| opendevreview | Andrey Kurilin proposed openstack/releases master: Release rally 3.3.0 https://review.opendev.org/c/openstack/releases/+/796634 | 12:03 |
| *** diablo_rojo__ is now known as diablo_rojo | 12:35 | |
| *** jbadiapa|afk is now known as jbadiapa | 12:35 | |
| *** elodilles_afk is now known as elodilles | 13:04 | |
| fungi | ERROR: openstack-infra/project-config/zuul.d/projects.yaml no release job specified for openstack/puppet-panko, one of ['publish-to-pypi'] needs to be included in ['puppet-openstack-check-jobs', 'puppet-openstack-module-unit-jobs', 'puppet-openstack-integration-jobs-scenario001', 'puppet-openstack-beaker-jobs', 'release-notes-jobs-python3'] or no release will be published | 13:55 |
| fungi | that? | 13:55 |
| ttx | yeah.. that error does not show on all the other puppet-* things despite them being similarly configured | 13:56 |
| fungi | i guess the question is why that first list has only one element | 13:56 |
| ttx | that message always was a bit hard to parse | 13:57 |
| ttx | why that first list has only one element, and why it works for all the others | 13:57 |
| fungi | so maybe somewhere in this function things are going strangely: https://opendev.org/openstack/releases/src/branch/master/openstack_releases/project_config.py#L178-L284 | 13:58 |
| hberaud | I expected to see our job fixed once these changes would be reverted | 13:58 |
| fungi | in theory it should be matching this: https://opendev.org/openstack/releases/src/branch/master/openstack_releases/project_config.py#L156-L159 | 13:59 |
| hberaud | yes https://opendev.org/openstack/project-config/src/branch/master/zuul.d/projects.yaml#L4711 | 14:00 |
| fungi | is release_type not set correctly for that project? | 14:00 |
| ttx | yeah, and again, it works ok with puppet-ovn which is executed just before and is configured exactly the same. That's why i was thinking the git repo was stuck at a previous state | 14:00 |
| hberaud | good question | 14:00 |
| ttx | like the project-config checkout used in the job is stuck to teh pre-revert version | 14:01 |
| ttx | (i.e. before https://review.opendev.org/c/openstack/project-config/+/793891 merged to restore the config) | 14:02 |
| fungi | this is what should detect it's puppet: https://opendev.org/openstack/releases/src/branch/master/openstack_releases/cmds/validate.py#L689-L690 | 14:02 |
| hberaud | fungi: puppet-panko and puppet-ovn seems to be configured the same | 14:02 |
| fungi | so puppetutils.looks_like_a_module() is probably returning false | 14:02 |
| fungi | looks like that conditional chain defaults to python if everything falls through, which looks like what we're seeing in the log | 14:03 |
| fungi | https://opendev.org/openstack/releases/src/branch/master/openstack_releases/puppetutils.py#L19-L26 | 14:04 |
| fungi | i'm going to bet if i go looking i'll find most of the puppet files have been deleted from the master branch | 14:05 |
| ttx | hah | 14:05 |
| ttx | could be, since it was retired | 14:06 |
| hberaud | good catch https://opendev.org/openstack/puppet-panko/ | 14:08 |
| hberaud | we pass the repo.name to this function, so, maybe we could use it to determine the deliverable type too... at least temporarly and to avoid to revert the repo abandon | 14:13 |
| ttx | is the need temporary? | 14:13 |
| hberaud | nope you're right | 14:14 |
| hberaud | UVWX will follow the same path | 14:14 |
| hberaud | at least | 14:14 |
| ttx | we should hit that " if repo.is_retired:" | 14:17 |
| ttx | https://opendev.org/openstack/releases/src/branch/master/openstack_releases/project_config.py#L195 | 14:17 |
| ttx | and therefore skip it | 14:18 |
| ttx | so maybe the solution is just to add "retired" in the repo flags | 14:20 |
| ttx | let me try that | 14:21 |
| opendevreview | Hervé Beraud proposed openstack/releases master: Adding more way to determine the deliverable type https://review.opendev.org/c/openstack/releases/+/796683 | 14:23 |
| hberaud | fungi, ttx: ^ | 14:23 |
| ttx | I don;t think that's necessary | 14:23 |
| hberaud | ok | 14:24 |
| ttx | We have a test in there for retired repos | 14:24 |
| ttx | we just failed to flag puppet-panko as retired | 14:24 |
| opendevreview | Thierry Carrez proposed openstack/releases master: Puppet OpenStack: Create final stable/train release https://review.opendev.org/c/openstack/releases/+/792820 | 14:24 |
| ttx | I bet that case happened in the past and that was the solution for it | 14:24 |
| hberaud | I see | 14:25 |
| ttx | If that works ^ we should probably put that flag into puppet-panko for UVWX | 14:25 |
| hberaud | yes | 14:25 |
| hberaud | If That work that's the right solution to follow | 14:26 |
| fungi | does it also merit mentioning in the project-team-guide section on project deprecations and retirements? | 14:27 |
| hberaud | Yes | 14:28 |
| hberaud | Can't hurt | 14:28 |
| hberaud | ttx: As long as we are there can you have a look too => https://review.opendev.org/c/openstack/releases/+/796456 | 14:29 |
| hberaud | s/too/to | 14:29 |
| ttx | haha, good catch | 14:30 |
| ttx | wondering if we could have a test case checking that | 14:31 |
| opendevreview | Merged openstack/releases master: Push validations-common 1.2.0 min version https://review.opendev.org/c/openstack/releases/+/796243 | 14:31 |
| hberaud | a so tiny change with possibly so huge impacts | 14:31 |
| hberaud | yes that can't hurt | 14:31 |
| ttx | basically what you did in 796485, but as a test case | 14:32 |
| hberaud | Yes I'll have a look | 14:32 |
| ttx | ok commented to that effect | 14:33 |
| ttx | If the test framework makes it super hard to test, we can just merge this... but it feels like it should be able to cover it | 14:33 |
| hberaud | sure | 14:33 |
| hberaud | thanks for the comment | 14:33 |
| *** gthiemon1e is now known as gthiemonge | 14:37 | |
| *** ykarel is now known as ykarel|away | 14:39 | |
| hberaud | \o/ that worked | 14:43 |
| hberaud | fungi, ttx: https://review.opendev.org/c/openstack/project-team-guide/+/796687 | 14:51 |
| opendevreview | Hervé Beraud proposed openstack/releases master: retire panko https://review.opendev.org/c/openstack/releases/+/796689 | 14:58 |
| opendevreview | Hervé Beraud proposed openstack/releases master: retire puppet-panko https://review.opendev.org/c/openstack/releases/+/796690 | 15:05 |
| opendevreview | Rico Lin proposed openstack/releases master: Release yaql 1.1.4 https://review.opendev.org/c/openstack/releases/+/796692 | 15:06 |
| opendevreview | Hervé Beraud proposed openstack/releases master: retire panko and python-pankoclient https://review.opendev.org/c/openstack/releases/+/796689 | 15:25 |
| opendevreview | Merged openstack/releases master: Puppet OpenStack: Create final stable/train release https://review.opendev.org/c/openstack/releases/+/792820 | 15:33 |
| opendevreview | Hervé Beraud proposed openstack/releases master: Fix validation to ensure that first release of series isn't a bugfix https://review.opendev.org/c/openstack/releases/+/796456 | 15:36 |
| opendevreview | Merged openstack/reno master: Moving to OFTC https://review.opendev.org/c/openstack/reno/+/796622 | 15:38 |
| *** marios|ruck is now known as marios|out | 15:45 | |
| hberaud | fungi: We just experienced a new SSH issue in one job https://zuul.opendev.org/t/openstack/build/1815e820706347478ac24f7af132d33f I think that we need to reenqueue this job http://lists.openstack.org/pipermail/release-job-failures/2021-June/001541.html | 15:58 |
| opendevreview | Andrey Kurilin proposed openstack/releases master: Release rally 3.3.0 https://review.opendev.org/c/openstack/releases/+/796634 | 16:08 |
| *** rpittau is now known as rpittau|afk | 16:11 | |
| fungi | hberaud: thanks, i'll take a look in a moment | 16:17 |
| hberaud | fungi: sure, thanks | 16:17 |
| fungi | looks like the recent ssh host key conflict failures have been in rackspace, they must be struggling with rogue virtual machines in their network again | 16:20 |
| fungi | hberaud: i don't think we can reqnqueue that one, it failed after building and publishing release artifacts to tarballs.o.o and puppetforge... it didn't hit the error until it was collecting logs at the end of the job | 16:23 |
| hberaud | Ah I see | 16:23 |
| *** ricolin_ is now known as ricolin | 17:49 | |
| opendevreview | Merged openstack/releases master: [trove] Transition Ocata to End of Life https://review.opendev.org/c/openstack/releases/+/795682 | 18:20 |
| opendevreview | Merged openstack/releases master: [designate] Transition Ocata to End of Life https://review.opendev.org/c/openstack/releases/+/795650 | 18:48 |
| opendevreview | Merged openstack/releases master: [freezer] Transition Ocata to End of Life https://review.opendev.org/c/openstack/releases/+/795651 | 18:50 |
| opendevreview | Merged openstack/releases master: [zaqar] Transition Ocata to End of Life https://review.opendev.org/c/openstack/releases/+/795686 | 18:54 |
| opendevreview | Merged openstack/releases master: [vitrage] Transition Ocata to End of Life https://review.opendev.org/c/openstack/releases/+/795683 | 18:56 |
| opendevreview | Merged openstack/releases master: [tacker] Transition Ocata to End of Life https://review.opendev.org/c/openstack/releases/+/795678 | 18:59 |
| *** diablo_rojo is now known as Guest2446 | 21:51 | |
| *** diablo_rojo is now known as Guest2457 | 23:20 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!