| *** y2kenny has quit IRC | 00:10 | |
| *** saneax_ has quit IRC | 01:03 | |
| *** wuchunyang has joined #zuul | 01:03 | |
| *** wuchunyang has quit IRC | 01:08 | |
| *** swest has quit IRC | 01:15 | |
| *** swest has joined #zuul | 01:29 | |
| *** rlandy has joined #zuul | 01:45 | |
| *** rlandy has quit IRC | 01:51 | |
| *** bhavikdbavishi has joined #zuul | 02:47 | |
| *** bhavikdbavishi has joined #zuul | 02:48 | |
| *** bhavikdbavishi1 has joined #zuul | 02:52 | |
| *** bhavikdbavishi has quit IRC | 02:54 | |
| *** bhavikdbavishi1 is now known as bhavikdbavishi | 02:54 | |
| *** Goneri has quit IRC | 03:00 | |
| *** wuchunyang has joined #zuul | 03:16 | |
| *** wuchunyang has quit IRC | 03:21 | |
| *** bhavikdbavishi1 has joined #zuul | 04:21 | |
| *** bhavikdbavishi has quit IRC | 04:22 | |
| *** bhavikdbavishi1 is now known as bhavikdbavishi | 04:22 | |
| *** evrardjp has quit IRC | 04:33 | |
| *** evrardjp has joined #zuul | 04:33 | |
| *** wuchunyang has joined #zuul | 04:59 | |
| *** saneax_ has joined #zuul | 05:00 | |
| *** sgw has quit IRC | 05:04 | |
| *** ysandeep|away is now known as ysandeep | 05:21 | |
| *** wuchunyang has quit IRC | 05:37 | |
| *** dry has quit IRC | 05:38 | |
| *** ysandeep is now known as ysandeep|brb | 05:38 | |
| *** reiterative has quit IRC | 05:42 | |
| *** reiterative has joined #zuul | 05:42 | |
| *** bhavikdbavishi1 has joined #zuul | 05:47 | |
| *** bhavikdbavishi has quit IRC | 05:49 | |
| *** bhavikdbavishi1 is now known as bhavikdbavishi | 05:49 | |
| *** ysandeep|brb is now known as ysandeep | 06:04 | |
| *** dustinc has quit IRC | 06:35 | |
| *** hashar has joined #zuul | 07:14 | |
| *** rpittau|afk is now known as rpittau | 07:18 | |
| *** bhavikdbavishi has quit IRC | 07:20 | |
| *** jcapitao has joined #zuul | 07:21 | |
| *** sshnaidm|afk is now known as sshnaidm | 07:23 | |
| *** tosky has joined #zuul | 07:38 | |
| *** bhavikdbavishi has joined #zuul | 07:40 | |
| openstackgerrit | Merged zuul/zuul master: Fix test race in test_client_dequeue_ref https://review.opendev.org/734032 | 07:41 |
|---|---|---|
| *** sugaar has joined #zuul | 07:54 | |
| *** jpena|off is now known as jpena | 07:57 | |
| avass | we're planning on upgrading our gerrit instance from 2.14 to 3.2. just checking if anyone else is running a newer version and if there could be any problems with zuul? | 08:01 |
| guillaumec | avass, I'm not really running an instance. I'm working on importing new tutorials, running 3.2.1, no issues :) | 08:07 |
| avass | ah yeah I guess quickstart is running the latest version of gerrit anyway | 08:36 |
| openstackgerrit | Sagi Shnaidman proposed zuul/zuul-jobs master: Add jobs for testing ensure-ansible https://review.opendev.org/734584 | 08:39 |
| guillaumec | avass, gerritcodereview/gerrit:latest was recently updated to 3.2.0, "latest" is just a tag, it does not necessarly point to the most recent version, https://groups.google.com/forum/#!searchin/repo-discuss/docker$20image%7Csort:date/repo-discuss/XKZSNtF5RwA/Evxsa-fsBgAJ | 08:54 |
| avass | guillaumec: yeha I know, but it usually maps to the latest docker image unless you already have it locally | 09:01 |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: zuul-web: support OPTIONS for protected endpoints https://review.opendev.org/734134 | 09:13 |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: zuul-web: refactor auth token handling code https://review.opendev.org/734139 | 09:14 |
| *** rpittau is now known as rpittau|bbl | 09:15 | |
| guillaumec | avass, it went from: "2020-06-01 15:20:34,847 INFO zuul.GerritConnection: Remote version is: 3.0.1 (parsed as (3, 0, 1))" (quick-start's scheduler log from review) | 09:17 |
| guillaumec | avass, to 2020-06-02 13:26:45,800 INFO zuul.GerritConnection: Remote version is: 3.2.0 (parsed as (3, 2, 0)) | 09:17 |
| *** jcapitao has quit IRC | 10:16 | |
| openstackgerrit | Sagi Shnaidman proposed zuul/zuul-jobs master: Add jobs for testing ensure-ansible https://review.opendev.org/734584 | 10:20 |
| *** jcapitao has joined #zuul | 10:22 | |
| *** jcapitao has quit IRC | 10:26 | |
| *** jcapitao has joined #zuul | 10:28 | |
| *** wuchunyang has joined #zuul | 10:28 | |
| *** wuchunyang has quit IRC | 10:33 | |
| *** jcapitao has quit IRC | 10:47 | |
| *** brendangalloway has joined #zuul | 10:49 | |
| brendangalloway | Hi - we're currently having an issue with many jobs failing due to retry limit | 10:49 |
| brendangalloway | I've enabled keep on the logs, and the only ouput is starting console and running ansible setup. | 10:50 |
| brendangalloway | Looking through the logs, I don't see any indication why the job failed, apart from a warning for timeout exceeded. Could that be the root? | 10:51 |
| *** bhavikdbavishi has quit IRC | 11:09 | |
| *** bhavikdbavishi has joined #zuul | 11:10 | |
| *** threestrands has quit IRC | 11:10 | |
| *** dennis_effa has joined #zuul | 11:20 | |
| *** fbo|off is now known as fbo | 11:22 | |
| *** jpena is now known as jpena|lunch | 11:40 | |
| *** brendangalloway has quit IRC | 11:44 | |
| fungi | if brendangalloway returns, i guess the answer is that if it's a pre playbook consistently hitting its configured timeout, then zuul will retry that until it reaches the retry limit and fail | 11:50 |
| *** rpittau|bbl is now known as rpittau | 11:54 | |
| *** bhavikdbavishi has quit IRC | 11:55 | |
| *** rlandy has joined #zuul | 12:16 | |
| zbr | raised https://github.com/ansible/ansible/issues/70003 related to the "failures" keyword we had yesterday, better to comment there if you want changes in ansible. | 12:20 |
| mordred | avass: we're also running a zuul against upstream zuul | 12:20 |
| mordred | avass: AND - opendev is about to work in upgrading away from 2.13 as well :) | 12:20 |
| avass | mordred: nice :) | 12:21 |
| mordred | avass: fwiw - I highly recommend setting up the checks plugin for your 3.2 deployment - zuul already has support for it | 12:21 |
| avass | mordred: oh is that ready? last I looked it was still experimental | 12:21 |
| mordred | it's running on gerrit upstream | 12:22 |
| mordred | it IS still experiemental in that it's not complete | 12:22 |
| mordred | but we've been explicitly workoing with the upstream checks plugins folks from zuul, so the interaction should be fairly solid with what's there - subchecks is the thing that's missing still | 12:22 |
| avass | ah cool | 12:23 |
| avass | I'll take a look at that whenever we upgrade | 12:23 |
| avass | mordred: ah so I'll guess we'll have to configure the checks for each project in gerrit | 12:27 |
| mordred | avass: yeah. and it's really a "Zuul" check | 12:29 |
| mordred | subchecks is the thing that will then let zuul report back which jobs are going to run for a given change | 12:30 |
| avass | yeah, I remember that discussion when you were in gothenburg. hopefully it's a bit more synced since I remember them having a different view of what subchecks should be :) | 12:31 |
| *** bhavikdbavishi has joined #zuul | 12:31 | |
| mordred | yeah. :) - corvus has been working with them on the spec | 12:31 |
| avass | nice :) | 12:32 |
| *** guilhermesp has quit IRC | 12:32 | |
| *** mnaser has quit IRC | 12:32 | |
| *** gmann has quit IRC | 12:33 | |
| *** lseki has quit IRC | 12:33 | |
| *** guilhermesp has joined #zuul | 12:33 | |
| *** lseki has joined #zuul | 12:33 | |
| *** gmann has joined #zuul | 12:33 | |
| *** mnaser has joined #zuul | 12:33 | |
| *** jpena|lunch is now known as jpena | 12:46 | |
| corvus | avass: oh i *don't* recommend setting up the checks plugin :) | 12:50 |
| corvus | mordred: ^ | 12:50 |
| avass | oh, how come? | 12:52 |
| corvus | not for use with zuul yet. it's going to be a lot harder to manage than just using streaming events. a lot of stuff beyond a simple check pipeline doesn't work as well. | 12:52 |
| corvus | subchecks isn't the only missing piece -- we also need webhooks, and more events emitted from the checks plugin | 12:53 |
| corvus | and the checks plugin needs to be able to match checkers on more than one project | 12:53 |
| avass | is it like 'do not rely on it' or more like 'do not use it, things will break'? :) | 12:55 |
| corvus | basically, it's at a stage where it *can* work to provide most of the functionality we have today. but it takes more work and its not as polished. | 12:55 |
| corvus | and is *far* less efficient and scalable | 12:56 |
| *** bhavikdbavishi has quit IRC | 12:56 | |
| *** bhavikdbavishi has joined #zuul | 12:58 | |
| mordred | avass: definitley listen to corvus and not me :) | 12:58 |
| corvus | there are mixed-configuration options that can make it better | 12:58 |
| corvus | so it also kind-of depends on the deployment situtation. it's worth looking into, but it's not at a stage where i would give a blanket recommendation to use it. | 13:00 |
| corvus | there's a reason there's a big warning in the zuul documentation about trying it out. :) | 13:00 |
| *** harrymichal has joined #zuul | 13:01 | |
| avass | yeah, that's what I was guessing :) | 13:01 |
| avass | anyway, since I have your attention. Want to take a look at: https://review.opendev.org/#/c/734774/13 for assigning instance profiles to nodes in aws? | 13:02 |
| avass | (and if you have time: https://review.opendev.org/#/c/733675/ but that change is a bit larger) | 13:02 |
| avass | :) | 13:02 |
| openstackgerrit | Sorin Sbarnea (zbr) proposed zuul/zuul master: Enable display of failures inside zuul console https://review.opendev.org/734833 | 13:09 |
| *** hashar has quit IRC | 13:28 | |
| guillaumec | mordred, small change for gerrit checks: https://review.opendev.org/#/c/732569/ | 13:29 |
| *** sgw has joined #zuul | 13:43 | |
| *** olaph has joined #zuul | 13:48 | |
| mordred | SpamapS: I know you don't really have much in the way of time for zuul reviews - but if you get bored it would be nice to have you give https://review.opendev.org/#/c/733675/ a once-over since you have zuul+terraform experience | 13:57 |
| *** bhavikdbavishi has quit IRC | 14:18 | |
| *** iurygregory_ has joined #zuul | 14:19 | |
| *** iurygregory has quit IRC | 14:19 | |
| *** iurygregory_ is now known as iurygregory | 14:19 | |
| openstackgerrit | Sagi Shnaidman proposed zuul/zuul-jobs master: Add jobs for testing ensure-ansible https://review.opendev.org/734584 | 14:29 |
| openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: Add linting rule to enforce no-same-owner policy https://review.opendev.org/727642 | 14:35 |
| openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: Add linting rule to enforce no-same-owner policy https://review.opendev.org/727642 | 14:39 |
| corvus | avass: done! i left comments on both | 14:46 |
| *** Goneri has joined #zuul | 14:46 | |
| openstackgerrit | Sagi Shnaidman proposed zuul/zuul-jobs master: Add jobs for testing ensure-ansible https://review.opendev.org/734584 | 14:51 |
| avass | corvus: ah yeah, boto3 doesn't seem to be clear to whether both the name and the arn is needed: https://boto3.amazonaws.com/v1/documentation/api/latest/reference/services/ec2.html#EC2.ServiceResource.create_instances | 14:52 |
| avass | and describe iam instance profile seem to need arn or id, which is different I believe :) | 14:52 |
| avass | corvus: but it could be that either the name or the arn is needed | 14:53 |
| corvus | avass: yeah, i agree that's ambiguous; though the links to the api docs i think end up on the page i found, so i'm leaning toward assuming it's only one is required; is it easy for you to test? | 14:54 |
| avass | corvus: not atm, but there's no hurry since I'm on vacation for a week now so I could probably take a look at it :) | 14:55 |
| *** jkt has joined #zuul | 15:01 | |
| avass | corvus: and yeah, I wanted to have it as a review comment to start with. but there wasn't a way to do that through zuul atm so I just stuck with a file comment for now, though I didn't know you could just omit the line :) | 15:01 |
| avass | corvus: and regarding moving the packer jobs to hashicorp, that was a way to group the jobs together a bit to avoid bloating the index too much | 15:03 |
| corvus | avass: yeah, it's probably fine -- it seems a little weird to group it by "owning company", but i also agree that having a page with one entry is not great | 15:10 |
| corvus | maybe packer could be in an image building jobs page, and terraform under deployment jobs or something | 15:10 |
| avass | yeah, either is fine | 15:10 |
| corvus | otoh, if using tf implies using packer, then grouping them together makes a lot of sense. but i don't get the impression that there's necessarily a strong implied relationship there | 15:11 |
| avass | no there isn't really, and I'm planning on setting up dib jobs instead for what we can | 15:11 |
| corvus | okay. no -1 on that, i'm happy to move them to hashicorp now and think about something else later :) | 15:12 |
| *** sshnaidm_ has joined #zuul | 15:15 | |
| openstackgerrit | Riccardo Pittau proposed zuul/zuul master: Correct syntax to be compatible with mysql8 https://review.opendev.org/735174 | 15:16 |
| *** sshnaidm has quit IRC | 15:16 | |
| *** bhavikdbavishi has joined #zuul | 15:19 | |
| mhu | sorry if I missed something, but what's the "RollCall-Vote" for in Gerrit? | 15:22 |
| corvus | mhu: it's only on some repos, usually governance ones, where members of a body have an official vote | 15:23 |
| *** hashar has joined #zuul | 15:23 | |
| corvus | mhu: we don't have any of those in zuul, but the openstack tc and opendev teams use it | 15:23 |
| mhu | got it, thanks | 15:24 |
| clarkb | you'll also find a prioritization as well as a backport category on some repos | 15:24 |
| openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: Terraform roles and jobs. https://review.opendev.org/733675 | 15:26 |
| openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: Terraform roles and jobs. https://review.opendev.org/733675 | 15:28 |
| avass | corvus: I believe that should take care of everything | 15:30 |
| corvus | cool, i think that can probably go in once we double check the output | 15:33 |
| avass | I was planning on setting up a check pipeline that runs 'plan' by assigning an instance-profile with a post-merge pipeline that applies the config, and maybe do some testing in the gate by spinning up temporary resources. still trying to figure out how far I can take this :) | 15:35 |
| corvus | avass: we're really happy with our "spin up a real $serivce" jobs in opendev's gate. it's not always easy, but it's really rewarding. | 15:40 |
| avass | yeah I can imagine | 15:40 |
| fungi | we tested an etherpad upgrade yesterday by proposing a broken change to our configuration management and telling zuul to set an autohold | 15:41 |
| fungi | and then had a usable sample deployment to poke around in | 15:41 |
| SpamapS | mordred: I'll peek in a bit. :) | 15:41 |
| avass | fungi: hah, yeah that's one way to do it | 15:43 |
| clarkb | avass: and that generated a bug report to etherpad and now we're waiting on that to figure out what our plans are before upgrading | 15:45 |
| clarkb | (it was actually a useful exercise) | 15:45 |
| *** panda has quit IRC | 15:51 | |
| *** panda has joined #zuul | 15:53 | |
| SpamapS | mordred: looks pretty good. I do worry about printing plans in code review systems.. as it's pretty easy to screw up and let a password through, but I suppose that's true in many other cases as well. | 15:54 |
| avass | SpamapS: well the passwords shouldn't be part of the logs anyway :) | 15:56 |
| *** rpittau is now known as rpittau|afk | 15:58 | |
| *** ysandeep is now known as ysandeep|away | 15:59 | |
| avass | SpamapS: but I guess the recommendation from terraform is to treat the state as sensitive data: https://www.terraform.io/docs/state/sensitive-data.html#recommendations | 16:00 |
| openstackgerrit | Fabien Boucher proposed zuul/zuul master: gitlab - add dynamic reconfig test https://review.opendev.org/735186 | 16:01 |
| openstackgerrit | Fabien Boucher proposed zuul/zuul master: gitlab - add test checking tenant reconfig when ref updated https://review.opendev.org/735187 | 16:01 |
| openstackgerrit | Fabien Boucher proposed zuul/zuul master: gitlab: add enqueue/dequeue unit tests https://review.opendev.org/735188 | 16:01 |
| openstackgerrit | Fabien Boucher proposed zuul/zuul master: gitlab - Add support for depends-on dependency https://review.opendev.org/735189 | 16:01 |
| mordred | SpamapS: woot, thanks! | 16:04 |
| * mordred cheers on fbo's gitlab work | 16:04 | |
| corvus | avass: happy with the new file comments? want to make leaving comments off by default? should we go ahead and approve it then? | 16:04 |
| avass | corvus: lgtm | 16:05 |
| corvus | tristanC: ^ feel free to +3 https://review.opendev.org/733675 after re-review | 16:07 |
| avass | and now when I think about it, it might make more sense to have file comments than review comments, since there could be multiple terraform jobs running at the same time | 16:08 |
| corvus | i have to suit up for a grocery shopping expidition, so i'll be biab. | 16:08 |
| avass | so that makes it easier to see which execution plan comes from which configuration | 16:08 |
| corvus | avass: i thought we added the job name to comments recently (so our linters job says "linters: you left out a space"). maybe we did that in the job itself; maybe you could do that here too? | 16:09 |
| corvus | (also, maybe we should make that automatic in zuul) | 16:09 |
| SpamapS | avass: My experience is that there's a pretty good chance somebody will fail to mark a piece of data in terraform as sensitive. which will make it print out in the plan. | 16:09 |
| SpamapS | And sometimes providers are hastily written, and don't mark things as sensitive either. | 16:10 |
| avass | corvus: yeah that's part of the tox_parse_output.py script | 16:10 |
| SpamapS | So.. yeah.. I'd just make sure that can be turned off. | 16:10 |
| avass | SpamapS: sure, but that's still just slightly worse since it would end up in the logs anyway | 16:10 |
| AJaeger | corvus: btw. I pushed https://review.opendev.org/677578 forward so far (configure-os-mirror) but will not have time to give it the final push, could you take it back or find another expert to help further, please? | 16:10 |
| avass | SpamapS: there could be a way to no_log it | 16:11 |
| avass | AJaeger: I can take a look at it, doesn't look like there's much left to do | 16:11 |
| avass | and I need to go get some food, I'll be back later :) | 16:12 |
| corvus | sounds like we need a centos/suse expert? | 16:12 |
| avass | oh, yeah that might be good :) | 16:14 |
| *** iurygregory has quit IRC | 16:15 | |
| openstackgerrit | Merged zuul/zuul master: Run upload-docker-image on release https://review.opendev.org/734902 | 16:25 |
| sshnaidm_ | AJaeger, avass, mordred and others, can you re-review please in your time https://review.opendev.org/#/c/730360 | 16:46 |
| *** sshnaidm_ is now known as sshnaidm | 16:46 | |
| *** sshnaidm is now known as sshnaidm|afk | 16:46 | |
| avass | sshnaidm|afk: my comment still stands, I'm not entirely convinced that ensure-ansible needs to be a role since it adds so little. mordred can you take a look at that as well ^ ? | 16:48 |
| avass | sshnaidm|afk: but I won't be in the way of adding it if other maintainers are ok with it. | 16:50 |
| sshnaidm|afk | avass, well, if no need - I can delete, it's easier than to create :) I'm open to suggestions | 16:51 |
| sshnaidm|afk | actually added it according to mordred request | 16:51 |
| AJaeger | corvus: I can help from SUSE side, we can ask dirk as well | 16:55 |
| *** jpena is now known as jpena|off | 17:00 | |
| AJaeger | corvus, avass, let me propose an update on top of 677578 on how to make openSUSE mirrors a bit more fine-granular - if we like it, we can squash it into 677578 | 17:05 |
| *** dennis_effa has quit IRC | 17:10 | |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: web UI: user login with OpenID Connect https://review.opendev.org/734082 | 17:16 |
| *** fbo is now known as fbo|off | 17:17 | |
| mhu | corvus: tristanC is moving out this week, he might be between flats right now, so not really available for reviews:) | 17:25 |
| mhu | mordred, corvus AJaeger what do you think of the idea of a zuul web CLI as in https://review.opendev.org/#/c/729236/ ? Or should it move to a separate project? | 17:37 |
| *** hashar has quit IRC | 17:41 | |
| corvus | mhu: ack thanks, i'll just carry over his previous +2 then | 17:46 |
| corvus | mhu: i think maybe a new project | 17:47 |
| mhu | corvus, on opendev? otherwise we can also host it on SF | 17:48 |
| corvus | mhu: i think it could be a zuul project. why don't you start a zuul-discuss ML list thread and gather feedback | 17:49 |
| corvus | zuul/zuul-cli or something | 17:50 |
| openstackgerrit | Andreas Jaeger proposed zuul/zuul-jobs master: Update openSUSE Mirrors https://review.opendev.org/735207 | 18:08 |
| openstackgerrit | Andreas Jaeger proposed zuul/zuul-jobs master: Update openSUSE Mirrors for configure-os-mirrors https://review.opendev.org/735207 | 18:09 |
| avass | corvus: looks like name or arn is enough, so I'll update that | 18:09 |
| AJaeger | avass, corvus, fungi, I updated the openSUSE mirrors for configure-os-mirrors in 735207, please tell me whether that works better - and is not over-engineered. | 18:12 |
| * AJaeger waits now for test results ;) | 18:12 | |
| AJaeger | oh, it will fail - I'll update test-playbooks/base-roles/configure-new-mirrors.yaml | 18:12 |
| openstackgerrit | Andreas Jaeger proposed zuul/zuul-jobs master: Update openSUSE Mirrors for configure-os-mirrors https://review.opendev.org/735207 | 18:14 |
| avass | corvus: and interestingly enough, name and arn may not be used at the same time :) | 18:14 |
| avass | that seems like something that should have been documented in boto | 18:15 |
| openstackgerrit | Andreas Jaeger proposed zuul/zuul-jobs master: DNM: Debug openSUSE mirror loop https://review.opendev.org/735209 | 18:23 |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: web UI: user login with OpenID Connect https://review.opendev.org/734082 | 18:25 |
| *** bhavikdbavishi has quit IRC | 18:29 | |
| *** bhavikdbavishi has joined #zuul | 18:30 | |
| openstackgerrit | Andreas Jaeger proposed zuul/zuul-jobs master: Update openSUSE Mirrors for configure-os-mirrors https://review.opendev.org/735207 | 18:35 |
| openstackgerrit | Andreas Jaeger proposed zuul/zuul-jobs master: DNM: Debug openSUSE mirror loop https://review.opendev.org/735209 | 18:35 |
| *** bhavikdbavishi has quit IRC | 18:45 | |
| openstackgerrit | Albin Vass proposed zuul/nodepool master: aws: add support for attaching instance profiles https://review.opendev.org/734774 | 18:46 |
| avass | corvus, mordred: that should fix it ^ | 18:46 |
| openstackgerrit | Andreas Jaeger proposed zuul/zuul-jobs master: Update openSUSE Mirrors for configure-os-mirrors https://review.opendev.org/735207 | 18:47 |
| openstackgerrit | Andreas Jaeger proposed zuul/zuul-jobs master: DNM: Debug openSUSE mirror loop https://review.opendev.org/735209 | 18:47 |
| openstackgerrit | Andreas Jaeger proposed zuul/zuul-jobs master: Update openSUSE Mirrors for configure-os-mirrors https://review.opendev.org/735207 | 18:55 |
| corvus | i'm going to re-enqueue the latest zuul release to check on the new dockerhub job | 18:57 |
| corvus | naturally that didn't work; i'll have an iterative improvement after lunch | 19:05 |
| *** rfolco|rover has joined #zuul | 19:08 | |
| openstackgerrit | Andreas Jaeger proposed zuul/zuul-jobs master: Update openSUSE Mirrors for configure-os-mirrors https://review.opendev.org/735207 | 19:12 |
| openstackgerrit | Albin Vass proposed zuul/nodepool master: WIP: aws: add support for uploading diskimages https://review.opendev.org/735217 | 19:15 |
| openstackgerrit | Andreas Jaeger proposed zuul/zuul-jobs master: Update openSUSE Mirrors for configure-os-mirrors https://review.opendev.org/735207 | 19:15 |
| *** rlandy is now known as rlandy|afk | 19:24 | |
| openstackgerrit | Andreas Jaeger proposed zuul/zuul-jobs master: Update openSUSE Mirrors for configure-os-mirrors https://review.opendev.org/735207 | 19:28 |
| AJaeger | avass, corvus, fungi, this should pass now, reviews welcome when you find time the next days ^ | 19:31 |
| AJaeger | corvus: do you want to merge https://review.opendev.org/731606 now? | 19:49 |
| corvus | AJaeger: sounds good; looks like all the suggestions can be done as followups if desired | 20:06 |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: web UI: user login with OpenID Connect https://review.opendev.org/734082 | 20:07 |
| *** rfolco|rover has quit IRC | 20:16 | |
| avass | corvus, AJaeger: I saw something that needs fixing, -2 to stop it from merging | 20:20 |
| avass | it shouldn't use a lookup if it's already using with_first_found | 20:21 |
| corvus | avass: ack, i cleared my vote | 20:24 |
| openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: Update guideline docs for os specific tasks https://review.opendev.org/731606 | 20:24 |
| avass | corvus, AJaeger: that should be better | 20:25 |
| avass | ah wait | 20:25 |
| openstackgerrit | Albin Vass proposed zuul/zuul-jobs master: Update guideline docs for os specific tasks https://review.opendev.org/731606 | 20:25 |
| avass | corvus, AJaeger: that's better :) | 20:25 |
| avass | corvus: I missed this the other day by the way: https://review.opendev.org/#/c/733478/ | 20:26 |
| *** rlandy|afk is now known as rlandy | 20:26 | |
| corvus | i think i can just +3 that one | 20:31 |
| avass | what's the post upload hook for in nodepool builder? | 20:33 |
| avass | ah, like verifying that the image is actually available in the provider? | 20:34 |
| openstackgerrit | James E. Blair proposed zuul/zuul-jobs master: Add test-build-container-image-release job https://review.opendev.org/735249 | 21:00 |
| openstackgerrit | Albin Vass proposed zuul/nodepool master: WIP: aws: add support for uploading diskimages https://review.opendev.org/735217 | 21:00 |
| corvus | avass: i think more for verifying the image works (but yes it would also implicitly verify it's available, but that's something that nodepool should detect as part of its upload process) | 21:02 |
| avass | corvus: ah so something like starting an instance from the image and testing it's functionality | 21:03 |
| avass | corvus: it doesn't seem to hard to add dib support for aws, just need to get a good understanding on how nodepool-builder works at the moment :) | 21:04 |
| corvus | avass: yep | 21:04 |
| mordred | corvus: test-build-container-image-release just makes me giggle | 21:08 |
| corvus | mordred: test-???-release! | 21:08 |
| mordred | corvus: ... profit | 21:08 |
| mordred | corvus: also - sorry about the unconditional use of label :) | 21:09 |
| mordred | corvus: you left --label "org.zuul-ci.change_url={{ zuul.change_url }}" | 21:09 |
| mordred | corvus: wouldn't that also need an if? or did you leave that to make sure the job fails properly | 21:09 |
| corvus | mordred: np :) and yeah -- that is present even in a ref pipeline. it's just poorly named. | 21:09 |
| avass | mordred: I was just about to comment on that :) | 21:10 |
| mordred | ah ... nod | 21:10 |
| avass | oh | 21:10 |
| mordred | yay for naming | 21:10 |
| corvus | it's our proud zuul v2 legacy | 21:10 |
| mordred | corvus: \o/ | 21:10 |
| mordred | corvus: should we even be setting that as a docker label then? | 21:10 |
| mordred | or - should we call the docker label something else? | 21:10 |
| corvus | mordred: https://opendev.org/zuul/zuul/commit/c12f3fe1defe8b61d59061363c9c04fb520dae18 | 21:11 |
| corvus | that's what it is for a tag | 21:11 |
| mordred | corvus: that's a good thing to have in the label I think | 21:11 |
| corvus | mordred: i think if we figure out what zuul.change_url should be, we should change the docker label, but until then, i can't think of a better docker label name :) | 21:11 |
| mordred | corvus: :) | 21:12 |
| mordred | corvus: looking at that link - gitea doens't seem to know about your key | 21:12 |
| mordred | I wonder if there is a way we can inform it of keys | 21:12 |
| mordred | (that's more of an opendev thought) | 21:12 |
| clarkb | keys for what? | 21:13 |
| mordred | clarkb: gpg key - but I just brought it up in #opendev | 21:13 |
| *** armstrongs has joined #zuul | 21:31 | |
| *** Defolos_ has left #zuul | 21:34 | |
| openstackgerrit | Merged zuul/zuul master: zookeeper-tls-howto: Remove unused keystorepassword https://review.opendev.org/733478 | 21:37 |
| *** armstrongs has quit IRC | 21:40 | |
| *** dustinc has joined #zuul | 21:55 | |
| guillaumec | mordred, corvus: I am looking at ansible/base/library/command.py, I'm looking at trace by putting more console.addLine because I've been unable to get the log from AnsibleModule.log. I am using ansible verbose 'vvvv', ANSIBLE_DEBUG, I read https://docs.ansible.com/ansible/latest/dev_guide/developing_program_flow_modules.html#ansible-no-log, can you give me a hint ? I am using zuul/zuul-executor container | 22:07 |
| openstackgerrit | James E. Blair proposed zuul/zuul-jobs master: Add test-build-container-image-release job https://review.opendev.org/735249 | 22:07 |
| *** threestrands has joined #zuul | 22:20 | |
| *** harrymichal has quit IRC | 22:21 | |
| guillaumec | found why, no syslog on tutorial node ^^ | 22:45 |
| corvus | guillaumec: sorry i was afk and don't fully understand the problem. it sounds like you have something working now? but if not, let me know, i'll try to help. | 23:03 |
| guillaumec | corvus, yes i do, and found root cause, preparing a change :) | 23:03 |
| mordred | \o/ | 23:04 |
| *** tosky has quit IRC | 23:10 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!