Saturday, 2020-04-04

*** cdearborn has quit IRC00:35
*** mwhahaha has quit IRC00:45
*** mwhahaha has joined #zuul00:48
*** jtanner has quit IRC00:52
*** tributarian has quit IRC00:52
*** tributarian has joined #zuul00:54
*** Goneri has quit IRC00:54
*** jtanner has joined #zuul00:55
*** mwhahaha has quit IRC00:57
*** mwhahaha has joined #zuul00:59
*** jtanner has quit IRC01:08
*** jtanner has joined #zuul01:11
*** saneax has joined #zuul01:29
*** swest has quit IRC01:31
*** swest has joined #zuul01:47
*** jamesmcarthur has quit IRC02:37
*** jamesmcarthur has joined #zuul02:37
*** jamesmcarthur has quit IRC02:43
*** saneax has quit IRC02:44
*** jamesmcarthur has joined #zuul02:46
*** zxiiro has quit IRC03:55
*** igordc has quit IRC04:23
*** evrardjp has quit IRC04:36
*** evrardjp has joined #zuul04:37
*** bhavikdbavishi has joined #zuul04:39
*** rfolco has quit IRC04:53
*** bhavikdbavishi has quit IRC05:38
*** jamesmcarthur has quit IRC05:52
*** jamesmcarthur has joined #zuul06:01
*** y2kenny has quit IRC06:07
*** jamesmcarthur has quit IRC06:08
*** bhavikdbavishi has joined #zuul06:16
*** bhavikdbavishi has quit IRC06:30
*** sgw has quit IRC06:44
*** bhavikdbavishi has joined #zuul07:30
*** bhavikdbavishi has quit IRC07:41
*** dpawlik has joined #zuul08:07
*** dpawlik has quit IRC08:12
*** bhavikdbavishi has joined #zuul09:03
*** bhavikdbavishi1 has joined #zuul09:06
*** bhavikdbavishi has quit IRC09:08
*** bhavikdbavishi1 is now known as bhavikdbavishi09:08
*** tosky has joined #zuul10:48
*** bhavikdbavishi has quit IRC10:58
*** bhavikdbavishi has joined #zuul10:59
*** bhavikdbavishi has quit IRC11:03
*** bhavikdbavishi has joined #zuul11:21
*** bhavikdbavishi1 has joined #zuul11:24
*** bhavikdbavishi has quit IRC11:25
*** bhavikdbavishi1 is now known as bhavikdbavishi11:25
*** bhavikdbavishi has quit IRC12:32
*** zxiiro has joined #zuul13:54
*** Goneri has joined #zuul14:23
AJaegermnaser, tobiash: https://review.opendev.org/717443 still has many " unable to map line for file comments: ", should those really be visible?14:31
AJaegershould we exclude ".tox"?14:32
AJaegerOr files not in git?14:32
openstackgerritMohammed Naser proposed zuul/zuul-jobs master: install-yarn: always install  https://review.opendev.org/71737514:52
openstackgerritMohammed Naser proposed zuul/zuul-jobs master: install-yarn: always install  https://review.opendev.org/71737515:03
openstackgerritMohammed Naser proposed zuul/zuul-jobs master: install-nodejs: test on all platforms  https://review.opendev.org/71747015:03
*** mordred has quit IRC15:20
*** mordred has joined #zuul15:20
openstackgerritAndreas Jaeger proposed zuul/zuul-jobs master: tox-parse_output: Ignore .tox directory  https://review.opendev.org/71748615:26
AJaegermnaser, mordred, tobiash, what do you think? ^15:27
mordredAJaeger: lgtm15:30
AJaegerthanks, mordred15:31
mnaserAJaeger: a test for that would be nice if you can :p15:43
mnaserit seems like it would be okay though, but i think we'd probably feel more confidnet (or maybe a depends-on the change that was being noisy?)15:43
*** rfolco has joined #zuul15:59
*** rfolco has quit IRC16:04
*** bhavikdbavishi has joined #zuul16:10
AJaegermnaser: pushing one now16:13
*** bhavikdbavishi has quit IRC16:16
*** bhavikdbavishi has joined #zuul16:17
AJaegermnaser: compare https://review.opendev.org/#/c/717493/ and https://review.opendev.org/#/c/717479/16:26
mnaserAJaeger: +3 :)16:30
AJaegerthanks16:31
*** evrardjp has quit IRC16:36
*** evrardjp has joined #zuul16:37
openstackgerritMohammed Naser proposed zuul/zuul-jobs master: DNM: refactor to ensure-repositories  https://review.opendev.org/71750316:47
AJaegermnaser: base that on top of the stack from https://review.opendev.org/#/c/716663/6 - or wait until we merge that...16:55
mnaserAJaeger: agreed, i'll do so so we avoid a bunch of merge conflicts16:56
AJaegerthanks16:56
openstackgerritMohammed Naser proposed zuul/zuul-jobs master: DNM: refactor to ensure-repositories  https://review.opendev.org/71750317:00
mnaserthis should hopefully help us more easily do cross-platforms jobs17:02
mnaserinstead of repeating the pattern everywhere17:02
openstackgerritMerged zuul/zuul-jobs master: tox-parse_output: Ignore .tox directory  https://review.opendev.org/71748617:04
*** bhavikdbavishi has quit IRC17:10
openstackgerritMohammed Naser proposed zuul/zuul-jobs master: DNM: refactor to ensure-repositories  https://review.opendev.org/71750317:15
*** bhavikdbavishi has joined #zuul17:22
openstackgerritMohammed Naser proposed zuul/zuul-jobs master: DNM: refactor to ensure-repositories  https://review.opendev.org/71750317:58
openstackgerritMohammed Naser proposed zuul/zuul-jobs master: DNM: refactor to ensure-repositories  https://review.opendev.org/71750318:08
*** igordc has joined #zuul18:09
*** bhavikdbavishi has quit IRC18:19
openstackgerritMohammed Naser proposed zuul/zuul-jobs master: DNM: refactor to ensure-repositories  https://review.opendev.org/71750318:24
mnaserwelp, we've been installing docker with the wrong key this whole time18:26
mordredmnaser: that doesn't seem awesome :)18:32
mordredmnaser: I like where you're going with that18:36
openstackgerritMohammed Naser proposed zuul/zuul-jobs master: DNM: refactor to ensure-repositories  https://review.opendev.org/71750318:39
mnasermordred: thanks, it seems like we have a lot of repeating logic and as i was trying to get the install-nodejs role to start supporting other platforms, i realized i was just going to copy pasta all this18:40
mnaserso include_vars per os + include that role and you're good to go18:40
mordred++18:41
openstackgerritMohammed Naser proposed zuul/zuul-jobs master: DNM: refactor to ensure-repositories  https://review.opendev.org/71750318:42
openstackgerritMohammed Naser proposed zuul/zuul-jobs master: DNM: refactor to ensure-repositories  https://review.opendev.org/71750318:52
openstackgerritMohammed Naser proposed zuul/zuul-jobs master: DNM: refactor to ensure-repositories  https://review.opendev.org/71750319:07
openstackgerritMohammed Naser proposed zuul/zuul-jobs master: ensure-repositories: Add role  https://review.opendev.org/71750719:36
openstackgerritMohammed Naser proposed zuul/zuul-jobs master: Rename install-docker to ensure-docker for consistency  https://review.opendev.org/71666319:41
openstackgerritMohammed Naser proposed zuul/zuul-jobs master: Rename install-kubernetes to ensure-kubernetes for consistency  https://review.opendev.org/71666719:41
openstackgerritMohammed Naser proposed zuul/zuul-jobs master: Rename install-openshift to ensure-openshift for consistency  https://review.opendev.org/71667519:41
openstackgerritMohammed Naser proposed zuul/zuul-jobs master: Rename install-podman to ensure-podman for consistency  https://review.opendev.org/71668219:41
openstackgerritMohammed Naser proposed zuul/zuul-jobs master: Rename install-devstack to ensure-devstack for consistency  https://review.opendev.org/71668519:41
openstackgerritMohammed Naser proposed zuul/zuul-jobs master: Rename install-javascript-packages to ensure-javascript-packages for consistency  https://review.opendev.org/71668719:41
openstackgerritMohammed Naser proposed zuul/zuul-jobs master: Rename install-nodejs to ensure-nodejs for consistency  https://review.opendev.org/71669219:41
openstackgerritMohammed Naser proposed zuul/zuul-jobs master: Rename install-yarn to ensure-yarn for consistency  https://review.opendev.org/71669319:41
openstackgerritMohammed Naser proposed zuul/zuul-jobs master: Rename install-pdk-dependencies to ensure-pdk-dependencies for consistency  https://review.opendev.org/71669519:41
openstackgerritMohammed Naser proposed zuul/zuul-jobs master: Rename install-if-python to ensure-if-python for consistency  https://review.opendev.org/71669819:41
openstackgerritMohammed Naser proposed zuul/zuul-jobs master: ensure-repositories: Add role  https://review.opendev.org/71750719:41
openstackgerritMohammed Naser proposed zuul/zuul-jobs master: ensure-nodejs: refactor to use ensure-repositories  https://review.opendev.org/71750819:52
openstackgerritMohammed Naser proposed zuul/zuul-jobs master: install-yarn: always install  https://review.opendev.org/71737519:58
openstackgerritMohammed Naser proposed zuul/zuul-jobs master: ensure-yarn: refactor to use ensure-repositories  https://review.opendev.org/71750919:58
openstackgerritMohammed Naser proposed zuul/zuul-jobs master: ensure-kubernetes: refactor to use ensure-repositories  https://review.opendev.org/71751020:00
openstackgerritMohammed Naser proposed zuul/zuul-jobs master: ensure-podman: refactor to use ensure-repositories  https://review.opendev.org/71751120:05
openstackgerritMohammed Naser proposed zuul/zuul-jobs master: ensure-docker: refactor to use ensure-repositories  https://review.opendev.org/71751220:13
*** mgoddard has quit IRC20:14
openstackgerritMohammed Naser proposed zuul/zuul-jobs master: cleanup: move tests to use ensure-repositories  https://review.opendev.org/71751320:29
*** mgoddard has joined #zuul21:23
*** tosky has quit IRC23:00

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