Thursday, 2018-05-24

openstackgerritRedHat RDO CI proposed openstack/tripleo-heat-templates stable/queens: GATE CHECK for TripleO  https://review.openstack.org/56722400:00
openstackgerritRedHat RDO CI proposed openstack/tripleo-quickstart-extras master: GATE CHECK for quickstart-extras  https://review.openstack.org/56044500:00
openstackgerritRedHat RDO CI proposed openstack/tripleo-heat-templates stable/ocata: GATE CHECK for TripleO  https://review.openstack.org/56429100:00
*** bdodd is now known as bdodd|afk00:06
*** slaweq has joined #tripleo00:11
*** zaneb has joined #tripleo00:15
*** slaweq has quit IRC00:15
*** rpioso is now known as rpioso|afk00:18
*** zaneb has quit IRC00:25
openstackgerritHarald Jensås proposed openstack/python-tripleoclient master: Deployed Server - portmap should match neutron API  https://review.openstack.org/56978900:27
openstackgerritHarald Jensås proposed openstack/tripleo-heat-templates master: Get ctlplane interface config from the subnet  https://review.openstack.org/56954700:27
*** edmondsw has quit IRC00:28
*** pblaho has quit IRC00:29
*** pblaho has joined #tripleo00:33
*** edmondsw has joined #tripleo00:35
*** edmondsw has quit IRC00:39
*** hamzy has quit IRC00:51
*** mdnadeem has joined #tripleo01:08
*** noslzzp has joined #tripleo01:08
*** bdodd|afk is now known as bdodd01:09
*** edmondsw has joined #tripleo01:11
*** slaweq has joined #tripleo01:11
*** mdnadeem has quit IRC01:13
*** slaweq has quit IRC01:15
*** edmondsw has quit IRC01:15
*** psahoo has joined #tripleo01:19
*** colonwq has joined #tripleo01:24
*** matbu has joined #tripleo01:28
*** mdnadeem has joined #tripleo01:30
*** matbu has quit IRC01:42
*** gvrangan has joined #tripleo01:43
*** gvrangan has quit IRC01:44
*** trwyth has joined #tripleo01:53
*** zshi has quit IRC01:57
*** trwyth has quit IRC01:57
*** trwyth has joined #tripleo01:58
*** skramaja has joined #tripleo02:09
openstackgerritSaravanan KR proposed openstack/python-tripleoclient master: DeployIdentifier should be provided for any parameter actions  https://review.openstack.org/56976502:20
*** trwyth1 has joined #tripleo02:21
*** dxiri has quit IRC02:21
*** trwyth has quit IRC02:23
openstackgerritMarius Cornea proposed openstack/tripleo-quickstart master: Remove ceph from fast forward upgrade job  https://review.openstack.org/57027802:23
openstackgerritMarius Cornea proposed openstack/tripleo-quickstart master: Remove ceph from fast forward upgrade job  https://review.openstack.org/57027802:24
*** rlandy|rover|bbl is now known as rlandy|rover02:26
*** trwyth1 has left #tripleo02:27
openstackgerritSteve Baker proposed openstack/tripleo-common master: Add a dry-run behaviour to image image_uploader  https://review.openstack.org/57030002:34
*** gvrangan has joined #tripleo02:35
*** gvrangan has quit IRC02:39
*** ramishra has joined #tripleo02:45
*** psachin has joined #tripleo02:45
*** hasanq has joined #tripleo02:46
*** hasanq has quit IRC02:48
*** rlandy|rover has quit IRC03:27
*** skramaja has quit IRC03:43
*** skramaja has joined #tripleo03:43
openstackgerritIan Wienand proposed openstack/diskimage-builder master: Save and close stdout on exit  https://review.openstack.org/57009403:46
openstackgerritIan Wienand proposed openstack/diskimage-builder master: Add log directory option to functional tests  https://review.openstack.org/57009503:46
openstackgerritIan Wienand proposed openstack/diskimage-builder master: Rename output log files  https://review.openstack.org/57009603:46
openstackgerritIan Wienand proposed openstack/diskimage-builder master: WIP: native zuulv3 tests  https://review.openstack.org/56677903:46
*** fragatina has quit IRC03:52
*** fragatina has joined #tripleo03:52
*** hamzy has joined #tripleo03:54
*** fragatina has quit IRC03:55
*** fragatina has joined #tripleo03:56
*** fragatina has quit IRC03:58
Tenguhello there03:58
*** fragatina has joined #tripleo03:58
*** rcernin has quit IRC03:58
*** hamzy has quit IRC03:59
*** pdeore has joined #tripleo04:00
*** gvrangan has joined #tripleo04:00
*** hamzy has joined #tripleo04:01
*** fragatina has quit IRC04:02
*** gvrangan has quit IRC04:03
*** shreshtha has joined #tripleo04:07
*** udesale has joined #tripleo04:08
*** gbarros has joined #tripleo04:15
*** rcernin has joined #tripleo04:17
*** cshastri has joined #tripleo04:18
*** noslzzp has quit IRC04:19
*** rcernin has quit IRC04:23
*** rcernin has joined #tripleo04:23
*** fragatin_ has joined #tripleo04:24
*** pcaruana has joined #tripleo04:24
*** pcaruana has quit IRC04:29
*** tzumainn has quit IRC04:30
jaosoriorGood morning!04:31
*** pdeore has quit IRC04:34
*** openstackgerrit has quit IRC04:34
Tenguhow are you to day, jaosorior ?04:35
*** d0ugal has quit IRC04:35
jaosoriorpretty good!04:35
jaosorioryou?04:35
jaosoriorso, this time I actually had breakfast :D04:35
*** ykarel|afk has joined #tripleo04:35
Tengu:D04:36
TenguI'm building a docker container in order to run tmate on a more.. hmm. closed/secured/foo env :)04:36
jaosorioryou should write a blog post about that04:39
jaosoriorI would like something similar but haven't dedicated the time04:39
Tengu:)04:40
*** shreshtha_ has joined #tripleo04:40
Tengulet's say I decide to take the time for the sake of my laptop security :)04:40
*** radeks_ has joined #tripleo04:40
*** shreshtha has quit IRC04:42
*** janki has joined #tripleo04:42
jaosoriorTengu: let me know when you have time so we can continue the work :D04:43
Tengujaosorior: I'm currently (re⁴)deploying the quickstart with a non-ipv6 profiles, as the ipv6, although allowing to validate your patch yesterday, can't be properly used on my ipv4-only network :). but more than that, I can work ;).04:45
*** shreshtha has joined #tripleo04:46
*** shreshtha is now known as shreshtha-away04:46
*** gbarros has quit IRC04:47
jaosoriorlets get back to business when you're done deploying04:47
Tengujaosorior: ok :). That should let me create my container :).04:48
Tenguhave some issues with tmate losing its server.04:48
*** shreshtha_ has quit IRC04:49
*** pdeore has joined #tripleo04:50
*** shreshtha_ has joined #tripleo04:51
*** openstackgerrit has joined #tripleo04:53
openstackgerrityatin proposed openstack/tripleo-heat-templates master: Use /usr/bin/env python to autodetect python  https://review.openstack.org/57021304:53
*** shreshtha-away has quit IRC04:54
*** rcernin has quit IRC04:56
openstackgerritMerged openstack-infra/tripleo-ci master: Pass dlrn_hash from the releases file  https://review.openstack.org/56995804:58
EmilienMhello folks04:59
EmilienMjaosorior: hey I have some reviews if you have time, they aren't big04:59
jaosoriorEmilienM: sure04:59
*** shreshtha_ has quit IRC04:59
EmilienMjaosorior: https://review.openstack.org/#/c/570237/ and the ones on top04:59
EmilienMjaosorior: except the WIP05:00
EmilienMthanks05:00
EmilienMlet me know if you need any review05:00
*** mrunge has quit IRC05:00
jaosoriorEmilienM: will do, thanks05:01
Tengujaosorior: found out the issue: https://hub.docker.com/r/sigma/tmate/ - apparently needs "privilege" mode.05:01
*** mrunge has joined #tripleo05:02
*** d0ugal has joined #tripleo05:03
*** radeks_ has quit IRC05:04
openstackgerritMerged openstack/tripleo-upgrade stable/queens: Move SSL undercloud validation out of UC script create tasks.  https://review.openstack.org/57013005:07
*** d0ugal has quit IRC05:10
*** ccamacho has quit IRC05:10
*** ratailor has joined #tripleo05:10
*** anilvenkata has joined #tripleo05:11
*** links has joined #tripleo05:13
*** d0ugal_ has joined #tripleo05:14
*** d0ugal__ has joined #tripleo05:23
*** links has quit IRC05:24
*** d0ugal_ has quit IRC05:24
*** links has joined #tripleo05:25
*** nyechiel_ has joined #tripleo05:33
Tengu\o/05:35
Tengujaosorior: I have a secured tmate in a container «o/05:35
*** links has quit IRC05:35
Tenguhit a couple of issues with docker, especially since the "-ti" option for exec doesn't actually provide a real TTY :]05:35
*** links has joined #tripleo05:36
*** links has joined #tripleo05:36
*** gbarros has joined #tripleo05:37
Tenguwow. it's actually working.05:38
*** pdeore_ has joined #tripleo05:39
EmilienMwe should change the /topic to it05:39
*** udesale_ has joined #tripleo05:39
*** EmilienM changes topic to "Welcome to Rocky. CI status: GREEEEEEEEN "wow it's actually working" | http://tripleo.org/ | https://docs.openstack.org/tripleo-docs/latest"05:40
TenguEmilienM: XD I wasn't speaking about CI05:40
Tengubut actually, if it's working, it's indeed a "wow-effect" :)05:40
EmilienM:D05:41
EmilienMI have the wow effect every day here05:41
*** pdeore has quit IRC05:41
Tengu:) can't wait to get your summary email05:42
*** udesale has quit IRC05:42
*** dparkes has quit IRC05:42
*** links has quit IRC05:42
EmilienMTengu: it'll have to wait, i'm on PTO after the summit, taking a bit of time off05:43
TenguEmilienM: whaaat? PTO? nah, you don't have the right to PTO05:43
Tengu:)05:43
EmilienM:-O05:45
EmilienMmy dad says "if you don't have the right, take the left"05:45
EmilienMI'll follow his advice on this one ;-)05:45
*** links has joined #tripleo05:46
Tengu^^05:48
jaosoriorI'll get my PTO in July :D 3 weeks off. It'll be great!05:48
*** jaganathan has quit IRC05:50
*** jaganathan has joined #tripleo05:50
*** marios has joined #tripleo05:56
*** pgadiya has joined #tripleo05:58
*** saneax has joined #tripleo05:59
*** yamahata has joined #tripleo06:03
*** links has quit IRC06:03
*** links has joined #tripleo06:04
*** jfrancoa has joined #tripleo06:05
*** gbarros has quit IRC06:06
*** saneax has quit IRC06:09
*** elgxl has joined #tripleo06:12
*** radek__ has joined #tripleo06:14
*** radeks_ has joined #tripleo06:14
*** kopecmartin has joined #tripleo06:15
*** quiquell|off is now known as quiquell06:15
*** links has quit IRC06:17
*** olap has joined #tripleo06:18
*** olap has quit IRC06:18
*** olap has joined #tripleo06:18
*** holser__ has joined #tripleo06:21
*** links has joined #tripleo06:22
openstackgerritEmilien Macchi proposed openstack/python-tripleoclient master: tripleo_deploy/upgrade: fail of upgrade_tasks don't return 0  https://review.openstack.org/57032106:28
*** dparkes has joined #tripleo06:29
openstackgerritEmilien Macchi proposed openstack/tripleo-heat-templates master: upgrade: remove packages at step 3  https://review.openstack.org/57011606:30
openstackgerritEmilien Macchi proposed openstack/tripleo-heat-templates master: nova-ironic: add support for post-upgrade cleanup  https://review.openstack.org/57024106:31
openstackgerritEmilien Macchi proposed openstack/tripleo-heat-templates master: WIP - add missing services for upgrade cleanup  https://review.openstack.org/57028306:31
*** dciabrin has joined #tripleo06:32
*** dciabrin_ has quit IRC06:33
EmilienMskramaja: thx for your review :)06:34
skramajawelcome. i had a small nit in one of your review..06:35
EmilienMskramaja: I think this is good now?06:35
*** udesale__ has joined #tripleo06:36
skramajachecking..06:36
Tenguif anyone is using tmate in order to share session with others: https://github.com/cjeanner/docker-tmate-client06:36
skramajayes EmilienM, done.06:37
EmilienMskramaja: ++ thanks06:37
EmilienMalright I'm off to bed :)06:37
skramajasure. good night.06:37
*** links has quit IRC06:38
*** links has joined #tripleo06:38
*** udesale_ has quit IRC06:38
EmilienMthanks!06:39
*** pdeore__ has joined #tripleo06:42
*** links has quit IRC06:42
*** rcernin has joined #tripleo06:43
*** alee has joined #tripleo06:43
*** pdeore_ has quit IRC06:45
*** lvdombrkr has joined #tripleo06:47
*** matbu has joined #tripleo06:47
*** links has joined #tripleo06:52
*** _jbadiapa is now known as jbadiapa06:53
*** pcaruana has joined #tripleo06:54
*** ukalifon has joined #tripleo06:59
*** ccamacho has joined #tripleo07:03
*** avivgt has joined #tripleo07:04
*** cylopez has joined #tripleo07:07
*** janki has quit IRC07:07
*** ccamacho has quit IRC07:07
*** ccamacho has joined #tripleo07:08
*** avivgt has quit IRC07:08
*** aufi has joined #tripleo07:10
*** ffiore has joined #tripleo07:16
*** masco has joined #tripleo07:20
*** bogdando has joined #tripleo07:20
*** suuuper has joined #tripleo07:21
*** tesseract has joined #tripleo07:23
openstackgerritLukas Bezdicka proposed openstack/tripleo-heat-templates master: Upgrades: Refactor httpd_enabled variable  https://review.openstack.org/55578107:25
openstackgerritLukas Bezdicka proposed openstack/tripleo-heat-templates master: Upgrades: Refactor playbooks to set facts  https://review.openstack.org/56229407:25
*** cylopez has left #tripleo07:28
*** ssbarnea_ has joined #tripleo07:28
silehthi, https://review.openstack.org/#/c/569956/ have two +2, but no +A, any reason ?07:29
*** ykarel|afk is now known as ykarel|lunch07:30
openstackgerritJose Luis Franco proposed openstack/tripleo-upgrade stable/queens: DNM: Test pike->queens upgrades.  https://review.openstack.org/56361607:32
*** waleedm has joined #tripleo07:36
*** jpena|off is now known as jpena07:37
*** kbyrne has quit IRC07:45
*** nyechiel_ has quit IRC07:46
openstackgerritBogdan Dobrelya proposed openstack/puppet-tripleo stable/queens: Rework GDPR compliant logrotate config  https://review.openstack.org/57032807:48
*** kbyrne has joined #tripleo07:49
*** udesale__ is now known as udesale07:53
*** aufi_ has joined #tripleo07:53
openstackgerritBogdan Dobrelya proposed openstack/tripleo-quickstart-extras master: Fix path and wire-in UC deploy role data file  https://review.openstack.org/56398807:55
*** aufi has quit IRC07:55
openstackgerritBogdan Dobrelya proposed openstack/python-tripleoclient master: Store UC hosts/portmaps heat params in output_dir  https://review.openstack.org/56413507:56
bogdandoo/ folks. I need help with debugging https://logs.rdoproject.org/16/566916/3/openstack-check/gate-tripleo-ci-centos-7-ovb-3ctlr_1comp-featureset035-master/Z219aad2e271a40d3ac6ad2d039999d31/undercloud/home/jenkins/overcloud_deploy.log.txt.gz#_2018-05-23_09_22_06 . The reproduce script does not reproduce that error, it passes on RDO cloud07:57
*** jistr is now known as jistr|iview07:57
bogdandothat's https://review.openstack.org/#/c/566916/07:58
*** sshnaidm|off is now known as sshnaidm07:59
*** janki has joined #tripleo08:09
*** ykarel|lunch is now known as ykarel08:12
openstackgerritDmitry Tantsur proposed openstack/tripleo-common master: Fix handling hardware types and drivers when generating fencing parameters  https://review.openstack.org/56789608:14
openstackgerritCédric Jeanneret proposed openstack/tripleo-common master: Set FixedIp by default for public-facing VIP  https://review.openstack.org/56981808:14
*** gkadam has joined #tripleo08:17
openstackgerritDmitry Tantsur proposed openstack/python-tripleoclient master: Stop supporting the removed pxe_ssh driver in `generate fencing parameters`  https://review.openstack.org/57034908:21
Tenguhey there, PTAL? :) https://review.openstack.org/#/c/570209/08:22
*** jaganathan has quit IRC08:22
*** anilvenkata has quit IRC08:28
*** anilvenkata has joined #tripleo08:29
*** jaganathan has joined #tripleo08:29
*** avivgt has joined #tripleo08:30
openstackgerritLukas Bezdicka proposed openstack/tripleo-heat-templates master: Upgrades: Refactor playbooks to set facts  https://review.openstack.org/56229408:35
*** rcernin has quit IRC08:36
*** b00tcat has joined #tripleo08:41
*** derekh has joined #tripleo08:42
openstackgerritCédric Jeanneret proposed openstack/tripleo-common master: TLS by default for the overcloud for VIPs and FQDNs  https://review.openstack.org/56936408:50
*** amoralej|off is now known as amoralej08:55
*** shreshtha has joined #tripleo08:58
*** jistr|iview is now known as jistr09:07
jaosoriorshardy: where is user-environment created?09:07
*** Tengu has quit IRC09:07
*** tdasilva has quit IRC09:08
jaosoriorjistr: setting EnablePublicTLS: true in t-h-t didn't help :/ cause we don't have access to the variables set in the resource registry from the environment nor the user environment :/09:08
*** Hazelesque has quit IRC09:08
jaosoriorjistr: talking about https://review.openstack.org/#/c/568736/6/overcloud-resource-registry-puppet.j2.yaml09:09
jaosoriorjistr: figuring out how to get it, or how to include it in a relevant environment by default.09:09
openstackgerritAna Krivokapic proposed openstack/instack-undercloud master: Upload validations to swift on undercloud install  https://review.openstack.org/56423309:10
*** akrivoka has joined #tripleo09:10
jaosoriorjistr: any ideas? :D09:13
marioso/ dev docs for Q upgrade if you have time please @ https://review.openstack.org/#/c/569443/ && http://logs.openstack.org/43/569443/4/check/build-openstack-sphinx-docs/7ef0bbf/html/install/developer/upgrades/major_upgrade.html09:14
*** morazi has joined #tripleo09:14
shardyjaosorior: it's created in tripleoclient, which is actually a bug as we should instead add each environment to plan-environment and let the environments get merged by heat on deploy09:15
shardyjaosorior: ref https://review.openstack.org/#/c/448209/ - turns out fixing that was quite hard as we need heat to handle retrieving the environments from swift to avoid the POST request limit09:16
shardyIIRC ramishra was looking into that09:16
*** salmankhan has joined #tripleo09:16
*** Hazelesque has joined #tripleo09:18
jistrjaosorior: i think it should be possible to get that info, but we may need to shuffle things around slightly. BTW we just got a green experimental upgrade job yesterday for scenario000 Q->master upgrade, so we should make sure the TLS by default will not break upgrades before we re-merge the patches (sorry :) )09:18
jistrjaosorior: i'm quite swamped today but i should be able to help on this tomorrow09:19
ramishrashardy: yeah, in my plan for this cycle. But I'm waiting for some reviews to https://review.openstack.org/#/q/status:open+branch:master+topic:bug/1755453 to move foward:)09:19
shardyramishra: ack thanks, will check out the patches09:19
jaosoriorshardy: so, you wanna merge everything into plan-environment.yaml instead of having plan-environment.yaml and user-environment.yaml?09:21
jaosoriorjistr: sure man, no worries. Trying my best not to break you guys :D . But it sure isn't easy to get this working X_x third week working on just this. Help is appreciated.09:22
shardyjaosorior: No we should pass a list of enabled environments then let heat merge them on the server side09:22
shardyjaosorior: currently we merge everything in tripleoclient (and also in the process templates mistral action)09:22
jaosoriorshardy: and that list of enabled environment would be in plan-environment.yaml, right?09:22
shardyjaosorior: yes, that's how the UI works now09:23
jaosoriorOK09:23
jaosoriorshardy: so, the thing is that I need to check a parameter that's in the resource registry https://review.openstack.org/#/c/568736/6/overcloud-resource-registry-puppet.j2.yaml09:23
jaosoriorthat one specifically09:23
shardyjaosorior: I'm not sure if we'll fix that this cycle though so probably best to assume the client side merging for now09:23
jaosoriorI wanted to get that variable from user-environment or plan-environment. But I'm starting to think I should just call swift to get the resource registry and check it there09:24
jaosoriori'm doing this because this needs to work with upgrades. So it's the case where an older overcloud is being managed by a newer tripleo-common. So we wouldn't enable TLS for older releases that don't have this parameter.09:24
shardyjaosorior: which resource_registry would you read from swift though?09:24
shardyYou need the merged output as ordering could change the value09:25
jaosoriorshardy: overcloud-resource-registry-puppet.yaml09:25
shardyjaosorior: but that could be overridden by a user environment?09:25
shardyjaosorior: would it be safer to look at the environment for the existing stack on upgrade?09:25
shardye.g openstack stack environment show overcloud or the mistral equivalent?09:25
shardythat's the only definitive answer about what values are in use09:26
jaosoriorshardy: right, this is what I'm doing right now https://review.openstack.org/#/c/569364/5/tripleo_common/actions/deployment.py (check line 348)09:26
jaosoriorshardy: I'm checking the environment (what the user passes), and it he EnablePublicTLS flag is set we deploy TLS. In older releases this flag shouldn't exist, so it'll default to False, and won't try to deploy TLS09:27
*** elgxl has quit IRC09:27
jaosoriorfor newer releases, the default is True, and you can disable it by setting it to false (that's in the environment that disables TLS)09:27
jaosoriorhowever, we need a way to get the default value for that; right now we only can see that variable if the user sets it09:28
jaosoriorshardy: cause I can't look at the environment on new deployments09:28
shardyjaosorior: isn't the default value for all parameters that exist in the flattened parameters we get from validation?09:28
jaosoriorvalidations is not always enabled09:28
shardyjaosorior: different type of validation09:29
shardywe always validate the plan AFAIK09:29
jaosoriorI don't know :D09:29
*** marios has quit IRC09:29
shardyjaosorior: the flattened parameters stuff I pointed to for derive parameters is always calculated AFAIK09:29
shardybut you can optionally disable some other validations e.g pre-deploy checks via ansible and the like09:29
jaosoriorshardy: wasn't it the case that we don't call that by default? That's the answer I got from yesterday09:30
shardyjaosorior: yes but if you move this to be done in a derive parameters workflow, we'd have to change that and do it by default09:30
*** wolverineav has joined #tripleo09:30
shardye.g add one line to the plan-environment09:30
openstackgerritJose Luis Franco proposed openstack/tripleo-upgrade stable/pike: Separate scripts creation tasks for under/overcloud.  https://review.openstack.org/57035709:30
*** marios has joined #tripleo09:31
jaosoriorso... honestly I don't really understand what that means. As far as I understood from skramaja yesterday, it wouldn't be a good idea to enable it by default because it makes it hardware dependent (that's how they get hardware parameters for NFV)09:31
*** zoli is now known as zoli|lunch09:31
shardyjaosorior: enabling a workflow for your stuff wouldn't be hardware dependent09:31
jaosoriorso, yeah, I haven't enabled that by default because I don't know how, nor what it means to do that.09:31
shardythe point is we don't run all the DPDK stuff by default09:32
*** Tengu has joined #tripleo09:32
jaosoriorshardy: but that's what the derive parameter workflow does09:32
shardyso, atm, we don't run any derive parameters workflows09:32
jaosoriorunless I'm getting very confused here09:32
shardyjaosorior: it's pluggable09:32
jaosoriorwhich I think I am09:32
shardyjaosorior: we can run one or more derive parameters workflows09:32
shardyatm we run zero09:32
skramajawe need to modify the existing workflows so that we avoid most of the instrospection data collection and other stuffs before applying formulas..09:32
shardyand for DPDK you can enable one09:32
shardyand there's another one for Ceph IIRC09:32
*** loacker has joined #tripleo09:32
jaosoriorlets check out the ceph one09:33
shardyskramaja: ack, but in theory we can make this pluggable so "derive parameters" is a general purpose way of including workflows that, uh, derive parameters?09:33
skramajaalso when it is enabled by default, there should be control wit the user to use it or not. for which we could add additional checks along with service presese..09:33
*** elgxl has joined #tripleo09:33
shardylike, for DPDK and ceph we derive parameters based on the introspection data09:33
shardybut in this case we're deriving it from different data09:33
skramajaif we add the vip requirement and no dpdk, getting instrospection data is a waste of effort.09:33
shardybut it's basically the same sequence?09:34
shardyskramaja: sure, we can make that optional09:34
skramajawhat i am saying is that we could tweak the workflow..09:34
shardyskramaja: yeah I think we're agreeing here :)09:34
shardydoes getting the introspection data actually take much time?09:34
shardyprobably depends on the number of nodes I guess09:34
skramajayes.. i agree to invoke by default with few additional changes to the workflow related to nfv and hci invocation..09:34
skramajanot much time. you dont need to force envery one to do introspection because of this..09:35
openstackgerritJose Luis Franco proposed openstack/tripleo-heat-templates stable/pike: DNM: Test undercloud upgrades in stable/pike.  https://review.openstack.org/56626409:35
shardywe probably just need a way to tell if each derive parameters workflow needs introspection data, and calculate a boolean based on which workflows are enabled09:36
jaosoriorTengu: just to catch you up, we're talking with shardy and skramaja about the issue we have, and the idea is that it would be better to do it as part of the "derive_parameters" workflow instead of deploy09:36
Tengujaosorior: hmm ok. I don't know that one, will dig in the doc.09:36
jaosoriorTengu: now, that workflow is not used by default at all, and currently doesn't fit our needs, but it would need to be refactored (as far as I've understood)09:37
Tenguah :)09:37
*** mdnadeem has quit IRC09:37
skramajashardy: yes. something like should be worked out. we could have feature mapping with hw dependecy..09:37
jistrskramaja, shardy: would the derive params workflow take priority over env files?09:37
shardyjaosorior: yeah my assumption was that this was already completely pluggable, but sounds like we may need a few tweaks09:37
skramajano.. user provided env has always the prirority jistr09:38
Tenguso this one right? https://specs.openstack.org/openstack/tripleo-specs/specs/pike/tripleo-derive-parameters.html09:38
jistr++ great09:38
shardyTengu: yes, that was implemented with Ceph and DPDK use-cases in mind, but it's a more general pattern where you need to calculate some parameters on deployment09:38
Tengushardy: hmm ok.09:39
*** cgoncalves is now known as cgoncalves|pto09:40
skramajashardy: https://github.com/openstack/tripleo-common/blob/master/workbooks/derive_params.yaml#L432 currently the features association with role is done here.09:40
skramajafor each role it will check the features associated with it and then get the introspection data.. we need to add a check if we need introspection data or not..09:41
skramajajaosorior: could you provide what are the paraemters that you are looking for to derive?09:41
jaosoriorskramaja: so... this is what we're doing https://review.openstack.org/#/c/569818/09:42
*** pmannidi has quit IRC09:42
jaosoriorskramaja: in a nutshell, if the user didn't provide anything, we get the allocation range for network and set the *FixedIPs parameter.09:43
jaosoriorit could be using network isolation (in which case, we use the external network), or it could be using the ctlplane network (in which case we use ControlFixedIPs)09:43
jaosoriorif the user provided the FixedIPs parameter, then we don't do anything09:43
skramajaok.09:44
jaosoriorfor this we need to get the allocation range for the ctlplane-subnet, and the ports that were already created in that subnet09:44
openstackgerritJuan Antonio Osorio Robles proposed openstack/tripleo-common master: Set FixedIp by default for public-facing VIP  https://review.openstack.org/56981809:47
openstackgerritJuan Antonio Osorio Robles proposed openstack/tripleo-common master: TLS by default for the overcloud for VIPs and FQDNs  https://review.openstack.org/56936409:47
jaosoriorTengu: the capitalized If was actually in the "Fixed IPs" patch ^^09:47
Tengujaosorior: damn.09:47
*** tdasilva has joined #tripleo09:47
jaosoriorskramaja: so... would you have time to make this derive_parameters workfloe default? We could then hook up our stuff there.09:48
*** mdnadeem has joined #tripleo09:50
jistrskramaja: i'm looking at the workflow code, just one question, how do we ensure that the parameter priorities are correct? default < derived < user-provided09:50
skramajayes jaosorior, i will involve the jaganathan to make the required chages as he is the one who is actively working on it. i will drop a mail explaing the expectations so that that is clear..09:51
jistrthe workflow just seems to edit the plan-env parameters directly?09:51
shardyThanks skramaja!09:51
jaosoriorskramaja: that would be awseme, thanks a lot!09:51
skramajajistr: https://github.com/openstack/tripleo-common/blob/2b6efe479eb87999431842b0ac203f9ad03c1092/tripleo_common/utils/plan.py#L14209:51
jaosoriorTengu: lets pospone the TLS by default work for now until we can hook up the TLS and VIP bits to the derive_parameters workflow09:52
Tengujaosorior: fine for me. I'll read a bit more about that.09:52
skramajahere is where the merging is done. the "derived_params" will be stored in the plan_environemnt.yaml and then merged as part of preocess templates..09:52
jaosoriorTengu: that's a good idea.09:52
Tenguand play a bit more with my tmate thingy.09:52
*** avivgt has quit IRC09:54
jistrskramaja: thanks wouldn't the priorities break when we merge shardy's patch? https://review.openstack.org/#/c/44820909:54
jistrshardy, skramaja: IIUC with that patch all user-provided env files from cmdline will end up in `environments` and thus taking lower prio than derived?09:55
jistrsorry for too many Qs i'm trying to build up understanding, as i'm a bit worried about how this all plays out together :)09:55
shardyjistr: doesn't user-environment.yaml already end up in the environments list, just as one big merged environment?09:56
jistrshardy: yes but user env has only resource_registry, no parameter_defaults AFAIK09:57
jistrat least that's what i saw when i poked into the plan... i didn't implement this so my knowledge isn't 100%09:58
jaosoriorjistr: that's correct. It only has the resource_registry09:58
jaosorioruser-provided parameters are in plan-environment.yaml09:58
shardyOk so that's another thing we'd need to fix to enable heat parameter_merge_strategies09:58
shardyin addition to my tripleoclient patch we'd need to rework tripleo-common so we don't do all the merged_params.update stuff09:59
shardytbh one reason I've delayed doing this is because it seems quite risky09:59
shardyso it's good to discuss it09:59
skramajaintially i had plans to write the derive params as separate file directly on the overcloud container..09:59
shardyThe way this should work, IMO, is we should have a list of environments where the ordering defines the precedence in the plan_environment10:00
shardywe shouldn't have any environment data e.g parameters etc merged and stored inside plan_environment10:00
jistrskramaja: i think that could fix it. Having everything in `environments`. `resource-registry-ouvercloud` as first, then `derived parameters` 2nd, then the rest of the files (user provided)10:00
shardythat's beyond the scope of this TLS discussion though I think10:00
shardyjistr: yeah that's what I had in mind10:01
skramajaok. when shardy revivies the patch, we have some work to do :)10:01
*** panda|off is now known as panda10:03
jaosoriorskramaja: in the meantime, the plan is to keep the current approach, right?10:03
jistrshardy: re beyond the scope of discussion, yes but it affects where can we put the TLS code right now to unblock jaosorior. If we know the `envrionments` patch and won't work well (yet) with how we handle derived parameters, i'm not sure if derived parameters is the best place. Or maybe better yet, yes let's use derived_parameters but that means we can't land the `environments` patch as it's currently.10:03
skramajayes jaosorior10:03
shardyjistr: ack, understood - I'm just trying to discuss it so we end up with the logic in a sane place, vs wedging a ton of python code into the deploy action which may be fragile and hard to maintain?10:04
shardynot trying to block but I expected the derive parameters interface to fit this requirement, if it doesn't then that's OK10:04
shardyalso I don't think impact on the environments patch I pushed should be a concern - that's been blocked for months for other reasons so I guess it's fairly low priority10:05
shardyI see it as technical debt which would be good to fix but probably not super critical from a user perspective10:05
jistrshardy: yes totally +1 re trying to bring some order into where we set what, and hopefully derived_parameters being the right place for this type of features10:06
*** wolverineav has quit IRC10:07
*** wolverineav has joined #tripleo10:07
jaosoriorshardy, jistr: It does sound like the best way to go forward, from both a logical and a maintainability point of view10:08
*** wolverin_ has joined #tripleo10:09
*** wolverineav has quit IRC10:09
*** anilvenkata has quit IRC10:09
jistrnow i wonder if we have all the info we need at the point when we call (or will call) the derive_parameters workflow10:10
*** anilvenkata has joined #tripleo10:10
*** wolverin_ has quit IRC10:10
shardyjistr: it's called right before the deploy starts so I assumed we'd have basically all data available10:10
jaosoriorjistr: well, the plan is to always call it (have it as a default). I'm not sure at what point of the deployment that happens though10:10
jaosoriorI see10:10
jistrshardy: that sounds good then10:10
*** wolverineav has joined #tripleo10:10
jistrshardy, jaosorior: we should be able to look into `parameter_defaults` and see the final state of `EnablePublicTLS` in that case, and decide based on that10:11
jaosoriorShould a call to the derive workflow be added to the deploy workflow? that way we can make sure that it gets called in both the CLI and the UI10:11
jaosoriorjistr, skramaja, shardy, Tengu ^^10:11
jistr+110:12
*** elgxl has quit IRC10:12
skramajahttps://github.com/openstack/python-tripleoclient/blob/master/tripleoclient/v1/overcloud_deploy.py#L21410:13
skramajaits currently invoked from cli.. the idea of not integrating with deploy woflow is --10:14
*** elgxl has joined #tripleo10:14
skramajafor ui, we wanted give an option for the user to see the parameters and then start the deployment/..10:14
jistrskramaja, shardy: also would be really nice to refactor it into being in the `environments` list, so that we know what came from where when debugging. I can imagine effectively amending the plan's global parameter_defaults directly without any way to see what came from derive_parameters (by inspecting the plan container later) could lead to some confusing situation + hard debugging.10:14
skramajain case if user want to alter it, he could do that in the UI..  though UI is not integrated it yet..10:15
shardyskramaja: Ok, but right now the derive params stuff just doesn't work via the UI?10:15
*** gfidente has joined #tripleo10:15
*** gfidente has quit IRC10:15
*** gfidente has joined #tripleo10:15
*** wolverineav has quit IRC10:15
shardyjistr: agreed, and that fits well with what I was trying to do for the user environments10:15
jaosoriorskramaja: sure, that's fine. So, for the  UI case... how do we make sure that the derive parameters get run by default?10:15
shardyany help appreciated ;)10:15
jaosoriorskramaja: else we would break the UI, cause the default endpodint map would have TLS endpoints.10:15
skramajaas of now plan-environment.yaml in the container will have the "derived_parameters" section seprately..10:15
jistrskramaja: ah right so that's good from debugging perspective10:16
shardyYeah but it still doesn't help if you have multiple derive parameters workflows10:16
Tengujaosorior: +4210:16
jistrshardy: yea i'd like to :D but we have so much stuff to do :D10:16
shardyit'd be better to have environments have derive-parameters-foo.yaml and derive-parameters-xyz.yaml10:16
skramajashardy: agreed, but what i am saying is that we have the difference today also.. all the derived_params are merged as one..10:17
* jistr currently prioritizing CI for updates/upgrades as lack of that has been a significant pain point for a long time10:17
shardyjistr: sure, just looking for opportunities to make incremental progress vs making more work to do later :)10:17
jistryea i understand :)10:17
skramajathis is needed because if you remember the DPDK and HCI usecases, both will update same prameters..10:17
skramajaso the order of executing the formulas play an importnat role..10:18
*** pkovar has joined #tripleo10:18
openstackgerritRoman Gorshunov proposed openstack/diskimage-builder master: Allow to rebuild arbitrary images  https://review.openstack.org/56869710:20
*** psahoo has quit IRC10:21
*** akrivoka has quit IRC10:25
*** akrivoka has joined #tripleo10:26
*** leanderthal has joined #tripleo10:27
jistrskramaja: what would be the ideal strategy for dealing with that case? merging a list / one of them overriding the other / sth else?  (i'm not familiar with the details)10:27
openstackgerritMerged openstack/instack-undercloud master: Fail in undercloud pre_upgrade if overcloud not in *_COMPLETE state  https://review.openstack.org/56934510:28
openstackgerritMerged openstack/python-tripleoclient master: Drop default inclusion of undercloud.yaml  https://review.openstack.org/56838610:28
skramajajistr: currently we do that. parameters are derived on per role basis and updated using <RoleName>Parameteres in the "derived_parameters" so it will be merged..10:29
skramajafor example NovaHostReservedMemory is done by both DPDK and HCI.10:29
*** dtantsur|afk is now known as dtantsur10:30
jistrack and i guess whatever runs later, wins10:30
jistr?10:30
shardyskramaja: do the values get combined, or is it just appending to lists or updating dicts for the same parameter?10:30
openstackgerritRabi Mishra proposed openstack/instack-undercloud stable/queens: Fail in undercloud pre_upgrade if overcloud not in *_COMPLETE state  https://review.openstack.org/57036710:30
shardybecause one of the motivations of moving to heat server side merging is it supports parameter_merge_strategy, which means you can control merging lists and maps from multiple environments10:31
shardydoesn't help if it's e.g a number or something which needs to be somehow combined tho10:31
skramajashardy: jistr what ever run last will win. in which case the order is important.10:32
shardyskramaja: yes, that's the same as environment merging though10:32
Tenguthe order is always important regarding the env -.-10:33
*** abishop has joined #tripleo10:33
* Tengu remembers being stung a lot of time with ceph, docker and HA env order inclusion -.-.'10:33
Tenguit was nasty bites.10:33
*** zoli|lunch is now known as zoli10:34
skramajathe order is for derived parameters is static and embeded as part of the workflow based on agreement between feature owners.. so it does matter..10:34
shardyskramaja: nobody disagrees with that, we're asking what sort of data is merged10:35
shardyis it e.g 3 merged with 4, where the second workflow knows they have to be added, or a data structure like [3] and [4] where you can reasonably know that lists are always appended?10:36
shardyheat handles the second case, but not the first10:36
*** olap has quit IRC10:38
jistrshardy, skramaja: also i wonder if we have the derive workflows smart enough to say "this is not a HCI env, don't emit any derived params for HCI". If so, i think a lot of the conflicts just wouldn't happen in practice.10:40
skramajashardy: https://github.com/openstack/tripleo-common/blob/master/workbooks/derive_params_formulas.yaml#L53610:40
*** ramishra has quit IRC10:40
jistrshardy, skramaja: and then handling shardy's example of 3, 4 => 7  vs. [3], [4] => [3, 4] personally i'd consider that a stretch goal10:41
*** maufart__ has joined #tripleo10:41
shardyjistr: yeah, my point was really that [3], [4] => [3, 4] (and similar with dicts) can be handled by heat parameter merging10:41
shardyso if we ever fix bug #1635409 we could use that in the list of environments from plan-environment10:42
openstackbug 1635409 in tripleo "parameter_merge_strategies: merge behaves like overwrite" [High,In progress] https://launchpad.net/bugs/1635409 - Assigned to Steven Hardy (shardy)10:42
*** ssbarnea_ has quit IRC10:42
*** aufi_ has quit IRC10:44
skramajai think we will have the goal to write it as environemt file as the next step..10:45
*** mdnadeem has quit IRC10:46
* jistr feels like having somewhat better understanding of the derived param context now10:49
jistrthanks skramaja10:49
* jistr gotta grab lunch before a chain of mtgs :/ tomorrow should be a better day :D10:49
Tengulunch. hmmm.10:51
Tengufancy idea.10:51
*** ramishra has joined #tripleo10:57
*** jpena is now known as jpena|lunch10:58
openstackgerritNir Magnezi proposed openstack/tripleo-heat-templates master: Make lb-mgmt-subnet a class B subnet  https://review.openstack.org/56813811:05
openstackgerritNir Magnezi proposed openstack/tripleo-heat-templates master: Make lb-mgmt-subnet a class B subnet  https://review.openstack.org/56813811:06
*** amoralej is now known as amoralej|off11:07
*** jpena|lunch is now known as jpena|off11:10
*** rh-jelabarre has joined #tripleo11:11
*** ansmith has quit IRC11:17
openstackgerritSylvain Afchain proposed openstack/tripleo-heat-templates master: skydive external_deploy_task migration  https://review.openstack.org/56127411:26
*** bfournie has quit IRC11:27
*** pradk has joined #tripleo11:28
*** ohamada has joined #tripleo11:28
*** udesale_ has joined #tripleo11:33
*** udesale has quit IRC11:35
*** olap has joined #tripleo11:42
*** pdeore__ has quit IRC11:42
*** udesale_ has quit IRC11:42
*** radeks__ has joined #tripleo11:43
*** rasca has quit IRC11:44
*** radeks_ has quit IRC11:45
*** loacker has quit IRC11:51
*** gfidente has quit IRC11:51
*** d0ugal has joined #tripleo11:55
*** gfidente has joined #tripleo11:55
*** gfidente has joined #tripleo11:55
*** d0ugal__ has quit IRC11:57
*** jpena|off is now known as jpena11:58
*** mvenesio has joined #tripleo12:00
*** mburned_out is now known as mburned12:01
*** dbecker has joined #tripleo12:04
*** bfournie has joined #tripleo12:04
* dtantsur LOL at "wow it's actually working"12:05
Tengu:]12:06
Tengudtantsur: my fault. although the context wasn't the CI -.-'.12:07
*** skramaja has quit IRC12:07
Tengu(although.. yes, wow, it IS working :D)12:08
openstackgerritOliver Walsh proposed openstack/instack-undercloud stable/ocata: WIP: prevent duplicate default cells being created by ocata upgrade  https://review.openstack.org/57008112:08
*** Guest19060 is now known as rbrady12:10
*** rbrady has joined #tripleo12:10
*** quiquell is now known as quiquell|lunch12:12
*** b00tcat has quit IRC12:13
*** ratailor has quit IRC12:14
*** tzumainn has joined #tripleo12:15
numansEmilienM, Hi, can you please take a look at this when you get some time - https://review.openstack.org/#/c/523804/12:15
*** waleedm has quit IRC12:15
*** elgxl has quit IRC12:18
sshnaidmjfrancoa, hey, are you aware of http://logs.openstack.org/91/564291/12/check/tripleo-ci-centos-7-undercloud-upgrades/7c33e27/logs/undercloud/home/zuul/undercloud_upgrade.log.txt.gz#_2018-05-24_09_09_32_718 ?12:20
mwhahahajaosorior: hi2u, re: https://review.openstack.org/#/c/565151/12:23
jaosoriormwhahaha: sup12:23
mwhahahajaosorior: i commented but we need to use something else for tls because that adds a requirement that haproxy needs to be on the keystone node12:23
*** lifeless_ has joined #tripleo12:24
*** lifeless has quit IRC12:24
jaosoriormwhahaha: understood12:24
jaosoriormwhahaha: originally that TLS bit was written on the Controller role (before we had composable roles).12:25
jaosoriorwhen composable roles came along, those tags were added to identify where to put the certs12:25
mwhahahajaosorior: the tags were before that but ok :D12:26
jaosoriornot sure what's the reasoning for needing both the controller and primary tags (I didn't add that). but if we can do without the primary tag, I'm fine with that12:26
jaosoriorit would just need to be fixed.12:26
mwhahahaso the problem here is that its' also used for the determination of the bootstrap node which is the keystone node12:27
mwhahahanot the haproxy node12:27
jaosoriormwhahaha: another thing is that I actually have no idea how it used to work before. Because MySQL also needs to be in the primary role in order to bootstrap the mysql users and databases12:27
jaosoriormwhahaha: right, primary tag == role that contains the bootstrap node12:27
mwhahahajaosorior: i have a feeling it broke because of config download and how e determine boostrap node12:28
mwhahahabecause i thought each role had their own bootstrap node12:28
jaosoriormwhahaha: they don't12:28
mwhahahabut that seems to no longer be the case12:28
*** rlandy has joined #tripleo12:28
jaosoriormwhahaha: I talked to shardy about this issue at some point. And he thought it would be better to get bootstrap nodes per-service12:28
jaosoriorI agree with his idea12:28
mwhahahai'll need to look into the queens version to see what was the case previously12:28
mwhahahabecause i think we need a bootstrap determination for each role12:29
jaosoriormwhahaha: it can't be per-role, because a service might be present in several roles12:29
shardythe problem is services can span more than one role, so in general I think a bootstrap node per service is a better pattern?12:29
jaosoriorI think a bootstrap node per service is the way to go12:29
shardyand we already have a bootstrap calculation per service12:30
mwhahahai guess, but we use bootstrap node in the deployment to determine if we run stuff12:30
mwhahahawhich is a problem12:30
mwhahahahttps://bugs.launchpad.net/tripleo/+bug/1768142/comments/412:30
openstackLaunchpad bug 1768142 in tripleo "tripleo-ci-centos-7-3nodes-multinode timing out frequently in the gate" [Critical,In progress] - Assigned to Alex Schultz (alex-schultz)12:30
*** rlandy is now known as rlandy|rover12:30
shardymwhahaha: yeah, we just need to ensure we check if it's a bootstrap node for each service, vs the historical one-and-only controller bootstrap node12:30
jaosoriormwhahaha: yes it is. It ties MySQL, Keystone, and HAProxy to always run together12:30
jaosoriorso, we need to fix it12:30
mwhahahak we should probably document that because that's not good12:30
mwhahahawhy can't we add a mysql tag and a haproxy tag12:31
mwhahahato solve for those12:31
shardywith containers and knowledge of per-service bootstrap nodes, are those services still coupled?12:31
shardymodulo bootstrap node bugs in puppet-tripleo12:31
*** udesale has joined #tripleo12:31
mwhahahayea because of where we run deployment bootstrap bits12:31
jaosoriormwhahaha: mysql also does database/user creation based on the manifest running on a bootstrap node12:31
mwhahahacheck that comment from the bug -^12:31
shardymwhahaha: Ok I assumed that bootstrapping could happen from any node provided it had the DB connection details12:32
mwhahahashardy: yea i think it can, which is why i think switching the tags back to the api role (keystone node) should be ok12:32
mwhahahabut i'm not sure about mysql users, it's working12:32
mwhahahabut it seems that tls adds a new wrinkle12:33
jaosoriorshardy: any node could do the mysql bootstrapping, but we only run the puppet manifest that does the actual bootstrapping from the role containing MySQL12:33
shardyjaosorior: Ok, and that's because the puppet-tripleo stuff looks at the old notion of bootstrap_server_id I guess?12:33
shardyor bootstrap_nodeid rather12:34
arxcruz|ruckmwhahaha: hi, fyi, although everything seems to be running okay, the queue is a little bit big :)12:34
mwhahahaarxcruz|ruck: cool thanks12:34
jaosoriorshardy: yep https://github.com/openstack/puppet-tripleo/blob/master/manifests/profile/base/database/mysql.pp#L17212:34
shardyhttps://github.com/openstack/tripleo-heat-templates/blob/master/overcloud.j2.yaml#L72412:34
shardyjaosorior: Ok, well IIRC short_service_bootstrap_node was supposed to be used instead, but there's probably a lot of places which haven't been updated to do that12:35
jaosoriorright12:36
jaosoriorshardy: then mwhahaha said, TLS is still an issue. Although for that we don't really need the bootstrap part. It would suffice if we just use a haproxy tag12:36
jaosoriorat some point I wanna get rid of the current TLS setup and do it via ansible though. So maybe this is a good excuse to do just that.12:38
*** holser__ has quit IRC12:38
*** holser__ has joined #tripleo12:39
jaosoriormwhahaha: we can do this, we can change the tags back, and I'll work on removing the heat resources dedicated for TLS, and use ansible instead.12:39
*** saneax has joined #tripleo12:39
*** ansmith has joined #tripleo12:39
beaglesshardy: can you have a look at https://review.openstack.org/#/c/567655/ to see if the logic makes sense to you (using the neutron user as a "sentinel value")12:40
jaosoriormwhahaha: of course, if you want to do the ansible bits yourself, that would also be nice :D I'm quite swamped with stuff to do as well :/12:40
mwhahahapfffft12:40
mwhahahai'll just move keystone back to the controller role12:40
mwhahaha:D12:40
beaglesshardy: still need mcornea's okay before merging this but feedback is welcome :)12:41
*** quiquell|lunch is now known as quiquell12:41
jaosoriormwhahaha: :(12:41
*** lifeless_ has quit IRC12:41
mwhahahai don't know why the haproxy stuff needs to be on the primary for tls, is it for determingin the hostname or something?12:42
openstackgerritAlex Schultz proposed openstack/tripleo-heat-templates master: Fix 3node deployment  https://review.openstack.org/57025212:42
openstackgerritMarios Andreou proposed openstack/tripleo-docs master: Minor fixes for Pike to Queens upgrade operator docs  https://review.openstack.org/57038012:43
bogdandoshardy: do you mean short_service_bootstrap_node should have replaced all hiera(bootstrap*) in puppets?12:43
bogdandolike https://github.com/openstack/puppet-tripleo/blob/master/manifests/profile/base/database/mysql.pp#L84 ?12:43
jaosoriormwhahaha: it doesn't, I actually don't know why the primary tag was added there (it's been there for a while). All we need is for this https://github.com/openstack/tripleo-heat-templates/blob/master/puppet/role.role.j2.yaml#L565 to know in what role to render the NodeTLSData12:43
*** lifeless has joined #tripleo12:44
mwhahahajaosorior: because you could have multiple 'controllers'12:44
*** trown|outtypewww is now known as trown12:44
mwhahahajaosorior: but we pick the 'primary'12:44
mwhahahafor bootstrap12:44
jaosoriormwhahaha: a whole role has the primary tag12:44
mwhahahaso i guess maybe the fix would be mark both as 'controller'12:44
mwhahaharight12:44
mwhahahathat's the bootstrap one12:44
mwhahahaso i think it was working previously because that's primarily used for keystone and the rest just works12:45
jaosoriormwhahaha: so every node with that role will have that tag and will get the certificate12:45
mwhahahaw/o any tag designations, the first role from the file becomes the primary node12:45
jaosoriorright12:45
mwhahahajaosorior: so it seems like we need to use a tag for tls12:45
mwhahahaso all nodes taged with tls gets the certs12:45
mwhahahaor something12:45
Tenguwhy not deploy the cert everywhere?12:46
jaosoriormwhahaha: or stop using NodeTLSData (which is what I want to do this release), and get ansible to provision the certs only in the HAProxy service profile12:46
mwhahahaor that12:46
mwhahahaanyway i'm working on figuring out how to fix 3node so we can figure that out later :D12:46
jaosoriorTengu: that cert is the public cert for HAProxy, we don't want the computes to have that cert.12:46
mwhahahaunraveling stuff is always run12:46
Tengujaosorior: mostly because of possible private key leak I guess - I don't see other reason in fact.. ?12:47
mwhahahas/run/fun12:47
shardybogdando: yes that is what I'm saying, but IIRC we only changed a few which were needed to enable composable HA12:47
jaosoriorTengu: correct. It's a bundle actually. Both the cert AND the key.12:47
Tenguand chain12:48
Tengubut ok.12:48
jaosoriorwell yeah, there are the intermediates there.12:48
bogdandoshardy: oh, that coould be a proper fix then! mwhahaha^^12:48
Tenguoh by the way.... any chance to get an integration with let's encrypt? that old topic :D12:48
*** rasca has joined #tripleo12:49
jaosoriorTengu: an alternative to this mess with tags and stuff is to remove this https://github.com/openstack/tripleo-heat-templates/blob/master/puppet/role.role.j2.yaml#L565-L573 entirely. And do the cert provisioning via ansible.12:49
jaosoriorTengu: so, if you want to dig into that, that's definitely very appreciated work :D12:49
Tengujaosorior: :D12:49
jaosoriorTengu: and yes! that would be awesome! Problem is always getting the time :D12:49
Tengujaosorior: well, dropping the TLSNodeData will be... well. complicated with backward compatibility, I guess.12:50
*** pkovar has quit IRC12:50
Tengu*NodeTLSData - anyway.12:50
*** pkovar1 has joined #tripleo12:50
*** pkovar1 is now known as pkovar12:50
jaosoriorTengu: maybe? we would be using the same parameters for the certs/keys12:50
Tengu2s12:50
Tenguah, ok, this isn't a file touched by the user.12:50
*** cylopez has joined #tripleo12:50
*** tvignaud has quit IRC12:51
Tenguso basically we'd drop that one from puppet, and push it somewhere else, with some other name, so that ansible can use the user-provided params.12:51
bogdandoshardy: hm, I see no changes for tripleo puppet at all http://codesearch.openstack.org/?q=short_service_bootstrap_node&i=nope&files=&repos=12:51
*** cshastri has quit IRC12:51
jaosoriorTengu: we would use deploy_step_tasks12:52
jaosoriorwe could even do it via host_prep_tasks12:53
Tengujaosorior: that would also impact the NodeTLSCAData I guess right?12:54
*** tcw has quit IRC12:54
*** pdeore__ has joined #tripleo12:54
jaosoriorTengu: it wouldn't inpact it, there's no dependency there. And NodeTLSCAData runs on every node (gotta persist the CA everywhere)12:54
*** pdeore__ has quit IRC12:54
shardybogdando: the string is e.g "mysql_short_bootstrap_node_name"12:55
Tenguhmm.12:55
jaosoriorTengu: yeah, I think we could do it via host_prep_tasks.12:56
openstackgerritCédric Jeanneret proposed openstack/tripleo-quickstart master: New config in order to get IPv6 with pacemaker on a single-nic isolated net  https://review.openstack.org/57020912:56
Tengushardy: -^ cleaned my config as you commented :).12:56
jfrancoasshnaidm: hey, is that happening in master?12:57
Tengujaosorior: any link to some doc regarding those steps?12:57
sshnaidmjfrancoa, ocata12:57
jaosoriorTengu: not sure. But this is how it looks like https://github.com/openstack/tripleo-heat-templates/blob/master/docker/services/swift-proxy.yaml#L24312:57
*** gfidente has quit IRC12:57
jfrancoasshnaidm: oh, I'll have a look at it. thanks for the notice12:57
sshnaidmjfrancoa, thanks12:58
jfrancoasshnaidm: is there any LP bug for it?12:58
sshnaidmjfrancoa, mm.. not sure12:58
jaosoriorshardy: at what point of the deployment does host_prep_Tasks run?12:58
shardybogdando: see https://review.openstack.org/#/c/421405/ - arguably at that point we should have converted all manifests, not only the pacemaker bits needed to enable composable ha12:58
shardybut if it helps we could do that now and remove the generic bootstrap_node stuff from t-h-t/hiera12:59
sshnaidmjfrancoa, I'll submit12:59
shardyjaosorior: early, before all the deploy steps12:59
shardyhttps://github.com/openstack/tripleo-heat-templates/blob/master/common/deploy-steps.j2#L29113:00
jaosoriorshardy: the ansible tasks don't get stored in the nodes nowadays, right?13:01
jaosoriorthey only get persisted on the undercloud13:01
shardyjaosorior: yup13:02
*** mcornea has joined #tripleo13:02
jaosoriorTengu: so yeah, effectively, both host_prep_tasks and deploy_step_tasks are ansible tasks that get ran on the overcloud hosts if a certain service is enabled.13:02
shardythat template is for the pre-config-download implementation13:02
Tengujaosorior: so in the TLS case, that service would be haproxy in its container I guess?13:02
openstackgerritBrent Eagles proposed openstack/tripleo-heat-templates master: Add acl to paths that are shared among related neutron processes  https://review.openstack.org/56765513:02
jaosoriorTengu: right, so the catch is that it gets run on the host, not the container.13:03
jaosoriorTengu: but so is the TLS setup currently. So that's not a big deal.13:03
jfrancoasshnaidm: thanks a lot, I think I didn't see it before because we were blocked by this other LP https://bugs.launchpad.net/tripleo/+bug/176910713:03
openstackLaunchpad bug 1769107 in tripleo "Undercloud upgrades job failing in stable/ocata" [Medium,Fix released] - Assigned to Jose Luis Franco (jfrancoa)13:03
shardyjaosorior: yeah, we could in theory make host_prep_tasks into step-0 deploy-steps tasks or something, but previously we couldn't because deploy_steps_tasks don't work when not using config-download13:03
*** myoung|off is now known as myoung13:03
Tengujaosorior: not a big issue, as the certificates are pushed on the host, and directory is mounted read-only in the containers13:03
jaosoriorexactly13:03
beaglesmarios: I've updated ^ as per you recommendations - I pushed for review before test deploying though so could you eyeball it for obvious syntax errors13:03
jaosoriorshardy: are we supporting folks turning off config-download?13:04
Tengujaosorior: only thing: ensure haproxy is properly reloaded after we push the certificate.13:04
Tengubut that's already managed I guess.13:04
Tenguand as it's a pre_task, it should be OK13:04
shardyjaosorior: no idea, I assumed it was at least deprecated13:04
*** tcw has joined #tripleo13:04
*** saneax has quit IRC13:05
jaosoriorTengu: there are two options. Either copy the script that's currently run. Or dissect the script and do each task via ansible. But yeah, the script right now reloads HAProxy if the cert is updated.13:05
Tengujaosorior: disecting the script would be the right way I think, for the sake of code reusability (is it even English?)13:06
jaosoriorit is13:06
Tengu:)13:06
Tengujaosorior: so. hm. basically, the change would be to move the TLS thingy in the docker/services/haproxy.yaml?13:09
*** tvignaud has joined #tripleo13:10
jaosoriorTengu: rather to puppet/services/haproxy.yaml, and then be called from the base up to puppet/services/pacemaker/haproxy.yaml docker/services/haproxy.yaml docker/services/pacemaker/haproxy.yaml13:11
*** tvignaud has quit IRC13:12
* Tengu pulling t-h-t master and preparing git grep13:13
*** vpickard_ is now known as vpickard13:14
*** jistr is now known as jistr|mtg13:14
*** toure|gone is now known as toure13:16
*** ykarel is now known as ykarel|away13:17
*** cylopez has left #tripleo13:18
openstackgerritMerged openstack/python-tripleoclient master: Make standalone role name configurable  https://review.openstack.org/56837813:18
openstackgerritMarios Andreou proposed openstack/tripleo-docs master: Add developer docs for Pike to Queens upgrade  https://review.openstack.org/56944313:19
*** gfidente has joined #tripleo13:20
*** zoli is now known as zoli|afk13:20
*** tcw has quit IRC13:21
bogdandomwhahaha, jaosorior, shardy: can we merge https://review.openstack.org/#/c/564135/ please?13:21
bogdandotoo much of rebasing after it gets wf+113:22
bogdandoand please do for https://review.openstack.org/#/c/563988/13:22
bogdandoI think we should not block master merges cuz of stable/foo hickups?13:22
bogdandosshnaidm: btw, that's another price we are now paying all :)13:23
sshnaidmbogdando, all this is *zero* price relative to what we'll pay with branches..13:24
*** links has quit IRC13:24
bogdandojust brought an example, it's really weird we can't merge into rocky when ocata fails13:24
Tengujaosorior: (still trying to understand, sorry) so in fact, we should rewrite this snippet to something in ansible, in the pre_host_tasks? puppet/extraconfig/tls/tls-cert-inject.yaml13:25
*** wolverineav has joined #tripleo13:26
jaosoriorTengu: yep13:26
*** tvignaud has joined #tripleo13:26
Tengudamned13:26
*** radeks_ has joined #tripleo13:26
TenguI can try to convert it as an ansible snippet. That would be a first step I guess.13:27
Tenguand a way for me to train a bit more my ansible.13:27
*** radeks__ has quit IRC13:29
*** radek__ has quit IRC13:30
bogdandosshnaidm: My point is we have a failure-triggering-noise, like intermittent/timeout things for each of jobs, say a probability of N, and the real faulure related to the subject patch, 1-N. Then, adding a two more OVB branched jobs and a two more zuul jobs makes each patch submitted into master highly likely blocked cuz of unrelated noise amplification13:30
bogdandoLet me try to calculate it ... :)13:31
sshnaidmbogdando, what is failure-triggering-noise??13:31
bogdandotimeouts, connectivity errors, all that makes us posting recheck13:31
bogdandoand is not related to real patches being tested13:31
bogdandoand now this noise gets amplified by a 4 times I think13:32
bogdandoso folks will post rechecs now like x4 time more often13:32
bogdandowhich will make zuul queues x4 larger, and time to wait for jobs x4 as well13:33
bogdandothat is what I call 'recheck storms'13:33
bogdandoand is the primary reason I created the jobs dependencies & pipelines topic13:33
*** avivgt has joined #tripleo13:34
bogdandocouldn't fine a better word for that noise sshnaidm13:34
bogdandofind*13:35
bogdandowe can call it distributed systems fee :)13:35
*** amoralej|off is now known as amoralej13:35
*** artom has quit IRC13:37
*** nyechiel_ has joined #tripleo13:39
*** tcw has joined #tripleo13:41
openstackgerritBrent Eagles proposed openstack/tripleo-heat-templates master: Add acl to paths that are shared among related neutron processes  https://review.openstack.org/56765513:44
sshnaidmbogdando, all you described is either infra failure and should be communicated to openstack-infra or product failures13:44
Tengujaosorior: doing the thing in ansible will be fun :). anyway. I call it a day, see you tomorrow!13:44
sshnaidmbogdando, "noise" doesn't exist in CI :)13:44
jaosoriorTengu: have a good one!13:45
sshnaidmbogdando, if it's product failure we need to submit a bug and fix it, no matter related it to patch or not13:45
bogdandosshnaidm: each time elastic recheck posts a comment it is highly likely cuz it saw that noise :)13:45
openstackgerritMerged openstack/python-tripleoclient master: Start standalone.conf  https://review.openstack.org/56947413:45
openstackgerritMerged openstack/python-tripleoclient master: Move undercloud.conf to the undercloud config namespace  https://review.openstack.org/56947713:45
bogdandoI'm mostly concerned of that noise amplification, not the product defects part of the issue13:45
sshnaidmbogdando, elastic recheck is not related here, it posts completely non relevant stuff, I was always against it13:46
bogdandoyou're right for the latter13:46
*** anilvenkata has quit IRC13:46
bogdandotho, the former takes a significant part of time we spend on moving patches forward13:46
bogdandoas well13:46
*** ykarel|away has quit IRC13:46
sshnaidmbogdando, product failures should be resolved, that's purpose of CI - to detect them. CI doesn't invent them..13:47
sshnaidmbogdando, if it's not fixed - it's not CI guilt, it's just a "messenger" :D13:48
openstackgerritMerged openstack/tripleo-common master: Add an "includes" prepare filter to match "excludes"  https://review.openstack.org/56968813:48
sshnaidmbogdando, if you can't move patch forward, it's a time to solve all remaining problems that exist now..13:49
sshnaidmbogdando, and to cover them in additional tests so we can catch it later13:49
sshnaidmbogdando, this is a normal development process as I always thought..13:50
*** udesale has quit IRC13:51
*** udesale has joined #tripleo13:51
sshnaidmjfrancoa, rlandy|rover arxcruz|ruck fyi: https://bugs.launchpad.net/tripleo/+bug/177317913:54
openstackLaunchpad bug 1773179 in tripleo "Undercloud upgrades fails in ocata because nova service failure" [High,Triaged] - Assigned to Ronelle Landy (rlandy)13:54
*** moguimar has quit IRC13:54
*** tesseract-RH has joined #tripleo13:54
jfrancoasshnaidm: I was rising the issue right now in the upgrades scrum call. It looks like the cause is that we don't have stable/ocata branch for tripleo-upgrade. So zuul is downloading master branch13:55
EmilienMnumans: sure13:55
*** tesseract-RH has quit IRC13:55
jfrancoasshnaidm: and the undercloud upgrade process is not the same13:55
*** tesseract-RH has joined #tripleo13:55
sshnaidmjfrancoa, hmm.. so we need to remove this job from ocata?13:55
jfrancoasshnaidm: exactly, that's what I was going to ask you13:55
openstackgerritMerged openstack/tripleo-heat-templates master: Fix upgrade_tasks for TripleO UI  https://review.openstack.org/57023713:55
*** tesseract has quit IRC13:56
sshnaidmjfrancoa, I'll submit a patch and let's discuss it, I want to be sure we don't delete it for mistake..13:56
trozetmwhahaha: can you help this tripleo noob understand what failed in this log http://logs.openstack.org/05/486905/37/experimental/tripleo-ci-centos-7-scenario008-multinode-oooq/cc41774/logs/undercloud/home/zuul/overcloud_deploy.log.txt.gz13:57
jfrancoasshnaidm: great, thanks13:57
rlandy|roversshnaidm: you're addressing the bug?13:58
sshnaidmrlandy|rover, yes13:58
mwhahahatrozet: it seems to have timed out during the deploy?13:58
rlandy|roverk - thanks13:58
*** pchavva has joined #tripleo13:58
trozetmwhahaha: what do you see that tells you that, cause i went over this log like 4x now13:58
mwhahahatrozet: because it didn't finish13:58
mwhahahatrozet: the log just stopped mid deploy13:59
*** derekh has quit IRC13:59
trozetmwhahaha: :/13:59
mwhahahatrozet: it was deploying Overcloud common deploy step tasks 3 which is the middle13:59
*** olivierb- has quit IRC13:59
trozetmwhahaha: so i guess i'll just try again? weird thing is the regular multinode job seems to pass everytime...i wonder if it could be a RAM issue?14:00
trozetmwhahaha: ODL takes up 2.5 GB14:00
mariosbeagles: was in a call ack thanks will check14:00
openstackgerritEmilien Macchi proposed openstack/tripleo-quickstart master: DNM - Revert "fs050: enable upgrade cleanup"  https://review.openstack.org/57039514:00
openstackgerritEmilien Macchi proposed openstack/tripleo-quickstart master: DNM - Revert "fs050: enable upgrade cleanup"  https://review.openstack.org/57039514:00
beaglesmarios: thanks -just as well, I added a line since ;)14:01
*** ohochman has quit IRC14:01
mwhahahatrozet: lol 2.5g?14:01
trozetmwhahaha: yeah java so efficient with garbage collection ya know?14:01
trozetheap for days...or minutes...14:02
jistr|mtggreen sc000 upgrade job from Queens to master! Any takers? please merge https://review.openstack.org/#/c/558425/ and the 2 depends-on patches14:02
*** jistr|mtg is now known as jistr14:02
*** leanderthal has quit IRC14:02
jistrGREEN UPGRADE JOBS FOR EVERYONE14:02
* jistr sellin it14:02
openstackgerritLukas Bezdicka proposed openstack/tripleo-heat-templates master: Upgrades: Refactor httpd_enabled variable  https://review.openstack.org/55578114:03
openstackgerritLukas Bezdicka proposed openstack/tripleo-heat-templates master: Upgrades: Refactor playbooks to set facts  https://review.openstack.org/56229414:03
openstackgerritSagi Shnaidman proposed openstack-infra/tripleo-ci master: Don't run undercloud-upgrades on ocata  https://review.openstack.org/57039814:03
sshnaidmjfrancoa, rlandy|rover ^^14:04
*** leanderthal has joined #tripleo14:04
jfrancoasshnaidm: ack, thanks14:05
trozetmwhahaha: you know how much ram these nodes have? or how to increase it...im looking for examples now14:05
*** lblanchard has joined #tripleo14:05
mwhahahatrozet: probably 8g and no14:06
trozettrown:^^14:06
trozetprobably here im thinking https://review.openstack.org/#/c/500872/11/config/general_config/featureset031.yml14:07
*** slaweq has joined #tripleo14:08
*** marrusl has joined #tripleo14:12
*** olap has quit IRC14:13
*** sai_ has quit IRC14:15
*** rook has quit IRC14:15
*** sai_ has joined #tripleo14:17
*** rook has joined #tripleo14:17
*** rook is now known as Guest1107914:18
*** rpioso|afk is now known as rpioso14:18
*** dbecker has quit IRC14:18
*** edmondsw has joined #tripleo14:20
*** olivierbourdon38 has joined #tripleo14:20
openstackgerritTim Rozet proposed openstack/tripleo-heat-templates master: Add OPNFV scenario environment  https://review.openstack.org/48690514:21
*** derekh has joined #tripleo14:21
*** Guest11079 has quit IRC14:26
*** sri_ has quit IRC14:28
openstackgerritEmilien Macchi proposed openstack/tripleo-quickstart master: Revert "fs050: enable upgrade cleanup"  https://review.openstack.org/57039514:29
*** rook_ has joined #tripleo14:29
*** sri_ has joined #tripleo14:30
*** pblaho has quit IRC14:30
shardytrozet: the multinode jobs use deployed_server: true e.g the nodes already exist and are provided by nodepool14:30
shardytrozet: I don't think you can increase the memory without increasing it for all jobs14:31
trozetshardy: happen to know how much ram?14:31
shardy8G IIRC, sec checking14:31
*** pblaho has joined #tripleo14:32
*** dparkes has quit IRC14:32
shardyhttps://github.com/openstack-infra/project-config/blob/master/nodepool/nl01.openstack.org.yaml#L6214:33
shardytrozet: I think that nodepool stuff is used but I'm not super familiar with it14:34
shardyIf you need more it may have to be via 3rd party coverage I guess14:34
shardyand/or disable services until you have enough spare14:34
trozetshardy: ok we could trigger 3rd party opnfv if this is a blocker.  I just rechecked again lets see if it passes14:35
trozetshardy: thanks for your help14:35
openstackgerritEmilien Macchi proposed openstack/tripleo-heat-templates master: nova-ironic: add support for post-upgrade cleanup  https://review.openstack.org/57024114:36
openstackgerritEmilien Macchi proposed openstack/tripleo-heat-templates master: WIP - add missing services for upgrade cleanup  https://review.openstack.org/57028314:36
shardytrozet: np - I'm not sure how much memory is available for the RDO 3rd party jobs14:36
trozetshardy: i mean not RDO 3rd party, but create a new 3rd party which triggers OPNFV jenkins jobs14:36
shardytrozet: ack yes I guess that would also work, I was looking for an existing job you could make use of14:37
trozetshardy: yeah14:37
*** lvdombrkr has quit IRC14:38
openstackgerritEmilien Macchi proposed openstack/python-tripleoclient master: tripleo_deploy/upgrade: fail of upgrade_tasks don't return 0  https://review.openstack.org/57032114:40
*** dparkes has joined #tripleo14:40
openstackgerritEmilien Macchi proposed openstack/tripleo-heat-templates master: Check container health as part of the deploy  https://review.openstack.org/56915314:41
openstackgerritEmilien Macchi proposed openstack/tripleo-heat-templates master: Check container health as part of the deploy  https://review.openstack.org/56915314:41
*** masco has quit IRC14:42
openstackgerritEmilien Macchi proposed openstack/tripleo-heat-templates master: Parameterized deployment hosts  https://review.openstack.org/56675214:42
*** etingof has quit IRC14:43
*** rleander_ has joined #tripleo14:44
*** mvenesio has quit IRC14:45
*** leitan has joined #tripleo14:45
openstackgerritEmilien Macchi proposed openstack/python-tripleoclient master: WIP: Handle config loading in tripleo deploy  https://review.openstack.org/56953514:45
*** leanderthal has quit IRC14:46
*** tesseract-RH has quit IRC14:47
*** zoli|afk is now known as zoli14:48
*** zoli is now known as zoli|wfh14:48
*** zoli|wfh is now known as zoli14:48
trozetshardy, dsneddon: what is the name of the script in tht to generate the os-net-config script from the nic template?14:49
*** rleander_ has quit IRC14:49
*** jtomasek has joined #tripleo14:55
shardytrozet: https://github.com/openstack/tripleo-heat-templates/blob/master/network/scripts/run-os-net-config.sh14:56
*** artom has joined #tripleo14:56
*** abishop has quit IRC14:56
*** dbecker has joined #tripleo14:57
openstackgerritAlex Schultz proposed openstack/puppet-tripleo master: Update netapp cinder configuration  https://review.openstack.org/57040614:57
*** maufart__ has quit IRC15:00
jfrancoajaosorior: hey, do you have a moment for a doubt regarding Barbican docker images in tripleo-common?15:01
*** slaweq has quit IRC15:01
*** matbu has quit IRC15:01
*** tesseract-RH has joined #tripleo15:02
bogdandoshardy: could you please comment or correct me for https://bugzilla.redhat.com/show_bug.cgi?id=1570938#c13 ?15:03
openstackbugzilla.redhat.com bug 1570938 in python-tripleoclient "[OSP13] Overcloud deployment fail when attempting to call heat-templates not from their default path (/usr/share/openstack-tripleo-heat-templates)." [High,Assigned] - Assigned to dpeacock15:03
*** tesseract-RH has quit IRC15:03
*** dbecker has quit IRC15:03
*** tesseract has joined #tripleo15:03
openstackgerritAlex Schultz proposed openstack/tripleo-heat-templates master: Don't run swift on 3node  https://review.openstack.org/57027515:05
openstackgerritOliver Walsh proposed openstack/instack-undercloud stable/ocata: WIP: prevent duplicate default cells being created by ocata upgrade  https://review.openstack.org/57008115:05
shardybogdando: I'll comment, I think the simpler answer is any environment files which are j2 rendered must be in the --templates path15:09
*** ooolpbot has joined #tripleo15:10
ooolpbotURGENT TRIPLEO TASKS NEED ATTENTION15:10
ooolpbothttps://bugs.launchpad.net/tripleo/+bug/177318815:10
*** ooolpbot has quit IRC15:10
openstackLaunchpad bug 1773188 in tripleo "puppet-tripleo unit tests broken due to recent upstream puppet module cleanup" [Critical,In progress] - Assigned to Alex Schultz (alex-schultz)15:10
openstackgerritAlex Schultz proposed openstack/puppet-tripleo master: Update netapp cinder configuration  https://review.openstack.org/57040615:10
*** matbu has joined #tripleo15:10
*** dxiri has joined #tripleo15:10
openstackgerritAlex Schultz proposed openstack/puppet-tripleo master: Rework GDPR compliant logrotate config  https://review.openstack.org/56997615:11
trozetshardy: yeah i thought there was some script that would render that script wiht the params from the template15:14
*** yamahata has quit IRC15:15
bogdandoshardy: thanks! I also linked related LP15:17
bogdandothat's so tricky...15:17
bogdandototally mind-blowing :D15:17
openstackgerritJose Luis Franco proposed openstack/tripleo-quickstart master: Remove ceph from major upgrade jobs.  https://review.openstack.org/56822915:19
shardybogdando: yeah, it's a historical mistake that we allowed unconstrained -e paths, which causes a lot of complexity in tripleoclient15:19
*** dbecker has joined #tripleo15:19
*** pcaruana has quit IRC15:19
*** rook_ is now known as rook15:20
*** slaweq has joined #tripleo15:25
*** dparkes has quit IRC15:26
*** kopecmartin has quit IRC15:26
*** leanderthal has joined #tripleo15:27
*** CristinaPauna has quit IRC15:29
*** bogdando has quit IRC15:30
*** ccamacho has quit IRC15:31
*** CristinaPauna has joined #tripleo15:32
*** gkadam has quit IRC15:32
*** edmondsw has quit IRC15:33
*** tcw has quit IRC15:34
*** jtomasek has quit IRC15:35
*** ukalifon has quit IRC15:36
*** gvrangan has joined #tripleo15:37
*** gvrangan has quit IRC15:38
openstackgerritEmilien Macchi proposed openstack/tripleo-quickstart master: Revert "Revert "Deploy container-multinode (fs010) with a containerized undercloud""  https://review.openstack.org/56026415:38
EmilienMchandankumar: could you look at my tempest error?15:38
*** janki has quit IRC15:40
*** slaweq has quit IRC15:40
*** gvrangan has joined #tripleo15:41
*** quiquell is now known as quiquell|off15:42
*** rbrady has quit IRC15:42
*** gvrangan has quit IRC15:45
*** gvrangan_ has joined #tripleo15:45
*** mburned is now known as mburned_out15:45
*** matbu has quit IRC15:48
*** ykarel|away has joined #tripleo15:51
*** gkadam has joined #tripleo15:53
pandajistr: do you have something in review for the fs000 full upgrade ?15:53
pandajob*15:53
*** gkadam has quit IRC15:54
jistrpanda: yes, this and the 2 depends-on https://review.openstack.org/#/c/558425/15:54
*** slaweq has joined #tripleo15:54
pandajistr: thanks15:55
*** marios has quit IRC15:55
*** tcw has joined #tripleo15:56
*** sshnaidm has quit IRC15:56
pandajistr: and this is for the job that upgrades both undercloud and overcloud ?15:57
*** edmondsw_ has joined #tripleo15:58
jistrpanda: just overcloud, just keystone + dependencies15:58
jistreven that has a run time between 2:30 and 2:4515:58
*** abishop has joined #tripleo15:58
pandajistr: ok so, did you discuss with was a full upgrade of undercloud and overcloud at all ?15:59
*** edmondsw_ has quit IRC15:59
pandaWes*15:59
*** edmondsw_ has joined #tripleo15:59
*** pkovar has quit IRC15:59
pandajistr: you ever discussed if a similar job was worth and added any benefit ?16:00
*** ykarel|away has quit IRC16:00
*** tcw has quit IRC16:01
*** tcw has joined #tripleo16:01
*** dbecker has quit IRC16:02
jistrpanda: yes we did. It fully validates the upgrade procedure as far as the framework itself is concerned. We already have the same job but for updates, it's gating now.16:02
pandajistr: ok, so you don't have a review for the full undercloud + overcloud job. At this point will need to be put in as third party, there's no way we can fit the current timeout16:03
*** gbarros has joined #tripleo16:04
*** pblaho has quit IRC16:04
*** pblaho has joined #tripleo16:04
*** ramishra has quit IRC16:06
jistrpanda: yes that's fine, not much we can do better there unless we manage to increase the gating time limit. But even if we can just get the sc000 upgrade job gating, it would be good because e.g. we were broken by tls by default, and config-download by default. Both of those breakages would be prevented16:07
*** matbu has joined #tripleo16:08
*** slaweq_ has joined #tripleo16:08
*** raildo has joined #tripleo16:09
*** dmellado has quit IRC16:09
*** avivgt has quit IRC16:09
*** ccamacho has joined #tripleo16:09
*** gouthamr has quit IRC16:09
*** ooolpbot has joined #tripleo16:10
ooolpbotURGENT TRIPLEO TASKS NEED ATTENTION16:10
ooolpbothttps://bugs.launchpad.net/tripleo/+bug/177318816:10
*** ooolpbot has quit IRC16:10
openstackLaunchpad bug 1773188 in tripleo "puppet-tripleo unit tests broken due to recent upstream puppet module cleanup" [Critical,In progress] - Assigned to Alex Schultz (alex-schultz)16:10
*** ohamada has quit IRC16:11
*** slaweq has quit IRC16:11
*** zshi has joined #tripleo16:12
*** stevebaker has quit IRC16:14
*** jaganathan has quit IRC16:19
*** leitan has quit IRC16:21
*** gvrangan_ has quit IRC16:21
*** leitan has joined #tripleo16:22
*** lucasagomes is now known as lucas-afk16:23
*** lucas-afk is now known as lucasagomes16:23
*** pcaruana has joined #tripleo16:23
*** alee has quit IRC16:25
*** gvrangan has joined #tripleo16:26
*** leitan has quit IRC16:26
*** edmondsw has joined #tripleo16:29
*** trown is now known as trown|lunch16:29
openstackgerritThomas Herve proposed openstack/puppet-tripleo master: Automatically configure neutron with designate  https://review.openstack.org/56998716:29
*** edmondsw has quit IRC16:30
*** edmondsw has joined #tripleo16:30
*** ffiore has quit IRC16:31
openstackgerritJohn Fulton proposed openstack/tripleo-heat-templates master: Set default application for Ceph Luminous openstack_pools  https://review.openstack.org/57004316:32
*** edmondsw_ has quit IRC16:32
*** pcaruana has quit IRC16:32
*** edmondsw has quit IRC16:34
*** edmondsw has joined #tripleo16:34
*** leanderthal has quit IRC16:35
*** holser__ has quit IRC16:37
*** slaweq has joined #tripleo16:38
*** slaweq_ has quit IRC16:41
*** zaneb has joined #tripleo16:42
*** lifeless has quit IRC16:43
*** udesale has quit IRC16:44
bandinimeh getting LP timeouts16:45
therveWell I get rdo cloud 50016:47
therveThe cloud is down, let's go home16:47
*** vpickard is now known as vpickard_16:47
*** matbu has quit IRC16:48
*** slaweq has quit IRC16:48
*** gvrangan has quit IRC16:48
*** rbrady has joined #tripleo16:48
*** slaweq has joined #tripleo16:49
openstackgerritMichele Baldessari proposed openstack/puppet-tripleo master: WIP Fix up property names in case of mixed case hostnames  https://review.openstack.org/57041316:49
bandini:)16:49
*** rbrady is now known as Guest329516:49
*** sshnaidm has joined #tripleo16:49
*** sshnaidm is now known as sshnaidm|off16:50
openstackgerritEmilien Macchi proposed openstack/tripleo-heat-templates master: Upgrades: Refactor httpd_enabled variable  https://review.openstack.org/55578116:50
openstackgerritEmilien Macchi proposed openstack/tripleo-heat-templates master: Upgrades: Refactor playbooks to set facts  https://review.openstack.org/56229416:51
Tenguhey guys, PTAL? :) https://review.openstack.org/#/c/570209/ you want to get a proper single-nic IPv6 in quickstart, don't you? :)16:53
openstackgerritAlan Bishop proposed openstack/puppet-tripleo master: Add missing nfs_mount_options for Cinder Netapp backend  https://review.openstack.org/57025616:54
*** zaneb has quit IRC16:55
*** alee has joined #tripleo16:56
*** olivierbourdon38 has quit IRC16:56
*** olivierbourdon38 has joined #tripleo17:00
*** derekh has quit IRC17:00
openstackgerritMichele Baldessari proposed openstack/puppet-tripleo master: WIP Fix up property names in case of mixed case hostnames  https://review.openstack.org/57041317:02
*** edmondsw_ has joined #tripleo17:02
*** amoralej is now known as amoralej|off17:03
*** gouthamr has joined #tripleo17:04
*** matbu has joined #tripleo17:04
*** edmondsw has quit IRC17:06
*** gbarros has quit IRC17:07
*** gvrangan has joined #tripleo17:08
*** slaweq_ has joined #tripleo17:08
*** anilvenkata has joined #tripleo17:10
*** ooolpbot has joined #tripleo17:10
ooolpbotURGENT TRIPLEO TASKS NEED ATTENTION17:10
ooolpbothttps://bugs.launchpad.net/tripleo/+bug/177318817:10
openstackLaunchpad bug 1773188 in tripleo "puppet-tripleo unit tests broken due to recent upstream puppet module cleanup" [Critical,In progress] - Assigned to Alex Schultz (alex-schultz)17:10
*** ooolpbot has quit IRC17:10
cmyster.817:10
*** jfrancoa has quit IRC17:11
*** dtantsur is now known as dtantsur|afk17:11
*** slaweq has quit IRC17:12
openstackgerritEmilien Macchi proposed openstack/puppet-tripleo master: Update netapp cinder configuration  https://review.openstack.org/57040617:13
EmilienMmwhahaha: fixed lint ^17:13
*** fragatin_ has quit IRC17:14
mwhahahaK17:14
*** stevebaker has joined #tripleo17:14
*** zaneb has joined #tripleo17:15
*** slaweq_ has quit IRC17:15
pabelangerEmilienM: where about's are you at the summit? Wanted to drop by and discuss something17:15
EmilienMpabelanger: 22017:16
EmilienMpabelanger: I'm free in 5 min17:17
openstackgerritAlan Bishop proposed openstack/puppet-tripleo master: Add missing nfs_mount_options for Cinder Netapp backend  https://review.openstack.org/57025617:18
openstackgerritMerged openstack/tripleo-heat-templates stable/queens: FFU Set NetworkDeploymentActions CREATE,UPDATE for ffwd-upgrade prepare  https://review.openstack.org/56823617:19
pabelangerEmilienM: okay, just outside17:20
*** zaneb has quit IRC17:22
EmilienMpabelanger: here in 2 min17:22
openstackgerritJames Slagle proposed openstack/tripleo-heat-templates stable/queens: Convert resource_name to number  https://review.openstack.org/57041817:22
*** panda is now known as panda|off17:23
*** tesseract has quit IRC17:25
*** jpena is now known as jpena|off17:28
openstackgerritJames Slagle proposed openstack/tripleo-heat-templates stable/pike: Convert resource_name to number  https://review.openstack.org/57041917:28
*** gvrangan has quit IRC17:28
openstackgerritJames Slagle proposed openstack/tripleo-heat-templates stable/ocata: Convert resource_name to number  https://review.openstack.org/57042017:29
*** edmondsw_ has quit IRC17:31
openstackgerritAlex Schultz proposed openstack/puppet-tripleo master: Rework GDPR compliant logrotate config  https://review.openstack.org/56997617:31
*** edmondsw has joined #tripleo17:31
*** fragatina has joined #tripleo17:35
*** edmondsw has quit IRC17:36
*** fragatin_ has joined #tripleo17:36
*** fragatina has quit IRC17:36
*** dparkes has joined #tripleo17:38
*** gfidente has quit IRC17:39
trozetmwhahaha: just to clarify...multinode-oooq means HA right?17:40
mwhahahatrozet: not necessarily17:40
trozetmwhahaha: well what decides that?17:41
mwhahahatrozet: multinode means 2 nodes in CI (one undercloud, one overcloud)17:41
mwhahahatrozet: so depends on what you mean 'ha'17:41
trozetmwhahaha: HA i mean 3 controllers17:41
mwhahahatrozet: if you mean >1 controller, that's only ovb17:41
trozetmwhahaha: oh, so then there is no reason to even include pacemaker17:41
mwhahahawell not necessarily17:41
mwhahahawe should be able to run 1 controller w/ pacemaker17:42
*** dmacpher has quit IRC17:42
trozetmwhahaha: but theres no reason to17:42
trozetmwhahaha: so i can just remove that from the scenario17:42
*** dmacpher has joined #tripleo17:42
mwhahahaif it's your scenario, sure17:42
*** raildo has quit IRC17:42
trozetmwhahaha: cause ODL isnt managed by pacemaker anyway17:42
mwhahahak17:42
*** gbarros has joined #tripleo17:42
trozetmwhahaha: whats the valeu of using pacemaker in a non-ha deployment for other scenarios?17:43
mwhahahatrozet: we enable it by default downstream17:43
trozetmwhahaha: yeah, i never knew why that was17:43
mwhahahatrozet: so it's a type of configuration that could be use17:43
mwhahahabecause it shouldn't matter17:44
mwhahahawhy do you want to disable it?17:44
mwhahahaperf problems or something?17:44
*** trown|lunch is now known as trown17:44
trozetmwhahaha: well if its not doing anything, why is it running :)17:44
mwhahahabecause it's a different code path17:44
mwhahahaand pulls in different services17:44
trozetmwhahaha: oh so for code path consistencty between deployments17:45
trozeti could see htat17:45
mwhahahait's ok if you want to turn it off in the odl scenario for upstream ci17:45
trozetmwhahaha: is it possible after we get thsi scenario working to make an ovb thing as well so we can have HA and non-ha coverage for ODL?17:46
mwhahahajust as long as you test it somewhere :D17:46
*** alee has quit IRC17:46
mwhahahanot sure, we'd have to check with the ci folks if we can do that17:46
*** beagles is now known as beagles_brb17:46
openstackgerritTim Rozet proposed openstack/tripleo-quickstart master: Updates OpenDaylight feature set 31  https://review.openstack.org/50087217:47
*** gbarros has quit IRC17:47
*** alee has joined #tripleo17:53
*** edmondsw has joined #tripleo17:53
*** slaweq has joined #tripleo17:53
*** dhill has joined #tripleo17:53
*** dhill is now known as Guest9612517:54
*** matbu has quit IRC17:54
*** wolverineav has quit IRC17:55
*** edmondsw has quit IRC17:58
openstackgerritTim Rozet proposed openstack/tripleo-heat-templates master: Add OPNFV scenario environment  https://review.openstack.org/48690518:00
*** loacker has joined #tripleo18:00
*** wolverineav has joined #tripleo18:01
*** psachin has quit IRC18:02
*** wolverineav has quit IRC18:04
*** wolverineav has joined #tripleo18:04
*** itlinux has joined #tripleo18:06
*** zoli is now known as zoli|gone18:08
*** zoli|gone is now known as zoli18:08
*** wolverineav has quit IRC18:09
*** ooolpbot has joined #tripleo18:10
ooolpbotURGENT TRIPLEO TASKS NEED ATTENTION18:10
ooolpbothttps://bugs.launchpad.net/tripleo/+bug/177318818:10
*** ooolpbot has quit IRC18:10
openstackLaunchpad bug 1773188 in tripleo "puppet-tripleo unit tests broken due to recent upstream puppet module cleanup" [Critical,In progress] - Assigned to Emilien Macchi (emilienm)18:10
*** salmankhan has quit IRC18:14
*** loacker has quit IRC18:14
*** slaweq_ has joined #tripleo18:15
openstackgerritJohn Fulton proposed openstack/tripleo-heat-templates master: Set default application for Ceph Luminous openstack_pools  https://review.openstack.org/57004318:16
openstackgerritJohn Fulton proposed openstack/tripleo-heat-templates master: Set default application for Ceph Luminous openstack_pools  https://review.openstack.org/57004318:17
*** slaweq has quit IRC18:18
*** slaweq_ has quit IRC18:20
*** shreshtha has quit IRC18:21
*** leitan has joined #tripleo18:22
*** itlinux has quit IRC18:22
*** matbu has joined #tripleo18:26
openstackgerritEmilien Macchi proposed openstack/tripleo-heat-templates master: Upgrades: Refactor playbooks to set facts  https://review.openstack.org/56229418:26
*** leitan has quit IRC18:26
*** abishop has quit IRC18:29
*** morazi has quit IRC18:31
openstackgerritEmilien Macchi proposed openstack/tripleo-heat-templates master: Upgrades: Refactor playbooks to set facts  https://review.openstack.org/56229418:34
*** anilvenkata has quit IRC18:36
*** rajinir has joined #tripleo18:37
*** beagles_brb is now known as beagles18:40
*** alee has quit IRC18:40
*** leitan has joined #tripleo18:44
*** leitan has joined #tripleo18:45
*** leitan has quit IRC18:45
sshnaidm|offrlandy|rover, trown https://review.openstack.org/#/c/570398/18:45
*** leitan has joined #tripleo18:45
*** leitan has joined #tripleo18:46
*** ansmith has quit IRC18:48
*** slaweq has joined #tripleo18:50
*** slaweq has quit IRC18:50
*** slaweq has joined #tripleo18:50
*** alee has joined #tripleo18:52
*** edmondsw has joined #tripleo18:53
*** wolverineav has joined #tripleo18:57
*** edmondsw has quit IRC18:58
*** slaweq has quit IRC19:02
*** matbu has quit IRC19:06
*** ChanServ has quit IRC19:07
*** ooolpbot has joined #tripleo19:10
ooolpbotURGENT TRIPLEO TASKS NEED ATTENTION19:10
ooolpbothttps://bugs.launchpad.net/tripleo/+bug/177318819:10
*** ooolpbot has quit IRC19:10
openstackLaunchpad bug 1773188 in tripleo "puppet-tripleo unit tests broken due to recent upstream puppet module cleanup" [Critical,In progress] - Assigned to Emilien Macchi (emilienm)19:10
*** ChanServ has joined #tripleo19:12
*** barjavel.freenode.net sets mode: +o ChanServ19:12
*** nyechiel_ has quit IRC19:14
*** gbarros has joined #tripleo19:15
*** dbecker has joined #tripleo19:15
*** Guest96125 has quit IRC19:16
*** suuuper has quit IRC19:18
*** ChanServ has quit IRC19:21
*** zaneb has joined #tripleo19:22
*** bcafarel has quit IRC19:24
*** bcafarel has joined #tripleo19:25
*** gvrangan has joined #tripleo19:25
*** ChanServ has joined #tripleo19:30
*** barjavel.freenode.net sets mode: +o ChanServ19:30
*** melwitt is now known as Guest3219819:33
*** beagles is now known as Guest8785119:34
*** fungi is now known as Guest7769519:34
*** sdake is now known as Guest1473519:34
*** v1k0d3n is now known as Guest4543819:34
*** dmsimard is now known as Guest9444819:34
*** flaper87 is now known as Guest6213919:34
*** honza is now known as Guest6099719:34
*** tristanC is now known as Guest1524819:34
*** redrobot is now known as Guest532719:34
*** vkmc has quit IRC19:34
*** vkmc has joined #tripleo19:34
*** vkmc has quit IRC19:34
*** vkmc has joined #tripleo19:34
*** salmankhan has joined #tripleo19:34
*** slaweq has joined #tripleo19:35
*** alee has quit IRC19:35
*** edmondsw has joined #tripleo19:35
*** Guest77695 is now known as fungi19:35
*** zaneb has quit IRC19:36
*** bandini has quit IRC19:36
*** zshi has quit IRC19:37
*** bandini has joined #tripleo19:38
*** edmondsw has quit IRC19:39
*** slaweq has quit IRC19:39
*** rlandy|rover is now known as rlandy19:42
*** rlandy is now known as rlandy|rover19:43
*** gvrangan has quit IRC19:48
*** raildo has joined #tripleo19:49
*** Guest94448 has quit IRC19:51
*** salmankhan has quit IRC19:51
*** dmsimard has joined #tripleo19:52
*** gvrangan has joined #tripleo19:56
*** salmankhan has joined #tripleo19:56
openstackgerritMerged openstack/tripleo-upgrade master: Pass --ssh-user option during fast forward upgrade run  https://review.openstack.org/57024519:57
*** Guest5327 is now known as redrobot19:57
*** gvrangan has quit IRC19:58
*** gvrangan has joined #tripleo20:00
*** zaneb has joined #tripleo20:00
openstackgerritMarius Cornea proposed openstack/tripleo-upgrade stable/queens: Pass --ssh-user option during fast forward upgrade run  https://review.openstack.org/57043520:01
openstackgerritBob Fournier proposed openstack/tripleo-docs master: Document how to use a composable network with Ironic in Overcloud  https://review.openstack.org/57043720:07
*** zaneb has quit IRC20:07
*** gbarros has quit IRC20:07
*** matbu has joined #tripleo20:09
*** ooolpbot has joined #tripleo20:10
ooolpbotURGENT TRIPLEO TASKS NEED ATTENTION20:10
ooolpbothttps://bugs.launchpad.net/tripleo/+bug/177318820:10
*** ooolpbot has quit IRC20:10
openstackLaunchpad bug 1773188 in tripleo "puppet-tripleo unit tests broken due to recent upstream puppet module cleanup" [Critical,In progress] - Assigned to Emilien Macchi (emilienm)20:10
*** salmankhan has quit IRC20:11
*** dhill has joined #tripleo20:12
*** dhill is now known as Guest6433220:12
*** etingof has joined #tripleo20:13
openstackgerritBob Fournier proposed openstack/tripleo-docs master: Document how to use a composable network with Ironic in Overcloud  https://review.openstack.org/57043720:19
*** slaweq has joined #tripleo20:22
*** dbecker has quit IRC20:24
*** lblanchard has quit IRC20:25
*** trown is now known as trown|outtypewww20:27
openstackgerritMichele Baldessari proposed openstack/puppet-tripleo master: Fix up property names in case of mixed case hostnames  https://review.openstack.org/57041320:32
bandinimandre: ^20:33
*** artom has quit IRC20:34
openstackgerritSteve Baker proposed openstack/ansible-role-tripleo-modify-image master: Test new repo  https://review.openstack.org/57044420:34
openstackgerritBob Fournier proposed openstack/tripleo-docs master: Document how to use a composable network with Ironic in Overcloud  https://review.openstack.org/57043720:35
openstackgerritSteve Baker proposed openstack/ansible-role-tripleo-modify-image master: Add .gitreview file  https://review.openstack.org/57044420:35
*** salmankhan has joined #tripleo20:38
EmilienMbandini: can you review https://review.openstack.org/#/c/570395/ please?20:39
stevebakerEmilienM: hey, do you have the rights to add tripleo groups/users to this repo? https://review.openstack.org/#/admin/groups/1916,members20:43
*** radeks_ has quit IRC20:44
EmilienMstevebaker: no you need to ask infra to add a first contributor20:44
EmilienMstevebaker: let me do it20:44
*** lifeless has joined #tripleo20:44
*** bfournie has quit IRC20:45
bandiniEmilienM: done20:46
stevebakerEmilienM: thanks20:46
*** Guest3295 is now known as rbrady-afk20:46
EmilienMthx20:46
*** Guest64332 has quit IRC20:47
*** raildo has quit IRC20:49
*** slaweq_ has joined #tripleo20:51
*** edmondsw has joined #tripleo20:52
*** zaneb has joined #tripleo20:52
*** zshi has joined #tripleo20:55
*** slaweq has quit IRC20:55
*** Guest87851 is now known as beagles20:57
*** bcafarel has quit IRC20:58
*** bcafarel has joined #tripleo20:59
rlandy|rovermwhahaha: hello - ever come across an error like this (ocata): https://ci.centos.org/artifacts/rdo/jenkins-tripleo-quickstart-promote-ocata-rdo_trunk-minimal-346/undercloud/home/stack/overcloud_deploy.log.gz21:02
EmilienMbandini, stevebaker: can one of you +2 https://review.openstack.org/#/c/570406/ please21:02
EmilienMto unblock gate on this repo21:02
EmilienMthanks21:02
mwhahaharlandy|rover: nope but that looks like neutron errored21:02
stevebakerEmilienM: yep21:03
rlandy|roverhmmm ... happy times21:03
mwhahaharlandy|rover: is it consistent or is the first time we've seen that?21:05
rlandy|rovermwhahaha: I've been debugging the ocata promotion all day - each run gives a different error. This is the first time I have seen that one though21:05
rlandy|roverthe last couple runs all died on the image md521:06
rlandy|roverso we made some progress21:06
mwhahaharlandy|rover: ok seems odd, let me know if you need help21:06
rlandy|roverthe last time we actually got to a deployment we had a failure on 2018-05-24 08:55:03 | 2018-05-24 08:54:59Z [overcloud.Controller.0.SshHostPubKey]: CREATE_FAILED  CREATE aborted21:07
rlandy|roverso I guess I'll run again to see which error we can actually reproduce21:07
rlandy|rovermwhahaha: thanks - will  ping you if/when I have anything reproducible21:07
mwhahaharlandy|rover: create aborted is usually a timeout, wonder if it's a perf problem21:08
rlandy|roverwe won't know until we can get that far again21:08
mwhahaha:D21:08
*** shardy has quit IRC21:08
*** shardy has joined #tripleo21:09
*** ooolpbot has joined #tripleo21:10
ooolpbotURGENT TRIPLEO TASKS NEED ATTENTION21:10
ooolpbothttps://bugs.launchpad.net/tripleo/+bug/177318821:10
*** ooolpbot has quit IRC21:10
openstackLaunchpad bug 1773188 in tripleo "puppet-tripleo unit tests broken due to recent upstream puppet module cleanup" [Critical,In progress] - Assigned to Emilien Macchi (emilienm)21:10
*** beagles is now known as beagles_biab21:14
*** salmankhan has quit IRC21:15
*** salmankhan has joined #tripleo21:20
*** slaweq has joined #tripleo21:21
*** edmondsw_ has joined #tripleo21:22
*** matbu has quit IRC21:23
*** slaweq_ has quit IRC21:24
*** edmondsw has quit IRC21:26
*** edmondsw_ is now known as edmondsw21:26
*** leitan has quit IRC21:27
openstackgerritTim Rozet proposed openstack/tripleo-heat-templates master: Add OPNFV scenario environment  https://review.openstack.org/48690521:28
*** marrusl has quit IRC21:33
*** lifeless_ has joined #tripleo21:34
*** lifeless has quit IRC21:34
mwhahahahmm seeing errors with fs001 on master, wonder if related to the rdo maintenance earlier  ResourceInError: resources.Controller: Went to status ERROR due to "Message: No valid host was found. , Code: 500"21:37
mwhahahahttps://logs.rdoproject.org/19/566419/8/openstack-check/gate-tripleo-ci-centos-7-ovb-3ctlr_1comp-featureset001-master/Z3cca71b2091045b1b46f3d4f38a9e729/undercloud/home/jenkins/overcloud_deploy.log.txt.gz#_2018-05-24_21_11_3221:37
*** akrivoka has quit IRC21:38
*** alee has joined #tripleo21:38
*** nyechiel_ has joined #tripleo21:39
*** salmankhan has quit IRC21:40
*** zaneb has quit IRC21:41
*** zaneb has joined #tripleo21:41
*** beagles_biab is now known as beagles21:45
openstackgerritAlex Schultz proposed openstack/tripleo-puppet-elements master: Additional OpenStack cleanup  https://review.openstack.org/56724021:47
*** wolverineav has quit IRC21:48
*** wolverineav has joined #tripleo21:49
*** salmankhan has joined #tripleo21:50
*** slaweq_ has joined #tripleo21:51
*** edmondsw_ has joined #tripleo21:52
*** wolverineav has quit IRC21:53
*** slaweq has quit IRC21:54
*** edmondsw has quit IRC21:56
*** salmankhan has quit IRC21:56
openstackgerritAlex Schultz proposed openstack/tripleo-puppet-elements master: Remove setup-puppet element  https://review.openstack.org/57046221:57
*** slaweq_ has quit IRC21:57
openstackgerritAlex Schultz proposed openstack/tripleo-puppet-elements master: Remove sensu, fluentd, collectd from the images  https://review.openstack.org/57046422:01
*** bfournie has joined #tripleo22:01
mwhahahapradk: please review -^ when you get a chance22:01
pradkwill do22:02
*** gvrangan has quit IRC22:07
*** rajinir has quit IRC22:07
*** edmondsw_ has quit IRC22:08
*** edmondsw has joined #tripleo22:09
openstackgerritAlex Schultz proposed openstack/tripleo-puppet-elements master: Remove opvercloud-cinder-volume element  https://review.openstack.org/57046522:10
*** ooolpbot has joined #tripleo22:10
ooolpbotURGENT TRIPLEO TASKS NEED ATTENTION22:10
ooolpbothttps://bugs.launchpad.net/tripleo/+bug/177318822:10
*** ooolpbot has quit IRC22:10
openstackLaunchpad bug 1773188 in tripleo "puppet-tripleo unit tests broken due to recent upstream puppet module cleanup" [Critical,In progress] - Assigned to Emilien Macchi (emilienm)22:10
*** edmondsw has quit IRC22:10
*** edmondsw has joined #tripleo22:10
*** slaweq has joined #tripleo22:10
*** rcernin has joined #tripleo22:11
*** artom has joined #tripleo22:13
*** slaweq has quit IRC22:15
*** salmankhan has joined #tripleo22:18
*** paramite_ has quit IRC22:21
*** edmondsw_ has joined #tripleo22:23
*** edmondsw_ has quit IRC22:24
*** salmankhan has quit IRC22:24
*** edmondsw_ has joined #tripleo22:26
*** edmondsw has quit IRC22:27
*** zaneb has quit IRC22:27
*** wolverineav has joined #tripleo22:28
*** alee has quit IRC22:28
*** edmondsw_ has quit IRC22:30
*** loacker has joined #tripleo22:31
openstackgerritEmilien Macchi proposed openstack/tripleo-heat-templates master: Remove UpgradeRemoveUnusedPackages from undercloud-upgrade-prepare.yaml  https://review.openstack.org/56932322:31
*** zaneb has joined #tripleo22:31
openstackgerritEmilien Macchi proposed openstack/tripleo-heat-templates master: Remove UpgradeRemoveUnusedPackages from undercloud-upgrade-prepare.yaml  https://review.openstack.org/56932322:31
*** lifeless_ has quit IRC22:34
openstackgerritJohn Fulton proposed openstack/tripleo-quickstart master: Update featureset024 to use ceph-ansible for Pike  https://review.openstack.org/57046922:42
*** matbu has joined #tripleo22:42
openstackgerritJohn Fulton proposed openstack/tripleo-heat-templates stable/pike: WIP: Add CI/development block devices environment files  https://review.openstack.org/57047322:54
*** nyechiel_ has quit IRC22:54
*** pchavva has quit IRC22:54
*** edmondsw has joined #tripleo22:56
openstackgerritMerged openstack/tripleo-common master: Fix Octavia amphora image RPM install  https://review.openstack.org/57016323:00
openstackgerritMerged openstack/tripleo-common stable/queens: Add a 'trash_output' flag to ansible playbook action  https://review.openstack.org/56688423:00
openstackgerritMerged openstack/python-tripleoclient master: Store UC hosts/portmaps heat params in output_dir  https://review.openstack.org/56413523:00
openstackgerritMerged openstack/tripleo-upgrade stable/pike: Separate scripts creation tasks for under/overcloud.  https://review.openstack.org/57035723:00
*** edmondsw has quit IRC23:00
*** nyechiel_ has joined #tripleo23:09
*** pmannidi has joined #tripleo23:09
*** ooolpbot has joined #tripleo23:10
ooolpbotURGENT TRIPLEO TASKS NEED ATTENTION23:10
ooolpbothttps://bugs.launchpad.net/tripleo/+bug/177318823:10
*** ooolpbot has quit IRC23:10
openstackLaunchpad bug 1773188 in tripleo "puppet-tripleo unit tests broken due to recent upstream puppet module cleanup" [Critical,In progress] - Assigned to Emilien Macchi (emilienm)23:10
*** matbu has quit IRC23:11
EmilienMrlandy|rover: can we land https://review.openstack.org/#/c/568946/ ?23:13
*** nyechiel_ has quit IRC23:13
*** dhill_ has joined #tripleo23:15
openstackgerritEmilien Macchi proposed openstack/tripleo-upgrade master: DNM, test  https://review.openstack.org/56873223:15
openstackgerritEmilien Macchi proposed openstack/tripleo-quickstart-extras master: execute the build / install of zuul changes in upgrade  https://review.openstack.org/56894623:16
openstackgerritEmilien Macchi proposed openstack-infra/tripleo-ci master: Add dry run option to toci_quickstart  https://review.openstack.org/56706023:16
openstackgerritEmilien Macchi proposed openstack/python-tripleoclient master: DNM Testing releases script upgrades with zuul change  https://review.openstack.org/56922923:16
EmilienMtrown|outtypewww, rlandy|rover: I rebased and retested ^23:16
EmilienMto see how it works, last test was May 18th!23:17
*** mcornea has quit IRC23:17
*** gbarros has joined #tripleo23:18
rlandy|roverEmilienM: checking dependent patches23:19
rlandy|roveroh ok - https://review.openstack.org/#/c/567060/ is the painful patch we can't get through23:20
rlandy|roverI'm ok with landing  https://review.openstack.org/#/c/568946/23:20
rlandy|roverthe other CI people may complain - we'll see23:20
*** matbu has joined #tripleo23:21
*** gbarros has quit IRC23:22
rlandy|roverEmilienM: got to run out for a bit - will check that testing when I get back23:24
*** rlandy|rover is now known as rlandy|rvr|bbl23:25
*** zaneb has quit IRC23:27
*** matbu has quit IRC23:30
*** jtomasek has joined #tripleo23:32
*** lifeless has joined #tripleo23:35
*** matbu has joined #tripleo23:36
*** alee has joined #tripleo23:40
*** gbarros has joined #tripleo23:41
EmilienMrlandy|rvr|bbl: I don't want CI people to complain and hate me :D23:44
* EmilienM wants peace23:44
*** gbarros has quit IRC23:44
*** pradk has quit IRC23:50

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