Friday, 2018-11-30

openstackgerritDamien Ciabrini proposed openstack/tripleo-common master: DNR undercloud/overcloud with mariadb 10.3.10 and galera 3.23  https://review.openstack.org/61735500:06
openstackgerritDan Radez proposed openstack/tripleo-common stable/queens: Install python2-networking-ansible in neutron-server  https://review.openstack.org/62104400:07
*** ade_lee has quit IRC00:09
*** wolverineav has quit IRC00:09
*** flaper87 has quit IRC00:09
*** ooolpbot has joined #tripleo00:10
ooolpbotURGENT TRIPLEO TASKS NEED ATTENTION00:10
ooolpbothttps://bugs.launchpad.net/tripleo/+bug/180587000:10
*** ooolpbot has quit IRC00:10
openstackLaunchpad bug 1805870 in tripleo "mistral unit tests failing in dlrn build of tripleo-common" [Critical,Triaged] - Assigned to Adriano Petrich (apetrich)00:10
*** wolverineav has joined #tripleo00:10
*** wolverineav has quit IRC00:12
*** vinaykns has quit IRC00:13
*** wolverineav has joined #tripleo00:13
*** flaper87 has joined #tripleo00:14
*** threestrands has joined #tripleo00:18
*** ansmith has joined #tripleo00:19
*** tosky has quit IRC00:27
*** rh-jelabarre has joined #tripleo00:41
*** aedc has quit IRC00:49
*** bfournie has joined #tripleo01:02
*** hamzy_ has joined #tripleo01:07
*** ooolpbot has joined #tripleo01:10
ooolpbotURGENT TRIPLEO TASKS NEED ATTENTION01:10
ooolpbothttps://bugs.launchpad.net/tripleo/+bug/180587001:10
*** ooolpbot has quit IRC01:10
openstackLaunchpad bug 1805870 in tripleo "mistral unit tests failing in dlrn build of tripleo-common" [Critical,Triaged] - Assigned to Adriano Petrich (apetrich)01:10
openstackgerritMerged openstack/tripleo-common stable/rocky: Generating fencing for capitalized MACs  https://review.openstack.org/61972501:14
*** ansmith has quit IRC01:19
*** boazel_ has joined #tripleo01:19
*** boazel has quit IRC01:22
openstackgerritDaniel Farrell proposed openstack/tripleo-heat-templates master: Use new ODL diagstatus heath check REST URL  https://review.openstack.org/62105201:34
*** wolverineav has quit IRC01:38
*** ooolpbot has joined #tripleo02:10
ooolpbotURGENT TRIPLEO TASKS NEED ATTENTION02:10
ooolpbothttps://bugs.launchpad.net/tripleo/+bug/180587002:10
openstackLaunchpad bug 1805870 in tripleo "mistral unit tests failing in dlrn build of tripleo-common" [Critical,Triaged] - Assigned to Adriano Petrich (apetrich)02:10
*** ooolpbot has quit IRC02:10
*** karthiks has quit IRC02:17
*** mrsoul has quit IRC02:22
*** mschuppert has quit IRC02:22
*** mcornea has quit IRC02:23
*** eck` is now known as eck`gone02:24
*** psachin has joined #tripleo02:40
mwhahahawe had a timeout on undercloud isntall setting up mistral, http://logs.openstack.org/78/620978/1/gate/tripleo-ci-centos-7-undercloud-oooq/191bbd9/logs/undercloud/home/zuul/undercloud_install.log.txt.gz#_2018-11-30_01_02_5803:07
*** ooolpbot has joined #tripleo03:10
ooolpbotURGENT TRIPLEO TASKS NEED ATTENTION03:10
ooolpbothttps://bugs.launchpad.net/tripleo/+bug/180587003:10
openstackLaunchpad bug 1805870 in tripleo "mistral unit tests failing in dlrn build of tripleo-common" [Critical,Triaged] - Assigned to Adriano Petrich (apetrich)03:10
*** ooolpbot has quit IRC03:10
*** apetrich has quit IRC03:15
openstackgerritMerged openstack/diskimage-builder master: Revert "Make tripleo-buildimage-overcloud-full-centos-7 non-voting"  https://review.openstack.org/62020103:18
*** ykarel|away has joined #tripleo03:22
*** rlandy has quit IRC03:31
*** psachin has quit IRC03:43
openstackgerritMerged openstack/diskimage-builder master: Add missing ws separator between words  https://review.openstack.org/61916903:53
*** udesale has joined #tripleo04:01
*** janki has joined #tripleo04:03
*** ooolpbot has joined #tripleo04:10
ooolpbotURGENT TRIPLEO TASKS NEED ATTENTION04:10
ooolpbothttps://bugs.launchpad.net/tripleo/+bug/180587004:10
*** ooolpbot has quit IRC04:10
openstackLaunchpad bug 1805870 in tripleo "mistral unit tests failing in dlrn build of tripleo-common" [Critical,Triaged] - Assigned to Adriano Petrich (apetrich)04:10
openstackgerrityatin proposed openstack/tripleo-quickstart-extras master: Fix regression of collect-logs package listing  https://review.openstack.org/62092704:16
*** ramishra has joined #tripleo04:29
*** ratailor has joined #tripleo04:50
*** wolverineav has joined #tripleo04:52
*** ooolpbot has joined #tripleo05:10
ooolpbotURGENT TRIPLEO TASKS NEED ATTENTION05:10
ooolpbothttps://bugs.launchpad.net/tripleo/+bug/180587005:10
openstackLaunchpad bug 1805870 in tripleo "mistral unit tests failing in dlrn build of tripleo-common" [Critical,Triaged] - Assigned to Adriano Petrich (apetrich)05:10
*** ooolpbot has quit IRC05:10
mwhahahamarios_|ruck, ssbarnea|rover, weshay: fyi i just realized -> https://bugs.launchpad.net/tripleo/+bug/1805968  Probably need to fix that before all the scenario standalone jobs get turned on :D05:14
openstackLaunchpad bug 1805968 in tripleo "standalone job does not run yum update during the container image prepare" [Critical,Triaged]05:14
*** wolverineav has quit IRC05:15
openstackgerritMerged openstack/tripleo-docs master: Add BIOS settings doc  https://review.openstack.org/57734305:16
*** threestrands_ has joined #tripleo05:17
*** threestrands has quit IRC05:20
*** ykarel|away has quit IRC05:24
jaosoriorstevebaker: around?05:29
jaosoriormwhahaha: any progress on the gate issues?05:32
mwhahahajaosorior: some of my patches for scenarios non voting are in the gate now. It looks like queens upgrades are broken but that's not new05:34
jaosoriormwhahaha: is the broken queens upgrade affecting the gate too?05:34
mwhahahajaosorior: other than that, we've at least reduced the check backlog by clearing our gate05:34
mwhahahaI don't think so because I don't think anything has a +1 zuul since ipgrades have been broken for weeks05:35
*** pcaruana has quit IRC05:35
jaosoriormwhahaha: is there a bug for that?05:36
mwhahahaMostly we need to address the time outs but we'll need to take a look at which jobs are having a issues after the scenario patches land05:36
mwhahahaI think so, it's not an alert through (should be)05:36
mwhahahaI've seen it before but I don't have it handy05:36
mwhahahaI think it's failing on the missing policy.json from heat05:37
jaosoriorunderstood, I'll try to poke people to see if anybody knows what bug it is. And add an alert to it.05:37
*** hakhande_ has joined #tripleo05:37
mwhahahaI recall wanting to look into it previously but I always get side tracked05:37
jaosoriormwhahaha: missing policy.json?05:37
jaosoriorheat using isn't using policy in code yet?05:38
mwhahahaI think it is but something changed so the undercloud update fails on it05:38
jaosoriorOK05:38
*** ykarel|away has joined #tripleo05:39
mwhahahahttps://bugs.launchpad.net/tripleo/+bug/179663905:39
openstackLaunchpad bug 1796639 in tripleo "[stable/queens] The tripleo-ci-centos-7-undercloud-upgrades is failing on queens for No such file or directory: '/etc/heat/policy.json" [Critical,Triaged]05:39
*** ykarel|away is now known as ykarel05:39
jaosoriormwhahaha: thanks, added the alert05:40
* mwhahaha wanders off to bed. Back in ~9 hours05:41
openstackgerritMerged openstack/tripleo-heat-templates stable/queens: Switch scenarios to non-voting  https://review.openstack.org/62097805:41
jaosoriormwhahaha: have a good one05:41
openstackgerritRabi Mishra proposed openstack/tripleo-heat-templates master: WIP set uuid facter explicitly  https://review.openstack.org/62107205:53
ramishrajaosorior: Hey! Are you ok with https://review.openstack.org/#/c/614457/ ? shardy has reviewed it already.05:56
jaosoriorramishra: I have the same question as Steven here: https://review.openstack.org/#/c/614457/10/roles/Standalone.yaml05:58
jaosoriorshouldn't we only need the tenant network?05:58
jaosoriorgiven that ctlplane is created by default05:58
jaosoriorsame here https://review.openstack.org/#/c/614457/10/roles/Standalone.yaml05:59
ramishrajaosorior: Is that question shardy asked? We do need all networks and create the ports as the ServiceNetMap has all those networks specified05:59
jaosoriorramishra: "Ack Ok - perhaps we could optimize this in future to remove the other networks, e.g we probably only need the ctlplane, External and Tenant networks?"05:59
ramishrajaosorior: Nope that would not work unless we override ServiceNetMap06:00
jaosoriorI thought we already did06:00
jaosoriorbut we don't06:01
jaosoriorgot it06:01
jaosoriorthat could be left for another iteration06:01
jaosoriorI think we should though06:01
ramishrajaosorior: No the earlier solution was not accpetable06:01
jaosoriorramishra: why wasn't it?06:01
ramishrajaosorior: I mean the one I proposed earlier, then the one shardy suggested is possibly a hack and would break in the future06:02
ramishraIt's all there in the discussion in the review06:02
jaosoriorah yeah06:02
jaosoriorthis one https://review.openstack.org/#/c/615363/2/roles/standalone/templates/standalone_config.yaml.j206:02
jaosoriorI actually like that solution better x_x that's the interfaces that we created for such a thing06:02
jaosoriorramishra: but anyway, the patches you pointed me to is fine. I'll +2 it for now but won't merge it given the situation in the gate.06:03
ramishrathe issue is, once we overide ServiceNetMap, we've to override everything in the map again if we want to change anything06:03
ramishrathat's a valid concern from alex and steve06:04
ramishraso, we don't change anything for undercloud/standalone as there is no network isolation for them atm06:04
ramishrawe can revisit the way to do it later06:05
jaosoriorramishra: well... that's what we ask deployers to do if they wanna change the network. So, if that's not an acceptable way of changing them, then we need to come up with a better way.06:05
jaosoriorramishra: but anyway, I did +2 your patch.06:05
ramishrayeah opinions differ, but we need a common way forward;)06:05
*** ooolpbot has joined #tripleo06:10
ooolpbotURGENT TRIPLEO TASKS NEED ATTENTION06:10
ooolpbothttps://bugs.launchpad.net/tripleo/+bug/179663906:10
ooolpbothttps://bugs.launchpad.net/tripleo/+bug/180587006:10
*** ooolpbot has quit IRC06:10
openstackLaunchpad bug 1796639 in tripleo "[stable/queens] The tripleo-ci-centos-7-undercloud-upgrades is failing on queens for No such file or directory: '/etc/heat/policy.json" [Critical,Triaged]06:10
openstackLaunchpad bug 1805870 in tripleo "mistral unit tests failing in dlrn build of tripleo-common" [Critical,Triaged] - Assigned to Adriano Petrich (apetrich)06:10
ykareljaosorior, ovb jobs are broken after https://review.openstack.org/#/c/619519/106:10
ykarelmarios_|ruck, ssbarnea|rover ^^06:10
jaosoriorykarel: are you gonna submit a patch to fix it?06:11
ykareljaosorior, okk will submit, need bug?06:12
jaosoriorykarel: or should I give it a try?06:12
ykareljaosorior, ok u can fix, just need to remove the debug task06:12
jaosoriorykarel: a bug would be nice. So we can alert on it and folks can keep track of that.06:12
ykareljaosorior, okk submitting bug first06:13
ykareljaosorior, https://bugs.launchpad.net/tripleo/+bug/180597206:15
openstackLaunchpad bug 1805972 in tripleo "OVB jobs failing with: "registered_overcloud_nodes.stdout": "VARIABLE IS NOT DEFINED!"" [Undecided,New]06:15
openstackgerritJuan Antonio Osorio Robles proposed openstack/tripleo-quickstart master: Reintroduce get-overcloud-nodes tasks in tripleo-inventory role  https://review.openstack.org/62107506:16
jaosoriorykarel: ^^06:16
openstackgerritKartikeya Jain proposed openstack/diskimage-builder master: Adding support for SLES 15 in element 'sles'  https://review.openstack.org/61918606:17
ykareljaosorior, +106:19
ykareljaosorior, also added promotion-blocker tag as it's blocking promotion of all realeases06:20
jaosoriorykarel: are you sure that's the issue?06:20
jaosoriorI mean, it looks like it'll break things06:21
jaosoriorhowever, the error seems different06:21
jaosoriorhttps://ci.centos.org/job/tripleo-quickstart-gate-master-delorean-quick-basic/7943/console06:21
jaosoriorykarel: or do you have another log?06:21
ykareljaosorior, see failed jobs: https://trunk.rdoproject.org/api-centos-rocky/api/civotes_detail.html?commit_hash=e77cead2ef9f3669b9dbd65229f4c01e28d13615&distro_hash=689771707fb5a44fbeb78a001092af22266690cf06:21
jaosoriorykarel: got it, thanks06:22
ykareljaosorior, the link u shared have some other issue06:22
ykarelsome syntax problem06:22
jaosoriorykarel: that was a failure in the patch that introduced this bug06:23
ykareljaosorior, looks like that job is already broken, let me check in some other patch06:23
ykareljaosorior, so if u see the comments in https://review.openstack.org/#/c/619519/, u can find that job passed sometimes on same patchset06:26
jaosoriorstrange06:26
ykarelor see https://ci.centos.org/job/tripleo-quickstart-gate-master-delorean-quick-basic/ and job 61951906:27
*** jfrancoa has joined #tripleo06:31
*** flaper87 has quit IRC06:32
jaosoriormandre: let me know when you're around06:34
*** ksambor has joined #tripleo06:38
*** radeks has joined #tripleo06:49
*** radeks_ has joined #tripleo06:50
*** radeks has quit IRC06:53
jaosoriorhjensas: is this still an issue https://bugs.launchpad.net/tripleo/+bug/1800166 ?06:55
openstackLaunchpad bug 1800166 in tripleo "tripleo-container-image-prepare :: unrecognized arguments: --log-file" [High,Triaged]06:55
Tenguhello there :)06:55
*** mmagr has joined #tripleo07:02
*** quiquell|off is now known as quiquell07:03
jaosoriormandre: submitted this https://bugs.launchpad.net/tripleo/+bug/180597707:03
openstackLaunchpad bug 1805977 in tripleo "tripleo image prepare command can't run certain for the overcloud" [High,Triaged]07:03
*** mschuppert has joined #tripleo07:06
openstackgerritMichele Baldessari proposed openstack/puppet-pacemaker master: Remove unneeded spec testing file  https://review.openstack.org/61989207:09
openstackgerritCédric Jeanneret proposed openstack/tripleo-heat-templates master: Ensure we get dedicated logging file for HAProxy  https://review.openstack.org/62060107:09
openstackgerritMarios Andreou proposed openstack-infra/tripleo-ci master: WIP: Add tripleo-ci-centos-7-scenario004-standalone job  https://review.openstack.org/61952007:09
*** ooolpbot has joined #tripleo07:10
ooolpbotURGENT TRIPLEO TASKS NEED ATTENTION07:10
ooolpbothttps://bugs.launchpad.net/tripleo/+bug/179663907:10
ooolpbothttps://bugs.launchpad.net/tripleo/+bug/180587007:10
*** ooolpbot has quit IRC07:10
openstackLaunchpad bug 1796639 in tripleo "[stable/queens] The tripleo-ci-centos-7-undercloud-upgrades is failing on queens for No such file or directory: '/etc/heat/policy.json" [Critical,Triaged]07:10
openstackLaunchpad bug 1805870 in tripleo "mistral unit tests failing in dlrn build of tripleo-common" [Critical,Triaged] - Assigned to Adriano Petrich (apetrich)07:10
openstackgerritCédric Jeanneret proposed openstack/tripleo-heat-templates master: Ensure we get dedicated logging file for HAProxy  https://review.openstack.org/62060107:10
*** kopecmartin|off has quit IRC07:10
*** hamzy__ has joined #tripleo07:11
*** hamzy_ has quit IRC07:12
*** rbrady has quit IRC07:12
*** kopecmartin has joined #tripleo07:13
*** threestrands_ has quit IRC07:13
*** apetrich has joined #tripleo07:16
*** skramaja has joined #tripleo07:19
*** pcaruana has joined #tripleo07:22
*** radeks_ has quit IRC07:26
*** radeks_ has joined #tripleo07:26
openstackgerritCédric Jeanneret proposed openstack/tripleo-heat-templates master: Ensure we actually pass parameters to parent template  https://review.openstack.org/62110507:27
*** cylopez has joined #tripleo07:30
*** cylopez has left #tripleo07:30
*** slaweq has joined #tripleo07:35
hjensasjaosorior: no, we had new container images promoted since that so the specific issue is gone now. (The fact that we can end up with RPM repos promoted and then several days of delay before we promote containers is still a problem. But let's close that bug. :) )07:35
*** radeks__ has joined #tripleo07:35
*** radeks_ has quit IRC07:35
mariosreviews please if you can spare them https://review.openstack.org/#/c/619504/ we are trying to merge the env files so testing scenario standalones is easier thanks07:36
jaosoriormarios: can't approve things yet07:36
*** florianf|afk is now known as florianf07:37
mariosjaosorior: ack i thought we were clear now at least i saw patches restored by alex?07:37
Tengusome are in the gate, but apparently it's still really slow.07:37
mariosjaosorior: thanks i will hold then but if you vote i'll -1 it07:37
openstackgerritFilippo Inzaghi proposed openstack/python-tripleoclient master: Add Python 3.6 classifier to setup.cfg  https://review.openstack.org/62111207:37
mariosTengu: zuul queue looks much better grafana reckons 7 hours07:38
mariosTengu: but yeah still long07:38
Tenguyeah. was something like 40 yesterday.07:38
Tenguwe might need some new power. lately it's going crazy07:39
openstackgerritMerged openstack/tripleo-docs master: [Documentation] Specify the name of the irc server.  https://review.openstack.org/62088707:39
openstackgerritMike Fedosin proposed openstack/tripleo-heat-templates master: Allow to skip docker reconfiguration during stack update  https://review.openstack.org/62062107:40
*** cylopez has joined #tripleo07:41
cylopezjaosorior: hello07:41
jaosoriorcylopez: sup07:41
cylopezjaosorior: I'm look about https://review.openstack.org/#/c/620876/, I have 2 ubunutu bionic in failure, I have no idea why07:42
jaosoriorcylopez: the error seems unrelated07:43
cylopezjaosorior: oh great07:43
jaosoriorso, might be that the ubuntu gate is broken07:43
jaosoriorgotta talk to the folks at puppet-openstack :/07:43
cylopezjaosorior: okok, is a blocker to get it merge ?07:43
*** ykarel is now known as ykarel|lunch07:43
jaosorioryes07:44
cylopezdamned07:44
cylopezjaosorior: I should ask for a "recheck" in few days so ?07:44
jaosoriorprobably07:44
cylopezjaosorior: thx and have a nice friday07:45
jaosoriorprobablythanks!07:46
Tengu:][07:46
*** cfontain has joined #tripleo07:48
hjensascylopez: jaosorior: I saw bandini push a patch to revert to the previous version of ubuntu in puppet-pacemaker yesterday ...07:49
cylopezhjensas: good to know, thx07:50
* bandini looks07:56
bandinicylopez, hjensas: my bionic failures were very specific. Fixed via https://review.openstack.org/#/c/619892/ fwiw07:57
openstackgerritRedHat RDO CI proposed openstack/tripleo-heat-templates stable/queens: GATE CHECK for TripleO  https://review.openstack.org/56722408:00
*** jpena|off is now known as jpena08:01
*** akrivoka has joined #tripleo08:03
*** jtomasek has joined #tripleo08:06
*** rcernin has quit IRC08:06
*** ooolpbot has joined #tripleo08:10
ooolpbotURGENT TRIPLEO TASKS NEED ATTENTION08:10
ooolpbothttps://bugs.launchpad.net/tripleo/+bug/179663908:10
openstackLaunchpad bug 1796639 in tripleo "[stable/queens] The tripleo-ci-centos-7-undercloud-upgrades is failing on queens for No such file or directory: '/etc/heat/policy.json" [Critical,Triaged]08:10
ooolpbothttps://bugs.launchpad.net/tripleo/+bug/180587008:10
*** ooolpbot has quit IRC08:10
openstackLaunchpad bug 1805870 in tripleo "mistral unit tests failing in dlrn build of tripleo-common" [Critical,Triaged] - Assigned to Adriano Petrich (apetrich)08:10
openstackgerritHarald Jensås proposed openstack/tripleo-heat-templates master: Add MTU to neutron networks and nic-config templates  https://review.openstack.org/59959308:10
openstackgerritHarald Jensås proposed openstack/tripleo-heat-templates master: Use mtu of Tenant network to control neutron global_physnet_mtu  https://review.openstack.org/61253308:10
openstackgerritNatal Ngétal proposed openstack/tripleo-quickstart master: [Configuration] Remove mitaka file.  https://review.openstack.org/61767008:17
therveapetrich: I think https://bugs.launchpad.net/tripleo/+bug/1805870, the failures are legit, and not in master08:19
openstackLaunchpad bug 1805870 in tripleo "mistral unit tests failing in dlrn build of tripleo-common" [Critical,Triaged] - Assigned to Adriano Petrich (apetrich)08:19
therveWe expect CI to be broken so often that we don't recognize legit failures anymore :D08:21
Tenguroohhh08:22
Tenguthat's nasty ;)08:22
*** shardy has joined #tripleo08:23
openstackgerritThomas Herve proposed openstack/python-tripleoclient stable/rocky: Undercloud: Disable persisting VIPs in /etc/hosts  https://review.openstack.org/62065208:24
*** shardy has quit IRC08:24
*** shardy has joined #tripleo08:24
*** amoralej|off is now known as amoralej08:24
openstackgerritNatal Ngétal proposed openstack/tripleo-docs master: [Documentation] Replace newton by rocky.  https://review.openstack.org/62091008:26
Hobbestigrouhi08:29
Hobbestigrousomeone can review this patch please https://review.openstack.org/#/c/620910/ ?08:30
openstackgerritMarios Andreou proposed openstack-infra/tripleo-ci master: WIP: Add new tripleo-ci-centos-7-scenario001-standalone job  https://review.openstack.org/61950808:33
Tenguhmm. that said, I found "current-tripleo" to be more stable than "current". current-tripleo seems to point to the changes that actually passed the CI.08:33
mariosTengu: current is just master, or at least latest build from master. current-tripleo is things that have passed upstream promotion http://logs.openstack.org/87/616187/1/check/openstack-tox-docs/164c631/html/contributor/stages-overview.html08:36
mariosTengu: so your finding is correct08:37
marios:)08:37
Tengu:)08:37
TenguI updated my lab in order to use current-tripleo as master was broken.08:37
Tengunow I can play :)08:37
*** ykarel|lunch is now known as ykarel08:38
*** xek has joined #tripleo08:38
*** tosky has joined #tripleo08:42
openstackgerritJiri Tomasek proposed openstack/tripleo-common master: Fix environments storting  https://review.openstack.org/62112208:43
*** ccamacho has joined #tripleo08:46
*** salmankhan has joined #tripleo08:50
jtomasekapetrich: ^ feel free to take that over:)08:51
*** jpich has joined #tripleo08:53
*** salmankhan has quit IRC08:56
*** holser_ has joined #tripleo09:05
*** ooolpbot has joined #tripleo09:10
ooolpbotURGENT TRIPLEO TASKS NEED ATTENTION09:10
ooolpbothttps://bugs.launchpad.net/tripleo/+bug/179663909:10
*** ooolpbot has quit IRC09:10
openstackLaunchpad bug 1796639 in tripleo "[stable/queens] The tripleo-ci-centos-7-undercloud-upgrades is failing on queens for No such file or directory: '/etc/heat/policy.json" [Critical,Triaged]09:10
cylopezhjensas: hello have time for a bug on os-net-config09:10
*** jaganathan__ has left #tripleo09:10
*** bogdando has joined #tripleo09:12
hjensascylopez: yes, I can plan it for the next sprint.09:13
*** dsneddon has quit IRC09:19
apetrichtherve, my bad. I will check that. It was late and tested only master09:22
therveapetrich: No worries :)09:22
d0ugaltherve: the failures are not in master? where are they?09:29
therved0ugal: patches?09:29
d0ugaloh09:29
d0ugalso we have a bug about errors in patches in gerrit?09:29
therveYep09:29
d0ugalhaha, that is confusing.09:30
openstackgerritThomas Herve proposed openstack/tripleo-common master: Set heatclient exceptions to verbose  https://review.openstack.org/61903909:31
apetrichtherve, I have a fix for that patch, mind if I submit?09:32
*** derekh has joined #tripleo09:33
apetrichtherve, nevermind. Just saw your fix09:33
* apetrich is still slow today. more coffee needed09:34
*** gfidente has joined #tripleo09:40
*** yprokule has joined #tripleo09:43
openstackgerritThomas Herve proposed openstack/tripleo-common master: Set heatclient exceptions to verbose  https://review.openstack.org/61903909:43
*** hberaud has joined #tripleo09:45
Hobbestigrouplease someone can review this patch https://review.openstack.org/#/c/611596/ ?09:49
*** aedc has joined #tripleo09:51
bogdandofolks, WDYT of cronie in containers and https://bugzilla.redhat.com/show_bug.cgi?id=165465909:52
openstackbugzilla.redhat.com bug 1654659 in cronie "Optimize cronie RPM spec for systemd requirements when installing it in containers" [Unspecified,New] - Assigned to tmraz09:52
bogdandowe have it installed in many services, how actually it works?09:52
bogdandowould it work w/o systemd in containers?09:52
bogdandomwhahaha, jistr: any ideas?09:53
* jistr no idea on this topic09:53
bogdandoowalsh: perchance?09:55
bogdandohttps://bugs.launchpad.net/tripleo/+bug/1701254 was the original issue for that09:56
openstackLaunchpad bug 1701254 in tripleo "Containerize crond" [High,Fix released] - Assigned to Dan Prince (dan-prince)09:56
bogdandoI hope that works https://bugzilla.redhat.com/show_bug.cgi?id=1654659#c610:01
openstackbugzilla.redhat.com bug 1654659 in cronie "Optimize cronie RPM spec for systemd requirements when installing it in containers" [Unspecified,New] - Assigned to tmraz10:01
openstackgerritSaravanan KR proposed openstack/tripleo-heat-templates master: Process the templates even if j2_excludes file is not present  https://review.openstack.org/62112910:02
bogdandofolks, PTAL https://review.openstack.org/#/c/620281/10:05
openstackgerritMarios Andreou proposed openstack/tripleo-quickstart-extras master: Add export ANSIBLE_LIBRARY to linting so we can find modules  https://review.openstack.org/62055610:08
*** ooolpbot has joined #tripleo10:10
ooolpbotURGENT TRIPLEO TASKS NEED ATTENTION10:10
ooolpbothttps://bugs.launchpad.net/tripleo/+bug/179663910:10
*** ooolpbot has quit IRC10:10
openstackLaunchpad bug 1796639 in tripleo "[stable/queens] The tripleo-ci-centos-7-undercloud-upgrades is failing on queens for No such file or directory: '/etc/heat/policy.json" [Critical,Triaged]10:10
openstackgerritSaravanan KR proposed openstack/tripleo-heat-templates master: Deprecate duplicate NFV environment files  https://review.openstack.org/62113310:18
openstackgerritRabi Mishra proposed openstack/tripleo-heat-templates master: Set facter variable 'uuid' explicitly in docker-puppet.py  https://review.openstack.org/62107210:19
*** akrivoka has quit IRC10:26
*** akrivoka has joined #tripleo10:27
*** slaweq has quit IRC10:32
*** jfrancoa has quit IRC10:34
*** dsneddon has joined #tripleo10:37
*** moguimar has joined #tripleo10:40
*** ramishra has quit IRC10:43
*** ramishra has joined #tripleo10:43
*** pcaruana has quit IRC10:44
thervehjensas: Hi, around? Wanted to chat about https://review.openstack.org/#/c/599593/10:48
hjensastherve: yes, I'm here.10:49
thervehjensas: I'd like to reduce the amount of duplication in there10:49
thervehjensas: Is that required to define the parameters everywhere?10:49
*** pcaruana has joined #tripleo10:50
hjensastherve: yeas, since we pass the parameter to the child stack we need to define the parameters. Otherwize Heat complain that we pass it an "unknown". It's like 'ls --foo /tmp' says "unrecognized option foo".10:51
hjensastherve: it's a pita. So if there is a way to get around it I'm all ears.10:52
therveLet's try to understand10:52
thervehjensas: Where are those values actually used?10:52
*** jfrancoa has joined #tripleo10:53
hjensastherve: in all the templates under network/config/10:53
hjensastherve: and here network/network.j210:54
therveOK10:54
thervehjensas: Like when you build MinViableMtu?10:54
hjensasnetwork/ports/port.j2 etc. So basically anything that is'nt static stuff used for CI.10:55
thervehjensas: Wouldn't it be possible to pass a hash map instead of individual names?10:56
thervemtus: {"ctlplane": 1500, "storage": 1500}, etc10:56
hjensastherve: so the biggest reason for the duplication is in https://review.openstack.org/#/c/599593/18/puppet/role.role.j2.yaml10:58
therveRight you pass explicitely the names10:58
hjensastherve: a hash map would work, but it would diverge from what we do for other things, like {{network.name}}IpSubnet, {{network.name}}InterfaceRoutes.10:59
*** udesale has quit IRC10:59
thervehjensas: Aren't those horrible to use too though?10:59
*** dtantsur|mtg is now known as dtantsur|afk11:00
therveLike ci/common/net-config-multinode-os-net-config.yaml has 28 parameters, it uses 3 of them11:01
hjensastherve: yeah, it's kind of crazy.11:02
*** iurygregory is now known as iury|lunch11:03
hjensastherve: but, for an actual user it's https://review.openstack.org/#/c/599593/18/network_data.yaml that need to change. Then nic-config templates can be rendered based on roles data. (as documented in https://review.openstack.org/#/c/596188/7/doc/source/install/advanced_deployment/network_isolation.rst)11:04
thervehjensas: Sure. It still has an impact though11:05
therveFirst it's basically impossible to review11:05
therveThere are 30 interesting lines hidden in 140011:05
*** hberaud is now known as hberaud|lunch11:05
therveThen it has a performance impact on Heat11:05
therveWe keep duplicating structure that we don't need, but Heat spends useless cycles processing those11:06
thervehjensas: That the very least, the parameters section should be jinja'ed11:07
therveIt's the very same for every single config, right?11:07
openstackgerritFlorian Fuchs proposed openstack/tripleo-validations master: Add a lookup plugin to get mistral workflow results  https://review.openstack.org/57512111:07
hjensastherve: yes, I totally get your point. I like the idea of using a hash map instead.11:07
therveIt's not perfect because we lose some validation11:08
therveBut I think it works here11:08
hjensastherve: I did'nt think adding jinja to the static CI content would be an option. But that would help loads when developing this stuff.11:09
owalshbogdando: hey, what was the Q?11:09
thervehjensas: Yeah it's possible we can't do that currently11:09
hjensastherve: I wonder how we could introduce a hash map and still be compatible with old templates.11:09
*** rfolco is now known as rfolco_doctor11:09
thervehjensas: Well, we can for introducing that one at least11:10
*** morazi has quit IRC11:10
*** ooolpbot has joined #tripleo11:10
ooolpbotURGENT TRIPLEO TASKS NEED ATTENTION11:10
ooolpbothttps://bugs.launchpad.net/tripleo/+bug/179663911:10
openstackLaunchpad bug 1796639 in tripleo "[stable/queens] The tripleo-ci-centos-7-undercloud-upgrades is failing on queens for No such file or directory: '/etc/heat/policy.json" [Critical,Triaged]11:10
*** ooolpbot has quit IRC11:10
therveNow for removing the existing ones, it's worth testing11:10
therveI can also look at changing heat so it some cases it doesn't complain when you pass an unknown parameter11:12
hjensastherve: Could we make heat accept missing parameters if they are not referenced by any resource?11:12
thervehjensas: Not sure I get that one11:12
therveYou mean accept parameters that aren't defined?11:13
*** dmacpher has joined #tripleo11:13
hjensastherve: I.e we pass parameter 'foo' and bar to the template, but only 'foo' is defined in parameters, and 'bar' is neveren referenced in the resources section. i.e no {get_param: bar}11:14
thervehjensas: Right. Yeah that's worth investigating.11:14
therveIt removes a source of validation, but I don't know how useful it is in practice11:14
hjensastherve: then again, validation wize it's kind of good to error on that situation.11:15
thervehjensas: Well, what matters is to fail when you do get_param on something not defined11:15
owalshbogdando: looks like you've figured it out anyway - rpm deps pulling in systemd, yea?11:16
thervehjensas: Passing something not used... It's bad when there is a typo, and you have a default on you parameter11:16
bogdandoowalsh: yea11:16
hjensastherve: yes, those things would be painful to troubleshoot.11:16
thervehjensas: OK, thanks for the dicussion, some food for thoughts :)11:17
* therve lunch11:17
hjensastherve: it would be cool if we could use a hash map, and then put custom constraint in each parameter in the template to ensure what is required is defined in the map.11:18
hjensastherve: that way we would'nt have to duplicate, but still we could raise validation error when appropriate.11:19
openstackgerritMarios Andreou proposed openstack/tripleo-quickstart-extras master: Add export ANSIBLE_LIBRARY to linting so we can find modules  https://review.openstack.org/62055611:20
*** ramishra has quit IRC11:20
openstackgerritMarios Andreou proposed openstack/tripleo-quickstart-extras master: Add export ANSIBLE_LIBRARY to linting so we can find modules  https://review.openstack.org/62055611:22
*** ramishra has joined #tripleo11:26
toskybogdando: about the RDO patches, wouldn't it be possible to use the condition already used in the spec file? %if 0%(?fedora} || 0%{?rhel} > 711:31
*** raildo has joined #tripleo11:32
*** holser_ is now known as holser|lunch11:33
*** iury|lunch is now known as iurygregory11:36
Hobbestigrouhi11:39
*** cfontain has quit IRC11:39
Hobbestigrouhttps://bugs.launchpad.net/tripleo/+bug/1652227/comments/411:39
openstackLaunchpad bug 1652227 in tripleo "[tripleo-incubator] "tox -e docs" fail" [Low,Triaged]11:39
Hobbestigrouquick question11:40
Hobbestigroui can change the status to won't fix ?11:40
openstackgerritQuique Llorente proposed openstack/tripleo-quickstart-extras master: Install correct python version packages at tempest  https://review.openstack.org/61872111:47
openstackgerritQuique Llorente proposed openstack-infra/tripleo-ci master: Enable tempest for fedora28 standalone  https://review.openstack.org/61805611:49
*** slaweq has joined #tripleo11:51
*** pbandark has joined #tripleo11:54
*** hberaud|lunch is now known as hberaud12:04
openstackgerritJanki Chhatbar proposed openstack/tripleo-heat-templates master: Don't mount data folder  https://review.openstack.org/62005312:10
*** ooolpbot has joined #tripleo12:10
ooolpbotURGENT TRIPLEO TASKS NEED ATTENTION12:10
ooolpbothttps://bugs.launchpad.net/tripleo/+bug/179663912:10
*** ooolpbot has quit IRC12:10
openstackLaunchpad bug 1796639 in tripleo "[stable/queens] The tripleo-ci-centos-7-undercloud-upgrades is failing on queens for No such file or directory: '/etc/heat/policy.json" [Critical,Triaged]12:10
openstackgerritRabi Mishra proposed openstack/tripleo-heat-templates master: Set facter variable 'uuid' explicitly in docker-puppet.py  https://review.openstack.org/62107212:10
*** dprince has joined #tripleo12:11
*** pcaruana has quit IRC12:11
openstackgerritGael Chamoulaud proposed openstack/tripleo-validations master: Avoid 'non-zero return code' msg when ceph-ansible isn't installed  https://review.openstack.org/62114812:12
openstackgerritKamil Sambor proposed openstack/tripleo-heat-templates master: Check and enable openvswitch package  https://review.openstack.org/61903112:13
dprincebogdando: Hi, I'm having a lot of trouble following the logic of all of our patches. Would a spec that describes explicitly what you plan on doing make sense?12:15
ssbarnea|rovermarios: what is http://logs.openstack.org/57/620457/3/check/heat-functional-orig-mysql-lbaasv2/9ed4f0e/logs/devstacklog.txt.gz ?12:15
dprincebogdando: in light of the recent mailing list discussions I'm not sure why you would still have this posted: https://review.openstack.org/#/c/620281/12:16
openstackgerritDan Radez proposed openstack/puppet-tripleo stable/queens: Add manifest for networking_ansible ML2 plugin  https://review.openstack.org/62114912:17
ssbarnea|rovermarios: ^ that one is on one fixing tickets, so unless we sort it it, nothing will merge.12:17
ssbarnea|roverohh, damn looks like flakiness.12:18
ssbarnea|roveri would put a query on it but I don't know what string to look for.12:18
*** slaweq has quit IRC12:24
*** udesale has joined #tripleo12:26
fultonjmarios: how's the scen001 multi to standalone conversion going?12:27
fultonjmarios: i think we were going to talk about that tuesday, but ran out of meeting time but wondered if there was anything you wanted input on from gfidente and me12:28
mariosfultonj: o/ we are almost there. latest issue we had was module path for the linting fixed with Depends-On: https://review.openstack.org/620556 # t-q-e12:28
mariosfultonj: the deployment is at least passing so we are almost at a green job12:29
mariosfultonj: gfidente sorry about the call, i was hoping it would be more useful to you12:29
mariosfultonj: mostly was interested in any input with the featureset_override we were discussing, as users of the job12:29
fultonjmarios: np, usually the call is.12:29
gfidentemarios I am not super expert of that anyway12:30
fultonjoh, i think i like featureset_override12:30
fultonjthough, i'm not sure i understand all of the implications12:30
mariosgfidente: fultonj can you folks sanity check the env files please scen1 /4 wrt ceph if you haven't already thanks https://review.openstack.org/#/c/619508/ https://review.openstack.org/619520 (depends on has the tht  env file)12:33
mariosgfidente: fultonj we quiquell and i agreed to start merging the envs at least12:34
*** rfolco_doctor is now known as rfolco12:34
fultonjmarios: i'm looking for the logs collected from the new scen00112:35
mariosgfidente: fultonj scen1 is voted already (but check if you see nits please and 4 still needs vote @ https://review.openstack.org/#/c/619504/ )12:35
mariosfultonj: thanks12:35
mariosfultonj: so they are both currently waiting for check after i updated the depends-on (the path thing)12:36
mariosfultonj: i mean for latest run. but lemme find you a 'good' one sec12:36
mariosfultonj: here is one with deployment passing http://logs.openstack.org/08/619508/7/check/tripleo-ci-centos-7-scenario001-standalone/08c9ed2/logs/undercloud/home/zuul/standalone_deploy.log.txt.gz12:38
fultonjmarios: tks12:38
mariosfultonj: thanks for checking. might be worth looking later to se the latest run once it happens if u have time thank you12:39
*** holser|lunch is now known as holser_12:39
openstackgerritDan Prince proposed openstack/tripleo-heat-templates master: Add DockerPuppetMountHostPuppet parameter  https://review.openstack.org/55084812:40
*** jpena is now known as jpena|lunch12:41
*** xek has quit IRC12:46
openstackgerritKamil Sambor proposed openstack/tripleo-heat-templates master: Check and enable openvswitch package  https://review.openstack.org/61903112:46
*** ansmith has joined #tripleo12:46
*** xek has joined #tripleo12:47
*** hakhande_ has quit IRC12:47
*** morazi has joined #tripleo12:49
*** iurygregory is now known as iurygregory|mtg12:59
*** zul has joined #tripleo13:06
*** vpickard_ is now known as vpickard13:07
*** salmankhan has joined #tripleo13:09
*** ooolpbot has joined #tripleo13:10
ooolpbotURGENT TRIPLEO TASKS NEED ATTENTION13:10
ooolpbothttps://bugs.launchpad.net/tripleo/+bug/179663913:10
*** ooolpbot has quit IRC13:10
openstackLaunchpad bug 1796639 in tripleo "[stable/queens] The tripleo-ci-centos-7-undercloud-upgrades is failing on queens for No such file or directory: '/etc/heat/policy.json" [Critical,Triaged]13:10
*** ratailor has quit IRC13:11
*** abishop has joined #tripleo13:13
*** trown|outtypewww has quit IRC13:17
*** trown|brb has joined #tripleo13:18
openstackgerritQuique Llorente proposed openstack/tripleo-quickstart-extras master: Install correct python version packages at tempest  https://review.openstack.org/61872113:18
openstackgerritCédric Jeanneret proposed openstack/paunch master: Correct error output  https://review.openstack.org/62115713:18
openstackgerritHervé Beraud proposed openstack/tripleo-heat-templates stable/rocky: Remove unnecessary logs due to misconfiguration from pacemaker  https://review.openstack.org/62115913:25
*** iurygregory|mtg is now known as iurygregory13:29
cylopezjaosorior: it's green now https://review.openstack.org/#/c/620876/13:30
cylopez:-)13:30
*** skramaja has quit IRC13:33
openstackgerritHervé Beraud proposed openstack/tripleo-heat-templates stable/queens: Remove unnecessary logs due to misconfiguration from pacemaker  https://review.openstack.org/61305413:35
*** rlandy has joined #tripleo13:36
*** jpena|lunch is now known as jpena13:40
pbandarkhi.. on undercloud node, after introspecting the nodes, when I execute `openstack hypervisor list` command, I can see undercloud IP for all available nodes in "Host IP" column. Is it expected? https://pastebin.com/XxXm6nDH13:43
*** vinaykns has joined #tripleo13:45
openstackgerritMerged openstack/diskimage-builder master: Add an element to configure iBFT network interfaces  https://review.openstack.org/39178713:46
*** mmagr has quit IRC13:46
*** udesale has quit IRC13:48
jaosoriormarios: at a quick glance, what do you think of this docs commit https://review.openstack.org/#/c/620921/ ?13:49
*** udesale has joined #tripleo13:49
*** dmacpher has quit IRC13:51
*** eck`gone is now known as eck`13:52
openstackgerritRabi Mishra proposed openstack/tripleo-heat-templates master: Set facter variable 'uuid' explicitly in docker-puppet.py  https://review.openstack.org/62107213:53
*** slaweq has joined #tripleo13:56
*** jcoufal has joined #tripleo13:57
*** EmilienM is now known as EvilienM13:58
Tengubandini: hmm, you didn't set a BJ session for the meeting about the podman/pcmk rpms, did you?14:01
bandiniTengu: I added it later :)14:04
*** amoralej is now known as amoralej|lunch14:04
bandiniTengu: https://redhat.bluejeans.com/4884650097/?src=meet_now14:05
*** tzumainn has joined #tripleo14:05
apetrichjtomasek, just to make sure I'm doing the correct thing. I'm looking at this https://review.openstack.org/#/c/621122/14:07
dprincebogdando: Trying to refine the idea here (constructively) but I'm having a hard time following some of your comments https://review.openstack.org/#/c/620909. Can you explain exactly which part you are -2'ing14:07
*** ksambor has quit IRC14:07
*** ykarel is now known as ykarel|away14:07
dprincebogdando: fwiw, I think a spec is a better place to refine an idea sometimes than the mailing list14:08
bogdandodprince: the blocker is "a no go as is wrt having puppet executed 1 time centrally..."14:08
fultonjgfidente: when you get a chance would you please reivew https://review.openstack.org/#/q/topic:bug/1800232 ?14:08
bogdandowhich is expanded in comments14:08
bogdandosee from hjensas , dprince14:09
bogdandoThere is a bunch of bootstrap-host based coordination tasks like creating things with openstack services puppet providers and doing db-sync et al tasks. So the approach need to be updated like once for configs plus once for bootstrap tasks of each a service.14:09
bogdandowhile the former should be done at undercloud, 1 time onlym indeed, the latter would require 1 per service run for control plane14:10
dprincebogdando: I'm aware of that. I created boostrap-host-exec14:10
bogdandoat least 1 run14:10
*** ooolpbot has joined #tripleo14:10
ooolpbotURGENT TRIPLEO TASKS NEED ATTENTION14:10
ooolpbothttps://bugs.launchpad.net/tripleo/+bug/179663914:10
*** ooolpbot has quit IRC14:10
openstackLaunchpad bug 1796639 in tripleo "[stable/queens] The tripleo-ci-centos-7-undercloud-upgrades is failing on queens for No such file or directory: '/etc/heat/policy.json" [Critical,Triaged]14:10
mariosjaosorior: in a call will check in a bit14:10
dprincebogdando: I think we can solve that problem differently perhaps.14:10
bogdandonot sure the spec explains that clearly, so I set a blocker flag14:10
dprincebogdando: so you are -2 because the spec needs to refine the idea more then? A bit harsh I think14:11
mariosjaosorior: makes sense but want to look more closely/not during a call :)14:11
mariosjaosorior: will add for next reviews14:11
bogdandowell, also as it looked like a no go to change what we have14:11
bogdandoas we'd still need to run puppet many times14:11
jtomasekapetrich: yes14:11
*** hjensas has quit IRC14:12
dprincebogdando: the idea is to minimize where puppet has to run. Even if we ran it still on the bootstrap nodes that would be a lesser offense. And I think even that could be improved too in time14:12
dprincebogdando: step at a time... but in the right direction14:12
bogdandook, if we can solve this, I'll remove the blocker, just update the spec please then14:13
bogdandoI thought we cant, rbh14:13
*** toure|biab is now known as toure14:13
bogdandojust wondering if would have to run it on computes still14:13
bogdandoe.g. for DVR/OVN14:13
bogdandothen we have a blocker again14:14
dprincebogdando: I was trying to propose a solutio to your problem on the list... some of these details can be worked out I think14:15
dprincebogdando: etcd is perhaps my favorite solution, but would likely involve a larger development effort14:15
jaosoriormarios: thanks14:16
dprincebogdando: this was sort of a hybrid, a step towards optimizing our configs without breaking what we support today14:16
beaglesbogdando, dprince: probably an issue wherever there are role specific parameters14:16
dprinceNot breaking upgrades, and supporting all the config options has always been the tricky part of our config management tooling14:17
beagles(i.e. on where puppet config stuff needs to run)14:17
dprincebeagles: perhaps a role specific "overlay" could be implemented14:17
beaglesinteresting idea14:18
dprincebeagles: I think we probably want to move towards the concept of pods/services and away from roles a bit in doing this.14:18
beaglesdprince, ah right14:19
dprincebeagles: so perhaps we'd actually make more unique services which configured things slightly differently14:19
bogdandodprince: running puppet multiple times is rather an orthogonal problem to having it in each layer14:19
bogdandobut they a causal related indeed )14:19
bogdandoin each container* via base layer14:19
dprincebogdando: As it is today where it exists in the base layer is the best place for it. I've been convinced of that for quite a while. Do note when we implemented docker-puppet.py we tried putting in in a few places in the base made the most sense14:21
bogdandodprince: I'd rather agree with that jistr evaluates14:21
bogdandoa) detach systemd off those, b) move puppet and manifests off the base to upper14:22
cschwedeHi! Can someone please review + approve this tiny fix: https://review.openstack.org/#/c/620650/ ? That would be great!14:22
dprincebogdando: these problems are related... because how you generate configs infuences where the tooling needs to be in the container layers14:22
*** ksambor has joined #tripleo14:22
bogdandothis solves all of the security concerns by the price of not much traffic increased14:22
dprincebogdando: I didn't read jistr's comments the same way. I think you misunderstood something14:22
bogdandobut that is only needed if we would not have that spec implemented that you're proposing, dprince, ofc14:22
bogdandootherwise, a golden config image stored only at UC is a way to go14:23
bogdandobtw, my latest comment was for exatly that14:23
bogdandocoult you please  clarify that also in the spec?14:24
*** janki has left #tripleo14:24
*** mcornea has joined #tripleo14:24
*** janki has joined #tripleo14:24
jistrbogdando, dprince: yea i was advocating *against* doing the layer restructuring because of the excessive bandwidth/space costs. FWIW i didn't get to read dprince's spec yet so i don't have a complete picture of the landscape right now, will try to catch up though.14:24
dprincebogdando: moving puppet out of the base image isn't something we should do until we have a better mechanism for generating config files. If you intend to do this you misunderstood something14:24
bogdandodprince: why? he evaluated and declined, while agree with the former only :)14:24
dprincebogdando: my spec, was an idea which would enable us to move things around14:25
jaosorior:qa14:25
bogdandoI agreed with an approach is feasible, had we detached systemd14:25
bogdandodprince: moving puppet out of the base image is that we can do right now14:26
dprincebogdando: minimizing dependencies in the base image is great. But approach it as a packaging issue for now. Not a layering issue I think14:26
bogdandoand change nothing but the runtime vs config tag14:26
bogdandobut the size / traffic impact would be high14:26
bogdandoif we removed systemd , it'd much better14:26
bogdandoand feasible IMO14:26
dprincebogdando: if you do it you will actually cost more storage/network traffic. I don't even need to run any tests to tell you that14:26
dprincebogdando: it defeats your purposes14:26
*** rbrady has joined #tripleo14:26
bogdandodprince: not that much if systemd goes away14:27
bogdandoa decent price for improved security14:27
dprincebogdando: "improved security". you keep going back to that. We don't have numbers for this yet do we?14:27
jistrdprince, bogdando: yea that was also one of my points in the first e-mail i sent to the thread. How serious/real is the security concern here.14:28
beaglescschwede, reviewed but we approvals are being held back until the gate gets in proper shape ( see channel status )14:28
cschwedebeagles: thx a lot!14:28
dprincebogdando: while I logically like the idea of moving puppet out of the base image, there is a storage/network cost to doing so. We know that. It isn't a guess14:29
bogdandodprince, jistr: I have no numbers neither do I know from where to get those, sorry. And I think the last a couple of releases, we have architecture choices dictated not really any numbers driven14:30
dprincebogdando: the cost is actually quite high. Several gigs of data I think14:30
bogdandosecurity concerns is never real until a user is hit with it, then it becomes very serios14:30
*** janki has quit IRC14:30
bogdandothat is why they created the defences in depth principle, to have things improved w/o getting any numbers14:31
dprincebogdando: we can make that decision I guess. If the team wants to have "logically correct" containers we can do that today. I don't think it is a good idea however. Leaving PUppet in the base image is a small cost for the network/storage benefits we know that exist14:31
bogdandodprince: sadly, systemd won't be decoupled until fedora29 I think14:33
dprincebogdando: what we can do, is refactor how are config files are generated. Consider things future things like etcd (again), and perhaps even smaller things like adjusting how/where we run puppet. Doing this will allow us to restructure the container layers to be more logical14:33
bogdandoso we'd have to postpone14:33
bogdandobut would be nice to do something with the spec clarified for all of the aforementioned places14:33
bogdandoand give it a try with a golden image14:33
*** ykarel|away has quit IRC14:33
bogdandoif the latter is never supposed to be pushed overcloud and/or used for running anything14:33
dprincebogdando: you can try the golden image today I think by setting all of the Docker*Image params to the same image. It might just work14:34
jistrdprince, bogdando: one thing i'm thinking from this conversation -- no improvement on this front can be realistically made until we decouple systemd from puppet. And after we do so, isn't the main security concern solved by that, even without the restructuring? I mean how dangerous is a puppet in a container, if i have bash in that container too? They won't give you much differing capabilities, will they?14:34
dprincejistr: I think the systemd decoupling problem stands on its own. We should do it regardless14:35
bogdandodprince: but trying a golden image buys nothing w/o a required change to remove config bits off runtime images14:35
bogdandoand we could see that doing only either of it is rather a counterproductive14:36
bogdandowe'd need both14:36
dprincejistr: but yes, solving that problem I think does resolve a good bit of the potential security concerns14:36
jistrdprince, bogdando: yea. So IMO let's focus on fixing the packaging, dropping systemd. And then we can follow up whether it's still worth to address this further, no?14:36
dprincebogdando: we've covered many different topics and I feel like you keep switching back and forth on them14:37
dprincebogdando: the reason the all-in-one (golden) image was suggested was a means to minimize network/storage traffic to your Edge nodes. That was a single topic, unrelated to other onces14:37
dprinceones14:37
bogdandodprince: yes, that's what the spec positions for, and I'm totally for it, if we could avoid run puppets for compute roles14:37
jistrbogdando: WDYT about focusing on the systemd drop for now? I think that adds a ton of value for all the possible follow up scenarios.14:38
dprincejistr: I'm in agreement with the systemd packaging fixes. +2 on all that.14:38
bogdandojistr: so we'd have to remove bash, ruby, puppet and it would be much better ;)14:38
bogdandoI had to fix containers for the missing perl, (perl!) to awk14:39
dprincebogdando: you can't remove Puppet until we address the other issues.14:39
bogdandoand now we want to keep python and ruby there :D14:39
bogdandoplease don't14:39
jistrif we drop python we need to drop openstack too :D14:40
bogdandojistr: so my take is, yes, it still would be a security issue even w/o systemd14:40
bogdandobut I better off stop eating jaosorior's cake now :D14:40
jaosoriorbogdando: security is for everyone, not just the security team ;)14:41
bogdandojistr: yes, I have some patches in work for decoupling systemd14:41
Tengua step by step approach is always the best. first the obvious things, then dig deeper in the meat.14:42
jistrbogdando: cool :) Personally i think dropping systemd from all the images is where we get the most value per amount of work/trouble.14:42
bogdandodprince: I consider all-in-one *config* image only, for UC only. And I'm against using it for far edges as a silver bullet for *runtime*14:42
bogdandoif we'd still need that image to config control plane, that's ok14:43
bogdandonot for runtime please14:43
dprincebogdando: it was a suggestion based on your LP ticket. Use it or not14:43
dprincebogdando: given the architecture today, my guess is that gives you the smallest network traffic and storage requirements to your edge nodes.14:43
bogdandoand that is totally related to removing any of the config bits off other containers, dprince14:43
bogdandojistr: http://lists.openstack.org/pipermail/openstack-discuss/2018-November/000393.html14:44
dprincebogdando: depending on the solution they might be related things. But there is an order of operations. By choosing to move PUppet out of the base image first it would actually make many of the problems (storage/network) much worse for edge clouds I think14:45
openstackgerritMerged openstack/puppet-pacemaker master: Remove unneeded spec testing file  https://review.openstack.org/61989214:45
dprincebogdando: all that because some of us *think* a puppet/ruby security update might be a problem. This isn't something we should do right now14:46
bogdandodprince: yes, we can't make this to puppet first. 1) decouple systemd, 2) the spec of yours with a golden image and etcd, 3) remove all config bits off runtime containers images14:46
bogdandothat's how I see it, jistr^^14:47
bogdandobut there is also a discussible 1.5)14:47
bogdando:)14:47
bogdandowhich is move puppet and modules on top of the layers14:47
bogdandoand improve security right now14:47
dprincebogdando: your last statement is close. (I almost agree with it). THe bit about the golden image however is an optional thing for the spec. But we can perhaps clarify that elsewhere14:48
bogdandowith a very little overhead of a 80MB layer14:48
dprincebogdando: golden image to me is an optimization that would be opt-in perhaps. Not always useful.... but perhaps useful in edge clouds14:48
bogdandodprince: we're understanding it differently still, dprince14:50
bogdandoI'm against using it for edge, runtime14:50
bogdandoand I'm failing to see how we could remove all config-time bits off containers w/o that *config* only a golden image14:50
bogdandonot an opt-in to me, but a mandatory part of the spec to add it there14:51
jistrbogdando: re (2) and (3), i still need to read the dprince's spec. I'm +2 on (1). For now i'm -1 on the discussible (1.5). Regarding (1.5) i think we've discussed it long enough, there's no agreement, and we can't do it anyway until (1) happens. So let's drop discussion about (1.5) for now and focus on (1) which we seem to all agree with. My 2 cents :)14:51
*** brault has quit IRC14:51
*** ykarel|away has joined #tripleo14:52
*** ykarel|away is now known as ykarel14:52
bogdandothe all-in-one runtime vs config in the spec scope is the most that concerns me, jistr14:53
bogdandoplease do not attempt to push an idea for golden image for a runtime folks14:53
bogdandoa no go14:53
dprincebogdando: I'm not pushing golden images, it was simply an idea. But do note that my spec does not require it14:55
bogdandodprince: could you elaborate?14:56
bogdando1) we have to run puppet for a CP that fits all those db manage14:56
bogdando2) we aiming to remove all config bits off images14:56
dprincejistr: agree that we have spent way to much time on this. the reason I keep talking about is there are still patches posted which are relayering some of our containers.... some with +2's on them14:56
bogdando3) so how would we do that w/o a config-time golden image?14:56
bogdandoplease explain in the spec, dprince14:57
* jistr looks for the patches14:57
bogdandoit's very confusing to get it otherwise, sorry14:57
dprincebogdando: yes, this is why I created the spec. Lets clarify it there14:58
bogdandook, great. thanks folks14:58
jistrwe shouldn't have +2s on the relayering right now, given there's no general agreement14:58
bogdandowe have nothing left WIP for relayering jistr14:58
bogdandowhat are those +2s you're talking about?14:59
dprincebogdando: sorry, the kolla image from this morning. Did you remove that?14:59
bogdandoyes14:59
bogdandoand that was a leftover14:59
bogdandothe patch was about another thing, changing FROM, dprince14:59
jistrok sounds like we're ok for now then :)14:59
bogdandoand I still think we should not use openstack-base for tripleoclient image15:00
bogdandobut that's unrelated to puppet15:00
bogdandodprince: ^^15:00
dprincebogdando: that is perhaps another thing, but we can debate that separately. The Puppet was what confused me15:00
bogdandomy point is if it works only from the openstack-base, we have dependencies issues for trilpeoclient15:01
bogdandoas it should be working jsut fine off the centos base15:01
bogdandoand install anything it wants via depencencies15:01
bogdandoand not pull in the openstack-base world15:01
dprincebogdando: that may be, but why not share a base image when you can?15:01
bogdandook, I'll revive the patch then, but removing puppet leftovers15:01
dprincebogdando: again, this goes towards layering.15:01
bogdandodprince: well, to end up with a smaller image?15:02
bogdandotripleo deps << openstack world15:02
bogdando(much less than)15:02
bogdandodprince: yeah, lightweight containers, think of it like that, not just layering :)15:05
bogdandoa huge topic indeed15:05
*** ooolpbot has joined #tripleo15:10
ooolpbotURGENT TRIPLEO TASKS NEED ATTENTION15:10
ooolpbothttps://bugs.launchpad.net/tripleo/+bug/179663915:10
*** ooolpbot has quit IRC15:10
openstackLaunchpad bug 1796639 in tripleo "[stable/queens] The tripleo-ci-centos-7-undercloud-upgrades is failing on queens for No such file or directory: '/etc/heat/policy.json" [Critical,Triaged]15:10
mwhahahaykarel: do you have a patch for a partial revert of https://review.openstack.org/#/c/619519/1/roles/tripleo-inventory/tasks/inventory.yml or do you want us to revert that patch15:10
ykarelmwhahaha, already have a patch https://review.openstack.org/#/c/621075/15:11
mwhahahak15:11
ykarelcan't get why the bug is not in alert?15:12
weshaymwhahaha, we can't have nice things15:12
mwhahahathat's ok, i don't like nice things anyway15:13
weshaylolz15:13
*** chandan_kumar is now known as chkumar|off15:14
mwhahahaweshay: so given that i cleared out the gate yesterday and it's already back 11+ hours, what failed in the gate15:15
*** jcoufal has quit IRC15:16
ssbarnea|rovermwhahaha: regarding https://review.openstack.org/#/q/status:open++topic:scenarios-nv -- the two queens branch are failing due to https://bugs.launchpad.net/tripleo/+bug/1796639 -- what can we do about it?15:16
openstackLaunchpad bug 1796639 in tripleo "[stable/queens] The tripleo-ci-centos-7-undercloud-upgrades is failing on queens for No such file or directory: '/etc/heat/policy.json" [Critical,Triaged]15:16
mwhahahassbarnea|rover: fix https://bugs.launchpad.net/tripleo/+bug/1796639 ?15:16
weshaymwhahaha, still is ovh bs115:16
* mwhahaha enjoys stating the obvious15:16
mwhahahai mean that bug has been open for 2 months now15:16
weshayhttp://dashboard-ci.tripleo.org/d/cEEjGFFmz/cockpit?orgId=1&panelId=61&fullscreen15:16
mwhahahaperhaps we should pay attnetion to it15:16
mwhahahalet me go spin up a reproducer i guess15:17
mwhahahai think i started to previously and got side tracked with the rest of the world falling apart15:17
weshayhrm.. couple real issues15:17
weshaymarios, http://logs.openstack.org/68/619468/3/gate/tripleo-ci-centos-7-standalone/d418ae2/logs/stackviz/#/testrepository.subunit/test-details/tempest.scenario.test_network_basic_ops.TestNetworkBasicOps.test_hotplug_nic15:18
openstackgerritNicolas Hicher proposed openstack-infra/tripleo-ci master: testenv-worker: add keystone v3 support  https://review.openstack.org/60447015:18
weshaythat failed in the gate.. let's add it to the skip list, w/ a bug for chandan and arx to look at15:18
ssbarnea|rovermwhahaha: my current nickname suffix does not allow me to *fix* things,...15:19
weshaychem, matbu marios /me looking at http://logs.openstack.org/92/616592/7/gate/tripleo-ci-centos-7-scenario000-multinode-oooq-container-updates/8147d4d/job-output.txt.gz#_2018-11-30_09_48_55_95859215:19
weshayhaven't seen that error before15:19
mwhahahassbarnea|rover: :D i'm going to tackle that one today15:19
ssbarnea|rovermwhahaha: thank you, I will assign it to you and look for the next one.15:20
weshaymarios, bug #2 http://logs.openstack.org/92/616592/7/gate/tripleo-ci-centos-7-scenario000-multinode-oooq-container-updates/8147d4d/logs/undercloud/home/zuul/overcloud_update_data_migration.log.txt.gz#_2018-11-30_09_48_5515:20
chemweshay: in the middle of something will have a look later15:20
weshaymwhahaha, two real bugs, rest are timeouts15:20
weshaychem++15:20
weshaythanks15:20
weshayI have to step out and get my stupid car fixed...15:21
mwhahahak15:21
weshaymarios, ssbarnea|rover can we get those two issues written up please and one skip list for basicops issue15:21
ssbarnea|roveradded to https://review.rdoproject.org/etherpad/p/ruckrover-sprint2315:23
openstackgerritMichele Baldessari proposed openstack/puppet-pacemaker master: Match node properties more strictly  https://review.openstack.org/62089215:25
mariosweshay: ack15:25
openstackgerritNicolas Hicher proposed openstack-infra/tripleo-ci master: vexxhost: ensure use private ip for te-broker  https://review.openstack.org/62118215:25
*** hberaud is now known as hberaud|afk15:26
*** dsneddon has quit IRC15:28
*** mmagr has joined #tripleo15:29
*** salmankhan has quit IRC15:29
*** Vorrtex has joined #tripleo15:30
*** dsneddon has joined #tripleo15:31
*** bnemec is now known as beekneemech15:34
jaosorioralright! off now. Have a good weekend everyone!15:36
*** ramishra has quit IRC15:38
*** hberaud|afk is now known as hberaud15:38
*** amoralej|lunch is now known as amoralej15:39
*** quiquell is now known as quiquell|off15:39
openstackgerritSergii Golovatiuk proposed openstack/python-tripleoclient master: Stop undercloud upgrade if deployment tasks fail  https://review.openstack.org/62090515:42
*** dansmith is now known as SteelyDan15:43
openstackgerritBogdan Dobrelya proposed openstack/paunch master: Add missing default-runtime support bits  https://review.openstack.org/60863215:44
chembandini: EvilienM ah, yes, fedora28 ... no upgrade there15:46
mariosssbarnea|rover: weshay so it hink those two are the same wes i mean 17:19 < weshay> chem, matbu marios /me looking at15:47
marioshttp://logs.openstack.org/92/616592/7/gate/tripleo-ci-centos-7-scenario000-multinode-oooq-container-updates/8147d4d/job-output.txt.gz#_2018-11-30_09_48_55_95859215:47
marios17:20 < weshay> marios, bug #215:47
marioshttp://logs.openstack.org/92/616592/7/gate/tripleo-ci-centos-7-scenario000-multinode-oooq-container-updates/8147d4d/logs/undercloud/home/zuul/overcloud_update_data_migration.log.txt.gz#_2018-11-30_09_48_5515:47
openstackgerritBogdan Dobrelya proposed openstack/paunch master: Add missing default-runtime support bits  https://review.openstack.org/60863215:47
mariosssbarnea|rover: weshay i filed https://bugs.launchpad.net/tripleo/+bug/180606815:47
openstackLaunchpad bug 1806068 in tripleo "centos-7-scenario000-multinode-oooq-container-updates job failing overcloud_online_data_migration.sh.txt.gz with "u'Error creating static inventory"" [Undecided,Triaged] - Assigned to Marios Andreou (marios-b)15:47
mariosweshay: nm i see the testops one looking now15:49
mariosssbarnea|rover: did you dig there already?15:49
*** thrash is now known as thrash|biab15:51
*** ade_lee has joined #tripleo15:53
*** ksambor has quit IRC15:57
openstackgerritJohn Fulton proposed openstack/tripleo-common master: Add python-notario package to mistral-executor container  https://review.openstack.org/62119315:57
*** mmagr has quit IRC16:02
*** hjensas has joined #tripleo16:04
*** artom is now known as temka16:08
*** ooolpbot has joined #tripleo16:10
ooolpbotURGENT TRIPLEO TASKS NEED ATTENTION16:10
ooolpbothttps://bugs.launchpad.net/tripleo/+bug/179663916:10
*** ooolpbot has quit IRC16:10
openstackLaunchpad bug 1796639 in tripleo "[stable/queens] The tripleo-ci-centos-7-undercloud-upgrades is failing on queens for No such file or directory: '/etc/heat/policy.json" [Critical,Triaged] - Assigned to Alex Schultz (alex-schultz)16:10
*** rbrady has quit IRC16:11
openstackgerritGiulio Fidente proposed openstack/tripleo-common master: Increase ansible gather_timeout to 30secs for config-download  https://review.openstack.org/62120716:15
openstackgerritMerged openstack/tripleo-common stable/rocky: Select message IDs properly when publishing to swift  https://review.openstack.org/61924916:20
*** yprokule has quit IRC16:23
*** pcaruana has joined #tripleo16:26
* mwhahaha flips tables16:26
mwhahahassbarnea|rover, weshay: so the reproducer for the queens upgrade job fails on setting up the queens repos16:26
ssbarnea|roverrlandy: ^^16:26
mwhahahaecho 'Failed to parse dlrn hash'16:27
*** thrash|biab is now known as thrash16:27
openstackgerritMarios Andreou proposed openstack/tripleo-quickstart master: Add test_black_regex for fs052 standalone for failing tempest  https://review.openstack.org/62122116:28
mwhahahacause the curl fails for https://mirror.regionone.rdo-cloud.rdoproject.org:8080/rdo/centos7-queens/d7d80c59146ce502ae82ad9ee93ab6409a3353f2_15fc9723/delorean.repo16:28
mariosweshay: ssbarnea|rover https://review.openstack.org/621221 for https://bugs.launchpad.net/tripleo/+bug/1806075 just adding skip for now as requested16:28
openstackLaunchpad bug 1806075 in tripleo "centos-7-standalone job failing tempest TestNetworkBasicOps" [Undecided,Triaged] - Assigned to Marios Andreou (marios-b)16:28
mwhahahacurl: (35) SSL received a record that exceeded the maximum permissible length.16:28
mwhahahadat ssl16:28
*** brault has joined #tripleo16:28
fultonjmarios: do you have a few minutes to give me a hand with https://review.openstack.org/#/c/617368 ?16:29
rlandyssbarnea|rover: probably pretty old16:29
mariosfultonj: can we please do it on monday is a bit late here and i still have some pending tasks before i can go16:30
fultonjmarios: certainly16:30
mariosthanks fultonj16:30
fultonjthank you16:30
mariosno thank you16:30
marios:D16:31
mariosk going16:31
mwhahaharlandy: fyi https://bugs.launchpad.net/tripleo/+bug/180607716:32
openstackLaunchpad bug 1806077 in tripleo "reproducer queens undercloud upgrade job fails trying to setup the queens repo" [High,Triaged]16:32
*** holser_ has quit IRC16:32
rlandymwhahaha: thanks for logging that - will review16:33
*** ccamacho has quit IRC16:35
*** ccamacho has joined #tripleo16:35
mwhahaharlandy: might be because the tripleo dlrn hash is bad or some reason. i'll keep poking at it because i need to reproduce this thing16:35
*** xek has quit IRC16:36
marios_|ruckfultonj: successful scen1 deploy from today if you're interested http://logs.openstack.org/08/619508/10/check/tripleo-ci-centos-7-scenario001-standalone/d3d5873/logs/undercloud/home/zuul/standalone_deploy.log.txt.gz16:36
rlandynot necessarily16:37
marios_|ruckfultonj: (from earlier)16:37
fultonjmarios_|ruck: cool16:37
*** bogdando has quit IRC16:37
mwhahaharlandy: well for whatever reason it was bad for me. anyway i replaced it with current-tripleo for now and will continue16:37
rlandyI'm trying that reproducer16:38
mwhahaharlandy: http://logs.openstack.org/75/620975/1/check/tripleo-ci-centos-7-undercloud-upgrades/6d3151c/logs/reproducer-quickstart.sh is the specific reproducer i was working with (not sure if i provided the right link)16:39
rlandyyeah - it was in the bug16:39
rlandyssbarnea|rover: ^^ trying it out16:40
mwhahahaof course it worked fine for me (the upgrade process)16:41
mwhahahaso in CI we aren't actually updating anything16:42
mwhahahahttp://logs.openstack.org/75/620975/1/check/tripleo-ci-centos-7-undercloud-upgrades/6d3151c/logs/undercloud/home/zuul/undercloud_upgrade.log.txt.gz16:42
mwhahahathat seems wrong16:42
ssbarnea|rovermwhahaha: you must use SSL16:42
mwhahahassbarnea|rover: i did, the issue was a bad hash. either way the job doesn't seem to be doing what it's supposed to16:42
mwhahahassbarnea|rover: the undercloud upgrade is supposed to update some packages (and they aren't)16:43
mwhahahai wonder if the repo config is bad in the job16:43
mwhahahahttp://logs.openstack.org/75/620975/1/check/tripleo-ci-centos-7-undercloud-upgrades/6d3151c/logs/undercloud/home/zuul/undercloud_upgrade.sh.txt.gz16:43
mwhahahatripleoclient is supposed to be updated, but did not happen16:43
ssbarnea|rovermwhahaha: 100% this web server is incorectly configured.16:43
ssbarnea|roverevem my browser refused to load that url16:43
mwhahahassbarnea|rover: it's limited to rdo cloud addresses16:44
ssbarnea|rovermwhahaha: i suspect HSTS policy forbits you from using HTTP.16:48
*** udesale has quit IRC16:49
mwhahahameh i got the process working with out that, still i think it's a job config problem because we didn't actually end up udpating any packages between pike/queens16:49
mwhahahawhich is weird16:49
*** RaisdbyRobots has joined #tripleo16:51
rlandydoesn't seem like reproducer issue16:52
ssbarnea|rovermwhahaha: run: curl -s -D- https://rdoproject.org/ | grep Strict16:53
ssbarnea|roveryou will understand why it fails. now the funny part is that my own curl is not affected by this, only the browser but i suspect a different version of curl may behave different16:54
*** abishop has quit IRC16:54
ssbarnea|roverthe current setting of rdoproect.org enforces use of SSL (HTTPS) for all subdomains.16:55
*** trown|brb is now known as regain16:55
*** regain is now known as trown16:55
*** abishop has joined #tripleo16:55
*** trown is now known as trown|lunch16:56
ssbarnea|rover... and that is cached for 1Y.16:56
*** rnoriega- has quit IRC16:56
*** rnoriega has joined #tripleo16:58
*** jpena is now known as jpena|off16:59
*** RaisdbyRobots has quit IRC16:59
*** jpich has quit IRC17:00
*** iurygregory has quit IRC17:00
*** tongl has joined #tripleo17:03
*** pcaruana has quit IRC17:04
openstackgerritMike Fedosin proposed openstack/ansible-role-container-registry master: Allow to skip docker reconfiguration  https://review.openstack.org/62124117:05
weshayssbarnea|rover, marios_|ruck fyi.. when skipping a tempest test.. that info goes here https://github.com/openstack/tripleo-quickstart-extras/blob/master/roles/validate-tempest/vars/tempest_skip_master.yml17:06
weshaywhen you skip due to a bug17:06
*** ooolpbot has joined #tripleo17:10
ooolpbotURGENT TRIPLEO TASKS NEED ATTENTION17:10
ooolpbothttps://bugs.launchpad.net/tripleo/+bug/179663917:10
*** ooolpbot has quit IRC17:10
openstackLaunchpad bug 1796639 in tripleo "[stable/queens] The tripleo-ci-centos-7-undercloud-upgrades is failing on queens for No such file or directory: '/etc/heat/policy.json" [Critical,Triaged] - Assigned to Alex Schultz (alex-schultz)17:10
marios_|ruckweshay: ack17:13
weshaymarios_|ruck, the why.. being.. we have a job that runs all the skip tests so we know when to take it back out of the skiplist and we always have a bug associated17:14
mariosweshay: makes sense17:17
ssbarnea|roverweshay: this approach requires two merges from ci (us): add workaround, remove workaround. we could avoid this by publishing the list of ignored tests somewhere and having a periodic job for updating it based on the bug status on LP. When the bug is closed the skip is *no* longer performed.17:17
weshayssbarnea|rover, we have that17:17
ssbarnea|rovercool17:17
weshayssbarnea|rover, talk to arx if you want details17:18
ssbarnea|roverweshay: too busy right now, trying to fix his other problem and I think I narrowed it down:  read https://bugs.launchpad.net/tripleo/+bug/180607717:20
openstackLaunchpad bug 1806077 in tripleo "reproducer queens undercloud upgrade job fails trying to setup the queens repo" [High,Triaged]17:20
weshayssbarnea|rover, ok.. queens is not huge priority atm17:20
ssbarnea|roverdefaulting to http:// something does not play well with SSL. I doubt is queens related17:21
*** hjensas has quit IRC17:21
ssbarnea|rovernot sure why I seen this only on queens because this kind of issue is related to infrastructure configuration.17:21
openstackgerritMarios Andreou proposed openstack/tripleo-quickstart-extras master: Add TestNetworkBasicOps.test_hotplug_nic to master skip list  https://review.openstack.org/62124817:21
mariosweshay: ssbarnea|rover ^17:21
*** ykarel is now known as ykarel|away17:22
*** shardy has quit IRC17:22
weshaythanks marios have a good weekend17:23
mariosu 217:23
*** moguimar has quit IRC17:25
weshayssbarnea|rover, anything I can help w/?17:25
ssbarnea|roverweshay: just want to tell me if my comments on https://bugs.launchpad.net/tripleo/+bug/1806077 makes sense. it should be easy to swich config for rdo proxy to ssl to avoid it.17:26
openstackLaunchpad bug 1806077 in tripleo "reproducer queens undercloud upgrade job fails trying to setup the queens repo" [High,Triaged]17:26
*** kopecmartin is now known as kopecmartin|off17:26
weshaywoot.. full pass on 3rd party :) https://review.openstack.org/#/c/621075/17:30
weshayssbarnea|rover, check that out :))17:31
weshayyou probably haven't seen that since you joined17:31
rascasofthey folks, are there specific limitations for the things you can include in the standalone deployment? https://docs.openstack.org/tripleo-docs/latest/install/containers_deployment/standalone.html17:31
ssbarnea|roverweshay: my mistake sent you wrong link, SSL received a record that exceeded the maximum permissible17:31
weshayrascasoft, bah.. get out of here17:31
weshaywho is this guy17:31
rascasoftI'm trying to include octavia via the usual -e option, but I'm miserably failing17:31
openstackgerritGoutham Pacha Ravi proposed openstack/tripleo-heat-templates master: Switch Manila API to httpd and support TLS  https://review.openstack.org/59456617:32
ssbarnea|roverhttps://bugs.launchpad.net/tripleo/+bug/180607717:32
openstackLaunchpad bug 1806077 in tripleo "reproducer queens undercloud upgrade job fails trying to setup the queens repo" [High,Triaged]17:32
rascasoftweshay, you'll never get rid of me17:32
weshayNOOOOOOOOOOOOOOOOo17:32
mwhahaharascasoft: we've never tried octavia, so not sure what limitations would be17:32
mwhahaharascasoft: best take that up with the networking folks17:32
mwhahaharodrigods: though if the octavia install requires mistral/swift/zaqar, none of that exists in standalone17:33
rascasoftmwhahaha, well at the moment I'm failing with a specific error, is this worth a bug or is too specific?17:33
mwhahaharodrigods: er sorry wrong person17:33
mwhahaharascasoft: probably? it's likely you're hitting some assumptions in the octavia deployment that needs to be handled specially for standalone17:33
rascasoftmwhahaha, ok, I'd go with a bug so everyone can be happy17:34
weshayrascasoft, so..  here's an example if a different set of services and params https://review.openstack.org/#/c/616592/7/ci/environments/scenario001-standalone.yaml17:34
mwhahaharascasoft: best ask cgoncalves or beagles about octavia+standalone. not sure if they've tried it17:34
beaglesrascasoft, almost certain we havent tried it17:35
rascasoftweshay, oh nice, maybe it can be useful, thanks for still loving me17:35
*** weshay is now known as he_hates_me17:35
beaglesrascasoft, a bug would be very welcome and we'll look at it asap17:35
* he_hates_me loves the xfl17:35
rascasoftbeagles, what are your feelings, it might work if I include all the dependencies? Do we want bugs on this topic or it's too experimental?17:35
rascasoftbeagles, oh ok17:36
mwhahahahe_hates_me: too obscure17:36
he_hates_meprobably17:36
*** he_hates_me is now known as weshay17:36
* mwhahaha gets the reference tho17:36
beaglesmwhahaha, do we have any standalone jobs that involve deployments using external_deploy_tasks yet?17:36
mwhahahabeagles: ceph17:37
weshayprobably should kick/ban rascasoft17:37
beaglesmwhahaha, okay so that mechanism is probably fine at least17:37
mwhahahabeagles: the external deploy tasks work fine, it's any assumptions on mistral/swift that don't exist17:37
mwhahahabeagles: so ceph has a param that can be used in the case of standalone for local file stuff17:38
rascasoftweshay, then I'll change my name, and then who knows, maybe I already did that and I am rascasoft and mwhahaha at the same time17:38
mwhahahabeagles: example is https://review.openstack.org/#/c/616592/7/ci/environments/scenario001-standalone.yaml@4217:38
rascasoftweshay, Invasion of the Body Snatchers17:38
mwhahahabeagles: so it depends on what the octavia installer needs and if any of those external service dependencies can be replaced with something static17:38
beaglesmwhahaha, ack17:41
rlandymwhahaha: I could reproduce the error17:41
rlandy2018-11-30 17:39:14 | + echo 'Failed to parse dlrn hash'17:41
rlandy2018-11-30 17:39:14 | Failed to parse dlrn hash17:41
*** hberaud is now known as hberaud|gone17:45
*** brault has quit IRC17:46
openstackgerritMerged openstack/ansible-role-redhat-subscription master: Set up repos when using portal with activation key  https://review.openstack.org/62093717:48
mwhahahaif ya'll could stop approving things that would be lovely17:49
*** openstackgerrit has quit IRC17:51
*** amoralej is now known as amoralej|off17:53
*** hberaud|gone is now known as hberaud17:54
apetrichmwhahaha, btw that bug was a real error but for a patch https://review.openstack.org/#/c/619039/17:57
mwhahahaapetrich: yea i saw17:57
*** hberaud is now known as hberaud|gone17:58
*** derekh has quit IRC18:00
mwhahahawell i can't reproduce the queens upgrade errror18:02
mwhahahaseems to be ci specific18:02
mwhahaha(great)18:02
*** gfidente has quit IRC18:04
*** cylopez has quit IRC18:04
mwhahahaoh damn it18:06
mwhahahait's install-built-repo18:06
tongl@mwhahaha: https://review.openstack.org/#/c/618796/, https://review.openstack.org/#/c/618798/, https://review.openstack.org/#/c/605056/ have been reviewed by Emilien and passed zull check. Do you think I need to ask Emilien for workflow +1 or you can do it?18:06
mwhahahatongl: i can but the CI gate is backed up so we should not be merging anything non-critical at the moment18:07
tonglWe are waiting for these patches to be merged so that we can package them downstream into RHOSP.18:08
tonglThese are the missing pieces for RHOSP integration with NSX neutron plugin.18:09
* mwhahaha shrugs18:09
mwhahahaeven if i do approve them, they likely won't be merged18:09
mwhahahaso if you're waiting, i would be proactively pulling them down because they are unlikely to land upstream in the near future18:09
*** ooolpbot has joined #tripleo18:10
ooolpbotURGENT TRIPLEO TASKS NEED ATTENTION18:10
ooolpbothttps://bugs.launchpad.net/tripleo/+bug/179663918:10
openstackLaunchpad bug 1796639 in tripleo "[stable/queens] The tripleo-ci-centos-7-undercloud-upgrades is failing on queens for No such file or directory: '/etc/heat/policy.json" [Critical,Triaged] - Assigned to Alex Schultz (alex-schultz)18:10
*** ooolpbot has quit IRC18:10
tonglokay, do you have any idea when they can be landed upstream?18:10
mwhahahawhen we can actually merged patches with out everything timing out18:11
*** jfrancoa has quit IRC18:16
*** ykarel|away has quit IRC18:17
*** wolverineav has joined #tripleo18:17
*** openstackgerrit has joined #tripleo18:26
openstackgerritMarius Cornea proposed openstack/tripleo-heat-templates master: Use InternalApi network for openshift_master_cluster_hostname  https://review.openstack.org/62125618:26
openstackgerritAlex Schultz proposed openstack/tripleo-quickstart-extras master: Add ability to disable yum update with gating repo  https://review.openstack.org/62125918:47
openstackgerritMarius Cornea proposed openstack/puppet-tripleo master: Set balance to source for openshift_master endpoints  https://review.openstack.org/62126018:49
openstackgerritMarius Cornea proposed openstack/tripleo-heat-templates master: Set balance to source for openshift_master endpoints  https://review.openstack.org/62126118:51
openstackgerritAlex Schultz proposed openstack/tripleo-quickstart-extras master: Add ability to disable yum update with gating repo  https://review.openstack.org/62125918:53
mwhahahaweshay, marios, ssbarnea|rover -^ should fix the upgrade job(s)18:53
openstackgerritAlex Schultz proposed openstack/tripleo-common stable/queens: Switch scenarios to non-voting  https://review.openstack.org/62097518:55
ssbarnea|rovermwhahaha: cool. i think that your SSL problem with rdoproject.com mirror will be also sorted till monday.18:56
openstackgerritMarius Cornea proposed openstack/tripleo-heat-templates master: Set balance to source for openshift_router endpoints  https://review.openstack.org/62126118:57
ssbarnea|rovermwhahaha: tbh I am not sure if disabling yum update is the best idea because we will no longer use latest code, meaning missing fixes or bugs.18:58
mwhahahassbarnea|rover: not true18:58
mwhahahassbarnea|rover: we do a yum update as part of the upgrade itself18:58
mwhahahassbarnea|rover: so this is prematurely yum updating packages18:58
mwhahahathis only affects upgrades18:58
mwhahahaso prior to the upgrade we need to swap out the repos, but we do updates as part of the upgrade execution18:59
mwhahahafor the undercloud that includes a yum update of python-tripleoclient followed by an actual yum update in the upgrade process18:59
mwhahahafor the overcloud that is also true18:59
ssbarnea|rovermwhahaha: ahh, in this case it should be ok.18:59
mwhahahayes18:59
mwhahahathats why it's still true by default18:59
mwhahahait only affects upgrades when we swap out the newer repos18:59
mwhahahait should just add the gating repo in and not update18:59
mwhahahabecause that gets handled by the subsequent upgrade processes19:00
ssbarnea|rovermakes sense. we skip it because we do it later anyway with upgrades, saving some time which is key for upgrades.19:00
mwhahahaso what was happening is in the undercloud update was that it wa getting the newer heat packages that removed the policy.json w/o a service restart19:00
mwhahahacausing the error19:00
mwhahahavery obscure and annoying :D19:01
openstackgerritAlex Schultz proposed openstack/puppet-tripleo stable/queens: Switch scenarios to non-voting  https://review.openstack.org/62097619:01
*** wolverineav has quit IRC19:02
*** wolverineav has joined #tripleo19:06
*** ooolpbot has joined #tripleo19:10
ooolpbotURGENT TRIPLEO TASKS NEED ATTENTION19:10
ooolpbothttps://bugs.launchpad.net/tripleo/+bug/179663919:10
openstackLaunchpad bug 1796639 in tripleo "[stable/queens] The tripleo-ci-centos-7-undercloud-upgrades is failing on queens for No such file or directory: '/etc/heat/policy.json" [Critical,In progress] - Assigned to Alex Schultz (alex-schultz)19:10
*** ooolpbot has quit IRC19:10
*** morazi has quit IRC19:11
dprincemwhahaha: hi, is the gate okay again?19:20
mwhahahano19:20
dprince:/19:20
mwhahahapretty much19:20
mwhahahawe need https://review.openstack.org/#/c/621248/ at least19:21
dprinceand a weekend19:21
mwhahahayes19:21
openstackgerritNatal Ngétal proposed openstack/tripleo-docs master: [Documentation] Add a note for tripleo container.  https://review.openstack.org/62127419:23
*** wolverineav has quit IRC19:37
*** wolverineav has joined #tripleo19:41
*** wolverineav has quit IRC19:42
*** wolverineav has joined #tripleo19:42
*** wolverineav has quit IRC19:42
*** wolverineav has joined #tripleo19:43
*** wolverineav has quit IRC19:43
*** wolverineav has joined #tripleo19:43
*** wolverineav has quit IRC19:44
*** wolverineav has joined #tripleo19:45
rascasoftmwhahaha, weshay, beagles, https://bugs.launchpad.net/tripleo/+bug/180611319:46
openstackLaunchpad bug 1806113 in tripleo "[tripleo] Standalone setup is unable to deploy Octavia" [Undecided,Triaged]19:46
rascasoftenvironment is accessible if you need it ;)19:46
mwhahahaoh yea it doesn't work because it's trying to get the default nova keypair19:47
mwhahahawhich doesn't exist19:47
mwhahahaso there needs to be a way to specify that outside of that19:47
mwhahahathere is no compute service19:47
*** wolverineav has quit IRC19:52
*** wolverineav has joined #tripleo19:53
weshayrascasoft, you can't just open bugs and quit..19:54
*** hjensas has joined #tripleo19:58
*** wolverineav has quit IRC19:59
*** rh-jelabarre has quit IRC20:03
*** rh-jelabarre has joined #tripleo20:05
*** morazi has joined #tripleo20:05
openstackgerritMerged openstack/tripleo-quickstart master: Revert "master/hack: Switch to CBS instead of buildlogs"  https://review.openstack.org/61962520:09
weshayhooooray20:09
*** ooolpbot has joined #tripleo20:10
ooolpbotURGENT TRIPLEO TASKS NEED ATTENTION20:10
ooolpbothttps://bugs.launchpad.net/tripleo/+bug/179663920:10
openstackLaunchpad bug 1796639 in tripleo "[stable/queens] The tripleo-ci-centos-7-undercloud-upgrades is failing on queens for No such file or directory: '/etc/heat/policy.json" [Critical,In progress] - Assigned to Alex Schultz (alex-schultz)20:10
*** ooolpbot has quit IRC20:10
*** jistr has quit IRC20:11
*** jistr has joined #tripleo20:13
*** radeks__ has quit IRC20:14
*** xek has joined #tripleo20:19
*** vpickard is now known as vpickard_20:28
openstackgerritMerged openstack/puppet-pacemaker master: Match node properties more strictly  https://review.openstack.org/62089220:32
*** radeks has joined #tripleo20:32
*** wolverineav has joined #tripleo20:33
*** slaweq has quit IRC20:34
*** matbu has quit IRC20:38
*** matbu has joined #tripleo20:46
*** openstackgerrit has quit IRC20:50
*** itlinux has joined #tripleo20:51
*** itlinux_ has joined #tripleo20:58
*** itlinux has quit IRC20:59
*** dprince has quit IRC21:04
*** raildo has quit IRC21:05
*** rlandy has quit IRC21:08
*** wolverineav has quit IRC21:08
*** ooolpbot has joined #tripleo21:10
ooolpbotURGENT TRIPLEO TASKS NEED ATTENTION21:10
ooolpbothttps://bugs.launchpad.net/tripleo/+bug/179663921:10
openstackLaunchpad bug 1796639 in tripleo "[stable/queens] The tripleo-ci-centos-7-undercloud-upgrades is failing on queens for No such file or directory: '/etc/heat/policy.json" [Critical,In progress] - Assigned to Alex Schultz (alex-schultz)21:10
*** ooolpbot has quit IRC21:10
weshaymwhahaha, I'll have to find it, but we're supposed to deal w/ using the gating repo only where appropriate https://review.openstack.org/#/c/621259/21:10
weshayre: that review21:10
mwhahahaweshay: what do you mean21:10
weshayso if the patch is on rocky for example.. the gating repo would be skipped in the upgrade phase/playbook where the release is set to master21:11
weshayif the patch is on master21:11
weshayfor instance than the gating repo would be enabled on master21:11
mwhahahaok i can join the ib_gating_repo_update with ib_gating_repo_enabled21:11
mwhahahaeither way we need to *not* always run the yum update on the upgrade jobs21:12
weshaymwhahaha, I'm not saying to do anything as much as I'm just thinking21:12
mwhahahathat's fine, what you say makes sense21:12
weshayit was complicated and I need to poke at this for a while21:12
weshaymwhahaha, /me wondering though21:12
weshayif your intent is disable any patches from landing on the env. during the upgrade/update portion21:12
mwhahahano21:12
mwhahahait is not21:12
mwhahahait is so that the yum update -y does not happen after install before upgrade21:13
mwhahahabecause the upgrade itself will pull in the appropriate updated packages (From the gating repo) at upgrade time21:13
*** openstackgerrit has joined #tripleo21:13
openstackgerritAlex Schultz proposed openstack/tripleo-quickstart-extras master: Add ability to disable yum update with gating repo  https://review.openstack.org/62125921:13
weshayOH21:13
mwhahahathat's what's breaking the queens upgrade21:13
mwhahahabecause that role is updating all the packages21:13
mwhahahaso the upgrade process breaks for the udnercloud21:14
*** Vorrtex has quit IRC21:14
weshayk k21:14
weshaythat makes sense21:14
mwhahahathe queens heat drops policy.json but the yum update doesn't restart the service21:14
mwhahahaso sadness ensues21:14
mwhahahait also explains why if you follow the normal procedure it always work21:14
weshayk21:14
mwhahahait only fails in CI because of this21:14
weshaythanks for explaining that21:14
weshayright21:14
openstackgerritDan Sneddon proposed openstack/os-net-config master: Fix incorrect application of 'ip' commands for bridges  https://review.openstack.org/62129121:20
*** radeks has quit IRC21:21
*** toure is now known as toure|gone21:26
*** salmankhan has joined #tripleo21:31
*** vinaykns has quit IRC21:34
*** chem has quit IRC21:34
*** wolverineav has joined #tripleo21:35
*** salmankhan has quit IRC21:36
*** abishop has quit IRC21:49
*** dpeacock has quit IRC21:52
*** dpeacock has joined #tripleo21:55
*** wolverineav has quit IRC21:59
*** wolverineav has joined #tripleo22:04
openstackgerritHarald Jensås proposed openstack/os-net-config master: Fix - restart VLAN interface on underlying device change  https://review.openstack.org/62129822:06
*** ooolpbot has joined #tripleo22:10
ooolpbotURGENT TRIPLEO TASKS NEED ATTENTION22:10
ooolpbothttps://bugs.launchpad.net/tripleo/+bug/179663922:10
openstackLaunchpad bug 1796639 in tripleo "[stable/queens] The tripleo-ci-centos-7-undercloud-upgrades is failing on queens for No such file or directory: '/etc/heat/policy.json" [Critical,In progress] - Assigned to Alex Schultz (alex-schultz)22:10
*** ooolpbot has quit IRC22:10
*** salmankhan has joined #tripleo22:11
*** wolverineav has quit IRC22:12
*** itlinux_ has quit IRC22:13
*** wolverineav has joined #tripleo22:13
*** wolverineav has quit IRC22:18
*** wolverineav has joined #tripleo22:19
*** mcornea has quit IRC22:20
mwhahahabandini: jfyi https://review.rdoproject.org/r/#/c/17630/22:26
*** wolverineav has quit IRC22:28
*** wolverineav has joined #tripleo22:29
openstackgerritHarald Jensås proposed openstack/os-net-config master: Fix - restart VLAN interface on underlying device change  https://review.openstack.org/62129822:34
openstackgerritHarald Jensås proposed openstack/os-net-config master: Fix - restart VLAN interface on underlying device change  https://review.openstack.org/62129822:36
*** slaweq has joined #tripleo22:36
*** xek has quit IRC22:37
*** slaweq has quit IRC22:43
*** apetrich has quit IRC22:47
*** slaweq has joined #tripleo22:50
*** slaweq has quit IRC22:52
*** wolverineav has quit IRC22:54
*** slaweq has joined #tripleo22:54
*** wolverineav has joined #tripleo22:55
*** cfontain has joined #tripleo22:58
*** wolverineav has quit IRC23:00
*** wolverineav has joined #tripleo23:00
*** apetrich has joined #tripleo23:02
*** itlinux has joined #tripleo23:07
*** ooolpbot has joined #tripleo23:10
ooolpbotURGENT TRIPLEO TASKS NEED ATTENTION23:10
ooolpbothttps://bugs.launchpad.net/tripleo/+bug/179663923:10
openstackLaunchpad bug 1796639 in tripleo "[stable/queens] The tripleo-ci-centos-7-undercloud-upgrades is failing on queens for No such file or directory: '/etc/heat/policy.json" [Critical,In progress] - Assigned to Alex Schultz (alex-schultz)23:10
*** ooolpbot has quit IRC23:10
*** slaweq has quit IRC23:22
*** wolverineav has quit IRC23:42
*** tosky has quit IRC23:42
*** wolverineav has joined #tripleo23:45
*** wolverineav has quit IRC23:48
*** wolverineav has joined #tripleo23:49
openstackgerritJill Rouleau proposed openstack/paunch master: Add systemd healthchecks to podman  https://review.openstack.org/62037223:50
*** salmankhan has quit IRC23:51
*** slaweq has joined #tripleo23:54
*** pbandark has quit IRC23:55

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