Friday, 2019-01-11

corvus*finally*  i have a playbook to retag a docker image00:31
corvushttps://hub.docker.com/r/jeblair/gitea/tags00:31
corvustest1 and latest are a retag og change_123400:31
corvusit's 3 ansible tasks, but thanks to the simplicity and clarity of the dockerhub api and documentation, it took me 2 hours to write them.00:33
*** openstackgerrit has quit IRC00:34
*** openstackgerrit has joined #opendev00:54
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: web: add jobs graph rendering  https://review.openstack.org/53786900:54
openstackgerritMerged openstack-infra/project-config master: Add CentOS NetworkManager testing node  https://review.openstack.org/62968000:54
openstackgerritJames E. Blair proposed openstack-infra/system-config master: Use promote pipeline for system-config images  https://review.openstack.org/62995701:06
corvusthere we go ^01:06
openstackgerritJames E. Blair proposed openstack-infra/system-config master: Use promote pipeline for system-config images  https://review.openstack.org/62995701:07
openstackgerritJames E. Blair proposed openstack-infra/system-config master: Use promote pipeline for system-config images  https://review.openstack.org/62995701:09
openstackgerritJames E. Blair proposed openstack-infra/system-config master: Use promote pipeline for system-config images  https://review.openstack.org/62995702:58
openstackgerritJames E. Blair proposed openstack-infra/system-config master: Use a mix-in job to reduce duplication  https://review.openstack.org/62998302:58
openstackgerritJames E. Blair proposed openstack-infra/system-config master: Use a mix-in job to reduce duplication  https://review.openstack.org/62998303:02
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: web: add version to info endpoint  https://review.openstack.org/60957105:35
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: web: add about dropdown to display zuul version  https://review.openstack.org/63002705:35
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: sql: add buildset uuid  https://review.openstack.org/63003406:45
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: web: add /{tenant}/buildsets route  https://review.openstack.org/63003506:45
openstackgerritRui Chen proposed openstack-infra/zuul master: Avoid using list branches with protected=1 in github driver  https://review.openstack.org/63003806:55
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: sql: add buildset uuid column  https://review.openstack.org/63003407:17
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: web: add /{tenant}/buildsets route  https://review.openstack.org/63003507:17
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: web: add buildsets page  https://review.openstack.org/63004107:17
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: sql: add buildset uuid column  https://review.openstack.org/63003407:46
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: web: add /{tenant}/buildsets route  https://review.openstack.org/63003508:17
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: web: add /{tenant}/buildsets route  https://review.openstack.org/63003509:07
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: web: add buildsets page  https://review.openstack.org/63004109:07
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: web: add /{tenant}/buildset/{uuid} route  https://review.openstack.org/63007809:07
openstackgerritTristan Cacqueray proposed openstack-infra/zuul master: web: add buildset page  https://review.openstack.org/63007909:07
openstackgerritTobias Henkel proposed openstack-infra/zuul master: Fix test_load_governor on large machines  https://review.openstack.org/63011810:08
openstackgerritAndriy Shevchenko proposed openstack/gertty master: Updatae home-page  https://review.openstack.org/63012010:11
openstackgerritArnaud Morin proposed openstack-infra/project-config master: Enable back GRA1 on OVH cloud  https://review.openstack.org/63013010:19
openstackgerritStephen Finucane proposed openstack-infra/project-config master: Retire gantt, python-ganttclient  https://review.openstack.org/63016211:34
openstackgerritTobias Henkel proposed openstack-infra/zuul master: Fix test_load_governor on large machines  https://review.openstack.org/63011811:39
openstackgerritMerged openstack-infra/project-config master: Enable back GRA1 on OVH cloud  https://review.openstack.org/63013011:50
openstackgerritAndriy Shevchenko proposed openstack/ansible-role-cloud-launcher master: Update home-page  https://review.openstack.org/63018512:11
openstackgerritSorin Sbarnea proposed openstack-infra/zuul-jobs master: Avoid zuul DISK_FULL failure with too big logs  https://review.openstack.org/63022413:37
openstackgerritSorin Sbarnea proposed openstack-infra/zuul-jobs master: Avoid zuul DISK_FULL failure with too big logs  https://review.openstack.org/63022413:46
openstackgerritWill Szumski proposed openstack-dev/pbr master: Do not globally replace path prefix  https://review.openstack.org/62900613:54
openstackgerritWill Szumski proposed openstack-dev/pbr master: Fix white space handling in file names  https://review.openstack.org/62916114:31
pabelangerwow, that is alot of openstackgerrit :)14:35
fungigerritrific14:38
openstackgerritPaul Belanger proposed openstack-infra/project-config master: Exclude pipeline config items for openstack-dev/ci-sandbox  https://review.openstack.org/63028615:19
pabelangercorvus: clarkb: I think ^ is the proper syntax to disable pipelines from being loaded in openstack-dev/ci-sandbox15:20
openstackgerritGraham Whaley proposed openstack-infra/openstack-zuul-jobs master: Add dco-signoff check files for Kata Containers  https://review.openstack.org/63029415:40
fungii suppose this is more an #opendev question now than an #openstack-infra one... the kata project has recently become much more interested in running jobs in our zuul. they use github as their primary means of code review and revision control management, so their workflows aren't terribly similar to our gerrit-using projects and they're looking at having at least a couple pipelines specific to their use case16:01
fungiwhich are unlikely to be shared by our other projects. should we be considering putting that in a new zuul tenant?16:01
corvusfungi: yes.  i think zuul should have a tenant too.16:02
fungii mentioned it in a comment on https://github.com/kata-containers/ci/issues/99 but of course it needs wider discussion16:02
mrhillsmanjust my .02, i am not sure how taxing a new is16:03
mrhillsmannew tenant is16:03
mrhillsmanbut maybe doing so can help things be more clearly defined and create better ux finding things16:04
corvusyeah, they're supposed to be very-low-cost (you probably want a project-config for each).16:04
corvusa while back, mordred suggested a third class of project ("tenant-project-config?") which was a little less privileged, but maybe additionally lets you define pipelines.  that could be handy.16:05
mrhillsmanthat does sound interesting16:06
mrhillsmancould be very handy indeed16:06
corvusfungi, clarkb: can you +3 https://review.openstack.org/629957  and we'll (1) see if we can get some images up, and (2) see if we have a working docker image promotion scheme :)16:07
fungiawesome! reviewing now16:08
fungii think i was able to mostly follow the image retagging discussion from earlier16:09
openstackgerritPaul Belanger proposed openstack-infra/zuul-jobs master: Add dco-license job  https://review.openstack.org/63030216:11
fungicorvus: lgtm, approved16:13
clarkbfungi: corvus I think there is a file path error in that change. I -1'd it just now16:17
fungioh, unapproved16:18
pabelangerfungi: clarkb: I've left a comment on kata dco-license job: https://review.openstack.org/630294/ should help them remove some code16:19
fungii just pointed that out to gwhaley in #kata too16:19
fungier, i mean in #kata-dev16:19
pabelangerOh16:19
pabelangerhmm16:19
pabelangerI think that job needs to be trusted?16:20
pabelangermaybe now16:20
pabelangernot*16:20
pabelangertesting now16:22
openstackgerritJames E. Blair proposed openstack-infra/system-config master: Use promote pipeline for system-config images  https://review.openstack.org/62995716:22
corvusclarkb, fungi: fixed, thanks!16:23
corvusi also tested the fix locally16:23
pabelangerfungi: okay cool, it works as expected: http://logs.openstack.org/08/630308/1/check/dco-license/c865186/ara-report/16:24
fungimy gertty is laggy16:25
fungibut approved now16:25
fungilatest iteration of 629957 i mean16:26
corvuspabelanger: both dco things lgtm, but i have a super nitpicky request on the job change :)16:28
pabelangercorvus: yes! fixing16:28
corvusi like that the message is configurable with a variable; let's a project add custom links, etc.16:29
openstackgerritPaul Belanger proposed openstack-infra/zuul-jobs master: Add dco-license job  https://review.openstack.org/63030216:30
openstackgerritStephen Finucane proposed openstack-infra/system-config master: Remove 'openstack-gantt' from statusbot monitoring  https://review.openstack.org/63032316:44
openstackgerritStephen Finucane proposed openstack-infra/project-config master: Retire gantt, python-ganttclient  https://review.openstack.org/63016216:44
openstackgerritJames E. Blair proposed openstack-infra/system-config master: Use a mix-in job to reduce duplication  https://review.openstack.org/62998316:51
openstackgerritJames E. Blair proposed openstack-infra/system-config master: Sketch of explicit multiple-inheritance  https://review.openstack.org/63033717:02
openstackgerritAndriy Shevchenko proposed openstack/os-performance-tools master: Update home-page  https://review.openstack.org/63034217:09
openstackgerritDavid Shrewsbury proposed openstack-infra/nodepool master: Add a timeout for the image build  https://review.openstack.org/62992317:09
openstackgerritMerged openstack-infra/system-config master: Use promote pipeline for system-config images  https://review.openstack.org/62995717:28
openstackgerritDavid Shrewsbury proposed openstack-infra/nodepool master: Add a timeout for the image build  https://review.openstack.org/62992317:32
openstackgerritJames E. Blair proposed openstack-infra/system-config master: Add build-image-jinja-init to check  https://review.openstack.org/63035117:35
corvusclarkb: ^ the fix for your other comment17:35
pabelangerclarkb: want to give https://review.openstack.org/#/q/topic:dco-license a review and maybe +A?17:47
clarkbpabelanger: can you see question on https://review.openstack.org/#/c/630302/2/ ?17:53
clarkbI've become increasingly wary of include_* after they completely broke our ansible on bridge17:54
pabelangerclarkb: replied17:56
clarkbhrm that is good to know but also somewhat scary17:57
clarkb(not sure if you saw the bug I filed around that, but tldr is you can crash ansible with nested includes17:57
clarkband the best part is ansible doesn't return failure.17:58
clarkbThis has been fixed in 2.8 but everything I tested before that is broken17:58
pabelangeroh, I don't think I seen that17:58
clarkbhttps://github.com/ansible/ansible/issues/4996917:59
corvusoh, hrm17:59
corvusi didn't notice that earlier17:59
corvusi thought we decided a long time ago to prefer "roles:" in zuul jobs for simplicity and consistency17:59
clarkbcorvus: I'm not aware of that if so, but I would be happy with such a rule given that bug :)18:00
clarkbshould I remove my +A?18:00
corvuswell, i know pabelanger and i talked about it at length.  i thought i convinced him.  i guess not.  :(18:00
corvusit's important to keep in mind that this is not just a system for ansible experts.  with jjb we were able to say "builders: [foo, bar, baz]" and construct jobs out of very simple building blocks18:01
corvuswe can do the same thing with ansible: "roles: [foo, bar, baz]"18:01
corvuswe know that's an easy system for folks to work with because of the huge success of jjb18:02
pabelangerYah, I don't have a big issue if we do that here in zuul-jobs, just I've stopped using roles myself as I don't want to have a mix of using roles.18:02
corvusif we do the same with zuul's ansible, we make it easier for folks who are new to ansible to use the system.18:02
pabelangerI can push up a patch in a few minutes18:02
corvuspabelanger: thanks.  i think it's a really good guideline (not strict rule, obviously there are times where include_role will be appropriate) for this repo.18:03
corvusand simple jobs like this are going to be the first ones that people look at.18:03
corvuspabelanger, clarkb: i could use more eyes on a problem that has me stumped.... the condition for this task was false: http://logs.openstack.org/57/629957/5/gate/system-config-upload-image-gitea/830ea25/ara-report/file/3e287763-fe4c-457c-8152-407c86075860/18:09
corvushere's the job definition: http://git.openstack.org/cgit/openstack-infra/system-config/tree/.zuul.yaml#n11818:09
corvuswhy wouldn't credentials be defined?18:10
openstackgerritPaul Belanger proposed openstack-infra/zuul master: Update docs since zuul-web requires zookeeper  https://review.openstack.org/63036518:12
corvus(one dead-end i just chased down was the fact that the secrets dict wasn't a list -- it turns out that's okay and zuul ends up with the same data either way)18:12
clarkbis our gate pipelien configured properly to allow post review secrets?18:13
pabelangerlooking18:13
mrhillsmansr.ht was mentioned yesterday, anyone seen https://www.phacility.com/phabricator/ - and if so, any thoughts on why not it? there are plenty of reasons why on the site :)18:15
clarkbmrhillsman: why not phabricator? we actually ran one as a poc. Its a giant ball of php without a proper api and it is hard to turn off the features you don't want18:16
openstackgerritMerged openstack-infra/zuul-jobs master: Add validate-dco-license role  https://review.openstack.org/62956518:17
clarkbwe specifically were interested int he bug tracker but turning everything off that wasn't the bug tracker was difficult iirc18:17
mrhillsman++18:18
clarkbalso rather than writing tools against a proper api everything wanted to be a plugin which meant writing php which we weren't interested in at the time18:18
clarkbother than that, I know wikimedia is transitioning to it (though still using gerrit for some stuff)18:18
mrhillsmani ran across it just before christmas and thought i would ask18:19
clarkbit also has a talk like a pirate mode that I think some people enjoy :)18:19
mrhillsmanas it seemed like was mentioned about sr.ht to be doing the same/similar to opendev18:19
mrhillsmanlol18:20
openstackgerritMerged openstack-infra/zuul-jobs master: Add dco-license job  https://review.openstack.org/63030218:20
clarkbre sr.ht my sense of it was that they are targetting a specific audience that is somewhat different than ours. Its code review over email and test jobs running on archlinux :)18:20
mrhillsmanwhat was interesting was - Supports Git, Mercurial, and SVN - as i wondered about opendev18:20
clarkbI don't think that is bad, but it is quite a bit different in its functional processes18:21
clarkbmrhillsman: ya with phabricator you don't review commits, you review diffs iirc. So really should support bzr and anything else too18:21
clarkbthis is good in that you can support many vcs systems easily, but then you lose some of the features you can buidl on top of native vcs support18:22
clarkb(its a tradeoff)18:22
mrhillsmanah ok, i see that18:23
openstackgerritPaul Belanger proposed openstack-infra/zuul-jobs master: Simplify dco-license job playbook  https://review.openstack.org/63036918:25
clarkbalso I've written down somewhere "emphasis on scalable flexible CI"18:25
clarkbI think for us thats a big part of what we intend to do18:25
pabelangerclarkb: corvus: ^dco-license playbook update18:25
clarkbpabelanger: +2 thanks18:25
mrhillsmanmakes sense18:25
clarkbmrhillsman: another thing that I think sets apart what we are trying to do is we want to run this as community infrastructure. Enable our users to be involved inthe running of services too.18:31
clarkbthat said I think variety here is a good thing as there are a number of workflows supported and no one does all the workflows18:31
mrhillsmanagreed and i also think the microservices vs monolith approach is appropriate to this end18:32
mrhillsmanif i just want zuul and not etherpad18:32
fungiwell, as was also pointed out sh.rt is aiming to be a for-pay-only service after the current alpha test18:34
fungigranted everyone needs some sort of funding model18:34
mrhillsmanyeah i saw that interesting faq18:34
fungii don't really see them as competition for opendev, but if there are interesting applications they end up building that we like i don't see why we couldn't run copies of them in opendev18:35
mrhillsman^18:35
fungibut it definitely does seem like they're bent on reimplementing the entire stack of tools rather than reusing some already useful things out there18:36
fungi(again, probably because of the desire to have browser-based interfaces with no javascript)18:36
mrhillsman"I am only accountable to you, and my only concern is with making the service as useful as possible for you. sr.ht's business model is designed explicitly to prioritize user needs over investor needs."18:37
mrhillsmanthis part of it throws me a bit and other ways of stating this i have seen open source projects use18:37
mrhillsmanusers are investors in the model ;)18:38
mrhillsmancould probably do better just dropping the comparisons in that section completely18:38
openstackgerritMerged openstack-infra/system-config master: Remove 'openstack-gantt' from statusbot monitoring  https://review.openstack.org/63032319:06
openstackgerritMerged openstack-infra/system-config master: Remove meeting-5 and meeting-cp from meetbot  https://review.openstack.org/62943919:32
pabelangerclarkb: fungi: I19:48
pabelangererr19:48
pabelangerclarkb: fungi: I'm now using dco-license job from zuul-jobs for ansible-network, kata should also be able to now19:48
fungithanks again pabelanger!!!19:48
pabelangernp! happy to help them19:48
openstackgerritMerged openstack-infra/system-config master: Update doc for ptgbot to match current behavior  https://review.openstack.org/62686919:49
fungii think this has gone a long way to showing them 1. how simple zuul jobs are to write, and 2. that zuul's standard library is a great place to collaborate on common job needs like this, and has a lot already in there to build on too19:49
corvusfungi: \o/19:49
corvusclarkb: yes, the gate pipeline has post-review set19:50
corvusoh i see the prbolme19:52
pabelangercorvus: where is images defined?19:53
corvuspabelanger: http://git.openstack.org/cgit/openstack-infra/system-config/tree/.zuul.yaml#n10819:53
corvusbut the problem is that the secret isn't available to the parent19:53
pabelangerthanks!19:53
pabelangerah19:54
openstackgerritMerged openstack-infra/zuul-jobs master: Simplify dco-license job playbook  https://review.openstack.org/63036920:05
openstackgerritMerged openstack-infra/zuul master: Update docs since zuul-web requires zookeeper  https://review.openstack.org/63036520:05
openstackgerritJames E. Blair proposed openstack-infra/system-config master: Fix docker image builds  https://review.openstack.org/63035120:06
corvusclarkb, fungi, mordred: ^ i think that should do it20:07
mordredcorvus: lgtm20:11
*** openstack has joined #opendev20:16
*** ChanServ sets mode: +o openstack20:16
*** corvus is now known as thecount21:02
*** thecount is now known as corvus21:02
openstackgerritColleen Murphy proposed openstack-infra/system-config master: Update review-dev.o.o to puppet 4  https://review.openstack.org/62966721:06
openstackgerritColleen Murphy proposed openstack-infra/system-config master: Upgrade all dev servers to puppet 4  https://review.openstack.org/63039121:06
openstackgerritClark Boylan proposed openstack-infra/system-config master: Test new Gerrit 2.13.12 war on review-dev  https://review.openstack.org/63039221:07
openstackgerritColleen Murphy proposed openstack-infra/system-config master: Update etherpad-dev.o.o to puppet 4  https://review.openstack.org/62966721:13
openstackgerritColleen Murphy proposed openstack-infra/system-config master: Upgrade all dev servers to puppet 4  https://review.openstack.org/63039121:13
openstackgerritMerged openstack-infra/system-config master: Fix docker image builds  https://review.openstack.org/63035121:13
openstackgerritDavid Shrewsbury proposed openstack-infra/nodepool master: Add a timeout for the image build  https://review.openstack.org/62992321:14
openstackgerritMerged openstack-infra/storyboard-webclient master: Change openstack-dev to openstack-discuss  https://review.openstack.org/62538821:16
openstackgerritJames E. Blair proposed openstack-infra/project-config master: Also report promote results to gerrit  https://review.openstack.org/63039321:20
openstackgerritJames E. Blair proposed openstack-infra/system-config master: Fix image promote playbook  https://review.openstack.org/63039521:24
openstackgerritMerged openstack-infra/project-config master: Also report promote results to gerrit  https://review.openstack.org/63039321:37
openstackgerritJames E. Blair proposed openstack-infra/system-config master: Fix image promote playbook  https://review.openstack.org/63039521:39
openstackgerritColleen Murphy proposed openstack-infra/system-config master: Update etherpad-dev.o.o to puppet 4  https://review.openstack.org/62966721:46
openstackgerritColleen Murphy proposed openstack-infra/system-config master: Upgrade all dev servers to puppet 4  https://review.openstack.org/63039121:46
openstackgerritJames E. Blair proposed openstack-infra/system-config master: Increase number of k8s nodes to 4  https://review.openstack.org/62803121:56
corvusmordred: what's the helm stuff in 626054?  was that just to generate the files in that change?21:58
mordredcorvus: yah. and is there as documentation of where those files came from22:01
openstackgerritMerged openstack-infra/system-config master: Test new Gerrit 2.13.12 war on review-dev  https://review.openstack.org/63039222:04
openstackgerritMerged openstack-infra/zuul-jobs master: Add role to move docs and artifacts to log root  https://review.openstack.org/62957122:07
openstackgerritMerged openstack-infra/system-config master: Fix image promote playbook  https://review.openstack.org/63039522:30
openstackgerritJames E. Blair proposed openstack-infra/system-config master: Build a gitea-init image  https://review.openstack.org/63040422:38
openstackgerritJames E. Blair proposed openstack-infra/system-config master: Use the v1.6.0 template instead of master  https://review.openstack.org/62993622:38
openstackgerritJames E. Blair proposed openstack-infra/system-config master: Update to gitea v1.6.3  https://review.openstack.org/62994222:38
corvusmordred: i abandoned your change https://review.openstack.org/629931 due to https://review.openstack.org/63040422:39
openstackgerritJames E. Blair proposed openstack-infra/system-config master: Build a gitea-init image  https://review.openstack.org/63040422:41
mordredcorvus: I have +2d 63040422:53
openstackgerritJames E. Blair proposed openstack-infra/system-config master: WIP: run a gerrit container  https://review.openstack.org/63040623:04
openstackgerritMerged openstack-infra/system-config master: Build a gitea-init image  https://review.openstack.org/63040423:48
*** pabelanger has quit IRC23:58

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