Tuesday, 2019-09-24

openstackgerritMerged openstack/tripleo-heat-templates master: Fix selinux context for glance-api  https://review.opendev.org/68276800:18
*** morazi has quit IRC00:37
*** morazi has joined #tripleo00:41
*** morazi has quit IRC00:52
EmilienMcloudnull: can we fix it in another ? (on top) https://review.rdoproject.org/r/#/c/2244500:55
*** morazi has joined #tripleo00:58
openstackgerritMerged openstack/paunch master: Fix discovering container names  https://review.opendev.org/67857701:00
*** morazi has quit IRC01:05
*** morazi has joined #tripleo01:07
*** gbarros has joined #tripleo01:09
cloudnullEmilienM +101:14
*** mschuppert has quit IRC01:15
* cloudnull doesn't have +2 there 01:15
openstackgerritMerged openstack/tripleo-heat-templates master: Ensure consistent mirroring across config-data  https://review.opendev.org/67458001:17
openstackgerritKevin Carter (cloudnull) proposed openstack/tripleo-heat-templates master: Move nested bash script into a real file  https://review.opendev.org/68327001:23
openstackgerritKevin Carter (cloudnull) proposed openstack/tripleo-heat-templates master: Move basic global functions into ansible tasks  https://review.opendev.org/68328601:24
openstackgerritKevin Carter (cloudnull) proposed openstack/tripleo-heat-templates master: Convert subprocess to a reusable method  https://review.opendev.org/68342901:24
openstackgerritKevin Carter (cloudnull) proposed openstack/tripleo-heat-templates master: Move globals into main level 0 case  https://review.opendev.org/68359001:25
openstackgerritKevin Carter (cloudnull) proposed openstack/tripleo-heat-templates master: Move config processing into consolodated methods  https://review.opendev.org/68367001:25
*** morazi has quit IRC01:45
*** morazi has joined #tripleo01:45
*** pierrepr1netti has joined #tripleo01:52
*** pierreprinetti has quit IRC01:56
openstackgerritpengyuesheng proposed openstack/tripleo-common master: Update the constraints url  https://review.opendev.org/68416002:04
openstackgerritpengyuesheng proposed openstack/tripleo-validations master: Update the constraints url  https://review.opendev.org/68416102:06
*** saneax has joined #tripleo02:08
*** weshay|rover has quit IRC02:20
openstackgerritMerged openstack/tripleo-common master: Add profile_tasks to ansible runs  https://review.opendev.org/68342502:27
*** rfolco has quit IRC02:27
*** dsneddon has quit IRC02:29
*** rlandy has quit IRC02:33
*** saneax has quit IRC02:40
*** ricolin has joined #tripleo02:48
*** rohit02 has joined #tripleo03:24
*** ykarel has joined #tripleo03:32
*** poopcat has quit IRC03:32
*** poopcat has joined #tripleo03:34
openstackgerritEmilien Macchi proposed openstack/paunch stable/stein: Revert "Add --cpuset-cpus support for both Docker and Podman"  https://review.opendev.org/68418003:39
openstackgerritEmilien Macchi proposed openstack/paunch stable/stein: Revert "Add --cpuset-cpus support for both Docker and Podman"  https://review.opendev.org/68418003:39
openstackgerritEmilien Macchi proposed openstack/paunch master: Add --cpuset-cpus support for both Docker and Podman  https://review.opendev.org/68125703:48
openstackgerritEmilien Macchi proposed openstack/paunch stable/stein: Improve volume validation  https://review.opendev.org/68338603:49
*** artom has quit IRC03:56
openstackgerrityatin proposed openstack/tripleo-quickstart master: [DNM] just test effect of container update  https://review.opendev.org/68418703:56
openstackgerritEmilien Macchi proposed openstack/paunch stable/stein: Add --cpuset-cpus support for both Docker and Podman  https://review.opendev.org/68418904:01
EmilienMmwhahaha: ^ sorry for the paunch mess, should be fixed (i reverted the backport, backported your volume patch and on top, cherry-pick of the cpu thing)04:01
*** skramaja has joined #tripleo04:05
*** jamesdenton has quit IRC04:07
openstackgerritMerged openstack/tripleo-upgrade master: Fix "unbound" variable  https://review.opendev.org/68401004:10
*** morazi has quit IRC04:17
*** morazi has joined #tripleo04:19
openstackgerritMerged openstack/tripleo-ansible master: paunch: disable check mode  https://review.opendev.org/68380204:21
*** ykarel has quit IRC04:24
*** gbarros has quit IRC04:29
*** jamesden_ has joined #tripleo04:43
*** ratailor has joined #tripleo04:43
*** ykarel has joined #tripleo04:55
*** abishop has joined #tripleo04:56
openstackgerritChandan Kumar (raukadah) proposed openstack/tripleo-quickstart master: [CentOS-7]Added release file for train  https://review.opendev.org/68393204:56
*** holser has joined #tripleo04:57
openstackgerritSergii Golovatiuk proposed openstack/tripleo-upgrade stable/stein: Fix "unbound" variable  https://review.opendev.org/68419104:57
*** ratailor has quit IRC05:00
*** udesale has joined #tripleo05:00
*** bnemec has quit IRC05:09
*** abishop has quit IRC05:10
*** brault has quit IRC05:12
*** brault has joined #tripleo05:15
*** lmiccini has joined #tripleo05:15
*** bnemec has joined #tripleo05:15
*** morazi has quit IRC05:25
*** ratailor has joined #tripleo05:26
*** jtomasek has joined #tripleo05:31
*** morazi has joined #tripleo05:31
*** dsneddon has joined #tripleo05:41
openstackgerritCédric Jeanneret (Tengu) proposed openstack/tripleo-validations master: New validation: detect failed containers  https://review.opendev.org/65678505:41
*** dsneddon has quit IRC05:46
*** holser has quit IRC05:53
*** yprokule has joined #tripleo05:58
*** marios has joined #tripleo05:59
*** jfrancoa has joined #tripleo06:00
*** jfrancoa has quit IRC06:04
*** kopecmartin|off is now known as kopecmartin06:05
*** surpatil has joined #tripleo06:05
*** slaweq has joined #tripleo06:09
*** saneax has joined #tripleo06:10
*** brault has quit IRC06:17
*** mschuppert has joined #tripleo06:23
*** morazi has quit IRC06:24
*** yolanda has quit IRC06:34
*** abishop has joined #tripleo06:42
*** rpittau|afk is now known as rpittau06:43
*** morazi has joined #tripleo06:44
*** tosky has joined #tripleo06:53
*** pcaruana has joined #tripleo06:56
*** gfidente has joined #tripleo06:59
*** hberaud|gone is now known as hberaud07:05
openstackgerrityatin proposed openstack/tripleo-quickstart master: [DNM] just test effect of container update  https://review.opendev.org/68418707:05
*** shyam89 has joined #tripleo07:07
*** tesseract has joined #tripleo07:07
*** paramite has joined #tripleo07:09
*** soniya29 has joined #tripleo07:12
*** xek has joined #tripleo07:16
*** hjensas has quit IRC07:23
*** chem has joined #tripleo07:24
Tenguhmmm... gate is terribly slow right?07:31
*** bogdando has joined #tripleo07:31
*** brault has joined #tripleo07:38
*** holser has joined #tripleo07:40
*** avivgta has joined #tripleo07:40
*** brault has quit IRC07:43
ksamborTengu: yeah and timeouts appears sometimes07:43
*** amoralej|off is now known as amoralej07:44
Tenguksambor: apparently, yes. apparently due to the container-prepare thingy.07:44
Tengusince the in-flight were deactivated last week.07:44
*** shyam89 has quit IRC07:45
*** shyam89 has joined #tripleo07:45
*** gfidente has quit IRC07:46
*** iurygregory has joined #tripleo07:47
*** shyam89 has quit IRC07:51
*** jfrancoa has joined #tripleo07:53
openstackgerritpengyuesheng proposed openstack/diskimage-builder master: Update the constraints url  https://review.opendev.org/68421607:53
openstackgerritpengyuesheng proposed openstack/tripleo-quickstart master: Update the constraints url  https://review.opendev.org/68422107:56
*** psachin has joined #tripleo08:00
*** florianf has joined #tripleo08:00
*** jpich has joined #tripleo08:01
openstackgerritpengyuesheng proposed openstack/ansible-role-container-registry master: Update the constraints url  https://review.opendev.org/68422608:02
*** alexmcleod has joined #tripleo08:05
*** cylopez has joined #tripleo08:07
*** ykarel is now known as ykarel|lunch08:08
*** hjensas has joined #tripleo08:09
*** apetrich has quit IRC08:12
*** tkajinam has quit IRC08:14
*** apetrich has joined #tripleo08:22
*** derekh has joined #tripleo08:28
*** jpena|off is now known as jpena08:30
*** hamdyk has joined #tripleo08:33
*** jbadiapa has joined #tripleo08:35
*** jbadiapa has quit IRC08:37
*** jbadiapa has joined #tripleo08:38
*** shyam89 has joined #tripleo08:41
*** jbadiapa has quit IRC08:42
*** holser has quit IRC08:43
*** holser has joined #tripleo08:44
*** lmiccini has quit IRC08:44
*** lyarwood has quit IRC08:45
*** mburned has quit IRC08:45
*** ykarel|lunch is now known as ykarel08:46
*** lyarwood has joined #tripleo08:46
openstackgerritBogdan Dobrelya (bogdando) proposed openstack/tripleo-ci master: Fallback to public IPv4 when there is no priv. IP  https://review.opendev.org/68209208:48
*** ccamacho has joined #tripleo09:01
*** rcernin has quit IRC09:04
*** brault has joined #tripleo09:05
*** ricolin_ has joined #tripleo09:05
*** ricolin has quit IRC09:08
*** brault has quit IRC09:09
openstackgerritChandan Kumar (raukadah) proposed openstack/tripleo-ci master: Added support for train for emit release file  https://review.opendev.org/68426009:17
*** florianf has quit IRC09:17
*** ricolin_ is now known as ricolin09:23
*** ratailor_ has joined #tripleo09:26
openstackgerritmathieu bultel proposed openstack/tripleo-common master: Implement Ansible fact cache for Mistral executor  https://review.opendev.org/68285509:26
*** shyam89 has quit IRC09:26
Tengumarios: heya! quick question: would it be possible to get some info about the swift storage dedicated for the logs? I suspect it might be the cause of the current timeouts we see in Zuul....09:27
*** ratailor has quit IRC09:28
Tengumarios: at least I don't see any timeout in this log: https://zuul.opendev.org/t/openstack/build/8b79b9ca3b9d4ff58c931d0bac313289/log/job-output.txt - but it has failed :/09:28
Tenguoh. sh*t... it's "timed_out", not "time out" nor "timeout"09:30
openstackgerritMartin Schuppert proposed openstack/tripleo-heat-templates stable/queens: [Queens] Run cellv2 host discovery via workflow or deploy_steps_tasks  https://review.opendev.org/68306909:30
mariosTengu: looking in the logs for the timeout now but not sure what you mean about swift storage for logs cos not sure we have much say about that sshnaidm do we have control on log server?09:31
Tengumarios: thought it might be linked since it was the last line there, but apparently the timeout happens during the tempest run -.-'.09:31
TenguI just didn't think to search for "timed_out" string in there. pfff.09:32
marios2019-09-23 19:31:27.524944 | primary | sort: write failed: standard output: Broken pipe on collect logs but not sure that it09:32
mariosat least the time jumps from 04 to 31 there09:32
mariosso maybe collect logs failed or times out if you already found the timeout09:32
Tengumarios: yeah, so that's apparently not main failure cause.09:32
Tenguhttps://zuul.opendev.org/t/openstack/build/8b79b9ca3b9d4ff58c931d0bac313289/log/job-output.txt#723609:32
*** saneax is now known as saneax|AFK09:34
*** saneax|AFK has quit IRC09:40
*** shyam89 has joined #tripleo09:40
*** brault has joined #tripleo09:42
*** brault has quit IRC09:42
*** brault has joined #tripleo09:42
*** hberaud is now known as hberaud|lunch09:51
openstackgerritSaravanan KR proposed openstack/tripleo-ansible master: Add kernelargs and tuned core configuration  https://review.opendev.org/67980909:51
owalshTengu: hey, I really don't understand the comment on https://review.opendev.org/681442... how can fixing a huge race condition be incompatible with the specs?09:54
openstackgerritHamdy Khader proposed openstack/tripleo-heat-templates master: Add heat templates to deploy mellanox neutron agent  https://review.opendev.org/67448609:55
openstackgerritHamdy Khader proposed openstack/tripleo-heat-templates master: Add support for Neutron Multi interface driver  https://review.opendev.org/67955809:55
*** florianf has joined #tripleo10:01
*** lmiccini has joined #tripleo10:01
owalshTengu: e.g https://review.opendev.org/681525 should have failed CI as the RPC healthchecks were not completely fixed. With the fix for the race we catch it https://review.opendev.org/68160510:02
openstackgerritSagi Shnaidman proposed openstack/tripleo-ci master: DNM: run ara undercloud  https://review.opendev.org/68426710:08
*** dsneddon has joined #tripleo10:09
*** brault has quit IRC10:11
*** cylopez has quit IRC10:11
*** openstackstatus has quit IRC10:12
*** openstack has joined #tripleo10:14
*** ChanServ sets mode: +o openstack10:14
*** openstackstatus has joined #tripleo10:14
*** ChanServ sets mode: +v openstackstatus10:14
*** morazi has quit IRC10:17
*** dtantsur|afk is now known as dtantsur10:17
bandiniis 'Exception: Not found image: docker://trunk.registry.rdoproject.org/tripleomaster/rhel-binary-cron:fd46c80632bbd7abc8411277bfc4819f6b95a485_bf6d0acd' known as an issue? (http://logs.rdoproject.org/47/669847/24/openstack-check/tripleo-ci-rhel-8-scenario001-standalone-rdo/07c8836/logs/undercloud/home/zuul/standalone_deploy.log.txt.gz)10:20
*** shyam89 has quit IRC10:23
*** pcaruana has quit IRC10:24
owalshmschuppert: hey, re https://review.opendev.org/683069 ...10:27
mschuppertowalsh: o/10:27
openstackgerritKamil Sambor proposed openstack/tripleo-heat-templates master: Support connecting OVN DB over SSL  https://review.opendev.org/68034510:28
owalshmschuppert: IIUC the workflow would be run twice when we have nova_compute + nova_ironic10:29
mschuppertowalsh: yes thats a scenario I am going to test right now and to see how to deal with. thats why I had it initially delecated to the nova_api10:30
owalshmschuppert: ack, I guess running twice is ok10:30
owalshmschuppert: but probably need to drop the params and hardcode the service/container10:31
amoralejhi, https://bugs.launchpad.net/tripleo/+bug/1845181 is blocking eventlet promotion in rdo, which is required for other projects10:31
openstackLaunchpad bug 1845181 in tripleo "Problem with tripleo_common git functionalities and eventlet 0.25.1" [Undecided,New]10:31
amoralejEmilienM, ^ fyi10:31
amoralejapetrich, i see you worked in the initial fix, ^ you may have a good solution for that10:31
mschuppertowalsh: yes, I added them only from the manager thing you work on. but as we said we won't support that on queens so I could hard code it10:32
openstackgerritSergii Golovatiuk proposed openstack/tripleo-heat-templates master: Replace include_role with import_role  https://review.opendev.org/68267210:32
owalshmschuppert: will need to support it in queens I expect10:32
mschuppertoh10:32
owalshhrmm, maybe default to hardcoded but use the params if they are set?10:33
*** shyam89 has joined #tripleo10:33
owalshwhich could be a follow-up patch when the manager thing is being backported10:34
mschuppertowalsh: ack10:34
mschuppertmakes sense10:34
apetrichamoralej, sorry, I was about to go to lunch. Is that all right if I look as soon as I'm back10:35
apetrich?10:35
amoralejsure10:35
amoralejno problem10:35
owalshmschuppert: the other thing I was wondering... when is workflow_tasks step5 run? Wondering if we'll have the same problem I've got with https://review.opendev.org/681042 (I need a step6!)10:36
owalshhttps://review.opendev.org/#/c/681042/12/deployment/nova/nova-compute-container-puppet.yaml@94610:36
mschuppertowalsh: it runs before deploy task step5, see  http://pastebin.test.redhat.com/80005810:39
owalshmschuppert: damn, nova_ironic start the container in step510:41
mschuppertowalsh: where? not in queens. in mater?10:43
mschuppertmaster10:43
owalshmschuppert: queens IIRC10:43
owalshmschuppert: https://github.com/openstack/tripleo-heat-templates/blob/stable/queens/docker/services/nova-ironic.yaml#L16310:44
owalshmschuppert: ah, no, step4 https://github.com/openstack/tripleo-heat-templates/blob/stable/queens/docker/services/nova-ironic.yaml#L11810:44
mschuppertyes, thats only discovery10:44
owalshmschuppert: but I need to move it to step5 in https://review.opendev.org/68104210:45
mschuppertowalsh: then we need a step 6 there as well10:45
*** pcaruana has joined #tripleo10:48
*** shyam89 has quit IRC10:48
mschuppertowalsh: or we add a wait_for_nova_api before we start the nova_compute in step410:50
*** weshay has joined #tripleo10:50
owalshmschuppert: but wait for how long?10:50
owalshpulling the image doesn't really have an upper bound (well, somthing else will timeout but....)10:51
mschuppertowalsh: yeah good point10:51
owalshmschuppert: actually adding a step6 for workflow tasks may not be that bad - https://github.com/openstack/tripleo-heat-templates/blob/stable/queens/common/deploy-steps.j2#L8710:53
owalshand we could call the 6th step 'post' ;-)10:55
mschuppertowalsh: yes :)10:55
*** ratailor_ has quit IRC10:58
*** iurygregory_ has joined #tripleo11:03
*** iurygregory has quit IRC11:05
openstackgerritHarald Jensås proposed openstack/tripleo-ansible master: Support 'ipversion' property in rules  https://review.opendev.org/68427711:06
openstackgerritHarald Jensås proposed openstack/tripleo-ansible master: Open ipv6-icmp traffic by default  https://review.opendev.org/68427811:06
apetrichamoralej, back. Looking11:06
openstackgerritwes hayutin proposed openstack/tripleo-heat-templates master: Add no_log to include_vars  https://review.opendev.org/68298811:08
*** udesale has quit IRC11:08
*** udesale has joined #tripleo11:09
Tenguis there a reason for tripleo-ci-centos-7-containers-multinode to often have timeouts? Apparently that's the only job hitting this issue now :(11:10
Tengujust trying to understand11:10
*** hberaud|lunch is now known as hberaud11:16
*** artom has joined #tripleo11:17
*** avivgta has quit IRC11:17
*** lucasagomes has joined #tripleo11:21
openstackgerritAdriano Petrich proposed openstack/tripleo-common master: Limit the eventlet version to lower than 0.25 as it fails tests  https://review.opendev.org/68428211:22
apetrichamoralej, It is the version 0.25 of the eventlet11:23
apetrichamoralej, wait is version 0.25 required for other projects11:24
amoralejapetrich, yes, we need to bump to 0.25.111:24
apetrichamoralej, that's an issue.11:24
apetrichok I will abandon my patch and rething that11:24
amoralejit's in upper-constraints so it's the version that the rest of projects are using to test upstream11:25
amoralejand in this case swift is actively requesting ti11:25
amoralejit11:25
amoralejhttps://review.rdoproject.org/r/#/c/22509/11:25
apetrichamoralej, ok. That is going to take a bit more time. I'm looking into it11:26
amoralejsure, no problem, thanks11:26
*** shyam89 has joined #tripleo11:27
*** ansmith has quit IRC11:31
*** amoralej is now known as amoralej|lunch11:32
weshayzbr|ruck, arxcruz|rover ok.. if you narrow the time in the job explorer.. you get a decent snapshot of container build times11:32
weshayhttp://dashboard-ci.tripleo.org/d/si1tipHZk/jobs-exploration?orgId=1&from=now-12h&to=now&var-influxdb_filter=branch%7C%3D%7Cmaster11:32
weshayEmilienM, fyi ^11:32
weshaysorry.. container-prep times on the undercloud/standalone11:32
weshaynote.. rdo periodic jobs hitting the rdo registry takes 6.5 minutes11:32
*** marios has quit IRC11:33
weshaytripleo-ci-centos-7-containers-multinode has a 50% pass rate :(11:34
*** jpena is now known as jpena|lunch11:34
openstackgerritmathieu bultel proposed openstack/tripleo-heat-templates master: Use update_serial as an ansible variable  https://review.opendev.org/68428711:38
zbr|ruckweshay: can we have a sync session? i could use some help. bj?11:38
Tenguweshay: why only this one for the container-prepare thingy? is it due to multi-node, or is there some other specifics?11:40
weshayhttps://trello.com/c/KkBkHEOD/1118-cixlp1844446tripleociproa-multiple-tripleo-jobs-timing-out-upstream-causing-gate-resets-in-train11:41
weshayTengu, ^11:41
openstackgerritDmitry Tantsur proposed openstack/tripleo-common master: Correct deprecated usage of ironicclient (broken with 3.0.0)  https://review.opendev.org/68428811:41
dtantsurarxcruz|rover: this ^^ may be it11:41
dtantsurI'm not sure how to test it with 3.0.0 though11:42
* dtantsur gets food now11:42
weshayTengu, zbr|ruck look at https://snapshot.raintank.io/dashboard/snapshot/SjZ16RtMXZCFbEIOjgZ7N7KKJvOpKBhB?orgId=211:42
weshaythat is data from yesterday11:42
Tenguweshay: yes, apparently only the multinode has timed_out right? why this one in particular?11:43
*** noama has joined #tripleo11:46
*** morazi has joined #tripleo11:49
*** rh-jelabarre has joined #tripleo11:54
*** bfournie has quit IRC11:54
*** boazel has joined #tripleo11:54
*** suuuper has joined #tripleo11:57
EmilienMamoralej: Ack11:58
*** marios has joined #tripleo11:58
*** jpich has quit IRC11:58
*** stevebaker has quit IRC11:59
*** derekh has quit IRC12:00
*** stevebaker has joined #tripleo12:00
apetrichEmilienM, amoralej|lunch that is a bit tricky. there's some bad interactions between gitpython and eventlet 0.25. I'm working on it12:05
*** cylopez has joined #tripleo12:05
EmilienMapetrich: thanks12:07
EmilienMweshay: looking12:07
*** ekultails has joined #tripleo12:07
*** rfolco has joined #tripleo12:10
openstackgerritChandan Kumar (raukadah) proposed openstack/tripleo-quickstart-extras master: Added train release tempest skip list  https://review.opendev.org/68429512:16
weshayEmilienM, on the agenda for tripleo mtg.. <weshay> thoughts on a daily periodic job to update master containers w/ update_repo: "delorean-current,quickstart-centos-ceph-nautilus" ?12:17
*** avivgta has joined #tripleo12:20
EmilienMweshay: yes12:25
apetrichamoralej|lunch, bbiab. have to pick up the kid at the nursery.12:25
EmilienMto reduce what is updated in each job?12:25
weshayEmilienM, aye.12:25
weshayEmilienM, btw.. the periodic rdo jobs.. pull containers in 5-6 min12:26
*** aakarsh has joined #tripleo12:27
EmilienMweshay: but did you apply https://review.rdoproject.org/r/#/c/22445/ ?12:29
weshayEmilienM, waiting to chat to rlandy about that12:29
weshayand the other folks working on the promoter atm.. I just pinged them all in #oooq12:30
weshayabout it12:30
*** jamesden_ has quit IRC12:34
*** jamesden_ has joined #tripleo12:34
*** jpena|lunch is now known as jpena12:35
*** shyam89 has quit IRC12:37
*** brault has joined #tripleo12:37
*** ade_lee_ has quit IRC12:38
*** rlandy has joined #tripleo12:39
*** brault has quit IRC12:40
*** brault has joined #tripleo12:40
*** ansmith has joined #tripleo12:42
*** jamesden_ is now known as jamesdenton12:44
*** bfournie has joined #tripleo12:46
*** amoralej|lunch is now known as amoralej12:46
*** gfidente has joined #tripleo12:47
*** psachin has quit IRC12:49
*** brault has quit IRC12:49
*** brault has joined #tripleo12:50
openstackgerritGael Chamoulaud proposed openstack/tripleo-validations master: Add new healthcheck-service-status validation  https://review.opendev.org/68237712:50
*** rohit02 has quit IRC12:52
*** aakarsh has quit IRC12:54
*** mcornea has joined #tripleo12:59
openstackgerritSorin Sbarnea proposed openstack/tripleo-common stable/stein: Add profile_tasks to ansible runs  https://review.opendev.org/68430512:59
*** jaosorior has quit IRC12:59
openstackgerritCarlos Goncalves proposed openstack/diskimage-builder master: WIP: CentOS 8 support  https://review.opendev.org/68430813:02
openstackgerritSorin Sbarnea proposed openstack/tripleo-common stable/rocky: Add profile_tasks to ansible runs  https://review.opendev.org/68430913:02
*** brault has quit IRC13:03
*** raildo has joined #tripleo13:04
openstackgerritSorin Sbarnea proposed openstack/tripleo-common stable/queens: Add profile_tasks to ansible runs  https://review.opendev.org/68431013:04
*** brault has joined #tripleo13:05
zbr|ruckweshay: EmilienM sshnaidm please help me backport the profiling ^13:06
*** jcoufal has joined #tripleo13:07
weshayzbr|ruck, can we get stein please?13:07
zbr|ruckweshay: stein is among them, there are 3 in total13:07
weshayah sorry13:08
* weshay looks13:08
sshnaidmzbr|ruck, ERROR Unable to find role in /var/opt/rh/rh-python35/lib/zuul/builds/bc245bb870f048ac8ee3a1b00ebd6199/ansible/pre_playbook_7/role_1/tripleo-ansible in https://review.opendev.org/#/c/684305/13:09
*** brault has quit IRC13:10
zbr|rucksshnaidm: what can I say.... unrelated :D13:10
*** iurygregory_ is now known as iurygregory13:11
*** saneax|AFK has joined #tripleo13:11
openstackgerritLuke Short proposed openstack/tripleo-heat-templates master: Convert firewall rules to use TripleO-Ansible  https://review.opendev.org/67723713:15
openstackgerritLuke Short proposed openstack/tripleo-heat-templates master: Use the Ansible role for tripleo-packages.  https://review.opendev.org/67862013:15
ykarelsshnaidm, zbr|ruck seems happening due to stable/stein branch in tripleo-ansible13:19
openstackgerritBogdan Dobrelya (bogdando) proposed openstack/ansible-role-tripleo-modify-image master: Add support for yum caching when buildah updates  https://review.opendev.org/68310013:19
ykarelwas that branch created intentionally?13:19
ykarelisn't tripleo-ansible going to be branchless?13:19
*** psachin has joined #tripleo13:20
cloudnullmornings13:20
EmilienMcloudnull: hey good morning, have you seen https://bugs.launchpad.net/bugs/1845170 and https://bugs.launchpad.net/bugs/1845175 yet?13:20
openstackLaunchpad bug 1845170 in tripleo "tripleo-ansible - uses 'proto' field to decide if rule is created in iptables or ip6tables" [High,In progress] - Assigned to Harald Jensås (harald-jensas)13:20
openstackLaunchpad bug 1845175 in tripleo "tripleo-ansible - firewall role does not open ipv6-icmp" [High,In progress] - Assigned to Harald Jensås (harald-jensas)13:20
sshnaidmykarel, good question, weshay cloudnull do we need stein in tripleo-ansible?13:21
EmilienMsshnaidm: no13:21
ykarelgood to clean up then https://github.com/openstack/tripleo-ansible/branches13:22
cloudnullEmilienM I have not seen the bugs, though there was chatter in the the firewall review about it - https://review.opendev.org/#/c/677237/13:23
*** ade_lee_ has joined #tripleo13:23
cloudnullhttps://review.opendev.org/#/c/684278/ https://review.opendev.org/#/c/684277/ - looks like reviews are up, now we just need the multinode job to not timeout :)13:25
Tenguweshay: heya! isn't your script for trello going nut?13:27
openstackgerritKarthik S proposed openstack/tripleo-heat-templates master: Fixed the path for neutron-base.yaml  https://review.opendev.org/68431813:27
weshayTengu, going nuts?13:27
weshayTengu, looks fine13:28
weshaywhy?13:28
Tenguweshay: yeah, duplicating cards in CIX board13:28
weshayTengu, lolz.. you have to update the bug first brotha13:28
weshay:)13:28
weshayTengu, stop cheating ;)13:28
weshayTengu, which bug are you referring to?13:28
Tenguhttps://trello.com/c/Ix9GT9O8/1126-cixbz1844856ospphase1osp16puppet-tripleopodmanjson-file-is-not-valid-arg-with-podman-14213:30
cloudnullykarel +1 feel free to remove stein branch. unless EmilienM or mwhahaha can think of how/where we're using it I don't think we need it.13:30
*** gbarros has joined #tripleo13:30
*** surpatil has quit IRC13:30
EmilienMI don't think we need it13:31
Tenguweshay: ah, there's an issue, LP id instead of BZ....13:31
Tenguweird.13:31
hjensasEmilienM: I'm working on [Bug 1845153] Re: puppet-tripleo - not possible to create IPv4 or IPv6 only rules13:31
openstackbug 1845153 in tripleo "puppet-tripleo - not possible to create IPv4 or IPv6 only rules" [High,Triaged] https://launchpad.net/bugs/1845153 - Assigned to Emilien Macchi (emilienm)13:31
EmilienMhjensas: ack, ok, I saw it unassigned so I took it :) I'll update launchpad13:32
hjensasEmilienM: specs, and an actual test run to go.13:32
EmilienMthanks for working on it !13:32
mwhahahaykarel, cloudnull: no it's not going to be branchless13:32
*** mcornea has quit IRC13:32
cloudnullhjensas - did you see https://review.opendev.org/#/c/684277/1/tripleo_ansible/roles/tripleo-firewall/tasks/tripleo_firewall_add.yml ?13:33
cloudnullotherwise looks good. thanks for working on it13:33
ykarelmwhahaha, you mean stable/stein is used or will be used?13:33
cloudnullwithout the default its resulting in https://427ec8d7b2c1b1e2ac8c-16c430a2363acd21a3fbf84e4ca76ae5.ssl.cf1.rackcdn.com/684277/1/check/tripleo-ansible-centos-7-molecule-tripleo-firewall/7f86851/reports.html13:33
weshayTengu, so we're good? anything to look at?13:33
hjensascloudnull: ok, thanks. I'll do an update later today.13:34
cloudnull++13:34
mwhahahaykarel: i don't think we referenceds tripleo-ansible in stein, that would be the question13:34
EmilienMright13:35
ykarelmwhahaha, it's not referenced, but wherever zuul tries to clone in branched projects, it will checkout stable/stein if it exists13:35
mwhahahaykarel: i'm missing the original context, i'm just commenting ont he branchlessness13:35
ykarelmwhahaha, see comment from RDO zuul:- https://review.opendev.org/#/c/684305/13:35
ykareland i think that error is happening due to existance of stable/stein in tripleo-ansible which has missing roles13:36
mwhahahaykarel: are we referencing tirpleo-ansible in zuul?13:36
mwhahahathat seems wrong13:36
ykarelmwhahaha, yes for container login part13:37
ykarelhttps://github.com/rdo-infra/review.rdoproject.org-config/blob/master/zuul.d/tripleo-rdo-base.yaml#L6213:37
mwhahahacan you specify a branch there13:37
ykarelchandankumar, fyi ^^13:37
*** soniya29 has quit IRC13:38
weshayiirc we were asked to reuse login..13:38
mwhahahaalso isn't the container login in the othe rrole?13:38
mwhahahanot in tripleo-ansible13:38
weshaypanda, ^13:38
mwhahahait's fine to reuse it if you can specify a branch13:38
ykarelfor required projects i think we can specify, but not sure for role13:39
ykareljpena, any idea ^^13:39
pandaweshay: mwhahaha: we are using ansible-role-container-registry for docker and tripleo-ansible for podman13:39
* mwhahaha points to why putting all roles in a single repo is a bad idea13:40
openstackgerritmathieu bultel proposed openstack/tripleo-heat-templates master: Use update_serial as an ansible variable  https://review.opendev.org/68428713:44
jpenaykarel: reading at https://zuul-ci.org/docs/zuul/user/config.html#attr-job.roles, it looks like we can use job.required-projects.override-checkout to select which branch we want to use for the role, see https://zuul-ci.org/docs/zuul/user/config.html#attr-job.required-projects.override-checkout13:46
*** mcornea has joined #tripleo13:47
ykareljpena, hmm seems it should work13:48
ykarelzbr|ruck, sshnaidm ^^13:48
mwhahahawe use that for puppet-ceph13:49
sshnaidmykarel, yeah, we use it for ovb repo13:49
sshnaidmykarel, but what we'll do in train then?13:50
ykarelsshnaidm, but i don't see ovb in roles: https://codesearch.rdoproject.org/?q=openstack-virtual-baremetal&i=nope&files=&repos=13:50
sshnaidmykarel, it's in upstream, not in rdo13:51
ykarelseems that is used at quickstart13:51
openstackgerritFrancesco Pantano proposed openstack/tripleo-heat-templates master: Allow using registry authentication to pull ceph related containers  https://review.opendev.org/68201513:51
mwhahahahttps://opendev.org/openstack/puppet-openstack-integration/src/branch/stable/queens/.zuul.yaml#L2513:51
mwhahahait might be on a branch13:51
mwhahahawhich doesn't show up in codesearch13:51
mwhahaha(kinda annoying when you try to find this stuff)13:51
sshnaidmhttps://github.com/openstack/tripleo-ci/blob/master/zuul.d/base.yaml#L33-L4113:52
sshnaidmykarel, ^13:52
ykarelsshnaidm, found, but it's in required-projects, not in roles:13:52
sshnaidmykarel, right13:53
openstackgerritFrancesco Pantano proposed openstack/tripleo-heat-templates master: Allow using registry authentication to pull ceph related containers  https://review.opendev.org/68201513:53
ykarelhmm codesearch indexes only master branch :(13:53
EmilienMbandini, weshay: https://wiki.centos.org/Manuals/ReleaseNotes/CentOS8.1905#head-b7a63b88b2399f78f44d05be1ff8eb95bc679d5a13:54
weshayEmilienM, aye.. I think we're still missing qcow2 images13:54
EmilienMright13:54
openstackgerritKevin Carter (cloudnull) proposed openstack/tripleo-heat-templates master: Move nested bash script into a real file  https://review.opendev.org/68327013:55
bandiniEmilienM: nice, can't wait :)13:55
openstackgerritKevin Carter (cloudnull) proposed openstack/tripleo-heat-templates master: Move basic global functions into ansible tasks  https://review.opendev.org/68328613:55
openstackgerritCarlos Goncalves proposed openstack/diskimage-builder master: WIP: CentOS 8 support  https://review.opendev.org/68430813:55
openstackgerritKevin Carter (cloudnull) proposed openstack/tripleo-heat-templates master: Convert subprocess to a reusable method  https://review.opendev.org/68342913:55
openstackgerritKevin Carter (cloudnull) proposed openstack/tripleo-heat-templates master: Move globals into main level 0 case  https://review.opendev.org/68359013:55
openstackgerritHamdy Khader proposed openstack/tripleo-heat-templates master: Add heat templates to deploy mellanox neutron agent  https://review.opendev.org/67448613:56
openstackgerritHamdy Khader proposed openstack/tripleo-heat-templates master: Add support for Neutron Multi interface driver  https://review.opendev.org/67955813:56
openstackgerritEmilien Macchi proposed openstack/paunch stable/stein: Add --cpuset-cpus support for both Docker and Podman  https://review.opendev.org/68418913:56
Tenguweshay: sooo I don't know what to do - BZ is ON_DEV, LP is "in progress" as well.... but apparently the script associate the LP id to the BZ....13:58
zbr|ruckEmilienM: weshay i was told that official qcow2 images will arrive in few days.13:58
Tenguweshay: https://bugzilla.redhat.com/show_bug.cgi?id=1754416 vs https://bugs.launchpad.net/tripleo/+bug/1844856 - and card title is [CIX][BZ:1844856]  <- that's the LP.13:59
openstackbugzilla.redhat.com bug 1754416 in puppet-tripleo "ovn-metadata-agent unable start haproxy" [High,On_dev] - Assigned to cjeanner13:59
openstackLaunchpad bug 1844856 in tripleo "podman 1.5 - json-file logging specified but not supported. Choosing k8s-file logging instead" [High,In progress] - Assigned to Cédric Jeanneret (cjeanner)13:59
weshayoh... we have a bug # collision13:59
Tengu:D13:59
*** chem has quit IRC14:00
mwhahaha#startmeeting tripleo14:00
openstackMeeting started Tue Sep 24 14:00:12 2019 UTC and is due to finish in 60 minutes.  The chair is mwhahaha. Information about MeetBot at http://wiki.debian.org/MeetBot.14:00
mwhahaha#topic agenda14:00
mwhahaha* Review past action items14:00
mwhahaha* One off agenda items14:00
mwhahaha* Squad status14:00
mwhahaha* Bugs & Blueprints14:00
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.14:00
mwhahaha* Projects releases or stable backports14:00
mwhahaha* Specs14:00
mwhahaha* open discussion14:00
*** openstack changes topic to " (Meeting topic: tripleo)"14:00
mwhahahaAnyone can use the #link, #action and #info commands, not just the moderatorǃ14:00
openstackThe meeting name has been set to 'tripleo'14:00
mwhahahaHi everyone! who is around today?14:00
*** openstack changes topic to "agenda (Meeting topic: tripleo)"14:00
dpeacocko/14:00
weshay0/14:00
ksamboro/14:00
Tengu\o/14:00
ekultailso/14:00
rlandyo/14:01
owalsho/14:01
marioshello14:01
*** Vorrtex has joined #tripleo14:01
chandankumarykarel: which branch? master ?14:01
chandankumar\o/14:02
*** dsneddon has joined #tripleo14:02
rfolcoo/14:02
mschupperto/14:02
ykarelchandankumar, yes14:03
ykarelo/14:03
chandankumarpatch coming14:03
mwhahahaalright let's get on with the meeting14:04
mwhahaha#topic review past action items14:04
mwhahahamwhahaha to close out train-m3 - DONE14:04
*** openstack changes topic to "review past action items (Meeting topic: tripleo)"14:04
mwhahahaI went ahead and closed out the launchpad milestone and we release m3 last week14:04
mwhahahaany questions on that?14:04
rh-jelabarreo/14:05
mwhahahasounds like nope14:05
mwhahahasync with weshay on ptl things - NOT DONE14:06
mwhahaha#action mwhahaha sync with weshay on ptl things14:06
mwhahahaweshay: do you have time later today so we can get just mark this one done14:06
weshayslackers14:06
weshayyes14:06
mwhahahak i'll ping you a bit later14:06
mwhahahathat's it on the past items14:06
mwhahahamoving on to more exciting things14:06
mwhahaha#topic one off agenda items14:06
mwhahaha#link https://etherpad.openstack.org/p/tripleo-meeting-item14:06
*** openstack changes topic to "one off agenda items (Meeting topic: tripleo)"14:06
mwhahahacentos8/pcmk ; clues when it will be available? should ci switch back to pcmk when it becomes available? (gfidente/bandini)14:06
mwhahahaIMHO the answer to pcmk would be yes14:07
mwhahahaweshay, do we have any thoughts on the turn around time for centos8, is that a question for rdo as well?14:07
*** dsneddon has quit IRC14:07
weshayonce we have a upstream nodepool image.. I'm hoping we'll see standalone running with in days.. and i'll provide an estimate then14:08
mwhahahak14:08
ykarelRDO would need time to prepare14:08
bandinimwhahaha: yes, once we can use centos8 we should switch back ci to pcmk14:09
bandiniand also work on ha by default14:09
mwhahahawe'll also finally be able to switch to podman on ovb as well14:09
*** bdodd has joined #tripleo14:09
mwhahahaand remove docker packaging14:09
mwhahahaso there's a bunch pending centos814:09
mwhahahahopefully we can switch early in the U cycle14:09
mjturekrfolco: does this look good? ppc container builds are failing due to kolla issue and this is the fix I think https://review.opendev.org/#/c/683997/14:09
weshaymjturek, in a mtg fyi14:10
rfolcomjturek, lets chat on #oooq14:10
mjturekwhoops, I'm  sorry14:10
ykarelyes hopefully for U cycle we will be able to go with CentOS14:10
bandiniaye14:10
ykarelbut more update will be there in coming days14:10
ykarelonce it's actually out14:10
mwhahahaany other pacemaker/centos8 thoughts?14:10
gfidenteweshay mwhahaha yeah and both scenario004 and scenario00114:10
*** lennyb has quit IRC14:11
cloudnullo/14:11
ykarelalso we would need other SIGs to be ready apart from cloud sig14:11
ykarellike virt and ceph to be ready with centos814:11
*** dsneddon has joined #tripleo14:12
mwhahaha#link http://mirror.centos.org/centos-8/14:12
mwhahahalooks like we're still pending qcow cloud images, but at least a repo is available14:12
ykarelregarding RDO and CentOS8, couple of info can be found at https://review.rdoproject.org/etherpad/p/moving-rdo-to-centos814:13
mwhahahak14:14
mwhahahaI think that's about it for now. centos8 is still a WIP14:14
mwhahahamoving on14:14
mwhahaha<weshay> quay registry14:14
weshayya...14:14
weshayjust a quick FYI.. we're exploring using quay ( key ) lolz.. in upstream.  patches are being tested now14:14
weshayexpect containers to live in both docker.io and quay for some time14:15
weshayhttps://review.rdoproject.org/r/#/c/22445/ <- being tested14:15
weshaythat's it14:15
weshaymwhahaha, ^14:16
*** skramaja has quit IRC14:16
mwhahahak14:16
mwhahahamoving on14:16
weshaynext item is mine as well14:16
mwhahaha<weshay> thoughts on a daily periodic job to update master containers w/ update_repo: "delorean-current,quickstart-centos-ceph-nautilus" ?14:16
mwhahahaw/ standalone, or other job.. if it passes update containers in docker.io or quay14:16
*** chem has joined #tripleo14:16
*** dsneddon has quit IRC14:17
weshayya.. I have arxcruz|rover and zbr|ruck looking at building a periodic (daily) job to update containers w/14:17
weshay "delorean-current,quickstart-centos-ceph-nautilus"14:17
sshnaidmit shouldn't even pass, just build and push14:17
mwhahahathat doesnt' seem correct14:17
*** dsneddon has joined #tripleo14:17
weshayand push to docker.io / quay.. to shorten the loop w/ containers prep14:17
weshaysshnaidm, k.. we'll consider that14:17
mwhahahashouldn't it match our includepkgs list (tripleo-repos current-tripleo-dev)14:17
sshnaidmanyway we update to these repos containers in every job, and it's merged only if it's passed14:18
weshayagain.. we're seeing very long container prep times in CI14:18
weshaycausing failures14:18
mwhahahawhich is the equivalent of CI14:18
mwhahahawhen was the last promotion?14:18
weshaysome examples can be found here:  https://snapshot.raintank.io/dashboard/snapshot/SjZ16RtMXZCFbEIOjgZ7N7KKJvOpKBhB?orgId=214:18
weshaymaster is at 4 days14:18
bogdandoweshay: yes, it's about ~1h++ for each job14:18
*** gbarros has quit IRC14:18
mwhahahahmm k14:18
weshayyou can see.. we have infra issues w/ ovh and fortnebula14:18
bogdandoonly to update the containers with modify image role14:18
weshaymost of the infra guys are out atm.. at ansible-fest14:19
weshayin rdo periodic jobs w/o yum updates.. containers prep is taking 5-6 minutes14:19
ykarelreally?14:19
*** michchap has quit IRC14:19
weshayso we're still exploring ways to get our job times back down to a reasonable time... updates will come14:19
weshayykarel,  aye14:20
mwhahahak14:20
ykarelokk nice, any link to logs14:20
weshayhttp://dashboard-ci.tripleo.org/d/si1tipHZk/jobs-exploration?orgId=1&from=now-12h&to=now&var-influxdb_filter=branch%7C%3D%7Cmaster14:20
weshayykarel,  look for the rdo provider14:20
weshayhrm.. let me get you a better link14:21
sshnaidmcontainer build takes about 15 minutes? maybe worth just to build them and not to update? :)14:21
ykareli see 25,21, 15 minutes14:21
*** hberaud is now known as hberaud|school-r14:21
mwhahahayou mean building them in CI?14:21
sshnaidmyeah14:21
mwhahahai think it'll probably take longer than that on some of these clouds14:22
mwhahahathough maybe we would get better cache results for packages than containers14:22
*** dsneddon has quit IRC14:22
mwhahahawe could try it14:22
weshayykarel, use https://snapshot.raintank.io/dashboard/snapshot/S9m9pvyn4iPSELe0bd566ZlMv4KvVWwO?orgId=214:22
mwhahahabut it requires an undercloud to build14:22
mwhahahabecause $reasons14:22
mwhahahai'm not sure it'd be a simple switch14:22
*** lennyb has joined #tripleo14:23
weshayhttp://logs.rdoproject.org/openstack-periodic-master/opendev.org/openstack/tripleo-ci/master/periodic-tripleo-ci-rhel-8-scenario001-standalone-master/7b77387/logs/undercloud/home/zuul/standalone_deploy.log.txt.gz14:23
bogdandoykarel: just checked, http://logs.rdoproject.org/openstack-periodic-master/opendev.org/openstack/tripleo-ci/master/periodic-tripleo-ci-rhel-8-scenario001-standalone-master/7b77387/logs/undercloud/var/log/tripleo-container-image-prepare.log.txt.gz is exactly 5 minutes14:24
ykarelrhel814:24
bogdandoykarel: 2019-09-24 01:32:26 - 2019-09-24 01:37:2114:24
weshaydoesn't matter14:24
mwhahahaanyway sounds like we need to continue investigating our options around containers14:25
mwhahahaanythign else?14:25
weshaynope14:25
weshayaye14:25
mwhahahak moving on14:25
mwhahaha#topic Squad status14:25
mwhahahaci14:25
mwhahaha#link https://etherpad.openstack.org/p/tripleo-ci-squad-meeting14:25
mwhahahaupgrade14:25
mwhahaha#link https://etherpad.openstack.org/p/tripleo-upgrade-squad-status14:25
*** openstack changes topic to "Squad status (Meeting topic: tripleo)"14:25
mwhahahaedge14:25
mwhahaha#link https://etherpad.openstack.org/p/tripleo-edge-squad-status14:25
mwhahahaintegration14:25
mwhahaha#link https://etherpad.openstack.org/p/tripleo-integration-squad-status14:25
mwhahahavalidations14:25
mwhahaha#link https://etherpad.openstack.org/p/tripleo-validations-squad-status14:25
mwhahahanetworking14:25
mwhahaha#link https://etherpad.openstack.org/p/tripleo-networking-squad-status14:25
mwhahahatransformation14:25
mwhahaha#link https://etherpad.openstack.org/p/tripleo-ansible-agenda14:25
mwhahahaany status highlights?14:25
ykarelweshay, i see http://dashboard-ci.tripleo.org/d/si1tipHZk/jobs-exploration?orgId=1&from=now-12h&to=now&var-influxdb_filter=branch%7C%3D%7Cmaster&var-influxdb_filter=provider%7C%3D%7Crdo-cloud and in this i see 25,21,1514:25
ykarelthe other link seems not working14:25
weshayykarel, k.. let's go out of channel14:26
ykarelack14:26
*** brault has joined #tripleo14:26
bogdandoykarel: but that was prolly a bad example, there was nothing to update at all :)14:27
* ykarel checked what's with rhel8, there is dnf issue, so it didn't run completely14:27
weshaythere are never container updates in periodic..14:27
openstackgerritKarthik S proposed openstack/tripleo-heat-templates master: Fixed the path for neutron-base.yaml  https://review.opendev.org/68431814:28
mwhahahaok moving on14:30
mwhahaha#topic bugs & blueprints14:30
mwhahaha#link https://launchpad.net/tripleo/+milestone/train-rc114:30
mwhahahaFor Train we currently have 27 blueprints and 521 (+15) open Launchpad Bugs. 28 train-rc1, 493 ussuri-1.  91 (-75) open Storyboard bugs.14:30
mwhahaha#link https://storyboard.openstack.org/#!/project_group/7614:30
*** openstack changes topic to "bugs & blueprints (Meeting topic: tripleo)"14:30
mwhahahaI cleaned up the tripleo-ui story board items last week. Additionally I closed out milestone 3.  I will review the open rc1 bugs to move them to ussuri-1 if they aren't really blocking the rc1.14:30
mwhahaha#action mwhahaha review train-rc1 open bugs14:30
mwhahahaany questions/concerns around bugs/blueprints?14:31
apetrichamoralej, so I've narrowed down the issue to this https://i.imgflip.com/3bex5p.jpg it is a bit weird. I'm trying to understand what causes that14:32
mwhahahasounds like that's in reference to https://bugs.launchpad.net/tripleo/+bug/184518114:33
openstackLaunchpad bug 1845181 in tripleo "Problem with tripleo_common git functionalities and eventlet 0.25.1" [Medium,In progress] - Assigned to Adriano Petrich (apetrich)14:33
mwhahahaalso an appropriate use of memes14:33
mwhahahaanything else?14:33
apetrichmwhahaha, it is.14:33
apetrichmwhahaha, so tempted to add that to the bug14:34
*** hberaud|school-r is now known as hberaud14:34
openstackgerritMerged openstack/tripleo-heat-templates stable/stein: Enable deep_compare by default for stonith resources  https://review.opendev.org/68340914:34
openstackgerritMerged openstack/tripleo-heat-templates stable/rocky: Enable deep_compare by default for stonith resources  https://review.opendev.org/68340414:34
openstackgerritMerged openstack/tripleo-heat-templates stable/queens: Enable deep_compare by default for stonith resources  https://review.opendev.org/68340514:34
openstackgerritMerged openstack/tripleo-validations master: Update the constraints url  https://review.opendev.org/68416114:34
openstackgerritMerged openstack/tripleo-upgrade stable/stein: Fix "unbound" variable  https://review.opendev.org/68419114:34
weshay\0/14:35
mwhahahaalright moving on14:35
mwhahaha#topic projects releases or stable backports14:35
*** openstack changes topic to "projects releases or stable backports (Meeting topic: tripleo)"14:35
mwhahahaany release related topics?14:35
*** jpich has joined #tripleo14:37
mwhahahamoving on14:37
mwhahaha#topic specs14:37
mwhahaha#link https://review.openstack.org/#/q/project:openstack/tripleo-specs+status:open14:37
*** openstack changes topic to "specs (Meeting topic: tripleo)"14:37
mwhahahathe U cycle will be starting soon, don't forget to update your specs14:37
mwhahahaany spec related items?14:38
mwhahaha#topic open discussion14:40
*** openstack changes topic to "open discussion (Meeting topic: tripleo)"14:40
mwhahahaanything else?14:40
*** kopecmartin is now known as kopecmartin|off14:40
mwhahahaalright thanks everyone14:43
mwhahaha#endmeeting14:43
*** openstack changes topic to "CI Status: Orange (be kind with recheck and +A, watch for zuul queue) | community irc meeting Tues@1400 UTC - tripleo-ci-community meeting Tues@1330 UTC | https://docs.openstack.org/tripleo-docs"14:43
openstackMeeting ended Tue Sep 24 14:43:03 2019 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)14:43
openstackMinutes:        http://eavesdrop.openstack.org/meetings/tripleo/2019/tripleo.2019-09-24-14.00.html14:43
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/tripleo/2019/tripleo.2019-09-24-14.00.txt14:43
openstackLog:            http://eavesdrop.openstack.org/meetings/tripleo/2019/tripleo.2019-09-24-14.00.log.html14:43
*** hamdyk has quit IRC14:46
openstackgerritSagi Shnaidman proposed openstack/openstack-virtual-baremetal master: DNM: test master  https://review.opendev.org/68436414:46
*** leanderthal has joined #tripleo14:48
*** mdbooth has joined #tripleo14:53
mdboothodyssey4me: Hey there. I'm trying to patch some code which you touched most recently ;) https://review.opendev.org/#/c/683972/14:54
mdboothodyssey4me: I'm getting a CI failure which afaict must be bogus, because I can't see where the failing job loads the ipxe-boot image.14:57
openstackgerritCarlos Goncalves proposed openstack/diskimage-builder master: WIP: CentOS 8 support  https://review.opendev.org/68430814:58
mdboothIOW I don't think the job is testing my change, which is also an interesting observation.14:58
mwhahahaweshay, cloudnull, EmilienM: fun fact, we duplicate the layers for all the containers when we write them out14:58
openstackgerritmathieu bultel proposed openstack/tripleo-validations master: Fix get plan_name default value for inventory script  https://review.opendev.org/68436914:59
sshnaidmmdbooth, the job doesn't do anything with this ipxe image, it uses the hosted one on CI tenant14:59
weshaymwhahaha, when we write them out.. at build time?14:59
mwhahahaweshay, cloudnull, EmilienM: when we upload them to the container image registry, we dont' de-dupe already fetched laywers14:59
*** ratailor has joined #tripleo15:00
mwhahahaweshay, cloudnull, EmilienM: http://paste.openstack.org/show/779191/15:00
weshaymarios, panda rfolco rlandy fyi ^15:00
mwhahahaso there's an optimization to be had if we cache the sha's and symlink them15:00
*** tosky has quit IRC15:00
mwhahahaso we don't fetch all the layers over and over again15:00
* mwhahaha flips tables15:00
mwhahahawell we may not be refetching them (we shouldn't be) but we are writing them out over and over15:01
mwhahahaand each time we write out the base layer it's taking like 24 sec on my test vm15:01
cloudnull:\15:01
*** pcaruana has quit IRC15:01
weshaymwhahaha, before you go all steve ballmer, is that something we can change in the container push?15:02
mwhahahano15:02
mwhahahait's code we've written15:02
mwhahahaif only we didn't have to implement out own container registry15:02
mdboothsshnaidm: That would make sense based on the failure of my grep-fu ;)15:03
odyssey4memdbooth: looking15:03
mwhahahait looks liek there might be soem de-dupe code but i need to dig deeper15:03
*** paramite has quit IRC15:03
*** ratailor has quit IRC15:03
mwhahahaoh i take that back, we do de-dupe it to an extent15:03
*** abishop has quit IRC15:04
mwhahahabut it's still taking way to long to write out these files15:04
mariosweshay: whats the paste ... that quay?15:04
Tengumwhahaha: rewrite it in go15:04
* Tengu already out15:04
mariosweshay: (also on the phone)15:04
*** ccamacho has quit IRC15:05
mwhahahaTengu: oh you15:05
*** gfidente has quit IRC15:05
Tengu:D15:05
mwhahahalooks like we might be hardlinking some stuff15:06
mwhahahabut i think it's the streaming of the content in python that might be causing issues15:06
*** gbarros has joined #tripleo15:08
mwhahahahttps://github.com/openstack/tripleo-common/blob/master/tripleo_common/image/image_export.py#L91-L9915:08
mwhahahai think that's where some of the perf problems are15:09
openstackgerritFrancesco Pantano proposed openstack/tripleo-heat-templates master: Allow using registry authentication to pull ceph related containers  https://review.opendev.org/68201515:09
mdboothodyssey4me: That particular change was motivated by cutting down the development cycle whilst trying to produce the follow-up patch, btw.15:10
Tengumwhahaha: well, the hashlib.sha256() calls might as well create some slowdowns..15:10
*** Vorrtex has quit IRC15:10
mdboothodyssey4me: 3 second rebuild vs 15 minutes or something like that.15:10
odyssey4memdbooth: yeah, it looks like it's got all the right bits as far as I can see - when I did the last modification I honestly thought that using dib was silly, but went with it anyway15:11
Tengumwhahaha: maybe we can add buffering=0  to the "open" ? that should allow a direct write instead of buffering/waiting15:11
mwhahahamaybe, going to poke at that15:11
odyssey4memdbooth: as sshnaidm the failing job is in RDO and defined outside of the openstack repositories... so you'll have to hunt down the third party CI owner15:12
odyssey4memdbooth: although, it's possible that the issue is the deletion of the resulting qcow2 file from the repo15:13
*** saneax|AFK has quit IRC15:14
mdboothodyssey4me: That was my primary suspect if it was an issue with my patch.15:15
mdboothodyssey4me: Ok, thanks. Appreciate your review on the patch and the follow-up addition to your ipxe boot script, btw.15:15
odyssey4memdbooth: while I'm not a fan of including the binary - it seems that there may be things relying on it... so we're kinda stuck for now with keeping it there15:15
sshnaidmmdbooth, seems like master is broken http://logs.rdoproject.org/72/683972/1/openstack-check/tripleo-ci-centos-7-ovb-3ctlr_1comp-featureset001-ovb-master/aa37cca/job-output.txt.gz15:16
sshnaidmbnemec, ^^15:16
mdboothodyssey4me: I can certainly put it back in. However, if we can find what is relying on it, I'm also happy to fix that instead.15:16
sshnaidmovb stack is not created http://logs.rdoproject.org/72/683972/1/openstack-check/tripleo-ci-centos-7-ovb-3ctlr_1comp-featureset001-ovb-master/aa37cca/logs/failed_ovb_stack.log15:18
sshnaidmResource CREATE failed: WaitConditionFailure: resources.baremetal_env.resources.bmc.resources.bmc_wait_condition: Signal 1 received15:18
*** brault has quit IRC15:18
sshnaidmbnemec, any ideas why could it happen maybe? ^15:18
*** brault has joined #tripleo15:19
*** brault has quit IRC15:19
bnemecsshnaidm: Sounds like bmc deployment failed. You'll need to look at the console log to see why.15:19
*** lmiccini has quit IRC15:23
*** leanderthal has quit IRC15:23
openstackgerritKevin Carter (cloudnull) proposed openstack/tripleo-ansible master: Pivot ovs-dpdk molecule to ansible tasks  https://review.opendev.org/68437915:25
openstackgerritKevin Carter (cloudnull) proposed openstack/tripleo-ansible master: Add kernelargs and tuned core configuration  https://review.opendev.org/67980915:25
*** lennyb has quit IRC15:25
sshnaidmbnemec, if node is not created, so it doesn't have console - isn't it?15:28
bnemecsshnaidm: If the node wasn't created I would have expected it to fail on that resource, not the wait condition.15:30
bnemecBut yeah, if the node is missing then that's a problem.15:30
*** lennyb has joined #tripleo15:30
*** iurygregory has quit IRC15:32
sshnaidmbnemec, seems like it's missing..15:32
sshnaidmok, will create a bug, seems like need to check with reproducer15:32
sshnaidmfyi https://bugs.launchpad.net/tripleo/+bug/184522515:34
openstackLaunchpad bug 1845225 in tripleo "OVB master branch job is broken, can't create bmc" [High,Triaged]15:34
sshnaidmweshay, ^15:35
openstackgerritAdriano Petrich proposed openstack/tripleo-common master: Make gitpython and eventlet work with eventlet 0.25.1  https://review.opendev.org/68438015:35
sshnaidmmdbooth, so for now you can ignore 3party ci on master, see https://bugs.launchpad.net/tripleo/+bug/184522515:37
openstackLaunchpad bug 1845225 in tripleo "OVB master branch job is broken, can't create bmc" [High,Triaged]15:37
weshaysshnaidm, thanks15:37
cloudnullmwhahaha I think you may be right about the performance issue with writing content, I think we're in some cases processing the same data stream 2, sometime 3, times per container.15:38
mwhahahacloudnull: i'm seeing delays in teh writing of chunks15:39
mwhahahai added some debugging in and it's hanging every one in a while15:39
cloudnullwe have things like so https://github.com/openstack/tripleo-common/blob/master/tripleo_common/image/image_export.py#L91-L99,15:39
mwhahahait might be a buffering thign on output15:39
mwhahahacloudnull: we're reading in chunks of 1meg15:39
mdboothweshay: Did I break it, btw, or was it already broken?15:39
cloudnullso its coming from the requests stream generator, to our generator, to the chunk writing loop15:40
mwhahahacloudnull: yea, i'm going to try turning off the write buffer15:40
cloudnull++15:40
weshaymdbooth, looks like it was broken before you got there... https://review.opendev.org/#/c/684364/15:40
weshaysshnaidm, has a no-op change.. and it's boosted15:41
* mwhahaha waits for ctl+c to actualy process15:41
openstackgerritHarald Jensås proposed openstack/puppet-tripleo master: Add 'ipversion' to firewall/rule.pp  https://review.opendev.org/68438415:42
cloudnulland it looks like we have two different stream processors, the same, but different thing seems to be happening here too  - https://github.com/openstack/tripleo-common/blob/master/tripleo_common/image/image_uploader.py#L1615-L164415:42
sshnaidmweshay, just a month before it was ok: https://review.opendev.org/#/c/675601/15:42
mdboothweshay: \o/ ;)15:42
openstackgerritHarald Jensås proposed openstack/tripleo-heat-templates master: Add DHCPv6 rules + set ipversion for dhcp rules  https://review.opendev.org/68438515:42
openstackgerritHarald Jensås proposed openstack/tripleo-ansible master: Support 'ipversion' property in rules  https://review.opendev.org/68427715:44
openstackgerritHarald Jensås proposed openstack/tripleo-ansible master: Open ipv6-icmp traffic by default  https://review.opendev.org/68427815:44
openstackgerritHarald Jensås proposed openstack/openstack-virtual-baremetal master: Add press Ctrl-B for iPXE command line  https://review.opendev.org/68438715:47
openstackgerritHarald Jensås proposed openstack/openstack-virtual-baremetal master: Add possiblity to build IPv6 IPXE iamge  https://review.opendev.org/68438815:47
mwhahahacloudnull, EmilienM: so i think the sluggish writing is actualy a function of the speed we pull from docker.io (and mirrors)15:51
mwhahahacloudnull, EmilienM: since we're using streams and fixed chunk sizes, we hang until the chunk is full, then we write it out.15:51
mwhahahacloudnull, EmilienM: What i'm seeing is that writting occurs, then hangs in the middle, then continues15:51
cloudnulldo you think we're being rate-limited from docker?15:51
mwhahahacloudnull, EmilienM: I think it's because we're using the fixed 1meg chunksize for the stream15:52
mwhahahacloudnull: i don't think it's a ratelimit, i think it's just a transit issue15:52
mwhahahathe mirroring should help but it doesn't seem to be, i wonder if cache hits are bad or something15:52
EmilienMshould we try to reduce the chunk size?15:52
mwhahahahttps://opendev.org/openstack/tripleo-common/src/branch/master/tripleo_common/image/image_uploader.py#L1350-L136015:52
mwhahahawe might want to drop it for the url requests15:53
mwhahahabecause it'll return what it has available if not specified15:53
mwhahahahttps://2.python-requests.org/en/master/api/#requests.Response.iter_content15:53
*** yprokule has quit IRC15:53
mwhahahalet me try setting to None in my env to see if I see the studdering downloads/writing15:53
*** ykarel is now known as ykarel|afk15:54
cloudnullI wonder if setting it to something like 8192 instead of 1048576 would be a better overall chuck size?15:54
mwhahahaon my test vm, i'm not iowating on disk so I don't think it's pending disk wait15:55
mwhahahawhich is why i think it's network15:55
mwhahahai think we might get better perf w/o it specified15:56
mwhahahai'm seeing a wider range of chunks15:56
mwhahaha841689015:56
mwhahaha3768747615:56
openstackgerritTakashi Kajinami proposed openstack/tripleo-heat-templates master: Use OS::TripleO::Services::Logging::NovaLibvirt to configure libvirt logging  https://review.opendev.org/68375215:56
openstackgerritTakashi Kajinami proposed openstack/tripleo-heat-templates master: DNM: Test libvirt logging  https://review.opendev.org/68381115:56
mwhahaha3585436015:56
cloudnull++ the default of 512 should perform fine, and we can raise it if needed15:57
mwhahahai specified None actually15:57
mwhahahaand let it figure it out15:57
mwhahahai think it returns what is available15:57
cloudnull+115:57
*** marios has quit IRC15:57
* mwhahaha proposes patch15:57
openstackgerritTakashi Kajinami proposed openstack/tripleo-heat-templates master: Use OS::TripleO::Services::Logging::NovaLibvirt to configure libvirt logging  https://review.opendev.org/68375215:58
openstackgerritTakashi Kajinami proposed openstack/tripleo-heat-templates master: DNM: Test libvirt logging  https://review.opendev.org/68381115:58
cloudnull"chunk_size must be of type int or None. A value of None will function differently depending on the value of stream. stream=True will read data as it arrives in whatever size the chunks are received. If stream=False, data is returned as a single chunk."15:58
cloudnullNone seems like the right option for our use15:58
cloudnullnice find/catch15:58
openstackgerritAlex Schultz proposed openstack/tripleo-common master: Remove chunk size for url stream  https://review.opendev.org/68439115:59
mwhahahalets see how that works, i'll fix tests ands tuff in a bit15:59
*** suuuper has quit IRC16:01
*** panda is now known as panda|bbl16:02
openstackgerritBrent Eagles proposed openstack/tripleo-quickstart master: Add support for running tempest against Octavia featuresets  https://review.opendev.org/66966616:03
openstackgerritBrent Eagles proposed openstack/tripleo-quickstart master: Set the amphora image to stein branch  https://review.opendev.org/67889916:03
bogdandomwhahaha, cloudnull: yea, that was indeed the great finding16:04
* bogdando rants around for now solving images streaming issues for containers...16:05
bogdandohow long to implement a registry? :)16:05
mwhahahayeaaaa16:05
*** alexmcleod has quit IRC16:06
mwhahahaif only there was a quality single host registry solution that didn't require the a super computer to serve files16:06
*** rpittau is now known as rpittau|afk16:08
*** cylopez has quit IRC16:10
*** udesale has quit IRC16:10
apetrichamoralej, EmilienM I have a patch up for https://bugs.launchpad.net/tripleo/+bug/1845181 from my local testing it fixes the issue.16:12
openstackLaunchpad bug 1845181 in tripleo "Problem with tripleo_common git functionalities and eventlet 0.25.1" [Medium,In progress] - Assigned to Adriano Petrich (apetrich)16:12
apetrichI'm going to clock out now. if you need anything just ping here that I will read before tomorrow16:13
amoralejapetrich, nice, let me do a quick test16:13
apetrichamoralej, I'm here waiting for the test then :)16:13
amoralejyou can go, i'll leave the results here16:14
apetrichamoralej, cheers16:14
*** dtantsur is now known as dtantsur|afk16:15
mwhahahameh still seeing stalls in processing, not sure where it's hanging now. I guess need more debug logs16:19
*** brault has joined #tripleo16:20
amoralejapetrich, i still see the issue with your patch :(16:21
*** jpich has quit IRC16:21
apetrichamoralej, let me double check. I probably messed something copying16:22
amoralejapetrich, how are you reproducing?16:22
openstackgerritHarald Jensås proposed openstack/tripleo-heat-templates master: Add DHCPv6 rules + set ipversion for dhcp rules  https://review.opendev.org/68438516:22
apetrichamoralej, two scripts based on the bug report. but I was coding on the one of the scripts and when it worked I copied to the tripleo-common. let me do it proper16:23
amoralejok16:23
amoralejnote it only fails when trying to commit the first time16:23
amoralejif there is a commit, it works16:24
apetrichamoralej, http://paste.openstack.org/show/779198/16:24
*** holser has quit IRC16:24
amoralejdunno why16:24
apetrichamoralej, that is due to the gitpython16:24
apetrichif it finds a head it uses it, but for the first commit there's no head and it lazy loads git.refs and creates a head16:24
amoralejok16:25
apetrichit is failing on the get path for the head creation16:25
*** brault has quit IRC16:27
amoralejapetrich, that script fails in my env with eventlet-0.25.116:27
apetrichamoralej, wha16:27
apetrichamoralej, not for me. with eventlet 0.24 and 0.2516:27
*** brault has joined #tripleo16:28
*** brault has quit IRC16:28
apetrichamoralej, ok. let me double check everything.16:28
*** brault has joined #tripleo16:28
*** bogdando has quit IRC16:28
amoralejhttp://paste.openstack.org/show/779200/16:28
amoralejmmm16:28
amoralejwith 0.24.1 passes16:28
amoralejlet me think what i may have done wrong16:28
*** jaosorior has joined #tripleo16:29
*** brault has quit IRC16:29
amoraleji'm running in fedora with python2, you are using on centos7?16:29
apetrichamoralej, no you are right. I did something weird. it is late16:29
amoralejok, no problem16:30
*** hberaud is now known as hberaud|gone16:30
apetrichamoralej, I see what I did. I didn't save a file from eventlet that I was debugging. my patch doesn't work. I will plow through it some more tomorrow16:31
apetrichamoralej, EmilienM sorry about that.16:31
*** brault has joined #tripleo16:32
amoralejno problem, can wait16:32
*** yoctozepto has quit IRC16:35
*** yoctozepto has joined #tripleo16:40
*** tesseract has quit IRC16:46
mwhahahacloudnull, EmilienM: i found our usage of futures is less than optimal. I shall propose a fix16:46
*** jaosorior has quit IRC16:51
openstackgerritMerged openstack/paunch stable/stein: Revert "Add --cpuset-cpus support for both Docker and Podman"  https://review.opendev.org/68418016:54
*** panda|bbl is now known as panda16:56
*** Vorrtex has joined #tripleo16:56
*** waleedm has joined #tripleo16:56
*** ramishra has quit IRC16:57
openstackgerritTakashi Kajinami proposed openstack/tripleo-heat-templates master: Use OS::TripleO::Services::Logging::NovaLibvirt to configure libvirt logging  https://review.opendev.org/68375216:59
openstackgerritTakashi Kajinami proposed openstack/tripleo-heat-templates master: DNM: Test libvirt logging  https://review.opendev.org/68381116:59
*** ricolin has quit IRC17:03
openstackgerritAdriano Petrich proposed openstack/tripleo-common master: Make gitpython and eventlet work with eventlet 0.25.1  https://review.opendev.org/68438017:16
apetrichamoralej, was able to squeeze a change before bedtime for the kids. See you tomorrow17:17
amoralejack, see you17:17
amoralejapetrich, worked fine in my env17:21
openstackgerritMerged openstack/paunch stable/stein: Improve volume validation  https://review.opendev.org/68338617:21
apetrichamoralej, nice to know :)17:27
EmilienMweshay: I'm going to add tripleorocky & tripleoqueens namespaces to quay.io, you'll get invit on the triploe ci account again17:31
*** brault has quit IRC17:31
EmilienMweshay: if you can accept so I finish the process and we won't have perm errors17:32
weshayk17:32
*** jpena is now known as jpena|off17:32
* weshay chatting w/ the folks now.. they will take the patch over17:32
*** hjensas has quit IRC17:36
*** waleedm has quit IRC17:36
EmilienMweshay: thanks17:43
EmilienMweshay: master/stein/rocky/queens is allset17:44
EmilienMweshay: I'm manually pushing current-tripleo in tripleomaster now17:45
weshayrlandy, panda rfolco ^17:57
weshayEmilienM, ya.. let's rekick the test patch that consumes containers.. now that we have the proxy17:58
* weshay logs into quay17:58
*** psachin has quit IRC17:59
*** ykarel|afk is now known as ykarel|away18:00
*** avivgta has quit IRC18:01
*** amoralej is now known as amoralej|off18:02
*** dsneddon has joined #tripleo18:05
*** Vorrtex has quit IRC18:07
*** Goneri has joined #tripleo18:10
openstackgerritAlex Schultz proposed openstack/tripleo-common master: Improve ThreadPoolExecutor usage  https://review.opendev.org/68441118:11
mwhahahacloudnull, EmilienM -^18:11
*** ykarel|away has quit IRC18:15
EmilienMmwhahaha: looking18:18
mwhahahait may not address everything but should give us a mor accurate picture of when things finish18:19
EmilienMnicely don18:20
*** jtomasek has quit IRC18:21
*** Vorrtex has joined #tripleo18:22
*** bdodd has quit IRC18:22
*** bdodd has joined #tripleo18:24
*** bdodd has quit IRC18:24
*** morazi has quit IRC18:25
mwhahahawonder if we should adjust the upload workers as well18:26
EmilienMshould we stop tripleo jobs in check to give a chance to your patches?18:28
weshaycan you do that?18:28
EmilienMweshay: if I abandon them yes18:29
EmilienMi suspect alex's patches which fix the world will not be tested before tomorrow if we do nothing18:30
weshaywe're not bleeding in the gate,  check queue is at 7hrs.. just fyi18:30
EmilienMyes I know18:30
EmilienMwhich is why i'm saying that18:30
EmilienMthoughts?18:31
*** ykarel|away has joined #tripleo18:35
EmilienMweshay: tripleomaster -- pushed. now pushing stein rocky and queens18:35
*** d0ugal has quit IRC18:37
EmilienMweshay: ah I'm blocked, please accept invit to join https://quay.io/organization/tripleostein/teams/tripleoci?tab=teams18:37
EmilienMmwhahaha: ^ thought son check?18:37
*** Goneri has quit IRC18:37
mwhahahano let's not do that for now18:38
mwhahahai'm running tests locally and will likely ahve more improvements18:38
EmilienMk18:38
mwhahahaat the moment i think well only be able to make certain smaller incremental fixes18:38
mwhahahamaybe the cumulative fixes will help but i don't think we have a single one to fix it all yet18:39
cloudnullmwhahaha nice18:39
EmilienMmwhahaha: you still believe in quay.io switch?18:40
cloudnullwe ran into issues with futures not raising an exception when there's a fault in a thread, we had to check change that in buildha.py to inspect the jobs - https://github.com/openstack/tripleo-common/blob/master/tripleo_common/image/builder/buildah.py#L170-L193 - but if this works then its a much cleaner solution18:40
mwhahahacloudnull: not sure if we've ever had an exception in that bit of code, but we'll see18:40
mwhahahaEmilienM: it won't hurt18:40
cloudnull++ fair enough18:40
cloudnullthe implementation of futures throught the tripleo-common code base is fairly copy/paste, I wonder if we shouldn't move this into some common method ?18:41
mwhahahanot necessarily18:42
mwhahahabut we'll see18:42
*** ykarel has joined #tripleo18:42
mwhahahathe copy image loop is duped, but the others were p.map invocations18:42
*** ykarel|away has quit IRC18:43
cloudnulldo we even need to use map ?18:45
cloudnullI suspect a simple thread should work just fine ?18:45
mwhahahai think map is correct18:49
mwhahahai need to review what's available in futures but i think that's what we're wanting18:49
cloudnullI guess map / submit one in the same given we're not using the ProcessPoolExecutor18:49
cloudnullmap will just auto-iterate over the the set, so +1 map seems like the better all around solution18:50
mwhahahaultimately i think map is what we're using everywhere18:52
mwhahahawhere we put the work together than then collect the results18:52
mwhahahaah map is in order18:55
mwhahahaand we don't care about the order for the layers i guess18:55
*** lucasagomes has quit IRC19:00
*** ansmith_ has joined #tripleo19:02
*** d0ugal has joined #tripleo19:02
*** ansmith has quit IRC19:04
*** morazi has joined #tripleo19:07
openstackgerritAlex Schultz proposed openstack/tripleo-common master: Improve ThreadPoolExecutor usage  https://review.opendev.org/68441119:07
openstackgerritFrancesco Pantano proposed openstack/tripleo-heat-templates master: Add registry authentication parameters to pull ceph related containers  https://review.opendev.org/67769919:16
tbarronweshay: so I'm still trying to locally repro the ceph-ansible issue from the rhel8 job in https://review.opendev.org/#/c/682853/ but I'm missing how to get the ceph-ansible pkg that has the issue installed on my standalone19:22
weshaytbarron, ah19:22
* weshay helps19:23
tbarronthe job pulls it from a pulp mirror, no baseurl19:23
weshaytbarron, seriously appologize we'd be more involved if the world were not on fire19:23
tbarronmy standalone is centos7 not rhel8 too ..19:23
openstackgerritKevin Carter (cloudnull) proposed openstack/tripleo-ansible master: Pivot ovs-dpdk molecule to ansible tasks  https://review.opendev.org/68437919:23
tbarronweshay: yah, feel free to tell me to go away and come back later19:23
openstackgerritKevin Carter (cloudnull) proposed openstack/tripleo-ansible master: Pivot ovs-dpdk molecule to ansible tasks  https://review.opendev.org/68437919:24
openstackgerritKevin Carter (cloudnull) proposed openstack/tripleo-ansible master: Add kernelargs and tuned core configuration  https://review.opendev.org/67980919:24
*** morazi has quit IRC19:25
*** morazi has joined #tripleo19:29
openstackgerritFrancesco Pantano proposed openstack/tripleo-heat-templates master: Allow using registry authentication to pull ceph related containers  https://review.opendev.org/68201519:34
*** morazi has quit IRC19:38
openstackgerritFrancesco Pantano proposed openstack/tripleo-heat-templates master: Allow using registry authentication to pull ceph related containers  https://review.opendev.org/68201519:42
openstackgerritAlex Schultz proposed openstack/tripleo-common stable/rocky: Add profile_tasks to ansible runs  https://review.opendev.org/68430919:43
openstackgerritAlex Schultz proposed openstack/tripleo-common stable/queens: Add profile_tasks to ansible runs  https://review.opendev.org/68431019:44
*** morazi has joined #tripleo19:46
*** ykarel has quit IRC19:47
EmilienMweshay: queens / rocky done, stein almost done19:47
*** rlandy has quit IRC19:50
*** rlandy has joined #tripleo19:54
weshayEmilienM, k.. cool20:02
*** noama has quit IRC20:07
cloudnullanyone around available to give https://review.opendev.org/#/c/684379 a nudge - fixes the ovs-dpdk jobs unblocking 67980920:08
openstackgerritJose Luis Franco proposed openstack/tripleo-upgrade master: Sync up upgrades and updates workarounds mechanism.  https://review.opendev.org/68441920:09
*** aakarsh has joined #tripleo20:13
*** jfrancoa has quit IRC20:17
*** ansmith has joined #tripleo20:19
*** ansmith_ has quit IRC20:20
*** ade_lee_ has quit IRC20:22
*** florianf has quit IRC20:25
*** morazi has quit IRC20:25
*** holser has joined #tripleo20:32
*** ansmith_ has joined #tripleo20:32
*** ansmith has quit IRC20:34
*** jcoufal has quit IRC20:34
*** ansmith_ has quit IRC20:36
*** Vorrtex has quit IRC20:38
*** morazi has joined #tripleo20:43
*** ade_lee_ has joined #tripleo20:59
openstackgerritEmilien Macchi proposed openstack/tripleo-repos master: Add support for CentOS8  https://review.opendev.org/68442621:00
*** jschlueter has quit IRC21:01
*** jschlueter has joined #tripleo21:03
*** ade_lee_ has quit IRC21:05
*** ade_lee_ has joined #tripleo21:06
*** mcornea has quit IRC21:09
*** rfolco has quit IRC21:11
*** rfolco has joined #tripleo21:11
*** xek has quit IRC21:11
openstackgerritMerged openstack/puppet-tripleo stable/rocky: Enable deep_compare for fencing resource  https://review.opendev.org/68339021:13
openstackgerritMerged openstack/puppet-tripleo stable/queens: Enable deep_compare for fencing resource  https://review.opendev.org/68339121:13
*** aakarsh has quit IRC21:16
*** artom has quit IRC21:17
*** artom has joined #tripleo21:18
openstackgerritMerged openstack/tripleo-ansible master: Pivot ovs-dpdk molecule to ansible tasks  https://review.opendev.org/68437921:26
openstackgerritLance Bragstad proposed openstack/tripleo-heat-templates master: Enable caching for keystone overcloud  https://review.opendev.org/68443221:33
lbragstad^ gave me a nice performance bump on basic overcloud installations21:34
lbragstad(i only timed keystone performance though)21:34
stevebakermwhahaha: hey, I need some puppet-nova advice21:42
mwhahahaorly21:42
stevebakermwhahaha: puppet-nova also configures novajoin, a completely seperate side-project. I need to do the novajoin equivalent of nova::policies, but it is not obvious to me what to call it or where to put it https://opendev.org/openstack/puppet-nova/src/branch/master/manifests/policy.pp21:44
* mwhahaha hates novajoin21:44
stevebakermwhahaha: it would be identical to nova::policies but it writes to /etc/novajoin/policy.json21:44
mwhahahaput it in metadata/novajoin21:45
stevebakerso metadata/novajoin/policy.pp? ok21:46
EmilienMécopy/pasta21:46
*** bfournie has quit IRC21:47
mwhahahastevebaker: yes21:51
stevebakermwhahaha: ok thanks21:51
*** raildo has quit IRC21:53
EmilienMstevebaker: I thought policy.json was somehow deprecated and replaced by policy in code21:57
*** holser has quit IRC21:57
mwhahahai thought the default is now in code21:57
mwhahahabut you could still have one21:57
*** weshay has quit IRC21:58
*** slaweq has quit IRC22:00
openstackgerritEmilien Macchi proposed openstack/tripleo-repos master: Add support for CentOS8  https://review.opendev.org/68442622:02
*** gchamoul has quit IRC22:04
*** rcernin has joined #tripleo22:07
*** gchamoul has joined #tripleo22:14
*** weshay has joined #tripleo22:19
EmilienMcloudnull: have you seen that before ? http://paste.openstack.org/show/779217/22:21
EmilienMseems like centos8 ships ansible 2.7.622:22
cloudnullno i have not22:23
EmilienMprobably the kernel ansible module isn't compatible with 2.22:23
EmilienM2.722:23
cloudnullwe've been testing with 2.822:23
cloudnullI think that's what upstream has been running ?22:23
EmilienMrigt, i'm now testing centos822:23
cloudnulloh.22:23
EmilienMhttps://trunk.rdoproject.org/rhel8-master/deps/latest/Packages/ansible-2.7.6-1.el8.noarch.rpm22:23
EmilienMthat's what is shipped in rdo for rhel822:24
cloudnullshould we be testing with 2.7 ?22:24
* cloudnull can change that if needed22:24
stevebakerEmilienM: I want to override the code policy because it limits calling novajoin to the nova user22:24
EmilienMcloudnull: my rhel8 env is down now22:25
EmilienMI don't know what version we run there22:25
EmilienMstevebaker: ack, thx22:25
cloudnullrhel8 ==ansible 2.8.422:26
cloudnullw/ osp1622:26
weshayEmilienM, the rhel8 rhui repos have 2.822:26
* weshay double checks my math22:26
EmilienMah so the rdo repo is wrong22:27
cloudnullEmilienM I would suspect that filter exists in 2.7 but the keyword arg is just not present .22:27
EmilienMright22:27
weshayansible-2.8.3-1.el8ae.noarch22:27
weshayEmilienM, you might want to hold back for a bit there cowboy22:27
EmilienMno, I'm trying to deploy w/ workarounds until it's finished22:28
EmilienMso we can find potential issues quickly22:28
weshaythat's why we have rhel22:28
weshaythe package set should be very close... /me hopes22:28
*** rfolco has quit IRC22:29
EmilienMhow do we handle these deps? is there a repo somewhere or it's manual?22:29
*** rfolco has joined #tripleo22:29
weshayEmilienM, it's in the RHUI22:31
weshayEmilienM, so.. the quay job returned22:31
weshay2019-09-24 01:23:48 | HTTPError: 401 Client Error: UNAUTHORIZED for url: https://quay.io/v2/tripleomaster/centos-binary-cron/manifests/current-tripleo22:31
weshayfirst job didn't hit that.. second one did..22:31
EmilienMok let me try22:32
EmilienMmhh22:33
weshayslow connection here22:33
weshaytrying it manually22:33
EmilienMme too22:34
weshaycloudnull, we have to keep EmilienM focused some times22:34
cloudnulllol22:34
EmilienMerror pulling image "quay.io/tripleomaster/centos-binary-aodh-api": unable to pull quay.io/tripleomaster/centos-binary-aodh-api: unable to pull image: Error determining manifest MIME type for docker://quay.io/tripleomaster/centos-binary-aodh-api:latest: Error reading manifest latest in quay.io/tripleomaster/centos-binary-aodh-api: manifest unknown: manifest unknown22:35
weshayworking for me w/ the tag current-tripleo22:39
EmilienMah I missed :current-trpleo22:39
EmilienMyea22:39
* weshay checks the test change22:39
EmilienMlet me check the cron22:39
weshayhrm... looks ok22:39
EmilienMit's public :/22:39
weshaywhich cron?22:39
EmilienMthe container22:40
EmilienMcentos-binary-cron22:40
EmilienMwhere do you see that error?22:40
* weshay gets22:40
weshayhttps://55cb9f8472ab875bc5fc-29df7c45e84d4e54520155da59ed8678.ssl.cf1.rackcdn.com/683212/3/check/tripleo-ci-centos-7-containers-multinode/e8709ed/logs/undercloud/home/zuul/undercloud_install.log.txt.gz22:41
weshaywe probably need to set the token in a pre.yaml ?22:42
weshayor should that just work like it does via cli22:42
EmilienMno22:42
EmilienMit should work fine22:42
EmilienMwhen did the job run?22:42
EmilienMI could pull the image on a node where I'm not logged22:43
*** bfournie has joined #tripleo22:43
*** ekultails has quit IRC22:43
EmilienMmaybe a transient error?22:44
EmilienMrecheck22:44
weshayEmilienM, maybe they got my email.. and said f.. off22:44
*** holser has joined #tripleo22:46
EmilienMlol22:47
EmilienMvery lickely22:47
EmilienMlikely damn I can't type with that lag22:48
EmilienM"libfacter was not found. Please make sure it was installed to the expected location."22:50
EmilienManother packaging issue22:51
*** dtantsur|afk has quit IRC22:51
mwhahahadid you install ruby-facter22:51
mwhahahaor something22:51
*** stevebaker has quit IRC22:51
*** tkajinam has joined #tripleo22:51
EmilienMyes, ruby-facter-1:3.9.3-9.el8ost.x86_6422:52
EmilienMcloudnull: FYI I requested a new ansible in RDO but also created https://bugs.launchpad.net/tripleo/+bug/1845272 for visibility22:52
openstackLaunchpad bug 1845272 in tripleo "tripleo-kernel fails on ansible 2.7" [Medium,Triaged]22:52
cloudnullI suspect we have several roles that will fail on 2.722:53
EmilienMI have facter-1:3.9.3-9.el8ost.x86_6422:53
cloudnullIf we think it best I can change our role test strategy to run w/ 2.722:53
EmilienMcloudnull: yeah, that's what I thought22:53
*** aakarsh has joined #tripleo22:55
*** stevebaker has joined #tripleo23:06
EmilienMmwhahaha: yeah I have /usr/lib64/libfacter.so.3.9.323:07
openstackgerritSteve Baker proposed openstack/tripleo-heat-templates master: Override policy for undercloud novajoin  https://review.opendev.org/68414223:08
*** rfolco has quit IRC23:13
*** rfolco has joined #tripleo23:14
*** stevebaker has quit IRC23:19
*** stevebaker has joined #tripleo23:20
*** ansmith has joined #tripleo23:21
*** chem has quit IRC23:26
*** holser has quit IRC23:29
*** weshay has quit IRC23:31
mwhahahacloudnull, EmilienM soooo https://opendev.org/openstack/tripleo-common/commit/e06cf4482da855d63ab21f8183cea66499e6f5f923:31
*** gbarros has quit IRC23:31
mwhahahacloudnull, EmilienM: that changed up our workers, which made it CPU-1 * CPU-123:31
EmilienMpushups23:31
mwhahahainstead of CPU-1 / 2 *  CPU-1 / 223:32
cloudnull?23:32
mwhahahaer CPU/2 * CPU/223:32
EmilienMmax(2, (processutils.get_worker_count() - 1))23:33
mwhahahathe entry point to our processing is https://opendev.org/openstack/tripleo-common/src/branch/master/tripleo_common/image/image_uploader.py#L1913-L191823:33
cloudnullthat should be cpu-1 or 223:33
mwhahahahrm wait23:33
mwhahahamayeb i'm looking at a bad copy23:33
mwhahahaeither way, we threadpool the uploaders23:33
mwhahahaand then threadpool the layers in each one23:33
mwhahahaso it's likely that we need to tone down the workers23:34
mwhahahabecause for each uploader, we end up doing various threading operations in it23:34
mwhahahafrom tag fetching to layer transfers23:34
cloudnullwe've been trying something similar here https://review.opendev.org/#/c/68273123:36
mwhahahaso we don't want them to be dynamic23:36
mwhahahabecause it's likely the source is the same place23:37
openstackgerritAlex Schultz proposed openstack/tripleo-common master: Improve ThreadPoolExecutor usage  https://review.opendev.org/68441123:37
cloudnullwouldn't we want to use more workers on a stronger system? like in the case of prod?23:37
mwhahahano23:37
mwhahahabecause http requests limitations23:37
mwhahahayou can't spawn 100 requests to docker.io23:38
mwhahahai mean you can23:38
mwhahahabut it's going to run terrible23:38
mwhahahakinda like what we've got now :D23:39
cloudnullbefore it would have been cpu/2 or 2, so on a 48 core system it'd have 24 workers, so maybe we need to change that worker setting, and set an upper limit?23:39
mwhahahait's possible23:40
mwhahahabut let's see if we can get it down for 8 cpu23:40
mwhahahait's not like we have more layers/containers23:40
cloudnulland if run on a power system we'd see something like 64+ workers :)23:40
cloudnullbut it makes sense. if we cant spawn that many connections to docker then I can see how itd become crazy slow23:41
mwhahahai might commit my logging improvements23:42
mwhahahamakes this multithreading stuff easier to follow23:42
cloudnull+223:46
mwhahahaoh god damn it23:49
mwhahahawho wants to guess what's wrong with this http://paste.openstack.org/show/779220/23:50
cloudnullprocesses not being cleaned up ?23:50
cloudnullsessions not closing ?23:51
mwhahahading23:51
mwhahahawant to guess how much we use this code23:51
cloudnull80%23:51
mwhahahahttps://github.com/openstack/tripleo-common/blob/master/tripleo_common/image/image_uploader.py#L1588-L161223:51
mwhahahaand we're doing it to ourselves23:52
mwhahahabecause apache23:52
*** gbarros has joined #tripleo23:52
mwhahahathats why i was seeing hangs23:52
mwhahahait hangs trying to close-wait to the registry23:52
mwhahahaand can't open more requests23:52
cloudnulltime to upgrade to NGINX?23:53
* cloudnull hides 23:53
mwhahahapfffft23:53
mwhahahaeh i'll fix it tomorrow23:58
*** gbarros_ has joined #tripleo23:59

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