| *** tosky has quit IRC | 00:11 | |
| *** cloudnull has joined #zuul | 03:15 | |
| *** ykarel has joined #zuul | 03:16 | |
| *** akrpan-pure has joined #zuul | 03:34 | |
| *** akrpan-pure has quit IRC | 04:34 | |
| *** cloudnull has quit IRC | 04:58 | |
| *** cloudnull has joined #zuul | 05:00 | |
| *** akrpan-pure has joined #zuul | 05:20 | |
| *** cloudnull has quit IRC | 05:31 | |
| *** evrardjp has quit IRC | 05:33 | |
| *** cloudnull has joined #zuul | 05:33 | |
| *** evrardjp has joined #zuul | 05:33 | |
| *** akrpan-pure has quit IRC | 05:49 | |
| *** cloudnull has quit IRC | 06:13 | |
| *** jfoufas1 has joined #zuul | 06:14 | |
| *** cloudnull has joined #zuul | 06:14 | |
| *** etp has quit IRC | 06:54 | |
| *** decimuscorvinus_ has quit IRC | 07:00 | |
| *** etp has joined #zuul | 07:01 | |
| *** decimuscorvinus has joined #zuul | 07:05 | |
| *** jfoufas1 has quit IRC | 07:08 | |
| *** cloudnull has quit IRC | 07:10 | |
| *** piotrowskim has joined #zuul | 07:12 | |
| *** cloudnull has joined #zuul | 07:15 | |
| *** etp has quit IRC | 07:17 | |
| *** etp has joined #zuul | 07:19 | |
| *** ykarel has quit IRC | 07:20 | |
| *** ykarel has joined #zuul | 07:26 | |
| openstackgerrit | Albin Vass proposed zuul/zuul master: Add messages to make the job setup more transparent https://review.opendev.org/c/zuul/zuul/+/777885 | 07:52 |
|---|---|---|
| *** rpittau|afk is now known as rpittau | 08:04 | |
| *** cloudnull has quit IRC | 08:10 | |
| *** cloudnull has joined #zuul | 08:14 | |
| *** jcapitao has joined #zuul | 08:16 | |
| *** tosky has joined #zuul | 08:20 | |
| *** jangutter_ has joined #zuul | 08:43 | |
| *** jangutter has quit IRC | 08:47 | |
| *** zbr1 has joined #zuul | 08:58 | |
| *** zbr has quit IRC | 09:01 | |
| *** zbr1 is now known as zbr | 09:01 | |
| *** zbr8 has joined #zuul | 09:06 | |
| *** ykarel is now known as ykarel|lunch | 09:06 | |
| *** zbr has quit IRC | 09:08 | |
| *** zbr8 is now known as zbr | 09:08 | |
| *** jfoufas1 has joined #zuul | 09:08 | |
| *** holser has joined #zuul | 09:16 | |
| *** holser has quit IRC | 09:17 | |
| *** holser has joined #zuul | 09:17 | |
| *** ttx has quit IRC | 09:20 | |
| *** ttx has joined #zuul | 09:21 | |
| *** hashar has joined #zuul | 09:21 | |
| *** holser has quit IRC | 09:24 | |
| *** zbr1 has joined #zuul | 09:27 | |
| *** zbr1 has quit IRC | 09:27 | |
| *** zbr has quit IRC | 09:29 | |
| *** holser has joined #zuul | 09:31 | |
| openstackgerrit | Simon Westphahl proposed zuul/zuul master: Improve typings in context of builds via ZooKeeper https://review.opendev.org/c/zuul/zuul/+/753578 | 09:35 |
| openstackgerrit | Simon Westphahl proposed zuul/zuul master: Make ZooKeeper mandatory for Scheduler https://review.opendev.org/c/zuul/zuul/+/756716 | 09:35 |
| openstackgerrit | Simon Westphahl proposed zuul/zuul master: Move setupZK() helper function to BaseTestCase class https://review.opendev.org/c/zuul/zuul/+/776639 | 09:35 |
| openstackgerrit | Simon Westphahl proposed zuul/zuul master: Initialize ZooKeeper connection in server rather than in cmd classes https://review.opendev.org/c/zuul/zuul/+/776640 | 09:35 |
| openstackgerrit | Simon Westphahl proposed zuul/zuul master: Make ConnectionRegistry mandatory for Scheduler https://review.opendev.org/c/zuul/zuul/+/757095 | 09:35 |
| openstackgerrit | Simon Westphahl proposed zuul/zuul master: Improve typings in context of 756716 and 757095 https://review.opendev.org/c/zuul/zuul/+/757148 | 09:35 |
| openstackgerrit | Simon Westphahl proposed zuul/zuul master: Instantiate executor client, merger, nodepool and app within Scheduler https://review.opendev.org/c/zuul/zuul/+/757149 | 09:35 |
| openstackgerrit | Simon Westphahl proposed zuul/zuul master: Improve typings in context of lock nodes on executor https://review.opendev.org/c/zuul/zuul/+/757097 | 09:35 |
| openstackgerrit | Simon Westphahl proposed zuul/zuul master: DNM: Reduce number of jobs for SOS development https://review.opendev.org/c/zuul/zuul/+/775081 | 09:35 |
| *** jpenag is now known as jpena | 09:42 | |
| *** zbr has joined #zuul | 09:43 | |
| *** ykarel|lunch is now known as ykarel | 09:54 | |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: Update tests/base.py to use proper git data https://review.opendev.org/c/zuul/zuul/+/742746 | 10:00 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: Fix gerrit merge commit change with zuul configuration https://review.opendev.org/c/zuul/zuul/+/762886 | 10:00 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: Fix zuul-client enqueue-ref when oldrev/newrev aren't provided https://review.opendev.org/c/zuul/zuul/+/765767 | 10:00 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: Improve merger getFilesChanges, Fix edge cases https://review.opendev.org/c/zuul/zuul/+/762887 | 10:00 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: Tenant reconfiguration: Add tests https://review.opendev.org/c/zuul/zuul/+/742747 | 10:00 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: Tenant reconfiguration: Reorg scheduler event process loop https://review.opendev.org/c/zuul/zuul/+/742748 | 10:00 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: Tenant reconfiguration: Scheduler ref-updated create/delete https://review.opendev.org/c/zuul/zuul/+/739198 | 10:00 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: Tenant reconfiguration: Allow ref-updated newrev+oldrev reconfiguration https://review.opendev.org/c/zuul/zuul/+/742749 | 10:00 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: Tenant reconfiguration: Get ref-updated modified files https://review.opendev.org/c/zuul/zuul/+/739078 | 10:00 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: tutorial: Add "Use zuul jobs" https://review.opendev.org/c/zuul/zuul/+/732068 | 10:12 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: tutorial: Add "gate pipeline" https://review.opendev.org/c/zuul/zuul/+/732069 | 10:12 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: tutorial: Add "job secrets" https://review.opendev.org/c/zuul/zuul/+/732070 | 10:12 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: tutorial: Add "job dependencies" https://review.opendev.org/c/zuul/zuul/+/732071 | 10:12 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: Rename quick-start to zuul-tutorial-quick-start https://review.opendev.org/c/zuul/zuul/+/737656 | 10:12 |
| *** nils has joined #zuul | 10:26 | |
| *** harrymichal has joined #zuul | 10:40 | |
| *** jangutter has joined #zuul | 10:41 | |
| *** jangutter has quit IRC | 10:42 | |
| *** jangutter has joined #zuul | 10:43 | |
| *** jangutter_ has quit IRC | 10:44 | |
| *** jfoufas1 has quit IRC | 10:50 | |
| *** fbo|off is now known as fbo | 10:52 | |
| *** jangutter_ has joined #zuul | 11:02 | |
| *** jangutter has quit IRC | 11:05 | |
| *** jcapitao is now known as jcapitao_lunch | 11:54 | |
| *** yoctozepto has quit IRC | 12:17 | |
| *** yoctozepto has joined #zuul | 12:17 | |
| *** saneax has joined #zuul | 12:20 | |
| *** rlandy has joined #zuul | 12:29 | |
| *** jpena is now known as jpena|lunch | 12:32 | |
| *** zbr has quit IRC | 12:34 | |
| *** zbr has joined #zuul | 12:36 | |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: Spec: external permissions for the REST admin API https://review.opendev.org/c/zuul/zuul/+/777629 | 12:42 |
| *** harrymichal has quit IRC | 12:49 | |
| *** ikhan has joined #zuul | 12:50 | |
| *** jcapitao_lunch is now known as jcapitao | 13:13 | |
| *** jfoufas1 has joined #zuul | 13:16 | |
| *** ykarel has quit IRC | 13:19 | |
| *** ykarel has joined #zuul | 13:20 | |
| *** jpena|lunch is now known as jpena | 13:24 | |
| *** sduthil has joined #zuul | 13:31 | |
| *** jangutter has joined #zuul | 13:33 | |
| *** jangutter_ has quit IRC | 13:37 | |
| *** jangutter_ has joined #zuul | 13:44 | |
| *** jangutter has quit IRC | 13:47 | |
| *** ikhan has quit IRC | 13:58 | |
| *** ikhan has joined #zuul | 13:59 | |
| *** jangutter has joined #zuul | 14:16 | |
| *** jangutter_ has quit IRC | 14:20 | |
| *** jangutte_ has joined #zuul | 14:20 | |
| *** jangutter has quit IRC | 14:23 | |
| jpena | mnaser: we have identified an issue with our networking configuration. Currently, the public network has two subnets (38.102.../24 and 38.129.../24). However, I see all routers from our tenant private networks end up with an IP from the first subnet | 14:37 |
| jpena | that's effectively preventing us from getting IPs from the second subnet, and I see no way to force the pool you get the subnet from when you create a router | 14:38 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: Tenant reconfiguration: Allow ref-updated newrev+oldrev reconfiguration https://review.opendev.org/c/zuul/zuul/+/742749 | 14:38 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: Tenant reconfiguration: Get ref-updated modified files https://review.opendev.org/c/zuul/zuul/+/739078 | 14:38 |
| jpena | sorry, wrong channel | 14:38 |
| openstackgerrit | James E. Blair proposed zuul/zuul master: Update zoned executor stats https://review.opendev.org/c/zuul/zuul/+/777852 | 15:17 |
| openstackgerrit | James E. Blair proposed zuul/zuul master: Optionally allow zoned executors to process unzoned jobs https://review.opendev.org/c/zuul/zuul/+/673840 | 15:23 |
| *** SaifAddin has joined #zuul | 15:31 | |
| *** harrymichal has joined #zuul | 15:31 | |
| SaifAddin | Hi, I have a pre-run playbook on a job, that is supposed to load variables to make them available for the actual run. | 15:38 |
| SaifAddin | The pre-run has a task that loads all variables inside a `set-facts:` - `cacheable: true` combination | 15:39 |
| SaifAddin | but that alone doesn't work, it is only working (i.e. finding variables) when I also put the variable values in the `vars` folder yaml | 15:39 |
| SaifAddin | But it is super ugly to have the variables repeated in two places | 15:40 |
| corvus | zuul-maint: 15:40 < openstackgerrit> James E. Blair proposed openstack/project-config master: Allow all registered users to edit hashtags on all Zuul projects https://review.opendev.org/c/openstack/project-config/+/778012 | 15:42 |
| corvus | SaifAddin: i don't know what the problem could be; all i can do at this point is reassure you that i would expect a cacheable fact to work and we use that frequently in zuul-jobs | 15:43 |
| SaifAddin | Thank you corvus, that is reasuring | 15:44 |
| SaifAddin | I am figuring out why the variables I put in the cacheable facts are not found until I put them also in the vars | 15:44 |
| tobiash | corvus: is it on purpose that zuul-client isn't listed there? ^ | 15:45 |
| corvus | tobiash: no... maybe my repo is out of date | 15:45 |
| corvus | tobiash: or maybe zuul-client just uses the zuul config file | 15:45 |
| corvus | tobiash: it's the latter -- it uses zuul.config | 15:46 |
| tobiash | k | 15:46 |
| *** sassyn has quit IRC | 15:48 | |
| tobiash | clarkb: could you put https://review.opendev.org/c/zuul/zuul/+/752056 (github request optimization) to your re-review list? I think the last ps was a rebase. | 15:50 |
| *** jfoufas1 has quit IRC | 16:00 | |
| openstackgerrit | Andy Ladjadj proposed zuul/zuul master: [web][config] move timezone component to preferences https://review.opendev.org/c/zuul/zuul/+/755929 | 16:11 |
| corvus | tristanC: you previously reviewed circular deps in https://review.opendev.org/685354 -- do you want to take another look at that and my followup in https://review.opendev.org/777843 ? | 16:13 |
| openstackgerrit | Andy Ladjadj proposed zuul/zuul master: [web][config] move timezone component to preferences https://review.opendev.org/c/zuul/zuul/+/755929 | 16:15 |
| openstackgerrit | Andy Ladjadj proposed zuul/zuul master: [web][config] move timezone component to preferences https://review.opendev.org/c/zuul/zuul/+/755929 | 16:17 |
| *** ykarel is now known as ykarel|away | 16:28 | |
| *** SaifAddin has quit IRC | 16:31 | |
| *** ykarel|away has quit IRC | 16:33 | |
| corvus | ianw: you previously reviewed this github optimization: https://review.opendev.org/753571 -- would you care to re-review? | 16:41 |
| *** ikhan has quit IRC | 16:42 | |
| corvus | swest_, tobiash: can you take a look at https://review.opendev.org/761756 ? i think primarily we want to make sure everyone understands the limitations with removing tags | 16:43 |
| tobiash | corvus: I wonder if we should just delete all tags before fetching, but that might impose significant costs on every fetch | 16:46 |
| corvus | tobiash: yeah that would be huge | 16:48 |
| corvus | tobiash: to be clear: there is no way to gracefully handle tag deletion with git. zuul is just encountering the same problems that any user would enconter. for that reason, nobody should ever delete tags. | 16:50 |
| corvus | now if all of the users are in the same room and you can tell them all to delete their tags, that will work :) | 16:50 |
| corvus | but that's sort of the exceptional case for a distributed version control system | 16:50 |
| tobiash | corvus: yeah, some of our users use this to test their release pipelines using test tags :/ | 16:50 |
| fungi | tobiash: i've found pushing prereleases and release candidate tags to be a useful solution, at least on projects i help maintain | 16:51 |
| corvus | tobiash: so it sounds like if those people never re-use the same tag, that might mostly work with this change. (though users are going to have to add --prune-tags to avoid ending up with a big pile of unused tags) | 16:52 |
| corvus | tobiash: if they re-use the same tag, then even this change isn't going to always work. | 16:52 |
| tobiash | they don't re-use tags and don't care if they are persistent | 16:53 |
| tobiash | but we all agree that's a bad practice | 16:53 |
| corvus | tobiash: okay, sounds like we're all on the same page. the main outcome i think we want to avoid is going down some rabbit hole of trying to make deleting tags "work for real" which we won't be able to achieve. | 16:56 |
| fungi | i'm okay with the idea as long as we wrap any mention of it with a big noticeable disclaimer that it's of limited utility and you're better off not deleting tags because git wasn't really designed to support that properly | 16:59 |
| *** rpittau is now known as rpittau|afk | 17:00 | |
| corvus | fungi: the change as written doesn't actually mention anything in docs or release notes, it just silently changes an internal behavior. would you be okay with that? :) | 17:02 |
| fungi | yeah, i mean. we're not updating the docs to mention anything people should expect to suddenly work, so seems fine | 17:04 |
| avass | tobiash: yes I've had to tell people to not delete tags way too many times | 17:19 |
| avass | though it would be easier if projects could re-enqueue release pipelines without admin access or creating a new tag somehow | 17:19 |
| avass | in case of intermittent errors | 17:20 |
| *** jcapitao has quit IRC | 17:22 | |
| fungi | in our (opendev's) case, release builds have enough nuanced side effects that reenqueuing may not work for some sorts of transient failures, and may even be destructive | 17:23 |
| fungi | we'd probably need to caution our users to design release jobs with complete idempotence in mind | 17:23 |
| fungi | which is not always easy | 17:24 |
| *** SaifAddin has joined #zuul | 17:25 | |
| SaifAddin | corvus is it possible than an empty main.yaml in vars to override all the variables set in tasks/main.yaml ?? | 17:25 |
| corvus | SaifAddin: i don't think so; i think it would only add variables for use in that role. | 17:29 |
| corvus | (but any vars explicitly set there might override cached facts? i'm unsure about that) | 17:29 |
| corvus | SaifAddin: if you want, you could upload a test change to the opendev/sandbox repository to try out the process. you can propose a new job with multiple playbooks and zuul will run it. we could all look at it together then. | 17:31 |
| *** rlandy is now known as rlandy|lunch | 17:36 | |
| SaifAddin | ahh that is cool, thanks. Will consider it | 17:40 |
| SaifAddin | but yes it is so strange, the job runs well only when I put the list of variables both inside the `set_fact:` AND when I put all the same variables (again) in the `vars/main.yml` | 17:41 |
| SaifAddin | just one or the other is not enough, driving me crazy | 17:41 |
| SaifAddin | ``` | 17:42 |
| SaifAddin | --- | 17:42 |
| SaifAddin | - name: setting_cache_variables | 17:42 |
| SaifAddin | set_fact: | 17:42 |
| SaifAddin | foo1: bar1 | 17:42 |
| SaifAddin | foo2: bar2 | 17:42 |
| SaifAddin | ``` | 17:42 |
| SaifAddin | (+ cacheable: true) | 17:43 |
| avass | fungi: for us it's often that either the artifacts got uploaded or they didn't | 17:43 |
| SaifAddin | And then the same in `vars/main.yaml` | 17:43 |
| SaifAddin | `foo1: bar1 | 17:43 |
| SaifAddin | foor2: bar2 | 17:43 |
| SaifAddin | ` | 17:43 |
| avass | SaifAddin: can you use a paste service like: http://paste.openstack.org/ for pasting code? :) | 17:44 |
| SaifAddin | Yes I am sorry. I thought triple beans would work for small snippets | 17:46 |
| *** ikhan has joined #zuul | 17:50 | |
| openstackgerrit | Merged zuul/zuul master: Remove an unneeded api call when creating check_runs https://review.opendev.org/c/zuul/zuul/+/752056 | 17:55 |
| openstackgerrit | Merged zuul/zuul master: Save superfluous api requests in check run reporting https://review.opendev.org/c/zuul/zuul/+/752079 | 17:55 |
| *** rlandy|lunch is now known as rlandy | 18:07 | |
| *** jpena is now known as jpena|off | 18:10 | |
| *** hashar has quit IRC | 18:10 | |
| *** SaifAddin has quit IRC | 18:10 | |
| corvus | clarkb, fungi, tobiash: fun fact: it looks like 'git fetch --tags' does cause local tags to update if remote tags are moved | 18:31 |
| clarkb | corvus: cool, so if we --prune-tags and --tags we might address a good portion of this? | 18:33 |
| corvus | clarkb: yeah, that's how it's looking to me. +1 for "write some tests to codify the behavior" :) | 18:33 |
| openstackgerrit | James E. Blair proposed zuul/zuul master: Prune git tags on fetch https://review.opendev.org/c/zuul/zuul/+/761756 | 18:34 |
| corvus | clarkb, fungi, tobiash, swest_: ^ i updated that with 2 tests: tag/delete/retag and moving a tag | 18:34 |
| fungi | corvus: however it won't remove a local tag when the remote is deleted, right? | 18:39 |
| fungi | that just winds up orphaned | 18:39 |
| openstackgerrit | Tobias Henkel proposed zuul/zuul master: Add optional support for circular dependencies https://review.opendev.org/c/zuul/zuul/+/685354 | 18:43 |
| openstackgerrit | Tobias Henkel proposed zuul/zuul master: Allow refreshing volatile data in canMerge check https://review.opendev.org/c/zuul/zuul/+/767084 | 18:45 |
| openstackgerrit | Tobias Henkel proposed zuul/zuul master: Check cycle items are mergeable before reporting https://review.opendev.org/c/zuul/zuul/+/743450 | 18:45 |
| corvus | fungi: it will because the change adds --prune-tags | 18:45 |
| fungi | ahh | 18:45 |
| fungi | got it | 18:45 |
| openstackgerrit | Tobias Henkel proposed zuul/zuul master: Support enqueuing behind circular dependencies https://review.opendev.org/c/zuul/zuul/+/777843 | 18:45 |
| tobiash | corvus: cdep needed a rebase ^ | 18:46 |
| fungi | so the combination of the two is enough to sync up local tags to remote tags even if they've been deleted and/or recreated | 18:46 |
| corvus | so basically, aiui, zuul will probably do the thing everyone imagines happens: it will magically keep up with whatever happens to tags upstream. | 18:46 |
| corvus | fungi: that's my understanding | 18:46 |
| fungi | neat | 18:46 |
| corvus | tobiash: thanks! | 18:46 |
| openstackgerrit | Tobias Henkel proposed zuul/zuul master: Make reporting asynchronous https://review.opendev.org/c/zuul/zuul/+/691253 | 18:46 |
| harrymichal | Hi folks, recently I started to use the dynamic badge and noticed the image does not get updated in the README preview on GitHub. I spent a bit of time digging and found this part of GItHub's documentation: https://docs.github.com/en/github/authenticating-to-github/about-anonymized-image-urls#an-image-that-changed-recently-is-not-updating. | 19:13 |
| harrymichal | Basically, the response with the image needs to contain value "Cache-Control" set to "no-cache". | 19:13 |
| harrymichal | Could that be set for the dynamic badge? | 19:14 |
| openstackgerrit | Tristan Cacqueray proposed zuul/zuul master: web: set cache-control header for the badge endpoint https://review.opendev.org/c/zuul/zuul/+/778054 | 19:23 |
| tristanC | harrymichal: perhaps ^ would do | 19:24 |
| *** hamalq has joined #zuul | 19:24 | |
| harrymichal | tristanC: Thank you! | 19:29 |
| *** piotrowskim has quit IRC | 20:12 | |
| openstackgerrit | Tristan Cacqueray proposed zuul/zuul master: Add project variables to the REST API https://review.opendev.org/c/zuul/zuul/+/778062 | 20:32 |
| *** mgagne has joined #zuul | 20:42 | |
| *** nils has quit IRC | 20:50 | |
| openstackgerrit | Merged zuul/zuul master: web: set cache-control header for the badge endpoint https://review.opendev.org/c/zuul/zuul/+/778054 | 21:13 |
| openstackgerrit | Kevin Carter proposed zuul/zuul-jobs master: Make .sh browsable on swift logs https://review.opendev.org/c/zuul/zuul-jobs/+/731795 | 22:22 |
| openstackgerrit | James E. Blair proposed zuul/zuul master: Update zoned executor stats https://review.opendev.org/c/zuul/zuul/+/777852 | 23:04 |
| openstackgerrit | James E. Blair proposed zuul/zuul master: Optionally allow zoned executors to process unzoned jobs https://review.opendev.org/c/zuul/zuul/+/673840 | 23:04 |
| fungi | probably been discussed before, but should we be limiting what we load from openstack/project-config and opendev/system-config in our zuul tenant definition? https://zuul.opendev.org/t/zuul/config-errors | 23:07 |
| corvus | fungi: probably so | 23:08 |
| corvus | we can at least pull out the pipeline defs | 23:08 |
| corvus | i'm assuming they're there for jobs and we may not get 100% clean on that | 23:09 |
| fungi | yeah, maybe. and i agree the openstack/project-config one should be simple at least. i'll start there | 23:10 |
| openstackgerrit | Merged zuul/zuul master: Don't query branch protection on pull request events https://review.opendev.org/c/zuul/zuul/+/753571 | 23:10 |
| corvus | clarkb: https://review.opendev.org/761756 look good now with the new tests? | 23:23 |
| clarkb | corvus: I'll look in a minute | 23:24 |
| corvus | no rush, just trying to clear the deck :) | 23:26 |
| fungi | corvus: okay, this is mildly confounding... https://opendev.org/openstack/project-config/src/branch/master/zuul/main.yaml#L1630 | 23:35 |
| fungi | it already shouldn't be trying to load pipeline definitions, right? | 23:35 |
| *** andy-ladjadj has joined #zuul | 23:36 | |
| fungi | okay, even more confusing. it's not actually alarming about it now | 23:36 |
| *** andy-ladjadj has quit IRC | 23:36 | |
| fungi | you saw it in there too though, right? i'm not imagining things? | 23:36 |
| clarkb | corvus: both tests in that change seem to cover the tag has moved scenario. Am I reading that correctly? | 23:37 |
| clarkb | corvus: I went ahead and approved it since the testing is only doing more than it claims and that should be fine :) | 23:40 |
| openstackgerrit | James E. Blair proposed zuul/zuul master: Make repo state buildset global https://review.opendev.org/c/zuul/zuul/+/738603 | 23:49 |
| corvus | clarkb: yes, though the first one is with zuul updating between the delete/add and the second is without | 23:50 |
| corvus | clarkb: i looked at it as the first one is "someone deleted the tag. then (much) later, someone readded the tag with the same name". versus "someone moved the tag" | 23:50 |
| corvus | fungi: yes, i still have the errors on my screen | 23:51 |
| corvus | fungi: i agree the pipelines are gone if i open in a new window | 23:51 |
| fungi | that's... unnerving | 23:54 |
| clarkb | corvus: gotcha | 23:55 |
| fungi | corvus: also /etc/zuul/main.yaml was last modified on the scheduler's filesystem ~28 hours ago (if my math is correct) | 23:57 |
| *** harrymichal has quit IRC | 23:58 | |
| *** harrymichal has joined #zuul | 23:58 | |
| fungi | and last full reconfigure a few minutes after that file was touched | 23:59 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!