| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 817869: Add an init phase to scheduler/web startup https://review.opendev.org/c/zuul/zuul/+/817869 | 00:35 | |
| @jim:acmegating.com | Clark, swest: ^ there wo go, i did get to it today :) | 00:36 |
|---|---|---|
| @clarkb:matrix.org | corvus: noted a small but important bug in there I think | 00:40 |
| @clarkb:matrix.org | (will avoid confusion) | 00:40 |
| @jim:acmegating.com | hah yes | 00:40 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 817869: Add an init phase to scheduler/web startup https://review.opendev.org/c/zuul/zuul/+/817869 | 00:41 | |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul] 815558: Replace asserts with exceptions https://review.opendev.org/c/zuul/zuul/+/815558 | 01:40 | |
| -@gerrit:opendev.org- Felix Edel proposed on behalf of Simon Westphahl: | 08:35 | |
| - [zuul/zuul] 815278: DNM: execute tests with two schedulers https://review.opendev.org/c/zuul/zuul/+/815278 | ||
| - [zuul/zuul] 815787: Refresh pipelines in tests when settled https://review.opendev.org/c/zuul/zuul/+/815787 | ||
| -@gerrit:opendev.org- Felix Edel proposed: [zuul/zuul] 818054: Make TestWeb.test_web_components work with multiple schedulers https://review.opendev.org/c/zuul/zuul/+/818054 | 08:35 | |
| @harrymichal:matrix.org | Hi folks! Is there a way to check my project .zuul.yaml file apart from pushing it to the repo? | 08:40 |
| -@gerrit:opendev.org- Matthieu Huin https://matrix.to/#/@mhuin:matrix.org proposed: [zuul/zuul] 817949: [WIP] SQL reporter: add substring search on some fields https://review.opendev.org/c/zuul/zuul/+/817949 | 09:54 | |
| -@gerrit:opendev.org- Matthieu Huin https://matrix.to/#/@mhuin:matrix.org proposed: [zuul/zuul] 817949: [WIP] SQL reporter: add substring search on some fields https://review.opendev.org/c/zuul/zuul/+/817949 | 11:34 | |
| -@gerrit:opendev.org- Matthieu Huin https://matrix.to/#/@mhuin:matrix.org proposed: [zuul/zuul] 817949: SQL reporter: add substring search on some fields https://review.opendev.org/c/zuul/zuul/+/817949 | 12:44 | |
| -@gerrit:opendev.org- Matthieu Huin https://matrix.to/#/@mhuin:matrix.org proposed: [zuul/zuul] 817949: zuul-web: add substring search on some fields https://review.opendev.org/c/zuul/zuul/+/817949 | 14:36 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 818101: Fix race in layoutstate https://review.opendev.org/c/zuul/zuul/+/818101 | 15:17 | |
| @jim:acmegating.com | i noticed that when doing some local startup sequence testing | 15:18 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 818101: Fix race in layoutstate https://review.opendev.org/c/zuul/zuul/+/818101 | 15:27 | |
| -@gerrit:opendev.org- Matthieu Huin https://matrix.to/#/@mhuin:matrix.org proposed: [zuul/zuul] 818106: example docker-compose: start web service after executor https://review.opendev.org/c/zuul/zuul/+/818106 | 15:32 | |
| @clarkb:matrix.org | corvus: so I don't forget: I was wondering if my config validation changes could somehow impact Zuul's ability to start with broken configs? In this case I think maybe we already fail if the tenant config itself is invalid? so no possible regression there? Also, is it time to start thinking about a functional test job with multiple schedulers to test things like rolling restarts to try and avoid introducing zk db data format changes that force us to start with a clean db? Or do we think we can emulate that well enough with the multiple scheduler support in the unittest framework? | 15:34 |
| @jim:acmegating.com | Clark: i'm not sure we want zuul to start with a broken main.yaml file, but i don't know how permissive we were before. | 15:39 |
| -@gerrit:opendev.org- Artem Goncharov proposed: [zuul/zuul-jobs] 818109: Add no_log to the log uploading module parameters https://review.opendev.org/c/zuul/zuul-jobs/+/818109 | 15:39 | |
| @jim:acmegating.com | Clark: the unit test framework is sufficient for testing multi-scheduler behavior; anything we want to test with 2 schedulers we can. | 15:40 |
| @jim:acmegating.com | upgrade testing would require a functional job though since it's multiple versions of the code. | 15:41 |
| @jim:acmegating.com | tbh, i'm on the fence about the risk/reward payoff there | 15:42 |
| @clarkb:matrix.org | I think I'm mostly concerned about cases where we make updates to the zk data schema/format and the old version isn't compatible with it or vice versa. | 15:43 |
| @clarkb:matrix.org | Though once the initial migration into zk is completed the opportunity for those problems drops off quickly | 15:44 |
| @jim:acmegating.com | Clark: agree on both points.... i wonder... | 15:54 |
| @jim:acmegating.com | could we use unit tests for this in a slightly unconventional manner...? split it into 2 unit tests, run part one on the old code, persist the zk data, then run part two on the new code | 15:55 |
| @jim:acmegating.com | that might be a lot simpler than having a whole functional system (with real gerrit, etc). and we get all the advantages of the unit test framework. | 15:56 |
| @clarkb:matrix.org | hrm ya a frozen zk state we can load up for a test should cover it | 15:56 |
| @clarkb:matrix.org | and then we can periodically update that data when we think we've crossed a safe threshold | 15:56 |
| @jim:acmegating.com | oh yeah that's even easier than what i was thinking. ;) | 15:57 |
| @jim:acmegating.com | (i was thinking of running phase1 every time, but you're right, we only need to run it once, and can freeze it) | 15:58 |
| @jim:acmegating.com | (then run it again if we want to re-make the data) | 15:58 |
| @fungicide:matrix.org | martymichal: doesn't look like anybody answered you earlier, so if you're still looking for suggestions, zuul is capable of testing job configuration when it's only proposed in a change/merge request/pull request as long as it's an untrusted repository and the configuration uses a pre-merge pipeline (conventionally something like a "check" pipeline) | 16:13 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 818116: Fix zuul-web startup config priming https://review.opendev.org/c/zuul/zuul/+/818116 | 16:13 | |
| @harrymichal:matrix.org | > martymichal: doesn't look like anybody answered you earlier, so if you're still looking for suggestions, zuul is capable of testing job configuration when it's only proposed in a change/merge request/pull request as long as it's an untrusted repository and the configuration uses a pre-merge pipeline (conventionally something like a "check" pipeline) | 16:16 |
| Yes, I'm aware of that. The point is that I'd like to run the check locally instead of constantly pushing to the branch. | ||
| @fungicide:matrix.org | martymichal: in that case, this proposed spec ioutlines the closest we can probably come to accommodating it: https://zuul-ci.org/docs/zuul/reference/developer/specs/zuul-runner.html | 16:19 |
| @jim:acmegating.com | i don't believe that includes job configuration changes | 16:20 |
| @fungicide:matrix.org | oh, good point | 16:20 |
| @fungicide:matrix.org | it's for recreating already merged job configuration | 16:20 |
| -@gerrit:opendev.org- Artem Goncharov proposed: [zuul/zuul-jobs] 807132: Fix zuul-jobs-test-upload-logs-s3 job https://review.opendev.org/c/zuul/zuul-jobs/+/807132 | 16:20 | |
| @jim:acmegating.com | a zuul job is part of a huge network of configuration objects that span multiple repos; to validate it, you need access to the whole thing. | 16:21 |
| @fungicide:matrix.org | in a way, it's sort of like asking how to test a train without putting it on a track... you can build your own track (i.e. run a local zuul deployment), but really the complexity involved in testing job configuration would require you to reproduce zuul itself | 16:22 |
| @jim:acmegating.com | it takes opendev about 15 minutes to bootstrap its configuration from scratch | 16:22 |
| @harrymichal:matrix.org | The problem is that any major changes to the .zuul.yaml file can very likely contain errors. And Zuul is quite bad at pointing out what's wrong with it. The only error I get is "unhashable type: 'list'" which sounds to me like some fundamental problem with the config and I'd be much more productive if I could check the config locally. | 16:28 |
| -@gerrit:opendev.org- Artem Goncharov proposed: [zuul/zuul-jobs] 818109: Add no_log to the log uploading module parameters https://review.opendev.org/c/zuul/zuul-jobs/+/818109 | 16:29 | |
| @clarkb:matrix.org | For errors like that we might be able to run just the vs schema checker locally | 16:31 |
| @harrymichal:matrix.org | But I guess there's no better approach than pushing to the branch. | 16:31 |
| @clarkb:matrix.org | Though there isn't a tool for that yet, and it will only catch schema problems | 16:31 |
| @harrymichal:matrix.org | > Though there isn't a tool for that yet, and it will only catch schema problems | 16:33 |
| Right. Anyway, I presume that particular error comes from Python, so does Python provide any other information about the error (e.g., where it encountered it)? That would help tremendously instead of having to search for a needle in a haystack of files. | ||
| @fungicide:matrix.org | also checking that the file is valid yaml can be done trivially, i guess | 16:33 |
| @fungicide:matrix.org | you might try pointing something like https://pypi.org/project/yamllint at it | 16:33 |
| @fungicide:matrix.org | or even just trying to use pyyaml's load() function on it | 16:34 |
| @clarkb:matrix.org | Ya that error is either at the yaml layer or in the vs schema error I think. The vs checker should tell you where it broke though so probably ppyaml? | 16:34 |
| -@gerrit:opendev.org- Zuul merged on behalf of Artem Goncharov: [zuul/zuul-jobs] 807132: Fix zuul-jobs-test-upload-logs-s3 job https://review.opendev.org/c/zuul/zuul-jobs/+/807132 | 16:34 | |
| @clarkb:matrix.org | in the vs case it will give you the yaml "path" not a line number but that should be sufficient | 16:35 |
| @harrymichal:matrix.org | Will try that. Thanks! | 16:35 |
| @harrymichal:matrix.org | > in the vs case it will give you the yaml "path" not a line number but that should be sufficient | 16:35 |
| That is already a good pointer. | ||
| @clarkb:matrix.org | is there a way to deeplink to the zuul config errors "page" | 16:36 |
| @clarkb:matrix.org | I want to ask openstack to kindly work on fixing those and being able to link directly would be nice | 16:36 |
| @fungicide:matrix.org | Clark: there is, though not to specifci errors i don't think, just a sec i'll remind myself | 16:37 |
| @fungicide:matrix.org | Clark: https://zuul.opendev.org/t/openstack/config-errors is what i've sent people to before | 16:37 |
| @clarkb:matrix.org | thanks, Looks like you get that if you click on any one error "block" | 16:38 |
| @fungicide:matrix.org | oh, yep! i think that must be new-ish | 16:38 |
| @fungicide:matrix.org | er, no, nevermind, that's how i was getting to it before. thought for a moment you were saying it would link to an individual error that way | 16:39 |
| -@gerrit:opendev.org- Zuul merged on behalf of Artem Goncharov: [zuul/zuul-jobs] 818109: Add no_log to the log uploading module parameters https://review.opendev.org/c/zuul/zuul-jobs/+/818109 | 16:45 | |
| @jim:acmegating.com | unhashable type 'list' doesn't sound like a config error, it sounds like a bug. especially if zuul didn't give you a line number. | 16:52 |
| @jim:acmegating.com | martymichal: zuul should give a line number, character number, snippet, and expected schema for any config errors. if it's unable to do that, then there's probably something more fundamental wrong. | 16:54 |
| @spamaps:spamaps.ems.host | Ugh. The kubernetes driver is proving very difficult to even modify to use a shared namespace. I wonder if anyone is using it and can offer any encouragement that it's worth the trouble. I think I might just run vms as I don't really want to manage my own Kubernetes cluster for zuul jobs. | 16:57 |
| @harrymichal:matrix.org | corvus To give you a reference: https://github.com/containers/toolbox/pull/918#issuecomment-966763201 | 16:58 |
| @clarkb:matrix.org | excluded_locations jumps out to me as a problem. | 17:01 |
| @clarkb:matrix.org | the zuul config doesn't so variable substitution like that. You can use yaml anchors instead though | 17:01 |
| @clarkb:matrix.org | so ya I suspect this is a yaml load issue? | 17:05 |
| @tristanc_:matrix.org | spamaps: have you looked at the openshiftpods driver?, i think it does what you are looking for | 17:06 |
| @clarkb:matrix.org | yaml tells me `found unhashable key` if I try to load something like: | 17:07 |
| ``` | ||
| foo = ['1', '2'] | ||
| bar = {{ foo }} | ||
| ``` | ||
| @clarkb:matrix.org | * yaml tells me `found unhashable key` if I try to load something like: | 17:07 |
| ``` | ||
| foo: ['1', '2'] | ||
| bar: {{ foo }} | ||
| ``` | ||
| -@gerrit:opendev.org- Zuul merged on behalf of Clark Boylan: [zuul/zuul] 818019: Add job name to BuildRequest repr https://review.opendev.org/c/zuul/zuul/+/818019 | 17:09 | |
| @fungicide:matrix.org | martymichal: probably what you want is `irrelevant-files: &excluded_locations ['.github/*', 'completion/*', 'doc/*', 'images/*', 'CODE-OF-CONDUCT.md', 'CONTRIBUTING.md', 'COPYING', 'GOALS.md', 'NEWS', 'README.md', 'SECURITY.md', 'toolbox']` in the first job, and then simply `irrelevant-files: *excluded_locations` in the rest of them | 17:17 |
| @jim:acmegating.com | martymichal: yeah, this is what zuul should have emitted based on that "excluded_locations" line: https://paste.opendev.org/show/811036/ | 17:19 |
| @jim:acmegating.com | i'm not sure why you didn't get that error | 17:19 |
| @fungicide:matrix.org | tristanC: ^ any idea if there's something about the way softwarefactory is configured that it didn't comment with the usual syntax error? | 17:21 |
| @jim:acmegating.com | oh, i am able to reproduce that locally | 17:25 |
| @jim:acmegating.com | okay, yeah it's the "irrelevant-files: {{ excluded_locations }}" that gets us the unhashable type | 17:26 |
| @jim:acmegating.com | remove that, and you get the error that i pasted before | 17:27 |
| @jim:acmegating.com | martymichal: so tl;dr: you should be able to fix it with fungi's suggestion, and i'll see if i can figure out why zuul got tripped up by the {{}} | 17:27 |
| @fungicide:matrix.org | i guess because it's invalid yaml, though that doesn't seem to match the error pyyaml gives me when attempting to load it | 17:28 |
| @fungicide:matrix.org | maybe it comes down to different versions of pyyaml raising slightly different exceptions | 17:29 |
| @tristanc_:matrix.org | fungi: i don't know why this zuul doesn't report the error properly, it's running version 4.6.0 | 17:33 |
| @jim:acmegating.com | fungi: yeah, it's in the yaml parser. | 17:33 |
| @fungicide:matrix.org | tristanC: sorry, turns out it's reproducible in master too | 17:34 |
| @jim:acmegating.com | tristanC: it does, that was my mistake; there are 2 errors and i stopped after the first. | 17:34 |
| @fungicide:matrix.org | so not specific to softwarefactory | 17:34 |
| @michiel:piscaer.com | When I use with Gitlab a merge and us the pipelines to test my code i see that Gitlab first does a merge and then starts the pipelines. But how does Zuul this together with Gitlab? | 17:37 |
| @clarkb:matrix.org | masterpe: zuul should always run jobs against the potential future state. This means for independent zuul pipelines you'll be merged against the target branch and for dependent pipelines you'll get the speculative state of any inflight changes as well as yourself merged against the target branch | 17:40 |
| @jim:acmegating.com | masterpe: zuul will not use the merge that gitlab does, it will perform its own merge | 17:44 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 818129: Report more YAML syntax errors https://review.opendev.org/c/zuul/zuul/+/818129 | 17:54 | |
| @jim:acmegating.com | martymichal, fungi, Clark, tristanC: ^ that should produce the error that fungi expected and should point to the line number on which it occurs. | 17:55 |
| @spamaps:spamaps.ems.host | > <@tristanc_:matrix.org> spamaps: have you looked at the openshiftpods driver?, i think it does what you are looking for | 17:58 |
| No I haven't. I'll take a look. Can you point me at some example jobs that make use of it? Thanks! | ||
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 818129: Report more YAML syntax errors https://review.opendev.org/c/zuul/zuul/+/818129 | 17:58 | |
| @jim:acmegating.com | (that was just a rebase) | 17:58 |
| @tristanc_:matrix.org | spamaps: in https://softwarefactory-project.io/cgit/third-party-ci-config/ you can find pre/base that runs with openshiftpods | 18:00 |
| @tristanc_:matrix.org | spamaps: and well, you might be interested in https://pagure.io/software-factory/k1s too | 18:01 |
| @spamaps:spamaps.ems.host | Cool thanks yes I just need some context because I am swimming in nodes and namespaces and service accounts and it's just a lot for PoC phase. :) | 18:06 |
| @clarkb:matrix.org | gtema: corvus fyi we're seeing odd swift uploads to ovh right now: https://zuul.opendev.org/t/openstack/build/bc0075f04a8941cf82dc536f76d6ecab/logs note the *** in the url error | 19:03 |
| @clarkb:matrix.org | doesn't seem to affect rax but rax has a separate cdn handling code path | 19:03 |
| @gtema:matrix.org | yes, I noticed this also on my installation | 19:03 |
| @gtema:matrix.org | not sure where the problem actually is | 19:03 |
| @jim:acmegating.com | Clark: seems like a quick revert is in order? and maybe test it next time? | 19:03 |
| @clarkb:matrix.org | corvus: Looking at the code I don't understand how this could be related, and the data is actually in the cloud if you fix the urls by hand (replace ***** with ovh for example) | 19:04 |
| @clarkb:matrix.org | but I think that may be in order | 19:04 |
| @clarkb:matrix.org | then we can rule it in or out | 19:04 |
| @clarkb:matrix.org | Basically it is the most likely reason right now | 19:04 |
| @jim:acmegating.com | who's pushing up the revert? | 19:05 |
| @clarkb:matrix.org | I'm running the opendev team meeting now if someone else can push it that would be great | 19:05 |
| @gtema:matrix.org | I'l open now | 19:06 |
| -@gerrit:opendev.org- Artem Goncharov proposed: [zuul/zuul-jobs] 817988: Revert "Add no_log to the log uploading module parameters" https://review.opendev.org/c/zuul/zuul-jobs/+/817988 | 19:06 | |
| @gtema:matrix.org | https://review.opendev.org/c/zuul/zuul-jobs/+/817988 | 19:06 |
| @jim:acmegating.com | thx approved | 19:07 |
| -@gerrit:opendev.org- Zuul merged on behalf of Artem Goncharov: [zuul/zuul-jobs] 817988: Revert "Add no_log to the log uploading module parameters" https://review.opendev.org/c/zuul/zuul-jobs/+/817988 | 19:18 | |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: | 19:20 | |
| - [zuul/zuul] 818101: Fix race in layoutstate https://review.opendev.org/c/zuul/zuul/+/818101 | ||
| - [zuul/zuul] 818129: Report more YAML syntax errors https://review.opendev.org/c/zuul/zuul/+/818129 | ||
| @gtema:matrix.org | corvus, Clark I think the problem is in Ansible. See example issue https://github.com/ansible/ansible/issues/71789. I need to think how to solve this | 20:03 |
| @jim:acmegating.com | gtema: could you no_log the task? | 20:04 |
| @clarkb:matrix.org | ya I think it is coming from heuritist_log_sanitize or similar in ansible | 20:04 |
| @gtema:matrix.org | yes, should be an option. Will test it now on my installation | 20:04 |
| @clarkb:matrix.org | and because cloud is in the url path and cloud is the name of the no_log option they collide? | 20:04 |
| @clarkb:matrix.org | Oh it might be explicitly removing the name of the cloud in the cloud dict from the url too | 20:09 |
| @clarkb:matrix.org | ya I think that is what actually happens because the name of the cloud is in the no_log dict so becomes part of the things to filter for | 20:11 |
| @gtema:matrix.org | I have seen some more cases where it censors more then only the cloud name, but for sure it is one part of the cleanup | 20:12 |
| @fungicide:matrix.org | interestingly thorough of it, though seems like a behavior which could break all sorts of things | 20:13 |
| @gtema:matrix.org | from what I have seen it censors potentially any content it finds in the no_log parameter | 20:13 |
| @gtema:matrix.org | there is a variety of issues in ansible about that | 20:14 |
| @gtema:matrix.org | "This is working as intended. Any value that is in a no_log field is masked, regardless of which field it is in. The values for all fields are compared to the set of no_log values and masked if there is a match." | 20:14 |
| @gtema:matrix.org | https://github.com/ansible/ansible/issues/71789#issuecomment-694463143 | 20:15 |
| @clarkb:matrix.org | _return_datastructure_name <- is doing it | 20:15 |
| @clarkb:matrix.org | "Return native stringified values from datastructures." | 20:15 |
| @clarkb:matrix.org | And since ovh is a value in the cloud dict it gets replaced in the url | 20:16 |
| @clarkb:matrix.org | this is not very user friendly | 20:16 |
| @clarkb:matrix.org | but maybe being cautious is the right choice for ansible | 20:16 |
| -@gerrit:opendev.org- Artem Goncharov proposed: [zuul/zuul-jobs] 818137: Apply no_log on the upload-logs-swift role https://review.opendev.org/c/zuul/zuul-jobs/+/818137 | 20:24 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 818116: Fix zuul-web startup config priming https://review.opendev.org/c/zuul/zuul/+/818116 | 20:59 | |
| @jim:acmegating.com | i don't understand why that test doesn't work in the gate, but i don't think it's worth fretting over too much right now, so i skipped it. | 21:00 |
| @jim:acmegating.com | i can't get it to fail locally :/ | 21:00 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 817869: Add an init phase to scheduler/web startup https://review.opendev.org/c/zuul/zuul/+/817869 | 21:06 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 818116: Fix zuul-web startup config priming https://review.opendev.org/c/zuul/zuul/+/818116 | 21:07 | |
| -@gerrit:opendev.org- Ian Wienand proposed: [zuul/nodepool] 818141: Bump DIB to 3.15.2 https://review.opendev.org/c/zuul/nodepool/+/818141 | 21:07 | |
| -@gerrit:opendev.org- Tobias Henkel proposed: [zuul/zuul] 818147: Remove reported_enqueue flag from queue item https://review.opendev.org/c/zuul/zuul/+/818147 | 21:40 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 817861: Support auth in multiple tabs https://review.opendev.org/c/zuul/zuul/+/817861 | 21:56 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed on behalf of Matthieu Huin https://matrix.to/#/@mhuin:matrix.org: | 21:57 | |
| - [zuul/zuul] 734850: web UI: allow a privileged user to dequeue a change https://review.opendev.org/c/zuul/zuul/+/734850 | ||
| - [zuul/zuul] 736772: web UI: allow a privileged user to re-enqueue a change https://review.opendev.org/c/zuul/zuul/+/736772 | ||
| - [zuul/zuul] 768115: Web UI: allow a privileged user to request autohold https://review.opendev.org/c/zuul/zuul/+/768115 | ||
| - [zuul/zuul] 768199: Web UI: add Autoholds, Autohold page https://review.opendev.org/c/zuul/zuul/+/768199 | ||
| - [zuul/zuul] 810699: Web UI: Show pipeline types as icons https://review.opendev.org/c/zuul/zuul/+/810699 | ||
| - [zuul/zuul] 781858: web UI: allow a privileged user to promote a change https://review.opendev.org/c/zuul/zuul/+/781858 | ||
| - [zuul/zuul] 802559: Web UI: Add "Create Autohold Request" form, improve API error messages https://review.opendev.org/c/zuul/zuul/+/802559 | ||
| - [zuul/zuul] 769943: Example Docker compose: keycloak integration https://review.opendev.org/c/zuul/zuul/+/769943 | ||
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: | 22:01 | |
| - [zuul/zuul] 816918: Upgrade react-router-dom https://review.opendev.org/c/zuul/zuul/+/816918 | ||
| - [zuul/zuul] 817861: Support auth in multiple tabs https://review.opendev.org/c/zuul/zuul/+/817861 | ||
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed on behalf of Matthieu Huin https://matrix.to/#/@mhuin:matrix.org: | 22:01 | |
| - [zuul/zuul] 734082: web UI: user login with OpenID Connect https://review.opendev.org/c/zuul/zuul/+/734082 | ||
| - [zuul/zuul] 734850: web UI: allow a privileged user to dequeue a change https://review.opendev.org/c/zuul/zuul/+/734850 | ||
| - [zuul/zuul] 736772: web UI: allow a privileged user to re-enqueue a change https://review.opendev.org/c/zuul/zuul/+/736772 | ||
| - [zuul/zuul] 768115: Web UI: allow a privileged user to request autohold https://review.opendev.org/c/zuul/zuul/+/768115 | ||
| - [zuul/zuul] 768199: Web UI: add Autoholds, Autohold page https://review.opendev.org/c/zuul/zuul/+/768199 | ||
| - [zuul/zuul] 810699: Web UI: Show pipeline types as icons https://review.opendev.org/c/zuul/zuul/+/810699 | ||
| - [zuul/zuul] 781858: web UI: allow a privileged user to promote a change https://review.opendev.org/c/zuul/zuul/+/781858 | ||
| - [zuul/zuul] 802559: Web UI: Add "Create Autohold Request" form, improve API error messages https://review.opendev.org/c/zuul/zuul/+/802559 | ||
| - [zuul/zuul] 769943: Example Docker compose: keycloak integration https://review.opendev.org/c/zuul/zuul/+/769943 | ||
| @jim:acmegating.com | Clark, tristanC: https://review.opendev.org/817869 and https://review.opendev.org/818116 are both ready to go -- they just got hit with (different) test races. | 22:54 |
| @clarkb:matrix.org | corvus: roger, I'll take a look momentarily. | 22:55 |
| @clarkb:matrix.org | corvus: does this zuul error: `Won't fetch project branches as read-only is set.` mean that zuul knows when a gerrit repo is read only? | 22:57 |
| @clarkb:matrix.org | if so that is neat | 22:57 |
| @jim:acmegating.com | Clark: nope, it means the component is a read-only consumer of the zk state | 22:58 |
| @jim:acmegating.com | (and that case is a cache-miss, so it can't proceed without a write to the state) | 22:59 |
| @jim:acmegating.com | it shouldn't typically happen... unless we have a bug like what that fixes :) | 23:00 |
| @clarkb:matrix.org | ok the current openstack zuul status seems to show that error. Is that what the zuul-web change should address? | 23:02 |
| @jim:acmegating.com | i think it should, yes. that error happens most easily at startup, but can happen any time. | 23:03 |
| @clarkb:matrix.org | got it | 23:03 |
| @clarkb:matrix.org | both lgtm | 23:09 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!