| opendevreview | Ian Wienand proposed zuul/zuul-jobs master: promote-container-image: add promote_container_image_method https://review.opendev.org/c/zuul/zuul-jobs/+/879009 | 00:50 |
|---|---|---|
| opendevreview | Ian Wienand proposed zuul/zuul-jobs master: remove-registry-tag: role to delete tags from registry https://review.opendev.org/c/zuul/zuul-jobs/+/878614 | 00:50 |
| opendevreview | Ian Wienand proposed zuul/zuul-jobs master: promote-container-image: use generic tag removal role https://review.opendev.org/c/zuul/zuul-jobs/+/878740 | 00:50 |
| opendevreview | Ian Wienand proposed zuul/zuul-jobs master: remove-registry-tag: update docker age match https://review.opendev.org/c/zuul/zuul-jobs/+/878810 | 00:50 |
| *** Trevor is now known as Guest9453 | 02:03 | |
| opendevreview | Ian Wienand proposed zuul/zuul-jobs master: promote-container-image: use generic tag removal role https://review.opendev.org/c/zuul/zuul-jobs/+/878740 | 02:49 |
| opendevreview | Ian Wienand proposed zuul/zuul-jobs master: remove-registry-tag: update docker age match https://review.opendev.org/c/zuul/zuul-jobs/+/878810 | 02:49 |
| ianw | looks like per rax messages afs01.ord.openstack.org was migrated and rebooted | 04:57 |
| ianw | i've logged into it and it seems happy | 04:57 |
| ianw | afs in general seems happy | 04:58 |
| ianw | fungi/tonyb: i've written an auditor, it's basically working i think | 05:41 |
| ianw | https://paste.opendev.org/show/bdsKZlZJeXTFOz5AQFpz/ is a sample of "wheels we have for centos-8-x86_64 that do not exist on pypi" | 05:41 |
| ianw | i.e. these might be helping. the inverse of that would be stuff we could delete | 05:42 |
| opendevreview | Michal Nasiadka proposed openstack/project-config master: Stop using Storyboard for Kayobe https://review.opendev.org/c/openstack/project-config/+/879055 | 05:56 |
| ianw | it's too long to paste but here's a feeling for the inverse | 05:58 |
| ianw | https://paste.opendev.org/show/bbOg4dNp9OGykwaoAeMP/ | 05:58 |
| frickler | just for reference this is the job I had to check u-c builds without our wheels. it finally passed last month after a lot of cleanup had happened https://review.opendev.org/c/openstack/requirements/+/813292 | 06:21 |
| opendevreview | Michal Nasiadka proposed openstack/project-config master: Stop using Storyboard for Kayobe https://review.opendev.org/c/openstack/project-config/+/879055 | 06:23 |
| *** amoralej|off is now known as amoralej | 06:39 | |
| frickler | hmm, why does zuul comment twice on my config error here? seems redundant https://review.opendev.org/c/openstack/kolla-ansible/+/879132 | 06:45 |
| *** jpena|off is now known as jpena | 07:23 | |
| opendevreview | Harald Jensås proposed openstack/diskimage-builder master: Update satellite_repo labels + add env var https://review.opendev.org/c/openstack/diskimage-builder/+/879137 | 08:00 |
| noonedeadpunk | Hey folks, I was wondering what is the reason not to have a comment for re-gating failed check? To limit this possibility only to ones who have +W ? | 09:07 |
| frickler | noonedeadpunk: at least one of the reasons it to avoid having patches with hard-failing jobs re-enter gate by enforcing a fresh clean check run to happen first | 09:10 |
| frickler | gate failures are much more costly due to speculative merging | 09:10 |
| noonedeadpunk | well, you still could -+W kinda? | 09:12 |
| frickler | it also reduces the chance of patches that cause intermediate failures getting merged | 09:12 |
| frickler | noonedeadpunk: no, if zuul voted -2, you'll need a v+1 before the next +W takes effect | 09:12 |
| noonedeadpunk | ah, yes, true | 09:13 |
| fungi | frickler: each zuul tenant a config is read in will comment, so maybe that project is in multiple tenants? | 11:13 |
| fungi | though it would probably help for it to indicate the tenant names in such cases, since a config change could in theory be erroneous in one tenant and not another | 11:13 |
| frickler | fungi: I don't think kolla-ansible is used in a different tenant and I haven't found that in the zuul config, either | 11:38 |
| frickler | fungi: checking open reviews, https://review.opendev.org/c/openstack/releases/+/878143/ seems due now? | 11:38 |
| opendevreview | Merged openstack/project-config master: Stop using Storyboard for Kayobe https://review.opendev.org/c/openstack/project-config/+/879055 | 11:44 |
| opendevreview | Merged openstack/project-config master: Stop using Storyboard for Placement projects https://review.opendev.org/c/openstack/project-config/+/878932 | 11:44 |
| opendevreview | Merged openstack/project-config master: Unpin nodeset for tag-releases https://review.opendev.org/c/openstack/project-config/+/878472 | 11:44 |
| fungi | frickler: oh, i didn't see you linked the change. skimming to quickly, sorry. i guess you're referring to how zuul added two identical inline comments on the file? i agree that seems odd | 12:07 |
| frickler | fungi: ah, checking the logs it is different pipelines, check vs. check-arm64. maybe that's indeed something that could be added to the message | 12:13 |
| frickler | 2023-03-31 06:39:06,479 INFO zuul.Pipeline.openstack.check-arm64: [e: 76990013af184fa4a7fc07cf28a82d80] Reporting item <QueueItem e4e9a82d0b5b4acfb82a1edbacbc3f0c for <Change 0x7fe87e253310 | 12:13 |
| frickler | openstack/kolla-ansible 879132,1> in check-arm64>, actions: [<GerritReporter: failure>] | 12:13 |
| fungi | frickler: and yes, thanks for spotting 878143, i meant to merge it on monday. i've pushed an update to correct the effective date and we can approve 878144 | 12:14 |
| fungi | oh, yes, multiple pipelines will also comment once per pipeline. good catch | 12:14 |
| fungi | i agree, pipeline (and tenant?) name in the comments could help, though also i can see some benefit to keeping them short | 12:15 |
| *** amoralej is now known as amoralej|lunch | 12:17 | |
| frickler | well the comment is pretty verbose already anyway IMO, citing the complete failing config item plus some more lines around it. I'll see if I can come up with a patch | 12:18 |
| fungi | i concur, specificity trumps brevity for that | 12:19 |
| fungi | looks like "StorPool OpenStack CI" is commenting on changes with zuul config errors | 12:50 |
| fungi | actually, i guess it's probably only erroring on changes to openstack/project-config because it's complaining about "Pipelines may not be defined in untrusted repos..." | 12:53 |
| fungi | they need to adjust their tenant to limit what configuration they read from our repositories | 12:54 |
| frickler | fungi: yes, but that's been going on for quite some time now. so far I considered it to be just below the level where I'd start to do something about it | 13:05 |
| opendevreview | Merged openstack/project-config master: Replace old Antelope cycle key with 2023.2/Bobcat https://review.opendev.org/c/openstack/project-config/+/878144 | 13:08 |
| *** amoralej|lunch is now known as amoralej | 13:11 | |
| mnasiadka | fungi: coming back to setting Kayobe to read-only in Storyboard - can I ask for that now? :-) | 13:37 |
| fungi | mnasiadka: i've got it on my to do list (was watching the change to make sure it deployed), but i have ptg sessions for the next 5 hours straight so not sure if i will get to it right away | 13:53 |
| mnasiadka | fungi: Understandable, I wasn't saying now now, just that it can be done in a convenient time :) | 13:53 |
| fungi | yep, i have several to do so will batch them up hopefully later today | 13:54 |
| mnasiadka | thanks :) | 13:55 |
| *** amoralej is now known as amoralej|off | 16:05 | |
| *** jpena is now known as jpena|off | 16:17 | |
| opendevreview | Dr. Jens Harbott proposed openstack/project-config master: Add nested-virt-debian-bullseye label to nodepool https://review.opendev.org/c/openstack/project-config/+/879196 | 16:20 |
| slittle1_ | does gerrit/projects.yaml still accept 'upstream:<url>' when defining a new project ? | 16:28 |
| clarkb | it should | 16:28 |
| slittle1_ | it was a good way to init the opendev repo from an upstream source | 16:29 |
| opendevreview | Thales Elero Cervi proposed openstack/project-config master: Add virtualization (virt) repo to StarlingX https://review.opendev.org/c/openstack/project-config/+/879076 | 16:36 |
| fungi | slittle1_: yes, as far as i know we haven't broken that | 16:42 |
| fungi | slittle1_: but also if you don't supply that, the project creation system will create a valid .gitreview file and init the project for you with just that file in it, then you can do normal code review to add other files | 16:43 |
| fungi | if you're going to supply your own content for importing, it's recommended to include a .gitreview file in the repository being imported or add one in your first change | 16:45 |
| fungi | slittle1_: also in the case of 879076 you've supplied a repository with a default branch named "main" which may not match the default branch convention for your other projects | 16:45 |
| fungi | though it looks like you also created a branch named master in it | 16:46 |
| fungi | both of those are going to end up getting imported | 16:46 |
| clarkb | you have to take extra steps if you want the default branch to be main in gerrit and gitea (I don't think anyone has tested that in production yet either) | 16:54 |
| clarkb | but it is theoretically possible. I added support for it | 16:54 |
| fungi | yeah, in this case it's more that the project has branches named master and main and both and going to end up imported | 17:03 |
| slittle1_ | I set 'master as the defailt branch, deleted the .zul.yaml, and deleted the 'main' branch. Please recheck https://review.opendev.org/c/openstack/project-config/+/879076 | 18:34 |
| fungi | thanks! will do | 18:34 |
| slittle1_ | The main goal was to preserve a long commit history | 18:34 |
| slittle1_ | and not re-review it all | 18:35 |
| fungi | makes sense, that's the usual reason for using an import there | 18:35 |
| fungi | i was just initially confused because github was defaulting to the empty "main" branch | 18:36 |
| fungi | looks like the checker ran cleanly this time, zuul will probably leave its +1 once the linters job finishes in a few minutes | 18:36 |
| opendevreview | Merged openstack/project-config master: Add virtualization (virt) repo to StarlingX https://review.opendev.org/c/openstack/project-config/+/879076 | 18:45 |
| fungi | slittle1_: i've added your gerrit account as the initial member of the new starlingx-virt-core group | 19:17 |
| opendevreview | Ian Wienand proposed zuul/zuul-jobs master: promote-container-image: add promote_container_image_method https://review.opendev.org/c/zuul/zuul-jobs/+/879009 | 22:17 |
| opendevreview | Ian Wienand proposed zuul/zuul-jobs master: remove-registry-tag: role to delete tags from registry https://review.opendev.org/c/zuul/zuul-jobs/+/878614 | 22:17 |
| opendevreview | Ian Wienand proposed zuul/zuul-jobs master: promote-container-image: use generic tag removal role https://review.opendev.org/c/zuul/zuul-jobs/+/878740 | 22:17 |
| opendevreview | Ian Wienand proposed zuul/zuul-jobs master: remove-registry-tag: update docker age match https://review.opendev.org/c/zuul/zuul-jobs/+/878810 | 22:17 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!