Tuesday, 2020-04-07

*** rlandy|afk is now known as rlandy00:22
*** jmasud has quit IRC00:54
*** jmasud has joined #oooq00:56
*** Goneri has quit IRC02:00
*** ysandeep|away is now known as ysandeep|rover02:02
ysandeep|roverweshay_, o/02:10
weshay_ysandeep|rover, hey02:10
weshay_ysandeep|rover, ovb is fixed02:10
weshay_was rdo cloud02:10
weshay_weee02:10
ysandeep|roverweshay_,  routing issue as we suspected?02:11
weshay_not sure if scenario001 is totally fixed .. it passed in promotion jobs.. so I bypassed the criteria and got master promoted02:11
weshay_ysandeep|rover, not sure.. they haven't said yet.. :(02:11
weshay_ysandeep|rover, so we're good on promotions http://rhos-release.virt.bos.redhat.com:3030/rhosp02:12
weshay_ysandeep|rover, I have a lot of ptl work to do this week.. it's m302:12
ysandeep|roverweshay_, no worries.. I will try to put an extra eye monitoring the queues.02:13
weshay_ysandeep|rover, k.. cool and be on the look out for improvements etc.. downtime is a good time to improve this stuff02:14
weshay_ysandeep|rover, this would have blocked that collectd puppet issue https://review.opendev.org/#/c/717760/02:15
ysandeep|roverweshay_, yes just saw your email about this.. added myself as an observer on this gerrit02:16
weshay_ysandeep|rover, also creating centos-8 train containers and jobs to unblock EmilienM02:16
weshay_https://review.opendev.org/#/c/717850/02:16
ysandeep|roverlooking02:17
weshay_I lost my login for my znc bouncer lolz02:17
weshay_DOH02:18
weshay_heh.. well.. molecule is working lolz https://80b696ebd10ce1a8679e-b5788ff47e43c8b1d4d7bc79b7243ed8.ssl.cf2.rackcdn.com/717850/1/check/openstack-tox-molecule/aa0bb20/tox/reports.html02:18
weshay_ysandeep|rover, looks like we need to build the overcloud-images as well.. to make the tests happy02:19
weshay_I have not updated the hackmd much today02:19
weshay_ysandeep|rover, was hoping you could xfer the ownership of the standalone hackmd you created to rh-openstack-ci02:20
weshay_in documentation02:20
ysandeep|roverweshay_, done that already, yesterday02:20
rlandyysandeep|rover: oh wow - is it morning now for you?02:20
ysandeep|roverrlandy, yes 7:50 in the morning :)02:21
rlandylol - I'm still trying to close out monday's work02:21
weshay_howdy rlandy  :)02:21
rlandyweshay_: hey02:21
rlandyone review for you ... based off yours ... https://review.opendev.org/#/c/717895/02:22
weshay_ah very nice02:22
rlandystill need the pre before marios wakes up02:22
* weshay_ looks02:22
weshay_rlandy, looks good02:23
rlandyweshay_: we'll need a few internal vars02:23
weshay_aye02:24
rlandyadding that to job def02:24
ysandeep|roverweshay_, failed to validate https://images.rdoproject.org/centos8/train/rdo_trunk/tripleo-ci-testing/ironic-python-agent.tar .. https://images.rdoproject.org/centos8/ only master here02:25
weshay_ysandeep|rover, ya.. we haven't built any of those02:25
weshay_so now we have the ol' chicken and egg problem02:26
weshay_meh02:26
ysandeep|roverweshay_, :) okay02:26
weshay_I'll build and push overcloud images tomorow..02:26
weshay_if you want to give it a go w/ chandankumar his head is all in overcloud-images atm I think02:26
ysandeep|roverweshay_, ack o/ i will keep an eye to know how to do that02:26
ysandeep|roverweshay_, so requirement is we need overcloud-images for centos8 train branch.. right?02:28
ysandeep|roverwill check if i can find some old review and take chandan to put them up before your morning :)02:30
weshay_ysandeep|rover, essentially parent off the centos-8 master job for overcloud-full and overcloud-ipa02:34
weshay_you can put up the zuul change.. and then run a test project w/o damaging anything...02:34
weshay_not sure if the image server will allow the job to create the dir structure..02:34
weshay_you'll need infra or myself to do that bit02:34
weshay_http://images.rdoproject.org/centos8/02:35
weshay_might be good to see how that all works02:35
ysandeep|roverRegarding parent off centos-8 master job for overcloud-full and overcloud-ipa - Just want to confirm - we need new jobs for centos8 train branch?02:38
ysandeep|roverweshay_, ^^02:41
weshay_ya.. we have centos-7... side project is centos-8 atm02:41
ysandeep|roverweshay_, thank you02:42
*** hamzy has quit IRC02:46
*** hamzy has joined #oooq02:46
*** ysandeep|rover is now known as ysandeep|afk03:13
*** rlandy has quit IRC03:37
*** ysandeep|afk is now known as ysandeep|rover04:18
*** ykarel|away is now known as ykarel04:20
*** jmasud has quit IRC04:54
*** jmasud has joined #oooq04:55
*** ratailor has joined #oooq05:00
*** marios has joined #oooq05:16
*** saneax has joined #oooq05:20
chandankumarysandeep|rover, weshay_ i will take care of that05:21
ysandeep|roverchandankumar, thanks!05:22
weshay_chandankumar, ysandeep|rover we need to spot check these https://review.rdoproject.org/zuul/builds?pipeline=openstack-promote-component%2005:22
weshay_make sure the right thing is happening05:23
weshay_see #rdo05:23
weshay_0/05:23
weshay_take care guys and thank you05:23
mariosweshay_: o_O what are you doing still up ?!05:23
ysandeep|roverweshay_, good night ..zzzzz...05:23
*** udesale has joined #oooq05:41
*** udesale has quit IRC06:09
*** udesale has joined #oooq06:13
*** bogdando has joined #oooq06:26
*** dpawlik has joined #oooq06:30
*** ratailor has quit IRC06:40
*** ratailor has joined #oooq06:47
*** amoralej|off is now known as amoralej07:03
*** apetrich has quit IRC07:24
*** tosky has joined #oooq07:24
zbro/07:27
zbrhttps://review.opendev.org/#/c/717723/ really needs review as is sprint work and blocking other work.07:28
zbralso https://review.opendev.org/#/q/topic:ensure+(status:open+OR+status:merged) about migration to ensure07:39
*** ysandeep|rover is now known as ysandeep|lunch07:44
chandankumarysandeep|lunch, Hello07:45
chandankumarysandeep|lunch, regarding overcloud build image, we need images for train?07:45
*** apetrich has joined #oooq07:47
*** pojadhav is now known as pojadhav|lunch07:49
*** jschlueter has quit IRC08:03
*** jschlueter has joined #oooq08:05
ysandeep|lunchchandankumar, yes08:23
*** jfrancoa has joined #oooq08:30
*** ccamacho has joined #oooq08:36
*** derekh has joined #oooq08:38
*** holser has joined #oooq08:39
*** skramaja has joined #oooq08:42
*** holser has quit IRC08:42
*** holser has joined #oooq08:42
*** ysandeep|lunch is now known as ysandeep|rover08:44
*** ykarel is now known as ykarel|lunch08:48
sshnaidm|afkzbr, where is the example of zuul failure with filter?08:57
*** sshnaidm|afk is now known as sshnaidm08:57
zbrsshnaidm: no idea, arxcruz should know better, he reported the issue, but i got confirmed from infra that we cannot use filters with zuul.08:58
zbrdue to security issues, as filters run on controller08:58
zbralways08:58
arxcruzsec08:58
zbrbut modules are ok08:58
sshnaidmzbr, ok, but where is the report?08:58
arxcruzsshnaidm: https://review.opendev.org/#/c/702676/08:59
arxcruzERROR Ansible plugin dir /var/lib/zuul/builds/cb344fc5b915483c95e10201a5945da3/ansible/pre_playbook_1/role_1/ansible-role-collect-logs/filter_plugins found adjacent to playbook /var/lib/zuul/builds/cb344fc5b915483c95e10201a5945da3/ansible/pre_playbook_1/role_1/ansible-role-collect-logs in non-trusted repo. in 2m 01s08:59
sshnaidmzbr, seems like a huge overhead for a such task, maybe to try use current jinja/ansible filters?08:59
zbrthat is a good question, irc logs for sure but that is a really bad place to document bugs.08:59
sshnaidmarxcruz, ack08:59
zbrsshnaidm: you are invited! but we warned, there are daemons around jinaja209:00
sshnaidmzbr, btw, we run a nested ansible, why should we care?09:00
zbrfew months back I tried to implement it in pure jinja2, and after two days, I decided that it does not worth the efforth09:00
sshnaidmoh, it's openstack-ansible role..09:01
zbri like jinja2 (maybe more than most of the people from the team), but it has its own limitations09:01
*** dtantsur|afk is now known as dtantsur09:01
sshnaidmzbr, can you please paste input and desired output of such filter? I'll look09:01
sshnaidmzbr, filters are fine, but module? it's kind of too far09:01
sshnaidmwhat's next, a special repository09:02
zbrthe desired output can be found in the molecule validation code, it does check for output there.09:02
sshnaidmzbr, thanks, but can you please link to it specifically?09:02
zbrsshnaidm: that was my first guess, but once you find more about filters/plugins/modules differences, you realise that modules are the least-problematic ones.09:03
zbrmy original desire was to add unittests to the module, instead of using molecule to test it, but I had no time. in fact the module itself documents the expected format!09:04
zbris in the doc section of the module09:04
sshnaidmzbr, can you paste link to input and output?09:04
sshnaidmarxcruz, so openstack-ansible doesn't run collect-logs in nested ansible, right?09:04
arxcruzsshnaidm: it doesn't run at all, this is just an attempt to replace their collect-logs script with our role09:05
sshnaidmarxcruz, in that case I think this filter issue is their smallest problem09:05
sshnaidmarxcruz, we have references to localhost and copying files to it, w/o nested it will try to run on the executor and fail09:06
arxcruzsshnaidm: i see09:06
sshnaidmarxcruz, so I don't think it's a small amount of work..09:06
arxcruzsshnaidm:  :(09:06
sshnaidmarxcruz, but will be fun09:07
sshnaidmdon't be sad09:07
arxcruzlol09:09
*** pojadhav|lunch is now known as pojadhav09:12
sshnaidmzbr, so, do you have input/ouput for a filter?09:12
chandankumarysandeep|rover, do you know why we need centos-8 based train images?09:16
sshnaidmarxcruz, tbh, I'd try firstly to run this role in zuul in our repos, not in openstack-ansible09:17
sshnaidmmuch easier09:17
chandankumarykarel|lunch, i got it09:18
ysandeep|roverweshay_, I am not entirely sure but I guess someone requested it .. wes mentioned "he is creating centos-8 train containers and jobs to unblock EmilienM"09:19
chandankumarykarel|lunch, https://review.rdoproject.org/r/2628509:19
ysandeep|roverchandankumar, I am not entirely sure but I guess someone requested it .. wes mentioned "he is creating centos-8 train containers and jobs to unblock EmilienM"09:19
pandachandankumar: ysandeep|rover ok if I promote back d189b188525ce9f703b8829c5c9995c2 for centos8/master ?09:22
chandankumarykarel|lunch, ^^09:23
pandachandankumar: he proposed it, he's ok with it09:23
chandankumarpanda, ok then go ahead09:23
pandachandankumar: but maybe you knew why wes forced the promotion waiving OVB jobs ...09:24
chandankumarpanda, I have no idea09:24
ysandeep|roverpanda, i have some idea09:24
pandaysandeep|rover: beware, ideas are dangerous09:24
pandaysandeep|rover: don't go around saying you have some.09:25
ysandeep|roverwes noticed some of the multinode jobs were timing out, he mentioned this happens when the content in upstream gets old and requires a lot of yum updates09:25
pandaysandeep|rover: so we needed promotion at all costs to fix multinode09:26
pandaysandeep|rover: ykarel|lunch so either we lose OVB jobs or we lose multinode jobs.09:26
pandaoh joy.09:26
ysandeep|roverhe did submit a change.. i think it was https://review.rdoproject.org/r/#/c/26281/.. the promotion-server did not pull the change.. so he edited it by hand.. i mean promote09:26
pandaysandeep|rover: yep, automatic code updates are disabled ....09:26
pandaysandeep|rover: you're confirming everything I thought09:27
ysandeep|roverpanda, i am glad i confirmed your thoughts09:28
pandaysandeep|rover: I'm glad you're glad09:28
ysandeep|roverhahaha09:28
chandankumarysandeep|rover, https://review.rdoproject.org/r/#/c/26287/ will build c8 images for train09:34
ysandeep|roverchandankumar, thank you!09:35
zbrpanda: if you can have another look at tox ugfrade one it would be great https://review.opendev.org/#/c/690057/09:37
pandazbr: I'm ugfrade I won't have time.09:38
*** ykarel|lunch is now known as ykarel09:41
sshnaidmzbr, sorry, I need to ask you again - any chance you have input and output example for your filter/module?09:41
pandazbr: done09:42
zbrsshnaidm: clearly you did not read the CR because both input and output examples are there: https://review.opendev.org/#/c/717723/4/library/flatten_nested_dict.py09:44
sshnaidmzbr, I did, and this didn't work09:45
sshnaidmzbr, I think there is incorrect example, that's why I'm asking09:45
ykarelpanda, ysandeep|rover /me not aware about multinode timeouts, i just see http://zuul.openstack.org/builds?job_name=tripleo-ci-centos-8-containers-multinode and from here i see jobs with older hash too finished before 2.5 hrs10:02
ykareland also not always container updates cause of timeout10:02
*** ykarel is now known as ykarel|meeting10:02
ykarel|meetingme ok to wait for wes so he can confirm10:02
chandankumarmarios, Hello10:31
chandankumarmarios, for internal container build, you have written playbook for downloading internal dlrn repos? If yes, I wanted to reuse it10:32
marioschandankumar: not yet but we will do10:32
marioschandankumar: there are some notes though on hackmd (rlandy scripts)10:33
chandankumarmarios, checking10:33
marioschandankumar: https://hackmd.io/6gE-LZYMSzmoP3IFt7puKg#script-for-local-clone-and-copy10:33
chandankumarmarios, thanks!10:36
*** ykarel|meeting is now known as ykarel10:37
mariossshnaidm: i banged my head trying to get that to work https://review.opendev.org/#/c/716639/6/roles/tripleo_overcloud_roles_generate/tasks/main.yml@11 ... i made it a list but for some reason my local env refused to pickup the change ( | join(' ')) ... not sure if there is some weird cache issue going on10:39
mariossshnaidm: i updated it anyway posted v8 just now https://review.opendev.org/#/c/716639/ lets see if it passes in ci10:39
mariossshnaidm: should do, afaics but let me know if you can spot issue thanks10:39
sshnaidmmarios, what is the error you got?10:40
mariossshnaidm: like it completely ignored what I put into the tasks/main... i made it like  {{ tripleo_overcloud_roles_generate_roles | join(' ') }} but it ignores the join in the cmd_output it just spits out "_generate_roles_cmd": " echo overcloud roles generate --roles-path /foo --output-file /home/m/molecule_roles_test.yml --skip-validate   [u'Compute', u'Controller']"10:42
mariossshnaidm:  will be easier to see once it runs in ci i'll harrass you again later ;)10:42
sshnaidmmarios, look at this as an example: https://github.com/openstack/tripleo-operator-ansible/blob/master/roles/tripleo_overcloud_node_introspect/molecule/default/converge.yml#L19-L3110:45
sshnaidmmarios, in assert blocks should be strings without variables, it's a final command10:45
mariossshnaidm: yeah i saw that also more like node delete does same.. problem isn't the match ... i used that for example https://review.opendev.org/#/c/716639/9/roles/tripleo_overcloud_roles_generate/molecule/default/converge.yml@7910:46
chandankumarkopecmartin, can you take care of this https://review.rdoproject.org/r/#/c/26289/?10:46
chandankumarmarios, we have added http://osp-trunk.hosted.upshift.rdu2.redhat.com/rhel8-osp17/tripleo-ci-testing/ toci testing dir on internal dlrn server10:47
mariossshnaidm: http://paste.openstack.org/raw/791723/10:47
mariossshnaidm: the problem is the *generated* command i.e. its ignoring the join in tasks/main not the assert match10:47
mariossshnaidm: (scroll to bottom of fpaste for error)10:48
sshnaidmmarios, well, I'd start from tests change, we shouldn't test if jinja works correctly, the final goal is to see a whole command and check that it's what we expect10:48
mariossshnaidm: k thanks waiting for the ci run and will revisit10:51
sshnaidmmarios, commented: https://review.opendev.org/#/c/716639/11:04
mariossshnaidm: thanks playing with it trying to get it to work11:05
mariossshnaidm: even though i said i'd switch it sucked me back in :/11:05
marios:D11:05
chandankumarmarios, https://review.opendev.org/715397 os_tempest is supported for train release11:09
marioschandankumar: for fs39?11:11
chandankumarmarios, yes11:11
marioschandankumar: this is centos8 fs39 though... didn't test fs39 centos7/train11:12
chandankumarmarios, ah, ok, got it11:12
marioschandankumar: as in my work/testing was all for centos8... i can change it to include ^^ but not sure it will work ok there didn't test that11:12
chandankumarmarios, we can deal train one in different patch11:13
marioschandankumar: ack added note there will have a look in a bit11:13
zbrpanda: does the https://review.opendev.org/#/c/690057/16/test-playbooks/ensure-tox.yaml looks better?11:27
zbrfound 3 bugs in one ansible task :p11:27
*** ysandeep|rover is now known as ysandeep|afk11:29
*** blueDuckRandom has joined #oooq11:40
*** ysandeep|afk is now known as ysandeep|rover11:56
*** sshnaidm is now known as sshnaidm|afk11:58
kopecmartinchandankumar: what is that? seems like hacking package is missing, but i have no idea why and where , i'm not a packager12:01
weshay_0/12:01
chandankumarkopecmartin, ok looking12:01
weshay_ysandeep|rover, ykarel chandankumar anything to hand off?12:02
weshay_https://hackmd.io/XMd_vskSTdGZl7ruZ1UckA12:03
weshay_reading through12:03
weshay_ykarel, wondering if there is any news re: the old python3-openstackclient package12:03
weshay_soniya29, good evening12:03
ykarelweshay_, false promotion12:03
soniya29weshay_: good evening. Meeting?12:03
weshay_ykarel, need more details12:03
ykarelwith adjusting criteria without ovb12:04
weshay_soniya29, one sec12:04
ykarelweshay_,  https://review.rdoproject.org/r/#/c/26281/2/ci-scripts/dlrnapi_promoter/config/CentOS-8/master.ini12:04
weshay_ykarel, that's not a false promotion12:04
ykarelweshay_, promoted wrong hash where ovb failed and repos were outdated https://trunk.rdoproject.org/api-centos8-master-uc/api/civotes_agg_detail.html?ref_hash=1c419f50bd835a5dd09338469b59b0c712:04
weshay_hrm...12:05
*** amoralej is now known as amoralej|lunch12:06
weshay_ykarel, k.. I'll look at this w/ panda12:06
ysandeep|roverweshay_, Got 2 new issues. determined one, checking other.. details in RR notes12:07
ykarelweshay_, ack12:07
weshay_ykarel, the promoter should take latest aggregated hash and promote that one12:07
ykarelweshay_, yes and that hash has issues,12:07
ykareldelorean.repo is wrong there12:07
weshay_ykarel, more detail on the wrong delorean.repo12:08
ykarelnot consistant with promoted contents12:08
weshay_soniya29, if your mic is not working.. pm in irc12:08
ykarelweshay_, see https://trunk.rdoproject.org/centos8-master/component/clients/promoted-components/delorean.repo vs dlrnapi --url https://trunk-centos8.rdoproject.org/api-centos8-master-uc promotion-get --promote-name promoted-components --component clients --limit 1 for small example, this for other componets went to tripleo-ci-testing and then to current-tripleo12:09
*** blueDuckRandom has quit IRC12:12
weshay_panda, will need about 1/2 hour of your time today12:13
pandaweshay_: sure12:14
weshay_ykarel, panda looks like this is not working https://logserver.rdoproject.org/openstack-promote-component/opendev.org/openstack/tripleo-ci/master/periodic-tripleo-centos-8-master-component-clients-promote-to-promoted-components/a119a9c/job-output.txt12:14
pandazbr: code doesn't even exist without tests.12:14
ykarelweshay_, likely because that hash is already promoted12:15
weshay_ykarel, hrm12:16
*** beagles_afk is now known as beagles12:17
mariossshnaidm|afk: weird... v11 is green in zuul but locally still fails... /me shrug https://review.opendev.org/#/c/716639/12:20
mariossshnaidm|afk: thanks for looking there ^12:20
*** rfolco has joined #oooq12:20
*** rlandy has joined #oooq12:24
pandamarios: can I see a log of the local failure ?12:24
mariospanda: let me get fpaste for latest run sec12:25
mariospanda: http://paste.openstack.org/raw/791731/12:26
chandankumararxcruz, can we use tripleo community meeting for tempest skip schema review?12:34
chandankumarzbr, sshnaidm|afk regarding ansible install, we can clone the ansible in ceph rpm build job and run make rpm there and put in ceph repo itself and install it12:35
weshay_panda, you have some time now?12:37
pandaweshay_: yes12:39
chandankumarweshay_, when you are free, I want to discuss about centos-8 train image build12:39
weshay_panda, ykarel https://meet.google.com/yjn-rwds-isr?authuser=112:39
weshay_ykarel, if you have time.. going to discuss the issues w/ promotions12:39
ykarelweshay_, sure joining12:40
weshay_panda, https://opendev.org/openstack/python-openstackclient/commits/branch/master12:54
weshay_ykarel, panda https://review.rdoproject.org/r/#/c/25767/12:58
weshay_ykarel, ssh promoter.rdoproject.org13:00
mariosrlandy: chandankumar: weshay_: o/ joining ?13:01
rlandymarios: sorry - joining13:02
*** udesale_ has joined #oooq13:03
*** udesale has quit IRC13:05
jfrancoaysandeep|rover: hi! just wondering if this issue is being tracked or not https://504a6afcff55c8f91b0c-2937bb652abe1e5f0c19a6a7c68db9e1.ssl.cf5.rackcdn.com/717861/3/check/tripleo-ci-centos-8-scenario001-standalone/3b7ad91/logs/undercloud/var/log/containers/stdouts/container-puppet-collectd.log , I think I've found the source of the problem and will submit a fix, but I would like to know if I need to open a LP bug first13:09
ysandeep|roverjfrancoa, we temporary fixed this yesterday.. let me find the lp13:10
jfrancoaysandeep|rover: great, thanks! that would be awesome (I did a quicksearch in LP but didn't find it)13:10
ysandeep|roverjfrancoa, https://bugs.launchpad.net/tripleo/+bug/187108613:10
openstackLaunchpad bug 1871086 in tripleo "tripleo-ci-centos-8-scenario001-standalone jobs failing with Container(s) with bad ExitCode: [''container-puppet-collectd'']" [Critical,In progress] - Assigned to wes hayutin (weshayutin)13:10
*** ratailor has quit IRC13:11
jfrancoaysandeep|rover: thanks a lot!13:12
ysandeep|roverjfrancoa, my pleasure, for now we pinned puppet-collectd to a previous good hash , proposed a pull request in puppet-collectd to fix the issue(make it compatible) and hardening our ci jobs so that these non tested puppet-* project codes don't gets in unnoticed.13:15
marioshttps://code.engineering.redhat.com/gerrit/gitweb?p=dist-git-manifests.git;a=blob_plain;f=combined-images.xml;hb=refs/heads/rhos-17.0-rhel-813:17
ysandeep|roverjfrancoa, please push a patch if you know a different way to fix this.13:23
*** Goneri has joined #oooq13:23
*** sshnaidm|afk is now known as sshnaidm13:27
rfolcotripleo ci community meeting starting in 2 min13:28
rfolcoping13:28
rfolcoarxcruz, rfolco, zbr, panda, sshnaidm, rlandy, marios, ysandeep, bhagyashris, soniya29, pojadhav, akahat, weshay, chandankumar13:28
rfolcohttps://meet.google.com/bqx-xwht-wky?authuser=113:28
soniya29rfolco, i am there13:28
bhagyashrisrfolco, me too13:28
rfolcoI see, I copy/paste the entire ping list :)13:28
jfrancoaysandeep|rover: I found the issue in the template, but if there are plans to rever the puppet-collectd patch, then it doesn't make sense to change it13:29
mariosrlandy: lets sync here... i'll try get the job definition out at least and you can continue to work on the pre and i'll take it when you leave... out tomorrow? or thursday13:30
ysandeep|roverykarel, weshay_ ^^ what do you think?13:30
*** amoralej|lunch is now known as amoralej13:30
rlandymarios: hey - I'm out from tomorrow13:31
mariosrlandy: ack13:31
rlandymarios: you're welcome to take over the review13:31
rlandymarios: all the reviews :)13:31
rlandymarios: I just left out one comment13:31
mariosrlandy: ok will do tomorrow anyway13:31
rlandyto copy back the macros13:31
rlandyso I'll add that and it's all yours13:31
rlandyI need to finish tls13:32
mariosrlandy: k13:32
chandankumarweshay_, ysandeep|rover https://hackmd.io/OAoW3fJkR72u5n-MTTBRzA?view#Deploy-standalone-using-tripleo-operator-ansible13:33
ysandeep|roverjfrancoa, Can you please point what was the issue in template?13:38
jfrancoaysandeep|rover: https://github.com/openstack/tripleo-heat-templates/blob/198c1f890dec38c31126d607a64059f3da9adc87/deployment/metrics/collectd-container-puppet.yaml#L262 the default is taking a string, when the puppet change made from this port option an integer. So removing the quotes should help13:40
ysandeep|roverykarel, weshay_ fyi.. whenever you are free ^^ we can unpin  puppet-collectd if we fix above..13:43
ykarelysandeep|rover, puppet-collectd is backward uncompatible, so quotes work with old puppet-collectd, and integer works with new puppet-collectd, this is what i understood the issue till now13:45
ysandeep|roveryes correct..13:47
rlandypojadhav: hi - I have some work to do on those reviews ... the role has moved to this repo https://opendev.org/x/tripleo-ipa/14:02
rlandyI am not sure how late you want to stay14:02
rlandypojadhav: I am out tomorrow to Friday but I can leave you email on where to pick up14:03
rlandychandankumar: sorry - didn't get to your review yesterday  - will look today14:03
pojadhavrlandy, sure14:03
rlandymarios: k - I updated the notes on the pre review - all yours14:04
rlandysend you info to log on to my test instance14:04
chandankumarrlandy, no problem14:04
mariosack thanks rlandy14:05
*** TrevorV has joined #oooq14:16
rlandyweshay_: hey - can I help with https://etherpad.openstack.org/p/tripleo-component-promotion-pipeline? ptg topic14:30
weshay_aye14:30
*** ysandeep|rover is now known as ysandeep|away14:33
*** skramaja has quit IRC14:46
*** skramaja has joined #oooq14:46
*** bogdando has quit IRC14:49
*** skramaja has quit IRC14:51
weshay_anybody need anything?14:54
* dpawlik thinking about new car14:57
marios:D dpawlik14:57
dpawlikmarios, you should write your wishes right now. Maybe weshay_ will help to make it true!14:58
* weshay_ calls oprash14:58
weshay_oprah14:58
weshay_YOU GET A NEW CAR14:58
rlandydpawlik has high ambitions ... atm I just want to go to the grocery store without a face mask15:02
dpawlikrlandy, you can always take a car and eat in mc donald :D15:04
dpawlikrlandy, without the mask15:04
dpawlikvia mc drive15:05
*** sshnaidm is now known as sshnaidm|afk15:07
*** sshnaidm|afk is now known as sshnaid15:27
*** sshnaid is now known as sshnaidm15:27
weshay_panda, are the changes we discussed still a good idea in your opinion? re: https://review.rdoproject.org/r/#/c/26294/15:31
weshay_panda, few more minutes of your time please when you have it15:40
pandaweshay_: here's the thing. We are not restricting the criteria for a hash to be promoted, we are widening them. Hashes will get promote if they never promoted OR they don't correspond to the latest promoted-component. This is as a workaround to an assumption that was recently broken with the packages. The broader criteria work in this case, but we may have some other cases where this doesn't work. I got15:41
pandafamiliar with the code a bit. There's no chance I could put satisfactory tests on this, so I can implement and merge, then we'll need to keep an eye on everything to see if something is breaking, and it may happen tomorrow of in one month. Without tests we are not covered against situations like this.15:41
weshay_panda, let's chat.. this is a little too complicated imhjo for irc15:41
weshay_panda, https://meet.google.com/ahs-soew-cpd?authuser=115:42
weshay_ykarel, are we sure consistent is clear of bad rsync artifacts?15:46
weshay_seemed like it was15:46
ykarelweshay_, yes current consistent are good wrt rsync issue15:47
weshay_ykarel, k15:48
mariosrlandy: see https://review.opendev.org/#/c/717895/2/roles/build-containers/templates/kolla-build.conf.j2 sound right? (I am going to add that to the job vars working on it now)15:52
rlandylooking15:52
rlandymarios: there is a redhat paste link with the vars15:53
rlandymarios: but you can choose it to be anything15:53
mariosrlandy: ah yes http://pastebin.test.redhat.com/852586 it was one of those 30000 tabs15:53
rlandymarios: np - it's on the hackmd - although that doc is also rather long these days15:54
mariosrlandy: but wrt namespace specifically ... not tripleo{{ osp_branch }} but more like rhosp... actually it doesnt matter we aren't pushing them anywhere come to think about it15:55
mariosrlandy: its all local build15:55
mariosrlandy: so tripleo{{ osp_branch }} should work fine too as long as it build ok  i.e. rhosp isn't expected by those dockerfiles15:55
rlandymarios: will check back in on sunday if you want changes reviewed ... wrt naming for eventual push weshay_ needs to confirm that with reldel15:55
mariosrlandy: k15:56
mariosrlandy: enjoy your time out15:56
rlandyright now, pick your poison15:56
mariosrlandy: are you out of isolation yet? or are they still passing you things at the end of a long stick :D15:56
rlandymarios: thanks - sorry for dumping these reviews, need to get TLS in some sort of working shape15:56
rlandymarios; yep - out of isolation - I took may maiden voyage to the grocery store last night15:57
mariosrlandy: no apologies i have also been working on this stuff as you know :) ... was planning on working the pre today anyway as we discussed earlier15:57
mariosrlandy: was it everything you imagined it would be ?! 18:57 < rlandy> marios; yep - out of isolation - I took may maiden voyage to the grocery store last night15:57
mariosrlandy: i am going to request permission to go for a run in a bit... i am so excited !15:57
rlandymarios: it was weird - shelves were pretty sparse15:57
marios:(15:58
rlandymarios: as in - no lettuce15:58
rlandybut you can still get 100+ varieties of breakfast cereal15:59
*** dpawlik has quit IRC16:00
rlandymarios: and the doctor's office sent out a message that they put an open tent in the garden - so you can now see the doctor w/o fear of getting corona - in full view of the neighbors16:01
rlandystrange times16:01
weshay_rfolco, where is the hackmd for the get_hash rework?16:01
rfolcohttps://hackmd.io/06K4wRKaT42axGljoDKtjQ16:02
weshay_rfolco, I need to add some notes16:02
weshay_rfolco, please move it to the org16:02
rfolcoweshay_, I transfered ownership already, but it does now show up16:02
rfolcoprobably because I created as personal gmail16:03
weshay_rfolco, add me as an admin16:03
rfolcoweshay_, you are admin16:03
mariosrlandy: :(16:03
weshay_rfolco, nope16:03
weshay_I am16:03
weshay_ok16:03
weshay_I see it now16:03
*** sshnaidm is now known as sshnaidm|afk16:03
weshay_rfolco, ok.. scrape it.. add a new one16:04
weshay_I'll do it16:04
rfolcoweshay_, yeah better to create a new one and remove that one16:04
weshay_rfolco, k.. archive.. or delete this one https://hackmd.io/06K4wRKaT42axGljoDKtjQ?both16:05
weshay_thanks16:05
rfolcoweshay_, do you have the new one?16:07
weshay_rfolco, https://hackmd.io/team/rh-openstack-ci?nav=overview16:07
*** udesale_ has quit IRC16:08
*** dtantsur is now known as dtantsur|afk16:08
*** ykarel is now known as ykarel|away16:10
mariosrlandy: just posting skeleton ... added the required-projects for now... will pickup tomorrow and try to wire it up with the pre and the tripleo-ci role review https://code.engineering.redhat.com/gerrit/#/c/196858/1/zuul.d/tripleo-build-containers.yaml16:18
mariosrlandy: bit of a maintenance issue... we'll need to keep the required-projects there ^^ updated16:20
rlandymarios; ack - commenting16:20
mariosrlandy: or we do some automation like tristan did at https://code.engineering.redhat.com/gerrit/#/c/196441/7/resources/osp-containers.dhall16:20
rlandywe need to do what chandankumar did in his review16:20
rlandycheckout out the right branch16:20
rlandyrest looks good - will add link16:20
mariosrlandy: ack will check it tomorrow and continue16:20
mariosrlandy: thanks16:20
mariosrlandy: have a good break16:20
rlandymarios: thanks - stay safe16:21
mariosu2 rlandy16:21
weshay_rlandy, where are the tls hackmd notes?16:24
rlandyweshay_: ade's notes? https://docs.google.com/document/d/1oTnB-x9V6OOOEWFPOxbzgCfy6_lL6oFlQVwKtffC1V4/edit?ts=5dde5c67#16:24
weshay_rlandy, hrm.. should I move that to a hackmd?16:25
rlandyweshay_: why bother?16:25
rlandywe have other work to do than move formats around16:25
*** marios is now known as marios|out16:28
*** marios|out has quit IRC16:30
*** jbadiapa has quit IRC16:57
*** amoralej is now known as amoralej|off17:28
chandankumarweshay_, do you have min to talk about train image build?17:51
*** holser has quit IRC17:52
weshay_chandankumar, I do17:55
weshay_https://meet.google.com/czg-hptr-njx?authuser=117:56
chandankumarweshay_, https://logserver.rdoproject.org/87/26287/1/check/periodic-tripleo-centos-8-buildimage-overcloud-full-train/262f590/job-output.txt17:59
*** holser has joined #oooq18:00
*** holser has quit IRC18:02
chandankumarweshay_, https://review.opendev.org/#/c/717043/18:04
chandankumarweshay_,  https://code.engineering.redhat.com/gerrit/19637318:04
chandankumarweshay_, https://hackmd.io/OAoW3fJkR72u5n-MTTBRzA?view#Deploy-standalone-using-tripleo-operator-ansible18:09
*** holser has joined #oooq18:13
*** ccamacho has quit IRC18:40
rlandyweshay_:  https://review.rdoproject.org/r/26304 Add x/tripleo-api and a repo in RDO18:56
rlandyrdo jobs are erroring out18:56
rlandyper https://review.opendev.org/#/c/706288/18:57
weshay_rlandy, https://review.rdoproject.org/r/#/c/26294/18:59
weshay_worst nightmare come tru18:59
weshay_will explain later19:00
rlandylooking19:01
rlandyoh dear ... https://bugs.launchpad.net/tripleo/+bug/187101019:03
openstackLaunchpad bug 1871010 in tripleo " Problem: package python3-tripleoclient-13.1.1-0.20200404091914.624a61f.el8.noarch requires validations-common" [Critical,Triaged]19:03
rlandyweshay_: better now?19:03
rlandythat the fix merged?19:03
weshay_no19:11
rlandyneed help?19:11
weshay_no.. :)19:11
weshay_thank oyu19:11
zbrrlandy: weshay_ : please wf https://review.opendev.org/#/q/topic:ensure+(status:open+OR+status:merged)19:15
*** saneax has quit IRC19:15
rlandylooks ok to me19:17
rlandyweshay_: any objection to calling these roles directly  https://opendev.org/x/tripleo-ipa/src/branch/master/tripleo_ipa/roles - rather than creating our own per https://review.opendev.org/#/c/706373/19:21
rlandythis linter commenting is so neat ... https://review.opendev.org/#/c/706373/31/roles/ipa-multinode/defaults/main.yml19:21
weshay_sec in a call19:23
*** derekh has quit IRC19:39
weshay_rlandy, back20:09
weshay_rlandy, FYI.. I just got access to zuul to trigger pipelines20:09
rlandyweshay_: for which zuul?20:10
weshay_so rlandy instead of test project jobs.. let me know if / when you need that20:10
*** jfrancoa has quit IRC20:10
rlandynot at the moment but could be interesting for rerunning components20:10
weshay_rlandy, if their roles work now20:10
weshay_that's fine20:10
weshay_aye.. I just triggered openstack-component-common20:11
rlandyit seems that there have been changes in the IPA work ...20:11
rlandyhttps://opendev.org/x/tripleo-ipa/src/branch/master/tripleo_ipa/roles20:11
weshay_ya.. ade has been very busy20:11
weshay_ya.. we approved that role about 1.5 months ago20:11
rlandyI'm trying to set up enough so that pooja can continues20:11
weshay_perfecto20:11
rlandyquestion is ...20:12
rlandywhat to do about the roles they have20:12
rlandyhttps://opendev.org/x/tripleo-ipa/src/branch/master/tripleo_ipa/roles20:12
rlandywe have out own version of those roles in https://review.opendev.org/#/c/706373/20:12
weshay_rlandy, seems like a good reason to setup a 1/2 sync w/ ade and talk about tea20:12
rlandytea?20:13
*** derekh has joined #oooq20:13
rlandyas in rooibos20:13
weshay_:)20:13
weshay_ya.. 1/2 next week at this point20:13
weshay_ur out thrs, friday20:13
rlandyand most of tomorrow20:13
rlandyI will meet with pooja though20:13
*** derekh has quit IRC20:14
rlandyso probably tuesday morning20:14
weshay_sounds perfect20:14
rlandyweshay_: did we get an info on the container reg when we are ready to push?20:15
weshay_no not yet.. but I expect another 2-3 weeks of work in that job20:16
rlandyk20:17
*** TrevorV has quit IRC20:55
*** rlandy is now known as rlandy|biab21:10
*** holser has quit IRC21:47
*** rlandy|biab is now known as rlandy21:54
*** jmasud has quit IRC22:30
*** jmasud has joined #oooq22:32
*** saneax has joined #oooq22:56
*** tosky has quit IRC23:26

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