Monday, 2019-09-16

*** dsneddon has joined #oooq00:05
*** d0ugal has quit IRC00:06
*** dsneddon has quit IRC00:10
*** d0ugal has joined #oooq00:22
*** dsneddon has joined #oooq00:44
*** dsneddon has quit IRC00:48
*** ccamacho has quit IRC01:09
*** dsneddon has joined #oooq01:22
*** dsneddon has quit IRC01:27
*** dsneddon has joined #oooq01:57
*** dsneddon has quit IRC02:02
*** dsneddon has joined #oooq02:30
*** dsneddon has quit IRC02:34
*** d0ugal has quit IRC02:39
*** ykarel|away has joined #oooq02:45
*** d0ugal has joined #oooq02:53
*** dsneddon has joined #oooq03:01
*** dsneddon has quit IRC03:06
*** dsneddon has joined #oooq03:42
*** aakarsh|2 has quit IRC03:42
*** gkadam has joined #oooq03:46
*** dsneddon has quit IRC03:46
*** gkadam has quit IRC03:47
*** surpatil has joined #oooq04:07
*** udesale has joined #oooq04:11
*** surpatil has quit IRC04:12
*** dsneddon has joined #oooq04:14
*** dsneddon has quit IRC04:19
*** ykarel|away has quit IRC04:35
*** soniya29 has joined #oooq04:42
*** dsneddon has joined #oooq04:51
*** dsneddon has quit IRC04:56
*** ykarel has joined #oooq04:56
*** dsneddon has joined #oooq05:27
*** surpatil has joined #oooq05:28
*** dsneddon has quit IRC05:31
*** ksambor has joined #oooq05:42
*** ksambor has quit IRC05:42
*** ksambor has joined #oooq05:42
*** ksambor has quit IRC05:43
*** ksambor has joined #oooq05:43
*** ratailor has joined #oooq05:44
*** ksambor has quit IRC05:44
*** ksambor has joined #oooq05:45
*** marios has joined #oooq05:57
*** dsneddon has joined #oooq05:57
*** dsneddon has quit IRC06:02
*** jfrancoa has joined #oooq06:02
*** kopecmartin|off is now known as kopecmartin06:16
*** dsneddon has joined #oooq06:30
*** dsneddon has quit IRC06:35
*** sshnaidm_ is now known as sshnaidm|rover06:37
*** holser has joined #oooq07:06
*** dsneddon has joined #oooq07:06
*** tosky has joined #oooq07:08
*** dsneddon has quit IRC07:11
*** bogdando has joined #oooq07:21
*** akahat has joined #oooq07:26
holserMorning07:27
holserhttps://lists.centos.org/pipermail/centos-devel/2019-September/017694.html :(07:27
*** ykarel is now known as ykarel|lunch07:28
*** amoralej has joined #oooq07:32
*** jpena|off is now known as jpena07:34
*** gchamoul has joined #oooq07:37
*** sshnaidm|rover is now known as sshnaidm|afk07:39
*** apetrich has joined #oooq07:44
*** dsneddon has joined #oooq07:46
*** dsneddon has quit IRC07:51
mariossshnaidm|afk: fyi merged that to config but shouldn't break anything job is non voting https://review.rdoproject.org/r/#/c/22259/07:54
mariossshnaidm|afk: (you are ruck? or is it panda now?)07:54
*** soniya29 has quit IRC07:54
*** chem has joined #oooq07:54
*** panda has quit IRC07:56
*** panda has joined #oooq07:58
*** matbu has joined #oooq08:06
*** soniya29 has joined #oooq08:09
*** dsneddon has joined #oooq08:24
*** dsneddon has quit IRC08:28
*** saneax has joined #oooq08:30
chandankumarpanda: hello08:34
chandankumarpanda: where this var is defined registry_login_enabled ?08:35
*** derekh has joined #oooq08:38
*** sshnaidm|afk is now known as sshnaidm|rover08:43
sshnaidm|rovermarios, ack08:46
sshnaidm|roverpanda, are you ruck today?08:47
chandankumarboom I have some logs now on container login08:47
*** ykarel|lunch is now known as ykarel08:49
*** skramaja has joined #oooq08:50
kopecmartinchandankumar: here's the task for interns, could you assign it to one of them? https://tree.taiga.io/project/tripleo-ci-board/task/128108:50
sshnaidm|roverdo we have interns?08:53
chandankumarsshnaidm|rover: yes, two interns08:57
*** dsneddon has joined #oooq08:57
chandankumarkopecmartin: sure08:57
kopecmartinchandankumar: thanks :)08:58
*** panda is now known as panda|ruck08:59
*** soniya29 has quit IRC09:00
*** dsneddon has quit IRC09:02
zbrmorning! anything I can help with?09:15
marioszbr: review that? https://review.rdoproject.org/r/#/c/22002/09:18
zbrmarios: afaik pushing a manifest requires you to push images from it before pushing it, was i wrong?09:19
marioszbr: yes you must have the containers tagged in the registry first as described in commit message09:27
*** dsneddon has joined #oooq09:31
surpatilhello surpatil here im a intern09:31
surpatilsuraj patil09:32
*** soniya29 has joined #oooq09:32
*** dtantsur|afk is now known as dtantsur09:32
soniya29kopecmartin, sshnaidm|rover, hello09:33
sshnaidm|roversurpatil, soniya29 hey folks09:33
surpatilhello sshnaidm|rover :)09:33
chandankumarjpena: sshnaidm|rover zbr https://review.rdoproject.org/r/#/c/22299/09:34
sshnaidm|roverchandankumar, but still worth to print status code09:36
*** dsneddon has quit IRC09:36
chandankumarsshnaidm|rover: you mean exit status?09:38
sshnaidm|roverchandankumar, yeah09:38
chandankumarok09:38
chandankumarupdating it09:38
zbrchandankumar: can  registry_login_podman expose secrets? do you have an example to show me (in private)09:40
zbrchandankumar: "unknown reason" is very bad anti-pattern. i want to see if we can avoid exposure without crippling user interface09:40
chandankumarzbr: fixed09:43
chandankumarwith exit status09:43
*** jaosorior has joined #oooq09:45
zbrsshnaidm|rover: chandankumar  please read my comment on https://review.rdoproject.org/r/c/2229909:50
zbrthe login needs to be secured on https://opendev.org/openstack/tripleo-ansible/src/branch/master/tripleo_ansible/roles/tripleo-podman/tasks/tripleo_podman_login.yml#L21-L2209:50
zbravoid giving credentials via CLI09:50
sshnaidm|roverzbr, via cli with env is more secure actually09:51
*** dsneddon has joined #oooq10:05
zbrsshnaidm|rover: chandankumar: see https://review.opendev.org/#/c/682329/2/tripleo_ansible/roles/tripleo-podman/tasks/tripleo_podman_login.yml10:06
*** dsneddon has quit IRC10:10
sshnaidm|roverzbr, commented, it's even worse than was before10:18
sshnaidm|roverzbr, the problem was in leaking "items"10:19
sshnaidm|roverzbr, and now it's leaked also in "cmd"10:20
zbrsshnaidm|rover: have you tried it? the only secret is the password, and is not leaked. see https://gist.github.com/ssbarnea/fc6bb10cf2b14b3765897f722e0e447f10:23
sshnaidm|roverzbr, you'd better test such things10:23
sshnaidm|roverzbr, see my comments10:23
zbrsshnaidm|rover: you are talking about debug mode, something that was exposing the secret even before.10:24
zbris normal and -v mode, nothing is visible10:24
sshnaidm|roverzbr, no any debug mode10:25
sshnaidm|roverzbr, just take this task and run, it's not difficult10:25
zbrhttps://seashells.io/v/fjxeJu3F -- i do not see any "secret" there10:26
sshnaidm|roverzbr, when failing, not when passed10:27
sshnaidm|roverzbr, when it's passed no any problem, we struggle with leaking when task failed10:27
sshnaidm|roverzbr, don't run your md5 task, run please a real task from your patch10:28
sshnaidm|roverzbr, it's not testing what you do10:28
*** udesale has quit IRC10:28
zbrsshnaidm|rover: i see what you say, in fact is not my patch exposes them, is the looping.10:35
zbrwhen using a loop+register you expose the looped content, in our case containing credentials. still, this is not something introduced by my patch.10:36
*** dsneddon has joined #oooq10:38
*** dsneddon has quit IRC10:43
*** tesseract has joined #oooq10:51
zbrsshnaidm|rover: i guess that the answer to the security question is....10:57
zbrthere is no security10:57
zbrclearly that stdin use improves the situation, but the final outcome is relative.11:00
sshnaidm|roverzbr, stdin clearly makes situation much worse, see my comments to patch11:05
sshnaidm|roverzbr, the real solution will be ignore_errors: true, capturint rc code and "fail" task if rc code is not 011:05
sshnaidm|roverzbr, then only "fail" task fails and no info is leaked11:06
*** marios has quit IRC11:09
*** tesseract has quit IRC11:09
zbrstdin is moving the secret from .cmd & invocation.module_args._raw_params to invocation.module_ars.stdin -- not really increasing the exposure.11:09
zbradvantage would be that allows us to also log the command line executed.11:10
*** dsneddon has joined #oooq11:11
zbrbut this does not fix the other issues, for those your approach will work (log only safe fields from registered result)11:11
chandankumarzbr: sshnaidm|rover In ordet to put final patch for enablig container login will I keep https://github.com/rdo-infra/review.rdoproject.org-config/blob/master/playbooks/tripleo-rdo-base/container-login.yaml#L6 default to false or true?11:16
chandankumaras in jobs these playbooks will be explicitly called11:16
*** dsneddon has quit IRC11:16
sshnaidm|roverchandankumar, default of what?11:17
sshnaidm|roverchandankumar, registry_login_enabled ?11:17
chandankumarsshnaidm|rover: registry_login_enabled11:17
chandankumaryes11:17
zbrchandankumar: good question, mainly adding no_log at that level makes the output barely usable, but at the same time i do not know how to secure it in a different way.11:18
chandankumarif we donot have default value it will leads to retry limit error in the logs11:18
sshnaidm|roverchandankumar, I think we can start with false firstly to see if it breaks nothing, (and add |bool)11:18
zbrchandankumar: how about this: replace conditon by looking if container_registry_logins is empty or not11:18
chandankumarsshnaidm|rover: zbr yes I can do that11:18
zbryou avoid using a boolean by checking if the dictionary with logings is empty11:18
zbris more pythonic11:18
sshnaidm|roverzbr, it's boolean flag afaiu11:19
sshnaidm|roverand I'd prefer to have it in order to switch on/off easily11:20
zbryeah i know, but i was trying to say: "do login if there is anything to login into", or am i missing somethign? do we have an execution path where we have credentials but we dont want to login?11:20
sshnaidm|roverzbr, yes, it could be such option11:21
zbrok. lets keep it like this11:22
*** dtantsur is now known as dtantsur|bbl11:26
*** jpena is now known as jpena|lunch11:30
bogdandoo/ PTAL https://review.opendev.org/#/c/682092/411:30
bogdandodo you know perchance where to dig to figure out the node indexes assigned?11:30
bogdandoykarel: ^^11:31
bogdandoso I have11:31
bogdando[overcloud]11:31
bogdandosubnode-211:31
bogdandoinstead of subnode-111:31
bogdandohttps://opendev.org/openstack/tripleo-quickstart/src/branch/master/roles/tripleo-inventory/tasks/inventory.yml#L50 looks the cause...11:32
bogdandonot sure where to adjust the things nicely11:33
ykarelbogdando, /me don't know that, but yes ^^ looks the right place where the inventory is populated11:33
bogdandoykarel: it parses /etc/nodepool/sub_nodes_private, but there is only a single IP...11:34
bogdandohm, why do we add +2 and not +111:35
bogdandosshnaidm|rover: around?11:35
* ykarel don't know +2 reasoning11:35
sshnaidm|roverbogdando, yep11:35
bogdandoso for a 1ctrlr node file and a multinode job, I have a valid distribution of primary/secondary in zuul and nodepool, even works for a single public IP and no private IP (with https://review.opendev.org/#/c/682092/)... The undercloud is matched as primary, and subnode-1 as secondary. But that +2 ruins the further deploynment11:38
bogdandoas I have no subnode-211:38
*** ccamacho has joined #oooq11:39
bogdandoI wonder how much of the things fall into cracks if I set +1 https://opendev.org/openstack/tripleo-quickstart/src/branch/master/roles/tripleo-inventory/tasks/inventory.yml#L5011:39
* bogdando away adjusting his fork11:39
sshnaidm|roverchandankumar, commented on https://review.rdoproject.org/r/#/c/22299/11:41
sshnaidm|roverbogdando, in some point it was subnode-1, subnode-211:44
sshnaidm|roverbogdando, as I see now it's "primary" and "secondary"11:44
*** udesale has joined #oooq11:45
*** soniya29 has quit IRC11:47
sshnaidm|roverbogdando, hmm.. still see it subnode-2 here: https://e6d7ddba3e6c9e0ac442-b854b998feabd4bf6926393c8ea9e138.ssl.cf5.rackcdn.com/682206/1/check/tripleo-ci-centos-7-containers-multinode/f7e7991/logs/11:47
*** dsneddon has joined #oooq11:48
bogdandosshnaidm|rover: git story point me to https://review.opendev.org/#/c/450351/11:48
sshnaidm|roverbogdando, yes, I remember why I did this :)11:49
bogdandolet's try +1 back and see how the world collapses?..11:49
sshnaidm|roverbogdando, or maybe just "secondary"11:51
*** dsneddon has quit IRC11:52
*** weshay has joined #oooq11:53
*** amoralej is now known as amoralej|lunch11:54
bogdandosshnaidm|rover: indeed11:57
bogdandoI'll try that11:57
bogdandobtw, I was reading about a new devops squad... Shall we merge quickstart with infrared? :)12:00
bogdandoweshay: hi ^^12:01
bogdandogot some fancy secrets about that?12:01
weshaybogdando, secrets.. that would be a terrible way to proceed12:01
weshaylol12:01
bogdando:D12:01
weshaybogdando, so there has been some open discussion about it12:02
sshnaidm|roverbogdando, we can merge with infrared... in way like killer whale merges with tuna12:03
bogdandohahah12:04
*** marios has joined #oooq12:06
*** soniya29 has joined #oooq12:09
weshaysshnaidm|rover, panda|ruck fyi.. the box that created cix's got pulled from infra.. /me setting it back up in tripleo-infra12:11
sshnaidm|roverweshay, you can add it in our playbooks, in some jumphost box for example12:12
weshayis the jumpbox still up?12:12
sshnaidm|roverweshay, no.. but there is hubbot12:14
weshaysshnaidm|rover,  k.. that works12:14
sshnaidm|roverweshay, I think we had a task to create a general toolbox? Now everybody create a separate instance for each script12:14
sshnaidm|roveriirc we even choose the name for this toolbox12:15
sshnaidm|roverlike half job is done12:15
weshayhrm.. can you raise that in ideas_for_improvement12:17
*** sanjayu_ has joined #oooq12:17
*** saneax has quit IRC12:18
*** rfolco has joined #oooq12:23
*** dsneddon has joined #oooq12:24
*** rlandy has joined #oooq12:25
weshaypanda|ruck, the stein ci.centos job is reporting failure to dlrn when the job passed12:28
weshaypanda|ruck,  can you get that fixed please https://ci.centos.org/job/tripleo-quickstart-promote-stein-current-tripleo-delorean-minimal/197/console12:29
weshayand make sure we promote stein rdo12:29
*** dsneddon has quit IRC12:30
sshnaidm|roverweshay, I had to increase deploy_timeout to make stein rdo job passing..12:30
sshnaidm|roverweshay, seems like we have a problem there, something is slowing down the deploy12:30
panda|ruckand causing the reporting to fail12:32
*** jpena|lunch is now known as jpena12:33
*** ratailor has quit IRC12:34
weshaypanda|ruck, fyi.. need repro https://trello.com/c/DvgaBFim/1104-cixlp1843259tripleociproa-periodic-rocky-fs020-job-fails-tempest-tests-tempestscenariotestsecuritygroupsbasicopstestsecuritygrou12:39
rfolcomarios, panda|ruck rlandy weshay would you please review ? this seems to be ready https://review.rdoproject.org/r/#/c/21872/12:41
mariosrfolco: ack will check in bit noting12:42
rlandyrfolco: you're promoting to docker.io?12:42
rlandyand reading rdo reg12:42
rlandyno internal regs?12:42
rlandy^^ discuss this at meeting12:42
rfolcorlandy, why you asking ? checking my patch again...12:43
* weshay notes infra also ate my bouncer12:49
*** amoralej|lunch is now known as amoralej12:57
*** dsneddon has joined #oooq12:57
weshaysshnaidm|rover, k.. the last job passed, but ya.. took a long time12:59
weshayperf regressions13:00
weshaywe need ara in the overcloud back13:00
rfolcoping scrum13:00
rfolcoping scrum13:00
sshnaidm|roverweshay, we should have ara on overcloud, we don't have on undercloud13:00
rfolcomarios, rlandy panda|ruck zbr arxcruz13:00
weshaysshnaidm|rover, oh really? that's something everyone should see /me looks13:00
sshnaidm|roverweshay, always had13:01
mariosrfolco: sory joining13:01
sshnaidm|roverweshay, https://logs.rdoproject.org/63/22163/4/check/tripleo-ci-centos-7-ovb-3ctlr_1comp-featureset001-master-vexxhost/0ecdaac/logs/ara_oooq_overcloud/ara-report/13:01
*** dsneddon has quit IRC13:02
rfolcozbr, rlandy13:02
*** bogdando has quit IRC13:06
*** bogdando has joined #oooq13:09
*** Goneri has joined #oooq13:11
weshayzbr, :) thanks for joining13:11
*** matbu has quit IRC13:11
*** bogdando has quit IRC13:11
*** matbu has joined #oooq13:12
weshaycan we supress the logging on  TASK [promoter : clone promoter dir from local dir if present] *****************13:17
*** aakarsh|2 has joined #oooq13:18
*** aakarsh|3 has joined #oooq13:21
zbrchandankumar: ok if I update 22299 to fix the looping?13:23
chandankumarzbr: yes13:23
*** aakarsh|2 has quit IRC13:23
chandankumarpanda|ruck: Do we need to enable container login for standalone jobs also na?13:24
panda|ruckchandankumar: for everything.13:24
-openstackstatus- NOTICE: The Gerrit service on review.opendev.org will be offline briefly starting at 14:00 UTC (that's roughly 30 minutes from now) for maintenance: http://lists.openstack.org/pipermail/openstack-discuss/2019-September/009064.html13:27
*** skramaja has quit IRC13:27
*** bogdando has joined #oooq13:27
sshnaidm|roverchandankumar, for periodic ones, right?13:34
*** dsneddon has joined #oooq13:35
weshaysshnaidm|rover, thanks for pointing out ara..13:35
weshaysshnaidm|rover,  looks like the upstream jobs are not rendering the ara page https://storage.gra1.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_879/682055/1/check/tripleo-ci-centos-7-containers-multinode/879e1c1/logs/ara_oooq_overcloud/13:36
weshayhttps://8729da22a49c2d7a78d7-1b102124c6fd0b6d71635117190cffc0.ssl.cf5.rackcdn.com/672969/3/check/tripleo-ci-centos-7-containers-multinode/a21975e/logs/ara_oooq_overcloud/ara-report/13:36
weshayis that something infra blocked us on?13:36
sshnaidm|roverweshay, yes, it's after infra changed their logs handling13:36
weshayfun13:36
chandankumarzbr: since we get rid of assert, we can do that same for docker part also13:38
zbryeah, already done that but I am also collapsing them into a single task13:39
*** dsneddon has quit IRC13:40
chandankumarpanda|ruck: sshnaidm|rover: I started testing with standalone one,  so we need to enable login for all the base which is defined here https://github.com/rdo-infra/review.rdoproject.org-config/blob/master/zuul.d/tripleo-rdo-base.yaml except container build jobs as they have different playbook?13:41
weshayzbr,  https://github.com/rdo-infra/ci-config/blob/master/ci-scripts/dlrnapi_promoter/config/RedHat-8/master.ini13:43
panda|ruckchandankumar: why selecting the jobs to login ? why not just enable it for everything ?13:43
sshnaidm|roverpanda|ruck, you don't auth with docker.io in regular jobs13:44
weshayhttps://meet.google.com/oei-zqpp-mia13:45
panda|ruckauth was not meant for docker.io13:45
panda|ruckauth was tied to the fact we wanted to close down rdo-registry13:45
weshaypanda|ruck,  sshnaidm|rover  do you guys have any idea if zoltan got the image server clean up script done? https://tree.taiga.io/project/morucci-software-factory/issue/296113:47
sshnaidm|roverweshay, no, but I can ask him13:48
*** chem` has joined #oooq13:48
*** chem has quit IRC13:49
zbrweshay: sshnaidm|rover : is this what we needed https://review.rdoproject.org/r/c/22306/ ?13:49
weshayzbr, please also add fs00113:50
weshayzbr,  then sshnaidm|rover  and panda|ruck can review13:50
*** holser has quit IRC13:50
*** surpatil has quit IRC13:51
zbrweshay: done, updated and also sorted list.13:52
weshayzbr, rock on.. thanks!13:53
weshayzbr,  this closes out that user story :)13:53
*** soniya29 has quit IRC13:54
weshaywell done folks https://tree.taiga.io/project/tripleo-ci-board/epic/94013:55
weshaychandankumar,  last one is auth :)13:55
weshaypanda|ruck,  sshnaidm|rover wow.. 92% pass rate in upstream :))13:57
weshaythat's as high as I've see that in the cockpit13:57
*** gchamoul has quit IRC13:58
weshayzbr,  sorry.. one more thing14:00
weshayzbr,  http://cistatus.tripleo.org/promotion/14:00
weshayscen001/002 need to be added to that config14:00
weshayzbr,  pull request to sshnaidm's sova repo14:00
-openstackstatus- NOTICE: The Gerrit service on review.opendev.org is offline briefly for maintenance: http://lists.openstack.org/pipermail/openstack-discuss/2019-September/009064.html14:01
*** ChanServ changes topic to "The Gerrit service on review.opendev.org is offline briefly for maintenance: http://lists.openstack.org/pipermail/openstack-discuss/2019-September/009064.html"14:01
zbrmarios: where?14:02
*** ykarel is now known as ykarel|afk14:03
weshayzbr https://meet.google.com/oei-zqpp-mia14:03
*** holser has joined #oooq14:07
*** dsneddon has joined #oooq14:07
weshayzbr, did you understand my sova comment?14:08
zbrzbr: did not see it, where?14:09
weshay<weshay> zbr,  sorry.. one more thing14:10
weshay<weshay> zbr,  http://cistatus.tripleo.org/promotion/14:10
weshay<weshay> scen001/002 need to be added to that config14:10
weshay<weshay> zbr,  pull request to sshnaidm's sova repo14:10
*** gchamoul has joined #oooq14:10
zbrweshay: ahh... so that is not using our config.14:11
weshayzbr, sova does not use the promotion ini files.. no14:11
*** dsneddon has quit IRC14:11
weshaysshnaidm|rover,  is this correct for the promotion jobs? https://github.com/sshnaidm/sova/tree/promotion14:12
weshayzbr, https://github.com/sshnaidm/sova/blob/promotion/tripleoci/config.py#L143-L14714:12
sshnaidm|roverweshay, yes14:13
weshaycool.. thanks14:13
zbrweshay: doing it. can't we just move sova to rdo? using github is a PITA14:13
weshayzbr,  yes that's an option, but I don't really care too much14:14
sshnaidm|roverzbr, firstly need to review this patch https://review.opendev.org/67863014:14
sshnaidm|roverweshay, ^14:14
weshayzbr,  retro idea for improvement14:14
sshnaidm|roveroh, gerrit is down14:14
zbryeah. once case where gh sucks less, it is up.14:16
zbranyway, gerrit uptime was really good, cannot complain.14:16
zbryou should have seen how often jenkins was going down downstream....14:17
zbrsshnaidm|rover: a hint on how to add the missing periodics in sova?14:17
sshnaidm|roverzbr, just add names to this list: https://github.com/sshnaidm/sova/blob/promotion/tripleoci/config.py#L143-L14714:18
sshnaidm|roverzbr, in pull request14:18
zbrthanks14:18
zbrsshnaidm|rover: OMG, not on master branch... :( :( :(14:19
zbrthat is why i was clueless14:19
sshnaidm|roverzbr, yeah, on "promotion" branch14:19
chandankumarzbr: now we have two reviews https://review.rdoproject.org/r/22299 and https://review.rdoproject.org/r/22305 can we merge in a single one?14:22
zbri created is separated as i was not ready yet and was too different. obbiosuly that we will merge only one of them14:23
zbrhttps://github.com/sshnaidm/sova/pull/7814:24
*** akahat has quit IRC14:26
chandankumarsshnaidm|rover: let's merge zbr patch then https://review.rdoproject.org/r/#/c/22305/14:26
chandankumarI will abandon mine14:26
zbrchandankumar: wait few more minutes to consolidate it, i want to make it a single task14:26
zbronly the var name is different14:26
chandankumarok14:26
*** ykarel_ has joined #oooq14:26
zbrless code/tasks = less bugs14:27
*** ykarel_ has quit IRC14:27
sshnaidm|roverzbr, I think you need registry_login_docker.results, not registry_login_docker14:29
zbryeah, thanks14:29
zbrtrying to figure out how to combine, i guess with ternary14:29
weshaychandankumar, rfolco  planning..14:30
*** ykarel|afk has quit IRC14:30
weshaychandankumar,  rfolco https://bluejeans.com/587845809714:30
weshaychandankumar, you there?14:31
chandankumarweshay: joined14:31
sshnaidm|roverzbr, see my comment to previous patch sets14:31
sshnaidm|roverzbr, with map(attribute), list and join14:31
*** sanjayu_ has quit IRC14:32
*** sanjayu_ has joined #oooq14:32
sshnaidm|roverzbr, actually if you loop over results, you don't need to combine14:33
*** ChanServ changes topic to "Docs: https://docs.openstack.org/tripleo-quickstart/latest/"14:33
-openstackstatus- NOTICE: The Gerrit outage portion of the current maintenance is complete and the service is back on line, however reindexing for renamed repositories is still underway and some Zuul job fixes are in the process of being applied14:33
*** dsneddon has joined #oooq14:41
*** dsneddon has quit IRC14:46
mariosarxcruz: sync tomorrow morning OK for you?14:46
mariosarxcruz: we can have a quick call i think its the easiest? are you busy in the morning14:47
arxcruzmarios: sure14:47
mariosarxcruz: i'll send an invite so we can make it official :D14:47
arxcruzmarios: 5 minutes before right? :D14:47
mariosarxcruz: taiga/review please?14:47
mariosarxcruz: now doing14:48
mariosbut 17:47 < marios> arxcruz: taiga/review please?14:48
arxcruzmarios: https://tree.taiga.io/project/tripleo-ci-board/task/1204?kanban-status=144727514:48
arxcruzmarios: brb14:48
mariosarxcruz: thanks check email in like 5 mins :D14:49
mariosweshay: cool, calendar seems to be integrated with meet :D14:50
mariosweshay: when you schedule stuffs14:50
mariosarxcruz: sent let me know if you want to move the time14:53
mariosarxcruz: anything works for me either side of that14:53
zbrsshnaidm|rover: let me know if this version looks ok https://review.rdoproject.org/r/c/22305/14:54
*** sanjayu_ has quit IRC14:54
sshnaidm|roverzbr, looking14:55
sshnaidm|roverchandankumar, arxcruz  did you see this error? http://logs.rdoproject.org/35/681635/1/openstack-check/tripleo-ci-rhel-8-standalone-rdo/b3c9c4f/logs/undercloud/home/zuul/tempest.log.txt.gz14:56
sshnaidm|roverchandankumar, arxcruz  it's from patch: https://review.opendev.org/#/c/681635/114:56
zbri do assume that podman/docker return code can be trusted14:56
bogdandorlandy: hi15:01
sshnaidm|roverzbr, I'd prefer to have 2 tasks as it was, you change the logic here and bring complexity15:01
rlandybogdando" hello15:01
bogdandorlandy, sshnaidm|rover: with https://review.opendev.org/#/c/682346/ and https://review.opendev.org/#/c/682092/ I have now a multi-node jobs deploying with CI repro15:01
sshnaidm|roverzbr, in original patch there was nothing about registry_login_docker defined or not15:01
zbri can replace that with the distro check15:02
bogdandothere is also a few https://review.rdoproject.org/r/#/q/topic:ci-repro+(status:open)15:02
zbrsshnaidm|rover: are these two tasks mutually exclusive or not?15:03
sshnaidm|roverzbr, they depend on distro as I can see15:03
sshnaidm|roverweshay, ready?15:03
rlandybogdando: k - will take a look when all the ci completes15:04
bogdandothanks15:04
sshnaidm|roverbogdando, rlandy consider also legacy pre playbooks in rdo15:07
*** pierrepr1netti is now known as pierreprinetti15:10
rlandymarios: rfolco: http://pastebin.test.redhat.com/797530 - query command for v2 repos15:10
*** ykarel has joined #oooq15:12
mariosrlandy: thank you checking & saving15:13
*** dsneddon has joined #oooq15:13
weshaysshnaidm|rover, I'm in15:13
chandankumarsshnaidm|rover: https://review.opendev.org/681352 might fix it15:15
chandankumara lot of patches coming soon related to nova less deploy15:16
*** dsneddon has quit IRC15:18
*** dtantsur|bbl is now known as dtantsur15:23
rlandypanda|ruck: hi - have a few minutes to discuss fixtures for promoter staging set up? http://pastebin.test.redhat.com/797545 - I'm adding the loval registry and the namespace15:39
rlandylocal15:39
panda|ruckrlandy: the error is related to an existing sqllite db file, means that the cleanup was not done correctly15:42
rlandypanda|ruck: can I manually clean up?15:43
*** amoralej is now known as amoralej|off15:44
panda|ruckrlandy: yes, the database and the tree are in /tmp/ for the pytest part15:47
weshayrfolco, we need to have a chat w/ Mike from IBM re:15:50
weshayhttp://lists.openstack.org/pipermail/openstack-discuss/2019-September/009374.html15:50
weshayrfolco,  can you please get in touch w/ him asap15:50
*** dsneddon has joined #oooq15:52
rfolcomjturek, ^^15:52
*** ccamacho has quit IRC15:52
rfolcomjturek, need to answer that thread ^15:53
rfolcobaha, fyi ^15:54
mjturekwill do!15:54
*** dsneddon has quit IRC15:57
*** ccamacho has joined #oooq15:58
mjturekThe rabbitmq issue confuses me as that's not preventing the builds...15:58
mjturekshould we be concerned there?15:58
*** dsneddon has joined #oooq16:00
rfolcoweshay, mjturek baha wanna chat about it ? what about now ?16:01
mjturekrfolco sure, doing some investigation now16:02
rfolcomjturek, ok, let me know if you need to chat. I'll be around.16:02
*** holser has quit IRC16:04
mariosttyl folks16:10
*** marios is now known as marios|out16:10
*** bogdando has quit IRC16:10
weshayzbr, ping me when you have 5min16:13
weshayre: centos816:13
mjturekrfolco, baha, weshay: Maybe we should have a chat as rfolco suggested to get us all on the same page16:15
mjturekI'm available whenever16:15
weshayme too16:15
weshayrfolco, you avail?16:15
rfolcoweshay, ok16:15
rfolcoweshay, your blue? https://bluejeans.com/587845809716:16
*** marios|out has quit IRC16:16
rfolcomjturek, baha ^16:17
zbrweshay: yeah16:18
*** kopecmartin is now known as kopecmartin|off16:18
*** ykarel is now known as ykarel|away16:24
*** ccamacho has quit IRC16:27
*** udesale has quit IRC16:32
*** ykarel|away has quit IRC16:34
*** rlandy is now known as rlandy|brb16:44
weshayzbr,  now a good time?16:46
zbryeah. bj?16:46
weshayzbr https://meet.google.com/fmo-dgtm-std16:47
mjturekweshay rfolco baha: responded here http://lists.openstack.org/pipermail/openstack-discuss/2019-September/009383.html16:47
rfolcomjturek, thx16:47
*** sshnaidm|rover is now known as sshnaidm|afk16:49
weshayzbr, https://tree.taiga.io/project/tripleo-ci-board/epic/108216:56
*** jpena is now known as jpena|off16:58
*** derekh has quit IRC17:00
*** jfrancoa has quit IRC17:13
*** dtantsur is now known as dtantsur|afk17:14
*** Goneri has quit IRC17:21
*** aakarsh|3 has quit IRC17:25
*** weshay has quit IRC17:25
*** jfrancoa has joined #oooq17:28
*** ykarel|away has joined #oooq17:29
*** rlandy|brb is now known as rlandy17:29
rfolcorlandy, can we chat?17:48
rlandyyes17:52
*** weshay has joined #oooq17:52
*** ccamacho has joined #oooq17:53
rfolcorlandy, https://meet.google.com/agw-dprx-qxe17:54
chandankumarzbr: rfolco rlandy weshay centos8 is coming https://twitter.com/CentOSProject/status/117365299630517043217:59
chandankumarwill there be centos8 ubi images also18:00
chandankumar?18:00
weshaychandankumar,  meh.. don't know18:00
weshaychandankumar,  zbr  is going to go after the nodepool image for us18:00
chandankumarweshay: cool, looking forward :-)18:00
chandankumarweshay: I think we can unblock the centos 8 epic, as it is no longer myth18:01
chandankumarweshay: zbr do we have cr repo for centos8?18:02
*** Goneri has joined #oooq18:02
weshaychandankumar,  nope.. I just got off the phone w/ zbr about it..18:02
weshaychandankumar, he'll be driving it18:03
chandankumarzbr: :-)18:03
chandankumarI hope all our python3 issues gets resolved18:03
chandankumarzbr: if you need any help nodepool image side for rdo, let me know I can help.18:07
zbrchandankumar: don't hyperventilate! that tweet does not mention centos 8, it could 7.7 on 24th.18:21
chandankumarzbr: yes, correct18:24
*** aakarsh has joined #oooq18:44
rfolcorlandy, whaaat... https://review.rdoproject.org/r/#/c/22256/ gate job failed?18:45
*** ykarel|away has quit IRC18:46
*** fmount has quit IRC18:52
*** aakarsh has quit IRC18:56
rlandyrfolco: sorry = my sister just dropped by .. back in a bit18:58
*** rlandy is now known as rlandy|biab18:59
rfolconp18:59
*** dsneddon has quit IRC19:05
*** holser has joined #oooq19:07
*** holser has quit IRC19:11
*** aakarsh has joined #oooq19:17
*** aakarsh|2 has joined #oooq19:21
*** aakarsh has quit IRC19:23
*** chem` has quit IRC19:28
*** ccamacho has quit IRC19:31
*** jtomasek has quit IRC19:34
*** holser has joined #oooq19:35
*** holser has quit IRC19:39
*** dsneddon has joined #oooq19:41
*** holser has joined #oooq19:51
*** dsneddon has quit IRC19:56
*** rlandy|biab is now known as rlandy19:57
rlandyrfolco: sorry - back19:57
rlandylooking at gate failure19:57
rlandyrerunning that19:57
rfolcorlandy, I am combining my 2 patches into a single one, and will rebase on top of yours when you fix local registry var19:58
rlandyrfolco: shoot - it merged19:58
rlandynon-voting19:58
rlandyugh19:58
rfolcorlandy, I don't know what happened there19:59
rfolcorlandy, rebase probably messed it up20:00
rlandyno idea - no log20:00
rlandywill recheck my patch20:00
*** pierreprinetti has quit IRC20:10
rlandyrfolco: test running now20:19
rlandywith variables20:19
rfolconice20:20
rlandywatching the molecule test to see what fails20:20
rfolcoI integrated the containers list patch in to the dlrn invokation patch20:20
rfolcoand also, when we rebase we can move cleanup part to separate file - https://review.rdoproject.org/r/#/c/21872/17/ci-scripts/infra-setup/roles/promoter/tasks/teardown.yml20:21
rlandyrfolco" sorry for earlier delay ... my sister and all her kids dropped by unexpectedly20:21
rfolcorlandy, I need the cleanup separated from the setup20:21
rfolcorlandy, stop apologizing :)20:21
*** jfrancoa has quit IRC20:21
rfolcoits all good timing, wan't blocked at all20:22
rfolcowasn't*20:22
rlandyrfolco: http://pastebin.test.redhat.com/797657 - molecule failure20:22
*** dsneddon has joined #oooq20:24
rlandy2019-09-16 20:21:41.488133 | rdo-fedora-stable |     __import__(name)20:25
rlandy2019-09-16 20:21:41.488282 | rdo-fedora-stable |   File "/home/zuul/src/review.rdoproject.org/rdo-infra/ci-config/.tox/molecule/lib/python2.7/site-packages/pytest_molecule/__init__.py", line 11, in <module>20:25
rlandy2019-09-16 20:21:41.488326 | rdo-fedora-stable |     from molecule.config import molecule_drivers20:25
rlandy2019-09-16 20:21:41.488370 | rdo-fedora-stable | ImportError: cannot import name molecule_drivers20:25
rlandyzbr: ^^?20:25
rlandythat is a new error20:25
*** dsneddon has quit IRC20:29
rlandyhttps://github.com/rdo-infra/ci-config/commits/master20:36
rlandydon;t see any changes there20:36
rlandyhttp://logs.rdoproject.org/55/22255/7/check/rdo-tox-molecule/8bd9cdd/tox/reports.html - hmmm - canb't see what versions were installed20:38
rlandyrfolco: ^^ idk - question for zbr20:41
rlandyidk what changed20:41
*** aakarsh|2 has quit IRC20:46
rlandyrfolco: ha - it's zbr's doing https://github.com/ansible/molecule/commit/745b370aff3edb99b3de9e7545b147e5eec58c7d20:48
rlandyzbr: ping, ping20:49
*** dsneddon has joined #oooq21:01
*** dsneddon has quit IRC21:06
*** panda|ruck has quit IRC21:10
rlandydo we lock our molecule version?21:15
*** aakarsh|2 has joined #oooq21:15
*** panda has joined #oooq21:16
*** dsneddon has joined #oooq21:23
rlandyrfolco: https://github.com/pycontribs/pytest-molecule/pull/40/files21:24
rlandyabr's fix ^^21:24
rlandyzbr's fix21:24
rlandynot merged yet21:24
rfolcorlandy, thx, will rebase on your patch later21:36
rlandyrfolco: the tests won;t run :(21:37
rlandysent email21:37
rlandyto ask zbr to push the fix21:37
*** holser has quit IRC21:54
*** holser has joined #oooq22:34
*** tosky has quit IRC22:35
*** Tengu has quit IRC23:11
*** holser has quit IRC23:35
*** rlandy has quit IRC23:43

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