Thursday, 2019-03-14

*** jamesmcarthur has joined #zuul00:13
*** jamesmcarthur has quit IRC00:16
*** jamesmcarthur has joined #zuul00:16
*** jamesmcarthur has quit IRC00:31
SpamapSyeah I'd recommend sudo over su for basically everything00:43
*** jamesmcarthur has joined #zuul00:57
*** jamesmcarthur has quit IRC00:57
*** jamesmcarthur has joined #zuul00:57
*** rlandy|bbl is now known as rlandy01:00
*** jamesmcarthur has quit IRC01:23
*** jamesmcarthur has joined #zuul01:24
*** jamesmcarthur has quit IRC01:26
*** jamesmcarthur has joined #zuul01:27
*** rlandy has quit IRC01:38
SpamapShttps://thenewstack.io/continuous-delivery-foundation-launches-with-jenkins-jenkins-x-spinnaker-tekton/02:38
*** jamesmcarthur has quit IRC02:47
*** jamesmcarthur has joined #zuul03:13
*** jamesmcarthur has quit IRC03:17
*** bjackman has joined #zuul03:44
bjackmancorvus, fungi No I have stdout and stderr merged in my job-output.json too, also in the result of my ansible "register"04:50
*** saneax has joined #zuul04:57
bjackmanBy the way, after months of back-burner work, I finally switched on Zuul gating in my org on Monday. Thanks for all you're help and the great SW y'all!04:59
bjackmanyour*****05:00
bjackman(I should be excited to see the gradual improvement in product quality but mostly I am excited that I can now more easily enforce psychopathic draconian coding style rules :D)05:01
SpamapSbjackman: congrats!!05:45
SpamapSbjackman: btw we've been using a code formatter and git hooks that just do it.. has been mostly positive. :)05:45
SpamapS(if folks bypass the git hooks the formatter fails in the gate too ;)05:46
*** raukadah is now known as chandankumar05:50
*** gtema has joined #zuul06:05
*** gtema has quit IRC06:06
*** [GNU] has joined #zuul07:01
*** gtema has joined #zuul07:54
*** themroc has joined #zuul08:01
*** hashar has joined #zuul08:04
*** pcaruana has joined #zuul08:07
*** pcaruana has quit IRC08:11
*** pcaruana has joined #zuul08:23
*** hashar has quit IRC08:49
*** bjackman has quit IRC08:58
*** jpena|off is now known as jpena09:00
*** bjackman has joined #zuul09:04
*** panda|rover|off has quit IRC09:07
*** hashar has joined #zuul09:11
badboycan someone explain to me what's the reason for using Jenkins when there's Zuul?09:19
mhubadboy, zuul is not an exact replacement for jenkins, and vice versa09:20
badboymhu: could elaborate on "not an exact replacement"?09:20
badboys/could/could\ you/09:21
mhujenkins is basically a build launcher, automated or manual09:21
mhuwhereas zuul does launch builds, but as a gate keeper09:21
*** bjackman has quit IRC09:21
mhu(sorry, in meeting right now, I'll try to elaborate later)09:22
badboymhu: ok, thx09:22
mhualso, just my 2cts so take it for what it is, just an opinion :)09:22
*** bjackman has joined #zuul09:24
*** sshnaidm is now known as sshnaidm|rover09:25
*** pcaruana has quit IRC09:48
*** gtema has quit IRC10:03
*** gtema has joined #zuul10:09
*** saneax has quit IRC10:21
*** yolanda has quit IRC11:02
*** yolanda has joined #zuul11:03
mhubadboy, so for example zuul's speculative merging or dependency handling is not part of jenkins' perimeter, that's not what jenkins is meant to do11:12
mhuOTOH zuul is all about automation whereas jenkins lets you also trigger builds manually, from a form where you can customize parameters11:14
mhuwhat i am trying to say is that zuul and jenkins overlap on some features but one isn't a complete replacement for the other11:14
mhubut yeah, on the overlap zuul is better than jenkins at what they do11:16
*** zbr has quit IRC11:16
*** saneax has joined #zuul11:27
*** pcaruana has joined #zuul11:32
*** saneax has quit IRC11:38
*** saneax has joined #zuul11:39
badboymhu: the only advantage of Jenkins, based on what you're saying, is triggering manual builds11:39
*** gcerami_ has joined #zuul11:57
*** saneax has quit IRC11:57
*** saneax has joined #zuul11:58
*** zbr has joined #zuul11:58
openstackgerritSimon Westphahl proposed openstack-infra/zuul master: Add more fields for SMTPReporter subject format  https://review.openstack.org/64330612:02
openstackgerritSimon Westphahl proposed openstack-infra/zuul master: Add more fields for SMTPReporter subject format  https://review.openstack.org/64330612:05
*** jpena is now known as jpena|lunch12:12
openstackgerritSimon Westphahl proposed openstack-infra/zuul master: wip: Spec for allowing circular dependencies  https://review.openstack.org/64330912:14
openstackgerritSimon Westphahl proposed openstack-infra/zuul master: wip: Spec for allowing circular dependencies  https://review.openstack.org/64330912:17
*** swest has joined #zuul12:18
*** rlandy has joined #zuul12:32
mhubadboy, yes but it can be an important use case, especially for QE12:33
*** gcerami_ is now known as panda12:40
*** bjackman_ has joined #zuul12:47
*** jamesmcarthur has joined #zuul12:47
*** bjackman has quit IRC12:49
*** jamesmcarthur has quit IRC13:27
openstackgerritJeremy Stanley proposed openstack-infra/zuul-jobs master: Support ~/.gnupg subdirs in remove-gpgkey role  https://review.openstack.org/64332813:32
fungiif any zuul/config reviewers have a moment, we found that ^ when trying to use the gnupg roles on newer ubuntu. it should be an entirely backward-compatible solution13:33
*** jamesmcarthur has joined #zuul13:46
*** bjackman_ has quit IRC13:49
*** gtema has quit IRC13:50
*** jpena|lunch is now known as jpena13:50
pabelangerfungi: left review14:02
pabelangerfungi: removed -1, I am unsure why we are using su -c there, over shell14:04
AJaegerfungi: looks fine - but linter job is unhappy, see http://logs.openstack.org/28/643328/1/check/openstack-zuul-jobs-linters/79afe4e/job-output.txt.gz#_2019-03-14_13_40_57_34110114:08
openstackgerritSimon Westphahl proposed openstack-infra/zuul master: Ensure correct lexical sorting of node requests  https://review.openstack.org/64335214:37
openstackgerritSimon Westphahl proposed openstack-infra/zuul master: Ensure correct lexical sorting of node requests  https://review.openstack.org/64335214:40
fungipabelanger: it's possible we used sh -c there because of the glob? if so i suppose we could replace it?14:47
fungi(do shell tasks handle globbing okay?)14:47
fungiAJaeger: thanks for the heads up, i guess i need to do ``; in ansible14:48
openstackgerritJeremy Stanley proposed openstack-infra/zuul-jobs master: Support ~/.gnupg subdirs in remove-gpgkey role  https://review.openstack.org/64332814:50
*** smyers has quit IRC14:50
fungipabelanger: AJaeger: ^14:50
openstackgerritTobias Henkel proposed openstack-infra/zuul master: Run tox-py35 jobs on xenial  https://review.openstack.org/64335914:51
*** Shrews has quit IRC14:51
*** smyers has joined #zuul14:51
*** mordred has quit IRC14:51
tobiashthis should unblock our jobs ^14:52
*** saneax has quit IRC14:52
*** bjackman_ has joined #zuul14:53
*** Shrews has joined #zuul14:53
openstackgerritTobias Henkel proposed openstack-infra/zuul master: Run tox-py35 jobs on xenial  https://review.openstack.org/64335914:54
*** mordred has joined #zuul14:57
*** yolanda has quit IRC14:57
pabelangertobiash: +314:59
tobiashpabelanger: thanks :)14:59
*** mordred has quit IRC15:23
*** gtema has joined #zuul15:29
*** mordred has joined #zuul15:29
*** swest has quit IRC15:41
*** yolanda has joined #zuul15:45
openstackgerritJeremy Stanley proposed openstack-infra/zuul-jobs master: Support ~/.gnupg subdirs in remove-gpgkey role  https://review.openstack.org/64332815:47
fungimordred: pabelanger: AJaeger: ^ there's the next revision based on feedback from all of you15:48
mordredfungi, corvus: don't hate me ... is the double \ needed if there's no wrapping sh -c ?15:48
fungii think the error was coming from yaml parsing?15:49
mordredah - cool15:50
fungish -c doesn't need (and indeed won't work with) the \ doubled there15:50
fungibut if i had just \; it looked like pyyaml's loader complained about an unrecognized escape sequence \;15:50
mordredyeah. kk. I see it now. thanks!15:50
fungithinking about that some more, i don't suppose the sh -c was being used to get ~ to resolve to the homedir?16:04
pabelangerpossible, but shell will resolve it properly too16:05
openstackgerritJames E. Blair proposed openstack-infra/zuul master: web: support more than one job in the filter  https://review.openstack.org/64338516:07
openstackgerritMerged openstack-infra/zuul-jobs master: Support ~/.gnupg subdirs in remove-gpgkey role  https://review.openstack.org/64332816:10
corvusmordred: i think a bionic issue hit our npm jobs?  http://logs.openstack.org/85/643385/1/check/zuul-build-dashboard/80ffd1b/ara-report/result/f2162c50-4348-4f81-a541-e05f304a62ac/16:32
corvusit looks like node version 8 is installed, but npm is not...16:33
corvuswas npm included automatically in xenial?16:34
*** themroc has quit IRC16:35
openstackgerritDavid Moreau Simard proposed openstack-infra/zuul-jobs master: Add a role to mirror a git repository to a remote git server  https://review.openstack.org/64339416:38
dmsimard^ fungi: a first stab at what you suggested on the ML, I'll write integration tests after lunch16:39
fungithanks dmsimard! i expect a lot of folks will find that very useful16:39
mordredcorvus: looking16:41
*** jamesmcarthur has quit IRC16:42
mordredcorvus: we're installing nodejs from bionic not from upstream nodesource16:43
mordredcorvus: http://logs.openstack.org/85/643385/1/check/zuul-build-dashboard/80ffd1b/ara-report/result/872d2e84-5f3e-41da-8fa5-d8b44033590d/16:43
mordredcorvus: so we might want to put in a pin to say "always get nodejs from this particular apt source regardless of version math"16:43
mordredfungi: do you know the apt pin cantrip for that ^^?16:44
corvusdmsimard: looks good -- but i left a quick suggestion on there you may want to look at before you get much further16:44
corvusmordred: oh! i missed that16:44
fungimordred: aha, saw a similar failure on one of the openstack/requirements jobs too16:45
fungii think we need an additional package which installing from the packages provided in bionic16:45
fungior is the desire to force to get it from nodesource anyway?16:45
mordredyes. tht would be my preference so16:46
mordredso that the job works consistently16:46
mordredbecause the distro and upstream packages have decided to package things differently16:46
*** jamesmcarthur has joined #zuul16:47
mordredfungi, corvus: patch coming16:49
corvusmordred: cool, i have an update to my patch standing by. once yours is up i'll add a depends-on and push mine16:49
openstackgerritMonty Taylor proposed openstack-infra/zuul-jobs master: Pin nodejs to come from nodesource repo  https://review.openstack.org/64339716:51
mordredcorvus: is that the right cantrip for copy?16:51
corvusmordred: i *think* so, but i never know when "files/" is automatic.16:52
mordredI think it's automatic in roles - doing a quick grep16:53
fungimordred: that looks right. it should be a sufficient priority that we always upgrade to it for the nodejs package (though insufficient if we want to be able to downgrade to it)16:53
openstackgerritJames E. Blair proposed openstack-infra/zuul master: web: support more than one job in the filter  https://review.openstack.org/64338516:53
mordredcorvus: fingers crossed16:53
*** sshnaidm|rover is now known as sshnaidm|afk17:02
*** gtema has quit IRC17:11
*** dkehn has quit IRC17:14
corvusmordred: welp.  it helps to spell "depends" correctly17:16
openstackgerritJames E. Blair proposed openstack-infra/zuul master: web: support more than one job in the filter  https://review.openstack.org/64338517:16
mordredcorvus: bah17:16
*** hashar is now known as hasharDinner17:18
dmsimardcorvus: your suggestion makes sense, thanks17:19
*** chandankumar is now known as chkumar24617:34
*** panda is now known as panda|off17:34
corvusmordred: huzzah! http://logs.openstack.org/85/643385/3/check/zuul-build-dashboard/e55539e/npm/html/status17:36
mordredcorvus: woot! I can confirm multi pattern works too17:38
pabelangerI got file not found on that url ^17:39
mordredyou have to remove /status17:39
corvusoh sorry its http://logs.openstack.org/85/643385/3/check/zuul-build-dashboard/e55539e/npm/html/17:39
pabelangerah17:39
pabelangeryah, works now17:39
*** panda|off has quit IRC18:01
*** panda has joined #zuul18:04
*** bjackman_ has quit IRC18:10
*** hasharDinner has quit IRC18:23
*** rlandy is now known as rlandy|biab18:25
*** jpena is now known as jpena|off18:27
fungimordred: do you expect 643397 should fix the cross-horizon-npm retry_limit result on https://review.openstack.org/643375 ?18:29
*** jamesmcarthur has quit IRC18:30
*** jamesmcarthur has joined #zuul18:33
*** hasharDinner has joined #zuul18:45
*** hasharDinner is now known as hashar18:55
openstackgerritMerged openstack-infra/zuul master: Run tox-py35 jobs on xenial  https://review.openstack.org/64335918:56
*** rlandy|biab is now known as rlandy19:02
openstackgerritMerged openstack-infra/zuul-jobs master: Pin nodejs to come from nodesource repo  https://review.openstack.org/64339719:05
openstackgerritDavid Moreau Simard proposed openstack-infra/zuul-jobs master: Add a role to mirror a git repository to a remote git server  https://review.openstack.org/64339419:07
openstackgerritDavid Moreau Simard proposed openstack-infra/zuul-jobs master: Do not merge: test zuul-jobs-upload-git-mirror job  https://review.openstack.org/64343719:12
openstackgerritDavid Moreau Simard proposed openstack-infra/zuul-jobs master: Do not merge: test zuul-jobs-upload-git-mirror job  https://review.openstack.org/64343719:14
openstackgerritDavid Moreau Simard proposed openstack-infra/zuul-jobs master: Do not merge: test zuul-jobs-upload-git-mirror job  https://review.openstack.org/64343719:23
openstackgerritDavid Moreau Simard proposed openstack-infra/zuul-jobs master: Do not merge: test zuul-jobs-upload-git-mirror job  https://review.openstack.org/64343719:28
*** yolanda has quit IRC19:46
openstackgerritDavid Moreau Simard proposed openstack-infra/zuul-jobs master: Add a role to mirror a git repository to a remote git server  https://review.openstack.org/64339420:09
openstackgerritDavid Moreau Simard proposed openstack-infra/zuul-jobs master: Do not merge: test zuul-jobs-upload-git-mirror job  https://review.openstack.org/64343720:10
*** jamesmcarthur has quit IRC20:36
pabelangerneed to #dadops for a bit, but would love to get a few reviews on https://review.openstack.org/642913/ to relax the keys folder permissions a little21:11
*** pcaruana has quit IRC22:21
*** rlandy is now known as rlandy|bbl22:22
*** hashar has quit IRC22:41
pabelangerI'm trying to debug something in zuul, but assume the following snip of tenant config: http://paste.openstack.org/show/747822/22:51
pabelangerwhy would a zuul merger need to process a change on the openstack connection?22:53
pabelangerthe reason I ask, is openstack/nova uploaded a 40 stack patchset to zuul.o.o, and right now it seems rdoproject tenant in sf.io zuul is trying to process all the same nova patches.22:54
pabelangerwhich, because there isn't a many zuul mergers available, has caused jobs in the ansible-network tenant to be blocked22:55
pabelangerI am trying to see if there is something on zuul that can be optimized22:55
clarkbpabelanger: I want to say that is a known bug22:55
clarkbzuul will do the merges for configs even if it isn't loading configs22:55
clarkbhowever if you are running jobs on that project it is gonna need to merge anyway22:56
clarkbso that it can provide the refs to the job(s)22:56
pabelangeryah, I don't believe we are running jobs, but trying to confirm22:56
pabelangerit more there to use depends-on, IIRC22:56
pabelangerhttps://review.rdoproject.org/zuul/project/openstack/nova22:58
pabelangerI think because that is giving 500 status, we don't have any jobs configured22:58
pabelangerokay, I'll have to check back in moring and maybe work on a patch23:11
pabelangerfor now, I suspect it will take a few hours (guessing 4hr) for rdoproject to catch up23:11
pabelangertobiash: tristanC: maybe ^ rings a bell23:12

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