| @theikki6:matrix.org | Hello friends! | 07:59 |
|---|---|---|
| I am wondering if anyone knows a way to exclude something from a reference? | ||
| For example; in a job I can include my vars and add more vars like this | ||
| - job: | ||
| name: foo_bar | ||
| vars: | ||
| <<: *my_wonderful_vars | ||
| new_var: true | ||
| is there a way to not inlcude a specific variable in the reference somehow? | ||
| @francesco_sc:matrix.org | Hi, I am tryiing to let github and zuul communicate, but I don't see the zuul pipelines under the github settings-branch tab, any idea? | 08:06 |
| -@gerrit:opendev.org- Felix Edel proposed: | 13:13 | |
| - [zuul/zuul] 818868: Combine different history approaches for merge jobs in tests https://review.opendev.org/c/zuul/zuul/+/818868 | ||
| - [zuul/zuul] 818205: WIP: Add source attribute to GitConnection https://review.opendev.org/c/zuul/zuul/+/818205 | ||
| -@gerrit:opendev.org- Felix Edel proposed on behalf of Simon Westphahl: [zuul/zuul] 815278: DNM: execute tests with two schedulers https://review.opendev.org/c/zuul/zuul/+/815278 | 13:13 | |
| @jim:acmegating.com | theikki6: not that i'm aware of. you might consider setting the value to 'null' and making sure your playbook treats that as if it was not present. | 14:46 |
| @jim:acmegating.com | francesco_sc: here's the docs for configuring zuul with github: https://zuul-ci.org/docs/zuul/reference/drivers/github.html#configure-github | 14:48 |
| -@gerrit:opendev.org- Matthieu Huin https://matrix.to/#/@mhuin:matrix.org proposed: [zuul/zuul-client] 819118: Support for "basic" authentication https://review.opendev.org/c/zuul/zuul-client/+/819118 | 14:52 | |
| @francesco_sc:matrix.org | Thanks corvus , I did what is written in the documentation, but I still can't see the pipeline on github. | 16:35 |
| I see the following message: Sorry, we couldn’t find any status checks in the last week for this repository. | ||
| Learn more about status checks on GitHub. | ||
| @jim:acmegating.com | they may only show up after jobs have run and zuul has reported once? | 16:44 |
| @francesco_sc:matrix.org | sorry corvus , I didn't get your question | 17:07 |
| @tobias.henkel:matrix.org | corvus: +2 with one typo on 809300 | 17:08 |
| @jim:acmegating.com | francesco_sc: i'm not sure a check will appear there until zuul has run jobs and reported | 17:09 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 809300: Add a zookeeper map to developer docs https://review.opendev.org/c/zuul/zuul/+/809300 | 17:10 | |
| @jim:acmegating.com | tobiash: thx fixed :) | 17:10 |
| @francesco_sc:matrix.org | OK, so how can I make check run for the first time? | 17:11 |
| @francesco_sc:matrix.org | I tried to open a PR with projects file: | 17:11 |
| - project: | ||
| name: myrepo | ||
| vars: | ||
| check: | ||
| fail-fast: true | ||
| jobs: | ||
| - gitlint | ||
| - small-checks | ||
| gate: | ||
| fail-fast: true | ||
| - noop | ||
| post: | ||
| jobs: | ||
| - noop | ||
| periodic: | ||
| jobs: | ||
| - noop | ||
| @jim:acmegating.com | francesco_sc: you might want to follow the tutorial here: https://zuul-ci.org/docs/zuul/tutorials/quick-start.html it's written for gerrit, but it walks you through the process of adding a project and running jobs. | 17:12 |
| @jim:acmegating.com | francesco_sc: if a project doesn't already have a check pipeline configuration, then zuul won't report errors in a PR that creates one. you may have errors in that code but won't see them because of that. start by adding a very simple empty pipeline config first. the tutorial recommends that; see https://zuul-ci.org/docs/zuul/tutorials/quick-start.html#configure-zuul-pipelines under `zuul.d/projects.yaml`) | 17:16 |
| @tobias.henkel:matrix.org | corvus, mhu : is it feasible to extend the web ui authentication work later to optionally require auth for read access as well? That would be useful for us such that we don't need to combine oauth2-proxy with the admin login | 17:22 |
| @mhuin:matrix.org | tobiash: you mean make zuul accessible only if you are authenticated? | 17:23 |
| @tobias.henkel:matrix.org | yes | 17:23 |
| @tobias.henkel:matrix.org | currently we have it behind a saml authenticating proxy (we're switching to keycloak and oidc soon) | 17:23 |
| @tobias.henkel:matrix.org | and we don't allow any anonymous access in our deployment | 17:24 |
| @mhuin:matrix.org | would that need to be tenant-scoped too? | 17:24 |
| @tobias.henkel:matrix.org | in the best case yes | 17:24 |
| @jim:acmegating.com | i think so, but the api info endpoints would probably still need to be publicly accessible; and maybe the tenant list depending on the details. | 17:26 |
| @jim:acmegating.com | basically -- the js web app gets the authentication url from the info endpoint | 17:27 |
| @tobias.henkel:matrix.org | I'd only care about the status, job, builds etc | 17:28 |
| @mhuin:matrix.org | I agree, but I think oauth2-proxy is the simplest way to achieve this. Any particular reason why you'd like to part from it? | 17:28 |
| @tobias.henkel:matrix.org | ideally the tenant list would be filtered if we go the tenant-scoped route | 17:28 |
| @tobias.henkel:matrix.org | I guess I have to try it out, oauth2-proxy is too simple for path based group requirements | 17:29 |
| @tobias.henkel:matrix.org | that requires a more complex setup using apache likely | 17:29 |
| @mhuin:matrix.org | ah, I see | 17:29 |
| @tobias.henkel:matrix.org | I just thought if zuul can do auth it might be a good idea if it can do all auth | 17:29 |
| @tobias.henkel:matrix.org | I guess at least if using oauth2 proxy it can forward the token so the user doesn't have to do double login? | 17:30 |
| @jim:acmegating.com | (to be clear: i don't see anything in the current stack that would prevent this work in the future, and i think the current stack will help enable it if we want to. if we choose to do it, it's additional work that we'd do later and not a refactor of the current stack) | 17:33 |
| @tobias.henkel:matrix.org | totally clear | 17:34 |
| @tobias.henkel:matrix.org | corvus, mhu : +2 with q on 810699 | 17:49 |
| @jim:acmegating.com | replied | 17:51 |
| @tobias.henkel:matrix.org | whoops, thanks :) | 17:52 |
| @tobias.henkel:matrix.org | just curious, will oidc-react automatically refresh the tokens? | 17:58 |
| @mhuin:matrix.org | yes | 17:58 |
| @mhuin:matrix.org | if you enable the console, you'll see a call to refresh the token one minute before it expires | 17:59 |
| @tobias.henkel:matrix.org | so 818295 is for those cases when the auth provider doesn't supply a refresh token | 17:59 |
| @jim:acmegating.com | actually i think in implicit flow we don't get a refresh token at all | 17:59 |
| @tobias.henkel:matrix.org | yeah, implicit forbids refresh tokens afaik | 18:00 |
| @jim:acmegating.com | so in normal circumstances we're "refreshing" by logging in again silently. so 818295 is for the cases where that fails | 18:00 |
| @jim:acmegating.com | and it might fail because of the SSO session timing out or being explicitly logged out | 18:00 |
| @tobias.henkel:matrix.org | k | 18:00 |
| @mhuin:matrix.org | exactly, that's how I tested it: I signed out off keycloak and let the token expire | 18:01 |
| @tobias.henkel:matrix.org | I think I'm through the stack | 18:02 |
| @tobias.henkel:matrix.org | thanks a lot for this work and sorry for the long review delay | 18:02 |
| @mhuin:matrix.org | thanks for the review! It's not an easy thing to evaluate | 18:04 |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 815207: Document (and warn about) tag job role checkouts https://review.opendev.org/c/zuul/zuul/+/815207 | 18:13 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed on behalf of Simon Westphahl: [zuul/zuul] 807868: Checkout playbook branch for roles running on tag https://review.opendev.org/c/zuul/zuul/+/807868 | 18:13 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 819156: Add SOS documentation and remove spec https://review.opendev.org/c/zuul/zuul/+/819156 | 18:32 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed: [zuul/zuul] 819156: Add SOS documentation and remove spec https://review.opendev.org/c/zuul/zuul/+/819156 | 18:33 | |
| @spamaps:spamaps.ems.host | > <@theikki6:matrix.org> Hello friends! | 19:34 |
| > | ||
| > I am wondering if anyone knows a way to exclude something from a reference? | ||
| > For example; in a job I can include my vars and add more vars like this | ||
| > | ||
| > - job: | ||
| > name: foo_bar | ||
| > vars: | ||
| > <<: *my_wonderful_vars | ||
| > new_var: true | ||
| > | ||
| > is there a way to not inlcude a specific variable in the reference somehow? | ||
| That level of composition is best done via roles, not yaml references,. | ||
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed on behalf of Felix Edel: | 21:51 | |
| - [zuul/zuul] 818773: Increase MySQL connection limit in tests https://review.opendev.org/c/zuul/zuul/+/818773 | ||
| - [zuul/zuul] 818774: Remove unused _get_reporter_from_connection_name helper method https://review.opendev.org/c/zuul/zuul/+/818774 | ||
| - [zuul/zuul] 818775: Add missing attributes to Gitlab MergeRequest https://review.opendev.org/c/zuul/zuul/+/818775 | ||
| - [zuul/zuul] 818054: Make TestWeb.test_web_components work with multiple schedulers https://review.opendev.org/c/zuul/zuul/+/818054 | ||
| - [zuul/zuul] 818203: Execute Github tests with only one scheduler https://review.opendev.org/c/zuul/zuul/+/818203 | ||
| - [zuul/zuul] 818862: Only use a single createScheduler() helper method in tests https://review.opendev.org/c/zuul/zuul/+/818862 | ||
| - [zuul/zuul] 818863: Limit scheduler_count to 1 for Scale out Scheduler tests https://review.opendev.org/c/zuul/zuul/+/818863 | ||
| - [zuul/zuul] 818864: Limit scheduler_count to 1 for broken tenant config tests https://review.opendev.org/c/zuul/zuul/+/818864 | ||
| - [zuul/zuul] 818867: Don't use RecordingMergeClient.history in TestNonLiveMerges https://review.opendev.org/c/zuul/zuul/+/818867 | ||
| - [zuul/zuul] 818868: Combine different history approaches for merge jobs in tests https://review.opendev.org/c/zuul/zuul/+/818868 | ||
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed on behalf of Simon Westphahl: | 21:51 | |
| - [zuul/zuul] 818215: Allow running tests with multiple schedulers https://review.opendev.org/c/zuul/zuul/+/818215 | ||
| - [zuul/zuul] 815787: Refresh pipelines in tests when settled https://review.opendev.org/c/zuul/zuul/+/815787 | ||
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul] 816918: Upgrade react-router-dom https://review.opendev.org/c/zuul/zuul/+/816918 | 22:09 | |
| -@gerrit:opendev.org- Zuul merged on behalf of Simon Westphahl: [zuul/zuul] 818360: Load job from pipeline state on executors https://review.opendev.org/c/zuul/zuul/+/818360 | 22:15 | |
| -@gerrit:opendev.org- Zuul merged on behalf of Matthieu Huin https://matrix.to/#/@mhuin:matrix.org: [zuul/zuul] 734082: web UI: user login with OpenID Connect https://review.opendev.org/c/zuul/zuul/+/734082 | 22:15 | |
| -@gerrit:opendev.org- James E. Blair https://matrix.to/#/@jim:acmegating.com proposed on behalf of Felix Edel: [zuul/zuul] 818868: Combine different history approaches for merge jobs in tests https://review.opendev.org/c/zuul/zuul/+/818868 | 22:26 | |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul] 817861: Support auth in multiple tabs https://review.opendev.org/c/zuul/zuul/+/817861 | 22:28 | |
| -@gerrit:opendev.org- Zuul merged on behalf of James E. Blair https://matrix.to/#/@jim:acmegating.com: [zuul/zuul] 809300: Add a zookeeper map to developer docs https://review.opendev.org/c/zuul/zuul/+/809300 | 23:00 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!