| *** tosky has quit IRC | 00:01 | |
| openstackgerrit | Merged zuul/zuul-jobs master: Clarity tox_environment accepts dictionary not list https://review.opendev.org/c/zuul/zuul-jobs/+/769433 | 00:05 |
|---|---|---|
| openstackgerrit | Merged zuul/zuul-jobs master: Document Python siblings handling for tox role https://review.opendev.org/c/zuul/zuul-jobs/+/768823 | 00:05 |
| *** ianychoi has joined #zuul | 00:14 | |
| *** zbr5 has joined #zuul | 00:22 | |
| *** zbr has quit IRC | 00:22 | |
| *** zbr5 is now known as zbr | 00:22 | |
| *** holser has quit IRC | 00:35 | |
| *** ikhan has quit IRC | 00:54 | |
| openstackgerrit | Merged zuul/zuul-jobs master: Add configuration to make logs public https://review.opendev.org/c/zuul/zuul-jobs/+/764483 | 01:02 |
| *** jamesmcarthur has quit IRC | 01:13 | |
| *** jamesmcarthur has joined #zuul | 01:15 | |
| *** icey has quit IRC | 01:15 | |
| *** icey has joined #zuul | 01:16 | |
| openstackgerrit | Merged zuul/zuul-jobs master: Allow to retrieve releasenotes requirements from a dedicated place https://review.opendev.org/c/zuul/zuul-jobs/+/769292 | 01:36 |
| *** jamesmcarthur has quit IRC | 01:36 | |
| *** jamesmcarthur has joined #zuul | 01:38 | |
| *** jamesmcarthur has quit IRC | 01:43 | |
| *** jamesmcarthur has joined #zuul | 01:48 | |
| *** zenkuro has quit IRC | 02:18 | |
| *** hamalq has quit IRC | 02:56 | |
| *** jamesmcarthur has quit IRC | 02:57 | |
| *** jamesmcarthur has joined #zuul | 02:57 | |
| *** jamesmcarthur has quit IRC | 03:03 | |
| *** jamesmcarthur has joined #zuul | 03:29 | |
| *** jamesmcarthur has quit IRC | 04:07 | |
| *** vishalmanchanda has joined #zuul | 05:11 | |
| *** evrardjp has quit IRC | 05:20 | |
| *** evrardjp has joined #zuul | 05:24 | |
| *** evrardjp has quit IRC | 05:33 | |
| *** evrardjp has joined #zuul | 05:35 | |
| *** ykarel has joined #zuul | 05:41 | |
| *** jamesmcarthur has joined #zuul | 06:04 | |
| *** jamesmcarthur has quit IRC | 06:09 | |
| openstackgerrit | Merged zuul/zuul-jobs master: Enable installing nimble siblings https://review.opendev.org/c/zuul/zuul-jobs/+/765672 | 07:13 |
| *** jcapitao has joined #zuul | 07:40 | |
| *** openstackgerrit has quit IRC | 07:47 | |
| *** jpena|off is now known as jpena | 07:51 | |
| *** hashar has joined #zuul | 08:03 | |
| *** piotrowskim has joined #zuul | 08:07 | |
| danpawlik | corvus, fungi, clarkb: hey, could you take a look on https://review.opendev.org/c/zuul/zuul/+/644927 and related to it changes? It got multiple times +2 . Thanks | 08:10 |
| *** rpittau|afk is now known as rpittau | 08:25 | |
| zbr | danpawlik: nice one | 08:34 |
| *** holser has joined #zuul | 08:34 | |
| *** tosky has joined #zuul | 08:39 | |
| *** saneax has quit IRC | 08:58 | |
| *** rfolco has joined #zuul | 09:05 | |
| *** jamesmcarthur has joined #zuul | 09:06 | |
| *** saneax has joined #zuul | 09:07 | |
| *** jamesmcarthur has quit IRC | 09:28 | |
| *** zenkuro has joined #zuul | 09:43 | |
| *** nils has joined #zuul | 09:50 | |
| *** ikhan has joined #zuul | 09:50 | |
| danpawlik | thanks for review zbr | 09:50 |
| *** hashar has quit IRC | 10:39 | |
| *** sshnaidm|afk is now known as sshnaidm|ruck | 11:19 | |
| *** jcapitao is now known as jcapitao_lunch | 11:40 | |
| *** zenkuro has quit IRC | 11:40 | |
| *** zenkuro has joined #zuul | 11:41 | |
| *** rlandy has joined #zuul | 12:16 | |
| *** jpena is now known as jpena|lunch | 12:49 | |
| *** jcapitao_lunch is now known as jcapitao | 12:55 | |
| *** vishalmanchanda has quit IRC | 13:01 | |
| *** rlandy is now known as rlandy|mtg | 13:04 | |
| jonass | avass: the aws diskimage support patch is now ready from my point of view. Already did some testing in our int environment. Did you also already manage to try it out? (https://review.opendev.org/c/zuul/nodepool/+/735217) | 13:23 |
| *** jamesmcarthur has joined #zuul | 13:26 | |
| avass | jonass: yeah I did some tests with it and it seems to work great | 13:26 |
| *** jamesmcarthur has quit IRC | 13:30 | |
| avass | jonass: I'll do a review on the change later :) | 13:34 |
| jonass | avass: Great :) I think there can still be done some improvements (e.g. build and upload vhd instead of raw). But those things can be done later in additional patches | 13:38 |
| *** rlandy|mtg is now known as rlandy | 13:44 | |
| *** jpena|lunch is now known as jpena | 13:51 | |
| *** saneax has quit IRC | 13:57 | |
| *** openstackgerrit has joined #zuul | 14:03 | |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: Example Docker compose: keycloak integration https://review.opendev.org/c/zuul/zuul/+/769943 | 14:03 |
| mordred | jonass: dib should already have support for building vhd - so hopefully that's an easy improvement | 14:09 |
| jonass | mordred: In theory yes, but I think i ran into a problem that at least under ubuntu the vhd-util package is required for that, which only existed until xenial (16.04). So i have to take a closer look at that. | 14:14 |
| mordred | jonass: oh - yeah - the vhd-util problem. we publish a copy into a PPA in opendev that we use for the vhd images we ahve to build | 14:29 |
| mordred | https://launchpad.net/~openstack-ci-core/+archive/ubuntu/vhd-util | 14:30 |
| mordred | fwiw | 14:30 |
| mordred | also: https://opendev.org/zuul/nodepool/src/branch/master/Dockerfile#L59-L69 - if you find that that vhd-util package helps - might be a good idea to update that comment to say "used for Rackspace and AWS" so that we don't accidentally remove it :) | 14:34 |
| *** vishalmanchanda has joined #zuul | 14:35 | |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: Web UI: allow a privileged user to request autohold https://review.opendev.org/c/zuul/zuul/+/768115 | 14:35 |
| *** jamesmcarthur has joined #zuul | 14:44 | |
| jonass | mordred: thanks for the hint, if so i will do that. | 14:46 |
| *** hashar has joined #zuul | 14:47 | |
| tobiash | mordred: alternatively qemu-img convert supports vhd as well do you know why dib uses vhd-util instead? | 14:58 |
| tobiash | ah there is a reason: https://docs.openstack.org/diskimage-builder/latest/developer/vhd_creation.html | 15:00 |
| *** zenkuro has quit IRC | 15:02 | |
| *** zenkuro has joined #zuul | 15:03 | |
| mordred | yeah. it's possible qemu-img is sufficient for aws, I'm not sure any of us have tried it. but I'd guess with their xen background the same issue might exist | 15:05 |
| tobiash | it was sufficient in our testing with aws | 15:08 |
| tobiash | I guess the image gets converted to some amazon standard during the import anyway | 15:09 |
| fungi | is aws xen based? if so, i didn't realize | 15:18 |
| tobiash | no idea, but it seems that qemu-img convert works for aws, but due to the xen support of dib I guess we cannot just use that in dib | 15:22 |
| mordred | tobiash: we could probably add some sort of flag to dib - to indicate you don't need the vhd-utils conversion and the qemu-img conversion would be fine | 15:24 |
| mordred | it would also be awesome if we'd reach a point where qemu-img would just work for rackspace too :) | 15:24 |
| mordred | fungi: it at least started off running xen | 15:25 |
| mordred | I have no idea what it runs today | 15:25 |
| corvus | danpawlik, tobiash: can you take a look at my comment on https://review.opendev.org/644927 ? | 15:28 |
| tobiash | corvus: makes sense | 15:31 |
| tobiash | corvus: is the get started stack ready to land? | 15:37 |
| clarkb | we have tested qemu-img a couple of times with rax and it has never worked. Making it a flag seems like a reasonable option | 15:37 |
| tobiash | plumbing such a flag into nodepool would be non-trivial however since it doesn't know if it builds rackspace vhd's or not | 15:38 |
| clarkb | tobiash: you could just set it as an env var like the other options for dib | 15:38 |
| clarkb | and users would haev to use the more conservative option if building for rax | 15:39 |
| clarkb | since that should work for rax and all others | 15:39 |
| tobiash | k, that would work | 15:39 |
| clarkb | (its a most fully featured image aiui) | 15:39 |
| fungi | tobiash: we need mnaser to take a look at a couple of changes in there which add vexxhost information and logo, otherwise yes i think so | 15:39 |
| mnaser | sorry, just +1'd them :) | 15:39 |
| fungi | thanks mnaser! | 15:39 |
| tobiash | k, then I'll +w them | 15:39 |
| fungi | tobiash: then yes, i'd say the get started series is good to go | 15:40 |
| danpawlik | corvus, thanks for review. Agree that I did not reply correctly to clarkb comments, there is just one comment from me which can be confusing for others. I will apply my changes into the change. | 15:40 |
| mnaser | fungi: fwiw aws used to be xen based but with their new and more recent 'nitro fancy stuff they are using kvm afaik | 15:43 |
| *** hashar is now known as hasharAway | 15:43 | |
| openstackgerrit | Merged zuul/zuul-website master: Adding Get Started page https://review.opendev.org/c/zuul/zuul-website/+/766520 | 15:45 |
| *** jamesmcarthur has quit IRC | 15:45 | |
| *** jamesmcarthur has joined #zuul | 15:46 | |
| openstackgerrit | Merged zuul/zuul-website master: Adjust title on Get Started page https://review.opendev.org/c/zuul/zuul-website/+/770490 | 15:46 |
| openstackgerrit | Merged zuul/zuul-website master: Add "Get Started" link to header https://review.opendev.org/c/zuul/zuul-website/+/770491 | 15:46 |
| openstackgerrit | Merged zuul/zuul-website master: Add vexxhost and opendev logos to get-started https://review.opendev.org/c/zuul/zuul-website/+/770492 | 15:47 |
| openstackgerrit | daniel.pawlik proposed zuul/zuul master: A Zuul reporter for Elasticsearch https://review.opendev.org/c/zuul/zuul/+/644927 | 15:48 |
| *** jamesmcarthur has quit IRC | 15:51 | |
| openstackgerrit | Merged zuul/zuul-website master: Link to opendev.org on get-started page https://review.opendev.org/c/zuul/zuul-website/+/770493 | 15:53 |
| danpawlik | corvus, tobiash, clarkb: related patchset for that change https://review.opendev.org/c/zuul/zuul/+/644927 has been squashed. Please review it in free time. Thanks | 15:53 |
| openstackgerrit | Merged zuul/zuul-website master: Add Acme Gating to get-started page https://review.opendev.org/c/zuul/zuul-website/+/770494 | 15:53 |
| *** jamesmcarthur has joined #zuul | 15:54 | |
| corvus | danpawlik: looks great, thanks! +1 from me; i assum tobiash will re-add his, and we should let clarkb +2. hopefully this makes it easier for him to do so :) | 15:55 |
| clarkb | fwiw I'm not sure that the docs update is correct. | 15:56 |
| clarkb | elasticsearch releases server specific client libs and theo ne we have chosen is for es7 iirc | 15:57 |
| clarkb | so we won't follow latest, we'll always support es7 unless that lib is changed | 15:57 |
| clarkb | oh I see we have uncapped it so ya nevermind | 15:57 |
| clarkb | but then when the next major elasticsearch python lib comes out it will only support the next version ofelasticsearch the server something like that | 15:58 |
| *** ykarel is now known as ykarel|away | 16:09 | |
| clarkb | corvus: danpawlik tobiash that lgtm now. The test and cleanup of the encoder in particular are much appreciated. I +2'd it but didn't approve since corvus only +1'd (though I think that was to ensure I could have another read of it first?) | 16:11 |
| corvus | clarkb: yep i +3d | 16:15 |
| openstackgerrit | Tristan Cacqueray proposed zuul/zuul master: Separate out executor concerns from AnsibleJob https://review.opendev.org/c/zuul/zuul/+/648643 | 16:16 |
| *** ykarel|away has quit IRC | 16:16 | |
| *** jamesmcarthur has quit IRC | 16:32 | |
| *** jamesmcarthur has joined #zuul | 16:34 | |
| openstackgerrit | Tristan Cacqueray proposed zuul/zuul master: runner: implement prep-workspace https://review.opendev.org/c/zuul/zuul/+/607082 | 16:35 |
| openstackgerrit | Tristan Cacqueray proposed zuul/zuul master: runner: add configuration schema https://review.opendev.org/c/zuul/zuul/+/640672 | 16:35 |
| openstackgerrit | Tristan Cacqueray proposed zuul/zuul master: runner: add prepare-workspace command line interface https://review.opendev.org/c/zuul/zuul/+/644770 | 16:35 |
| openstackgerrit | Tristan Cacqueray proposed zuul/zuul master: runner: add execute sub-command https://review.opendev.org/c/zuul/zuul/+/630944 | 16:35 |
| openstackgerrit | Tristan Cacqueray proposed zuul/zuul master: runner: add job parameters listing https://review.opendev.org/c/zuul/zuul/+/644795 | 16:35 |
| openstackgerrit | Tristan Cacqueray proposed zuul/zuul master: web: add depends-on support to the freeze job API https://review.opendev.org/c/zuul/zuul/+/639022 | 16:35 |
| *** jamesmcarthur has quit IRC | 16:38 | |
| *** tosky has quit IRC | 16:41 | |
| *** tosky has joined #zuul | 16:42 | |
| *** jamesmcarthur has joined #zuul | 16:50 | |
| *** jcapitao has quit IRC | 16:59 | |
| *** piotrowskim has quit IRC | 17:04 | |
| *** jpena is now known as jpena|off | 17:05 | |
| *** jamesmcarthur has quit IRC | 17:29 | |
| *** jamesmcarthur has joined #zuul | 17:30 | |
| *** sshnaidm|ruck is now known as sshnaidm|afk | 17:30 | |
| *** zenkuro has quit IRC | 17:31 | |
| *** nils has quit IRC | 17:51 | |
| *** hamalq has joined #zuul | 18:03 | |
| *** rpittau is now known as rpittau|afk | 18:05 | |
| openstackgerrit | Andy Ladjadj proposed zuul/zuul-jobs master: [ensure-python] install python version only if not present https://review.opendev.org/c/zuul/zuul-jobs/+/770656 | 18:06 |
| *** cloudnull has quit IRC | 18:12 | |
| *** cloudnull has joined #zuul | 18:12 | |
| openstackgerrit | Merged zuul/zuul master: A Zuul reporter for Elasticsearch https://review.opendev.org/c/zuul/zuul/+/644927 | 18:52 |
| corvus | jamesmcarthur: ^ that might be worth a small marketing push | 18:52 |
| openstackgerrit | Andy Ladjadj proposed zuul/zuul-jobs master: [ensure-python] install python version only if not present https://review.opendev.org/c/zuul/zuul-jobs/+/770656 | 18:53 |
| corvus | i mean, it's not in a released version, but it's merged and available for folks running CD | 18:53 |
| jamesmcarthur | corvus: oh, that's awesome! | 18:53 |
| jamesmcarthur | thank you for the flag :) | 18:53 |
| corvus | np | 18:53 |
| corvus | jamesmcarthur: i expected the get-started page to be live now but isn't; i will look into that this afternoon and update you | 18:54 |
| jamesmcarthur | corvus: I was wondering about that. I saw the patch was approved + the extra stuff you added. | 18:55 |
| jamesmcarthur | Thanks for checking on it. | 18:55 |
| fungi | corvus: jamesmcarthur: we had an afs problem earlier today and i'm still working on getting things caught back up, so the site publication is delayed | 18:56 |
| fungi | it's queued behind a full volume release of the tarballs site | 18:57 |
| jamesmcarthur | ah ha! | 18:57 |
| fungi | but yeah i planned to give you all a heads up once that update is live | 18:58 |
| openstackgerrit | Andy Ladjadj proposed zuul/zuul-jobs master: [ensure-python] install python version only if not present https://review.opendev.org/c/zuul/zuul-jobs/+/770656 | 19:04 |
| *** rlandy is now known as rlandy|biab | 19:04 | |
| *** jamesmcarthur has quit IRC | 19:06 | |
| *** jamesmcarthur_ has joined #zuul | 19:11 | |
| openstackgerrit | Oleksandr Kozachenko proposed zuul/zuul master: Add swift proxy to contrib https://review.opendev.org/c/zuul/zuul/+/708233 | 19:18 |
| openstackgerrit | Andy Ladjadj proposed zuul/zuul-jobs master: [ensure-python] install python version only if not present https://review.opendev.org/c/zuul/zuul-jobs/+/770656 | 19:24 |
| *** hamalq has quit IRC | 19:29 | |
| *** paladox has quit IRC | 19:34 | |
| *** hasharAway has quit IRC | 19:36 | |
| *** paladox has joined #zuul | 19:39 | |
| *** zenkuro has joined #zuul | 19:50 | |
| openstackgerrit | Tristan Cacqueray proposed zuul/zuul master: Separate out executor concerns from AnsibleJob https://review.opendev.org/c/zuul/zuul/+/648643 | 19:53 |
| *** jamesmcarthur_ has quit IRC | 19:54 | |
| *** jamesmcarthur has joined #zuul | 19:54 | |
| *** rlandy|biab is now known as rlandy | 20:01 | |
| openstackgerrit | Andy Ladjadj proposed zuul/zuul-jobs master: [ensure-python] install python version only if not present https://review.opendev.org/c/zuul/zuul-jobs/+/770656 | 20:08 |
| *** jamesmcarthur has quit IRC | 20:14 | |
| openstackgerrit | Andy Ladjadj proposed zuul/zuul-jobs master: [ensure-python] install python version only if not present https://review.opendev.org/c/zuul/zuul-jobs/+/770656 | 20:24 |
| openstackgerrit | Tristan Cacqueray proposed zuul/zuul master: Separate out executor concerns from AnsibleJob https://review.opendev.org/c/zuul/zuul/+/648643 | 20:52 |
| *** ikhan has quit IRC | 20:54 | |
| *** vishalmanchanda has quit IRC | 21:41 | |
| openstackgerrit | Tristan Cacqueray proposed zuul/zuul master: Separate out executor concerns from AnsibleJob https://review.opendev.org/c/zuul/zuul/+/648643 | 22:00 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: Fix zuul_run_command ret when command is not found https://review.opendev.org/c/zuul/zuul/+/735475 | 22:13 |
| openstackgerrit | Guillaume Chauvel proposed zuul/zuul master: Fix zuul_run_command ret when command is not found https://review.opendev.org/c/zuul/zuul/+/735475 | 22:40 |
| openstackgerrit | Tristan Cacqueray proposed zuul/zuul master: runner: implement prep-workspace https://review.opendev.org/c/zuul/zuul/+/607082 | 22:52 |
| openstackgerrit | Tristan Cacqueray proposed zuul/zuul master: runner: add configuration schema https://review.opendev.org/c/zuul/zuul/+/640672 | 22:52 |
| *** rlandy has quit IRC | 23:36 | |
| corvus | tobiash: you had a +2 on https://review.opendev.org/732066 previously; i'll leave it to you to re-add / +W | 23:38 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!