Friday, 2020-01-17

*** ekultails has quit IRC00:01
rdogerritrdo-trunk created rdoinfo master: Bump rdoinfo train tags to latest upper-constraints  https://review.rdoproject.org/r/2459000:03
rdogerritrdo-trunk created rdoinfo master: Bump rdoinfo ussuri-uc tags to latest upper-constraints  https://review.rdoproject.org/r/2459100:05
rdogerritrdo-trunk proposed rdoinfo master: Promote CBS tags update for stein-testing  https://review.rdoproject.org/r/2454600:05
rdogerritrdo-trunk created rdoinfo master: Promote CBS tags update for train-testing  https://review.rdoproject.org/r/2459200:07
rdogerritSagi Shnaidman proposed rdo-infra/ci-config master: Add internal projects to zuul builds  https://review.rdoproject.org/r/2458900:16
rdogerritMerged rdo-infra/ci-config master: Return nginx.conf for local run  https://review.rdoproject.org/r/2458800:22
*** tkajinam_ has quit IRC00:43
*** tkajinam has joined #rdo00:43
*** tkajinam_ has joined #rdo01:00
*** tkajinam has quit IRC01:00
*** bnemec has quit IRC01:31
rdogerritMerged rdo-infra/ci-config master: Add internal projects to zuul builds  https://review.rdoproject.org/r/2458902:47
*** ktdreyer has quit IRC02:48
*** logan- has quit IRC02:48
*** logan_ has joined #rdo02:50
*** logan_ is now known as logan-02:50
*** osmanlicilegi has quit IRC02:56
*** osmanlicilegi has joined #rdo02:56
*** rfolco has joined #rdo02:57
*** rfolco has quit IRC03:02
*** apetrich has quit IRC03:13
*** rh-jelabarre has quit IRC03:22
*** brault has joined #rdo03:29
*** rlandy has quit IRC04:06
*** eggs has quit IRC04:12
rdogerritrdo-trunk created openstack/hardware-distgit rpm-master: python-hardware: failed to build 7c4c388  https://review.rdoproject.org/r/2459304:20
*** raukadah is now known as chandankumar04:46
*** udesale has joined #rdo04:49
*** ykarel|afk is now known as ykarel04:50
*** mcornea has joined #rdo04:51
rdogerritChandan Kumar created rdo-jobs master: [DNM] https://review.opendev.org/#/c/703014/  https://review.rdoproject.org/r/2459404:56
*** mcornea has quit IRC04:57
*** zaneb has quit IRC05:08
*** jdandrea has quit IRC05:23
*** rpittau|afk has quit IRC05:34
*** rpittau|afk has joined #rdo05:34
rdogerritYatin Karel proposed openstack/hardware-distgit rpm-master: Remove hardware-cardiff from package  https://review.rdoproject.org/r/2457805:35
*** surpatil has joined #rdo05:48
*** soniya29 has joined #rdo05:49
*** udesale has quit IRC05:56
*** udesale has joined #rdo05:56
rdogerritMerged rdoinfo master: Pin neutron-lib and python-hardware to py2 compatible tag  https://review.rdoproject.org/r/2456505:57
rdogerritrdo-trunk created openstack/hardware-distgit queens-rdo: python-hardware-0.23.0-1  https://review.rdoproject.org/r/2459506:00
rdogerritrdo-trunk created openstack/hardware-distgit rocky-rdo: python-hardware-0.23.0-1  https://review.rdoproject.org/r/2459606:00
*** soniya29 has quit IRC06:00
rdogerritrdo-trunk created openstack/hardware-distgit stein-rdo: python-hardware-0.23.0-1  https://review.rdoproject.org/r/2459706:01
*** marios has joined #rdo06:05
*** aufi has joined #rdo06:05
rdogerritChandan Kumar proposed rdo-jobs master: [DNM] https://review.opendev.org/#/c/703014/  https://review.rdoproject.org/r/2459406:08
*** bandini has joined #rdo06:08
rdogerritYatin Karel created openstack/hardware-distgit rpm-master: Dummy commit to rebuild python-hardware  https://review.rdoproject.org/r/2459806:09
rdogerritMerged openstack/hardware-distgit rpm-master: Dummy commit to rebuild python-hardware  https://review.rdoproject.org/r/2459806:19
rdogerritMerged openstack/hardware-distgit stein-rdo: python-hardware-0.23.0-1  https://review.rdoproject.org/r/2459706:20
rdogerritMerged openstack/hardware-distgit queens-rdo: python-hardware-0.23.0-1  https://review.rdoproject.org/r/2459506:21
rdogerritMerged openstack/hardware-distgit rocky-rdo: python-hardware-0.23.0-1  https://review.rdoproject.org/r/2459606:25
rdogerritYatin Karel proposed rdoinfo master: Bump rdoinfo train tags to latest upper-constraints  https://review.rdoproject.org/r/2459006:31
rdogerritYatin Karel proposed rdoinfo master: Bump rdoinfo ussuri-uc tags to latest upper-constraints  https://review.rdoproject.org/r/2459106:32
rdogerritYatin Karel proposed rdoinfo master: Bump rdoinfo ussuri-uc tags to latest upper-constraints  https://review.rdoproject.org/r/2459106:34
*** SurajPatil has joined #rdo06:36
*** tkajinam__ has joined #rdo06:37
*** SurajPatil has quit IRC06:37
*** tkajinam_ has quit IRC06:39
*** numans has quit IRC06:39
*** surpatil has quit IRC06:40
*** tkajinam_ has joined #rdo06:42
*** soniya29 has joined #rdo06:42
*** tkajinam__ has quit IRC06:44
*** surpatil has joined #rdo06:49
*** jaosorior has joined #rdo06:51
*** jfrancoa has joined #rdo06:52
*** lmiccini has joined #rdo07:04
rdogerritMerged rdoinfo master: Bump rdoinfo train tags to latest upper-constraints  https://review.rdoproject.org/r/2459007:06
*** zaneb has joined #rdo07:09
*** chem_ has joined #rdo07:13
*** jlibosva has joined #rdo07:15
*** chem has quit IRC07:18
*** ramishra has quit IRC07:18
*** avivgt has quit IRC07:18
*** chasmo77 has quit IRC07:18
*** jrist has quit IRC07:19
*** migarcia has quit IRC07:19
*** lhinds has quit IRC07:19
*** tbarron has quit IRC07:19
*** tobias-urdin has quit IRC07:19
*** ianw has quit IRC07:19
*** mjturek has quit IRC07:19
*** zigo has quit IRC07:19
*** EmilienM has quit IRC07:19
*** rnoriega_ has quit IRC07:19
*** brtknr has quit IRC07:19
*** dmacpher has joined #rdo07:20
*** openstackstatus has quit IRC07:20
*** dmacpher_ has quit IRC07:22
*** slaweq has joined #rdo07:35
*** ykarel is now known as ykarel|lunch07:45
*** brault has quit IRC07:56
*** apevec has joined #rdo07:59
*** ramishra has joined #rdo08:05
*** avivgt has joined #rdo08:05
*** jrist has joined #rdo08:05
*** migarcia has joined #rdo08:05
*** tbarron has joined #rdo08:05
*** lhinds has joined #rdo08:05
*** brtknr has joined #rdo08:05
*** tobias-urdin has joined #rdo08:05
*** ianw has joined #rdo08:05
*** mjturek has joined #rdo08:05
*** zigo has joined #rdo08:05
*** EmilienM has joined #rdo08:05
*** rnoriega_ has joined #rdo08:05
*** maciejjozefczyk has joined #rdo08:06
*** jcapitao has joined #rdo08:07
*** zzzeek has quit IRC08:08
*** nijaba has quit IRC08:08
*** cylopez has joined #rdo08:09
*** rodolof has quit IRC08:09
*** tkajinam_ has quit IRC08:11
*** zzzeek has joined #rdo08:11
*** nijaba has joined #rdo08:11
*** rodolof has joined #rdo08:12
*** tesseract has joined #rdo08:16
*** tmazur has joined #rdo08:21
*** holser has joined #rdo08:27
*** rpittau|afk is now known as rpittau08:28
*** tosky has joined #rdo08:29
*** holser has quit IRC08:31
*** ykarel|lunch is now known as ykarel08:31
*** holser has joined #rdo08:32
*** iurygregory has joined #rdo08:32
*** dmellado has quit IRC08:34
*** dmellado has joined #rdo08:35
*** ccamacho has joined #rdo08:35
*** gfidente has joined #rdo08:38
*** amoralej|off is now known as amoralej08:40
amoralejykarel, centos8 images is fixed and I see weirdo jobs passing again, right?08:42
ykarelamoralej, yes those are passing08:43
ykarelamoralej, now new issue with centos7 image,08:43
ykareli noticed that in tripleo jobs08:43
amoralejwhat's the new issue?08:43
ykarelamoralej, corrupted cirros image08:44
ykarelso tempest was failing08:44
amoralejthat's in upstream jobs or the ones in SF zuul?08:44
ykarelamoralej, sf zuul08:44
ykarelamoralej, https://review.rdoproject.org/r/#/c/24546/08:44
ykareland similar train08:44
ykarelalso periodic job failed08:44
amoralejare sf guys aware of it?08:46
amoralej Could not open '/var/lib/cinder/conversion/tmpfizqaKstandalone@tripleo_ceph': Image is not in qcow2 format08:47
amoralejmmm08:47
amoralejyeah, looks like corrupted image08:47
ykarelamoralej, yes08:48
*** jpena|off is now known as jpena08:48
ykarelamoralej, i confirmed ^^ by holding node for the job08:48
amoralejjpena, ^08:48
amoralejcould centos7 image be rebuilt in sf nodepool?08:48
jpenawhich one, upstream-centos-7 ?08:49
ykarelamoralej, http://paste.openstack.org/show/788516/08:49
jpenaLet me check08:49
amoralejcached cirros image is corrupted08:49
amoralejjpena, btw, centos8 image is fixed08:49
ykareljpena,  i just asked in sf-ops for this08:49
*** ralonsoh has joined #rdo08:50
*** iurygregory_ has joined #rdo09:01
*** iurygregory has quit IRC09:03
*** rcernin_ has joined #rdo09:04
*** rcernin has quit IRC09:04
*** jpich has joined #rdo09:09
*** ktibi has joined #rdo09:17
*** xek has joined #rdo09:21
*** ktibi has quit IRC09:29
*** chem_ has quit IRC09:36
*** derekh has joined #rdo09:36
*** chem_ has joined #rdo09:37
*** apetrich has joined #rdo09:47
*** paramite has joined #rdo09:48
*** Jeffrey4l has quit IRC09:50
*** alexmcleod has joined #rdo09:56
amoralejjpena, ykarel wrt https://lists.rdoproject.org/pipermail/users/2020-January/000660.html10:00
amoralejwdyt about doing some puppet promotion for centos8-train until we have oooq promotion on it?10:01
amoraleji think it'd be good for community users needing train on centos8 as starlingx10:01
jpenaamoralej: sounds fine, we could at least have some sanity check10:01
ykarelamoralej, ok for me10:01
amoralejack, i'll reply10:02
*** dtantsur|afk is now known as dtantsur10:06
*** aufi has quit IRC10:07
rdogerritAlfredo Moralejo created rdo-infra/ci-config master: Add puppet promotion pipeline for centos8-train  https://review.rdoproject.org/r/2459910:17
*** ccamposr__ has quit IRC10:18
*** Jeffrey4l has joined #rdo10:21
*** jaosorior has quit IRC10:29
*** ykarel is now known as ykarel|afk10:31
*** rcernin_ has quit IRC10:34
rdogerritSorin Sbarnea (zbr) proposed rdo-jobs master: rdo-tox-molecule: switch nodeset to cloud-centos-8  https://review.rdoproject.org/r/2458410:36
*** apetrich has quit IRC10:38
*** apetrich has joined #rdo10:40
*** ccamposr has joined #rdo10:41
*** fmount|afk is now known as fmount10:47
rdogerritSorin Sbarnea (zbr) proposed rdo-jobs master: rdo-tox-molecule: switch nodeset to cloud-centos-8  https://review.rdoproject.org/r/2458410:50
*** soniya29 has quit IRC11:01
rdogerritSorin Sbarnea (zbr) proposed rdo-jobs master: rdo-tox-molecule: switch nodeset to cloud-centos-8  https://review.rdoproject.org/r/2458411:03
rdogerritAlfredo Moralejo proposed rdo-infra/ci-config master: Add puppet promotion pipeline for centos8-train  https://review.rdoproject.org/r/2459911:12
*** bhagyashris has joined #rdo11:16
*** ccamel has joined #rdo11:16
*** aufi has joined #rdo11:16
*** camelCaser has quit IRC11:16
*** soniya29 has joined #rdo11:27
*** surpatil has quit IRC11:33
*** ykarel|afk is now known as ykarel11:59
*** pcaruana has joined #rdo11:59
rdogerritSagi Shnaidman created rdo-infra/ci-config master: Read password before starting Grafana services  https://review.rdoproject.org/r/2460011:59
*** sshnaidm|afk is now known as sshnaidm|off12:00
*** zbr|rover is now known as zbr|drover12:09
*** dsneddon has quit IRC12:12
*** jcapitao is now known as jcapitao|afk12:13
rdogerritSorin Sbarnea (zbr) proposed rdo-jobs master: rdo-tox-molecule: switch nodeset to cloud-centos-8  https://review.rdoproject.org/r/2458412:13
rdogerritSorin Sbarnea (zbr) proposed rdo-jobs master: rdo-tox-molecule: switch nodeset to cloud-centos-8  https://review.rdoproject.org/r/2458412:13
rdogerritMerged rdo-infra/ci-config master: Read password before starting Grafana services  https://review.rdoproject.org/r/2460012:17
*** rfolco has joined #rdo12:27
*** akahat has joined #rdo12:32
*** eharney has joined #rdo12:37
*** iurygregory_ is now known as iurygregory12:40
*** dmellado has quit IRC12:41
*** dmellado has joined #rdo12:44
*** udesale_ has joined #rdo12:45
*** jpena is now known as jpena|lunch12:46
*** dsneddon has joined #rdo12:48
*** marios is now known as marios|call12:48
*** udesale has quit IRC12:48
rdogerritMerged rdo-infra/ci-config master: Add puppet promotion pipeline for centos8-train  https://review.rdoproject.org/r/2459912:49
*** ykarel is now known as ykarel|afk12:50
*** rh-jelabarre has joined #rdo12:51
*** jdandrea has joined #rdo12:55
*** dsneddon has quit IRC12:56
*** jdandrea has quit IRC12:57
*** marios|call is now known as marios12:58
*** dsneddon has joined #rdo13:06
*** rlandy has joined #rdo13:11
*** aufi has quit IRC13:17
*** jcapitao|afk is now known as jcapitao13:19
*** imcleod has joined #rdo13:25
*** gfidente has quit IRC13:27
*** derekh has quit IRC13:32
*** jpena|lunch is now known as jpena13:39
*** gfidente has joined #rdo13:42
*** pcaruana has quit IRC13:45
*** soniya29 has quit IRC13:47
*** ykarel|afk is now known as ykarel13:51
rdogerritSagi Shnaidman proposed rdo-jobs master: WIP: rhel-8 molecule tripleo-ansible job  https://review.rdoproject.org/r/2367013:51
*** leanderthal has joined #rdo13:53
rdogerritDaniel Alvarez proposed rdoinfo master: Bump OVS/OVN to 2.12 in RDO Train  https://review.rdoproject.org/r/2396014:01
*** derekh has joined #rdo14:02
*** nweinber__ has joined #rdo14:11
*** ekultails has joined #rdo14:16
*** pcaruana has joined #rdo14:22
*** amoralej is now known as amoralej|lunch14:26
*** aedc has joined #rdo14:36
*** mcornea has joined #rdo14:39
*** jdandrea has joined #rdo14:53
*** aedc has quit IRC14:55
*** marios is now known as marios|call14:57
rdogerritSorin Sbarnea (zbr) proposed rdo-jobs master: rdo-tox-molecule: switch nodeset to cloud-centos-8  https://review.rdoproject.org/r/2458415:02
*** zbr|drover has quit IRC15:08
*** lmiccini has quit IRC15:10
*** amoralej|lunch is now known as amoralej15:12
*** lmiccini has joined #rdo15:12
*** zbr has joined #rdo15:16
*** zbr is now known as zbr|drover15:16
*** jpich has quit IRC15:39
*** jpich has joined #rdo15:39
*** jcapitao is now known as jcapitao|afk15:48
*** aedc has joined #rdo15:52
*** leanderthal has quit IRC15:52
*** aufi has joined #rdo15:58
*** cylopez has quit IRC15:59
*** udesale_ has quit IRC16:01
*** udesale has joined #rdo16:03
*** jcapitao|afk is now known as jcapitao16:03
*** migarcia is now known as migarcia|away16:06
*** migarcia|away is now known as migarcia16:06
*** udesale has quit IRC16:10
rdogerritSorin Sbarnea (zbr) proposed rdo-jobs master: rdo-tox-molecule: switch nodeset to cloud-centos-8  https://review.rdoproject.org/r/2458416:14
zbr|drovertristanC: hi! do you have few minutes to chat about the two upstream patches related to docker?16:16
*** slaweq has quit IRC16:16
tristanCzbr|drover: well i'm not familiar with docker, i only use podman... but maybe i can help? what's up16:17
zbr|droverone simple is revoke-sudo chokes on rdo because our username is not called zuul, so I proposed https://review.opendev.org/#/c/703065/816:18
*** bandini has quit IRC16:18
*** marios|call is now known as marios16:18
zbr|droverthe more complex one is related to install-docker, which fails on centos-8 because docker-ce repository has some outdated dependencies.16:19
*** maciejjozefczyk has quit IRC16:19
zbr|droverit can install succesfully only with --nobest or by manually pre-installing a specific rpm before trying to install docker itself, fix proposed at https://review.opendev.org/#/c/703053/116:20
zbr|droveryou don't need to know docker, is rpm stuff :D16:20
*** lmiccini has quit IRC16:24
*** marios is now known as marios|out16:28
tristanCzbr|drover: i left new comments on the changes. it would be good to have feedback from the other cores and user affected by the issue you are trying to address.16:28
zbr|droverouch... so you are against because it breaks your hack :D16:30
tristanCzbr|drover: it's not because of the hack, it's because you made revoke-sudo requires SETUID capabilities, and this is going to break user who do not give SETUID to their jobs using revoke-sudo16:31
dmsimardzbr|drover: docker doesn't have a centos8 repo yet, only 716:33
dmsimardhttps://download.docker.com/linux/centos/16:33
dmsimardI actually ran into a similar issue recently16:33
*** marios|out has quit IRC16:33
zbr|drovertristanC: that two tasks cannot fail, the only issue i see there is that it would not work with your hack, but I could adapt the code to keep the hardcoded removal of that file.16:34
zbr|droveri am more worried to make the role behave correctly, as now is not working with the newer centos-8 image16:34
zbr|droverprobably because zuul is part wheel group (i suspect)16:35
zbr|droverif you update the image to mimic the hack, i would not mind as I would have one less blocking patches16:36
tristanCzbr|drover: well, why zuul is part of the wheel group?16:36
zbr|drovertristanC: no idea, I did not add it myself, but I can confirm that my proposed code does work.16:36
zbr|drovertristanC: look at https://review.rdoproject.org/r/#/c/24584/ (molecule job)16:37
*** udesale has joined #rdo16:37
*** ykarel is now known as ykarel|away16:37
zbr|droverif you try it without using the revoke-sudo patch, it will fail at the task that verifies that you no longer have sudo (last one)16:38
zbr|drovermeaning that it removes the file, but that is not enough, sudo still works after.16:38
tristanCzbr|drover: oh, i see, my bad, let me fix the image then16:38
zbr|drovertristanC: thanks. it should buy us more time to properly fix the upstream role16:39
zbr|droverfixing upstream role is our interest, or we will endup drowning in rdo workarounds.16:39
tristanCzbr|drover: i fail to see how zuul is being added to the wheel group, are you sure of that issue?16:39
zbr|drovertristanC: not sure, as I didn't start the image myself, by i only read the logs (look inside json file, contains all details)16:40
zbr|droverthe console is kinda brief, but all 3 tasks are with more detail in json file.16:40
tristanCzbr|drover: i'll rebuild the image with https://softwarefactory-project.io/r/1733716:41
tristanCthat's actually a bug on our side, since we refactored the virt-customize stuff into role, we missed that sudoer filename fix16:42
zbr|droveri don't really know why we didn't use the "API" (gpasswd)16:42
tristanCzbr|drover: build is running here: https://softwarefactory-project.io/nodepool-log/cloud-centos-8-0000000005.log16:48
rdogerritMerged rdoinfo master: Promote CBS tags update for train-testing  https://review.rdoproject.org/r/2459216:49
tristanConce this is completed and updloaded, i think your molecule job will run without depending on https://review.opendev.org/#/c/703065/816:49
zbr|drovertristanC: maybe you also happen to know a trick that I could use to avoid being blocked by the install-docker patch.16:51
tristanCthen for the docker fix, i can't tell what is the best thing to do. I don't mind your change if other install-docker user are ok with it, otherwise i think we should wait for docker to provide a version that works on centos16:51
zbr|droveri tried to run the task in run-pre but discovered that this runs too late, zuul roles being executed before job pre, thus failing.16:52
tristanCzbr|drover: you can work around the issue by providing your own pre-run playbook and make it install the right thing before including the install-docker role16:52
zbr|drovertristanC: we cannot wait, tell this to weshay_16:52
zbr|droverthey knew about this bug since november, Docker Inc has Red Hat stuff at the back-of-back of their priorities....16:53
*** rpittau is now known as rpittau|afk16:53
tristanCzbr|drover: let's backup a bit, why are you using docker on centos?16:53
zbr|drovermost of molecule scenarios are using docker backend for testing ansible code.16:54
tristanCzbr|drover: why molecule doesn't use podman instead?16:54
zbr|droverpodman is not ready to be used as replacement, and I bet the driver author can back my story, sshnaidm|off (sadly off)16:54
tristanCzbr|drover: what is not ready?16:54
zbr|droverin fact even the docker from centos was too old and buggy to be used for testing, and that is what forced us to use the upstream version.16:56
amoralejykarel|away, we have first puppet promotion for centos8-train16:56
tristanCzbr|drover: what bugs are you referring to? I need to step away shortly for lunch, but i'd be happy to look into why can't use podman after16:56
zbr|droverplan is to also enable podman and to switch scenarios by scenario to podman when we see it working. but that scenario by scenario16:57
zbr|droverthe job itself will have to provide both tools/backends.16:57
*** dtantsur is now known as dtantsur|afk16:58
ykarel|awayamoralej, cool16:58
tristanCzbr|drover: if podman can't be used in place of docker on centos, then that's a podman issue we need to fix. so please, let me know what is not working and why the docker version you are installing is better suited for molecule16:58
*** udesale has quit IRC17:00
zbr|drovertristanC: just one example: https://github.com/ansible/molecule/pull/248417:01
zbr|droverand you know very well which version we ship17:01
zbr|droverwe had to disable molecule tests on podman driver, because of that.17:02
weshay|ruckhello17:02
weshay|ruckwhat do I need to know here?17:02
zbr|droverwhen we will have 1.6 we can start testing, but until then, molecule needs a working docker.17:02
zbr|droverweshay|ruck:  tristan was curious why we really need docker for molecule job17:03
rdogerritChandan Kumar created rdo-jobs master: [wip] fs020 with latest master podman  https://review.rdoproject.org/r/2460117:04
*** jfrancoa has quit IRC17:04
*** imcleod has quit IRC17:04
weshay|ruckzbr|drover, like.. docker vs. podman or  containers vs. nodepool node17:04
zbr|drover1st17:05
weshay|ruckwhat is the status of molecule support for podman17:05
zbr|droversadly today sagi is off as he is the area knowledge on this17:05
weshay|rucklast I knew about it.. we were helping that along17:05
weshay|ruckright17:05
*** alexmcleod has quit IRC17:05
weshay|ruckafaik folks.. molecule had some issues when running w/ podman and we working to fix it17:06
zbr|drovermainly molecule does not have an active CI with podman and requires 1.5.1+17:06
weshay|ruckand I'm afaid I don't have all the specifics noted17:06
zbr|droverso use of podman is out of the question for the moment, we need to keep using docker until podman driver becomes a valid replacement17:07
weshay|ruckzbr|drover, last work I see from sshnaidm|off was in october https://github.com/sshnaidm/molecule/commits/master17:07
zbr|droverwhich will happen, I am sure17:07
zbr|droverin fact podman driver also requires ansible 2.9+ if it remember well17:08
weshay|ruckthat merged17:08
sshnaidm|offwhat is the current issue with podman-molecule?17:08
weshay|ruckthere is the man17:09
sshnaidm|offthere are various problems with specific versions of podman and cgroups17:09
weshay|rucksshnaidm|off, are there any remaining issues w/ podman + molecule that you are aware of?17:09
zbr|droversshnaidm|off: hi! i was trying to explain tristan that we still need docker for rdo-tox-molecule17:09
*** jcapitao has quit IRC17:09
*** aufi has quit IRC17:09
sshnaidm|offweshay|ruck, I know the issue with with cgroups v2 only17:09
zbr|drovermaybe tristanC does not know that each molecule scenario/test declares which backend it wants to use, so is not the job that decides what is used, is the tests inside.17:10
weshay|rucktristanC, dmsimard what is the issue w/ also having docker avail?17:10
sshnaidm|offzbr|drover, can we at least try to run it with podman? to know how much it's bad/good17:10
sshnaidm|offif we don't run it, we'll never have it working17:11
zbr|droversshnaidm|off: weshay|ruck : read this to underatand what we are debating about https://review.opendev.org/#/c/703053/117:12
zbr|droveri have docker-ce on centos-8, since ~2 mo ago, works fine. we just need to hack the install a little bit.17:13
zbr|droverreal all my comments, including link to https://github.com/geerlingguy/ansible-role-docker/issues/17117:13
tristanCweshay|ruck: iiuc, docker doesn't distribute working centos-8 package, https://download.docker.com/linux/centos/17:13
weshay|ruckoh loveerly17:14
weshay|ruckso they got our flowers re: "hey docker, we think you suck"17:14
sshnaidm|offzbr|drover, I'm not against using docker temporary until we ensure it works good with podman, but if we never run jobs with molecule-podman we'll be stuck with docker forever17:14
zbr|drovertristanC: the el7 repo is working,17:14
weshay|ruckzbr|drover, sounds like it's no docker for centos-8 dude17:14
zbr|droverweshay|ruck: you call me stupid?17:15
tristanCzbr|drover: the molecule issue you link seems to be about a missing podman cp command. iirc podman does have a cp command17:15
weshay|ruck?17:15
sshnaidm|offtristanC, which issue?17:15
tristanCsshnaidm|off: +1, please test using podman in place of docker and let see what is broken and what needs to be fixed17:15
tristanCsshnaidm|off: the issue zbr|drover linked earlier, it's https://github.com/ansible/molecule/pull/248417:16
tristanC(afk lunch now17:16
sshnaidm|offtristanC, I'm for that, but we may require something working meanwhile if there is a problem with podman, always good to have a backup17:16
zbr|droverhttps://www.google.com/search?client=firefox-b-d&q=instaling+docker+on+centos+8 -- any of the top 10 results will tell you how to do it.17:16
sshnaidm|offzbr|drover, I need more details in this issue17:19
sshnaidm|offzbr|drover, what podman version, cgroups version, ansible version17:19
weshay|rucktristanC, so.. zbr|drover is saying the centos-7 of docker works on 817:19
weshay|ruckperhaps we can check that out17:19
*** chandankumar is now known as raukadah17:20
weshay|rucktristanC, I guess it's how you feel about https://review.opendev.org/#/c/703053/1/roles/install-docker/tasks/upstream-yum.yaml17:21
*** tosky has quit IRC17:21
*** holser has quit IRC17:23
sshnaidm|offtristanC, the default centos8 podman version 1.4.2-stable2 won't work because of https://github.com/containers/libpod/pull/3391/commits/c962d214e6156bd4d89e0eb20246549eca272ba717:24
sshnaidm|offand this commit is only from 1.4.3 :(17:24
sshnaidm|offif there is way to install newer podman package it will help17:24
*** holser has joined #rdo17:29
*** holser has quit IRC17:29
*** tmazur has quit IRC17:29
*** jpich has quit IRC17:30
tristanCzbr|drover: i didn't realize the backend was declared in the molecule scenario. Since docker and podman are similar container-runtime, could molecule use podman instead of docker if the podman command is available?17:32
*** aedc has quit IRC17:32
tristanCweshay|ruck: zbr|drover: in the meantime, i think we can just bake the containerd.io package in the cloud-centos-8 image17:33
tristanCsshnaidm|off: i see, it seems like a nice to have fix in the podman distributed centos. not sure what is the cherry-pick/backport/patch process for the package though. isn't a more recent version available through centos-cr or stream or modularity?17:34
zbr|drovertristanC: just finished talk with wes, we got in idea, can we pre-build an node image and use that?17:35
rdogerritChandan Kumar proposed rdo-jobs master: [wip] fs020 with latest master podman  https://review.rdoproject.org/r/2460117:35
zbr|droverlike a "centos-8-preloaded" that has all extra deps/hacks on it?17:35
tristanCzbr|drover: yes sure, but that's just postponing the main issue, that is we shouldn't be using docker on centos-817:35
sshnaidm|offtristanC, no idea.. there is not even ansible or pip in c8, what's wrong with this distro..17:36
zbr|drovertristanC: do you want me to propose an ubuntu image? that is what we have upstream, and works fine ;)17:36
tristanCand if we are going to taint the image with docker, then we might as well install the right podman version17:36
zbr|drovertristanC: that image is special for tox/molecule is not for deployments17:37
zbr|droveris a "qa" system, not one that is supposed to match what we ship.17:37
sshnaidm|offtristanC, would be nice17:38
zbr|droverthe reality is that for molecule I we will need an image that has: docker, podman, libvirt maybe even vagrant.17:38
zbr|droverall the virtualization backends that may be needed by various functional tests.17:39
tristanCsshnaidm|off: let me have a look, perhaps it's not difficult to get a more recent podman in centos.   for pip you need to install python36 ( e.g. https://softwarefactory-project.io/paste/show/1717/ )17:39
zbr|droversshnaidm|off: i fixed pip, just remember is called pip3 ;)17:40
sshnaidm|off:E17:40
zbr|droversee my merged fix from https://review.opendev.org/#/c/702978/17:40
sshnaidm|offand why do they hate ansible?17:40
zbr|droversshnaidm|off: i think is not hate, is business model ;)17:41
tristanCzbr|drover: iiuc, because molecule test declare the container runtime, then i think you should just skip or fail the scenario that declares docker on centos17:42
zbr|droverthe release stream, subscription stuff,... but I am personally not afraid to use pip, in the end YOLO17:42
tristanCzbr|drover: i would fix molecule to not declare the container runtime, so that molecule could choose podman or docker based on what's available17:42
zbr|droverwelll, I think I know how to name the magic image,  "centos-8-yolo", sounds ok? i am serious.17:43
tristanCzbr|drover: what's the purpose of such image?17:43
zbr|drovertristanC: it is possible to override the driver via an evironment variable but in practice it will work in less than 1/5 cases.17:43
zbr|droverbecause drivers have different options needed, in very few cases you can get around and switch it with the override env var.17:44
zbr|droverworked fine for few basic cases between podman and docker17:44
tristanCzbr|drover: yeah, that sounds like a limitation...17:44
zbr|droverand my plan is to simplify this, especially between podman and docker.17:45
tristanCzbr|drover: podman and docker are meant to be compatible and swapable17:45
zbr|droverlike adding a native fallback between them17:45
zbr|drovertristanC: it seems that marketing worked better for you than for me :D17:45
tristanCzbr|drover: ok, so once molecule can fallback, we can drop docker from centos-8 right?17:45
*** jpena is now known as jpena|off17:45
zbr|drovermy desktop is a macos (yes...), and I use docker remoting perfectly (including with molecule), podman remoting system is far from usable.17:46
zbr|drovertristanC: yes, that was the idea.17:47
tristanCzbr|drover: alright, then let's not bother with a special image name, let's bake that docker fix until molecule can use podman instead, then we'll drop docker17:47
zbr|droverif i suceed to make the backend-fallback work, we will be able remove docker without changing any repository or job definition.17:47
zbr|drovertristanC: is risky, because newer podman may go into conflict, I would not want to have of reference image backed with docker.17:48
tristanCzbr|drover: sounds like the right plan to me17:48
zbr|droveri know is easier but i am afraid someone will be really pissed off.17:49
tristanCzbr|drover: why would 'newer podman may go into conflict' ?17:49
zbr|droverfor example current version of upstream podman had some bugs which made impossible to install both. they installed man-packages that overlapped.17:50
zbr|droveri think is already fixed, but you got the idea.17:50
tristanCzbr|drover: that is news to me, i never had issue updating centos or fedora packages...17:51
zbr|droverstill, if I have a YOLO machine, I could even bake a latest podman from upsream on it to test if newer version version works bettter than vanilla.17:51
*** gfidente has quit IRC17:51
zbr|drovertristanC: read https://github.com/containers/libpod/issues/4791 still open :D17:52
tristanCzbr|drover: looking at this from another angle, when molecule test docker on centos-8, it is testing something nobodies is going to use now because docker on centos-8 does not work yet.  Thus I don't understand why we are going through so many work around for such a thing... Fixing molecule to use podman instead sounds like the best things to do now.17:52
zbr|drovertristanC: you are looking at it at the wrong angle: these jobs are not testing software compatibility with OS from the image itself, they are testing virtualized stuff inside docker/podman containers.17:54
tristanCzbr|drover: right, that can happens when you install extra stuff in /usr/share. It sounds like docker-ce should be using /usr/local/share or /opt/docker to avoid conflict with the distribution17:54
zbr|droverAFAIK, we could easily use the ubuntu image that is already used by openstack-tox-molecule job.17:54
zbr|droverwhich is in fact doing the same thing: installing docker-ce from upstream, and not from ubuntu repos.17:55
zbr|droverbut this would not play well with dogfooding, so better to stick on a redhat platfrom17:56
tristanCzbr|drover: is there another issue you had with podman beside the cp directory thing that is already fixed in 1.4.3 ?17:57
zbr|drovertristanC: is NOT fixed, I am 100% sure, read all comments in the bug. The first attempt to fix was in 1.4.3, but the real fix was in 1.5.117:59
*** aedc has joined #rdo17:59
zbr|droveri know for sure, as I faces it few weeks back, there is a bz about it.17:59
tristanCzbr|drover: the bug is fixed https://github.com/containers/libpod/issues/338417:59
*** derekh has quit IRC18:00
tristanCzbr|drover: i'm curious what is not working and why can't molecule use podman, but until you have implemented the fallback, iiuc, the plan is to install the containerd.io package in the cloud-centos-8 image so that you have some time to do the molecule fallback implementation right?18:02
tristanCzbr|drover: do you need help to add the containerd.io package in the cloud-centos-8 image?18:02
zbr|drovertristanC: i think I can do it. so your idea is to install only this package, so prepare to so the upstream role will work.18:03
tristanCzbr|drover: seems like the easiest thing to do. perhaps also bake the docker-ce.repo in case the containerd.io package depends on some extra requirements provided by docker18:03
zbr|droverthis may be a safe hack, as it would not install docker for everyone. sounds like a good compromise to me.18:03
zbr|droverthe repository needs to be installed first if I remember well. i will do some tests, and attempt to do minimal alteration to make it work.18:04
tristanCzbr|drover: thanks, let me know when the changes are up for review18:05
zbr|droverall this wasted time because Docker Inc, is slow to fix things.18:05
zbr|droversure, and thanks a LOT about helping on that.18:05
tristanCzbr|drover: you're welcome :)18:05
*** aedc has quit IRC18:06
zbr|drovercool, apparently i can install that rpm directly without setting the rpm, this will make task easier.18:09
tristanCzbr|drover: this might be helpful too: https://softwarefactory-project.io/r/#/c/17339/1/nodepool/virt_images/README.md18:11
*** ralonsoh has quit IRC18:11
*** amoralej is now known as amoralej|off18:12
zbr|droveri already did something similar. if I have time I will propose some changes to make it even easier to test locally.18:12
*** imcleod has joined #rdo18:13
*** ekultails has quit IRC18:23
*** ekultails has joined #rdo18:24
*** social has quit IRC18:39
*** bnemec has joined #rdo18:44
*** bnemec is now known as beekneemech18:45
*** jlibosva has quit IRC18:48
*** pcaruana has quit IRC18:51
*** migarcia is now known as migarcia|away18:54
zbr|drovertristanC: review https://softwarefactory-project.io/r/#/c/17341/ -- i want to split changes19:09
zbr|droverto avoid introducing bugs19:09
tristanCzbr|drover: wfm19:10
zbr|drovertristanC: if you do not mind i may want to add some extra checks like ansible-lint on that config repo, when I find the time.19:11
tristanCzbr|drover: that would be useful, but add them to https://softwarefactory-project.io/cgit/software-factory/sf-config/tree/ansible/roles/sf-repos/files/config so that all sf users would benefit from them19:13
zbr|droverin fact we use tox-linters to run these, so probably I do not need to add any new playbooks/roles, only to enable the job and add few config files.19:14
zbr|droverbut only one line of zuul config, likely.19:14
zbr|droveranother option could be to extend the purpose of config-check and make it run extra tests when the right files are detected.19:16
zbr|droverwe could even bypass the need for tox.19:17
tristanCzbr|drover: there are ci implication too, we run many config-check/config-update on each software-factory changes as part of the integration test. I would suspect a tox setup to slows down the whole ci.19:17
tristanCand because you need the first config-check to succeed in order to setup nodepool-provider, we run config-check on the executor directly19:18
zbr|drovertristanC: do you have support for caching folder between builds?19:18
tristanCzbr|drover: zuul doesn't let you do that unfortunately19:18
zbr|drovernot yet, we will get there.19:18
*** tesseract has quit IRC19:32
zbr|drovertristanC: also updated https://softwarefactory-project.io/r/#/c/17340/ - which could prove useful for future updates19:34
*** tosky has joined #rdo19:41
*** aedc has joined #rdo19:52
*** openstackstatus has joined #rdo20:02
*** ChanServ sets mode: +v openstackstatus20:02
*** jtomasek has quit IRC20:03
*** imcleod has quit IRC20:17
*** aedc has quit IRC20:26
*** haleyb has quit IRC20:28
*** haleyb has joined #rdo20:31
*** rh-jelabarre has quit IRC20:36
*** nweinber__ has quit IRC20:36
*** aedc has joined #rdo20:36
*** eharney has quit IRC20:59
*** imcleod has joined #rdo21:10
zbr|drovertristanC: take a look at https://softwarefactory-project.io/r/#/c/17343/ -- result of last hours21:25
zbr|droverthere is a lot of duplication across elements/ and virt_images/ -- i am not sure why.21:26
*** rlandy has quit IRC21:36
*** aedc has quit IRC21:42
*** paramite has quit IRC21:54
tristanCzbr|drover: elements/virt-customize is the legacy, we are migrating to virt_images21:55
zbr|drovertristanC: and what stops us from removing them?21:56
zbr|droverfew more images to "port" to the new one?21:56
tristanCzbr|drover: the time it would take to refactor the standalone playbook elements into the new virt_images roles21:56
zbr|drovertristanC: you got yourself a buddy to help, i think i like building images.21:57
zbr|droveri was impressed how fast it it run locally and that it was mostly idempotent21:58
tristanCzbr|drover: that's nice to hear, but we likely need to groom and plan such effort with the team first21:58
zbr|droversure, small bits21:59
tristanCe.g. before rewritting all the image, we would create a story here https://tree.taiga.io/project/morucci-software-factory/ first21:59
*** eharney has joined #rdo22:26
*** rh-jelabarre has joined #rdo22:33
*** xek has quit IRC22:36
*** jbrooks has quit IRC22:42
*** imcleod has quit IRC22:45
*** slaweq has joined #rdo22:51
*** apevec has quit IRC22:57
*** cyberworm54 has quit IRC23:10
*** cyberworm54 has joined #rdo23:10
*** slaweq has quit IRC23:14
*** imcleod has joined #rdo23:26
*** rh-jelabarre has quit IRC23:27
*** mcornea has quit IRC23:40
rdogerritSagi Shnaidman created rdo-infra/ci-config master: Increase interval for zuul stats  https://review.rdoproject.org/r/2460223:58

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