Monday, 2019-12-02

*** jamesmcarthur has joined #zuul00:20
*** jamesmcarthur has quit IRC00:26
*** tosky has quit IRC00:26
*** jamesmcarthur has joined #zuul00:58
*** jamesmcarthur has quit IRC01:17
*** spsurya has joined #zuul01:28
*** bhavikdbavishi has joined #zuul02:39
*** bhavikdbavishi has quit IRC02:41
*** bhavikdbavishi has joined #zuul02:42
*** bhavikdbavishi1 has joined #zuul02:44
*** bhavikdbavishi has quit IRC02:46
*** bhavikdbavishi1 is now known as bhavikdbavishi02:46
*** bstinson has quit IRC03:15
*** bstinson has joined #zuul03:22
*** bstinson has quit IRC03:39
*** bstinson has joined #zuul03:51
*** jamesmcarthur has joined #zuul05:46
*** jamesmcarthur has quit IRC05:50
*** raukadah is now known as chkumar|rover05:54
*** pcaruana has joined #zuul06:32
*** chkumar|rover is now known as chkumar|ruck06:51
*** jcapitao has joined #zuul08:01
*** jangutter has joined #zuul08:09
*** sugaar has joined #zuul08:24
*** tosky has joined #zuul08:31
*** hashar has joined #zuul08:46
*** jpena|off is now known as jpena08:48
*** ttx has quit IRC08:50
*** ttx has joined #zuul08:50
*** themroc has joined #zuul08:56
*** bogdando has joined #zuul10:02
bogdandohi, could you please take a look https://review.opendev.org/#/c/696337/ ?10:02
bogdandoI cannot apply it to my local zuul via depends-on for some reason... And I need this to use custom pypi mirror for fedora 31 subnodes10:03
*** bogdando has left #zuul10:18
*** tosky_ has joined #zuul10:26
sugaarHi, somebody knows where nodepool takes the nodes from in the zuul/doc/source.admin/example demo??10:26
sugaarI am trying to configure it to use some nodes in a kubernetes cluster in minikube but It can't find the .kub/config file. So I was wondering how it does it by default10:27
*** tosky has quit IRC10:28
*** bogdando has joined #zuul10:34
bogdandois there a way to live test by local zuul itself a change in zuul-jubs? e.g. https://review.opendev.org/#/c/696337/10:35
bogdandoThis change looks never applied by zuul executor running my CI job10:35
bogdandoand there is no traces for 696337 in all logs10:35
*** tosky_ is now known as tosky10:38
ianwbogdando: because configure-mirrors is included from the base job, you need to test it via base-test; see  https://opendev.org/opendev/base-jobs/src/branch/master/zuul.d/jobs.yaml#L510:55
bogdandoianw: thanks!11:12
bogdandoianw: IIUC, I'll need to commit something to my local gerrit project copy of opendev/base-jobs for my local zuul?11:33
bogdandoand the step 4 Propose a change to a job to reparent it to base-test is not quite clear to me. If I have my job under test is foo -> multinode -> base, what shall I reparent?11:35
bogdandothat's multinode, right?11:36
bogdandolike http://paste.openstack.org/show/6bLOJNKOEUDcyMvSpeOi/ ?..11:39
openstackgerritBogdan Dobrelya (bogdando) proposed zuul/zuul-jobs master: DNM testing custom pypi mirrors (696337)  https://review.opendev.org/69686711:41
*** jcapitao is now known as jcapitao|lunch12:03
bogdandoianw: I'm sorry but do you have some example for that https://opendev.org/opendev/base-jobs/src/branch/master/zuul.d/jobs.yaml#L5 implemented as a DNM patch?12:05
bogdandoI'm lost in understanding hose steps12:05
bogdandothose*12:05
*** hashar has quit IRC12:07
*** rfolco has joined #zuul12:09
*** hashar has joined #zuul12:10
*** jpena is now known as jpena|lunch12:26
Shrewssugaar: the demo example uses the nodepool static driver (https://opendev.org/zuul/zuul/src/branch/master/doc/source/admin/examples/etc_nodepool/nodepool.yaml#L9) and uses a docker node as the target node (https://opendev.org/zuul/zuul/src/branch/master/doc/source/admin/examples/docker-compose.yaml#L89)12:50
Shrewssugaar: I think tristanC might be able to help you with kubernetes configuration if you're having problems with it12:51
*** rlandy has joined #zuul12:57
pabelangerianw: bogdando: base-test will not work, unless it is also untrusted. Otherwise, you need to merge change for trusted projects13:12
*** jcapitao|lunch is now known as jcapitao13:14
*** jamesmcarthur has joined #zuul13:15
sugaarShrews nice, thanks!13:22
fungihow we do it is to merge the alteration we want to prove to the base-test job, then optionally if the only jobs which expose the problem in question are parented through another trusted repo job to base we have to merge a duplicate of that job with a different name parented to base-test, and then finally propose a change to the affected untrusted job altering its parentage so that it chains up to base-test13:23
sugaarShrews I try to workout with that info now by myself, and if I don't manage I will contact him.13:23
*** jpena|lunch is now known as jpena13:24
pabelangerfungi: issue here, is we cannot merge the role change, as it will hit production live. base / base-test playbooks don't really work if same role is in both. We need to rename role, then merge13:25
AJaegerbogdando: ^13:26
fungipabelanger: you need a similar role with a different name for base-test in that case13:27
fungior, yeah, what you said13:27
*** jamesmcarthur has quit IRC13:35
bogdandopabelanger: I can add opendev/base-jobs as untrusted for my local zuul, just not sure of the next steps :)13:41
bogdandopabelanger: I think I got it. Then I will be able to use depends-on: https://review.opendev.org/696867 for example, in my local gerrit w/o merging it upstream or whatever13:49
bogdandoso IUUC that patch only needs a copy of the base-test job ported in13:50
bogdandoincluding roles and playbooks and defaults perhaps?..13:50
pabelangerbogdando: is this rdoproject?13:51
bogdandono13:51
pabelangerkk13:51
pabelangeryou should look to how we setup rdocloud, I believe configure-mirrors is located in untrusted project by default13:52
bogdandojust doing some tripleo standalone (based on multinode zuul parent) but for fedora 3113:52
bogdandothanks I'll check13:52
bogdandopabelanger: yes, I have zuul/zuul-jobs, which contains that configure-mirrors  in untrusted13:53
pabelangerk, that should work now13:54
openstackgerritBogdan Dobrelya (bogdando) proposed zuul/zuul-jobs master: DNM testing custom pypi mirrors (696337)  https://review.opendev.org/69686713:58
bogdandomhm, that does Base jobs must be defined in config projects mean?14:01
openstackgerritBogdan Dobrelya (bogdando) proposed zuul/zuul-jobs master: DNM testing custom pypi mirrors (696337)  https://review.opendev.org/69686714:02
tristanCsugaar: you have to setup the ~nodepool/.kube/config similarly to the ~nodepool/.config/openstack/clouds.yaml, that is with the information and token of your provider14:10
*** rlandy is now known as rlandy|mtg14:14
*** jamesmcarthur has joined #zuul14:17
openstackgerritBogdan Dobrelya (bogdando) proposed zuul/zuul-jobs master: DNM testing custom pypi mirrors (696337)  https://review.opendev.org/69686714:23
*** Goneri has joined #zuul14:41
bogdandopabelanger, ianw, fungi: sorry, I have to give up on https://review.opendev.org/696867. I can't get how to test https://review.opendev.org/696337 manually14:41
bogdandoIf I had some real example from gerrit to compare the required steps...14:44
*** jamesmcarthur has quit IRC14:49
fungibogdando: thanks, i missed some of the background on this so was speaking in the abstract. since there's a real-world example i'll take a look and see if i can provide a more concrete plan to test it14:52
bogdandofungi: thanks a lot14:52
*** bhavikdbavishi has quit IRC14:53
clarkbfwiw I think the mirror config redo plan js the proper way to address this14:53
clarkbthen each mirrored item can be set independe tly14:53
mordred++15:00
*** jamesmcarthur has joined #zuul15:03
sugaartristanC I am trying to do it first using minikube rather than the could provider, but do you reckon that it would be easier to go straight away to the could provider?15:10
*** rlandy|mtg is now known as rlandy15:18
fungibogdando: while i agree with clarkb and mordred that the in-progress overhaul of mirror configuration would be a better place to spend time, the way to test that change would be to first propose a copy to zuul/zuul-jobs in roles/configure-mirrors-test and get that merged, then propose a change to opendev/base-jobs where you alter playbooks/base-test/pre.yaml to use configure-mirrors-test instead of15:24
fungiconfigure-mirrors and get that merged too, then propose a do-not-merge change to zuul/zuul-jobs which sets the parent on the unittests job to base-test and then propose a trivial do-not-merge change to some other project like zuul/zuul with a depends-on to your other do-not-merge change in zuul/zuul-jobs which gets a unit test job to run, and see if it works and exercises the pypi mirror normally but through15:24
fungithe new variable15:24
fungiand then also propose reverts of the merged changes for zuul/zuul-jobs and opendev/base-jobs15:25
*** sgw has joined #zuul15:26
tristanCsugaar: it's up to you. I guess that if you don't need cloud virtual machines, then k8s would be easier15:27
bogdandofungi: thanks15:27
bogdando> that the in-progress overhaul of mirror configuration would be a better place to spend time15:27
bogdandowhere is all that code located?15:27
fungibogdando: it's sketched out in zuul-jobs documentation so far: https://zuul-ci.org/docs/zuul-jobs/mirror.html15:31
openstackgerritDavid Moreau Simard proposed zuul/zuul master: DNM: Test zuul-stream-functional with ara 1.3  https://review.opendev.org/69462215:35
fungibogdando: in short, the configure-mirrors role was very opendev-specific and just sort of got copied over. trying to plumb more and more variables through it in an uncoordinated fashion to support uses outside opendev is going to lead to an unmaintainable mess (we might argue it already is) so the idea is to completely replace it with a new role structured for general use and flexibility15:36
bogdandothanks I'll try if zuul_site_mirror_info works for me15:38
*** tflink_ is now known as tflink15:39
openstackgerritMerged zuul/nodepool master: Delete images by id  https://review.opendev.org/69367215:41
corvusbogdando: it's still early enough that if it doesn't, we can change the design :).   if it does look good, and you wanted to make a new role (can probably copy the old role for most of it) and update it to match that design, that would be welcome.  i started on something similar for the os mirrors here, but haven't finished it yet: https://review.opendev.org/67757815:59
*** jcapitao is now known as jcapitao|afk16:14
*** chkumar|ruck is now known as raukadah16:17
sugaartristanC where are stored ~nodepool/.kube/config ~nodepool/.config/openstack/clouds.yaml? are they in the container that launches nodepool? because I can't see them, or do I need to add a new user to my laptop for nodepool?16:22
tristanCsugaar: you need to provides those files16:24
*** jamesmcarthur has quit IRC16:24
Shrewssugaar: you might want to check how we start minikube in our zuul-jobs:install-kubernetes role: https://opendev.org/zuul/zuul-jobs/src/branch/master/roles/install-kubernetes/tasks/minikube.yaml16:25
*** jcapitao|afk is now known as jcapitao16:25
*** jamesmcarthur has joined #zuul16:26
Shrewsi *think* minikube populates the .kube/config file for you, iirc16:26
Shrewsyou just have to tell it where it is16:26
corvustristanC, clarkb: https://review.opendev.org/695050 and 2 children are ready when you have a minute (podman speculative container builds)16:35
clarkbk. If I make it home before dark I'll take a look. I'm about to travaerse mountain passes that have received unusual amounts of early snow16:38
corvusclarkb: oh, er, it's not really important enough to risk a donner party kind of situation :)16:40
sugaarShrews tristanC thanks for the help, sorry it is being a bit confusing for me. I am very new with kubernetes, cloud providers and all this ecosystem and I am trying to learn about nodepool and zuul in the same time16:41
openstackgerritBogdan Dobrelya (bogdando) proposed zuul/zuul-jobs master: DNM testing custom pypi mirrors (696337)  https://review.opendev.org/69691816:42
corvussugaar: :) some of us are still learning about all of those things16:42
openstackgerritBogdan Dobrelya (bogdando) proposed zuul/zuul-jobs master: DNM testing custom pypi mirrors (696337)  https://review.opendev.org/69691816:43
Shrewssugaar: no problem at all. but be aware that afaik, none of us are experts with minikube (i created that role that uses it with a lot of trial and error, and I'm pretty ignorant of kubernetes setup in general). but happy to help with nodepool/zuul stuff as much as possible16:43
*** hashar has quit IRC16:51
openstackgerritBogdan Dobrelya (bogdando) proposed zuul/zuul-jobs master: DNM testing custom pypi mirrors (696337) #2  https://review.opendev.org/69692917:01
bogdandofungi: I tried https://review.opendev.org/#/c/696929/17:03
bogdandoand all the steps (but nothing merged yet ofc), I think it will be failing anyway as there is no base-test job defined in zuul/zuuljobs17:04
bogdandothat patch corresponds to the step "then propose a do-not-merge change to zuul/zuul-jobs which sets the parent on the unittests job"17:05
bogdandoI have also tried pointing out https://review.opendev.org/696918 dependency for my local gerrit, which has17:08
bogdando        untrusted-projects:17:08
bogdando            projects:17:08
bogdando            - zuul/zuul-jobs17:08
bogdando            - zuul/zuul-base-jobs17:08
bogdando            - opendev/base-jobs17:08
bogdando(just to repeat the steps if it was running upstream but w/o the need to merge anything)17:08
bogdandoand still there is no attempts to pull  that 696918 in :(17:08
bogdandopabelanger: ^^17:08
pabelangerbogdando: where do you call configure-mirrors role in your local install17:08
pabelangerwhich job?17:08
bogdandocorrection , the dependency set for my zuul was for https://review.opendev.org/#/c/696921/17:09
bogdandopabelanger: there is a job on top of multinode I run for my zuul, it has depends-on: https://review.opendev.org/69692117:09
bogdandothat one depends on https://review.opendev.org/69691817:10
pabelangerbogdando: but, we need to know where the job is defined, in your local zuul17:10
bogdandoall that follows the steps fungi described earlier17:10
bogdandojust that nothing has to be merged for my local zuul cuz untrusted projects ...17:10
pabelangerit sounds like, you call configure-mirrors from trustred playbook17:10
pabelangerlocally17:10
tristanCcorvus: thanks, i'll have a look shortly17:12
bogdandopabelanger: zuul/zuul-jobs is in untrusted-projects, and it contains configure-mirrors17:13
*** sshnaidm is now known as sshnaidm|afk17:13
bogdandoopendev/base-jobs is also untrusted so I presumed I can omit the "merge" steps17:14
pabelangerbogdando: so, I think we are missing each other. What I am trying to understand, in your playbooks, what job calls configure-mirrors?17:14
pabelangerfor zuul.a.c, that is our 'base' job https://github.com/ansible/ansible-zuul-jobs/blob/master/playbooks/base/pre.yaml which lives in an untrusted project17:15
bogdandothe base I guess, my_job -> multinode -> base17:15
pabelangerright17:15
pabelangerand is base trusted?17:15
pabelangerI am guess yes17:15
pabelangerwhich means, you cannot do speclative testing17:15
corvus(it's the playbook's repo that determines whether speculative merging happens)17:16
bogdandopabelanger: sorry what do you mean if base is trusted?17:16
pabelangerbogdando: if you look at rdoproject, we creaated the idea of base-minimal (config-project) and base (untrusted)17:16
bogdandobase is not a repo?..17:16
pabelangerbogdando: base in your case, is likely a config-project: https://zuul-ci.org/docs/zuul/user/config.html#security-contexts17:17
bogdandopabelanger: here is untrusted et al https://pastebin.com/YDSv3G2917:18
pabelangerbogdando: can you please see which job playbook as the role 'configure-mirrors'17:19
pabelangerBased on what you are discribing, I believe that will live in zuul-config17:20
pabelangerdescribing*17:20
*** jcapitao has quit IRC17:20
bogdandopabelanger: my local gerrit path http://localhost:9000/t/tripleo-ci-reproducer/job/base do you mean?.. https://pastebin.com/yyFWCsJn17:22
bogdandook I got it, so it's trusted17:23
pabelangerright17:23
pabelangerso, none of this will work, until your change is merged17:23
bogdandosigh :)17:23
pabelangeror you move configure-mirror later in the job run, as untrusted job17:23
pabelangerthat is what we do in zuul.a.c and pretty sure how we also setup rdoproject17:24
bogdandomhm, I would love to try, not sure how to do that17:24
pabelangerhttps://github.com/ansible/ansible-zuul-jobs/blob/master/zuul.d/jobs.yaml#L217:24
pabelangeris an example of untrusted 'base' that parents to trusted 'base-minimal'17:24
bogdandopabelanger: how gets this ansible/ansible-zuul-jobs catched up by zuul?17:26
bogdandovia config-projects17:26
bogdando?17:26
pabelangerbogdando: https://github.com/ansible/project-config/blob/master/zuul/tenants.yaml#L517:26
pabelangerthat is our tenant config17:26
pabelangeransible-zuul-jobs, is just untrusted17:27
bogdandopabelanger: thanks!!17:27
pabelangernp17:27
*** bogdando has quit IRC17:31
*** Goneri has quit IRC17:40
*** michael-beaver has joined #zuul17:41
openstackgerritMerged zuul/zuul master: Ensure correct re-enqueue on changing gate queue  https://review.opendev.org/69657817:43
*** jangutter has quit IRC17:43
*** Goneri has joined #zuul17:54
*** jpena is now known as jpena|off17:59
openstackgerritJames E. Blair proposed zuul/zuul-jobs master: WIP: openshift speculative containers  https://review.opendev.org/69693918:01
*** spsurya has quit IRC18:25
*** igordc has joined #zuul18:45
*** decimuscorvinus has quit IRC19:06
*** decimuscorvinus has joined #zuul19:07
*** sgw has quit IRC19:09
*** jamesmcarthur has quit IRC19:19
*** hashar has joined #zuul19:27
*** jangutter has joined #zuul19:44
*** jangutter has quit IRC19:48
*** fdegir4 has joined #zuul19:54
*** donnyd_ has joined #zuul19:54
*** persia_ has joined #zuul19:56
*** mugsie_ has joined #zuul19:56
*** pcaruana has quit IRC20:02
*** fdegir has quit IRC20:02
*** reiterative has quit IRC20:02
*** Guest72500 has quit IRC20:02
*** persia has quit IRC20:02
*** mugsie has quit IRC20:02
*** johanssone has quit IRC20:02
*** donnyd has quit IRC20:02
*** clarkb has quit IRC20:02
*** donnyd_ is now known as donnyd20:02
*** johanssone has joined #zuul20:03
*** clarkb has joined #zuul20:03
*** pcaruana has joined #zuul20:05
*** michael-beaver has quit IRC20:10
openstackgerritJames E. Blair proposed zuul/zuul-jobs master: WIP: openshift speculative containers  https://review.opendev.org/69693920:24
*** hashar has quit IRC20:33
ianwcorvus: i really need some help getting speculative testing working for these nodepool container jobs; do you have a minute for me to bring you up to speed?20:34
corvusianw: yep20:35
ianwcorvus: thanks; the problem for me at the moment is testing https://review.opendev.org/#/c/696486/20:35
ianwit just fails with "This change depends on a change with an invalid configuration."20:36
corvusianw: i'm not sure i'm convinced that we want a job running in the nodepool repo to depend on a job defined in system-config20:36
ianwwell, where do you think the job to build the container image that will run our production nodepool containers should be?20:38
ianw(this may be more an infra discussion, if we're not discussing the configuration issues)20:39
corvusi think opendev should use the upstream zuul and nodepool container images when it switches to running in containers (i understand that's the thing that needs to change since they don't include dib).  but i don't think they should depend on opendev operations.20:40
corvuswe want zuul to be a standalone project -- it's not just something that opendev runs20:41
corvuswe have contributors who aren't interested in contributing to opendev, and we'd like to retain them and attract more of them :)20:41
corvusmoreover, if opendev does make its own nodepool images (as it does for gerrit) because of local ops issues, it should have the freedom to do that without impacting all zuul users20:42
ianwi agree, which is why i felt building the images from system-config means that they are custom images for opendev, and can make choices that may not be appropriate/useful for the generic images (such as shipping our PPA with vhd support, etc)20:44
ianwhowever, i would also like those images tested20:44
corvuscan they be tested in system-config?20:45
pabelangerre: vhd support, I think we should either pull that into zuul or finally merge that code into debian directly. As, for rackspace support for nodepool, we need to install opendev repos20:46
fungipabelanger: the "vhd support" being patched xen-specific image generation? if so, why not in diskimage-builder?20:47
ianwpabelanger: well, i basically offered to work with RAX on getting support into qemu-img for their systems (https://opendev.org/openstack/diskimage-builder/src/branch/master/doc/source/developer/vhd_creation.rst) and in so many words they were not interested20:47
fungiis that "support for their sustems" or "support for xen"?20:48
fungis/sustems/systems/20:48
pabelangerI cannot remember all the details, will read ianw link. But qemu-utils vhd support in linux doesn't work out of box20:48
pabelangerso, in our case we are looking to use rackspace in zuul.a.c, but need to pull in opendev ppa20:49
ianwpabelanger: yeah, that link has the details :)  i spent quite a while, and broke the gate investigating it all :)20:49
*** mattw4 has joined #zuul21:13
corvuspabelanger: i'd be curious to hear the reasons you're looking into a packer.io nodepool builder, if you care to share21:20
pabelangermostly, there is a few network appliances that have packer.io as builder. So it was an easy win to use them.  However, really the workflow we need to support is boot existing qcow2 in qemu, run ansible-playbook to provision things, then compress and save.  A quick look around the current builders, seems to show packer.io as leader in this (native ansible hooks).21:22
pabelangerwe do that manually today, then manually upload into cloud. Which, isn't scaling for us21:23
corvuspabelanger: ah, makes sense, thanks21:23
pabelangerI've thought of doing it as a zuul-job, but then need to figure out upload part into nodepool-builder server, then trigger upload to cloud some how21:25
pabelangerwith new flag to point to disk-image-create command, I'm going to POC with bash script21:25
pabelangerthen see how to move more into nodepool correctly21:25
corvuspabelanger: if you want to explore the zuul job route, you can upload it to the cloud and configure nodepool-launcher to use a 'cloud-image'21:30
pabelangercorvus: yup, so far we haven't done that route to avoid adding nodepool credentials to zuul job but agree, on the short list as solutions.21:31
tristanCusing a zuul-job to build and upload cloud-image would be great, but how could nodepool detect and use the new image then?21:40
Shrewsiirc, if you're using cloud images, you just reference the image by name21:42
pabelangeryou'd have to first delete the image, then upload new21:42
corvusdo names have to be unique?21:47
tristanCperhaps nodepool could list image and pick the most recent one when if the name matches it's config?21:48
corvusif so, maybe we could add regex support so you can append a timestamp to the image name and then use the "latest" cloud image21:48
corvustristanC: yeah, and that if names don't have to be unique21:48
Shrewscorvus: you can also specify image id21:49
pabelangerno, names don't need to be unique IIRC, but nodepool will get confused if n+1 images with same name21:50
pabelangerwell, openstacksdk21:50
tristanCwell, not sure how the user would get the first image, but that process of using zuul-job to update nodepool image sounds like a better user experience, e.g. build logs available in zuul builds page, image update frequency with periodic pipeline, ...21:50
corvusif the image build can run on a *real* cloud image, that solves the bootstrap problem21:51
pabelangeryes, that is hold back for us too (getting image into public)21:51
corvusif the cloud supports renaming images, you could have a test/promote system21:52
corvus(even if it doesn't, you could still do that, it would just be slower)21:52
corvus(and have an outage window)21:52
pabelangerI do like the idea of test/promote of images :)21:55
pabelangertime to run, #dadops21:56
*** mugsie_ is now known as mugsie21:59
*** pcaruana has quit IRC22:11
*** sgw has joined #zuul22:30
*** sgw has quit IRC22:35
*** tosky has quit IRC23:41
openstackgerritIan Wienand proposed zuul/zuul-jobs master: [wip] copy listed siblings to docker build  https://review.opendev.org/69698723:53

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!