Friday, 2020-04-10

*** ysandeep|out is now known as ysandeep00:34
openstackgerritTristan Cacqueray proposed zuul/zuul-operator master: Add initial withCertManager input toggle  https://review.opendev.org/71884000:39
tristanCcorvus: 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 implemented00:42
*** zxiiro has quit IRC00:56
*** swest has quit IRC01:23
openstackgerritMerged zuul/zuul master: kubernetes-operator: add job volume to the spec  https://review.opendev.org/70663901:24
openstackgerritMerged zuul/zuul master: kubernetes-operator: change attribute to camelCase  https://review.opendev.org/70719001:24
*** swest has joined #zuul01:39
*** Goneri has quit IRC01:47
openstackgerritMerged zuul/zuul-operator master: Update attributes to camelCase  https://review.opendev.org/70719302:31
*** ysandeep is now known as ysandeep|off03:28
*** y2kenny has quit IRC03:38
*** sgw has quit IRC03:38
*** bhavikdbavishi has joined #zuul03:55
*** bhavikdbavishi1 has joined #zuul03:58
*** bhavikdbavishi has quit IRC04:00
*** bhavikdbavishi1 is now known as bhavikdbavishi04:00
*** sgw has joined #zuul04:06
*** cdearborn has quit IRC04:17
*** evrardjp has quit IRC04:37
*** evrardjp has joined #zuul04:37
*** bhavikdbavishi has quit IRC04:38
*** bhavikdbavishi has joined #zuul04:49
openstackgerritIan Wienand proposed zuul/zuul-jobs master: ensure-pip: export ensure_pip_virtualenv_command  https://review.opendev.org/71822404:53
openstackgerritIan Wienand proposed zuul/zuul-jobs master: fetch-zuul-cloner: use ensure-pip  https://review.opendev.org/71788204:53
openstackgerritIan Wienand proposed zuul/zuul-jobs master: fetch-subunit-output test: use ensure-pip  https://review.opendev.org/71822504:53
openstackgerritIan Wienand proposed zuul/zuul-jobs master: ensure-tox: use ensure-pip role  https://review.opendev.org/71766304:53
openstackgerritIan Wienand proposed zuul/zuul-jobs master: Update Fedora to 31  https://review.opendev.org/71765704:53
*** gtema has joined #zuul06:08
*** bhavikdbavishi has quit IRC06:11
*** bhavikdbavishi has joined #zuul06:11
*** gtema has quit IRC06:13
webknjaz@tobiash: Mind verifying that https://github.com/cherrypy/cheroot/pull/277 fixes your problem?06:17
*** dpawlik has joined #zuul06:20
*** bhavikdbavishi has quit IRC06:24
*** gtema has joined #zuul06:32
*** gtema has quit IRC06:33
*** bhavikdbavishi has joined #zuul06:38
*** bhavikdbavishi has quit IRC06:54
*** jcapitao has joined #zuul06:59
*** gtema has joined #zuul07:02
*** gtema has quit IRC07:03
*** bhavikdbavishi has joined #zuul07:10
*** bhavikdbavishi has quit IRC07:22
*** tosky has joined #zuul07:32
*** rpittau|afk is now known as rpittau07:36
*** hashar has joined #zuul07:52
*** bhavikdbavishi has joined #zuul08:47
*** bhavikdbavishi has quit IRC09:08
*** bhavikdbavishi has joined #zuul09:08
*** bhavikdbavishi1 has joined #zuul09:19
*** bhavikdbavishi has quit IRC09:21
*** bhavikdbavishi1 is now known as bhavikdbavishi09:21
*** sassyn has joined #zuul10:03
sassynhi all10:03
sassynI'm a new baby when it come to zuul10:04
sassynI running Gerrit with Jenkins interrogation10:04
sassynI want to move to zuul, so zuul will trigger jobs from gerrit, and will run jobs in jenkins10:04
sassynI have install zuul + german + zookeeper, and I still have question if I really need the nodepool10:05
sassynI don't want to auto create slave/workers since I allready have jenkins slaves.10:06
sassynSo the ansible part is not something I need in my setup10:06
sassynmy question how can I run a jobs that exists  in the the jenkins ?10:06
sassynis this supported?10:07
sassynI 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 documenation10:07
sassynThank you10:07
*** persia_ is now known as persia10:07
*** armstrongs has joined #zuul10:10
*** rpittau is now known as rpittau|bbl10:19
*** armstrongs has quit IRC10:20
jktsassyn: you're following a six-year-old article. Since that tim, Zuul had moved on; now there's no place for Jenkins in there10:41
jktsassyn: a good place to start reading is https://zuul-ci.org/10:42
* jkt afk10:42
*** jcapitao is now known as jcapitao_lunch10:58
*** bhavikdbavishi has quit IRC11:10
*** bhavikdbavishi has joined #zuul11:20
*** rpittau|bbl is now known as rpittau11:58
*** jcapitao_lunch is now known as jcapitao12:18
fungisassyn: 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/zuul12:55
sassynfugni12:56
fungibut basically we started to work on removing jenkins from the system architecture in early 201612:56
sassynthank you12:56
openstackgerritTristan Cacqueray proposed zuul/zuul-operator master: Increase scheduler wait to timeout  https://review.opendev.org/71816212:57
openstackgerritTristan Cacqueray proposed zuul/zuul-operator master: Add schema validation error message  https://review.opendev.org/71899912:57
sassynI figure it out already..  https://opensource.com/article/20/2/zuul point it out this in the comment below12:57
sassynmy problem is that I have 400 jsalves12:57
sassynand 200+ jobs in jenkins12:57
sassynchange it will be hard12:57
fungiin 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 be12:58
sassynZuul seems hard to understand12:59
sassynI still not getting to full picture12:59
fungiobviously 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 them12:59
sassynmost of our jobs use Jenkinsfile13:00
sassynand pipelines13:00
sassynall are in the gerrit repo13:00
fungisassyn: there's an architecture diagram at https://zuul-ci.org/docs/zuul/discussion/components.html with explanations of the individual components13:00
fungiin case you haven't found that yet13:01
sassynyes, read it13:01
sassynfungi - thank u!13:01
sassynTired of sitting home with this corona13:01
fungialso 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.html13:02
fungiif you have specific questions which don't seem to be addressed by the documentation, please do ask them13:02
fungisassyn: 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/qRYRWb42CuQ13:08
sassynperfect!13:09
sassynare u are one of the developers?13:09
fungii'm one of the zuul maintainers, yes (jeremy stanley). we're listed here: https://zuul-ci.org/docs/zuul/reference/governance.html#zuul-maintainers13:12
fungibut there are plenty of other people who are doing development work on zuul in addition to us13:12
fungi(i haven't really written that much of it by comparison)13:12
*** cdearborn has joined #zuul13:23
*** bhavikdbavishi has quit IRC13:39
mnasercorvus: 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
corvusmnaser: 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 easily14:19
openstackgerritTristan Cacqueray proposed zuul/zuul-operator master: Add a zuul-ensure-database-passwords role  https://review.opendev.org/71788014:19
mnasercorvus: 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 dont14:20
corvusmnaser: yep14:21
mnasercorvus: 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
mnaserbecause ansible usually doesnt put localhost inside all14:21
corvusmnaser: yeah, don't inherit from that job, just use the role14:21
mnaserokay, that clears it up14:22
corvusmnaser: (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
mnaseri will hack on something and push it up soon14:22
mnaser++ makes sense14:22
mnaserzuul-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 role14:24
tristanCmnaser: wfm14:25
corvusthat name looks good -- can we talk about tests?14:26
corvusi think we keep going back and forth about whether new roles should have tests14:26
corvusmaybe we should go ahead and decide that they should?14:26
mnasersure.  i struggled to come up with some reasonable set of tests personally, because anything i 'imagined' ended up testing the actual ansible module14:26
corvusmnaser: that's probably fine :)14:27
corvusmnaser: 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 errer14:28
mnaserso run the playbook and then check /etc/yum.repos.d/foo.repo, etc?14:28
corvusmnaser: i'd be satisfied at "run the role" and stop there, but certainly also adding "check..." wouldn't hurt.  :)14:29
mnaserthat seems fair, also, i was thinking that any role that uses it should be included in the 'files' section for that job14:29
mnaserbut worried that might get out of hand very quickly14:29
corvusmnaser: yeah, that would be an option, but i think with dedicated test we could avoid that14:30
mnaserok, 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
corvusthe "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 playbook14:30
mnaseri actually think for something like this, it's good to do a dedicated playbook because breaking this role would break a lot of things14:31
corvus(the idea of that was to at least make "just run the role" tests easy to do)14:31
corvusmnaser: 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
corvusyeah, i dig the idea :)14:31
mnaserok cool, i'll try to do the opendev/openstack stuff now as i don't think it's a massive effort for now14:32
corvussassyn: 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
corvushttps://zuul-ci.org/docs/zuul-jobs/general-jobs.html#job-run-test-command14:35
openstackgerritTristan Cacqueray proposed zuul/zuul-operator master: Increase scheduler wait to timeout  https://review.opendev.org/71816214:55
openstackgerritTristan Cacqueray proposed zuul/zuul-operator master: Add schema validation error message  https://review.opendev.org/71899914:55
openstackgerritTristan Cacqueray proposed zuul/zuul-operator master: Add a zuul-ensure-database-passwords role  https://review.opendev.org/71788014:55
*** rpittau is now known as rpittau|afk14:57
*** armstrongs has joined #zuul15:37
*** Goneri has joined #zuul15:40
openstackgerritTristan Cacqueray proposed zuul/zuul-operator master: Add a zuul-ensure-database-passwords role  https://review.opendev.org/71788015:42
*** armstrongs has quit IRC15:45
*** bhavikdbavishi has joined #zuul15:53
*** bhavikdbavishi1 has joined #zuul15:58
*** jcapitao has quit IRC16:00
*** bhavikdbavishi has quit IRC16:00
*** bhavikdbavishi1 is now known as bhavikdbavishi16:00
-openstackstatus- NOTICE: review.opendev.org is being restarted for scheduled maintenance; see http://lists.opendev.org/pipermail/service-announce/2020-April/000003.html16:07
*** bhavikdbavishi has quit IRC16:23
*** evrardjp has quit IRC16:37
*** evrardjp has joined #zuul16:37
*** dpawlik has quit IRC16: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.html17:04
openstackgerritTristan Cacqueray proposed zuul/zuul-operator master: Add zuul-registry deployment  https://review.opendev.org/71065017:06
openstackgerritAndreas Jaeger proposed zuul/zuul-jobs master: Fix check_jobs_documented linter  https://review.opendev.org/71905417:09
*** hashar has quit IRC17:20
-openstackstatus- NOTICE: The etherpad migration is still in progress; revised estimated time of completion 18:30 UTC17:54
*** mugsie has quit IRC17:59
*** EmilienM has quit IRC17:59
*** Open10K8S has quit IRC17:59
*** mugsie has joined #zuul18:03
*** EmilienM has joined #zuul18:03
*** Open10K8S has joined #zuul18:03
*** EmilienM has quit IRC18:12
*** EmilienM has joined #zuul18:13
*** jamesmcarthur has joined #zuul18:54
*** jamesmcarthur has quit IRC19:30
*** jamesmcarthur has joined #zuul19:35
*** sgw has quit IRC19:36
*** jamesmcarthur has quit IRC19:40
*** jamesmcarthur has joined #zuul19:45
*** sgw has joined #zuul19:54
openstackgerritTristan Cacqueray proposed zuul/nodepool master: config: add missing voluptuous required attributes  https://review.opendev.org/71909419:56
*** nhicher has quit IRC20:02
*** nhicher has joined #zuul20:02
-openstackstatus- NOTICE: Due to a database migration error, etherpad.opendev.org is offline until further notice.20:10
openstackgerritTristan Cacqueray proposed zuul/zuul-operator master: Add zuul-preview service  https://review.opendev.org/71841920:34
openstackgerritTristan Cacqueray proposed zuul/zuul-operator master: Add zuul-registry deployment  https://review.opendev.org/71065020:38
*** sassyn has quit IRC20:38
*** sshnaidm|off has quit IRC20:46
*** jamesmcarthur_ has joined #zuul20:57
*** jamesmca_ has joined #zuul20:58
*** jamesmcarthur has quit IRC21:00
*** jamesmcarthur_ has quit IRC21:02
openstackgerritTristan Cacqueray proposed zuul/zuul-operator master: Add initial withCertManager input toggle  https://review.opendev.org/71884021:32
openstackgerritTristan Cacqueray proposed zuul/zuul-operator master: Add gearman tls secret provided by cert-manager  https://review.opendev.org/71911021:32
-openstackstatus- NOTICE: Maintenance on etherpad.opendev.org is complete and the service is available again22:26
paladoxcorvus 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
corvuspaladox: 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 tab22:49
paladoxah22:49
corvuspaladox: but if you want to access it directly, you can add 'index.html' to that22:49
paladoxthanks!22:49
corvusit seems that people are drawn to the "log url" metadata rather than the big logs tab at the top22:49
corvusi think that's a ui problem we should address22:49
corvuspaladox: also, hey, cool, a test failure!  does that look like a legit failure?22:50
paladoxIt says "non-zero return code" but i'm not sure why.22:50
paladoxIt builds locally and also works22:50
paladoxfixes a bug that i found22:50
paladoxhttps://bugs.chromium.org/p/gerrit/issues/detail?id=1257522:50
corvuspaladox: is "bazelisk test plugins/download-commands/..." the way you run tests for that plugin?22:51
paladoxoh!!22:51
paladoxi guess using bazel should also be the same22:51
* paladox tries22:51
paladoxfound the issue i think22:52
paladoxit has no tests22:52
paladoxso when running `bazel test plugins/download-commands/...` it shows as a sucess but returns a error code 422:52
paladox(when i did echo $?)22:52
paladoxcorvus ^22:53
corvusyep that's what zuul saw too: https://ci.gerritcodereview.com/t/gerrit/build/753ed28956bb474da16d70351fba6d37/console#2/0/3/testnode22:53
paladoxso i guess we would have to put running tests behind a flag22:54
corvusi did, let me push up a change22:54
paladoxohh22:54
corvuspaladox: remote:   https://gerrit-review.googlesource.com/c/zuul/config/+/262772 Do not run tests for the download-commands plugin [NEW]22:56
paladoxcorvus awesome! thank you! +1'd22:56
*** hashar has joined #zuul22:58
*** hashar has quit IRC23:13
*** jamesmca_ has quit IRC23:19
*** jamesmcarthur has joined #zuul23:20
*** tosky has quit IRC23:20
openstackgerritTristan Cacqueray proposed zuul/zuul-operator master: Add initial withCertManager input toggle  https://review.opendev.org/71884023:24
openstackgerritTristan Cacqueray proposed zuul/zuul-operator master: Add gearman tls secret provided by cert-manager  https://review.opendev.org/71911023:24
*** jamesmcarthur has quit IRC23:25
*** jamesmcarthur has joined #zuul23:33

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