*** ysandeep|out is now known as ysandeep | 00:34 | |
openstackgerrit | Tristan Cacqueray proposed zuul/zuul-operator master: Add initial withCertManager input toggle https://review.opendev.org/718840 | 00:39 |
---|---|---|
tristanC | corvus: mordred: so i think it should be simple to support both cert-manager and internal openssl/zk-ca usage by the operator, ^ is a demo of how such withCertManager toggle could be implemented | 00:42 |
*** zxiiro has quit IRC | 00:56 | |
*** swest has quit IRC | 01:23 | |
openstackgerrit | Merged zuul/zuul master: kubernetes-operator: add job volume to the spec https://review.opendev.org/706639 | 01:24 |
openstackgerrit | Merged zuul/zuul master: kubernetes-operator: change attribute to camelCase https://review.opendev.org/707190 | 01:24 |
*** swest has joined #zuul | 01:39 | |
*** Goneri has quit IRC | 01:47 | |
openstackgerrit | Merged zuul/zuul-operator master: Update attributes to camelCase https://review.opendev.org/707193 | 02:31 |
*** ysandeep is now known as ysandeep|off | 03:28 | |
*** y2kenny has quit IRC | 03:38 | |
*** sgw has quit IRC | 03:38 | |
*** bhavikdbavishi has joined #zuul | 03:55 | |
*** bhavikdbavishi1 has joined #zuul | 03:58 | |
*** bhavikdbavishi has quit IRC | 04:00 | |
*** bhavikdbavishi1 is now known as bhavikdbavishi | 04:00 | |
*** sgw has joined #zuul | 04:06 | |
*** cdearborn has quit IRC | 04:17 | |
*** evrardjp has quit IRC | 04:37 | |
*** evrardjp has joined #zuul | 04:37 | |
*** bhavikdbavishi has quit IRC | 04:38 | |
*** bhavikdbavishi has joined #zuul | 04:49 | |
openstackgerrit | Ian Wienand proposed zuul/zuul-jobs master: ensure-pip: export ensure_pip_virtualenv_command https://review.opendev.org/718224 | 04:53 |
openstackgerrit | Ian Wienand proposed zuul/zuul-jobs master: fetch-zuul-cloner: use ensure-pip https://review.opendev.org/717882 | 04:53 |
openstackgerrit | Ian Wienand proposed zuul/zuul-jobs master: fetch-subunit-output test: use ensure-pip https://review.opendev.org/718225 | 04:53 |
openstackgerrit | Ian Wienand proposed zuul/zuul-jobs master: ensure-tox: use ensure-pip role https://review.opendev.org/717663 | 04:53 |
openstackgerrit | Ian Wienand proposed zuul/zuul-jobs master: Update Fedora to 31 https://review.opendev.org/717657 | 04:53 |
*** gtema has joined #zuul | 06:08 | |
*** bhavikdbavishi has quit IRC | 06:11 | |
*** bhavikdbavishi has joined #zuul | 06:11 | |
*** gtema has quit IRC | 06:13 | |
webknjaz | @tobiash: Mind verifying that https://github.com/cherrypy/cheroot/pull/277 fixes your problem? | 06:17 |
*** dpawlik has joined #zuul | 06:20 | |
*** bhavikdbavishi has quit IRC | 06:24 | |
*** gtema has joined #zuul | 06:32 | |
*** gtema has quit IRC | 06:33 | |
*** bhavikdbavishi has joined #zuul | 06:38 | |
*** bhavikdbavishi has quit IRC | 06:54 | |
*** jcapitao has joined #zuul | 06:59 | |
*** gtema has joined #zuul | 07:02 | |
*** gtema has quit IRC | 07:03 | |
*** bhavikdbavishi has joined #zuul | 07:10 | |
*** bhavikdbavishi has quit IRC | 07:22 | |
*** tosky has joined #zuul | 07:32 | |
*** rpittau|afk is now known as rpittau | 07:36 | |
*** hashar has joined #zuul | 07:52 | |
*** bhavikdbavishi has joined #zuul | 08:47 | |
*** bhavikdbavishi has quit IRC | 09:08 | |
*** bhavikdbavishi has joined #zuul | 09:08 | |
*** bhavikdbavishi1 has joined #zuul | 09:19 | |
*** bhavikdbavishi has quit IRC | 09:21 | |
*** bhavikdbavishi1 is now known as bhavikdbavishi | 09:21 | |
*** sassyn has joined #zuul | 10:03 | |
sassyn | hi all | 10:03 |
sassyn | I'm a new baby when it come to zuul | 10:04 |
sassyn | I running Gerrit with Jenkins interrogation | 10:04 |
sassyn | I want to move to zuul, so zuul will trigger jobs from gerrit, and will run jobs in jenkins | 10:04 |
sassyn | I have install zuul + german + zookeeper, and I still have question if I really need the nodepool | 10:05 |
sassyn | I don't want to auto create slave/workers since I allready have jenkins slaves. | 10:06 |
sassyn | So the ansible part is not something I need in my setup | 10:06 |
sassyn | my question how can I run a jobs that exists in the the jenkins ? | 10:06 |
sassyn | is this supported? | 10:07 |
sassyn | I see it here http://www.joinfu.com/2014/01/understanding-the-openstack-ci-system/ and some movies in youtube - but I can't find the offical documenation | 10:07 |
sassyn | Thank you | 10:07 |
*** persia_ is now known as persia | 10:07 | |
*** armstrongs has joined #zuul | 10:10 | |
*** rpittau is now known as rpittau|bbl | 10:19 | |
*** armstrongs has quit IRC | 10:20 | |
jkt | sassyn: you're following a six-year-old article. Since that tim, Zuul had moved on; now there's no place for Jenkins in there | 10:41 |
jkt | sassyn: a good place to start reading is https://zuul-ci.org/ | 10:42 |
* jkt afk | 10:42 | |
*** jcapitao is now known as jcapitao_lunch | 10:58 | |
*** bhavikdbavishi has quit IRC | 11:10 | |
*** bhavikdbavishi has joined #zuul | 11:20 | |
*** rpittau|bbl is now known as rpittau | 11:58 | |
*** jcapitao_lunch is now known as jcapitao | 12:18 | |
fungi | sassyn: also here's an article which explains a bit about how zuul evolved, in case you're interested in the reasons for those decisions: https://opensource.com/article/20/2/zuul | 12:55 |
sassyn | fugni | 12:56 |
fungi | but basically we started to work on removing jenkins from the system architecture in early 2016 | 12:56 |
sassyn | thank you | 12:56 |
openstackgerrit | Tristan Cacqueray proposed zuul/zuul-operator master: Increase scheduler wait to timeout https://review.opendev.org/718162 | 12:57 |
openstackgerrit | Tristan Cacqueray proposed zuul/zuul-operator master: Add schema validation error message https://review.opendev.org/718999 | 12:57 |
sassyn | I figure it out already.. https://opensource.com/article/20/2/zuul point it out this in the comment below | 12:57 |
sassyn | my problem is that I have 400 jsalves | 12:57 |
sassyn | and 200+ jobs in jenkins | 12:57 |
sassyn | change it will be hard | 12:57 |
fungi | in opendev we had over 10,000 jenkins jobs we converted to ansible, so it's doable but yes i too understand how hard that can be | 12:58 |
sassyn | Zuul seems hard to understand | 12:59 |
sassyn | I still not getting to full picture | 12:59 |
fungi | obviously we didn't do it by hand, we were already templating the jobs with a tool we wrote called "jenkins job builder" and so wrote some scripts to translate much of them | 12:59 |
sassyn | most of our jobs use Jenkinsfile | 13:00 |
sassyn | and pipelines | 13:00 |
sassyn | all are in the gerrit repo | 13:00 |
fungi | sassyn: there's an architecture diagram at https://zuul-ci.org/docs/zuul/discussion/components.html with explanations of the individual components | 13:00 |
fungi | in case you haven't found that yet | 13:01 |
sassyn | yes, read it | 13:01 |
sassyn | fungi - thank u! | 13:01 |
sassyn | Tired of sitting home with this corona | 13:01 |
fungi | also we're trying to put together a comprehensive set of definitions for the terminology we use around zuul here: https://zuul-ci.org/docs/zuul/reference/glossary.html | 13:02 |
fungi | if you have specific questions which don't seem to be addressed by the documentation, please do ask them | 13:02 |
fungi | sassyn: also since you're already using gerrit, you may be interested in the ongoing work to use zuul for development of gerrit itself: https://groups.google.com/forum/#!msg/repo-discuss/uJFyPnFlbio/ZBpmTXJtFwAJ https://groups.google.com/forum/#!topic/repo-discuss/qRYRWb42CuQ | 13:08 |
sassyn | perfect! | 13:09 |
sassyn | are u are one of the developers? | 13:09 |
fungi | i'm one of the zuul maintainers, yes (jeremy stanley). we're listed here: https://zuul-ci.org/docs/zuul/reference/governance.html#zuul-maintainers | 13:12 |
fungi | but there are plenty of other people who are doing development work on zuul in addition to us | 13:12 |
fungi | (i haven't really written that much of it by comparison) | 13:12 |
*** cdearborn has joined #zuul | 13:23 | |
*** bhavikdbavishi has quit IRC | 13:39 | |
mnaser | corvus: carrying on the conversation regarding mirroring here, as this is a "zuul"-ism. do you think it would make sense for the upload-git-mirror job to have an empty nodeset in zuul/zuul-jobs or ? | 14:15 |
corvus | mnaser: that would mean that it would only work in a trusted repo by default; i hesitate to adopt that pattern for jobs in zuul-jobs -- in the current situation, any developer can add that to their own repo easily | 14:19 |
openstackgerrit | Tristan Cacqueray proposed zuul/zuul-operator master: Add a zuul-ensure-database-passwords role https://review.opendev.org/717880 | 14:19 |
mnaser | corvus: ok, so i think someting i may have personally learned yesterday is that trusted repos have the power of executing things inside arbitrarily in the executor when normal untrusted ones dont | 14:20 |
corvus | mnaser: yep | 14:21 |
mnaser | corvus: but in this case, i'm trying to think how i could make that job nodeless if the playbooks in the parent job don't point at 'hosts: localhost' | 14:21 |
mnaser | because ansible usually doesnt put localhost inside all | 14:21 |
corvus | mnaser: yeah, don't inherit from that job, just use the role | 14:21 |
mnaser | okay, that clears it up | 14:22 |
corvus | mnaser: (this is one of the reasons we try not to put logic in playbooks in zuul-jobs, so that if the answer to a problem is "don't use the job" then it's no big deal, all the work is in the roles) | 14:22 |
mnaser | i will hack on something and push it up soon | 14:22 |
mnaser | ++ makes sense | 14:22 |
mnaser | zuul-maint: tobiash notes a pretty good point wrt 'ensure-repositories' naming at https://review.opendev.org/#/c/717507/ -- before i go ahead and update the whole stack, i'd appreciate other comments so we can come to a consensus on a ood name for the role | 14:24 |
tristanC | mnaser: wfm | 14:25 |
corvus | that name looks good -- can we talk about tests? | 14:26 |
corvus | i think we keep going back and forth about whether new roles should have tests | 14:26 |
corvus | maybe we should go ahead and decide that they should? | 14:26 |
mnaser | sure. i struggled to come up with some reasonable set of tests personally, because anything i 'imagined' ended up testing the actual ansible module | 14:26 |
corvus | mnaser: that's probably fine :) | 14:27 |
corvus | mnaser: i think these are kind of like python unit tests; the main value is often in just running the code to make sure it doesn't have a syntax errer | 14:28 |
mnaser | so run the playbook and then check /etc/yum.repos.d/foo.repo, etc? | 14:28 |
corvus | mnaser: i'd be satisfied at "run the role" and stop there, but certainly also adding "check..." wouldn't hurt. :) | 14:29 |
mnaser | that seems fair, also, i was thinking that any role that uses it should be included in the 'files' section for that job | 14:29 |
mnaser | but worried that might get out of hand very quickly | 14:29 |
corvus | mnaser: yeah, that would be an option, but i think with dedicated test we could avoid that | 14:30 |
mnaser | ok, fair, i'll try to carve sometime to do the rename of the role and a basic test (i'll probably get sucked into adding zypper support too probably to support all platforms) | 14:30 |
corvus | the "test-playbooks/simple-role-test.yaml" playbook is good for a "just run the role" test; if you want to do a check, then it'll need a dedicated playbook | 14:30 |
mnaser | i actually think for something like this, it's good to do a dedicated playbook because breaking this role would break a lot of things | 14:31 |
corvus | (the idea of that was to at least make "just run the role" tests easy to do) | 14:31 |
corvus | mnaser: okay no argument there. you can just file the simple-role-test playbook away for future use :) | 14:31 |
mnaser | (if you look at the rest of the stack i pretty much removed every reference and usage of apt_key/apt_repo/rpm_key/yum_repo/etc) | 14:31 |
mnaser | + | 14:31 |
mnaser | ++ | 14:31 |
corvus | yeah, i dig the idea :) | 14:31 |
mnaser | ok cool, i'll try to do the opendev/openstack stuff now as i don't think it's a massive effort for now | 14:32 |
corvus | sassyn: our general approach for migrating from jenkins to zuul was to reduce the use of jenkins plugins (that was easy -- we found that lots of them broke jenkins anyway so we had been moving away from them for years) and turn our jobs into shell scripts. then it's easy to tell ansible to run a shell script. zuul even has a job for that already: | 14:35 |
corvus | https://zuul-ci.org/docs/zuul-jobs/general-jobs.html#job-run-test-command | 14:35 |
openstackgerrit | Tristan Cacqueray proposed zuul/zuul-operator master: Increase scheduler wait to timeout https://review.opendev.org/718162 | 14:55 |
openstackgerrit | Tristan Cacqueray proposed zuul/zuul-operator master: Add schema validation error message https://review.opendev.org/718999 | 14:55 |
openstackgerrit | Tristan Cacqueray proposed zuul/zuul-operator master: Add a zuul-ensure-database-passwords role https://review.opendev.org/717880 | 14:55 |
*** rpittau is now known as rpittau|afk | 14:57 | |
*** armstrongs has joined #zuul | 15:37 | |
*** Goneri has joined #zuul | 15:40 | |
openstackgerrit | Tristan Cacqueray proposed zuul/zuul-operator master: Add a zuul-ensure-database-passwords role https://review.opendev.org/717880 | 15:42 |
*** armstrongs has quit IRC | 15:45 | |
*** bhavikdbavishi has joined #zuul | 15:53 | |
*** bhavikdbavishi1 has joined #zuul | 15:58 | |
*** jcapitao has quit IRC | 16:00 | |
*** bhavikdbavishi has quit IRC | 16:00 | |
*** bhavikdbavishi1 is now known as bhavikdbavishi | 16:00 | |
-openstackstatus- NOTICE: review.opendev.org is being restarted for scheduled maintenance; see http://lists.opendev.org/pipermail/service-announce/2020-April/000003.html | 16:07 | |
*** bhavikdbavishi has quit IRC | 16:23 | |
*** evrardjp has quit IRC | 16:37 | |
*** evrardjp has joined #zuul | 16:37 | |
*** dpawlik has quit IRC | 16:53 | |
-openstackstatus- NOTICE: etherpad.openstack.org will be offline for about 30 minutes while it is migrated to a new server with a new hostname; see http://lists.opendev.org/pipermail/service-announce/2020-April/000003.html | 17:04 | |
openstackgerrit | Tristan Cacqueray proposed zuul/zuul-operator master: Add zuul-registry deployment https://review.opendev.org/710650 | 17:06 |
openstackgerrit | Andreas Jaeger proposed zuul/zuul-jobs master: Fix check_jobs_documented linter https://review.opendev.org/719054 | 17:09 |
*** hashar has quit IRC | 17:20 | |
-openstackstatus- NOTICE: The etherpad migration is still in progress; revised estimated time of completion 18:30 UTC | 17:54 | |
*** mugsie has quit IRC | 17:59 | |
*** EmilienM has quit IRC | 17:59 | |
*** Open10K8S has quit IRC | 17:59 | |
*** mugsie has joined #zuul | 18:03 | |
*** EmilienM has joined #zuul | 18:03 | |
*** Open10K8S has joined #zuul | 18:03 | |
*** EmilienM has quit IRC | 18:12 | |
*** EmilienM has joined #zuul | 18:13 | |
*** jamesmcarthur has joined #zuul | 18:54 | |
*** jamesmcarthur has quit IRC | 19:30 | |
*** jamesmcarthur has joined #zuul | 19:35 | |
*** sgw has quit IRC | 19:36 | |
*** jamesmcarthur has quit IRC | 19:40 | |
*** jamesmcarthur has joined #zuul | 19:45 | |
*** sgw has joined #zuul | 19:54 | |
openstackgerrit | Tristan Cacqueray proposed zuul/nodepool master: config: add missing voluptuous required attributes https://review.opendev.org/719094 | 19:56 |
*** nhicher has quit IRC | 20:02 | |
*** nhicher has joined #zuul | 20:02 | |
-openstackstatus- NOTICE: Due to a database migration error, etherpad.opendev.org is offline until further notice. | 20:10 | |
openstackgerrit | Tristan Cacqueray proposed zuul/zuul-operator master: Add zuul-preview service https://review.opendev.org/718419 | 20:34 |
openstackgerrit | Tristan Cacqueray proposed zuul/zuul-operator master: Add zuul-registry deployment https://review.opendev.org/710650 | 20:38 |
*** sassyn has quit IRC | 20:38 | |
*** sshnaidm|off has quit IRC | 20:46 | |
*** jamesmcarthur_ has joined #zuul | 20:57 | |
*** jamesmca_ has joined #zuul | 20:58 | |
*** jamesmcarthur has quit IRC | 21:00 | |
*** jamesmcarthur_ has quit IRC | 21:02 | |
openstackgerrit | Tristan Cacqueray proposed zuul/zuul-operator master: Add initial withCertManager input toggle https://review.opendev.org/718840 | 21:32 |
openstackgerrit | Tristan Cacqueray proposed zuul/zuul-operator master: Add gearman tls secret provided by cert-manager https://review.opendev.org/719110 | 21:32 |
-openstackstatus- NOTICE: Maintenance on etherpad.opendev.org is complete and the service is available again | 22:26 | |
paladox | corvus hi, it appears that clicking on the storage link (log) on https://ci.gerritcodereview.com/t/gerrit/build/753ed28956bb474da16d70351fba6d37 says "No such object: gerrit_zuul_logs2/33/262733/1/check/gerrit-plugin-build/753ed28/" | 22:48 |
corvus | paladox: yeah it's a limitation of google storage we haven't worked around in zuul yet. easiest/best thing to do is use the logs/console tab | 22:49 |
paladox | ah | 22:49 |
corvus | paladox: but if you want to access it directly, you can add 'index.html' to that | 22:49 |
paladox | thanks! | 22:49 |
corvus | it seems that people are drawn to the "log url" metadata rather than the big logs tab at the top | 22:49 |
corvus | i think that's a ui problem we should address | 22:49 |
corvus | paladox: also, hey, cool, a test failure! does that look like a legit failure? | 22:50 |
paladox | It says "non-zero return code" but i'm not sure why. | 22:50 |
paladox | It builds locally and also works | 22:50 |
paladox | fixes a bug that i found | 22:50 |
paladox | https://bugs.chromium.org/p/gerrit/issues/detail?id=12575 | 22:50 |
corvus | paladox: is "bazelisk test plugins/download-commands/..." the way you run tests for that plugin? | 22:51 |
paladox | oh!! | 22:51 |
paladox | i guess using bazel should also be the same | 22:51 |
* paladox tries | 22:51 | |
paladox | found the issue i think | 22:52 |
paladox | it has no tests | 22:52 |
paladox | so when running `bazel test plugins/download-commands/...` it shows as a sucess but returns a error code 4 | 22:52 |
paladox | (when i did echo $?) | 22:52 |
paladox | corvus ^ | 22:53 |
corvus | yep that's what zuul saw too: https://ci.gerritcodereview.com/t/gerrit/build/753ed28956bb474da16d70351fba6d37/console#2/0/3/testnode | 22:53 |
paladox | so i guess we would have to put running tests behind a flag | 22:54 |
corvus | i did, let me push up a change | 22:54 |
paladox | ohh | 22:54 |
corvus | paladox: remote: https://gerrit-review.googlesource.com/c/zuul/config/+/262772 Do not run tests for the download-commands plugin [NEW] | 22:56 |
paladox | corvus awesome! thank you! +1'd | 22:56 |
*** hashar has joined #zuul | 22:58 | |
*** hashar has quit IRC | 23:13 | |
*** jamesmca_ has quit IRC | 23:19 | |
*** jamesmcarthur has joined #zuul | 23:20 | |
*** tosky has quit IRC | 23:20 | |
openstackgerrit | Tristan Cacqueray proposed zuul/zuul-operator master: Add initial withCertManager input toggle https://review.opendev.org/718840 | 23:24 |
openstackgerrit | Tristan Cacqueray proposed zuul/zuul-operator master: Add gearman tls secret provided by cert-manager https://review.opendev.org/719110 | 23:24 |
*** jamesmcarthur has quit IRC | 23:25 | |
*** jamesmcarthur has joined #zuul | 23:33 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!