| openstackgerrit | Ian Wienand proposed zuul/nodepool master: [dnm] testing docker sibling copy https://review.opendev.org/696988 | 00:03 |
|---|---|---|
| SpamapS | pabelanger: we use packer.io on our AWS nodepool.. but we just build them in a zuul job, and then we have a tag filter that we use to find the latest one we built | 00:04 |
| SpamapS | Which is why we landed the change for tag filters in nodepool. :) | 00:05 |
| SpamapS | As cool as nodepool-builder is.. zuul's a pretty good image builder too. :) | 00:05 |
| openstackgerrit | Ian Wienand proposed zuul/zuul-jobs master: [wip] copy listed siblings to docker build https://review.opendev.org/696987 | 00:08 |
| *** rlandy has quit IRC | 00:33 | |
| openstackgerrit | Ian Wienand proposed zuul/zuul-jobs master: [wip] copy listed siblings to docker build https://review.opendev.org/696987 | 01:23 |
| *** yolanda has quit IRC | 01:25 | |
| *** rfolco has quit IRC | 01:30 | |
| *** rfolco has joined #zuul | 01:38 | |
| *** rfolco has quit IRC | 01:39 | |
| *** rfolco has joined #zuul | 01:40 | |
| *** bhavikdbavishi has joined #zuul | 01:40 | |
| openstackgerrit | Ian Wienand proposed zuul/zuul-jobs master: [wip] copy listed siblings to docker build https://review.opendev.org/696987 | 01:40 |
| openstackgerrit | Ian Wienand proposed zuul/zuul-jobs master: [wip] copy listed siblings to docker build https://review.opendev.org/696987 | 01:56 |
| *** Goneri has quit IRC | 01:57 | |
| *** bhavikdbavishi has quit IRC | 01:57 | |
| *** rfolco has quit IRC | 02:07 | |
| openstackgerrit | Ian Wienand proposed zuul/zuul-jobs master: build-docker-image: add option to install siblings https://review.opendev.org/696987 | 02:26 |
| *** bhavikdbavishi has joined #zuul | 02:41 | |
| *** bhavikdbavishi1 has joined #zuul | 03:34 | |
| *** bhavikdbavishi has quit IRC | 03:36 | |
| *** bhavikdbavishi1 is now known as bhavikdbavishi | 03:36 | |
| *** jangutter has joined #zuul | 03:59 | |
| *** jangutter has quit IRC | 04:03 | |
| *** igordc has quit IRC | 04:34 | |
| *** jangutter has joined #zuul | 04:46 | |
| *** jangutter has quit IRC | 04:51 | |
| *** raukadah is now known as chkumar|ruck | 05:37 | |
| *** mattw4 has quit IRC | 06:00 | |
| *** jangutter has joined #zuul | 06:47 | |
| *** jangutter has quit IRC | 06:52 | |
| openstackgerrit | Felix Schmidt proposed zuul/zuul master: Store information about gate resets in MQTT and SQL reporter https://review.opendev.org/696670 | 07:04 |
| *** yolanda has joined #zuul | 07:22 | |
| *** AJaeger has quit IRC | 07:38 | |
| *** AJaeger has joined #zuul | 07:44 | |
| *** fdegir4 is now known as fdegir | 07:46 | |
| *** johanssone has quit IRC | 07:51 | |
| *** johanssone has joined #zuul | 07:52 | |
| *** jangutter has joined #zuul | 08:04 | |
| *** jcapitao has joined #zuul | 08:14 | |
| *** bhavikdbavishi has quit IRC | 08:21 | |
| *** bhavikdbavishi has joined #zuul | 08:22 | |
| *** saneax has joined #zuul | 08:23 | |
| *** reiterative has joined #zuul | 08:26 | |
| *** bhavikdbavishi1 has joined #zuul | 08:29 | |
| *** tosky has joined #zuul | 08:29 | |
| *** bhavikdbavishi has quit IRC | 08:30 | |
| *** bhavikdbavishi1 is now known as bhavikdbavishi | 08:30 | |
| openstackgerrit | Simon Westphahl proposed zuul/zuul master: Fix exceptions on reconfig with changing window https://review.opendev.org/697023 | 08:36 |
| *** hashar has joined #zuul | 08:41 | |
| *** jpena|off is now known as jpena | 08:48 | |
| openstackgerrit | Simon Westphahl proposed zuul/zuul master: Fix exceptions on reconfig with changing window https://review.opendev.org/697023 | 08:55 |
| *** bogdando has joined #zuul | 09:12 | |
| openstackgerrit | Simon Westphahl proposed zuul/zuul master: Optionally allow zoned executors to process unzoned jobs https://review.opendev.org/673840 | 09:15 |
| *** mhu has joined #zuul | 09:16 | |
| *** saneax has quit IRC | 09:44 | |
| openstackgerrit | Ian Wienand proposed zuul/zuul-jobs master: build-docker-image: add option to install siblings https://review.opendev.org/696987 | 09:59 |
| openstackgerrit | Bogdan Dobrelya (bogdando) proposed zuul/zuul-jobs master: DNM testing custom pypi mirrors (696337) https://review.opendev.org/696929 | 10:07 |
| *** pcaruana has joined #zuul | 10:08 | |
| *** bogdando has left #zuul | 10:11 | |
| *** saneax has joined #zuul | 10:21 | |
| *** electrofelix has joined #zuul | 10:22 | |
| *** mhu has quit IRC | 10:25 | |
| *** sshnaidm|afk is now known as sshnaidm | 10:26 | |
| *** mhu has joined #zuul | 10:30 | |
| *** dmellado has quit IRC | 10:33 | |
| *** dmellado has joined #zuul | 10:35 | |
| *** bogdando has joined #zuul | 10:44 | |
| bogdando | pabelanger: hi, around? | 10:44 |
| openstackgerrit | Tobias Henkel proposed zuul/zuul master: Fix updating change with draft change on top https://review.opendev.org/697055 | 10:45 |
| bogdando | I'm trying to test https://review.opendev.org/696337 via forked base-jobs at my local zuul, as a config-project. I followed example: | 10:47 |
| bogdando | https://github.com/ansible/ansible-zuul-jobs/blob/master/zuul.d/jobs.yaml#L2 | 10:47 |
| bogdando | https://github.com/ansible/project-config/blob/master/zuul/tenants.yaml#L5 | 10:47 |
| bogdando | but for my case (https://github.com/bogdando/base-jobs/commit/21319184009e611f4a4da5742ce192ae50cee1d9 ), there is KeyError if I use gihub.com source... | 10:47 |
| bogdando | was the problem that each source listed in main.yaml must be also defined in projects.jaml? | 10:49 |
| bogdando | pabelanger: so for that example I have KeyError: 'github.com' in scheduler logs | 11:07 |
| bogdando | what did I miss when following https://github.com/ansible/project-config/blob/master/zuul/tenants.yaml#L13 example?.. | 11:08 |
| bogdando | for tentant config I use: | 11:11 |
| bogdando | source: | 11:11 |
| bogdando | github.com: | 11:11 |
| bogdando | config-projects: | 11:11 |
| bogdando | - bogdando/base-jobs | 11:11 |
| bogdando | gerrit: | 11:11 |
| bogdando | untrusted-projects: | 11:11 |
| bogdando | - test1 | 11:11 |
| bogdando | - test2 | 11:11 |
| bogdando | opendev.org: | 11:11 |
| bogdando | untrusted-projects | 11:11 |
| bogdando | ... | 11:11 |
| *** mhu has quit IRC | 11:13 | |
| *** mhu has joined #zuul | 11:18 | |
| bogdando | pabelanger: nevermind, I think I'm using wrong approach | 11:54 |
| *** zbr_ has quit IRC | 11:55 | |
| *** zbr has joined #zuul | 11:56 | |
| *** gtema has joined #zuul | 12:02 | |
| *** jcapitao is now known as jcapitao|lunch | 12:02 | |
| *** rfolco has joined #zuul | 12:12 | |
| *** rfolco has quit IRC | 12:23 | |
| *** rfolco has joined #zuul | 12:24 | |
| sugaar | tristanC, Shrews can you confirm if I am fdoing the right approach? It doesn't work right now because the nodepool container can't find the mounted k8s container but I am working on that, it is not related to nodepool I guess is more about containers https://paste.gnome.org/pssfuj23e | 12:27 |
| *** gtema has quit IRC | 12:31 | |
| tristanC | sugaar: the paste is 404 | 12:32 |
| sugaar | try now | 12:37 |
| sugaar | https://paste.gnome.org/pdu4aqhid | 12:37 |
| tristanC | sugaar: the last line should rather be nodepool -c /etc/nodepool/nodepool.yaml | 12:40 |
| sugaar | oh that's true! I changed so many times that I end up messing it up. thanks for that | 12:43 |
| *** jpena is now known as jpena|lunch | 12:45 | |
| *** bogdando has left #zuul | 13:09 | |
| *** jcapitao|lunch is now known as jcapitao | 13:16 | |
| *** rlandy has joined #zuul | 13:18 | |
| *** rlandy is now known as rlandy|mtg | 13:32 | |
| *** jpena|lunch is now known as jpena | 13:49 | |
| *** rlandy|mtg is now known as rlandy | 14:02 | |
| *** Goneri has joined #zuul | 14:12 | |
| *** chkumar|ruck is now known as raukadah | 14:21 | |
| *** jhesketh has quit IRC | 14:27 | |
| *** jhesketh has joined #zuul | 14:28 | |
| *** pcaruana has quit IRC | 14:33 | |
| *** smyers has quit IRC | 14:47 | |
| *** saneax has quit IRC | 14:52 | |
| *** michael-beaver has joined #zuul | 14:55 | |
| *** igordc has joined #zuul | 14:57 | |
| *** pcaruana has joined #zuul | 15:04 | |
| *** igordc has quit IRC | 15:17 | |
| *** igordc has joined #zuul | 15:25 | |
| *** igordc has quit IRC | 15:30 | |
| *** jlvillal has quit IRC | 15:34 | |
| *** jlvillal has joined #zuul | 15:35 | |
| *** jcapitao is now known as jcapitao|afk | 15:46 | |
| *** jcapitao|afk is now known as jcapitao | 16:09 | |
| dmsimard | tobiash: I came across something I had a question on, added a comment in https://review.opendev.org/#/c/638801/ | 16:19 |
| tobiash | dmsimard: appearently it works: https://storage.bhs1.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_951/697055/1/check/tox-py37/9511e98/zuul-info/inventory.yaml | 16:21 |
| tobiash | check out the message in the inventory ;) | 16:21 |
| dmsimard | tobiash: how, though ? :p | 16:21 |
| dmsimard | tobiash: reading it like that it's modifying a dict that isn't used afterwards | 16:22 |
| tobiash | zuul_vars is a reference to the value of all_vars['zuul | 16:22 |
| tobiash | and there the message is just updated | 16:22 |
| tobiash | so in fact it *is* modifying all_vars | 16:23 |
| dmsimard | ok, what confused me was 'vars': all_vars, vs 'vars': zuul_vars | 16:23 |
| dmsimard | s/what/was/ | 16:23 |
| dmsimard | thanks :p | 16:23 |
| *** electrofelix has quit IRC | 16:29 | |
| sugaar | can I limit or constrain the number of nodes requested by nodepool? | 16:31 |
| pabelanger | max-servers in a provider will do that | 16:31 |
| sugaar | pabelanger thanks | 16:33 |
| clarkb | pabelanger: sugaar: max-nodes-per-job is a tenant setting that gives you better control | 16:35 |
| clarkb | max-servers in a provider implies a ceiling for max-nodes-per-job since zuul and nodepool try to schedule all nodes for a job in a single provider, but you can further limit this with max-nodes-per-job | 16:35 |
| *** hashar has quit IRC | 16:39 | |
| pabelanger | +1 | 16:40 |
| sugaar | clarkb so if I understood correctly max-servers limits the number of total nodes and max-nodes-per-job the number of nodes allocated to a task? | 16:40 |
| clarkb | sugaar: max-servers limits how many total instances can run at any time in a provider across all jobs. max-nodes-per-job restricts it on a per job basis | 16:41 |
| openstackgerrit | James E. Blair proposed zuul/zuul-jobs master: WIP: openshift speculative containers https://review.opendev.org/696939 | 16:41 |
| sugaar | clarkb all right, thanks for the clarification | 16:43 |
| fungi | well, limits how many instances can be requested per build of a job | 16:44 |
| fungi | so if you set max-nodes-per-job=5 then people can't create 6-node jobs | 16:44 |
| fungi | s/create/run/ | 16:44 |
| *** jpena is now known as jpena|brb | 16:44 | |
| fungi | but it doesn't limit how many nodes in aggregate can be used by concurrently-running builds of the same job | 16:45 |
| *** jlvillal has quit IRC | 16:45 | |
| *** jlvillal has joined #zuul | 16:46 | |
| *** jlvillal has quit IRC | 16:50 | |
| *** sgw has joined #zuul | 17:10 | |
| *** jlvillal has joined #zuul | 17:11 | |
| *** jcapitao has quit IRC | 17:12 | |
| openstackgerrit | Matthieu Huin proposed zuul/zuul master: authentication config: add optional max_token_age, skew https://review.opendev.org/642408 | 17:13 |
| *** jlvillal has quit IRC | 17:14 | |
| *** jpena|brb is now known as jpena | 17:23 | |
| clarkb | corvus: did you want to make the change suggested at https://review.opendev.org/#/c/695251/4/roles/build-container-image/common.rst seems like a reasonable update | 17:25 |
| corvus | clarkb, tristanC: yeah, responded on change. let's hold approval for that. | 17:31 |
| clarkb | corvus: I left a couple comments on the two child changes as well. I didn't approve and will wait for updates | 17:41 |
| openstackgerrit | James E. Blair proposed zuul/zuul-jobs master: WIP: openshift speculative containers https://review.opendev.org/696939 | 17:42 |
| corvus | thanks | 17:42 |
| *** jhesketh has quit IRC | 17:42 | |
| *** jhesketh has joined #zuul | 17:44 | |
| *** rlandy is now known as rlandy|brb | 17:45 | |
| *** mattw4 has joined #zuul | 17:50 | |
| openstackgerrit | Merged zuul/nodepool master: Support optional post upload hooks https://review.opendev.org/676266 | 17:51 |
| *** mattw4 has quit IRC | 17:51 | |
| *** mattw4 has joined #zuul | 17:52 | |
| mordred | tobiash: tristanC pointed out that podman build can use -v mounting when doing container image builds (where docker build can't) - this just made me realize this can be used as a way to set mirror locations in things like sources.list (-v the sources.list file) - without impacting the target image ... which seems like something that might make you happy | 18:00 |
| mordred | clarkb: ^^ also - we should ponder the use of that in opendev too, as we think about mirror things | 18:01 |
| clarkb | I know some of the container folk do not want our mirrors during build time | 18:02 |
| clarkb | we probably want to make it configurable at least | 18:03 |
| mordred | clarkb: that's mostly because they don't want our mirrors to wind up in the final image | 18:03 |
| clarkb | though if we finish the mirror rebuilds and ssl them maybe we cans top worrying | 18:03 |
| mordred | yeah | 18:03 |
| clarkb | mordred: well and the packages are untrusted | 18:03 |
| mordred | yah - that too | 18:03 |
| mordred | in any case- I agree, it should be optional | 18:03 |
| tristanC | mordred: i've also a wip for extended layers management with buildah here: https://github.com/containers/buildah/pull/1790 | 18:04 |
| mordred | but it would also be nice to plumb through optional support for using all of the mirrors we have in the new world of mirror registries something something | 18:04 |
| *** michael-beaver has quit IRC | 18:05 | |
| tobiash | mordred: awesome :) | 18:10 |
| *** igordc has joined #zuul | 18:15 | |
| *** sshnaidm is now known as sshnaidm|afk | 18:21 | |
| *** rlandy|brb is now known as rlandy | 18:33 | |
| *** jpena is now known as jpena|off | 18:33 | |
| *** michael-beaver has joined #zuul | 18:50 | |
| *** gmann is now known as gmann_afk | 19:00 | |
| *** bhavikdbavishi has quit IRC | 19:04 | |
| *** bhavikdbavishi has joined #zuul | 19:04 | |
| *** bhavikdbavishi has quit IRC | 19:11 | |
| openstackgerrit | James E. Blair proposed zuul/zuul-jobs master: WIP: openshift speculative containers https://review.opendev.org/696939 | 20:21 |
| *** gmann_afk is now known as gmann | 20:25 | |
| Shrews | If we could integrate this (https://aws.amazon.com/codeguru/) with zuul, we'd almost totally eliminate the need for humans in the sw dev process | 20:26 |
| mordred | Shrews: is that a service that will leave code review comments on my whitespace? | 20:29 |
| mordred | Shrews: oh - we'd have to rewrite zuul in Java | 20:30 |
| pabelanger | $0.75 per 100 lines of code scanned per month | 20:31 |
| pabelanger | sign me up | 20:31 |
| mordred | "For example, if you have a typical pull request with 500 lines of code, it would only cost $3.75 to run CodeGuru Reviewer on it." | 20:31 |
| pabelanger | would be intersting to test out | 20:32 |
| * fungi does napkin math to figure out how much they would charge to review openstack's change volume | 20:33 | |
| fungi | answer: too much | 20:33 |
| mordred | fungi: yah. I did similar math - and I didn't even get to the hard math :) | 20:33 |
| fungi | i couldn't find a napkin | 20:33 |
| mordred | mine is dirty | 20:33 |
| *** mhu has quit IRC | 20:36 | |
| Shrews | i wonder what they define a "line" as? does '{\n' cost $0.75? i could review that for half that price and get it correct most of the time | 20:38 |
| * Shrews will review mordred's whitespace issues for $0.30/per line... what a Cyber Tuesday bargain! | 20:40 | |
| mordred | Shrews: you say that now - just wait until my next batch of whitespace issues! | 20:40 |
| *** michael-beaver has quit IRC | 21:05 | |
| *** rfolco has quit IRC | 21:08 | |
| *** rfolco has joined #zuul | 21:09 | |
| *** smyers has joined #zuul | 21:25 | |
| *** rfolco has quit IRC | 21:26 | |
| openstackgerrit | Merged zuul/zuul-jobs master: build-docker-image: add option to install siblings https://review.opendev.org/696987 | 21:26 |
| *** Goneri has quit IRC | 21:33 | |
| openstackgerrit | James E. Blair proposed zuul/zuul-jobs master: WIP: openshift speculative containers https://review.opendev.org/696939 | 21:40 |
| *** gtema has joined #zuul | 21:42 | |
| *** gtema has quit IRC | 21:45 | |
| *** pcaruana has quit IRC | 22:06 | |
| openstackgerrit | James E. Blair proposed zuul/zuul-jobs master: use-buildset-registry: Vendor pytoml and remarshal https://review.opendev.org/695050 | 22:08 |
| openstackgerrit | James E. Blair proposed zuul/zuul-jobs master: Add build-container-image role https://review.opendev.org/695251 | 22:08 |
| openstackgerrit | James E. Blair proposed zuul/zuul-jobs master: use-buildset-registry: Add podman support https://review.opendev.org/695051 | 22:08 |
| openstackgerrit | James E. Blair proposed zuul/zuul-jobs master: use-buildset-registry: Vendor pytoml and remarshal https://review.opendev.org/695050 | 22:10 |
| openstackgerrit | James E. Blair proposed zuul/zuul-jobs master: Add build-container-image role https://review.opendev.org/695251 | 22:23 |
| openstackgerrit | James E. Blair proposed zuul/zuul-jobs master: use-buildset-registry: Add podman support https://review.opendev.org/695051 | 22:23 |
| openstackgerrit | James E. Blair proposed zuul/zuul-jobs master: use-buildset-registry: Add podman support https://review.opendev.org/695051 | 22:31 |
| openstackgerrit | James E. Blair proposed zuul/zuul-jobs master: WIP: openshift speculative containers https://review.opendev.org/696939 | 22:32 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!