Tuesday, 2018-11-27

openstackgerritRedHat RDO CI proposed openstack/tripleo-heat-templates stable/pike: GATE CHECK for TripleO  https://review.openstack.org/60224800:00
openstackgerritRedHat RDO CI proposed openstack/tripleo-heat-templates master: GATE CHECK for TripleO  https://review.openstack.org/60429800:00
*** wolverineav has quit IRC00:04
*** wolverineav has joined #tripleo00:08
*** slaweq has joined #tripleo00:16
*** tosky has quit IRC00:16
*** boazel has joined #tripleo00:18
openstackgerritEmilien Macchi proposed openstack/tripleo-heat-templates master: upgrade: remove Docker containers now managed by Podman  https://review.openstack.org/61520900:20
*** boazel_ has quit IRC00:21
*** slaweq has quit IRC00:24
*** bfournie has joined #tripleo00:34
openstackgerritEmilien Macchi proposed openstack/tripleo-common master: image_uploader: fail if registry isn't reachable  https://review.openstack.org/62011700:37
openstackgerritMerged openstack/tripleo-heat-templates stable/rocky: Add customized libvirt-guests unit file to properly shutdown instances  https://review.openstack.org/61877100:39
openstackgerritEmilien Macchi proposed openstack/tripleo-common master: image_uploader: fail if registry isn't reachable  https://review.openstack.org/62011700:53
*** sai_p has quit IRC00:55
*** slaweq has joined #tripleo01:16
*** slaweq has quit IRC01:24
*** wolverin_ has joined #tripleo01:41
*** wolverin_ has quit IRC01:42
*** wolverineav has quit IRC01:42
*** wolverineav has joined #tripleo01:42
*** aedc has quit IRC01:45
*** slaweq has joined #tripleo02:13
*** wolverineav has quit IRC02:16
*** wolverineav has joined #tripleo02:16
*** wolverineav has quit IRC02:16
openstackgerritIan Wienand proposed openstack/diskimage-builder master: Capture ramdisk test run logs  https://review.openstack.org/61940002:16
openstackgerritIan Wienand proposed openstack/diskimage-builder master: Fix unit tests for elements  https://review.openstack.org/61938702:16
openstackgerritIan Wienand proposed openstack/diskimage-builder master: package-installs: provide for skip from env var  https://review.openstack.org/61911902:16
openstackgerritIan Wienand proposed openstack/diskimage-builder master: simple-init: allow for NetworkManager support  https://review.openstack.org/61912002:16
openstackgerritIan Wienand proposed openstack/diskimage-builder master: Make tripleo-buildimage-overcloud-full-centos-7 non-voting  https://review.openstack.org/62020002:16
openstackgerritIan Wienand proposed openstack/diskimage-builder master: Revert "Make tripleo-buildimage-overcloud-full-centos-7 non-voting"  https://review.openstack.org/62020102:16
*** wolverineav has joined #tripleo02:17
*** wolverineav has quit IRC02:21
*** ramishra has joined #tripleo02:22
*** mrsoul has joined #tripleo02:23
*** slaweq has quit IRC02:24
*** mschuppert has quit IRC02:27
*** psachin has joined #tripleo02:40
*** Baggypants12000 has quit IRC02:47
*** agopi has joined #tripleo02:49
*** eck` is now known as eck`gone03:11
openstackgerritSteve Baker proposed openstack/os-apply-config stable/pike: fixes how os-apply-config handles invalid json  https://review.openstack.org/62020703:11
*** slaweq has joined #tripleo03:16
*** artom has quit IRC03:21
*** artom has joined #tripleo03:24
*** slaweq has quit IRC03:24
*** dsneddon has quit IRC03:24
openstackgerritAlan Bishop proposed openstack/tripleo-heat-templates stable/queens: Configure cinder's access to the nova API  https://review.openstack.org/62020903:26
*** mcornea has quit IRC03:50
*** dsneddon has joined #tripleo03:51
*** udesale has joined #tripleo03:54
*** dsneddon has quit IRC03:56
*** ykarel|away has joined #tripleo03:57
*** ykarel|away is now known as ykarel04:06
*** therve has quit IRC04:08
*** slaweq has joined #tripleo04:11
*** rh-jelabarre has quit IRC04:17
*** rh-jelabarre has joined #tripleo04:20
*** slaweq has quit IRC04:24
*** udesale has quit IRC04:31
*** ykarel has quit IRC04:46
*** dsneddon has joined #tripleo04:46
*** janki has joined #tripleo04:49
openstackgerritJanki Chhatbar proposed openstack/puppet-tripleo stable/queens: Pass variable from puppet-tripleo to puppet-neutron  https://review.openstack.org/62001304:55
*** dsneddon has quit IRC04:56
*** dsneddon has joined #tripleo05:01
*** ykarel has joined #tripleo05:03
*** dsneddon has quit IRC05:06
*** bshephar has joined #tripleo05:16
*** chandan_kumar has joined #tripleo05:26
*** chandan_kumar is now known as chkumar|ruck05:28
Tenguhello there :)05:31
*** dsneddon has joined #tripleo05:31
*** ratailor has joined #tripleo05:39
*** hakhande_ has joined #tripleo05:41
*** dsneddon has quit IRC05:41
openstackgerritMerged openstack/tripleo-heat-templates master: Stop upgrade if a task on one node fails  https://review.openstack.org/61926505:42
openstackgerritMerged openstack/tripleo-heat-templates master: Added Dell EMC SC multipath support  https://review.openstack.org/61245105:42
*** slaweq has joined #tripleo06:11
openstackgerritCarlos Camacho proposed openstack/instack-undercloud stable/rocky: Fix instack-undercloud nits  https://review.openstack.org/61946706:12
*** tzumainn has quit IRC06:20
openstackgerritzhongshengping proposed openstack/puppet-pacemaker master: Deprecate include_class  https://review.openstack.org/62021906:22
*** slaweq has quit IRC06:24
*** ccamacho has quit IRC06:30
*** dsneddon has joined #tripleo06:37
*** dsneddon has quit IRC06:42
jankiHi Tengu :)06:51
Tenguhi janki :)06:51
jankijamesdenton, jistr marios help merge this cherry-pick https://review.openstack.org/#/c/612238/06:52
*** ksambor has joined #tripleo06:56
*** rcernin has quit IRC06:58
*** bshephar has quit IRC07:02
*** ahosam has joined #tripleo07:02
*** dsneddon has joined #tripleo07:05
*** jfrancoa has joined #tripleo07:06
*** mschuppert has joined #tripleo07:07
openstackgerritGael Chamoulaud proposed openstack/tripleo-validations stable/rocky: Check the keystone token flush on the keystone_cron container  https://review.openstack.org/62022507:12
openstackgerritGael Chamoulaud proposed openstack/tripleo-validations stable/rocky: Fix containerized services task  https://review.openstack.org/62022707:14
*** ahosam has quit IRC07:16
*** pcaruana has joined #tripleo07:24
openstackgerritVieri proposed openstack/tripleo-validations master: Add Python 3.6 classifier to setup.cfg  https://review.openstack.org/62023207:24
*** kopecmartin|off is now known as kopecmartin07:26
openstackgerritHarald Jensås proposed openstack/tripleo-heat-templates master: Add Storage network to IronicConductor role  https://review.openstack.org/61798707:27
openstackgerritRadoslaw Smigielski proposed openstack/tripleo-common master: Fix AnsibleAction.run() when argument is not a dict  https://review.openstack.org/62023607:28
*** yprokule has joined #tripleo07:29
openstackgerritRadoslaw Smigielski proposed openstack/tripleo-common master: Fix non-exiting link to TripleO-common bugs  https://review.openstack.org/61999907:30
*** cylopez has joined #tripleo07:30
*** cylopez has left #tripleo07:30
*** quiquell has joined #tripleo07:33
*** hjensas has joined #tripleo07:34
openstackgerritQuique Llorente proposed openstack-infra/tripleo-ci master: Simplify regex removing capturing groups  https://review.openstack.org/61948807:35
*** ykarel is now known as ykarel|lunch07:35
mschuppertjaosorior: morning, if possible could you please review https://review.openstack.org/619586 again. I have addressed your comment.07:39
*** slaweq has joined #tripleo07:43
*** therve has joined #tripleo07:43
openstackgerritgaobin proposed openstack/tripleo-docs master: modify judgment logic  https://review.openstack.org/62024707:53
*** ccamacho has joined #tripleo07:56
openstackgerritCarlos Goncalves proposed openstack/puppet-tripleo master: Include octavia::controller  https://review.openstack.org/62011807:56
openstackgerritCarlos Goncalves proposed openstack/tripleo-heat-templates master: Fix Octavia hieradata keys  https://review.openstack.org/62010607:57
openstackgerritCarlos Camacho proposed openstack/instack-undercloud stable/rocky: Fix instack-undercloud nits  https://review.openstack.org/61946707:59
openstackgerritRedHat RDO CI proposed openstack/tripleo-heat-templates master: GATE CHECK for TripleO  https://review.openstack.org/60429808:00
openstackgerritRedHat RDO CI proposed openstack/tripleo-heat-templates stable/rocky: GATE CHECK for TripleO  https://review.openstack.org/60429308:00
*** saneax has joined #tripleo08:03
*** cfontain has joined #tripleo08:04
*** gkadam has joined #tripleo08:08
*** cfontain has quit IRC08:09
*** florianf|afk is now known as florianf08:13
openstackgerritQuique Llorente proposed openstack-infra/tripleo-ci master: Simplify regex removing capturing groups  https://review.openstack.org/61948808:14
mariosfultonj: sshnaidm|afk panda|rover quiquell rfolco weshay strongly considering +A here any thoughts/objections please. https://review.openstack.org/#/c/616872/08:16
mariosquiquell: and this? https://review.openstack.org/#/c/619468/3 but you need recheck08:17
mariosah you did already08:17
*** nawar has joined #tripleo08:17
nawarhi guys08:18
quiquellmarios: What jobs where isuing the ceph thing apart from standalone scn1|4 ?08:18
quiquellmultinode scn1|4 ?08:18
nawarnew problem :( with deploying, I'm trying network isolation with multiple-nics but I'm getting this error with keepalived "Error: Evaluation Error: Error while evaluating a Function Call, invalid address:  at /etc/puppet/modules/tripleo/manifests/profile/base/keepalived.pp:66:38 on node overcloud-contro08:19
nawarller-0.localdomain", I'm not configuring ha, it's simple deployment controller with computeHCI08:19
quiquellmarios: workflowed the ceph stuff08:20
quiquellmarios: about https://review.openstack.org/#/c/619468 you can workflow it after we have zuul blessing08:22
quiquellmarios: Let's do reviewing of standalone scenarios stuff and merge us much as possible08:22
*** jtomasek has joined #tripleo08:22
mariosquiquell: ah right yeah that too...08:23
*** dsneddon has quit IRC08:23
mariosquiquell: thought you meant 'lets discuss this afternoon about the container cli override because of the featureset stuff' i didn't read your sentence properly :) ignore previous08:24
*** cfontain has joined #tripleo08:24
quiquellmarios: We are possitive on you featureset_override ? did everything change after I drop ?08:25
*** shardy has joined #tripleo08:25
quiquells/you/using/g08:25
mariosquiquell: no we aren't08:25
quiquellmarios: Yoiu can workflow scenario001 env file https://review.openstack.org/#/c/616592/08:25
quiquellmarios: nope ?08:25
quiquelldamn...08:25
mariosquiquell: we will continue discussion this afternoon with stakeholders/anyone intereseted. invited the ceph folks too.08:25
mariosquiquell: and we make a call08:25
quiquellmarios: ok then we don't merge the container_cli override08:26
mariosquiquell: right thats what i was trying to decide08:26
quiquellmarios: but we can merge the env files, they are deploying correctly08:26
quiquellmarios: lint stuff is different08:26
mariosquiquell: like its only useful for featureset_override, otherwise we have to pass it another way08:26
quiquellmarios: yep we cannot merge jobs or featureset_override fixes08:26
quiquellmarios: but we can merge env files08:26
mariosquiquell: replied to matbu here with more context https://review.rdoproject.org/r/#/c/17496/308:26
mariosquiquell: so i think it makes sense to hold the env files a bit please08:27
mariosquiquell: because we didn't even get 1 green run yet. we need that first, then we can sanity check we got all the services we expect in that scenario08:27
mariosquiquell: we didn't validate that yet. i think they are easy to merge anyway08:28
mariosquiquell: agree/disagree?08:28
quiquellmarios: we can merge then and fix later08:28
quiquellmarios: going to easy the depends-on hell08:28
quiquellmarios: as long as we dont break the voting jobs08:28
mariosquiquell: yeah i guess that's right but i'd rather see it run green once at least before i push to merge .. though i'm up for adding the +2s and lining up the ci runs etc08:30
openstackgerritgaobin proposed openstack/tripleo-docs master: fixed Useless variable declaration  https://review.openstack.org/62025608:30
mariosmatbu: did you see my comment here https://review.rdoproject.org/r/#/c/17496/3 the issue we have is specific to standalone they bypass the 'usual' tripleo-common mistral action with the module_path08:31
*** ykarel|lunch is now known as ykarel08:32
*** paramite has joined #tripleo08:32
matbumarios: hey o/08:33
matbumarios: yep i was checking my email, i didn't click on the link08:33
matbumarios: but thanks for the answer :)08:33
quiquellmarios, matbu: it right that this is a big change that have to be tested, maybe is just better to add the --module-path to python-tripleoclient calls08:35
quiquellmatbu: more or less the issue is that we are not using mistral at standalone08:35
quiquellmatbu: so it does not read module_path var08:35
mariosquiquell: yes that sounds right08:35
mariosquiquell: add here https://github.com/openstack/python-tripleoclient/blob/master/tripleoclient/v1/tripleo_deploy.py#L816 well in here anyway08:36
matbumarios: quiquell yep maybe safer08:37
mariosmarios recaffeine08:38
mariosbrb08:38
*** tosky has joined #tripleo08:38
*** Baggypants12000 has joined #tripleo08:39
openstackgerritCarlos Camacho proposed openstack/instack-undercloud stable/rocky: Fix instack-undercloud nits  https://review.openstack.org/61946708:40
*** cfontain has quit IRC08:43
*** cfontain has joined #tripleo08:44
openstackgerritNatal Ngétal proposed openstack/tripleo-common stable/queens: [Queens only] Fix the default for docker registry.  https://review.openstack.org/61132908:47
*** dsneddon has joined #tripleo08:47
*** jaganathan has joined #tripleo08:48
*** cfontain has quit IRC08:48
*** jpena|off is now known as jpena08:51
shardyHey folks, if anyone has time to check https://review.openstack.org/#/c/605492/ that would be great, next in the series reworking bootstrap_node things and finally unblocked/passing CI \o/08:59
*** amoralej|off is now known as amoralej09:01
*** xek has joined #tripleo09:01
*** jpich has joined #tripleo09:05
*** gfidente has joined #tripleo09:05
*** udesale has joined #tripleo09:09
*** holser_ has joined #tripleo09:09
*** holser_ has quit IRC09:10
*** holser_ has joined #tripleo09:10
openstackgerritJose Luis Franco proposed openstack/tripleo-upgrade master: Avoid adding ContainerImagePrepare in roles_data twice.  https://review.openstack.org/62026209:12
*** ahosam has joined #tripleo09:15
mschupperthi, can I please get reviews on https://review.openstack.org/619586 , this is related to a blocker.09:18
*** dmacpher has joined #tripleo09:19
*** suuuper has joined #tripleo09:20
*** cfontain has joined #tripleo09:20
*** ccamacho has quit IRC09:20
*** bogdando has joined #tripleo09:23
Tenguhmm, gerrit seems slow.09:28
*** skramaja has joined #tripleo09:30
*** ccamacho has joined #tripleo09:30
openstackgerritCarlos Camacho proposed openstack/instack-undercloud stable/rocky: Fix instack-undercloud nits  https://review.openstack.org/61946709:34
openstackgerritJose Luis Franco proposed openstack/tripleo-common master: Fix ironic-pxe healthcheck.  https://review.openstack.org/61772809:35
*** derekh has joined #tripleo09:40
*** cfontain has quit IRC09:41
*** xek has quit IRC09:44
*** xek has joined #tripleo09:48
openstackgerritCarlos Camacho proposed openstack/instack-undercloud stable/rocky: Fix instack-undercloud nits  https://review.openstack.org/61946709:51
*** aedc has joined #tripleo09:51
*** hakhande_ has quit IRC09:53
*** hakhande_ has joined #tripleo09:53
*** dsneddon has quit IRC09:54
openstackgerritSorin Sbarnea proposed openstack/tripleo-quickstart-extras master: Fix failure to install atop on fedora28  https://review.openstack.org/61959510:02
ykarelssbarnea|bkp2, so seeing the existing implementation it looks like it wanted to install from EPEL repo10:07
ykarelwhich is rarely enabled in all jobs10:07
ykarelsaying by seeing: atop_location=$(repoquery --repoid=osepel --repofrompath=osepel,$NODEPOOL_EPEL_MIRROR/7/x86_64/ --location atop)10:07
nawarwhy I'm getting this error with keepalived "Error: Evaluation Error: Error while evaluating a Function Call, invalid address:  at /etc/puppet/modules/tripleo/manifests/profile/base/keepalived.pp:66:3810:07
openstackgerritSorin Sbarnea proposed openstack/tripleo-quickstart-extras master: Use include_role on tempest to avoid misleading output  https://review.openstack.org/62014610:10
openstackgerritSorin Sbarnea proposed openstack/tripleo-quickstart-extras master: Use include_role on tempest to avoid misleading output  https://review.openstack.org/62014610:12
*** akrivoka has joined #tripleo10:13
*** xek has quit IRC10:18
openstackgerritSorin Sbarnea proposed openstack/tripleo-quickstart-extras master: Fix failure to install atop on fedora28  https://review.openstack.org/61959510:19
*** aedc has quit IRC10:21
*** xek has joined #tripleo10:23
openstackgerritMarios Andreou proposed openstack-infra/tripleo-ci master: Removes EXTRA_TAGS from toci_gate_test&quickstart.sh.j2  https://review.openstack.org/60476810:25
ykarelssbarnea|bkp2, is osepel deployed somewhere?10:25
ykarelthis is different from originaly deploy method which was not enabling any repo, just installing pacake10:26
*** ramishra has quit IRC10:26
ykarelmay be u have considered that fact10:26
*** aedc has joined #tripleo10:29
openstackgerritJose Luis Franco proposed openstack/tripleo-upgrade master: Do not adjust roles data for minor updates.  https://review.openstack.org/62026210:33
*** udesale has quit IRC10:33
*** udesale has joined #tripleo10:34
*** moguimar has joined #tripleo10:36
*** udesale has quit IRC10:39
openstackgerritDerek Higgins proposed openstack/tripleo-docs master: [WIP] Add documentation on using net-ansible in overcloud  https://review.openstack.org/62010110:44
openstackgerritQuique Llorente proposed openstack/python-tripleoclient master: Catch socket.timeout waitting port api  https://review.openstack.org/61974110:45
*** dsneddon has joined #tripleo10:47
Hobbestigrouhi10:49
Hobbestigroui have a question about this patch https://review.openstack.org/#/c/620003/10:49
*** ramishra has joined #tripleo10:49
Hobbestigroui have remove the tox part to build the documentation, because is not necessary on this project10:51
Hobbestigroui have also modify the test-requirements.txt to remove the documentation dependencies https://review.openstack.org/#/c/620003/3/test-requirements.txt10:53
Hobbestigrouthe patch fail because i have change the requirements and zuul check that10:54
openstackgerritCarlos Camacho proposed openstack/instack-undercloud stable/rocky: Fix instack-undercloud nits  https://review.openstack.org/61946710:55
Hobbestigrouand tripleo-upgrade use ansible https://github.com/openstack/tripleo-upgrade/blob/master/requirements.txt#L210:55
Hobbestigroubut it's not present in openstack global requirements https://github.com/openstack/requirements/blob/master/global-requirements.txt10:56
Hobbestigrouit's normal because ansible is not a global dependency of openstack10:56
Hobbestigroubut i know which zuul template i can use for tripleo-upgrade https://review.openstack.org/#/c/620003/3/zuul.d/layout.yaml10:57
jistrHobbestigrou: i also just found this https://github.com/openstack/tripleo-upgrade/blob/master/ansible-requirements.txt10:59
*** dsneddon has quit IRC10:59
jistrHobbestigrou: so i think maybe the best way would be to try dropping Ansible from https://github.com/openstack/tripleo-upgrade/blob/master/requirements.txt10:59
openstackgerritQuique Llorente proposed openstack/python-tripleoclient master: Catch socket.timeout waitting port api  https://review.openstack.org/61974110:59
Hobbestigroujistr, thanks10:59
jistryprokule: ^ sounds ok?11:00
Hobbestigrouyes it's possible to remove ansible from global requirements11:00
openstackgerritNatal Ngétal proposed openstack/tripleo-upgrade master: [Configuration] Remove the build documentation.  https://review.openstack.org/62000311:03
*** udesale has joined #tripleo11:06
*** holser_ is now known as holser|lunch11:07
Hobbestigrousomeone can review this please https://review.openstack.org/#/c/617670/ ?11:08
Hobbestigrouand this one also please https://review.openstack.org/#/c/619519/11:08
Hobbestigrou?11:08
*** moguimar has quit IRC11:11
openstackgerritQuique Llorente proposed openstack/python-tripleoclient master: Add --module-path to ansible-playbook calls  https://review.openstack.org/62029211:14
quiquellmarios: ^11:14
quiquellmarios: for scenarios11:14
*** moguimar has joined #tripleo11:15
openstackgerritQuique Llorente proposed openstack/python-tripleoclient master: Add --module-path to ansible-playbook calls  https://review.openstack.org/62029211:17
openstackgerritJose Luis Franco proposed openstack/ansible-role-redhat-subscription master: Define job templates in zuul config.  https://review.openstack.org/62029311:18
mariosack quiquell checking in a sec11:18
mariosquiquell: looks nice and clean11:19
mariosquiquell: module path andnothing else +111:19
quiquellmarios: pleas can you testing with stdl and scn don't have time now11:19
mariosquiquell: ack will do11:20
*** moguimar has quit IRC11:21
nawarwhat is the use of keepalived in overcloud if I want to enable ha will use pacemaker?11:22
nawarand can I disable it?11:22
openstackgerritQuique Llorente proposed openstack-infra/tripleo-ci master: Simplify regex removing capturing groups  https://review.openstack.org/61948811:28
*** dsneddon has joined #tripleo11:30
openstackgerritMarios Andreou proposed openstack-infra/tripleo-ci master: Add new tripleo-ci-centos-7-scenario001-standalone job  https://review.openstack.org/61950811:31
*** ahosam has quit IRC11:31
openstackgerritMarios Andreou proposed openstack-infra/tripleo-ci master: Add tripleo-ci-centos-7-scenario004-standalone job  https://review.openstack.org/61952011:32
*** chkumar|ruck is now known as chkumar|away11:32
*** salmankhan has joined #tripleo11:35
openstackgerritMarios Andreou proposed openstack-infra/tripleo-ci master: Add standalone scenario002 job  https://review.openstack.org/61870911:35
openstackgerritMarios Andreou proposed openstack-infra/tripleo-ci master: Add new tripleo-ci-centos-7-scenario003-standalone job  https://review.openstack.org/61876111:36
*** dsneddon has quit IRC11:36
mariosquiquell: ^ done11:36
mariosrfolco: we added Depends-On: https://review.openstack.org/620292 # python-tripleoclient fix for linting issue (set module path in standalone deploy)11:37
*** raildo has joined #tripleo11:37
*** salmankhan1 has joined #tripleo11:38
*** udesale has quit IRC11:38
quiquellmarios: thanks mate, let's see how it breaths11:39
quiquellmarios: if we merge this non voting jobs we can test reviews without test reviews11:39
quiquellmarios: same we did with f2811:40
*** salmankhan has quit IRC11:40
*** salmankhan1 is now known as salmankhan11:40
quiquellas long as we don't break the others11:40
openstackgerritSorin Sbarnea proposed openstack/python-tripleoclient master: Add missing py37 toxenv and corrected default envlist  https://review.openstack.org/61760611:40
*** xek has quit IRC11:40
*** ahosam has joined #tripleo11:41
*** salmankhan has quit IRC11:42
*** udesale has joined #tripleo11:45
*** rfolco is now known as rfolco_doctor11:45
*** udesale has quit IRC11:45
*** udesale has joined #tripleo11:46
*** gfidente has quit IRC11:47
*** moguimar has joined #tripleo11:51
*** udesale has quit IRC11:52
*** udesale has joined #tripleo11:53
*** ratailor has quit IRC11:53
*** moguimar has quit IRC11:56
*** moguimar has joined #tripleo11:56
*** gfidente has joined #tripleo11:56
openstackgerritJuan Antonio Osorio Robles proposed openstack/tripleo-docs master: Update parameters for ipsec deployment  https://review.openstack.org/61955711:58
jaosoriormarios: fixed the grammar issue, thanks for the review ^^11:58
*** janki has quit IRC11:59
jaosoriorjistr: is it cool if we merge this https://review.openstack.org/#/c/619733/2 ?11:59
*** hakhande_ has quit IRC12:01
*** hakhande_ has joined #tripleo12:02
jistrjaosorior: yup, merging it12:02
jaosoriorhjensas: around?12:04
*** amoralej is now known as amoralej|lunch12:04
*** bshephar has joined #tripleo12:05
*** ahosam has quit IRC12:05
*** ahosam has joined #tripleo12:05
*** dsneddon has joined #tripleo12:09
openstackgerritMerged openstack/tripleo-docs master: Some mixing errors  https://review.openstack.org/61807112:10
openstackgerritMerged openstack/tripleo-docs master: Reorganise upgrade/update/ffwd upgrade docs into their own section  https://review.openstack.org/61973312:10
*** xek has joined #tripleo12:11
*** aedc has quit IRC12:11
*** ccamacho has quit IRC12:12
jaosoriorhjensas: anyway, commented on this https://review.openstack.org/#/c/616392/12:13
*** holser|lunch is now known as holser_12:15
EmilienMmandre: hello :) please look https://review.openstack.org/#/c/620168/ and tell me if it's fine12:16
*** bshephar has quit IRC12:18
openstackgerritDerek Higgins proposed openstack/tripleo-docs master: Remove redundant mentions of boot_option  https://review.openstack.org/61926112:18
jaosoriorderekh: I'll merge https://review.openstack.org/#/c/619261/ now given that the fix was a simple typo, do you agree? or still wanna wait for other folks to check it out?12:19
*** udesale has quit IRC12:20
openstackgerritMarios Andreou proposed openstack/ansible-role-tripleo-modify-image master: Replace scenario001-multinode with scenario001-standalone  https://review.openstack.org/62030112:20
*** ahosam has quit IRC12:20
openstackgerritMarios Andreou proposed openstack/tripleo-common master: Replace scenario001-multinode with scenario001-standalone  https://review.openstack.org/62030312:20
openstackgerritMarios Andreou proposed openstack/puppet-tripleo master: Replace scenario001-multinode with scenario001-standalone  https://review.openstack.org/62030412:20
openstackgerritMarios Andreou proposed openstack/tripleo-heat-templates master: Replace scenario001-multinode with scenario001-standalone  https://review.openstack.org/62030512:20
jaosoriormarios: yay!12:20
*** ahosam has joined #tripleo12:20
derekhjaosorior: thanks, go ahead but it depends on about 4 other patches so wont merge until they do12:20
jaosoriorderekh: oh! got it, I'll review them then12:21
mariosjaosorior: well not quite yet but part of the task just getting ready12:21
derekhjaosorior: thanks12:21
mariosjaosorior: we didn't get the new job green yet12:21
*** jaganathan has quit IRC12:22
mariosjaosorior: this merged btw https://review.openstack.org/#/c/619733/12:22
mariosjaosorior: oh you voted :)12:22
mariosthanks jistr ccamacho12:22
jaosoriormarios: haha yep. I voted and then asked jistr to give it his holy blessing12:22
openstackgerritEmilien Macchi proposed openstack/os-apply-config stable/ocata: fixes how os-apply-config handles invalid json  https://review.openstack.org/62030612:24
marioshttps://docs.openstack.org/tripleo-docs/latest/ didn't show up yet12:24
openstackgerritVieri proposed openstack/tripleo-validations master: fix tox python3 overrides  https://review.openstack.org/60824112:24
jaosoriorEmilienM: was this a clean commit https://review.openstack.org/#/c/620306/1 ?12:25
jaosorior* clean cherry-pick12:25
openstackgerritGabriele Cerami proposed openstack-infra/tripleo-ci master: Fix overcloud node log collection in ovb jobs  https://review.openstack.org/61866912:25
jaosoriormarios: it takes a bit12:25
*** cfontain has joined #tripleo12:25
jaosoriorit just merged some minutes ago12:25
mariosack thanks jaosorior so have to find a new one now for this week...12:25
marios**will be smaller ...12:25
marios(much)!12:25
mariosjaosorior: found it am just gonna re-brand and sell this one as related to our effort :) https://review.openstack.org/616187 sorted till next week then ...12:26
mariosdont tell anyone12:27
jaosoriormarios: lol, sounds good to me12:27
jaosoriorderekh: wanna fix the comments from dtantsur|mtg on this review https://review.openstack.org/#/c/618727/3 ? or wanna do it in a subsequent one?12:28
jistrmarios: thanks for doing these12:28
*** ansmith has quit IRC12:29
derekhjaosorior: I left them as is because he +2'd it, I'm happy to change them now if you want to reduce further patches12:32
mariosack jistr somethign that bugged me for a while since i did the dev docs for q12:32
jaosoriorderekh: given that the patch works, I'm fine merging that as-is. feel free to address the comments in a subsequent patch.12:32
*** udesale has joined #tripleo12:32
*** udesale has quit IRC12:32
derekhjaosorior: ok, will do, thanks12:32
*** bfournie has quit IRC12:33
jaosoriormarios: addressed your comment here https://review.openstack.org/#/c/619557/12:34
*** udesale has joined #tripleo12:34
*** salmankhan has joined #tripleo12:34
*** jpena is now known as jpena|lunch12:36
openstackgerritBogdan Dobrelya proposed openstack/tripleo-common master: Add puppet-tripleo container  https://review.openstack.org/61974412:36
*** ccamacho has joined #tripleo12:41
*** ykarel is now known as ykarel|away12:42
EmilienMjaosorior: yes12:43
jaosoriorack12:43
*** dciabrin has quit IRC12:43
EmilienMjaosorior: can you review https://review.openstack.org/#/c/615209/ please?12:45
openstackgerritBogdan Dobrelya proposed openstack/tripleo-common master: Add puppet-tripleo container  https://review.openstack.org/61974412:46
EmilienMjaosorior: the result is that none of the docker containers that were automatically started are in docker anymore http://logs.openstack.org/09/615209/25/check/tripleo-ci-centos-7-containerized-undercloud-upgrades/c864e82/logs/undercloud/var/log/extra/docker/docker_allinfo.log.txt.gz12:46
*** morazi has joined #tripleo12:46
*** jfrancoa has quit IRC12:46
EmilienMjaosorior: after the upgrade to podman12:46
jaosoriorneutron-dns seems to still be up though. Is that expected?12:46
EmilienMjaosorior: yeah this one isn't managed by paunch but by neutron, we need to handle that12:47
jaosoriorgot it12:47
EmilienMjaosorior: i'm working on the next steps, pruning the containers not used anymore and migrates the one managed by neutron12:48
*** moguimar has quit IRC12:49
ykarel|awayEmilienM, i rechecked the buildlogs revert patch, the broken rpm(container-selinux) is removed from buildlogs, let's see the job results12:51
EmilienMykarel|away: nice! thank you12:51
*** trown has quit IRC12:51
*** trown|brb has joined #tripleo12:53
*** udesale has quit IRC12:54
*** abishop has joined #tripleo12:55
*** ykarel|away has quit IRC12:56
*** jfrancoa has joined #tripleo12:59
*** akrivoka has quit IRC13:00
*** arxcruz is now known as arxcruz|brb13:00
*** nawar has left #tripleo13:00
*** akrivoka has joined #tripleo13:00
*** moguimar has joined #tripleo13:00
openstackgerritBogdan Dobrelya proposed openstack/tripleo-common master: Add puppet-tripleo container  https://review.openstack.org/61974413:02
openstackgerritBogdan Dobrelya proposed openstack/tripleo-common master: Remove puppet et al bits from the base image layer  https://review.openstack.org/62031013:02
*** ccamacho has quit IRC13:04
*** ccamacho has joined #tripleo13:04
*** amoralej|lunch is now known as amoralej13:06
*** toure|gone is now known as toure13:06
openstackgerritHonza Pokorny proposed openstack/tripleo-common master: Add a way to update ContainerImagePrepare params  https://review.openstack.org/61907113:06
openstackgerritJose Luis Franco proposed openstack/tripleo-heat-templates master: Invoke redhat-subscription role during upgrade.  https://review.openstack.org/61049313:08
*** suuuper has quit IRC13:09
*** salmankhan has quit IRC13:12
*** aedc has joined #tripleo13:15
*** ahosam has quit IRC13:17
*** salmankhan has joined #tripleo13:18
openstackgerritBrent Eagles proposed openstack/tripleo-heat-templates master: neutron-cleanup skips ports marked skip_cleanup=true  https://review.openstack.org/61906013:23
*** salmankhan has quit IRC13:24
openstackgerritSlawek Kaplonski proposed openstack/tripleo-heat-templates master: Remove external_network_bridge Neutron option  https://review.openstack.org/60476413:25
*** devep has joined #tripleo13:26
*** devep_ has joined #tripleo13:26
*** gfidente has quit IRC13:28
*** rlandy has joined #tripleo13:28
*** ykarel|away has joined #tripleo13:28
*** gfidente has joined #tripleo13:29
Hobbestigrousomeone can check this patch please https://review.openstack.org/#/c/618161/13:33
Hobbestigrou?13:33
openstackgerritBrent Eagles proposed openstack/tripleo-heat-templates master: neutron-cleanup skips ports marked skip_cleanup=true  https://review.openstack.org/61906013:34
openstackgerritCarlos Camacho proposed openstack/instack-undercloud stable/rocky: Fix instack-undercloud nits  https://review.openstack.org/61946713:34
*** ade_lee has quit IRC13:37
*** jpena|lunch is now known as jpena13:38
mandreEmilienM: only if you tell me how to get the ansible-role-redhat-subscription role to work :)13:42
EmilienMmandre: DEAL13:42
EmilienMmandre: are you using it standalone or with tripleo?13:42
openstackgerritJiri Stransky proposed openstack/tripleo-common stable/rocky: Set DeployIdentifier on update/upgrade/ffwd prepare actions  https://review.openstack.org/62031613:42
*** dprince has joined #tripleo13:43
mandreEmilienM: I'm using with tripleo, but I found that rhsm_repos is ignored when using rhsm_activation_key, is that expected?13:43
EmilienMyes13:44
openstackgerritSteven Hardy proposed openstack/tripleo-heat-templates master: Don't disable keepalived in nonha-arch.yaml  https://review.openstack.org/60718013:44
EmilienMwhen you have an activation key you manage the repos in your subscription interface13:44
EmilienMI thought we documented that, one sec13:44
mandreEmilienM: it's inconsistent with the current documentation13:44
EmilienMyes13:44
EmilienMindeed, I need to fix that13:45
*** trown|brb is now known as trown13:45
EmilienMand the github doc is broken too13:45
EmilienMok it needs some love indeed13:45
mandreEmilienM: the only doc I found was on https://access.redhat.com/documentation/en-us/red_hat_openstack_platform/13/html/advanced_overcloud_customization/ansible-based-registration13:45
EmilienMyes and there is https://docs.openstack.org/tripleo-docs/latest/install/advanced_deployment/rhsm.html13:46
EmilienMmandre: but afik these 2 params aren't compatible. I'll fix the doc today13:47
EmilienMmandre: the reason is really on RHSM side13:47
EmilienMmandre: i'm back in 1h3013:48
EmilienMmandre: see https://review.openstack.org/#/c/611334/ for history13:48
EmilienMand https://launchpad.net/bugs/179838213:48
openstackLaunchpad bug 1798382 in tripleo "Overcloud deployment with Rhsm role that uses activation key fails with: "msg": "parameters are mutually exclusive: activationkey, autosubscribe"" [Medium,Triaged] - Assigned to Emilien Macchi (emilienm)13:48
mandreEmilienM: hmm, do you know how to enable a given repo on the RHSM side then? will autoattach option will enable *all* the available repos for the subscription? in my tests it looks like not13:49
mandreEmilienM: thanks for the pointers, I'll dig into it13:49
EmilienMmandre: it should, and you can manage these repos in the Red Hat website I think, or in your Satellite instanec13:50
*** mcornea has joined #tripleo13:51
openstackgerritSorin Sbarnea proposed openstack/tripleo-docs master: Add documentation on deploying Designate  https://review.openstack.org/58583313:52
*** ansmith has joined #tripleo13:54
openstackgerritDougal Matthews proposed openstack/tripleo-common master: Better surface the error status and message when there is a failure  https://review.openstack.org/61733013:55
openstackgerritDougal Matthews proposed openstack/tripleo-common master: Fix broken YAQL expressions in heat_environments.yaml  https://review.openstack.org/61731113:55
*** udesale has joined #tripleo13:56
jaosorior#startmeeting tripleo14:00
openstackMeeting started Tue Nov 27 14:00:55 2018 UTC and is due to finish in 60 minutes.  The chair is jaosorior. Information about MeetBot at http://wiki.debian.org/MeetBot.14:00
openstackUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.14:00
*** openstack changes topic to " (Meeting topic: tripleo)"14:00
openstackThe meeting name has been set to 'tripleo'14:00
jaosorior#topic agenda14:01
*** openstack changes topic to "agenda (Meeting topic: tripleo)"14:01
Tenguah :)14:01
d0ugalo/14:01
ccamachoo/ hi TripleOers!14:01
fultonjo/14:01
jaosorior* Review past action items14:01
jaosorior* One off agenda items14:01
jaosorior* Squad status14:01
bogdandoo/14:01
jaosorior* Bugs & Blueprints14:01
ksamboro/14:01
jaosorior* Projects releases or stable backports14:01
jaosorior* Specs14:01
marios\o14:01
jaosorior* open discussion14:01
jaosoriorAnyone can use the #link, #action and #info commands, not just the moderatorǃ14:01
jaosoriorHey folks! who's around?14:01
panda|rovero/14:01
rlandyo/14:01
dpeacocko/14:01
weshay0/14:02
openstackgerritEmilien Macchi proposed openstack/ansible-role-redhat-subscription master: Fix table in the README  https://review.openstack.org/62031714:02
EmilienMo/14:02
shardyo/14:02
jtomaseko/14:03
matbuo/14:04
*** cfontain has quit IRC14:04
chemo/14:04
beagleso/14:04
jaosorior#topic review past action items14:05
*** openstack changes topic to "review past action items (Meeting topic: tripleo)"14:05
jaosoriorNone.14:05
jaosorior#topic one off agenda items14:05
*** openstack changes topic to "one off agenda items (Meeting topic: tripleo)"14:05
jaosorior#link https://etherpad.openstack.org/p/tripleo-meeting-items14:05
*** udesale has quit IRC14:05
jaosoriorFirst topic:     (jtomasek) Since the Workflows squad has been reorganized and UI&Validations squad is taking over the tripleo-common responsibilities, I'd like to propose we give UI squad cores right to +2 tripleo-common patches14:05
*** udesale has joined #tripleo14:05
jfrancoao/14:06
*** jrist has joined #tripleo14:06
jristo/14:06
*** suuuper has joined #tripleo14:06
jaosoriorjtomasek: around?14:06
abishopo/14:06
jtomasekjaosorior: as described ^, should I send an email to openstack-dev about it? Or is discussing it here enough?14:06
jristui team is hitting stuff that is very UI specific that needs +2 workflow14:07
jaosoriorjtomasek: sending a mail would be nice too.14:07
*** cfontain has joined #tripleo14:07
jtomasekjaosorior: ok, will do14:07
jaosoriorjrist: understood.14:07
jristjaosorior: what's your initial reaction?14:08
openstackgerritThomas Herve proposed openstack/tripleo-common master: Don't update run UpdateParametersAction errors  https://review.openstack.org/62031914:08
chemccamacho: try this instead "openstacksdk>=0.17.2"14:08
chemccamacho: and remove the tox line14:08
jaosoriorjrist, jtomasek: My initial reaction is to hit stackalaytics and check how much are folks from the UI squad reviewing tripleo-common stuff.14:08
chemccamacho: I had it working locally14:08
openstackgerritJose Luis Franco proposed openstack/ansible-role-redhat-subscription master: Define job templates in zuul config.  https://review.openstack.org/62029314:08
jristjaosorior: noted, thanks14:09
shardyhistorically there's been a lot of discussion and good testing/reviews of the workbooks and actions, so IMO this should be fine14:09
openstackgerritCarlos Camacho proposed openstack/instack-undercloud stable/rocky: Fix instack-undercloud nits  https://review.openstack.org/61946714:10
jaosoriorjrist, jtomasek, shardy: I understand the practical reasons; and I want to say it's cool and all. But I don't see folks doing a whole lot of reviews.14:10
jristyup, understood.14:11
*** tzumainn has joined #tripleo14:11
jtomasekjaosorior: I understand your concerns, we can revisit the topic again later14:11
ccamachochem so now, what about running them with python2 holser_ has a point we dont have py3 in rocky, right?14:12
holser_chem I think we need to setup setup.cfg back to python214:12
holser_as it will create a huge divergence between upstream and downstream14:13
jaosoriorjtomasek: sounds good to me.14:13
jaosoriorAnybody else has a take on this topic?14:13
chemholser_: ccamacho well I did the validation using python3 and it was ok14:13
thervejtomasek: Does that mean UI is handling all tripleo-common bugs? :)14:13
* Tengu votes +1 on that14:13
holser_chem, there will be python2 downstream anyway...14:14
jtomasektherve: yes, they're all on apetrich now, who joined UI squad from workflows14:14
therveHum okay14:14
jristtherve: depends is always the answer14:15
jristdepends14:15
jristbut sort of, yeah.14:15
d0ugaltherve: unless you want them? :)14:15
therved0ugal: Not really, but I end up fixing them anyway14:15
therveThat's why I'm asking :)14:15
jristhaha14:15
d0ugaltherve: I noticed, thanks!14:15
jtomasektherve++14:16
*** ykarel|away has quit IRC14:16
jaosoriorjtomasek, jrist: Lets do this, send a mail to the mailing list so more folks have time to give their thoughts there.14:17
openstackgerritJiri Stransky proposed openstack/tripleo-heat-templates stable/rocky: Stop upgrade if a task on one node fails  https://review.openstack.org/62032014:17
jristroger that14:17
jtomasekjaosorior: ack14:17
*** thrash has joined #tripleo14:17
jaosorioralright, the second topic is from panda|rover: featureset overriding and developer experience14:18
panda|roveryes, I'm introducing this, but would like to discuss it in the CI community meeting.14:19
panda|roverAs part of the design to replace scenarios with standalone, a mechanism that overrides variables defined in featuresets was introduced in the CI workflow. This was deemed necessary to satisfy requirements given by tripleo developers, like the ability to change scenario more easily during the local workflow. I'd like to understand what these requirements are and in what way overriding featureset is the thing14:19
*** cfontain has quit IRC14:19
panda|roverthat you need. Featureset design is one of the long lived and most robust design in our workflow, and its non overridability has protected us from regressions in the past, and it represented the last source of truth to understand what a job is doing. Since we are also changing the practice to map a job to a featureset 1:1 (all the standalone scenarios will now map to fs052) this represents also a big change14:19
panda|roverin out to intepret the jobs. I want to understand what was the request that rendered this big change necessary, and eventually work on an alternative solution together.14:19
panda|roverso, if some developers would be able to spend some time to explain it in the community meeting, I'd appreciate it, thanks.14:19
jaosoriorpanda|rover: might also be a good idea to bring it up to the mailing list. Given that not everybody might be able to join to the CI community meeting (I can't join cause I have a conflicting meeting, for instance)14:20
panda|roverjaosorior: ok, I'll do that, thanks.14:21
jaosoriorthank you!14:21
jaosoriorthe next topic is mine: (jaosorior) Lets do some docs!14:22
jaosoriorSo, based on some of the community feedback that was taken from the Summit; one of the pain points of folks was stale and difficult to find documentation14:22
jaosoriorin order to address this, marios and I synced up and decided to start a small initiative to work on the docs14:23
jaosoriorNamely, lets do small changes to improve the docs (readability, structure, and removing stale bits)14:23
jaosoriorand we're tracking this via this bug: https://bugs.launchpad.net/tripleo/+bug/180464214:23
openstackLaunchpad bug 1804642 in tripleo "[DOCS] Improve upstream documentation" [Low,In progress] - Assigned to Marios Andreou (marios-b)14:23
jaosoriorI know everybody's very busy, so we have the goal of doing one change a week14:24
*** hgibson has joined #tripleo14:24
openstackgerritSteven Hardy proposed openstack/tripleo-quickstart-extras master: Switch httpboot directory for baremetal underclouds  https://review.openstack.org/58534414:24
jaosoriorso, if you're interested in joining, help is very much appreciated!14:24
mariosjaosorior: thanks (also listening to another call) 1/week if you want to join us please do small changes/improvements14:24
*** srini_ has quit IRC14:24
TenguI can probably get some time out of the "idling" while a deploy is running on my lab :)14:25
Tenguthat will also improve my knowledge.14:25
jaosoriorTengu: that's how I've been approaching it. take some idle time, read the docs, and try to find something to fix.14:26
mariosjust remember to include the related-bug so we can have a nice track of our improvement14:26
jaosoriorso far it's been a nice learning experience, even if we just passed the first week.14:26
openstackgerritBogdan Dobrelya proposed openstack/tripleo-heat-templates master: Optimize config containers used for puppet steps  https://review.openstack.org/62006114:26
Tengumarios: for sure14:26
jaosoriorlots to read still! :D14:26
Tengujaosorior: yeah. will try to ;).14:26
marioswelcome Tengu :) you now owe one patch by friday14:26
openstackgerritHonza Pokorny proposed openstack/tripleo-quickstart-extras master: Bump OVB version  https://review.openstack.org/62032514:26
*** saneax has quit IRC14:27
jaosoriorThe next topic is this one:     (bogdando) https://bugs.launchpad.net/tripleo/+bug/1804822 optimize base layer for security and edge cases - feedback and code review needed14:27
openstackLaunchpad bug 1804822 in tripleo "Reduce kolla containers image size by moving off puppet bits we override for tripleo" [High,In progress] - Assigned to Bogdan Dobrelya (bogdando)14:27
bogdandoyeah, I hope it self containing, nothing to add here14:28
bogdandojust more eyes needed please14:28
*** rbrady has joined #tripleo14:28
jaosoriorbogdando: I'll check it out after the meetings, thanks for bringing it up!14:28
bogdandotl;dr the battle for images size and security14:28
dprincebogdando: I'm not convinced puppet is your biggest problem14:29
bogdandojust a one of many, dprince14:29
dprincebogdando: I would prefer we hold on pushing any patches perhaps until we agree14:29
bogdandoofc, I'm pushing updates for review only14:29
bogdandoto not develop things detached14:29
dprincebogdando: I think the --volumes-from thing is a bit janky. it is more complex to debug for example14:30
bogdandoall those have wip-114:30
bogdandodprince: we could use paunch to start containers14:30
bogdandoso debugging will be very easy14:30
*** xek has quit IRC14:30
dprincethe way the layers are built today uses less space, because puppet is in the base layer.14:30
Tengupaunch creates systemd units...14:30
bogdandoand solid with containers & paunch 101 UX14:30
bogdandodprince: yes, but puppet brings in systemd, ruby and more14:31
dprincebogdando: 2 things I would stress. Lets not just change our base layers around without hard data. Suspicion isn't enough to go and change these things14:32
bogdandoso we have at least 3 unrelated subjects for security maintenance for *each* container14:32
dprincebogdando: and you may consider alternative (radical) containers to better accomidate edge clouds14:32
dprincebogdando: what is your main issue? space, or network bandwidth?14:32
bogdandowhat do you mean a suspicion? I think it's time to address that tech debt, at very least from the security standpoints14:32
bogdandothose look very generic, not just suspicious things to be proved14:33
shardyintroducing more dependencies on the host will make it harder when we eventually want to adopt an immuntable host OS like coreos, FWIW14:33
dprincebogdando: my vision for how to address the puppet technical debt is quite different from yours I think14:33
openstackgerritJason E. Rist proposed openstack/tripleo-ui master: Enable deletion of deployment upon successful deployment  https://review.openstack.org/62033214:33
dprincebogdando: the LP ticket you filed mentioned space and network bandwidth...14:33
bogdandoshardy: the change does not bring dependencies for host, only reorg containers14:33
dprincebogdando: https://bugs.launchpad.net/tripleo/+bug/180482214:33
openstackLaunchpad bug 1804822 in tripleo "Reduce kolla containers image size by moving off puppet bits we override for tripleo" [High,In progress] - Assigned to Bogdan Dobrelya (bogdando)14:33
dprinceimage size is your main concern?14:34
bogdandodprince: I updated it for security concerns todayt14:34
bogdandoand would be nice to alter cronie to not bring in systemd14:34
bogdando#link https://review.rdoproject.org/r/#/q/topic:base-container-reduction14:34
dprincebogdando: I spoke to slage yesterday. One idea for edge might be to go the other direction and try using 1 container for everything14:34
*** xek has joined #tripleo14:34
bogdando#link https://review.openstack.org/#/q/topic:base-container-reduction14:35
dprincebogdando: like, we used to have an overcloud-full.qcow2 contain all the packages. If you want to only have a minimum amount of space, then that is likely your best option14:35
jaosoriordprince: I do have to agree with bogdando on the security side. Whenever a security vulnerability is found on one of the dependencies that are pulled because of puppet, we need to update the base, and thus update all of the containers.14:35
bogdandodprince: > my vision for how to address the puppet technical debt is quite different from yours I think14:35
bogdandolet's please consolidate on that14:35
dprincejaosorior: do we have data on this? Like there are other Openstack things in our base container that are likely much more commonly security patched14:36
bogdandoin mail lists, code review, and alternative patches! it's very welcome14:36
dprincejaosorior: I guess what confuses me is why puppet. Its one of perhaps many things14:36
bogdando# link http://lists.openstack.org/pipermail/openstack-discuss/2018-November/000212.html14:36
openstackgerritDerek Higgins proposed openstack/tripleo-quickstart-extras master: Setup Ironic in Overcloud  https://review.openstack.org/50972814:37
jaosoriordprince: understood. I'll bring this topic up with my team and get more heads around the security side of this.14:37
*** cfontain has joined #tripleo14:37
dprincebogdando: I will write a spec for my thoughts on the puppet containerization thoughts I have. But do note they are more about "logical" containers (keeping the containers self-contained) and do not perhaps fit with what your space/disk concerns are after14:37
dprincejaosorior: look at the history, how often are we currently bumping the base layer for puppet and its deps. Is it really a problem?14:38
Tenguwell, if we can drop systemd from containers, that would already be a nice thing.14:38
bogdandojaosorior, dprince: not just puppet, but systemd and ruby :)14:38
dprincejaosorior: compare that to other packages.14:38
openstackgerritSorin Sbarnea proposed openstack/tripleo-docs master: Add simple diagram for contributor guide promotion stages  https://review.openstack.org/61618714:38
bogdandomy strong opinion is it's time to stop adding those into all containers14:38
bogdandoso the size is a 2nd issue , and yes, for edge cases only14:39
dprincebogdando: but they get used in all the containers?14:39
dprincebogdando: you are essentially asking us to stop using RPM dependencies in our containers... and just start bind mounting random things from other places into them14:39
jaosorioralright, I suggest we continue this discussion in the BZ or the mailing list14:40
dprinceI mean I guess both containers would be built with their own RPMs14:40
Tenguif this allows to prevent having to rebuild all the containers when there are security concerns...14:40
bogdandodprince: jaosorior: > I guess what confuses me is why puppet. Its one of perhaps many things14:41
bogdandoyes, one of many, in the linked mail list, hjensas brought more of that14:41
bogdandoand mwhahaha did some optimizations, earlier in the topic14:41
jaosoriorbogdando: mind starting another mail thread with a new topic so this discussion is easier to track?14:41
shardyWe do already have parameters which allow splitting ServiceNameImage (runtime) and ServiceNameConfigImage14:41
shardyso we could use different images, double how many are built and remove the runtime puppet etc deps14:42
bogdandodprince: yes, they are in all containers in tripleo14:42
shardybut yeah, ML sounds like a good place to discuss14:42
bogdandonot used maybe ,but stay here runtime and may be exploited, all of that security things :)14:42
dprinceshardy: if you have to download the config image once why not just use it to run the service?14:42
shardydprince: to avoid the need to restart service containers if e.g puppet gets updated which was mentioned above14:43
shardydprince: but yeah, I do agree14:43
dprinceshardy: the reason the Config image parameters exist was more along the lines of optimization I think, to eliminate generating things like nova.conf more than once14:43
bogdandosorry, that's it from my side14:43
shardyjust pointing out we already provide an interface which could enable "lean" runtime containers if folks really want that for hardening etc14:43
openstackgerritMarios Andreou proposed openstack/tripleo-docs master: Add simple diagram for contributor guide promotion stages  https://review.openstack.org/61618714:43
*** xek_ has joined #tripleo14:43
dprinceshardy: its all pre-optimization to me at this point. Like how often is puppet or ruby updated that something else in the base container isn't?14:44
* dprince thinks we are chasing the wrong problem14:44
mwhahahavery rarely14:44
mwhahahabecause we don't switch versions14:44
shardydprince: yeah, I'm just saying different images could be cleaner than bind mounts between containers etc, not arguing for the change itself14:44
jaosoriorAlright, lets move on with the agenda and continue this topic in the ML14:45
dprinceshardy: cool. And I'm saying having a container be able to configure itself is very useful. Furthermore a logical 'service' container is a very good thing14:45
Tengudprince: well, between systemd, puppet and ruby, there are many security concernes, almost every month... and also, what's the point keeping them in runtime containers when they are useless? reducing attack surface is a must, regarding security ;). anyway, will also jump in the LP + ML.14:45
dprincethat said, if you want to break the rules for edge and have an all-in-one container then we have parameters in place that should allow you to do that14:46
*** xek has quit IRC14:46
dprincebut don't go re-layer everything for the sake of pre-optimizing container layers14:46
jistrFWIW removing puppet/ruby from service containers is going to make situation worse w/r/t space usage likely14:46
jistrwould we start building 2 sets of base images?14:47
bogdandoshardy: note, those bind mounts between containers is a daily approach for pods14:47
jistrone for services, other for config14:47
Tengumounts between containers is also a known and well used thing in rancher world.14:47
dprincebogdando: sure, but is having a service be able to configure itself really need to involve a separate pod?14:47
TenguI'd say yes, dprince... removing not-runtime things is a good idea. That will require some chances in the architecture, but still... I support bogdando on that :).14:49
bogdandoshardy: I wish I could just use podman pods, but I have hope to backport that for Queens14:49
dprincerather than this, I would like to see use use --volumes from for the /var/lib/config-data directory14:49
dprincehaving that be in a data container makes so much sense14:49
dprinceand IMO is a more correct "podlike" construct14:49
shardybogdando: IMHO backporting something like this would be very risky indeed14:49
bogdandobut we want to support edge there...14:50
bogdandoDCN14:50
dprincebut what is being suggested here is basically teasing apart functional containers...14:50
dprincethose are two very different things14:50
bogdandothink of 30,000 or 40,000 nodes getting a base layer with the world plugged in14:50
shardybogdando: if you need a solution that works on queens personally I'd just build two sets of containers vs the sidecar thing14:51
shardybut that's just my opinion14:51
bogdandoI think that can work as well14:51
bogdandowould be nice to play with neat podman pods14:51
dprinceshardy: thats actually more data that he'll download I think14:51
bogdandow/o legacy )14:51
shardyit's still a risk from an update perspective but it miminizes the architecural changes14:51
*** krypto has joined #tripleo14:52
jistrdprince: +1. I don't follow how can we save on downloading things if we build two sets of images, it's likely going to have the opposite effect AFAICT?14:52
jistri understand the security argument though14:52
*** gkadam has quit IRC14:52
*** rfolco_doctor is now known as rfolco14:53
bogdandodprince: I'm not sure all-in-one container is a good idea tbh14:53
jistrbut as long as we generate config locally on each node, the 30 or 40K nodes are going to have to download images with world plugged in anyway14:53
dprincebogdando: given what you are asking for it is the way you'll download the least data to these nodes14:53
shardyjistr, dprince: yeah, I was just saying it's a less risky way of achieving the hardening goal vs the image size thing14:53
bogdandojistr: > well, between systemd, puppet and ruby, there are many security concernes, almost every month... and also, what's the point keeping them in runtime containers when they are useless? reducing attack surface is a must, regarding security ;). anyway, will also jump in the LP + ML.14:53
Tenguhm, we seriously should continue this discussion on the ML and get over the mtg though (ping jaosorior ;))14:53
bogdandono, just a separate image with config-time dependencies14:54
bogdandoand config_image to leverage volumes off it14:54
bogdandothat's what I proposed in the lined topics14:54
jaosoriorTengu: at this point there's 5 min left.14:54
kryptohi all in triple-o is it possible to have separate roles like RabbitMQ and load balancer so that it can be installed on separate nodes14:54
jistrbogdando, shardy: yea re hardening it makes sense i think14:54
dprincebogdando: honestly, the best solution for you would be using packages on the host, generating the config files on the host. And then having an all-in-one container for all the services which lets them run in an isolated mannner14:54
jaosoriorAnyway14:54
jaosoriorjust for status:14:54
jaosorior#topic bugs & blueprints14:54
*** openstack changes topic to "bugs & blueprints (Meeting topic: tripleo)"14:54
jaosorior#link https://launchpad.net/tripleo/+milestone/stein-214:54
jaosoriorFor Stein we currently have 29 (for stein-2) and 3 (for stein-3) blueprints open in Launchpad.14:54
jaosoriorBugs: 780 (+15) stein-2, 5 (+2) stein-3.  102 (+1) open Storyboard bugs.14:54
openstackbug 36202 in gnupg (Ubuntu) "duplicate for #780 race condition on ~/.gnupg/random_seed when signing" [Medium,Fix released] https://launchpad.net/bugs/36202 - Assigned to Daniel Silverstone (dsilvers)14:54
dprincebogdando: i.e. taking a step back from containers14:54
jaosorior#link https://storyboard.openstack.org/#!/project_group/7614:54
mwhahahakrypto: yes, see the Messaging role we provide14:54
jaosorior#topic open discussion14:55
*** openstack changes topic to "open discussion (Meeting topic: tripleo)"14:55
kryptoThanks mwhahaha14:55
*** salmankhan has joined #tripleo14:58
*** salmankhan has quit IRC14:58
jaosoriorAlright, folks! thanks for joining14:58
jaosorior#endmeeting14:58
*** openstack changes topic to "CI Status: Green but recheck slowly | https://docs.openstack.org/tripleo-docs/latest/"14:58
openstackMeeting ended Tue Nov 27 14:58:51 2018 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)14:58
openstackMinutes:        http://eavesdrop.openstack.org/meetings/tripleo/2018/tripleo.2018-11-27-14.00.html14:58
openstackMinutes (text): http://eavesdrop.openstack.org/meetings/tripleo/2018/tripleo.2018-11-27-14.00.txt14:58
openstackLog:            http://eavesdrop.openstack.org/meetings/tripleo/2018/tripleo.2018-11-27-14.00.log.html14:58
bogdandojistr, not for day 2. One have to only update that config-image, not all containers, pushing changes off the base layer upside all the way up... If puppet/systemd/ruby gets a CVE14:59
weshayk.. moving to ci open office14:59
weshayrfolco, https://bluejeans.com/411356779815:00
weshayssbarnea|bkp2, sshnaidm|afk ^15:01
rfolcoweshay, joining in a min15:01
mariosgfidente: fultonj: you want to join us on ci community call? " https://bluejeans.com/411356779815:03
gfidentemarios yes joining15:03
fultonjmarios: yes15:03
gfidentemarios thanks15:03
*** bfournie has joined #tripleo15:05
jistrbogdando: ack, yea i agree re security. Re downloads i think it's going to be the same or worse still. Either we'll have to have per-service config image or the single config image will need to carry all of OpenStack anyway (puppet modules depend on OpenStack packages for some actions AFAIK), so you'll still download the world even if you update just the config image, unless i'm missing something15:05
*** devep_ has quit IRC15:05
*** devep has quit IRC15:05
Tengujistr: at least World won't be loaded in the runtime containers, exposing flawed libraries and the like ;).15:06
jistrTengu: yup agreed on that15:06
Tengusecurity has always been a tradeoff.15:06
Tenguconvenience vs security, usability vs security, bandwidth vs security, KISS vs security, and the like15:07
chemKISS is a worthy goal :)15:07
Tenguyeah, and that one can lead to security ;).15:08
Tengubtw, if we apply KISS to the containers, we will actually drop a lot of useless load.15:08
jistryea i'm fine making tradeoffs, just trying to get to the bottom of individual points, because in any decision making it helps to have the pros/cons clear. I mean we shouldn't use download size as a "pro" in this discussion when (according to my understandin anyway) it's going to end up being a "con".15:09
Tenguof course15:09
openstackgerritQuique Llorente proposed openstack/python-tripleoclient master: Catch socket.timeout waitting port api  https://review.openstack.org/61974115:09
Tenguopen process means open discussions, weighting all aspects, and so on ;).15:09
Tenguone more thing: if runtime containers don't have World, security update might NOT affect them, meaning: no service disturbence, as we should not need any container restart.15:10
Tenguthat's the nice thing if we got either all-in-one config container, or per-service config container. either way: as long as (service+libs) aren't affected, we don't need to restart the container. yay!15:11
Tenguno more shattering rabbitmq cluster running after its messages15:11
Tengujust for that.. I'm all in :D15:12
jistrTengu, bogdando: just a heads up, to allow rolling upgrades we'll likely need per-service config containers15:13
*** skramaja has quit IRC15:13
jistrso a preferred technical solution would be to bolt on config stuff on top of individual service images i think15:14
dtantsur|mtgconfig containers wut?..15:15
Tenguyeah, if individual config-container are needed, that's probably the right thing to do, for the sake of common layers.15:15
jistralternatively we could go the other way -- bolt on service stuff on top of base image with config stuff15:15
Tenguhm, that's what we have already.... ?15:15
Tengubolting that way would let puppet/systemd/ruby/others in runtime containers15:15
jistrTengu: i meant a separate tree for config images only. The service images would start from a base image without config stuff15:16
Tenguah, ok. not sure this is the best regarding space/bw15:17
*** salmankhan has joined #tripleo15:17
Tenguas the config will get part or all of the service thingy anyway.15:17
Tengu(due to dependencies)15:17
jistrTengu: personally i'd prefer "bolt on config stuff on top of individual service images" because having a single base image seems easier to reason about15:17
Tengu+115:17
jistrTengu: i think from space perspective both are bad :)15:17
Tenguso, base image -> per-service image -> per-service config image  ?15:18
jistrTengu: yes that's the sanest i think15:18
jistrTengu: but it results in downloading config stuff N times instead of once15:18
Tenguyep.15:18
Tenguand why not a config "all-in-one"?15:18
TenguOK it will probably duplicate part of openstack things. but we download it once, and get smaller per-service containers.15:19
Tenguwe avoid most of the duplication15:19
jistrTengu: because that would block us from aiming for rolling per-service upgrades15:19
Tenguwhy so?15:19
Tenguas long as service upgrade isn't triggered, we don't need to run the "service specific parts".. ?15:20
jistrif we intend to run one service in major/minor version X and other service in minor/major version Y, we need matching-version config image15:20
Tenguah. humpf. so people are actually doing that? is it sane anyway?15:20
dtantsur|mtgrolling upgrades? well, people who care about their cloud uptime do ;)15:21
jistrTengu: people aren't doing that with TripleO right now but i think it's been suggested that it'd be nice to be able to do that. IIRC some old blog posts, CERN used to do upgrades service-after-service15:22
openstackgerritRafael Folco proposed openstack-infra/tripleo-ci master: Deprecate tripleo stable/{newton,ocata} jobs  https://review.openstack.org/61933715:22
dtantsur|mtgthey have to do it. otherwise it's too hard to recover if something fails in the middle.15:22
jistryea. Currently TripleO takes node-by-node approach, which makes a lot of sense coming from package-based world, but as we get more and more established with containers, it will probably be better/safer to go service-by-service at some point15:23
dtantsur|mtgand of course downtime15:23
Tengujistr: erf...15:23
dtantsur|mtginstance-of-service-by-instance-of-service, to be more precise15:24
jistrdtantsur|mtg: yes, to preserve uptime15:24
dtantsur|mtglike, ironic-conductor-1, then ironic-conductor-2, then ironic-api-1, then ironic-api-2, ...15:24
bogdandohjensas, mwhahaha, jaosorior, shardy, jistr, Tengu, dprince: http://lists.openstack.org/pipermail/openstack-discuss/2018-November/000244.html15:25
jaosoriorbogdando: ack15:25
*** shardy has quit IRC15:26
jistrdtantsur|mtg: yea. On high level rolling across components, and within one component, rolling across service instances15:26
bogdandojistr: I think this has nothing to individual containers, that is still a thing15:26
jistrdtantsur|mtg, Tengu: currently that's all scifi :) but we shouldn't close any doors towards that15:26
bogdandojust empowered by a "pod" with puppet et al bits15:26
bogdandoconsumed from another common container15:27
jistrbogdando: nothing with containers, but it is affected by how images are structured15:27
bogdandoindividual config images will keep working as is, jistr15:28
dtantsur|mtgjistr: scifi for tripleo - yes15:28
*** Vorrtex has joined #tripleo15:29
*** salmankhan has quit IRC15:29
bogdandojust that they all stop having puppet* in it15:29
jistrdtantsur|mtg: well we can't rolling-upgrade some services (e.g. Rabbit) so it's a bit more than just TripleO15:29
bogdandolater we may want to move off all clients there15:30
dtantsur|mtgright, I mostly mean for openstack services that do support it15:30
*** mjturek has joined #tripleo15:30
bogdandolike openstack client and whatnot?15:30
dtantsur|mtg(nova, ironic, cinder, ...?)15:30
bogdandoso containers for runtime will only have needed libs to run15:30
dtantsur|mtgthe best way to upgrade rabbit is stop using rabbit for rpc :)15:30
bogdandoa service15:30
bogdandobut we can start from puppet15:30
bogdandoand its nasty dependencies )15:31
*** rbrady has quit IRC15:33
bogdandojistr: >so you'll still download the world even if you update just the config image, unless i'm15:33
bogdandoI think you do. A side car container contains puppet and deps, while config image now stops containing it, as of the base layer getting leaner15:33
bogdandoand that's basically it... nothing that that15:33
bogdandothan15:33
bogdandonow if puppet gets a CVE fixed you do not pull in all layers on top of the base, for all containers15:34
bogdandobut only a base layer + 1 a common side car container15:34
bogdandoconfig/runtime images are now not afected by unrelated updates15:34
jistrbogdando: that was my point in discussion with Tengu though, we probably can't have 1 common side car container image if we want to support rolling upgrades15:34
bogdandowhy not? it contains puppet, maybe clients later, just like a single image would do15:35
Tengujistr: unless playing on image tag maybe?15:35
jistrbogdando: because you want different ones in one cloud at the same time. They have to match the services15:36
bogdandowe just split image runtime, but from a container started off it, it looks the same as if started from the old image where we have all included there15:36
bogdandonothing changes15:36
*** ykarel|away has joined #tripleo15:36
bogdandoso you can version that image15:36
Hobbestigrousomeone can review this please patch please https://review.openstack.org/#/c/620003/ ?15:36
jistrTengu: that could be a way perhaps... still it feels a bit strange to make one service config image affect all the other services15:36
bogdandothe same way as you would do for the original image that contains puppet15:36
jistrTengu: no per-service fixes/tweaks are possible15:37
bogdandoI see no difference really...15:37
bogdandoJust that you have N+1 images of a vesrion X vs Y, compared to the original N images of X vs Y15:38
bogdandojust that N images duplicate all that is there in that extra 1 image15:38
bogdandojust in the base layer15:38
bogdandojistr: ^^15:38
jistrbogdando: yea, i'm thinking whether we can get away with bundling the config capabilities of all services into a single image in this scenario... maybe (?)15:39
bogdandonot sure, that looks like another topic and I have to wrap my head around it )15:40
bogdandothat's probably only the question of when to merge layers15:40
bogdandobuild time, like that all-on-one15:40
bogdandoor runtime, what pods/volume-from can do for individual container merges15:41
bogdandoor merge nothing and have it all dupicated off the common base layer, as we do15:41
bogdandojistr: ^^15:41
jistrTengu, bogdando: the single-config-image vs. per-service-config-image ways can perhaps be combined too, if we discover we have to. E.g. have a single image mostly, and if some services are too special then they can still get their own custom one.15:42
jistrand maybe playing on image tag would indeed be sufficient for rolling upgrades here15:42
bogdandomerging run time, in pods, look more a wise approach to me15:42
bogdandowhen bloating it up build time15:42
bogdandothink of k8s pods that work as only build-time all in one? :D15:43
bogdandodo we want that?15:43
*** dciabrin has joined #tripleo15:43
*** shardy has joined #tripleo15:44
jistrbogdando: do you have a link with some reading re runtime merging? i'm not familiar with that15:46
bogdandojistr: that's a pod in k8s15:46
bogdandothe pod is a container15:47
bogdandoand all containers in it may consume its volumes15:47
bogdandothat's that I called merging runtime15:47
*** jcoufal has joined #tripleo15:47
*** ade_lee has joined #tripleo15:47
bogdandoso logically you see a pool of volume resources for all of containers in pod15:47
*** ksambor has quit IRC15:47
bogdandoan opposite to pods, would be to build a single container and create containers using volumes off it15:48
bogdandothat's how I get it, I may be wrong...15:48
jistrbogdando: ok i am familiar with that :) but afaik that's merging *volumes*, not *layers*, right?15:48
bogdandofor the latter case we have shared nothing for containers started off that all in one15:48
bogdandothink of it as logical units15:48
bogdandowe want to see puppet from a container w/o it15:49
bogdandowe can either add a layer, into the base or other one, or use a pod15:49
jistrhm so you'd want to share things like /usr/bin/puppet and /usr/lib/<whatever-puppet-needs> via a volume?15:50
bogdandoyes15:50
bogdandothat's my plan15:50
jistrwell i've never tried that, but at a first glance it sounds dangerous/fragile to me15:50
bogdandosee https://review.openstack.org/#/c/620061/4/docker/docker-puppet.py15:50
bogdandowell, not sure. Why? k8s uses that for each pod. Not sure if for binaries, we could search for examples15:51
bogdandobtw, I don't know if pods use volume-from or just nsenter15:52
*** dtrainor__ is now known as dtrainor15:52
dtrainoris there a way to see a list of resource classes?  or is this information only availble per a node's resource_class field, and the in the 'resources' attribute of the 'properties' field of a flavor?15:52
*** jcoufal has quit IRC15:52
jistrbogdando: i think that's kinda the difference i see. I don't mind as we mount /usr/share/openstack-puppet/modules, because we know that we only need whatever is in that dir. But how would we sanely track whatever dynamic libs are getting loaded when we execute /usr/bin/puppet ?15:53
openstackgerritSam Doran proposed openstack/ansible-role-redhat-subscription master: Fix table in README  https://review.openstack.org/62034515:53
bogdandojistr: right, and that is an open question :)15:53
dtantsur|mtgdtrainor: you can fetch them from Placement, I think. but doing $ openstack node list --fields resource_class may be easier15:53
bogdandolet's think of alternatives15:53
bogdandomaybe indeed, stop using strange RPM dependencies?15:54
dtrainordtantsur|mtg, thanks.  i was actually going to ping you about this later on, i think there may have been a regression re: node tagging / resource matching from rocky -> queens15:54
*** quiquell is now known as quiquell|off15:54
bogdandoor stop installing RPMs in containers )15:54
*** gfidente has quit IRC15:54
dtrainorbut i first wanted to prove this :)15:54
jistrbogdando: the build-time layering sounds saner to me..15:54
bogdandolike I want cronie and get systemd pulled in for freeeee bonus15:54
dtantsur|mtgdtrainor: there may be, there may be not :) nova scheduler is dark and full of terrors15:54
*** gfidente has joined #tripleo15:54
dtrainorindeed.15:54
*** hberaud has joined #tripleo15:55
bogdandojistr: that is not achievable to drop off config time bits then15:55
dtrainordtantsur|mtg, like this?  (undercloud.example.com, 5396d128-0fbb-4f73-a2ea-1f1c067b0f28) ram: 0MB disk: 0MB io_ops: 0 instances: 0 does not have 4096 MB usable ram before overcommit, it only has 0 MB. host_passes /usr/lib/python2.7/site-packages/nova/scheduler/filters/ram_filter.py:4715:55
mwhahahaweshay: so i think the gate is terrible because we added the scenario jobs back as voting to the stable branches.15:55
bogdandounless maintaining a two trees15:55
dtantsur|mtgdtrainor: I suspect you need to update your flavors15:55
*** devep has joined #tripleo15:55
bogdandoor using docker build time dependencies15:55
bogdandonot a thing for podman AFAIK15:56
dtantsur|mtgdtrainor: https://docs.openstack.org/ironic/latest/install/configure-nova-flavors.html all this resources stuff, do you have it?15:56
bogdandoand not really fits into what docker-puppet.py does15:56
dtrainorwhich updates might you suggest?  i'm using the default flavors deployed with a downstream queens deployment15:56
dtrainori do, yes15:56
*** udesale has quit IRC15:56
dtrainorthey're set to 0, and per the docs that's ok to disable scheduling based on standard properties for a bare metal flavor15:57
*** udesale has joined #tripleo15:57
bogdandodprince, jistr: speaking of https://review.openstack.org/#/c/620061/4/docker/docker-puppet.py, why do we bind mount /usr/share/openstack-puppet/modules/ from host?15:58
openstackgerritMerged openstack/tripleo-heat-templates master: Upgrades: Ensure idempotency of pacemaker services  https://review.openstack.org/61957515:58
openstackgerritMerged openstack/tripleo-common master: Catch heat exceptions in DeployStackAction  https://review.openstack.org/61397415:58
*** udesale has quit IRC15:58
bogdandoI thought we want to consume what we have in containers?15:58
bogdandowhy including things into the base layer then overriding it in docker-puppet from host?15:59
*** yprokule has quit IRC15:59
openstackgerritSteven Hardy proposed openstack/tripleo-heat-templates master: Remove common bootstrap_nodeid from deploy_steps/tripleo-packages.yaml  https://review.openstack.org/60568215:59
jistrbogdando: i think the puppet *modules* are not distributed with containers maybe? via RPMs still? while puppet *executables* are in the containers15:59
bogdandojistr: they are15:59
bogdandopuppet-tripleo pulls in the world15:59
bogdandoand it's in the base later16:00
bogdandolayer16:00
openstackgerritRedHat RDO CI proposed openstack/tripleo-heat-templates master: GATE CHECK for TripleO  https://review.openstack.org/60429816:00
bogdandothat's not what I understand in DF still )16:00
dtrainordtantsur|mtg, re: "disable scheduling based on standard properties" was a quote from that very same doc you linked.  i believe my understanding of that process is correct16:00
bogdandohad we wanted only puppet, we'd include puppet, not puppet-tripleo16:00
jistrack then i don't know16:00
* bogdando sigh16:00
bogdandomwhahaha: perchance?16:01
mwhahahaque?16:01
bogdandomwhahaha:16:01
bogdandospeaking of https://review.openstack.org/#/c/620061/4/docker/docker-puppet.py, why do we bind mount /usr/share/openstack-puppet/modules/ from host?16:01
bogdandoI thought we want to consume what we have in containers?16:01
bogdandowhy including things into the base layer then overriding it in docker-puppet from host?16:01
mwhahahabecause puppet-tripleo is used by all the services16:01
mwhahahaso it's global to all openstack services16:01
mwhahahawhich is why it would make sense to be in the base layer16:01
*** dansmith has quit IRC16:02
bogdandomwhahaha: but why overriding /usr/share/openstack-puppet/modules/ from host?16:02
*** dansmith has joined #tripleo16:02
bogdandoisn't that pointless to include into container that path from puppet-tripleo & deps  then?16:02
mwhahahaprobably, but dprince has a patch to stop using the host puppet16:02
mwhahahathis is one of those things that we need to weigh the pros/cons of16:03
bogdandoI saw that, but the question remains, why do we want something in the base layer what we always override16:03
mwhahahabecause there's an operational impact for either method16:03
* mwhahaha shrugs16:03
mwhahahait was like that when i got here16:03
bogdandothe same :D16:03
mwhahahait's easier to hack in changes if they come from the host than in the container16:04
mwhahahaso i'm not sure16:04
*** cfontain has quit IRC16:04
mwhahahawe could drop puppet-tripleo and just leave puppet from the base16:04
mwhahahabut that prevents it from being used only from containers16:04
*** cfontain has joined #tripleo16:04
mwhahahalike if we switch to an atopic base os16:04
mwhahahaof course who knows if we'll ever be able to do that16:05
bogdandowe'll do, you'll see! :)16:05
mwhahahaso i still think you need puppet in the base container16:06
mwhahahaother wise you'll dupe that in all the containers and actually have space problems16:06
bogdandomwhahaha: my plan was to use a pod-like way to runtime refer it16:06
panda|rovermwhahaha: to the best of you knowledge, do you think any of the scenarios we'll have to define in the custom environment file will require a different set of "ci" feature from the one required for another environment ?16:06
bogdandosee that jistr writes above:4:50:13 PM GMT+1 - jistr: hm so you'd want to share things like /usr/bin/puppet and /usr/lib/<whatever-puppet-needs> via a volume?16:07
mwhahahapanda|rover: not for the standalone, we don't have any basic features from a quickstart perspective that we would change16:07
* bogdando thinks of using nsenter for that...16:07
mwhahahabogdando: honestly this doesn't solve anything of use. It would be better if we could generate configs/containers and ship only what we needed from the UC. but that's even more complicated16:08
Tenguplease don't16:08
Tengunsenter is a mess, really.16:08
panda|rovermwhahaha: ok16:08
Tengubogdando: if we can avoid nsenter, it's really, really better.16:09
Tenguwe already had some funky issues with it, and it's really a pain to debug/trace -.-16:09
openstackgerritGael Chamoulaud proposed openstack/tripleo-validations master: Use containerized location of keystone.conf file  https://review.openstack.org/62035016:09
bogdandoTengu: ok, I'll look what can be done16:10
*** salmankhan has joined #tripleo16:11
*** salmankhan has quit IRC16:12
jristhonza, jtomasek, dtrainor, florianf eyes? https://review.openstack.org/#/c/620332/16:13
openstackgerritMerged openstack/tripleo-docs master: Update parameters for ipsec deployment  https://review.openstack.org/61955716:15
openstackgerritThomas Herve proposed openstack/tripleo-common stable/rocky: Catch heat exceptions in DeployStackAction  https://review.openstack.org/62035216:18
Tengubogdando: avoiding nsenter will make the code less complex, more maintainable, easier to fix/debug/understand :). switching namespace on the fly is nice, but probably a bad idea for the future.16:18
*** aedc has quit IRC16:20
jistrTengu, bogdando: separating service vs. config images at build time (either having a single know-it-all config image, or bolting config parts on top of service images) so far sounds the most sane to me, to achieve the security hardening16:20
*** aedc has joined #tripleo16:20
jistr(from what we've discussed so far)16:21
*** artom has quit IRC16:22
*** artom has joined #tripleo16:22
*** cfontain has quit IRC16:22
openstackgerritJose Luis Franco proposed openstack-infra/tripleo-ci master: Create tripleo-ansible-role-job template for ansible projects.  https://review.openstack.org/62035316:23
*** thrash is now known as thrash|biab16:25
*** mfedosin has joined #tripleo16:25
dtrainorjrist, ack, i'll be able to look in a bit16:27
*** jcoufal has joined #tripleo16:28
*** apetrich has quit IRC16:28
*** jfrancoa has quit IRC16:28
openstackgerritSorin Sbarnea proposed openstack/tripleo-quickstart-extras master: Fix failure to install atop on fedora28  https://review.openstack.org/61959516:29
openstackgerritRajini Karthik proposed openstack/tripleo-heat-templates stable/rocky: Added Dell EMC SC multipath support  https://review.openstack.org/62035516:29
*** hakhande_ has quit IRC16:30
openstackgerritRajini Karthik proposed openstack/tripleo-heat-templates stable/rocky: Added Dell EMC SC multipath support  https://review.openstack.org/62035516:31
openstackgerritRajini Karthik proposed openstack/tripleo-heat-templates stable/queens: Added Dell EMC SC multipath support  https://review.openstack.org/62035816:34
mfedosinweshay, hello!16:35
mfedosinI just found an issue with the reproducer on 'stack create' task...16:35
mfedosinand wanted to understand whether it's common or just for me16:35
mfedosinhere is my command ./reproducer-quickstart.sh --create-virtualenv true --remove-stacks-keypairs true --nodestack-prefix repro -a16:36
mfedosinand the results are http://paste.openstack.org/show/736123/16:36
dprincebogdando, mwhahaha: you have to keep in mind the entire TripleO deployment used to be on baremetal16:37
dprincethis is why we bind mounted /usr/share/openstack-puppet/modules/. It was a step towards containerization16:37
dprincebut also keep in mind Pacemaker still runs on the baremetal host as well and currently requires puppet there16:38
bogdandojistr: yes, but that makes the Edge DCN story very sad16:41
bogdandoindeed, dprince I forgot about pacemaker on host :o16:42
*** apetrich has joined #tripleo16:42
bogdandojistr: though, may be adding config bits as a small layer on top ...16:44
bogdandonot the base layer ofc16:44
bogdandomay be a way to go indeed16:44
*** sri_ has joined #tripleo16:44
thervemfedosin: What does "openstack stack show repromultinode_stack" says?16:45
openstackgerritGael Chamoulaud proposed openstack/tripleo-validations master: Run pcs command only if the pacemaker service is active  https://review.openstack.org/62036016:47
dprincebogdando: if you add 'config bits' (i.e. puppet, etc) in the layers on top it will actually cost you much more in terms of network bandwidth and disk usage16:47
bogdandojistr: commented as well, nsenter...16:48
bogdandoTengu: (sorry for that)16:48
Tengubogdando: :)16:48
bogdandodprince: not, if you think of Day 216:48
Tenguanyway. see you tomorrow folks!16:48
bogdandowhen you have to push the changed layer and all changes on top for each container as a minor update16:48
dprincebogdando: if you'll hold on for 1 day I will write you a spec for how I'd like to re-layer puppet logically16:49
*** agopi is now known as agopi|food_run16:49
bogdandothat's not an emergency you know :) I can hold off as long as we want it. let's make it clearly implemented! thanks dprince!16:49
mfedosintherve,  $ openstack stack show repromultinode_stack16:49
mfedosinStack not found: repromultinode_stack16:49
dprincebogdando: regarding the root cause of your comments I'm hearing a general "I don't like how containers work", so lets bind mount things ad-hoc16:49
bogdandodprince: please hear me right. bind-mount is not like host or something. Just a standard way of doing things in pods!16:50
thervemfedosin: Do you still have  /tmp/reproduce-tmp.mqp25/reproprovision_multinode_stack.log ?16:50
bogdandoeach pod in k8s world does that16:50
bogdandowhen you define a volume in a pod and use it in a container defined nearby16:51
dprincebogdando: I dispute that. Bind mounting applications and libraries should not be done. It is the equivalent of using an /nfs share for your applications16:51
bogdandothat's established, what else we can do? only follow it16:51
dprincebogdando: --volumes from should be used to share application state. Not application code16:51
mfedosintherve, yes, one second16:51
bogdandodprince: yes, that's a nuance indeed...16:51
dprincethis is where we disagree, and I think we need to have the team agree on our best practices. Otherwise, we'll have what I consider a "hack" in our codebase16:52
bogdandosure16:52
bogdandolet's avoid more hacks :)16:52
mfedosintherve, http://paste.openstack.org/show/736126/16:52
mfedosinit says ERROR: The Parameter (image) was not provided.16:52
* jistr also -1 on bind-mounting executables/libs16:53
therveYeah16:53
bogdandodprince: wait, but init containers does share code16:53
bogdandoand any bootstrap logic one wants for a pod!16:53
bogdandoIIRC16:53
thervemfedosin: Content of /tmp/reproduce-tmp.mqp25/params_multinode.yaml ?16:53
bogdandoso one can have its containers bootstrapped using missing code16:53
bogdandoconsumed from the init container16:53
bogdandoam I right?16:54
bogdandodo share*16:54
*** arxcruz|brb is now known as arxcruz16:54
mfedosintherve, http://paste.openstack.org/show/736127/16:54
bogdandoand init containers may be using arbitrary images16:54
bogdandoso the same volumes from approach, just from other interface16:55
jistri haven't seen a container sharing code yet i think.. Once i tred to do it to get tensorflow running in container, mounting nvidia driver libs, but i noped out of that quite quickly :)16:55
dprincebogdando: init containers are ephemeral (they go away quickly just like our config containers do). The images can be different sure, but the init container images should contain all the application binaries and libaries they need to complete on their own16:55
dprincebogdando: I'm not sure I see any differences here other than you can use an alternate image16:55
*** moguimar has quit IRC16:55
bogdandodprince: init container abstracts off the missing code in the main container16:56
dprincebogdando: but do note... if that is your idea layering puppet outside of the base layer is going to give you a much large network download and disk footprint as I said before16:56
bogdandoone may be using distinct images16:56
jistrbogdando: yes but it doesn't actualy export or import binaries between containers. Just acts on data using its own binaries.16:56
thervemfedosin: I fixed something around that area not too long ago. I don't know what's going on16:56
dprincebogdando: you say abstracts off, I would say "builds upon". I would think each container has its own complete set of libraries and applications16:56
bogdandojistr: hm, right...16:57
bogdandosigh16:57
thervemfedosin: What does "openstack image list --long -f value -c ID --public --property latest=centos-7-latest" give you?16:57
bogdandonsenter is not a hack :D16:58
bogdandoit's been around for decades isn't it? just a tool16:58
mfedosintherve, $ openstack image list --long -f value -c ID --public --property latest=centos-7-latest16:58
mfedosinissubclass() arg 1 must be a class16:58
therveHeh16:59
bogdandowe'd need only wrap a single puppet command with it16:59
bogdandodprince, jistr: ^^16:59
bogdandoand problem solved16:59
weshaymfedosin, /me looking.. panda|rover fyi can you assist mfedosin16:59
dprincebogdando: the use of nsenter was a hack IMO because it was used in a manner that was trying to replicate a previous work around: https://review.openstack.org/#/c/620062/16:59
bogdandook, but that's not a case for this bug right?16:59
thervemfedosin: Do you have an old client or something?16:59
mfedosinit's more or less fresh... openstack 3.17.017:00
weshaymfedosin, can you point me at the log / job file you pulled the reproducer script from17:00
jistrbogdando, dprince: i'm not so sure. Read my review on your patch please. Either we'd need to "mount" puppet binary or we'd need to "mount" the OpenStack service binaries, assuming that no image has both OpenStack and Puppet binaries in it.17:00
dprincebogdando: the docker in docker pattern was a work around from a previous release. Rather than try to duplicate that pattern with podman (which doesn't have a daemon or a socket) I thought it better to get to the root of the problem and launch things more securely, exposing less privs to the runtime service containers themselves17:00
mfedosinweshay, http://logs.openstack.org/65/613165/10/check/tripleo-ci-centos-7-scenario009-multinode-oooq/1aac355/logs/17:00
openstackgerritEmilien Macchi proposed openstack/ansible-role-redhat-subscription master: sat6: pull & deploy katello certs  https://review.openstack.org/62017617:00
weshaythanks17:00
*** jpich has quit IRC17:01
bogdandojistr: oh right, so chroot maybe17:01
mfedosintherve, openstack image list works fine http://paste.openstack.org/show/736128/17:01
bogdandobut still the same problem ...17:01
bogdandoof switching it17:01
openstackgerritGael Chamoulaud proposed openstack/tripleo-validations master: Fix inspector.conf file path again.  https://review.openstack.org/62036517:01
bogdandoinside of puppet invokation17:01
jistrbogdando: yes, it's all either-or. Switching rather than combining.17:02
bogdandosecurity is hard :D17:02
*** salmankhan has joined #tripleo17:03
bogdandoand I feel not so sure now17:03
bogdandofor all that thing17:03
bogdandolet's try another day then, c ya folks17:03
jistryea i think if we want to split, we'll likely have to go build-time17:03
bogdandoright...17:03
jistr++ on security but -- on data efficiency17:03
weshaymfedosin, trying in my env w/ the same script17:05
mfedosinweshay, cool! let me know the results17:06
panda|rovermmmhh we have no wa to add -vvvv to the ansible-playbook in the reproducer ...17:08
weshaymfedosin, is your personal tenant clean? ie.. all servers, networks removed from previous work?17:08
*** sshnaidm|afk is now known as sshnaidm17:08
*** bogdando has quit IRC17:08
weshaymfedosin, this just worked for me, from your log it looks like the heat stack did not complete properly17:08
panda|roverand get more meaningful errors17:09
weshaywhich could be a resources issue or a fluke w/ rdo17:09
weshaypanda|rover, it's during the heat stack create though17:09
mfedosinweshay, what command did you use?17:09
weshaypanda|rover, reproprovision_multinode_stack.log17:10
weshay bash -x reproducer-quickstart.sh --create-virtualenv true --remove-stacks-keypairs true --nodestack-prefix repro -a17:10
panda|roverERROR: The Parameter (image) was not provided.17:10
mwhahahajrist, https://review.openstack.org/#/c/619077/ not sure if this affects tripleo or not. jfyi17:18
jristred alert17:19
*** psachin has quit IRC17:21
*** derekh has quit IRC17:23
*** ykarel|away has quit IRC17:28
*** apetrich has quit IRC17:30
*** jcoufal has quit IRC17:31
*** dmacpher has quit IRC17:32
*** sai_p has joined #tripleo17:34
*** zaneb has quit IRC17:34
openstackgerritJill Rouleau proposed openstack/paunch master: WIP: Add systemd healthchecks to podman  https://review.openstack.org/62037217:37
openstackgerritAlex Schultz proposed openstack/tripleo-heat-templates master: Switch debug to use logging classes  https://review.openstack.org/62037317:37
ChaserjimAfter adding https://github.com/openstack/tripleo-heat-templates/commit/6c17d0f1c3c8c28c401dae56d88c6dd4075bf046#diff-f84644f1b5951f535bc7c3f4151a9a90 to my code base, I now encounter the scaled-out compute with no containers running. Any thoughts?17:39
*** thrash|biab is now known as thrash17:40
*** agopi|food_run is now known as agopi17:41
*** aedc has quit IRC17:41
*** trown is now known as trown|lunch17:42
openstackgerritThomas Herve proposed openstack/tripleo-common master: Don't upcall run on UpdateParametersAction errors  https://review.openstack.org/62031917:46
*** apetrich has joined #tripleo17:50
*** florianf is now known as florianf|afk17:53
*** hberaud is now known as hberaud|gone17:54
*** ansmith has quit IRC17:59
*** salmankhan has quit IRC18:00
*** zaneb has joined #tripleo18:01
dtrainordtantsur|mtg, I'm still struggling a bit with a filter that's failing.  I'm not sure if it's because the baremetal resources are set to 0 (like the doc says to do, to match based on resources instead of standard resources.  The terminal error always seems to be:18:01
dtrainorram: -8192MB disk: -81920MB io_ops: 0 instances: 0 does not have 4096 MB usable ram before overcommit, it only has 0 MB18:01
*** ansmith has joined #tripleo18:01
*** ahosam has joined #tripleo18:03
*** krypto has quit IRC18:04
*** bnemec has quit IRC18:06
*** jcoufal has joined #tripleo18:06
*** bnemec has joined #tripleo18:06
*** jpena is now known as jpena|off18:09
*** jtomasek has quit IRC18:12
*** devep has quit IRC18:12
*** devep has joined #tripleo18:12
*** kopecmartin is now known as kopecmartin|off18:16
*** eck`gone is now known as eck`18:16
openstackgerritEmilien Macchi proposed openstack/ansible-role-redhat-subscription master: Make it more explicit that rhsm_activation_key and rhsm_repos are exclusive  https://review.openstack.org/62038018:18
EmilienMmandre: ^18:18
*** wolverineav has joined #tripleo18:24
openstackgerritEmilien Macchi proposed openstack/tripleo-docs master: RHSM: some updates for Rocky  https://review.openstack.org/62038218:25
EmilienMmarios, jaosorior : finally found the time to push the weekly doc patch :P ^18:25
*** ykarel|away has joined #tripleo18:27
*** dprince has quit IRC18:28
*** rlandy is now known as rlandy|brb18:28
*** suuuper has quit IRC18:33
*** amoralej is now known as amoralej|off18:34
cgoncalveshi folks. I updated some packages on the undercloud (THT, puppet-tripleo, puppet-octavia) and ran 'openstack undercloud upgrade'. only 2 containers have been restarted (neutron-dnsmasq-dhcp and nova-placement). is it safe to assume everything's in order?18:34
cgoncalves(container keepalived was also restarted)18:35
EmilienMcgoncalves: no18:36
EmilienMyou need to upgrade python-tripleoclient18:36
EmilienMlike it's documented18:36
EmilienMhttps://docs.openstack.org/tripleo-docs/latest/upgrade/undercloud.html18:37
EmilienMsudo yum update python-tripleoclient* openstack-tripleo-common openstack-tripleo-heat-templates18:37
EmilienMthen "openstack undercloud upgrade"18:37
cgoncalvesEmilienM, why? I have no updates for triploclient or tripleo-common18:38
EmilienMah it's not an upgrade?18:38
cgoncalvesno. I want to test some scratch builds18:39
EmilienMcontainers which restart are the ones with new image or/and new config only18:39
cgoncalvesok, makes sense since the packages I updated (THT, puppet-tripleo and puppet-octavia) do not actually bring any new config or image18:41
cgoncalvesjust wanted to make sure before I trigger OC deploy :)18:41
cgoncalvesthanks18:41
openstackgerritEmilien Macchi proposed openstack/ansible-role-redhat-subscription master: sat6: pull & deploy katello certs  https://review.openstack.org/62017618:42
openstackgerritEmilien Macchi proposed openstack/ansible-role-redhat-subscription master: sat6: pull & deploy katello certs  https://review.openstack.org/62017618:44
*** tongl has joined #tripleo18:47
*** dougbtv has joined #tripleo18:48
*** zbitter has joined #tripleo18:49
*** zaneb has quit IRC18:52
*** salmankhan has joined #tripleo18:54
*** salmankhan has quit IRC18:54
*** salmankhan1 has joined #tripleo18:54
*** eck` is now known as eck`gone18:56
*** morazi has quit IRC18:56
*** eck`gone is now known as eck`18:56
*** salmankhan1 is now known as salmankhan18:56
*** ykarel|away has quit IRC18:56
*** trown|lunch is now known as trown18:58
*** zbitter is now known as zaneb19:02
*** wolverin_ has joined #tripleo19:03
*** wolverineav has quit IRC19:05
*** vinaykns has joined #tripleo19:06
*** ahosam has quit IRC19:08
*** rlandy|brb is now known as rlandy19:09
*** salmankhan has quit IRC19:11
openstackgerritMerged openstack/tripleo-common master: Update swift_backup workflow success status message  https://review.openstack.org/61909919:12
*** devep has quit IRC19:15
*** salmankhan has joined #tripleo19:15
*** salmankhan has quit IRC19:15
*** salmankhan has joined #tripleo19:16
*** salmankhan has quit IRC19:17
*** salmankhan has joined #tripleo19:17
*** zaneb has quit IRC19:20
*** holser_ has quit IRC19:24
openstackgerritMerged openstack/puppet-tripleo master: [ui] Add option to configure apache expires  https://review.openstack.org/58957219:26
*** salmankhan has quit IRC19:28
*** gfidente has quit IRC19:28
openstackgerritNoam Angel proposed openstack/tripleo-docs master: change "--override-config-file" to "--override-ansible-cfg"  https://review.openstack.org/61150319:30
*** zaneb has joined #tripleo19:31
openstackgerritHarald Jensås proposed openstack/tripleo-heat-templates master: ctlplane pre-alloc IPs - deprecated/non-deprecated role name  https://review.openstack.org/62039119:33
*** moguimar has joined #tripleo19:34
*** moguimar has quit IRC19:34
*** shardy has quit IRC19:35
weshayjaosorior, so.. we had some failures w/ centos 7.6 but things appear to be passing atm https://tree.taiga.io/project/tripleo-ci-board/issue/309?kanban-status=202773319:39
*** wolverin_ has quit IRC19:41
*** wolverineav has joined #tripleo19:41
*** wolverineav has quit IRC19:42
*** wolverineav has joined #tripleo19:42
openstackgerritEmilien Macchi proposed openstack/tripleo-docs master: RHSM: some updates for Rocky  https://review.openstack.org/62038219:46
openstackgerritEmilien Macchi proposed openstack/tripleo-docs master: RHSM: some updates for Rocky  https://review.openstack.org/62038219:47
openstackgerritMerged openstack/tripleo-heat-templates stable/queens: Add config option for ODL IPv6 deployment  https://review.openstack.org/61224219:50
*** salmankhan has joined #tripleo19:51
*** wolverineav has quit IRC19:53
vinayknshello channel I'm working on https://bugzilla.redhat.com/show_bug.cgi?id=161360120:03
openstackbugzilla.redhat.com bug 1613601 in openstack-tripleo-heat-templates "THT should have an option to disable Panko completely." [Medium,On_dev] - Assigned to vkapalav20:03
vinayknsis there a way in heat templates that can do condition checking to avoid https://github.com/openstack/tripleo-heat-templates/blob/master/docker/services/ceilometer-agent-notification.yaml#L93 if panko is disabled..?20:04
*** devep has joined #tripleo20:06
*** dprince has joined #tripleo20:09
*** jcoufal has quit IRC20:09
*** devep has quit IRC20:09
*** jtomasek has joined #tripleo20:17
*** lennyb has quit IRC20:20
*** jrist has quit IRC20:22
openstackgerritTrevor Vardeman proposed openstack/tripleo-common master: Add manifest list image support to the kolla builder  https://review.openstack.org/62039720:22
*** wolverineav has joined #tripleo20:24
openstackgerritJill Rouleau proposed openstack/paunch master: WIP: Add systemd healthchecks to podman  https://review.openstack.org/62037220:26
*** akrivoka has quit IRC20:31
*** wolverineav has quit IRC20:33
*** wolverineav has joined #tripleo20:34
hjensasvinaykns: https://docs.openstack.org/heat/latest/template_guide/hot_spec.html#if20:35
hjensasvinaykns: the problem is how to figure out if panko was disabled.20:37
vinayknshjensas: I can have a new variable that's always true, and then in the env file having value as false if it needs to be disabled.20:38
hjensasvinaykns: yes, that should work.20:39
vinayknsmwhahaha: thx for the input.!20:40
hjensasvinaykns: it would be cool if one could check the resource registry for OS::TripleO::Services::PankoApi:, to see if it is OS::Heat::None or something. Not sure we can.20:40
vinayknshjensas: yeah that would be the ideal way to check I guess, but even I'm not sure if one can do that.20:41
*** slaweq_ has joined #tripleo20:44
*** jtomasek has quit IRC20:45
*** salmankhan has quit IRC20:52
openstackgerritNagasai Vinaykumar Kapalavai proposed openstack/tripleo-heat-templates master: Remove ties between ceilometer and panko  https://review.openstack.org/62040220:52
openstackgerritNagasai Vinaykumar Kapalavai proposed openstack/tripleo-heat-templates master: Update to the ceilometer publisher list  https://review.openstack.org/60084320:55
openstackgerritNagasai Vinaykumar Kapalavai proposed openstack/tripleo-heat-templates master: Remove ties between ceilometer and panko  https://review.openstack.org/62040220:56
*** morazi has joined #tripleo20:58
*** dprince has quit IRC20:58
*** slaweq_ has quit IRC21:03
*** ahosam has joined #tripleo21:04
openstackgerritEmilien Macchi proposed openstack/tripleo-heat-templates master: WIP - upgrade2podman: prune everything in Docker  https://review.openstack.org/62040521:05
openstackgerritEmilien Macchi proposed openstack/tripleo-heat-templates master: WIP - upgrade2podman: prune everything in Docker  https://review.openstack.org/62040521:06
*** raildo has quit IRC21:14
*** jrist has joined #tripleo21:15
*** Vorrtex has quit IRC21:18
*** jcoufal has joined #tripleo21:28
*** wolverineav has quit IRC21:32
*** wolverineav has joined #tripleo21:34
*** xek_ has quit IRC21:35
*** wolverineav has quit IRC21:38
*** wolverineav has joined #tripleo21:38
*** ansmith has quit IRC21:41
*** salmankhan has joined #tripleo21:45
*** pcaruana has quit IRC21:46
*** salmankhan has quit IRC21:46
*** salmankhan1 has joined #tripleo21:46
*** salmankhan1 is now known as salmankhan21:49
*** trown is now known as trown|outtypewww21:49
openstackgerritSteve Baker proposed openstack/tripleo-quickstart master: DNM install tar-split  https://review.openstack.org/62041322:01
*** slaweq has quit IRC22:03
openstackgerritSteve Baker proposed openstack/tripleo-common master: Implement python based uploader  https://review.openstack.org/61601822:03
openstackgerritSteve Baker proposed openstack/tripleo-common master: WIP make python uploader the default  https://review.openstack.org/61601922:03
*** redrobot has quit IRC22:08
*** saneax has joined #tripleo22:15
*** abishop has quit IRC22:16
*** slaweq has joined #tripleo22:19
*** slaweq has quit IRC22:24
*** ahosam has quit IRC22:24
*** ansmith has joined #tripleo22:26
*** bfournie has quit IRC22:27
*** wolverineav has quit IRC22:36
*** wolverineav has joined #tripleo22:37
*** wolverineav has quit IRC22:39
*** wolverin_ has joined #tripleo22:39
*** wolverin_ has quit IRC22:49
*** wolverineav has joined #tripleo22:50
*** saneax has quit IRC22:50
*** vinaykns has quit IRC22:55
*** jcoufal has quit IRC22:56
*** radeks has quit IRC22:57
*** rcernin has joined #tripleo22:57
*** wolverineav has quit IRC22:59
*** wolverineav has joined #tripleo23:02
*** salmankhan has quit IRC23:11
*** wolverin_ has joined #tripleo23:12
*** aedc has joined #tripleo23:14
*** wolverin_ has quit IRC23:15
*** wolverin_ has joined #tripleo23:15
*** wolverineav has quit IRC23:15
*** mjturek has quit IRC23:16
openstackgerritMerged openstack/puppet-tripleo stable/queens: Add support for ODL-OVS IPv6 deployment  https://review.openstack.org/61223823:21
*** ade_lee has quit IRC23:21
*** rlandy has quit IRC23:27
openstackgerritMerged openstack/diskimage-builder master: Make tripleo-buildimage-overcloud-full-centos-7 non-voting  https://review.openstack.org/62020023:32
openstackgerritJill Rouleau proposed openstack/tripleo-ansible master: Add podman_container module  https://review.openstack.org/62042323:47
*** ade_lee has joined #tripleo23:55

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