Tuesday, 2019-07-23

rdogerritrdo-trunk created rdoinfo master: Promote CBS tags update for rocky-testing  https://review.rdoproject.org/r/2155800:04
*** Vorrtex has quit IRC00:20
*** d0ugal has quit IRC00:34
*** d0ugal has joined #rdo00:48
*** imcleod has joined #rdo00:53
*** ccamposr has quit IRC00:55
*** ccamposr has joined #rdo00:55
*** lhinds has quit IRC00:57
*** eharney has quit IRC00:57
*** lhinds has joined #rdo00:58
*** ccamposr has quit IRC01:04
*** ccamposr has joined #rdo01:04
*** imacdonn has quit IRC01:18
*** imacdonn has joined #rdo01:18
*** jcoufal has joined #rdo01:52
*** fmount has joined #rdo01:52
*** jcoufal has quit IRC01:56
*** imcleod has quit IRC02:22
*** imcleod has joined #rdo02:44
*** imcleod has quit IRC02:50
*** imcleod has joined #rdo03:01
*** artom has quit IRC03:15
*** cyberworm54 has joined #rdo03:16
*** igarcia_ has quit IRC03:18
*** ykarel has joined #rdo03:23
*** cyberworm54 has quit IRC03:25
*** cyberworm54 has joined #rdo03:25
*** dsneddon has quit IRC03:29
*** imcleod has quit IRC03:30
*** artom has joined #rdo03:32
*** gkadam has joined #rdo03:49
*** gkadam has quit IRC03:50
*** dsneddon has joined #rdo03:54
*** sabedevops has joined #rdo03:55
*** sabedevops has quit IRC03:57
*** imcleod has joined #rdo03:58
rdogerritYatin Karel proposed rdo-jobs master: [DNM] Test scenario001 with exclude rc11  https://review.rdoproject.org/r/2017104:00
*** dsneddon has quit IRC04:01
*** udesale has joined #rdo04:01
*** igarcia_ has joined #rdo04:07
*** cyberworm54 has quit IRC04:09
*** rcernin has quit IRC04:13
*** igarcia__ has joined #rdo04:13
*** rcernin has joined #rdo04:14
*** igarcia_ has quit IRC04:15
*** rcernin has quit IRC04:20
rdogerritYatin Karel proposed rdo-jobs master: [DNM] Test scenario001 with exclude rc11  https://review.rdoproject.org/r/2017104:21
*** imcleod has quit IRC04:24
rdogerritMerged openstack/kolla-distgit stein-rdo: openstack-kolla-8.0.0-1  https://review.rdoproject.org/r/2155504:29
rdogerritMerged openstack/manila-distgit queens-rdo: openstack-manila-6.3.1-1  https://review.rdoproject.org/r/2155704:31
rdogerritMerged openstack/manila-distgit stein-rdo: openstack-manila-8.0.1-1  https://review.rdoproject.org/r/2155604:32
*** gfidente has quit IRC04:43
*** pcaruana has joined #rdo04:43
*** raukadah is now known as chandankumar04:44
*** marios has joined #rdo04:48
*** marios|ruck has joined #rdo04:49
*** dsneddon has joined #rdo04:49
*** ramishra has quit IRC04:54
*** ykarel is now known as ykarel|afk04:55
*** ykarel|afk has quit IRC05:02
*** dsneddon has quit IRC05:03
*** dsneddon has joined #rdo05:06
*** dsneddon has quit IRC05:14
*** ykarel|afk has joined #rdo05:18
*** ykarel|afk is now known as ykarel05:18
*** aufi has joined #rdo05:20
*** ramishra has joined #rdo05:29
*** aakarsh has joined #rdo05:42
*** dpawlik has joined #rdo05:44
*** dsneddon has joined #rdo05:47
*** aakarsh has quit IRC05:48
*** dsneddon has quit IRC05:57
*** ccamposr has quit IRC06:03
*** ccamposr has joined #rdo06:05
*** soniya29 has joined #rdo06:10
*** ccamposr has quit IRC06:11
*** ccamposr has joined #rdo06:12
rdogerritChandan Kumar proposed config master: Adds rhel8 periodic-tripleo-buildimage* jobs and common base  https://review.rdoproject.org/r/2135706:16
*** apetrich has quit IRC06:20
*** soniya29 has quit IRC06:23
*** dsneddon has joined #rdo06:27
*** paramite has quit IRC06:35
*** paramite has joined #rdo06:36
*** paramite has joined #rdo06:37
*** paramite has quit IRC06:38
*** paramite has joined #rdo06:39
*** paramite has quit IRC06:41
*** paramite has joined #rdo06:41
*** ttsiouts has joined #rdo06:48
*** jpich has joined #rdo06:55
*** jlibosva has joined #rdo06:58
*** dmacpher has joined #rdo06:58
*** gfidente has joined #rdo06:59
*** jfrancoa has joined #rdo07:02
*** jtomasek has joined #rdo07:04
*** paramite has quit IRC07:05
*** paramite has joined #rdo07:06
*** paramite has quit IRC07:07
*** Goneri has joined #rdo07:07
*** paramite has joined #rdo07:07
*** slaweq has joined #rdo07:07
*** paramite has quit IRC07:08
*** paramite has joined #rdo07:08
*** tesseract has joined #rdo07:09
*** rpittau|afk is now known as rpittau07:09
*** iurygregory has joined #rdo07:14
*** dgurtner has joined #rdo07:14
*** holser has joined #rdo07:17
*** zbr_ has joined #rdo07:20
*** aufi_ has joined #rdo07:20
*** irclogbot_1 has quit IRC07:20
*** openstackstatus has quit IRC07:20
*** nmagnezi has quit IRC07:21
*** mpeterson has quit IRC07:21
*** mpeterson has joined #rdo07:21
*** nmagnezi has joined #rdo07:22
*** holser has quit IRC07:22
*** jfrancoa has quit IRC07:22
*** irclogbot_0 has joined #rdo07:22
*** zbr has quit IRC07:23
*** mrda has quit IRC07:23
*** aufi has quit IRC07:23
*** mrda has joined #rdo07:23
*** Anticimex has quit IRC07:24
rdogerritChandan Kumar proposed config master: Add rhel-8 containers buildah master periodic job  https://review.rdoproject.org/r/2129407:25
*** beagles has quit IRC07:26
*** paramite has quit IRC07:26
*** paramite has joined #rdo07:26
*** tosky has joined #rdo07:28
*** paramite has quit IRC07:28
*** paramite has joined #rdo07:28
*** holser has joined #rdo07:28
*** Anticimex has joined #rdo07:29
*** mburned_out is now known as mburned07:31
*** holser has quit IRC07:36
rdogerritMerged config master: Add CI config for collectd-sensubility  https://review.rdoproject.org/r/2153307:37
*** holser has joined #rdo07:37
*** holser has quit IRC07:40
*** jfrancoa has joined #rdo07:40
*** holser has joined #rdo07:41
*** ttsiouts has quit IRC07:42
*** ttsiouts has joined #rdo07:43
*** ttsiouts has quit IRC07:48
rdogerritChandan Kumar proposed config master: Add periodic-tripleo-rhel-8-master-containers-build-push  https://review.rdoproject.org/r/2129407:48
*** ykarel is now known as ykarel|lunch07:50
rdogerritChandan Kumar proposed config master: Adds rhel8 periodic-tripleo-buildimage* jobs and common base  https://review.rdoproject.org/r/2135707:52
rdogerritMerged config master: Add periodic-tripleo-rhel-8-master-containers-build-push  https://review.rdoproject.org/r/2129407:52
rdogerritMerged config master: Adds rhel8 periodic-tripleo-buildimage* jobs and common base  https://review.rdoproject.org/r/2135707:55
*** apetrich has joined #rdo07:57
*** dtantsur|afk is now known as dtantsur07:59
sshnaidmI think we are good to go with https://softwarefactory-project.io/r/#/c/15957/08:03
*** Goneri has quit IRC08:09
*** Goneri has joined #rdo08:11
rdogerritArx Cruz proposed rdo-infra/ansible-role-tripleo-ci-zuul-runner master: Create zuul-runner playbooks  https://review.rdoproject.org/r/2133208:12
*** holser has quit IRC08:22
*** ttsiouts has joined #rdo08:22
chandankumarmarios|ruck: your patch is merged now08:26
marios|ruckchandankumar: ack yeah thanks noticed it moved the task to qe08:28
rdogerritYatin Karel proposed rdo-jobs master: [DNM] Test scenario001 with rc12  https://review.rdoproject.org/r/2017108:32
rdogerritArx Cruz proposed rdo-jobs master: Added periodic-tripleo-ci-rhel-8-standalone-master job  https://review.rdoproject.org/r/2127008:33
*** panda has quit IRC08:38
*** panda has joined #rdo08:38
rdogerritSorin Sbarnea (zbr) proposed config master: Adds empty molecule job  https://review.rdoproject.org/r/2155408:41
rdogerritSorin Sbarnea (zbr) proposed config master: Make ovb-manage install_openstack_deps distro aware  https://review.rdoproject.org/r/2146308:41
*** holser has joined #rdo08:44
*** morazi has joined #rdo08:45
*** soniya29 has joined #rdo08:51
*** ccamposr__ has joined #rdo08:56
*** ykarel|lunch is now known as ykarel08:57
*** ccamposr has quit IRC08:59
*** Goneri has quit IRC09:11
*** holser has quit IRC09:13
*** Goneri has joined #rdo09:16
*** holser has joined #rdo09:22
*** apetrich has quit IRC09:36
*** aufi_ has quit IRC09:45
*** jaosorior has joined #rdo09:52
rdogerritSagi Shnaidman created rdo-infra/ansible-role-tripleo-ci-reproducer master: Use new images with reproducer  https://review.rdoproject.org/r/2155909:53
*** soniya29 has quit IRC10:00
*** dpawlik has quit IRC10:02
*** dpawlik has joined #rdo10:03
*** holser has quit IRC10:05
*** ttsiouts has quit IRC10:11
*** ttsiouts has joined #rdo10:12
*** kopecmartin|off is now known as kopecmartin10:12
*** ttsiouts has quit IRC10:16
*** dsneddon has quit IRC10:37
*** apetrich has joined #rdo10:53
rdogerritChandan Kumar proposed config master: Add RHEL 8 openstack-periodic pipeline  https://review.rdoproject.org/r/2149511:06
rdogerritChandan Kumar proposed rdo-jobs master: Added periodic-tripleo-ci-rhel-8-standalone-master job  https://review.rdoproject.org/r/2127011:06
*** dsneddon has joined #rdo11:08
*** dsneddon has quit IRC11:14
rdogerritArx Cruz created config master: Adding RedHat to the epel-release logic  https://review.rdoproject.org/r/2156011:21
*** holser has joined #rdo11:23
*** ccamposr__ has quit IRC11:26
*** ttsiouts has joined #rdo11:30
*** b3nt_pin has joined #rdo11:35
*** ykarel is now known as ykarel|afk11:40
*** dsneddon has joined #rdo11:41
*** holser has quit IRC11:57
rdogerritArx Cruz proposed rdo-jobs master: WIP: Add ovb check and periodic fs001 job running on RHEL 8  https://review.rdoproject.org/r/2141811:57
*** dsneddon has quit IRC11:57
*** rh-jelabarre has joined #rdo12:00
*** eharney has joined #rdo12:01
*** udesale has quit IRC12:04
*** udesale has joined #rdo12:04
*** Goneri has quit IRC12:06
*** imcleod has joined #rdo12:06
*** dsneddon has joined #rdo12:08
rdogerritChandan Kumar proposed rdo-jobs master: Added periodic-tripleo-ci-rhel-8-standalone-master job  https://review.rdoproject.org/r/2127012:11
rdogerritSorin Sbarnea (zbr) proposed config master: Make ovb-manage install_openstack_deps distro aware  https://review.rdoproject.org/r/2146312:13
*** dsneddon has quit IRC12:14
*** ttsiouts has quit IRC12:17
*** ttsiouts has joined #rdo12:17
*** Goneri has joined #rdo12:18
*** imcleod has quit IRC12:19
*** imcleod has joined #rdo12:20
*** ttsiouts has quit IRC12:22
*** ccamacho has joined #rdo12:22
*** jpich has quit IRC12:30
*** jpich has joined #rdo12:30
*** imcleod has quit IRC12:34
*** ykarel|afk is now known as ykarel12:34
*** imcleod has joined #rdo12:38
*** imcleod has quit IRC12:43
*** dsneddon has joined #rdo12:45
rdogerritSagi Shnaidman created rdo-infra/ansible-role-tripleo-ci-reproducer master: Update README with examples for trusted repo  https://review.rdoproject.org/r/2156212:52
sshnaidmzbr_, panda weshay arxcruz marios|ruck take a look please ^^12:53
*** dsneddon has quit IRC12:53
*** dsneddon has joined #rdo12:57
*** imcleod has joined #rdo12:58
arxcruzsshnaidm, i would like your help, because on fedora-29 for zuul-runner, i can't find the key used to connect, so zuul keeps getting ssh error, because can't find tripleo-ci-id_rsa key12:58
sshnaidmarxcruz, let's talk after mtg13:00
arxcruzsshnaidm, ok13:00
*** dsneddon has quit IRC13:02
*** sshnaidm has quit IRC13:03
rdogerritUser marios created config master: Add ansible_python_interpreter for rhel8 buildimage jobs  https://review.rdoproject.org/r/2156313:04
*** apetrich has quit IRC13:07
rdogerritCarlos Goncalves proposed openstack/octavia-tempest-plugin-distgit stein-rdo: python-octavia-tests-tempest-1.1.0-1  https://review.rdoproject.org/r/2150013:07
*** sshnaidm has joined #rdo13:12
rdogerritCarlos Goncalves proposed openstack/octavia-tempest-plugin-distgit rocky-rdo: python-octavia-tests-tempest-1.1.0-1  https://review.rdoproject.org/r/2149913:14
*** ttsiouts has joined #rdo13:17
rdogerritArx Cruz proposed config master: Adding RedHat to the epel-release logic  https://review.rdoproject.org/r/2156013:18
*** openstackstatus has joined #rdo13:22
*** ChanServ sets mode: +v openstackstatus13:22
rdogerritCarlos Goncalves proposed openstack/octavia-tempest-plugin-distgit queens-rdo: python-octavia-tests-tempest-1.1.0-1.51e91b4git  https://review.rdoproject.org/r/2149813:22
*** ktibi has joined #rdo13:30
*** ykarel is now known as ykarel|away13:39
*** tosky_ has joined #rdo13:40
*** tosky has quit IRC13:42
*** ekultails has joined #rdo13:44
*** ykarel|away has quit IRC13:44
rdogerritSorin Sbarnea (zbr) proposed config master: Make ovb-manage install_openstack_deps distro aware  https://review.rdoproject.org/r/2146313:46
rdogerritArx Cruz proposed config master: Adding RedHat to the epel-release logic  https://review.rdoproject.org/r/2156013:49
*** ttsiouts has quit IRC13:53
*** ttsiouts has joined #rdo13:54
*** iurygregory has quit IRC13:55
*** iurygregory has joined #rdo13:55
*** ttsiouts has quit IRC13:58
*** ttsiouts has joined #rdo13:58
*** dsneddon has joined #rdo14:01
sshnaidmarxcruz, where is your error about key in fedora-29?14:02
arxcruzsshnaidm, https://review.rdoproject.org/r/#/c/21332/14:03
arxcruzi'm trying to get something similar as in the reproducer14:03
*** rpittau is now known as rpittau|afk14:03
rdogerritMerged config master: Add ansible_python_interpreter for rhel8 buildimage jobs  https://review.rdoproject.org/r/2156314:05
*** dsneddon has quit IRC14:08
*** tosky_ is now known as tosky14:08
*** Vorrtex has joined #rdo14:09
*** ccamposr has joined #rdo14:09
*** dprince has joined #rdo14:10
*** ykarel|away has joined #rdo14:11
rdogerritSagi Shnaidman proposed rdo-jobs master: DNM: rerun periodic jobs  https://review.rdoproject.org/r/2133814:11
*** aakarsh has joined #rdo14:12
*** jcoufal has joined #rdo14:16
*** paramite has quit IRC14:20
*** paramite has joined #rdo14:20
*** imcleod has quit IRC14:22
*** imcleod has joined #rdo14:22
*** apetrich has joined #rdo14:26
*** ykarel|away has quit IRC14:34
*** ykarel|away has joined #rdo14:37
rdogerritSagi Shnaidman proposed rdo-jobs master: DNM: rerun periodic jobs  https://review.rdoproject.org/r/2133814:44
*** iurygregory has quit IRC14:45
rdogerritSorin Sbarnea (zbr) proposed config master: Adds empty molecule job  https://review.rdoproject.org/r/2155414:47
*** iurygregory has joined #rdo14:47
*** gfidente has quit IRC14:49
*** paramite has quit IRC14:53
rdogerritSorin Sbarnea (zbr) proposed config master: Adds empty molecule job  https://review.rdoproject.org/r/2155414:53
*** paramite has joined #rdo14:53
*** paramite has quit IRC14:55
*** paramite has joined #rdo14:55
*** holser has joined #rdo14:57
*** paramite has quit IRC14:57
*** paramite has joined #rdo14:57
*** marios|ruck has quit IRC15:04
*** marios has quit IRC15:04
*** marios has joined #rdo15:07
*** marios|ruck has joined #rdo15:08
*** mburned is now known as mburned_out15:09
rdogerritSagi Shnaidman created config master: Configure release for image build jobs  https://review.rdoproject.org/r/2156415:13
sshnaidmmarios, ^^15:13
marios|rucksshnaidm: ack15:15
*** aakarsh has quit IRC15:19
*** holser has quit IRC15:20
*** ccamacho has quit IRC15:21
rdogerritMerged config master: Configure release for image build jobs  https://review.rdoproject.org/r/2156415:21
*** jfrancoa has quit IRC15:22
*** rh-jelabarre has quit IRC15:23
*** dmacpher has quit IRC15:23
*** morazi has quit IRC15:25
rdogerritSagi Shnaidman proposed rdo-jobs master: DNM: rerun periodic jobs  https://review.rdoproject.org/r/2133815:25
*** rh-jelabarre has joined #rdo15:25
*** dprince has quit IRC15:27
rdogerritGabriele Cerami proposed rdo-jobs master: [WIP] add buildimage rhel8 check jobs  https://review.rdoproject.org/r/2153515:27
*** dpawlik has quit IRC15:30
*** ktibi has quit IRC15:32
*** zaneb has quit IRC15:43
*** dsneddon has joined #rdo15:46
*** dsneddon has quit IRC15:52
*** jaosorior has quit IRC15:53
rdogerritSorin Sbarnea (zbr) proposed rdo-jobs master: Add ovb check and periodic fs001 job running on RHEL 8  https://review.rdoproject.org/r/2141815:53
*** ttsiouts has quit IRC15:54
*** ttsiouts has joined #rdo15:55
rdogerritSorin Sbarnea (zbr) proposed rdo-jobs master: Add ovb check and periodic fs001 job running on RHEL 8  https://review.rdoproject.org/r/2141815:55
*** ttsiouts_ has joined #rdo15:57
*** ttsiouts has quit IRC15:57
*** dsneddon has joined #rdo15:59
*** ttsiouts_ has quit IRC15:59
*** iurygregory has quit IRC16:02
*** tesseract has quit IRC16:07
*** dsneddon has quit IRC16:08
*** marios has quit IRC16:08
*** marios|ruck has quit IRC16:08
rdogerritSagi Shnaidman created config master: Add workspace variable to buildimage playbooks  https://review.rdoproject.org/r/2156516:09
rdogerritSagi Shnaidman proposed config master: Add workspace variable to buildimage playbooks  https://review.rdoproject.org/r/2156516:10
rdogerritChandan Kumar proposed rdo-jobs master: Added periodic-tripleo-ci-rhel-8-standalone-master job  https://review.rdoproject.org/r/2127016:11
sshnaidmpanda, this what happens when not testing in reproducer, how would functional test help here? https://review.rdoproject.org/r/#/c/21565/16:13
rdogerritMerged rdo-jobs master: Add ovb check and periodic fs001 job running on RHEL 8  https://review.rdoproject.org/r/2141816:14
*** dprince has joined #rdo16:15
rdogerritMerged config master: Add workspace variable to buildimage playbooks  https://review.rdoproject.org/r/2156516:15
rdogerritSagi Shnaidman proposed rdo-jobs master: DNM: rerun periodic jobs  https://review.rdoproject.org/r/2133816:15
*** ohsnap has joined #rdo16:17
*** dsneddon has joined #rdo16:18
*** jpich has quit IRC16:18
pandasshnaidm: I have no idea what this change is about, and the commit message is not explaining why it was needed.16:21
pandasshnaidm: so it' adding a variable that was not defined. Ok. THe functiona test launches this playbook, and at the end checks that file were arrchived in the correct directory16:22
*** dsneddon has quit IRC16:23
pandaso if we were expecting the files to be archived in /home/zuul/workspace, the functional test contains a check that archive file exist in /home/zuul/workspace16:23
pandaa simple stat: with failed_when: not file.exists should be enough.16:24
pandaand you know the result immediately, without having to wait for the whole job to run and arrive to post to collect logs.16:24
sshnaidmpanda, can you submit a patch with example?16:24
sshnaidmpanda, it fails in "pre.yaml", yet long before collecting logs16:24
pandasshnaidm: it fails in both, patch incoming, but I'll keep it simple.16:28
*** ekultails has quit IRC16:28
rdogerritGabriele Cerami proposed rdo-jobs master: add buildimage rhel8 check jobs  https://review.rdoproject.org/r/2153516:32
sshnaidmykarel|away, when you're back tomorrow, can you take a look please as rhel8 build log? Like all these missing packages, do we miss something? http://logs.rdoproject.org/38/21338/11/check/periodic-tripleo-rhel-8-master-buildimage-overcloud-full-master/326458f/logs/build.log16:36
*** dsneddon has joined #rdo16:36
chandankumarsshnaidm: https://logs.rdoproject.org/38/21338/11/check/periodic-tripleo-rhel-8-master-buildimage-overcloud-full-master/326458f/logs/16:38
chandankumarsshnaidm: why it does not collects /etc/yum.repos.d?16:38
*** dtantsur is now known as dtantsur|afk16:38
chandankumarand dnf.log to get full picture what packages are missing16:38
sshnaidmchandankumar, it barely collect anything..16:38
chandankumarsshnaidm: is it possible to make them collecting or tweak it to collect those files?16:40
sshnaidmchandankumar, I'll look, worth to run our collect-logs role there16:41
sshnaidmat least it tries to build something16:41
*** jlibosva has quit IRC16:41
*** dsneddon has quit IRC16:43
*** Goneri has quit IRC16:44
*** dsneddon has joined #rdo16:45
sshnaidmdmsimard, hi16:52
dmsimardo/16:52
sshnaidmdmsimard, can you please take a look at this patch? https://review.rdoproject.org/r/#/c/21554/16:57
sshnaidmdmsimard, I'd like to be sure we don't break anything in config repo16:57
sshnaidmdmsimard, afaik you can force reverting it in case things are going wrong :)16:57
dmsimardlooking16:58
ykarel|awaysshnaidm, looking16:58
ykarel|awaylooks like missing repos16:59
ykarel|awaythose packages should come from rhel8 delorean repo16:59
rdogerritGabriele Cerami created config master: [DNM] POC of functional tests on buildimage playbooks  https://review.rdoproject.org/r/2156616:59
ykarel|awaythe python3-client ones16:59
pandasshnaidm: this is just a non working POC, but it give the idea ^16:59
sshnaidmykarel|away, ack, will check erpos16:59
sshnaidm*repos16:59
dmsimardsshnaidm: why does it need to live in the config repo ? could it not be in another repo ?17:00
dmsimard(genuinely curious, lacking context)17:01
sshnaidmdmsimard, zbr_ wants to run it on config repo exactly17:01
sshnaidmdmsimard, it should run functional molecule jobs on config repo patches17:01
zbr_dmsimard: we do have ansible roles inside config, so we can test patches proposed on them.17:02
rdogerritChandan Kumar proposed rdo-jobs master: Added periodic-tripleo-ci-rhel-8-standalone-master job  https://review.rdoproject.org/r/2127017:02
*** udesale has quit IRC17:03
zbr_dmsimard: i asked corvus yesterday about it and he told me that it would be no issue with it. in the end is another job (one that does not need secrets, not even an expensive one).17:04
dmsimardzbr_: wouldn't trusted execution context end up getting in the way ?17:05
dmsimardi.e, you would have been able to test that molecule patch in an untrusted repo without landing it17:06
zbr_i doubt, i do not see it how it would be different than tox-linters one, in fact they both inherit the same basic tox job.17:06
*** chandankumar is now known as raukadah17:08
dmsimardzbr_: I don't have a problem with that particular patch fwiw, it will work -- but I feel like it is cumbersome to use the config repo for things that do not require trusted execution context17:09
zbr_dmsimard: that is another question. in fact ovb-manage itself needs access to credentials, but we can test some parts of it without them.17:11
zbr_ansible code can move from one repo to another alongside its testing code, no problem with that.17:12
*** dprince has quit IRC17:13
dmsimardcredentials can live in non-config repos too :p17:13
sshnaidmdmsimard, how?17:13
*** aakarsh has joined #rdo17:13
dmsimardsshnaidm: you can define a secret in any project as long as it is encrypted with the project key17:13
dmsimardfor example in ara I have a secret which is a private ssh key to be able to mirror things to github17:14
dmsimardjobs from projects can use secrets whether they are defined in a config repo or not -- there is a config value for secrets that define which projects are able to use it17:16
sshnaidmdmsimard, I think we have it in config repo so that people can't submit a patch with "cat $PATH_TO_MY_PASSWORD_FILE"17:16
dmsimardwait, nevermind that last one -- the config value for defining access is on the jobs, not the secrets17:17
sshnaidmdmsimard, and can I submit a patch to your ara repo with "cat ~/.ssh/id_rsa" and to see your key?17:18
dmsimardsshnaidm: in my particular case, no, because that secret is only used in a post pipeline job so I would have to merge the malicious patch before17:19
sshnaidmdmsimard, and this post pipeline job doesn't run on patches, right?17:20
dmsimardit runs after the patch merges, yeah17:20
dmsimardsshnaidm: there's a lot of secrets in use upstream and I would like to think that we can't easily print them -- if we can, we need to let upstream know17:20
sshnaidmdmsimard, yeah, currently we have to hold some roles inside config repo for that reason17:21
dmsimardit's a safe assumption :)17:22
sshnaidmdmsimard, so that people can't run patch on them17:22
*** paramite has quit IRC17:22
dmsimardbut it also gets in your way17:22
*** paramite has joined #rdo17:22
sshnaidmdmsimard, although I think it can be improved and maybe some part dealing with secrets can be extracted, but anyway17:22
sshnaidmdmsimard, so what do you think about this patch? is it fine to try to merge it?17:23
sshnaidmdmsimard, I just don't want to have config repo broken for some reason and you have superpowers :)17:23
*** ramishra has quit IRC17:24
*** ykarel|away has quit IRC17:35
*** panda is now known as panda|off17:45
dmsimardsshnaidm: Changes to the jobs in the config-repo aren't effective until after they merge, after which they could fail because of something as silly as a typo that was missed during review. Keeping them in another repository helps iterate faster without needing to land anything so I would do that if you can. If there are constraints or concerns (lik17:47
dmsimarde secrets), I would bring it up in #zuul.17:47
sshnaidmdmsimard, yeah, that's we know..17:49
*** sshnaidm is now known as sshnaidm|afk17:50
*** jcoufal_ has joined #rdo17:59
*** jcoufal has quit IRC18:03
*** holser has joined #rdo18:33
*** jcoufal_ has quit IRC18:52
*** holser has quit IRC18:53
*** dsneddon has quit IRC18:59
*** dtrainor has quit IRC19:05
*** dsneddon has joined #rdo19:16
*** kopecmartin is now known as kopecmartin|off19:16
*** panda|off has quit IRC19:23
*** panda has joined #rdo19:25
*** dgurtner has quit IRC19:30
rdogerritSorin Sbarnea (zbr) created config master: DNM: test security of config  https://review.rdoproject.org/r/2156719:47
*** tosky has quit IRC19:47
zbr_dmsimard: i am really wondering about which secret is sshnaidm|afk afraid of exposing via the new job? that job does not need any credentials defined, it works almost identically with tox-linters.19:52
sshnaidm|afkzbr_, I don't think you'll expose secrets, I think your job won't run, but I'm fine to merge it to try19:54
zbr_lets try it please19:54
sshnaidm|afkzbr_, I let dmsimard to do this, if something is broken in config because of it - only he has superpowers to force merges19:55
dmsimardI'm not the only one able to force submit things but I can help if you're in a pinch :p19:56
sshnaidm|afkdmsimard, maybe apevec too) but I didn't see anybody else in admins there19:56
sshnaidm|afkdmsimard, ok, I'll merge it then and will try to submit another patch to config repo, and we'll see how it goes19:57
dmsimardsure, I'm here for about an hour or two still.19:57
zbr_lets go, once merged it should take ~5min to get confirmation that it works or not. a simple recheck on other change would be enough.19:58
sshnaidm|afkzbr_, btw, where do you install molecule in this patch?19:58
zbr_sshnaidm|afk: is 100% tox19:59
sshnaidm|afkzbr_, and pytest?19:59
zbr_yep19:59
sshnaidm|afkok, I see19:59
zbr_tox->pytest->pytest-molecule->molecule->ansible20:00
rdogerritMerged config master: Adds empty molecule job  https://review.rdoproject.org/r/2155420:01
sshnaidm|afkzbr_, ^^20:01
zbr_yep, testing now20:02
rdogerritSagi Shnaidman created config master: Add dummy comment to other comment  https://review.rdoproject.org/r/2156820:02
zbr_look at https://review.rdoproject.org/zuul/status -- 2146320:02
zbr_already queued it20:02
zbr_and because is no voting, even if it is broken it will not affect the final result.20:02
zbr_now it would be up to us to decide what we run inside in order to validate the code, mostly we mock what we can, and we skip what we cannot.20:03
zbr_even 50% coverage is better than 0%.20:04
zbr_i am not saying that this replaces full integration jobs, but will provide fast feedback on our changes, which are very often around boostraping stuff (rpm,pip,...)20:05
*** slaweq has quit IRC20:08
zbr_sshnaidm|afk: got results from first execution: job run, failed the additional test (expected), not affecting final results.20:08
zbr_see http://logs.rdoproject.org/63/21463/13/check/rdo-tox-molecule/e930c07/tox/reports.html20:09
zbr_i will take care of it tomorrow, now we have a way to test the changes.20:09
sshnaidm|afkzbr_, better to limit it to specific paths with "files:"20:09
*** dtrainor has joined #rdo20:09
rdogerritMerged config master: Add dummy comment to other comment  https://review.rdoproject.org/r/2156820:10
sshnaidm|afkzbr_, but in general, if you can test something with secrets - get this functionality out of this repo, that should be the correct way20:10
zbr_sshnaidm|afk: yeah, in fact I this it may already have a default pattern for roles/ and .*molecule.*, but I need to double check.20:10
sshnaidm|afks/with/without/20:11
zbr_we will need to investigate this aspect, with all that rhel8 work we are better to find a proper solution.20:11
zbr_my idea was to modify check queue to be trigger only once a core adds a "looks-safe" label. so no CI would run without core blessing.20:12
zbr_that approach could rule out "hacker" attempts, and we should be able to enable it only for trusted repos.20:12
*** jroll has joined #rdo20:12
*** igarcia__ has quit IRC20:12
jrollred hat noob here: what's the right way to file a bug against the centos openstack repos? there's an old package in the rocky and stein repos which has a security issue and it would be cool to see it updated :)20:12
zbr_gerrit can have any number of labels.20:12
*** cyberworm54 has joined #rdo20:13
zbr_jroll: not sure about bug, depends on the affected project. is that an openstack package or just something we package?20:14
jrollzbr_: it's novnc20:15
jrollspecifically hoping to pull in this bug fix: https://github.com/novnc/noVNC/issues/748, per https://bugs.launchpad.net/horizon/+bug/165643520:15
openstackLaunchpad bug 1656435 in OpenStack Security Advisory "XSS in noVNC" [Undecided,Invalid]20:15
zbr_jroll: by guess "bugzilla", and this may help https://www.rdoproject.org/rdo/bugzilla-ownership/20:16
jrollalright, thanks20:18
*** aakarsh has quit IRC20:19
*** jtomasek has quit IRC20:21
*** jroll has left #rdo20:23
*** Vorrtex has quit IRC20:28
*** zaneb has joined #rdo20:41
*** pcaruana has quit IRC20:50
*** paramite has quit IRC20:56
*** paramite has joined #rdo20:56
*** slaweq has joined #rdo20:56
*** slaweq has quit IRC21:01
*** paramite has quit IRC21:17
*** paramite has joined #rdo21:17
*** holser has joined #rdo21:22
*** paramite has quit IRC21:23
*** paramite has joined #rdo21:23
*** paramite has quit IRC21:24
*** paramite has joined #rdo21:25
*** zaneb has quit IRC21:31
*** altlogbot_1 has quit IRC21:33
*** paramite has quit IRC21:33
*** irclogbot_0 has quit IRC21:33
*** paramite has joined #rdo21:33
*** paramite has quit IRC21:34
*** paramite has joined #rdo21:34
*** altlogbot_2 has joined #rdo21:35
*** irclogbot_0 has joined #rdo21:35
*** paramite has quit IRC21:54
*** paramite has joined #rdo21:54
*** paramite has quit IRC21:59
*** irclogbot_0 has quit IRC21:59
*** paramite has joined #rdo21:59
*** paramite has quit IRC22:00
*** paramite has joined #rdo22:00
*** altlogbot_2 has quit IRC22:01
*** cyberworm54 has quit IRC22:01
*** cyberworm54 has joined #rdo22:08
*** holser has quit IRC22:09
*** cyberworm54 has quit IRC22:22
*** altlogbot_3 has joined #rdo22:23
*** imcleod has quit IRC22:25
*** cyberworm54 has joined #rdo22:25
*** dsneddon has quit IRC22:26
*** altlogbot_3 has quit IRC22:27
*** cyberworm54 has quit IRC22:33
*** dsneddon has joined #rdo22:36
*** cyberworm54 has joined #rdo22:45
*** altlogbot_1 has joined #rdo23:15
*** rcernin has joined #rdo23:16
*** altlogbot_1 has quit IRC23:19
*** altlogbot_3 has joined #rdo23:29
*** irclogbot_2 has joined #rdo23:33
*** imcleod has joined #rdo23:38

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