Friday, 2019-07-05

*** rlandy|bbl is now known as rlandy00:10
*** slaweq has joined #tripleo00:16
*** slaweq has quit IRC00:24
*** slaweq has joined #tripleo01:14
*** slaweq has quit IRC01:25
*** rfolco has quit IRC01:47
*** mschuppert has quit IRC01:57
*** apetrich has quit IRC01:58
EmilienMbandini: if you can review https://review.opendev.org/#/c/669222/ please02:03
EmilienMand https://review.opendev.org/#/c/669179/02:03
EmilienMand https://review.opendev.org/#/c/669173/02:03
EmilienMhttps://review.opendev.org/#/c/665739/6 and https://review.opendev.org/#/c/665743/02:04
openstackgerritEmilien Macchi proposed openstack/puppet-tripleo stable/queens: Fix bootstrap node for nova conductor  https://review.opendev.org/66668802:07
openstackgerritEmilien Macchi proposed openstack/puppet-tripleo stable/queens: Fix bootstrap node for nova conductor  https://review.opendev.org/66668802:08
openstackgerritEmilien Macchi proposed openstack/puppet-tripleo stable/queens: Fix bootstrap node for nova conductor  https://review.opendev.org/66668802:08
EmilienMamoralej|off: please review https://review.rdoproject.org/r/#/c/21180/ when you can02:09
*** mcornea has joined #tripleo02:14
*** slaweq has joined #tripleo02:15
*** slaweq has quit IRC02:24
*** skramaja has joined #tripleo02:49
*** poopcat has quit IRC03:13
*** slaweq has joined #tripleo03:14
openstackgerritafazekas proposed openstack/tripleo-common master: Use py3 compatible expression in j2  https://review.opendev.org/66924303:21
*** ramishra has joined #tripleo03:23
*** slaweq has quit IRC03:25
*** ramishra has quit IRC03:28
*** ramishra has joined #tripleo03:29
*** ykarel|away has joined #tripleo04:15
*** slaweq has joined #tripleo04:16
*** rcernin has quit IRC04:17
*** slaweq has quit IRC04:24
openstackgerritRabi Mishra proposed openstack/tripleo-common master: Stop writing allnodes_vars in inventory  https://review.opendev.org/66924704:24
*** ykarel|away is now known as ykarel04:31
*** udesale has joined #tripleo04:49
*** shyamb has joined #tripleo04:52
*** tkajinam has quit IRC04:54
*** tkajinam has joined #tripleo04:55
*** shyamb has quit IRC04:57
*** shyamb has joined #tripleo05:04
*** rcernin has joined #tripleo05:10
*** slaweq has joined #tripleo05:16
*** slaweq has quit IRC05:24
*** tkajinam_ has joined #tripleo05:25
*** tkajinam has quit IRC05:27
*** cfontain has joined #tripleo05:29
*** cfontain_ has joined #tripleo05:30
*** cfontain has quit IRC05:34
*** ykarel has quit IRC05:34
*** cfontain_ has quit IRC05:34
*** cfontain has joined #tripleo05:35
*** cfontain has quit IRC05:39
*** cfontain_ has joined #tripleo05:41
*** rcernin has quit IRC05:41
*** ratailor has joined #tripleo05:42
*** cfontain_ has quit IRC05:45
*** shyamb has quit IRC05:49
*** ykarel has joined #tripleo05:58
*** rcernin has joined #tripleo05:58
openstackgerritRajesh Tailor proposed openstack/tripleo-heat-templates master: Add new parameter NovaSchedulerLimitTenantsToPlacementAggregate  https://review.opendev.org/66925306:04
*** marios has joined #tripleo06:04
*** amoralej|off is now known as amoralej06:10
*** slaweq has joined #tripleo06:13
*** slaweq has quit IRC06:17
*** jfrancoa has joined #tripleo06:20
*** jfrancoa has quit IRC06:21
*** shyamb has joined #tripleo06:24
*** slaweq has joined #tripleo06:27
*** jfrancoa has joined #tripleo06:28
*** saneax has joined #tripleo06:32
*** slaweq has quit IRC06:35
*** mschuppert has joined #tripleo06:38
*** slaweq has joined #tripleo06:46
*** cgrosjean has joined #tripleo06:51
openstackgerritMartin Schuppert proposed openstack/tripleo-heat-templates master: Revert "Also set nova::network::neutron::dhcp_domain"  https://review.opendev.org/66927006:53
*** cfontain has joined #tripleo06:59
*** jaosorior has joined #tripleo07:02
*** jtomasek has joined #tripleo07:03
*** rpittau|afk is now known as rpittau07:06
*** xek has joined #tripleo07:06
openstackgerritChandan Kumar (raukadah) proposed openstack/python-tripleoclient master: Added for support for generating overcloud clouds.yaml  https://review.opendev.org/66458607:14
openstackgerritChandan Kumar (raukadah) proposed openstack/tripleo-heat-templates master: Reuse clouds_yaml.py from tripleo-common in tht  https://review.opendev.org/66560407:14
*** raukadah is now known as chandankumar07:14
*** hoonetorg has quit IRC07:16
openstackgerritRajesh Tailor proposed openstack/tripleo-heat-templates master: Add new parameter NovaSchedulerLimitTenantsToPlacementAggregate  https://review.opendev.org/66925307:18
*** bogdando has joined #tripleo07:20
*** florianf has joined #tripleo07:20
*** tesseract has joined #tripleo07:21
*** shyamb has quit IRC07:23
*** ccamacho has joined #tripleo07:23
*** cgrosjean has quit IRC07:28
openstackgerritFrancesco Pantano proposed openstack/tripleo-puppet-elements master: [WIP] Add ceph-grafana-dashboards to overcloud image  https://review.opendev.org/66921207:29
*** hoonetorg has joined #tripleo07:29
*** pkopec has joined #tripleo07:30
*** ykarel is now known as ykarel|lunch07:35
*** pbandark has quit IRC07:37
*** jfrancoa has quit IRC07:39
*** cylopez has joined #tripleo07:40
*** holser_ has joined #tripleo07:42
*** jpich has joined #tripleo07:44
*** rcernin has quit IRC07:45
*** cylopez has quit IRC07:45
*** holser__ has joined #tripleo07:48
*** holser_ has quit IRC07:48
openstackgerritMarios Andreou proposed openstack/tripleo-ci master: Adds build-images role - migrates image build in ci to a role  https://review.opendev.org/66796607:49
*** jfrancoa has joined #tripleo07:51
*** holser_ has joined #tripleo07:52
*** holser__ has quit IRC07:54
*** cfontain has quit IRC07:55
*** cylopez has joined #tripleo08:00
*** tosky has joined #tripleo08:01
*** gkadam has joined #tripleo08:02
*** rcernin has joined #tripleo08:02
*** xek has quit IRC08:10
*** pbandark has joined #tripleo08:10
*** tkajinam_ has quit IRC08:11
*** marios has quit IRC08:12
*** lucasagomes has joined #tripleo08:14
openstackgerritArx Cruz proposed openstack/tripleo-quickstart-extras master: WIP Add playbook and role to run tobiko  https://review.opendev.org/65542308:16
openstackgerritCarlos Camacho proposed openstack/tripleo-heat-templates master: Stop services for unupgraded controllers  https://review.opendev.org/66909508:20
openstackgerritCédric Jeanneret (Tengu) proposed openstack/tripleo-heat-templates master: Run NetworkConfig only if configuration script exists  https://review.opendev.org/66865208:21
openstackgerritFrancesco Pantano proposed openstack/tripleo-puppet-elements master: [WIP] Add ceph-grafana-dashboards to overcloud image  https://review.opendev.org/66921208:28
*** pbandark has quit IRC08:32
*** shyamb has joined #tripleo08:35
openstackgerritSaravanan KR proposed openstack/os-net-config stable/stein: Restructuring the method of vf-representor renaming  https://review.opendev.org/66928908:37
openstackgerritSaravanan KR proposed openstack/os-net-config stable/rocky: Restructuring the method of vf-representor renaming  https://review.opendev.org/66929008:37
openstackgerritSaravanan KR proposed openstack/os-net-config stable/queens: Restructuring the method of vf-representor renaming  https://review.opendev.org/66929108:37
openstackgerritRabi Mishra proposed openstack/tripleo-heat-templates master: Remove puppet/deploy-artifacts.yaml  https://review.opendev.org/66929208:39
openstackgerritNatal Ngétal proposed openstack/tripleo-common master: [Core] Be pep8 compliant.  https://review.opendev.org/63424108:42
*** cfontain has joined #tripleo08:44
*** ykarel|lunch is now known as ykarel08:44
*** rcernin has quit IRC08:49
openstackgerritBogdan Dobrelya (bogdando) proposed openstack/tripleo-heat-templates master: Ensure purging of the deleted httpd module configs  https://review.opendev.org/66917808:57
*** rcernin has joined #tripleo09:05
*** jistr has quit IRC09:07
*** jistr has joined #tripleo09:14
openstackgerritMerged openstack/tripleo-common master: Python 3 fixes for tripleo-hieradata role  https://review.opendev.org/66921909:15
*** rcernin has quit IRC09:16
*** marios has joined #tripleo09:25
openstackgerritMerged openstack/tripleo-quickstart master: Avoids use of ignore_errors while loading OS specific variables  https://review.opendev.org/61711809:25
*** ykarel_ has joined #tripleo09:33
*** ykarel has quit IRC09:35
*** ykarel_ is now known as ykarel|meeting09:35
*** pdeore has joined #tripleo09:35
*** jaosorior has quit IRC09:36
openstackgerritFrancesco Pantano proposed openstack/tripleo-puppet-elements master: [WIP] [EXP] Add ceph-grafana-dashboards to overcloud image  https://review.opendev.org/66921209:48
Tengufolks, care to make that one merge? https://review.opendev.org/#/c/668964/09:54
openstackgerritMerged openstack/python-tripleoclient stable/stein: Add log_path_dir parameter  https://review.opendev.org/66906809:58
*** cgrosjean has joined #tripleo10:05
*** shyamb has quit IRC10:15
openstackgerritMartin Schuppert proposed openstack/tripleo-heat-templates master: Don't use the z flag in case NovaNfsEnabled is true  https://review.opendev.org/66931710:17
*** ykarel_ has joined #tripleo10:18
*** ykarel|meeting has quit IRC10:20
*** mschuppert has quit IRC10:32
*** brault has joined #tripleo10:32
*** xek has joined #tripleo10:36
*** cgrosjean has quit IRC10:36
*** brault has quit IRC10:37
*** ykarel_ is now known as ykarel10:43
openstackgerritGael Chamoulaud proposed openstack/tripleo-common master: Get all the available Ansible variables for each validations  https://review.opendev.org/66932410:50
*** shyamb has joined #tripleo10:51
*** pbandark has joined #tripleo10:53
chandankumarhjensas: Hello10:53
chandankumarhjensas: https://review.opendev.org/#/q/topic:overcloud_cloudyaml+(status:open+OR+status:merged) patches are now ready for review10:54
openstackgerritFrancesco Pantano proposed openstack/tripleo-heat-templates master: Add dashboard component as new ceph composable service.  https://review.opendev.org/66657810:58
*** pdeore has quit IRC10:59
*** ykarel is now known as ykarel|afk11:01
*** pbandark has quit IRC11:03
*** slagle-july-4th is now known as slagle11:04
*** ykarel|afk has quit IRC11:07
openstackgerritMarios Andreou proposed openstack/tripleo-ci master: Add tripleo-build-images-base job & reparent existing build jobs  https://review.opendev.org/66932911:10
*** shyamb has quit IRC11:14
*** brault has joined #tripleo11:14
openstackgerritMarios Andreou proposed openstack/tripleo-ci master: Adds build-images role - migrates image build in ci to a role  https://review.opendev.org/66796611:16
mariospanda: thanks for voting - updated please re-vote? ^11:16
skramajahjensas: few backports for review - https://review.opendev.org/#/q/topic:bug/1830716+status:open11:24
*** apetrich has joined #tripleo11:26
*** ykarel|afk has joined #tripleo11:27
*** ykarel|afk is now known as ykarel11:28
*** ratailor has quit IRC11:28
openstackgerritSorin Sbarnea proposed openstack/tripleo-quickstart master: WIP: Adds RHEL8 testing to molecule  https://review.opendev.org/66922311:41
*** shyamb has joined #tripleo11:41
openstackgerritSorin Sbarnea proposed openstack/tripleo-quickstart master: WIP: Adds RHEL8 testing to molecule  https://review.opendev.org/66922311:41
*** mcornea has quit IRC11:43
*** brault has quit IRC11:46
*** brault has joined #tripleo11:47
* beagles wonders when the last time was that someone simply ran "overcloud deploy" with all defaults 11:48
*** ykarel_ has joined #tripleo11:50
*** ykarel has quit IRC11:54
*** florianf has quit IRC11:56
openstackgerritCarlos Camacho proposed openstack/tripleo-common master: Add playbook to stop containers  https://review.opendev.org/66893811:58
*** openstackgerrit has quit IRC12:04
*** cgrosjean has joined #tripleo12:05
*** openstackgerrit has joined #tripleo12:06
openstackgerritSorin Sbarnea proposed openstack/tripleo-ci master: WIP: Adds RHEL8 to molecule  https://review.opendev.org/66934312:06
*** rh-jelabarre has joined #tripleo12:06
openstackgerritHarald Jensås proposed openstack/tripleo-heat-templates master: Drop EC2MetadataIp parameter and its uses  https://review.opendev.org/66537312:07
*** florianf has joined #tripleo12:11
openstackgerritBrent Eagles proposed openstack/tripleo-quickstart-extras master: Add a tasks to download amphora image for octavia deployments  https://review.opendev.org/66934412:15
openstackgerritCarlos Camacho proposed openstack/tripleo-common stable/stein: Add playbook to stop containers  https://review.opendev.org/66934512:17
*** jcoufal has joined #tripleo12:19
*** ansmith has joined #tripleo12:25
*** xek has quit IRC12:26
*** xek_ has joined #tripleo12:26
*** rfolco has joined #tripleo12:26
*** iurygregory has joined #tripleo12:31
*** rlandy has joined #tripleo12:31
*** abishop has joined #tripleo12:31
*** psachin has joined #tripleo12:33
*** cfontain has quit IRC12:34
*** skramaja_ has joined #tripleo12:39
*** skramaja has quit IRC12:40
*** skramaja_ has quit IRC12:42
*** ansmith has quit IRC12:45
*** udesale has quit IRC12:51
*** udesale has joined #tripleo12:51
*** tesseract has quit IRC12:52
*** ansmith has joined #tripleo12:52
*** tesseract has joined #tripleo12:54
*** shyamb has quit IRC12:55
*** shyamb has joined #tripleo12:55
*** ykarel_ is now known as ykarel12:57
shyambHi13:04
shyambykarel: I have uploaded a yum repo file through swift artifacts upload way13:05
shyambto overcloud13:05
*** ansmith has quit IRC13:05
shyamblooks like it is changing ownership of '/etc' directory altogether13:05
ykarelshyamb, /me have not used this, but i think it should not change ownership of /etc directory, it should just touch the repo file13:13
ykarelshyamb, btw what it changed?which owner and permissions?13:13
shyambykarel: http://paste.openstack.org/show/753891/13:13
shyambit was root:root earlier13:14
shyambwe reproduced on two clouds13:14
shyambNow it's heat-admin:hugetlbfs13:14
shyambThis is the script we are using "https://github.com/openstack/tripleo-common/blob/newton-eol/scripts/upload-swift-artifacts"13:16
ykarelshyamb, hmm looks bad, permission should not change for base directories13:18
shyambyeah13:18
*** cgrosjean has quit IRC13:19
*** cgrosjean has joined #tripleo13:19
openstackgerritSorin Sbarnea proposed openstack/tripleo-ci master: WIP: Adds RHEL8 to molecule  https://review.opendev.org/66934313:20
*** shyamb has quit IRC13:25
*** raildo has joined #tripleo13:26
owalshykarel: I expect the problem is the ownership of the files/dirs shyamb is uploading. E.g if they are owned by stack:stack on the undercloud I'd guess that uid/gid is heat-admin:hugetlbfs on the overcloud nodes13:29
*** amoralej is now known as amoralej|lunch13:29
openstackgerritFrancesco Pantano proposed openstack/tripleo-heat-templates master: Add dashboard component as new ceph composable service.  https://review.opendev.org/66657813:29
openstackgerritEmilien Macchi proposed openstack/tripleo-ci master: Revert "Make tripleo-ci-fedora-28-standalone non voting"  https://review.opendev.org/66935613:36
EmilienMmarios: ^ please look13:36
openstackgerritEmilien Macchi proposed openstack/tripleo-ci master: Revert "Make tripleo-ci-fedora-28-standalone non voting"  https://review.opendev.org/66935613:36
EmilienMSorin too ^13:37
ykarelowalsh, ack possibly, but why permission got changed for base directory(/etc)13:38
owalshykarel: because that dir was in the tarball13:38
*** zaneb has joined #tripleo13:38
ykarelowalsh, okk so https://github.com/openstack/tripleo-heat-templates/blob/master/puppet/deploy-artifacts.sh#L18 is executed13:39
owalshykarel: yup13:39
EmilienMcan someone review https://review.opendev.org/#/c/669173/ please13:39
openstackgerritCarlos Camacho proposed openstack/tripleo-heat-templates master: Stop services for unupgraded controllers  https://review.opendev.org/66909513:40
owalshykarel: I'd guess there is something like /home/stack/upload_dir/etc/yum.repos.d/foo.repo on the undercloud, and the ownership of the etc dir ain't root13:40
owalshso tar chowns it when it extracts the tarball13:41
owalshand it just happens that the undercloud stack user and the overcloud heat-admin users have the same uid (first user created)13:42
ykarelowalsh, hmm ^^ looks correct, but from where hugetlbfs came13:42
owalshykarel: undercloud stack gid -> overcloud hugetlbfs gid13:42
ykarelowalsh, let me check in my setup13:43
owalshykarel: ack, I can't think of anything else that could cause this13:43
ykareli don't have a setup with stack user13:44
ykarelowalsh, hmm i see gid of hugetlbfs is 1000 on overcloud13:45
owalshykarel: yup, I'd expect stack:stack to be 1000:1000 on the undercloud13:46
owalshor whatever the first user created is called13:47
ykarelowalsh, right, just checked hugetlbfs:x:1000:openvswitch on overcloud and zuul:x:1000: on undercloud in /etc/group13:48
ykarelhttps://logs.rdoproject.org/openstack-periodic-master/opendev.org/openstack/tripleo-ci/master/periodic-tripleo-ci-centos-7-ovb-1ctlr_2comp-featureset020-master/d1be891/logs/undercloud/etc/group.txt.gz13:49
*** ayoung has joined #tripleo13:49
owalshykarel: ack. Looks like shyamb is offline now though13:49
ykarelowalsh, yes13:49
fultonjEmilienM: i'll review it13:49
EmilienMfultonj: thx13:50
openstackgerritEmilien Macchi proposed openstack/paunch master: Reduce the usage of "podman inspect" command  https://review.opendev.org/66573113:50
*** gkadam_ has joined #tripleo13:50
owalshykarel: I'll drop him an email13:50
ykarelowalsh, ack13:50
*** gkadam has quit IRC13:52
openstackgerritMerged openstack/tripleo-heat-templates stable/stein: Add container engine authentication support  https://review.opendev.org/66922213:52
cloudnullmornings13:55
*** ekultails has joined #tripleo13:56
*** sabedevops has joined #tripleo13:59
*** ratailor has joined #tripleo14:00
*** morazi has joined #tripleo14:01
mariosEmilienM: ack was on call looking in a sec14:01
*** amoralej|lunch is now known as amoralej14:02
*** sabedevops has quit IRC14:02
openstackgerritChandan Kumar (raukadah) proposed openstack/tripleo-quickstart master: Added RedHat-8 release file  https://review.opendev.org/66905714:06
*** ansmith has joined #tripleo14:07
openstackgerritMerged openstack/tripleo-common master: Stop writing allnodes_vars in inventory  https://review.opendev.org/66924714:18
*** cylopez has quit IRC14:18
openstackgerritBogdan Dobrelya (bogdando) proposed openstack/puppet-tripleo master: Apache MPM event for better TCP keep alive in WSGI  https://review.opendev.org/66886214:20
*** ansmith has quit IRC14:24
openstackgerritKevin Carter (cloudnull) proposed openstack/tripleo-ansible master: Add sphinx_python to the reno jobs  https://review.opendev.org/66936314:26
*** zul has quit IRC14:29
openstackgerritKevin Carter (cloudnull) proposed openstack/tripleo-ansible master: Add legacy variable scenario tests  https://review.opendev.org/66760914:30
openstackgerritGael Chamoulaud proposed openstack/python-tripleoclient master: Allow running validations by name using Ansible by default.  https://review.opendev.org/65516514:31
openstackgerritGael Chamoulaud proposed openstack/python-tripleoclient master: Allow running validations by group using Ansible by default  https://review.opendev.org/66885514:31
openstackgerritKevin Carter (cloudnull) proposed openstack/tripleo-ansible master: Remove irrelevant files  https://review.opendev.org/66858214:33
openstackgerritChandan Kumar (raukadah) proposed openstack/tripleo-common master: Added support of overcloud os_cloud  https://review.opendev.org/66456814:35
chandankumarEmilienM: mwhahaha feel free to take a look on these patches https://review.opendev.org/#/q/topic:overcloud_cloudyaml+(status:open+OR+status:merged)14:36
openstackgerritCédric Jeanneret (Tengu) proposed openstack/ansible-role-chrony master: Ensure service is enabled  https://review.opendev.org/66936514:38
cloudnullany cores around mind kicking https://review.opendev.org/#/c/669363 through?14:39
Tengucloudnull: can't do anything outside of tripleo-validations ;_;14:39
cloudnullnp.14:40
*** jcoufal has quit IRC14:40
* cloudnull can't do anything ... :D14:41
*** ykarel is now known as ykarel|afk14:41
openstackgerritSorin Sbarnea proposed openstack/tripleo-repos master: Fix mirror replacement logic for empty ones  https://review.opendev.org/66936614:42
openstackgerritSorin Sbarnea proposed openstack/tripleo-quickstart master: WIP: Adds RHEL8 testing to molecule  https://review.opendev.org/66922314:42
openstackgerritArx Cruz proposed openstack/tripleo-quickstart-extras master: WIP Add playbook and role to run tobiko  https://review.opendev.org/65542314:47
*** chandankumar is now known as raukadah14:48
openstackgerritSorin Sbarnea proposed openstack/tripleo-repos master: Fix mirror replacement logic for empty ones  https://review.opendev.org/66936614:51
*** vinaykns has joined #tripleo14:59
openstackgerritArx Cruz proposed openstack/tripleo-quickstart-extras master: WIP Add playbook and role to run tobiko  https://review.opendev.org/65542315:01
*** bogdando has quit IRC15:02
fultonjcloudnull: i have a question you might know the answer to in https://review.opendev.org/#/c/668294 . would you mind taking a look?15:04
fultonjcloudnull: basically i want molecue to use tags15:05
* fultonj +2'd https://review.opendev.org/#/c/669363 15:07
fultonji hit that same issue in my 66829415:08
*** tesseract-RH has joined #tripleo15:08
*** xek_ has quit IRC15:09
*** tesseract has quit IRC15:10
openstackgerritMerged openstack/tripleo-ansible master: Add sphinx_python to the reno jobs  https://review.opendev.org/66936315:10
openstackgerritGael Chamoulaud proposed openstack/tripleo-docs master: Add support for running validations through ansible-playbook  https://review.opendev.org/66936815:11
*** holser__ has joined #tripleo15:11
*** holser_ has quit IRC15:12
*** rpittau is now known as rpittau|afk15:12
*** saneax has quit IRC15:14
openstackgerritGael Chamoulaud proposed openstack/tripleo-common master: Get all the available Ansible variables for each validations  https://review.opendev.org/66932415:18
gchamoulEmilienM: hi, could you please approve this one when you have 2 min? https://review.opendev.org/#/c/668964/15:23
EmilienMgchamoul: sure thing15:24
EmilienMand done15:24
cloudnullfultonj sorry was afk a minute, looking now15:25
openstackgerritSorin Sbarnea proposed openstack/tripleo-repos master: WIP: Fix mirror replacement logic for empty ones  https://review.opendev.org/66936615:27
cloudnullfultonj it looks like we can pass in ansible arguments, we just need to extend our entry point so that sees them15:28
openstackgerritSorin Sbarnea proposed openstack/tripleo-repos master: WIP: Fix mirror replacement logic for empty ones  https://review.opendev.org/66936615:28
openstackgerritSorin Sbarnea proposed openstack/tripleo-ci master: WIP: Adds RHEL8 to molecule  https://review.opendev.org/66934315:30
*** cfontain_ has joined #tripleo15:31
*** marios is now known as marios|out15:32
*** lucasagomes has quit IRC15:33
*** jfrancoa has quit IRC15:36
*** marios|out has quit IRC15:37
*** morazi has quit IRC15:39
fultonjcloudnull: how do i extend the entry point to pass molecule arguments?15:43
fultonjhttps://tox.readthedocs.io/en/latest/example/basic.html#integration-with-setup-py-test-command15:43
fultonj^ ?15:44
zbrfultonj: depends on the jobs. For those doing tox->pytest->molecule you can only define env vars, no arguments because {posargs} are for pytest itself.15:46
zbrwhat do is that I run molecule directly when i need extra params15:46
fmountHi there, during my ceph_dashboard work I found that a package is needed in the overcloud host and I used this: https://review.opendev.org/#/c/669212/ to install it. Is this a good way to do that?15:47
fmount Should I create a tht review that Depends-On the tripleo-puppet-elements one?15:47
fmountor I need to do some extra work?15:47
*** amoralej is now known as amoralej|off15:47
openstackgerritKevin Carter (cloudnull) proposed openstack/tripleo-ansible master: Extend pytest so it can accept ansible args  https://review.opendev.org/66937815:47
cloudnullfultonj ^15:48
cloudnullmind giving that a spin15:48
cloudnullit should give you the ability to pass in arbitrary ansible arguments to a given test15:48
zbrcloudnull: I already dropped using the tests/test_molecule.py as i replaced it with pytest-molecule. we may need to add features there.15:48
fultonjcloudnull: yeah, i'll try it thank you15:48
fultonjEmilienM: i'm trying to help fmount with his question above but i'm not sure. can you help?15:49
gchamoulEmilienM: thx!15:49
fmountfultonj: thx, EmilienM I created this review: https://review.opendev.org/#/c/666578 that Depends-On the tripleo-puppet-elements one, but I don't see the package installed in the host15:50
fmountmaybe there is something missing or it's not a correct depends chain15:51
cloudnullzbr we're still using test_molecule.py in tripleo-ansible.15:52
zbrcloudnull: i did not had time to propose the change to remove it :p15:52
openstackgerritGael Chamoulaud proposed openstack/tripleo-validations master: Add different logic when running against the Undercloud or the Controller  https://review.opendev.org/66566115:55
*** ratailor_ has joined #tripleo15:56
*** ratailor_ has quit IRC15:56
*** ratailor has quit IRC15:56
openstackgerritJohn Fulton proposed openstack/tripleo-ansible master: WIP: Add molecule test structure to ceph roles  https://review.opendev.org/66829416:01
openstackgerritLuke Short proposed openstack/paunch master: Handle defined containers that are stopped.  https://review.opendev.org/64291016:16
*** florianf has quit IRC16:16
fultonjcloudnull: am i using your change incorrectly? http://paste.openstack.org/show/75390016:16
*** iurygregory has quit IRC16:19
*** jpich has quit IRC16:20
cloudnullyes that looks right, but let me give that a spin locally to see whats wrong16:21
*** cgrosjean has quit IRC16:22
openstackgerritMerged openstack/tripleo-heat-templates master: Make nova ephemeral storage backend configurable per-role  https://review.opendev.org/64830116:22
*** rlandy is now known as rlandy|brb16:23
*** udesale has quit IRC16:28
*** dmacpher_ has joined #tripleo16:34
*** rlandy|brb is now known as rlandy16:35
openstackgerritMerged openstack/tripleo-heat-templates master: Disable Docker in default Standalone environments  https://review.opendev.org/66917316:36
*** gkadam_ has quit IRC16:37
*** dmacpher has quit IRC16:37
openstackgerritKevin Carter (cloudnull) proposed openstack/tripleo-ansible master: Extend pytest so it can accept ansible args  https://review.opendev.org/66937816:39
cloudnullfultonj if you're still about, mind giving that a spin ?16:39
fultonjcloudnull: yes, thanks i'll try it16:39
*** cfontain_ has quit IRC16:41
openstackgerritSorin Sbarnea proposed openstack/tripleo-repos master: Fix mirror replacement logic for empty ones  https://review.opendev.org/66936616:43
openstackgerritSorin Sbarnea proposed openstack/openstack-virtual-baremetal master: Removed deprecarted --centos-mirror param  https://review.opendev.org/66939616:46
openstackgerritJohn Fulton proposed openstack/tripleo-ansible master: WIP: Add molecule test structure to ceph roles  https://review.opendev.org/66829416:49
openstackgerritSorin Sbarnea proposed openstack/openstack-virtual-baremetal master: Removed deprecated --centos-mirror param  https://review.opendev.org/66939616:49
openstackgerritGael Chamoulaud proposed openstack/tripleo-validations master: Remove useless meta directory for the Ansible ntp role  https://review.opendev.org/66939716:51
*** slaweq has quit IRC16:52
openstackgerritGael Chamoulaud proposed openstack/tripleo-validations master: Support multi-arch in deploy image validations  https://review.opendev.org/66673016:53
openstackgerritFrancesco Pantano proposed openstack/tripleo-heat-templates master: Add dashboard component as new ceph composable service.  https://review.opendev.org/66657816:53
openstackgerritKevin Carter (cloudnull) proposed openstack/tripleo-ansible master: Extend pytest so it can accept ansible args  https://review.opendev.org/66937816:54
cloudnullsorry fultonj ^ one more revision (i think) so that cleanup is envoked.16:55
* cloudnull testing locally, and that seems to work when passing extra ansible args in with some of our existing roles 16:55
fultonjcloudnull: no problem, ps2 worked for me also!16:56
cloudnullawesome!16:56
fultonjcloudnull: i'll test again with ps3 (finally cleanup)16:57
fultonjthanks cloudnull16:57
cloudnullthank you!16:58
*** tesseract-RH has quit IRC17:01
fultonjcloudnull: because of the way my roles depend on eachother you can test one role and it will include testing all of the others17:01
openstackgerritFrancesco Pantano proposed openstack/tripleo-heat-templates master: Add dashboard component as new ceph composable service.  https://review.opendev.org/66657817:01
fultonjcloudnull: so i'm wondering if i should only have a tox.ini modification for that one role17:01
cloudnull++ just the one likely makes the most sense17:01
cloudnullmaybe one per-scenario (assuming you have more than one)17:02
openstackgerritJohn Fulton proposed openstack/tripleo-ansible master: Add molecule test structure to ceph roles  https://review.opendev.org/66829417:04
*** dmacpher__ has joined #tripleo17:05
*** ykarel|afk is now known as ykarel|away17:07
*** dmacpher_ has quit IRC17:07
cloudnullthese were rebased due to the reno change we needed to land, any chance a core can re-workflow them https://review.opendev.org/#/c/668582 https://review.opendev.org/#/c/667609 ?17:07
fultonjcloudnull: i can17:08
EmilienMfultonj: looking17:08
EmilienMfmount, fultonj : I know17:08
EmilienMyou need to Depends-On: I56d103f3b3696393b0a630446e8b55fd18a0164e17:08
EmilienMOVB jobs are run in RDO THird Party, so an external Zuul17:09
EmilienMand AFIK it doesn't support URLs.17:09
EmilienMit needs to be the Change ID in Gerrit.17:09
cloudnullthanks fultonj17:09
EmilienMif your issue was with OVb jobs.17:09
openstackgerritBrent Eagles proposed openstack/tripleo-quickstart-extras master: Add a tasks to download amphora image for octavia deployments  https://review.opendev.org/66934417:09
EmilienM(btw only OVB jobs exercise testing of t-p-e patches)17:09
fultonjthanks EmilienM17:10
fmountoh thanks EmilienM17:11
openstackgerritJohn Fulton proposed openstack/tripleo-ansible master: Add molecule test structure to ceph roles  https://review.opendev.org/66829417:11
*** holser__ has quit IRC17:11
EmilienMfultonj, fmount : I think it was depending on zuul v317:15
EmilienMwhich AFIK has been deployed in RDO but since it's a fork I'm not sure this feature was rebased17:15
EmilienMbetter ask tristanC17:15
EmilienMfultonj, fmount : FYI http://lists.openstack.org/pipermail/openstack-dev/2018-February/127516.html17:16
fmountoh thanks for this ^17:17
EmilienMnp17:19
ykarel|awayEmilienM, tristanC is out till mid july, is there some issue?17:20
EmilienMykarel|away: can we use Depends-On: URL in RDO CI?17:20
ykarel|awayEmilienM, yes17:21
EmilienMeg: https://review.opendev.org/#/c/66657817:21
ykarel|awayurl should be opendev.org17:21
ykarel|awayEmilienM, ^^ looks fine17:21
*** saneax has joined #tripleo17:21
ykarel|awayEmilienM, from job logs i can see Updated: openstack-tripleo-puppet-elements-11.0.1-0.20190705134545.4a9a0f5.el7.noarch17:22
ykarel|awaymeans depends on is working17:22
EmilienMykarel|away: thanks17:22
fmountEmilienM: ykarel|away we basically need to install a package on the overcloud host when the ceph_dashboard is enabled: this package provides all the grafana templates for ceph17:22
fmountso basically https://review.opendev.org/#/c/66657817:22
fmountDepends-On: https://review.opendev.org/#/c/669212/17:23
fmountIs the pattern I'm following correct?17:23
fmountor we should avoid doing that?17:23
fmountEmilienM: ykarel|away ^ wdyt?17:23
ykarel|awayfmount, ^^ pattern is correct and package is installed in undercloud17:23
ykarel|awayme seeing https://logs.rdoproject.org/78/666578/24/openstack-check/tripleo-ci-centos-7-ovb-3ctlr_1comp-featureset001/341a334/logs/undercloud/var/log/yum.log.txt.gz17:23
fmountykarel|away: oh, I want to install that package on the overcloud hosts17:24
fmountit creates a path that we pass as a volume to the grafana-server container17:25
ykarel|awayfmount, u want tripleo-puppet-elements package to be installed in overcloud?17:26
ykarel|awayor u want to apply that element while creating overcloud image?17:26
EmilienMthat ^17:27
fultonjto apply that element while creating overcloud image17:27
fmountykarel|away: fultonj exactly ^17:27
ykarel|awayso since that package is installed in undercloud, it must be used while creating image17:27
ykarel|awayhttps://logs.rdoproject.org/78/666578/24/openstack-check/tripleo-ci-centos-7-ovb-3ctlr_1comp-featureset001/341a334/logs/undercloud/home/zuul/overcloud_image_build.log.txt.gz17:27
openstackgerritEmilien Macchi proposed openstack/puppet-tripleo master: Replace hiera('service_names') by hiera('enabled_services')  https://review.opendev.org/66941417:28
ykarel|awayso it's being used17:28
ykarel|awayas per the logs17:28
ykarel|awayceph-grafana-dashboards package is installed17:29
fultonjcloudnull: it seems I can update molecule.yaml so theres only one ceph job which tests a job which depends on the others17:34
fultonjcloudnull: but that i could also keep a tox.ini entry for each job in case a developer wants to run less tests17:35
fultonjif they're just working on one rule17:35
cloudnull++17:35
fmountykarel|away: yes but I want to find the package here > http://logs.openstack.org/78/666578/24/check/tripleo-ci-centos-7-scenario001-standalone/19e7b8b/logs/undercloud/var/log/extra/rpm-list.txt.gz17:35
fmountykarel|away: I think I miss something :/17:36
ykarel|awayfmount, but ^^ is standalone, where tripleo-puppet-elements does have any effect17:36
fmountykarel|away: ack, this explain why I don't see the pkg17:37
ykarel|awayack17:39
* ykarel|away out17:39
*** psachin has quit IRC17:42
openstackgerritJohn Fulton proposed openstack/tripleo-ansible master: Add molecule test structure to ceph roles  https://review.opendev.org/66829417:43
*** ykarel|away has quit IRC17:44
openstackgerritMatthew Thode proposed openstack/diskimage-builder master: disable autounmask for emerge  https://review.opendev.org/66942217:49
openstackgerritEmilien Macchi proposed openstack/tripleo-heat-templates master: Remove service_names hieradata file  https://review.opendev.org/66942417:50
*** saneax has quit IRC17:52
*** saneax has joined #tripleo17:53
*** cfontain has joined #tripleo17:53
*** cfontain has quit IRC17:55
*** slaweq has joined #tripleo17:56
*** poopcat has joined #tripleo18:02
*** slaweq has quit IRC18:05
rlandyEmilienM: do you happen to know if there are ceph containers built for rhel 8 that are available for rdo deployments in rdocloud?18:16
rlandywe have ceph on the upshift registry but it's internal18:17
cloudnullanyone around want to give https://review.opendev.org/#/c/669378 a review18:18
fultonj      ceph_image: daemon18:23
fultonj      ceph_namespace: docker.io/ceph18:23
fultonj      ceph_tag: v4.0.0-stable-4.0-nautilus-centos-7-x86_6418:23
fultonjrlandy: ^18:23
fultonjwill work on rhel818:23
rlandyfultonj: thank you!!18:23
fultonjrlandy: sure, it's based on cent7 but should be sufficient and it's fully upstream18:24
rlandyhttps://logs.rdoproject.org/70/21270/21/check/periodic-tripleo-ci-rhel-8-standalone-master/93f5301/logs/undercloud/home/zuul/standalone_container_prep.log.txt.gz18:24
rlandyhmmm ... actually that's what we tried18:24
fultonjrlandy: and it failed?18:24
openstackgerritEmilien Macchi proposed openstack/tripleo-heat-templates master: Remove service_names hieradata file  https://review.opendev.org/66942418:25
fultonjyou can use registry.redhat.io/rhceph-beta/rhceph-4-rhel8:latest but it requires auth which you can do with ContainerImageRegistryCredentials18:25
fultonjbut maybe you dont' want to hard code ContainerImageRegistryCredentials18:25
fultonjwhich is understanble18:26
rlandyit did ...18:26
rlandywe used docker_registry_host: trunk.rdoproject.org18:26
rlandy - which may be wrong18:26
fultonjis there a log, maybe i can debug?18:26
rlandyhttps://logs.rdoproject.org/70/21270/21/check/periodic-tripleo-ci-rhel-8-standalone-master/93f5301/logs/undercloud/home/zuul/standalone_deploy.log.txt.gz#_2019-07-05_10_45_3818:26
rlandyfultonj: this is rdo master on rhel 818:27
rlandyin fact all the rdo on rhel containers are missing - but ceph should be independent18:28
openstackgerritRonelle Landy proposed openstack/tripleo-quickstart master: Added RedHat-8 release file  https://review.opendev.org/66905718:31
fultonjhttp://logs.openstack.org/73/669173/2/check/tripleo-ci-centos-7-scenario001-standalone/92f419c/logs/undercloud/home/zuul/containers-prepare-parameters.yaml.txt.gz18:32
fultonjrlandy: ^ standalone001 uses the same ceph_* parameters under ContainerImagePrepare and is green18:33
rlandycentos7 as opposed to rhel818:34
fultonjrlandy: yes on centos7 host os not rhel8 host os18:34
fultonjbut i've tested the same container on rhel8 in my env and it works on rhel818:34
* rlandy compares params18:34
rlandyok - reverting params - let's see what happens18:35
fultonjif it was a rhel8 issue I'd expect an error about the container itself not working for some reason18:35
fultonj2019-07-05 10:45:55 | requests.exceptions.HTTPError: 404 Client Error: Not Found for url: https://trunk.rdoproject.org/v2/18:35
fultonjrlandy: is it only when you deploy with ceph that you get ^ ?18:36
fultonjhttps://logs.rdoproject.org/70/21270/21/check/periodic-tripleo-ci-rhel-8-standalone-master/93f5301/logs/undercloud/home/zuul/standalone.sh.txt.gz18:37
rlandyit's because I've set the registry to where we want to build the containers - setting it back to docker.io to use centos7 containers on rhel818:37
fultonjlooks like a standalone deploy without ceph18:37
rlandysince you say it works18:37
fultonjworks for ceph18:38
fultonjthe centos7-based nautilus contianer worked on rhel8 in my env18:38
openstackgerritRonelle Landy proposed openstack/tripleo-quickstart master: Added RedHat-8 release file  https://review.opendev.org/66905718:38
openstackgerritMerged openstack/tripleo-ansible master: Add legacy variable scenario tests  https://review.opendev.org/66760918:47
openstackgerritRonelle Landy proposed openstack/tripleo-quickstart master: Added RedHat-8 release file  https://review.opendev.org/66905718:58
openstackgerritRonelle Landy proposed openstack/tripleo-quickstart master: Added RedHat-8 release file  https://review.opendev.org/66905719:00
zbrcloudnull: would you like to become a core on pytest-molecule? I kinda dislike being the single point of failure on it ;)19:04
zbri agree with your last bug, docker should not be mandatory. still I am wondering how to make it smarter, probably I need to read the molecule.yml file and check the driver from there.19:05
EmilienMzbr: FYI https://review.opendev.org/#/c/669356/19:05
openstackgerritMerged openstack/tripleo-ansible master: Remove irrelevant files  https://review.opendev.org/66858219:05
zbrEmilienM: added a +1 on it, its all I got atm, but I fully support it. People ignored it for too long.19:07
cloudnullzbr sure. I'd be happy to help out with that plugin19:07
zbrcloudnull: re selinux, not sure which approach is the best. it used to be a fatal error, now is only a warning. i wanted to fail fast as I got quite often issues related to its absence.19:10
zbrbut maybe i am trying to add too much logic into it19:11
*** cfontain has joined #tripleo19:18
fultonjcloudnull: https://review.opendev.org/#/c/668294/9/zuul.d/molecule.yaml@14619:20
fultonjcloudnull: i could but i know run-ceph-ansible already has the deps defined to run the other jobs19:20
fultonjif i add other ceph roles that aren't tested they could get added to molecule.yaml19:20
*** slaweq has joined #tripleo19:21
openstackgerritMerged openstack/tripleo-heat-templates stable/stein: Per-Role krb-service-principal for CompactServices  https://review.opendev.org/66879719:25
*** ansmith has joined #tripleo19:25
fultonjbut maybe it will ensure that any new job *ceph* job is tested with the existing ones19:25
*** tosky has quit IRC19:25
cloudnull++ with all the deps being captured in the molecule.yml file maybe its best to keep extra roles out of the zuul config.19:27
fultonjcloudnull: ok, i guess i'll leave as is19:28
cloudnullsounds good.19:28
cloudnullwe can always adjust later if needed, but having now thought about it a bit more, for the ceph case and probably a lot of our more complex roles, it is likely better to capture those deps in the molecule config19:29
fultonjack19:30
fultonjhttp://logs.openstack.org/94/668294/9/check/tripleo-ansible-centos-7-molecule-tripleo-ceph-run-ansible/80a5fab/tox/ansible-execution.log19:30
fultonjseems to fail in CI but work on my workstation19:30
*** ramishra has quit IRC19:31
cloudnullim not 100% sure, but when I see that error https://pasted.tech/pastes/4781fc618546c240dc05c706d887cab4ef1c9082.raw - the issue is selinux related.19:33
cloudnullmaybe the role needs to install libselinux-python (python3-selinux) into the target containers?19:34
fultonjthanks i'll try that19:35
fultonjthe only reason i have this task https://review.opendev.org/#/c/668294/9/tripleo_ansible/roles/tripleo-ceph-common/tasks/main.yml@5019:36
fultonjis so molecule doesn't fail when we get to this task https://opendev.org/openstack/tripleo-ansible/src/branch/master/tripleo_ansible/roles/tripleo-ceph-work-dir/tasks/prepare.yml#L2919:38
openstackgerritRonelle Landy proposed openstack/tripleo-quickstart master: Added RedHat-8 release file  https://review.opendev.org/66905719:39
openstackgerritMerged openstack/tripleo-heat-templates stable/rocky: Per-Role krb-service-principal for CompactServices  https://review.opendev.org/66879819:44
openstackgerritRafael Folco proposed openstack/tripleo-ci master: DNM: Apply kolla patch to support rhel8  https://review.opendev.org/66948719:54
openstackgerritEmilien Macchi proposed openstack/tripleo-heat-templates master: Introduce param_config interface  https://review.opendev.org/66109319:54
*** cfontain has quit IRC19:57
*** cfontain has joined #tripleo19:58
openstackgerritJames Slagle proposed openstack/tripleo-common master: Add fqdn and cloud_domain templates to tripleo-hieradata  https://review.opendev.org/66890919:59
openstackgerritEmilien Macchi proposed openstack/tripleo-heat-templates master: Glance: wire param_config  https://review.opendev.org/66079119:59
*** cylopez has joined #tripleo19:59
*** saneax has quit IRC20:03
*** cfontain has quit IRC20:03
*** raildo has quit IRC20:13
*** cylopez has left #tripleo20:23
*** dciabrin has quit IRC20:27
*** sabedevops has joined #tripleo20:36
*** sabedevops has quit IRC20:37
openstackgerritRonelle Landy proposed openstack/tripleo-quickstart master: Added RedHat-8 release file  https://review.opendev.org/66905720:38
openstackgerritJohn Fulton proposed openstack/tripleo-ansible master: Add molecule test structure to ceph roles  https://review.opendev.org/66829420:49
*** dtrainor has quit IRC21:04
*** dtrainor has joined #tripleo21:09
*** jjoyce has quit IRC21:13
openstackgerritFrancesco Pantano proposed openstack/tripleo-heat-templates master: Add dashboard component as new ceph composable service.  https://review.opendev.org/66657821:43
*** ekultails has quit IRC21:48
*** jjoyce has joined #tripleo21:49
*** dciabrin has joined #tripleo21:54
*** jtomasek has quit IRC21:54
*** cfontain has joined #tripleo22:00
*** cfontain has joined #tripleo22:05
*** cfontain has quit IRC22:09
*** cfontain has joined #tripleo22:10
*** holser_ has joined #tripleo22:20
*** jistr has quit IRC22:24
*** jistr has joined #tripleo22:24
*** cfontain has joined #tripleo22:34
*** cfontain_ has joined #tripleo22:37
*** cfontain_ has quit IRC22:37
*** cfontain has quit IRC22:38
*** cfontain_ has joined #tripleo22:44
*** cfontain_ has quit IRC22:48
*** pkopec is now known as pkopec|afk22:50
*** rlandy has quit IRC23:07
*** cfontain_ has joined #tripleo23:09
*** zaneb has quit IRC23:09
*** zaneb has joined #tripleo23:10
*** cfontain_ has quit IRC23:14
*** zaneb has quit IRC23:15
*** cfontain_ has joined #tripleo23:16
*** zaneb has joined #tripleo23:16
*** zaneb has quit IRC23:17
*** zaneb has joined #tripleo23:18
*** zaneb has quit IRC23:19
*** cfontain_ has quit IRC23:20
*** zaneb has joined #tripleo23:20
*** zaneb has quit IRC23:22
*** zaneb has joined #tripleo23:23
*** cfontain_ has joined #tripleo23:23
*** cfontain has joined #tripleo23:26
*** vinaykns has quit IRC23:27
*** cfontain_ has quit IRC23:28
*** cfontain has quit IRC23:30
*** holser_ has quit IRC23:33
*** cfontain_ has joined #tripleo23:40
*** cfontain_ has joined #tripleo23:41
*** cfontain_ has joined #tripleo23:42
*** cfontain_ has joined #tripleo23:44
*** cfontain_ has quit IRC23:49

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