Wednesday, 2020-04-08

openstackgerritMerged openstack/tripleo-ansible master: container_puppet_config: don't show a warning if config_hash is empty  https://review.opendev.org/71816400:04
openstackgerritFlavio Fernandes proposed openstack/tripleo-heat-templates master: Add Octavia OVN Provider configuration  https://review.opendev.org/70769500:08
openstackgerritFlavio Fernandes proposed openstack/tripleo-heat-templates stable/train: Add Octavia OVN Provider configuration  https://review.opendev.org/71491600:13
openstackgerritEmilien Macchi proposed openstack/tripleo-ansible stable/train: Relax get_failed_containers() filter  https://review.opendev.org/71828100:21
*** dsneddon has quit IRC00:21
openstackgerritEmilien Macchi proposed openstack/tripleo-ansible stable/train: tripleo_container_rm: fix check mode  https://review.opendev.org/71828200:22
openstackgerritEmilien Macchi proposed openstack/tripleo-ansible stable/train: tripleo_container_manage: add check mode testing  https://review.opendev.org/71828300:22
openstackgerritEmilien Macchi proposed openstack/tripleo-ansible stable/train: container_puppet_config: don't show a warning if config_hash is empty  https://review.opendev.org/71828500:23
openstackgerritEmilien Macchi proposed openstack/tripleo-heat-templates stable/train: Include {{step}} when setting facts  https://review.opendev.org/71752600:24
*** morazi has quit IRC00:37
*** jmasud has quit IRC00:55
*** jmasud has joined #tripleo00:56
weshay_tkajinam, kind of00:59
weshay_EmilienM, why is container-prep trying to write to the remote registry?00:59
weshay_https://logserver.rdoproject.org/openstack-periodic-master/opendev.org/openstack/tripleo-ci/master/periodic-tripleo-ci-centos-8-standalone-full-tempest-api-master/64747ba/logs/undercloud/home/zuul/standalone_deploy.log.txt.gz00:59
tkajinamMay I ask your attention about https://bugs.launchpad.net/tripleo/+bug/1870448 ?01:00
openstackLaunchpad bug 1870448 in tripleo "no voting unittest jobs for puppet-tripleo master" [Undecided,In progress] - Assigned to Takashi Kajinami (kajinamit)01:00
tkajinamweshay_, ^^^01:00
tkajinamnow puppet-tripleo doesn't have any voting unit tests, and actually unit tests are broken now, because of my change :-P01:02
weshay_puppet-openstack-unit-5.5-centos-8success (non-voting) see it worked01:02
tkajinamweshay_, but it's non-voting01:02
weshay_aye.. but your change fixes that bit01:03
tkajinamweshay_, yes. I already submit a fix for current failure, and IMO we need to make it voting.01:04
weshay_aye.. I +2'd01:05
tkajinamweshay_, May I ask your review about https://review.opendev.org/#/c/718277/ ?01:05
tkajinamwe need this to fix current failure...01:05
tkajinamso we need this get merged before we make the job voting01:06
weshay_aye.. see that.. k.. both look good01:07
tkajinamweshay_, thx. I hope that the fix enables vote for unit tests, but as I'd like to ask some help if it doesn't work as expected.01:09
tkajinambecause zuul job definitions is still a bit too tricky for me... :-P01:10
tkajinamweshay_, anyway thanks. sorry for ping you in your late at night (I guess)01:11
weshay_tkajinam, no worries mate :)01:11
weshay_tkajinam, it looked right man.. you got it!01:11
tkajinamweshay_, :-)01:12
openstackgerritwes hayutin proposed openstack/tripleo-ci master: make scenario001 non-voting until packages are sane  https://review.opendev.org/71829001:13
*** ysandeep|away is now known as ysandeep|rover01:17
weshay_ysandeep|rover, EmilienM 0/01:33
EmilienMwhat's up01:33
weshay_ah.. this is local write fail01:33
weshay_    ERROR:root:Image prepare failed: [tripleomaster/centos-binary-neutron-server] Write Failure: HTTPSConnectionPool(host='trunk.registry.rdoproject.org', port=443): Read timed out.01:33
EmilienMmhh01:33
weshay_https://logserver.rdoproject.org/openstack-periodic-master/opendev.org/openstack/tripleo-ci/master/periodic-tripleo-ci-centos-8-standalone-full-tempest-api-master/64747ba/logs/undercloud/home/zuul/standalone_deploy.log.txt.gz01:33
weshay_all the jobs are failing on it01:33
EmilienMwth01:33
weshay_    socket.timeout: The read operation timed out01:34
weshay_maybe the registry went down :(01:34
weshay_sorry I read that wrong01:34
EmilienMmaster?01:34
weshay_ya :(01:34
weshay_trying to get the packages straight.. this is not helping01:34
* weshay_ checks registry01:35
EmilienMis the rdo registry up?01:35
weshay_https://registry.rdoproject.org:8443/oapi/v1/namespaces/tripleomaster/imagestreamtags/01:35
weshay_ya01:35
weshay_seems to be01:35
EmilienMk01:35
weshay_EmilienM, sorry to disturb01:35
EmilienMnp01:36
EmilienMI'm out now01:36
openstackgerritLance Bragstad proposed openstack/python-tripleoclient stable/train: Add logic to clean up ipa on stack delete  https://review.opendev.org/71678401:36
openstackgerritMerged openstack/tripleo-ansible master: tripleo_container_manage: add check mode testing  https://review.opendev.org/71812901:39
*** artom has quit IRC02:03
openstackgerritTakashi Kajinami proposed openstack/instack-undercloud stable/rocky: Add missing README.rst in elements  https://review.opendev.org/71829502:04
*** ysandeep|rover is now known as ysandeep|brb02:10
*** apetrich has quit IRC02:11
*** avivgt has quit IRC02:25
*** rlandy has quit IRC02:37
*** Goneri has quit IRC02:40
openstackgerritRonelle Landy proposed openstack/tripleo-quickstart master: WIP: Add tripleo-ansible repo for IPA roles  https://review.opendev.org/70741902:51
*** rh-jelabarre has quit IRC03:00
openstackgerritMerged openstack/tripleo-heat-templates stable/train: Switch to docker pull  https://review.opendev.org/71816703:09
*** chem has quit IRC03:14
*** chem has joined #tripleo03:19
*** psachin has joined #tripleo03:29
*** jdandrea has quit IRC03:34
*** ysandeep|brb is now known as ysandeep|rover03:42
*** cdearborn has quit IRC03:59
*** skramaja has joined #tripleo04:03
*** ykarel|away is now known as ykarel04:24
*** evrardjp has quit IRC04:36
*** evrardjp has joined #tripleo04:37
*** zaneb has quit IRC04:48
*** zaneb has joined #tripleo04:49
openstackgerritRabi Mishra proposed openstack/tripleo-common master: Revert "Revert removal of OrchestrationDeployAction"  https://review.opendev.org/71830604:53
*** jmasud has quit IRC04:53
openstackgerritRabi Mishra proposed openstack/tripleo-common master: Revert "Revert removal of OrchestrationDeployAction"  https://review.opendev.org/71830604:54
*** jmasud has joined #tripleo04:56
openstackgerritAde Lee proposed openstack/tripleo-heat-templates stable/train: Create DNS entries in IPA for openstack services  https://review.opendev.org/71819104:59
openstackgerritRabi Mishra proposed openstack/python-tripleoclient master: Add releasenote for removal of overcloud remote execute command  https://review.opendev.org/71830805:07
*** ratailor has joined #tripleo05:17
*** ed_b has quit IRC05:19
*** udesale has joined #tripleo05:22
*** dwilde has quit IRC05:22
*** udesale has quit IRC05:23
*** udesale has joined #tripleo05:23
*** lmiccini has joined #tripleo05:29
*** abdysn has joined #tripleo05:38
*** marios has joined #tripleo05:53
*** Abdallahyas has joined #tripleo06:03
*** abdysn has quit IRC06:07
*** jfrancoa has joined #tripleo06:08
*** bandini has joined #tripleo06:12
openstackgerritJose Luis Franco proposed openstack/tripleo-heat-templates master: Change Collectd ports type to numbers.  https://review.opendev.org/71820006:20
openstackgerritMichele Baldessari proposed openstack/tripleo-heat-templates stable/train: WIP Move the haproxy iptables rules creation to host_prep_tasks  https://review.opendev.org/71820106:21
*** iurygregory has quit IRC06:25
*** iurygregory has joined #tripleo06:27
*** bogdando has joined #tripleo06:27
openstackgerritCédric Jeanneret (Tengu) proposed openstack/tripleo-operator-ansible master: Allow to set log-file option for container-prepare  https://review.opendev.org/71690606:27
openstackgerritBogdan Dobrelya (bogdando) proposed openstack/python-tripleoclient master: Allow force resetting passwords  https://review.opendev.org/71808206:49
*** jcapitao has joined #tripleo06:54
*** florianf has joined #tripleo06:56
*** slaweq_ has joined #tripleo06:57
*** maciejjozefczyk has joined #tripleo07:00
*** abdysn has joined #tripleo07:05
*** cylopez has joined #tripleo07:07
openstackgerritSagi Shnaidman proposed openstack/tripleo-quickstart master: Sync fs035 job with fs001 settings  https://review.opendev.org/70898807:08
*** Abdallahyas has quit IRC07:09
*** rpittau|afk is now known as rpittau07:17
*** amoralej|off is now known as amoralej07:20
openstackgerritRajesh Tailor proposed openstack/tripleo-heat-templates master: Add new parameter NovaSchedulerEnableIsolatedAggregateFiltering  https://review.opendev.org/71832907:20
*** ysandeep|rover is now known as ysandeep|lunch07:24
mariosneeds reviews please https://review.opendev.org/#/c/716639/1207:29
*** suuuper has joined #tripleo07:31
*** ccamacho has joined #tripleo07:39
*** dtantsur|afk is now known as dtantsur07:41
*** cylopez has quit IRC07:41
*** paramite has joined #tripleo07:42
*** cylopez has joined #tripleo07:42
sshnaidm|afkmarios, commented07:42
*** sshnaidm|afk is now known as sshnaidm07:42
openstackgerritFrancesco Pantano proposed openstack/tripleo-docs master: Customizing crushmap using device classes  https://review.opendev.org/71810907:46
openstackgerritamolkahat proposed openstack/tripleo-quickstart master: Ported featureset060.yml to os_tempest.  https://review.opendev.org/71809107:48
mariossshnaidm: o/ please check again i still don't follow the objection commnted https://review.opendev.org/#/c/716639/12/roles/tripleo_overcloud_roles_generate/molecule/default/converge.yml@2207:48
mariossshnaidm: and thanks for the review time07:49
*** bandini has quit IRC07:54
sshnaidmmarios, I've tried to explain, sorry if it's not clear07:54
mariossshnaidm: its clear what you want. i don't understand the benefit. you gav me an example about 'C o n t r '  but that isn't true as i showed there07:55
sshnaidmmarios, it's really the fundamental testing base, you don't use hidden values in your expected results07:55
mariossshnaidm: they aren't hidden they are passed as vars in the very same block/test case07:56
mariossshnaidm: very explicitly passed nothing hidden there07:56
sshnaidmmarios, no, you process them before07:56
*** bandini has joined #tripleo07:56
sshnaidmmarios, think about it as a black box testing07:57
mariossshnaidm: i am calling the role with some vars. and then testing with the same vars.07:57
mariossshnaidm: not copying them over/ the very same ones07:57
openstackgerritCédric Jeanneret (Tengu) proposed openstack/tripleo-operator-ansible master: Add a standalone job for container_image_prepare  https://review.opendev.org/71833407:58
openstackgerritCédric Jeanneret (Tengu) proposed openstack/tripleo-operator-ansible master: Add a standalone job for container_image_prepare  https://review.opendev.org/71833408:00
*** tosky has joined #tripleo08:00
*** dmellado has quit IRC08:01
*** aedc_ has joined #tripleo08:02
openstackgerritamolkahat proposed openstack/tripleo-ci master: Port tripleo-ci-centos-7-scenario012-standalone job to os_tempest  https://review.opendev.org/71799508:04
*** dmellado has joined #tripleo08:05
*** aedc has quit IRC08:05
*** jpich has joined #tripleo08:05
*** lucasagomes has joined #tripleo08:08
*** dpawlik has joined #tripleo08:08
ykarelsshnaidm, hey08:10
ykarelsshnaidm, u looked at deletion of tripleo-ansible stein branch?08:10
ykareli still see jobs failing08:10
*** aedc_ has quit IRC08:11
openstackgerritBogdan Dobrelya (bogdando) proposed openstack/tripleo-common stable/train: Fix misused timeout values in the deployment workflow  https://review.opendev.org/71833908:13
openstackgerritMerged openstack/tripleo-ci master: make scenario001 non-voting until packages are sane  https://review.opendev.org/71829008:13
sshnaidmykarel, hey, tbh I don't know how I delete stein branch, maybe need weshay_ for that08:14
ykarelsshnaidm, ohhk i think need infra help on it08:15
openstackgerritChandan Kumar (raukadah) proposed openstack/tripleo-ci master: Revert "make scenario001 non-voting until packages are sane"  https://review.opendev.org/71834008:15
ykarelsshnaidm, and also cleanup https://github.com/openstack/releases/blob/master/deliverables/stein/tripleo-ansible.yaml#L1608:16
*** pkopec has joined #tripleo08:16
openstackgerritBogdan Dobrelya (bogdando) proposed openstack/tripleo-common stable/train: [Train only] Fix misused timeout values in the deployment workflow  https://review.opendev.org/71833908:16
*** aedc has joined #tripleo08:17
openstackgerritMerged openstack/tripleo-operator-ansible master: Allow to set log-file option for container-prepare  https://review.opendev.org/71690608:18
openstackgerritFrancesco Pantano proposed openstack/tripleo-docs master: Customizing crushmap using device classes  https://review.opendev.org/71810908:19
*** rcernin has quit IRC08:27
*** ysandeep|lunch is now known as ysandeep|rover08:29
*** dmacpher has quit IRC08:32
*** derekh has joined #tripleo08:33
*** dmacpher has joined #tripleo08:33
openstackgerritGrzegorz Grasza proposed openstack/tripleo-heat-templates stable/train: Create DNS entries in IPA for openstack services  https://review.opendev.org/71819108:34
marioscopy/paste from oooq 11:32 < marios> folks me and sshnaidm are having a discussion there https://review.opendev.org/#/c/716639/12/roles/tripleo_overcloud_roles_generate/molecule/default/converge.yml  I need reviews there please if you have time and have an opinion about that08:34
*** slaweq_ is now known as slaweq08:36
sshnaidmysandeep|rover, ykarel, do you know if we have any patch for "find_client" issue, that I can depend on it?08:39
openstackgerritamolkahat proposed openstack/tripleo-ci master: Removed centos7-scenario012 job.  https://review.opendev.org/71834608:44
openstackgerritGrzegorz Grasza proposed openstack/tripleo-heat-templates stable/train: Add new composable service for IpaClient  https://review.opendev.org/71812008:44
openstackgerritGrzegorz Grasza proposed openstack/tripleo-heat-templates stable/train: Create DNS entries in IPA for openstack services  https://review.opendev.org/71819108:44
ykarelsshnaidm, needs master promotion08:45
sshnaidmsigh08:45
ysandeep|roversshnaidm, if you are reffering to image upload issue failing with 'Client' object has no attribute 'find_image'?  I don't think there is any patch.. afaik.. we are trying to get promotion with a good hash to fix things.08:45
sshnaidmysandeep|rover, ack08:46
*** tkajinam has quit IRC08:51
*** ykarel is now known as ykarel|lunch08:53
openstackgerritBogdan Dobrelya (bogdando) proposed openstack/python-tripleoclient stable/train: Fix misused deployment vs connection timeouts  https://review.opendev.org/71834708:53
*** jmasud has quit IRC08:54
*** jmasud has joined #tripleo08:56
*** jpich has quit IRC08:59
*** jpich has joined #tripleo09:00
*** alexmcleod has joined #tripleo09:03
openstackgerritGrzegorz Grasza proposed openstack/python-tripleoclient stable/train: Add logic to clean up ipa on stack delete  https://review.opendev.org/71678409:04
openstackgerritamolkahat proposed openstack/tripleo-ci master: Removed centos7-scenario012 job. Added job tripleo-ci-centos-7-scenario012-standalone in depricated-jobs.yaml file.  https://review.opendev.org/71834609:10
openstackgerritMichele Baldessari proposed openstack/tripleo-heat-templates master: WIP Move the haproxy iptables rules creation to host_prep_tasks  https://review.opendev.org/71815909:15
openstackgerritSagi Shnaidman proposed openstack/python-tripleoclient master: Pass params to tripleo_plan_parameters_update as dictionary  https://review.opendev.org/71786509:17
openstackgerritMichele Baldessari proposed openstack/tripleo-heat-templates stable/train: WIP Move the haproxy iptables rules creation to host_prep_tasks  https://review.opendev.org/71820109:17
*** dmacpher has quit IRC09:20
openstackgerritFrançois Charlier proposed openstack/tripleo-quickstart-extras master: OOO Validations command changed in OpenStack Train  https://review.opendev.org/71835309:21
*** gfidente has joined #tripleo09:22
*** dmacpher has joined #tripleo09:23
*** elGamal has quit IRC09:51
*** elfGamal has joined #tripleo09:51
*** jbadiapa has joined #tripleo09:59
*** holser has joined #tripleo10:12
*** ykarel|lunch is now known as ykarel10:12
openstackgerritSagi Shnaidman proposed openstack/tripleo-operator-ansible master: Add tripleo_overcloud_profiles_list role  https://review.opendev.org/71837410:13
openstackgerritMerged openstack/tripleo-quickstart master: Fix default value of tempestconf overrides  https://review.opendev.org/71606010:14
openstackgerritFlavio Fernandes proposed openstack/tripleo-heat-templates master: Re-enable driver agent for scenario 10  https://review.opendev.org/71463910:16
openstackgerritBogdan Dobrelya (bogdando) proposed openstack/python-tripleoclient stable/train: Fix misused deployment vs connection timeouts  https://review.opendev.org/71834710:23
*** slaweq has quit IRC10:23
*** rpittau is now known as rpittau|bbl10:24
*** slaweq has joined #tripleo10:26
*** ysandeep|rover is now known as ysandeep|afk10:30
openstackgerritSagi Shnaidman proposed openstack/tripleo-operator-ansible master: Add tripleo_overcloud_profiles_list role  https://review.opendev.org/71837410:30
*** chem has quit IRC10:31
*** chem has joined #tripleo10:33
*** mrebisz has joined #tripleo10:39
mrebiszHi so maybe someone can help trying to run tripleo standalone on centos8 when running yum install python3-tripleoclient I do get the following error http://paste.openstack.org/show/791796/10:49
mrebiszcan anyone clarify what is wrong here10:50
*** florianf has quit IRC10:50
openstackgerritRajesh Tailor proposed openstack/tripleo-heat-templates master: Add new parameter NovaSchedulerEnableIsolatedAggregateFiltering  https://review.opendev.org/71832910:50
*** Abdallahyas has joined #tripleo10:53
*** florianf has joined #tripleo10:54
openstackgerritSagi Shnaidman proposed openstack/tripleo-operator-ansible master: Add tripleo_overcloud_profiles_match role  https://review.opendev.org/71838810:54
*** jtomasek has joined #tripleo10:55
*** sshnaidm is now known as sshnaidm|afk10:56
*** abdysn has quit IRC10:57
*** jcapitao is now known as jcapitao_lunch11:07
*** morazi has joined #tripleo11:09
*** abdysn has joined #tripleo11:13
chandankumarbandini, marios https://review.opendev.org/#/c/718200/2 to clear scenario111:15
*** Abdallahyas has quit IRC11:16
*** apetrich has joined #tripleo11:16
*** ratailor has quit IRC11:16
*** ed_b has joined #tripleo11:17
openstackgerritMerged openstack/tripleo-common stable/train: [stable/train only] Use nodes as hostnames if ironic lookup fails  https://review.opendev.org/71779811:20
*** ysandeep|afk is now known as ysandeep|rover11:31
*** mrebisz has quit IRC11:35
*** rh-jelabarre has joined #tripleo11:45
openstackgerritOliver Walsh proposed openstack/tripleo-common stable/train: Preserve existing placement password during extraction  https://review.opendev.org/71840511:49
*** raildo has joined #tripleo11:53
openstackgerritJose Luis Franco proposed openstack/tripleo-heat-templates stable/train: DNM: Patch for testing /var/lib/mistral issue.  https://review.opendev.org/71840611:55
*** artom has joined #tripleo12:08
*** jcapitao_lunch is now known as jcapitao12:14
*** tkajinam has joined #tripleo12:14
*** Goneri has joined #tripleo12:16
*** rlandy has joined #tripleo12:19
*** sshnaidm|afk is now known as sshnaidm12:19
sshnaidmcloudnull, seems like patch didn't work https://review.opendev.org/#/c/717865/ in https://logserver.rdoproject.org/14/25814/12/check/tripleo-ci-centos-8-ovb-3ctlr_1comp-featureset001/a3da3c3/logs/undercloud/home/zuul/overcloud_deploy.log.txt.gz12:19
sshnaidmcloudnull, I wonder if "isintstance.." didn't apply there12:19
sshnaidmcloudnull, I'll try w/o it12:20
*** amoralej is now known as amoralej|lunch12:20
openstackgerritamolkahat proposed openstack/tripleo-quickstart master: Ported featureset060.yml to os_tempest.  https://review.opendev.org/71809112:21
weshay_sshnaidm++12:21
openstackgerritSagi Shnaidman proposed openstack/python-tripleoclient master: Pass params to tripleo_plan_parameters_update as dictionary  https://review.opendev.org/71786512:22
*** mrebisz has joined #tripleo12:22
*** rpittau|bbl is now known as rpittau12:23
cloudnullmornings12:23
mrebiszHi did anyone see the fallowing error. Trying to deploy tripleo and getthing this in my face no idea how to fix it. http://paste.openstack.org/show/791800/12:24
sshnaidmramishra, cloudnull I wonder if this should be actually dictionary https://github.com/openstack/tripleo-ansible/blob/master/tripleo_ansible/ansible_plugins/modules/tripleo_plan_parameters_update.py#L6512:25
cloudnullsshnaidm https://review.opendev.org/#/c/717865/8/tripleoclient/v1/overcloud_deploy.py@690 does that need to be updated?12:26
sshnaidmcloudnull, no, seems like messed up12:27
sshnaidmtried gerrit interface for changing a patch :/12:27
cloudnullmeh it happens :D12:28
openstackgerritSagi Shnaidman proposed openstack/python-tripleoclient master: Pass params to tripleo_plan_parameters_update as dictionary  https://review.opendev.org/71786512:29
ade_leemwhahaha, EmilienM cloudnull hjensas https://review.opendev.org/#/c/718120 and https://review.opendev.org/#/c/718191 please12:29
sshnaidmcloudnull, so, if it's the final destination for "params": https://github.com/openstack/tripleo-common/blob/master/tripleo_common/utils/plan.py#L4912:30
sshnaidmcloudnull, then it should be string actually12:30
openstackgerritDamien Ciabrini proposed openstack/tripleo-heat-templates master: Support for mariadb's ed25519 authentication  https://review.opendev.org/71347012:30
sshnaidmmaybe ramishra knows12:30
*** rlandy has quit IRC12:31
*** rlandy has joined #tripleo12:31
ysandeep|rovermrebisz, 'ERROR! the playbook: /usr/share/ceph-ansible/site-container.yml.sample could not be found' --> seems like missing ceph-ansible package on your undercloud.. which docs are you following?12:32
openstackgerritEmilien Macchi proposed openstack/python-tripleoclient stable/train: Ensure the undercloud output dir constant is consistent  https://review.opendev.org/71841512:33
openstackgerritSagi Shnaidman proposed openstack/tripleo-ansible master: Change params type to be string  https://review.opendev.org/71841612:33
sshnaidmcloudnull, ^^12:33
openstackgerritGrzegorz Grasza proposed openstack/tripleo-heat-templates master: Skip both tenant and management networks when generating certs  https://review.opendev.org/69684212:33
cloudnullsshnaidm that module is used and works elsewhere.12:35
cloudnullI'm almost certain that the data type needs be a dictionary12:38
sshnaidmcloudnull, I see12:38
sshnaidmok, let's continue with the first approach..12:38
*** xek_ has joined #tripleo12:40
openstackgerritSagi Shnaidman proposed openstack/tripleo-operator-ansible master: Add tripleo_overcloud_profiles_match role  https://review.opendev.org/71838812:41
openstackgerritCédric Jeanneret (Tengu) proposed openstack/tripleo-operator-ansible master: Allows to get the standalone deploy reproducer script  https://review.opendev.org/71842112:42
openstackgerritRonelle Landy proposed openstack/tripleo-quickstart master: WIP: Add tripleo-ansible repo for IPA roles  https://review.opendev.org/70741912:50
openstackgerritTakashi Kajinami proposed openstack/tripleo-heat-templates master: Remove useless nova::ironic::common::api_endpoint  https://review.opendev.org/71842912:50
*** jmasud has quit IRC12:54
*** jmasud has joined #tripleo12:56
mrebiszysandeep|rover im going with https://docs.openstack.org/project-deploy-guide/tripleo-docs/latest/deployment/standalone.html12:56
*** chem has quit IRC12:56
*** ykarel is now known as ykarel|afk12:56
cloudnullsshnaidm if you have some extra time, I'd really appreciate it if you could take a look at https://review.opendev.org/#/c/71818412:56
cloudnull-cc ramishra12:56
ysandeep|rovermrebisz, do you need ceph in your standalone ?12:57
openstackgerritRonelle Landy proposed openstack/tripleo-quickstart-extras master: DNM: Testing ipa multinode  https://review.opendev.org/70637312:58
ysandeep|rovermrebisz, if yes, then install ceph-ansible and rerun deployment12:58
ysandeep|rovermrebisz, "sudo -E tripleo-repos current-tripleo-dev ceph", will setup necessary repositories to install ceph-ansible12:58
*** chem has joined #tripleo12:58
mrebiszthanks im quite sure that was done but it wont hurt13:00
ysandeep|rovermrebisz, we can  check "rpm -qa | grep -i ceph-ansible" -- see if its there..13:01
*** saneax has quit IRC13:01
openstackgerritwes hayutin proposed openstack/tripleo-ci master: ensure we can override the registry namespace  https://review.opendev.org/71843313:03
openstackgerritKevin Carter (cloudnull) proposed openstack/tripleo-ansible master: Improve deployer UX by limiting failure output potential  https://review.opendev.org/71498513:03
openstackgerritTakashi Kajinami proposed openstack/tripleo-heat-templates master: Add missing region_name for ironic access from nova  https://review.opendev.org/71843513:05
openstackgerritRonelle Landy proposed openstack/tripleo-ci master: DNM: Testing standalone IPA running on multinode  https://review.opendev.org/70628813:05
openstackgerritKevin Carter (cloudnull) proposed openstack/tripleo-common master: Remove invalid mistral envocation and references  https://review.opendev.org/71156213:05
cloudnullsshnaidm have you seen https://logserver.rdoproject.org/50/717750/1/openstack-check/tripleo-ci-centos-8-ovb-3ctlr_1comp-featureset001-vexxhost/858ba93/logs/undercloud/home/zuul/overcloud_deploy.log.txt.gz13:08
cloudnullthat output from the debug stanza looks odd ?13:09
openstackgerritKevin Carter (cloudnull) proposed openstack/tripleo-ansible master: DNM - add debug for param  https://review.opendev.org/71775013:10
sshnaidmcloudnull, yeah, I think it's actually string: "rendered_parameters: |- "13:11
cloudnullsshnaidm maybe the output was truncated? while it mostly looks like a dictionary, it starts with `s': ['1.1.1.1', '1.0.0.1'],`13:11
sshnaidmcloudnull, right, confused me as well13:11
cloudnullwhich would make sense why it sees it as a string13:11
cloudnullwhere are those params defined ?13:12
*** cdearborn has joined #tripleo13:13
sshnaidmparams = env.pop('parameter_defaults', None) https://github.com/openstack/python-tripleoclient/blob/master/tripleoclient/v1/overcloud_deploy.py#L278:L27813:13
sshnaidmI believe there ^13:13
openstackgerritTakashi Kajinami proposed openstack/tripleo-heat-templates master: Add missing region_name for ironic access from nova  https://review.opendev.org/71843513:13
openstackgerritKevin Carter (cloudnull) proposed openstack/tripleo-ansible master: DNM - add debug for param  https://review.opendev.org/71775013:14
openstackgerritMerged openstack/tripleo-docs master: Document scale-down with hostnames  https://review.opendev.org/71813913:14
cloudnullsshnaidm I added two debug statements .13:15
cloudnullone before and after the set.13:15
cloudnullwe should be able to see exactly what is being passed into the playbook13:15
*** skramaja has quit IRC13:16
openstackgerritTakashi Kajinami proposed openstack/tripleo-heat-templates master: Remove useless nova::ironic::common::api_endpoint  https://review.opendev.org/71842913:17
*** udesale_ has joined #tripleo13:18
openstackgerritBogdan Dobrelya (bogdando) proposed openstack/tripleo-docs master: Document how to reset service passwords via CLI  https://review.opendev.org/71844313:19
*** udesale has quit IRC13:20
mrebiszysandeep|rover ok had some sucess now a bit of a diffrent issue: keystoneauth1.exceptions.discovery.DiscoveryFailure: Could not find versioned identity endpoints when attempting to authenticate. Please check that your auth_url is correct. Service Unavailable (HTTP 50313:21
ramishrasshnaidm, cloudnull I'm thinking it's probably something to do with json vs yaml in extravars for 'params'.. It seems to be working when it's yaml and is converted to dict with templating and not for json13:21
ramishranot sure if it's truncated as rendered_parameters looks ok in https://logserver.rdoproject.org/50/717750/1/openstack-check/tripleo-ci-centos-8-ovb-3ctlr_1comp-featureset001-vexxhost/858ba93/logs/undercloud/home/zuul/overcloud_deploy.log.txt.gz13:22
openstackgerritLukas Bezdicka proposed openstack/tripleo-heat-templates stable/train: [Q->T] Introduce Queens to Train Upgrade  https://review.opendev.org/71348213:24
*** rlandy is now known as rlandy|pto13:24
ysandeep|rovermrebisz, sry its end of day for me.. i can give some pointers tomorrow..13:25
*** ysandeep|rover is now known as ysandeep|away13:25
ramishradebug is probably truncating it..13:25
mrebiszAnyone elese knows what the keystoneauth1.exceptions.discovery.DiscoveryFailure: Could not find versioned identity endpoints when attempting to authenticate. Please check that your auth_url is correct. Service Unavailable (HTTP 503 error could be all about? and how to fix it?13:28
*** jdandrea has joined #tripleo13:28
openstackgerritLukas Bezdicka proposed openstack/tripleo-heat-templates stable/train: [Q->T] Introduce Queens to Train Upgrade  https://review.opendev.org/71348213:30
mwhahahasounds like a service is down, check the status in haproxy. it's likely keystone isn't up13:30
lbragstadhey folks - quick question on how scale down works13:34
lbragstadi'm running this during a node delete https://opendev.org/openstack/tripleo-heat-templates/src/branch/master/deployment/nova/nova-compute-container-puppet.yaml#L111213:34
lbragstad and i'm getting http://paste.openstack.org/raw/791803/ - deletgate_to: localhost should run that task on the undercloud in this case, right?13:35
EmilienMfrom the overcloud, can you reach overcloud keystone?13:35
EmilienMsource overcloudrc; openstack endpoint list13:35
openstackgerritLukas Bezdicka proposed openstack/tripleo-upgrade stable/train: Switch install-docker with ensure-docker  https://review.opendev.org/71844913:35
lbragstadEmilienM from the undercloud?13:36
lbragstadEmilienM yeah - http://paste.openstack.org/show/791804/ i can reach the overcloud keystone service from the undercloud as the root, stack, and tripleo-admin users13:37
*** ykarel|afk is now known as ykarel13:37
EmilienMoh wait13:38
*** morazi has quit IRC13:38
openstackgerritDamien Ciabrini proposed openstack/tripleo-common master: mariadb: add pynacl dependency to support ed25519  https://review.opendev.org/71845313:38
openstackgerritDavid Hill proposed openstack/tripleo-heat-templates stable/rocky: Add ApacheMaxMaxClients property to set apache::mod::prefork::maxclients  https://review.opendev.org/71845413:44
*** bandini has quit IRC13:46
*** morazi has joined #tripleo13:46
*** bandini has joined #tripleo13:46
openstackgerritDavid Hill proposed openstack/tripleo-heat-templates stable/rocky: Add ApacheMaxMaxClients property to set apache::mod::prefork::maxclients  https://review.opendev.org/71845413:47
abdysnmrebisz: -^13:47
*** amoralej|lunch is now known as amoralej13:47
openstackgerritDavid Hill proposed openstack/tripleo-heat-templates stable/rocky: Add ApacheMaxMaxClients property to set maxclients  https://review.opendev.org/71845413:49
mrebiszmwhahaha So silly question how can I check that?13:50
*** dtantsur is now known as dtantsur|brb13:50
mwhahahacheck the status of containers13:50
mwhahahaon the controller13:50
mrebiszHmm ok noob question since im new to this. But I am going with https://docs.openstack.org/project-deploy-guide/tripleo-docs/latest/deployment/standalone.html and thats happening on tripleo deploy so Im sure how to do what your suggesting13:52
mwhahahahaproxy status can be viewed on the contorllers with: watch 'echo "show stat" | nc -U /var/lib/haproxy/stats | cut -d "," -f 1,2,5-11,18,24,27,30,36,50,37,56,57,62 | column -s, -t'13:52
mwhahahasudo podman ps -a13:52
*** dwilde has joined #tripleo13:54
openstackgerritBogdan Dobrelya (bogdando) proposed openstack/paunch stable/train: [Train only] Process labels before building container run args  https://review.opendev.org/71845913:57
openstackgerritBogdan Dobrelya (bogdando) proposed openstack/paunch stable/stein: Process labels before building container run args  https://review.opendev.org/71846013:57
weshay_ade_lee, you able to join the tls sync on the 13th?13:58
openstackgerritBogdan Dobrelya (bogdando) proposed openstack/paunch stable/queens: Process labels before building container run args  https://review.opendev.org/71846313:59
openstackgerritLuke Short proposed openstack/tripleo-common stable/train: [Train-Only] Expose --limit, --skip-tags, and --tags in config-download.  https://review.opendev.org/71668414:03
ade_leeweshay_, what tls sync?14:04
ade_leeweshay_, I see it now and yes14:05
weshay_ade_lee, context is ... rlandy and pooja are back on it.. have been working it for the last two weeks or so and ready to sync up w/ the work you guys have done on roles and new projects14:07
weshay_cloudnull, so we're talking about dropping the stein branch on tripleo-ansible.. assume you agree?14:07
ade_leeweshay_, great14:08
mwhahahawhat's the history on that request?14:08
openstackgerritMerged openstack/os-net-config master: Run ifup on a bond when a slave interface is restarted  https://review.opendev.org/71743214:08
weshay_sshnaidm, is discussing it #openstack-release14:08
* mwhahaha is uncertain why we need to nuke it14:08
weshay_sshnaidm, perhaps we make it a topic in #tripleo mtg prior to bringing it to the release team14:08
weshay_so more folks have context?14:09
* weshay_ checks email list14:09
weshay_#too many things14:09
cloudnullweshay_ +114:09
cloudnulltripleo-ansible was essentially a shell repo in Stein14:10
sshnaidmmwhahaha, https://review.opendev.org/#/c/717774/14:10
weshay_https://review.opendev.org/#/c/718468/14:10
mwhahahacan't you override the branch14:11
sshnaidmmwhahaha, we tried.. and then built in every job master branch of tripleo-ansible instead of branch of the patch14:12
mwhahahaperhaps we shouldn't be using tripleo-ansible as we are14:12
sshnaidmiirc it for for reusing existing roles14:13
sshnaidmbut yeah, it may create conflict of interest..14:13
openstackgerritGiulio Fidente proposed openstack/tripleo-ansible master: Add admin user to mistral group  https://review.opendev.org/71846914:13
openstackgerritMarios Andreou proposed openstack/tripleo-operator-ansible master: Adds overcloud roles generate role - creates roles_data.yaml  https://review.opendev.org/71663914:15
mariosparamite: i updated for your comments at ^^^14:15
mariossorry paramite meant for panda ^^14:15
mariosmwhahaha: can you check the discussion when you have time please and revote ... me and sshnaidm are having a discussion there https://review.opendev.org/#/c/716639/12/roles/tripleo_overcloud_roles_generate/molecule/default/converge.yml14:15
mariosperhaps wait for molecule to run first ;/14:16
mariosthanks14:16
mwhahahamarios: i'm not a fan of using the jinja in the expected values14:17
mwhahahafwiw14:17
mariosmwhahaha: yes that is what the discussion was about14:17
mariosmwhahaha: i mean, it is passed in a block so the same vars that go in the input are expected in the output14:17
marios i don't see the problem but... this is the same discussion i already had with sshnaidm on the review and here earlier14:18
openstackgerritEmilien Macchi proposed openstack/tripleo-ansible master: tripleo_all_nodes_data: sort enabled_services (idempotency)  https://review.opendev.org/71847114:18
mwhahahawe've seen similar patterns break down in the python bits14:18
mwhahahai prefer to be explicit14:18
mariostidied up the block in v13 easier to read perhaps https://review.opendev.org/#/c/716639/13/roles/tripleo_overcloud_roles_generate/molecule/default/converge.yml14:18
marios17:18 <@mwhahaha> we've seen similar patterns break down in the python bits14:18
mariossigh14:18
mariosok14:18
mariosmwhahaha: break down where in ci you mean ?14:19
mariosmwhahaha: still not clear what breaks14:19
mariosmwhahaha: i mean this is the molecul scenario we're talking about14:19
mwhahahathe point of those tests in that repo is to test the vars make to the output as expected14:20
mariosmwhahaha: yes exactly, so i use the *exact same* vars: for input and output check14:20
mwhahahai'm not seeing the benefit as defined14:20
mwhahahait's actually more code the way you have defined it14:21
mariosmwhahaha: how is it more code.14:21
mariosmwhahaha: the benefit is easier maintenance, cos you only have to update the vars: and not update the test14:21
mwhahahai don't agree it's easier maintance14:22
mwhahahahttps://github.com/openstack/tripleo-operator-ansible/blob/master/roles/tripleo_config_generate_ansible/molecule/default/converge.yml#L35-L45 is vastly less than https://review.opendev.org/#/c/716639/12/roles/tripleo_overcloud_roles_generate/molecule/default/converge.yml@6514:23
mariosmwhahaha: how so14:24
mariosmwhahaha: in https://github.com/openstack/tripleo-operator-ansible/blob/master/roles/tripleo_config_generate_ansible/molecule/default/converge.yml#L35-L4514:24
mariosmwhahaha: you have to go in and explicitly change lines 45 AND 39 if you want to update the test14:24
mariosmwhahaha: in my case you'd only have to update the var, on line 3914:24
mariosmwhahaha: so how is it *vastly* less?14:24
openstackgerritDavid Hill proposed openstack/tripleo-heat-templates master: Add ApacheMaxMaxClients property to set maxclients  https://review.opendev.org/71847214:25
mwhahahawhy  "{{ ansible_env.HOME }}"14:26
mwhahahawhen you can jsut define a random value14:26
mwhahahai'm not sold on your maintenace claims14:26
mariosmwhahaha: hold on though that's changing the subject. so do you still object to the parameterization?14:27
mwhahahayes14:27
mariosmwhahaha: i mean honestly i'm in the process of changing it cos if 2 of you are saying no14:27
mwhahahajust define it statically please14:27
mariosmwhahaha: then fine, but i really don't believe it14:27
mariosmwhahaha: i don't get the benefit14:27
mariosmwhahaha: all i see is cost14:27
mariosmwhahaha: it really doesn't make sense to me14:27
mariosmwhahaha: and apparently panda too14:27
openstackgerritFlavio Fernandes proposed openstack/tripleo-heat-templates master: Add Octavia OVN Provider configuration  https://review.opendev.org/70769514:27
mariosmwhahaha: you claimed it was vastly more maintenance to do it my way but i showed you it is in fact the opposite14:28
mariosmwhahaha: so what is the benefit other than 'easier for humans to read'14:28
mwhahahawhen you statically define it it is much easier to understand the expectations rather than having to unwind dynamic code in your head14:28
mariosmwhahaha: but even that is debatable.. its all there in the vars RE 17:28 < marios> mwhahaha: so what is the benefit other than 'easier for humans to read'14:28
mwhahahaplease don't do it, the precieved maintence code is not worth the complexity14:28
mariosmwhahaha: again, i don't see any added complexiy, in fact the opposite14:29
mariosmwhahaha: but doing ...14:29
mwhahahajust because you don't doesn't mean it's not there14:29
mwhahahawe already doo too much complex shit14:29
mwhahahaplease don't continue with that14:29
mariosmwhahaha: well, sure but its not enough to say 'added complexity' as an abstract without pointint to the thing that is adding it. what in my review adds complexity14:29
mwhahahadude, i get your point. i'm just sayign it's not worth it over all14:30
mariosmwhahaha: thanks for taking the time to discuss14:31
*** florianf has quit IRC14:32
ekultailsHey folks, does anyone know where the container-update.py file from tripleo-common ends up being installed to?14:36
EmilienMekultails: it's probably not afict, (check in setup.cfg)14:37
EmilienMyou would need to patch that file and add it, and it'll be shipped.14:38
ekultails*checking setup.cfg*14:39
ekultailsEmilienM: It's not mentioned in the setup.cfg either (but other scripts are). Does that mean it is not used? I thought it was ran during an Overcloud update.14:40
openstackgerritKevin Carter (cloudnull) proposed openstack/tripleo-ansible master: WIP Create derived parameters playbook, modules, and role  https://review.opendev.org/71818414:40
EmilienMekultails: idk how the script is used but it's not shipped14:42
EmilienMchem: ^ do you know?14:43
chemEmilienM: lemme check14:44
ykarelsshnaidm, https://review.opendev.org/#/c/717865/ didn't worked?14:44
sshnaidmykarel, nope, running some debug on it https://review.opendev.org/#/c/717750/14:45
ykarelsshnaidm, ohhk14:45
*** udesale_ has quit IRC14:46
ykarelsshnaidm, i see in some jobs those parameters are printed14:46
sshnaidmykarel, where?14:46
openstackgerritwes hayutin proposed openstack/tripleo-ci master: Revert "make scenario001 non-voting until packages are sane"  https://review.opendev.org/71847714:46
ykarelsshnaidm, iirc i saw in containers-multinode job upstream14:46
sshnaidmhmm.. looking14:46
openstackgerritDave Wilde (d34dh0r53) proposed openstack/tripleo-heat-templates stable/train: Add new composable service for IpaClient  https://review.opendev.org/71812014:47
chemEmilienM: ekultails it seems nowhere is the reply14:50
chemEmilienM: ekultails that file is nowhere to be seen on a live env14:51
lbragstadhjensas cloudnull https://review.opendev.org/#/c/716784/ should be good for another pass - thanks for all the help and reviews14:51
EmilienMchem: so why do we have it? can we kill it?14:52
ekultailschem: EmilienM: lol That is what I was seeing, too. I guess the file is not actually used. I will put up a Do Not Merge patch to remove the file and see what happens.14:52
*** Abdallahyas has joined #tripleo14:53
EmilienMekultails: oh I'm sure CI will be fine14:53
EmilienMthe question is who is using it14:53
*** abdysn has quit IRC14:54
chemEmilienM: I've never seen it being  used in update (at least recently)14:54
openstackgerritLuke Short proposed openstack/tripleo-common master: [DNM] Remove container-update.py  https://review.opendev.org/71848214:54
sshnaidmcloudnull, I wonder how it prints variables in multinode job: https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_ace/718184/3/check/tripleo-ci-centos-8-containers-multinode/acebcfb/logs/undercloud/home/zuul/overcloud_deploy.log14:55
*** weshay_ is now known as weshay|ruck14:55
chemEmilienM: ekultails this file is docker centric, so I guess it's safe to remove it to train14:55
chemand stein14:55
EmilienMbandini: have you already seen that on the undercloud, /etc/pki/tls/private/overcloud_endpoint.pem is always re-generated for some reasons14:55
EmilienMchem: cool; thanks14:55
EmilienMekultails: so yeah, please rm it14:56
*** hoonetorg has quit IRC14:56
ekultailsEmilienM: chem: Thanks for the guidance!14:56
bandiniEmilienM: nope never noticed that (not too familiar with those bits)14:56
sshnaidmcloudnull, oh, it has "--verbose" which we don't have in ovb14:56
sshnaidmand why that14:57
EmilienMbandini: ok no prob14:57
EmilienMbandini: i think juan did these bits14:57
openstackgerritLuke Short proposed openstack/tripleo-common master: Remove container-update.py  https://review.opendev.org/71848214:57
bandiniack14:57
openstackgerritDave Wilde (d34dh0r53) proposed openstack/tripleo-heat-templates stable/train: Create DNS entries in IPA for openstack services  https://review.opendev.org/71819114:58
openstackgerritMarios Andreou proposed openstack/tripleo-operator-ansible master: Adds overcloud roles generate role - creates roles_data.yaml  https://review.opendev.org/71663914:59
weshay|ruckmwhahaha, EmilienM release engineers are looking for release notes to be merged for puppet-tripleo I think per: https://review.opendev.org/#/c/718297/15:01
weshay|ruckor I have to add something to the release bits..15:01
EmilienMjust ship it15:02
EmilienM...15:02
* mwhahaha hates the version debate every release15:02
*** TrevorV has joined #tripleo15:02
weshay|ruck:)15:02
weshay|ruckk.. just checking in w/ you guys to confirm..15:03
weshay|ruckk.. sounds like "trailing release.. so we'll get the release notes in at a later time"15:03
*** ykarel is now known as ykarel|away15:04
EmilienMade_lee: who is the best contact to talk about ssl certs generation in tripleo15:06
ade_leeEmilienM, probably me15:07
EmilienMade_lee: there is a bug (not reported yet) where /etc/pki/tls/private/overcloud_endpoint.pem is always re-generated when deploying standalone or undercloud15:07
openstackgerritSagi Shnaidman proposed openstack/tripleo-quickstart master: Add verbosity for overcloud deploy in OVB  https://review.opendev.org/71848715:07
EmilienMI wonder if it's in certmonger_certificate puppet provider15:08
EmilienMor in certmonger-haproxy-refresh.sh15:08
openstackgerritPiotr Kopec proposed openstack/tripleo-ansible master: Add new role to manage persistent memory devices  https://review.opendev.org/71715815:08
ade_leeEmilienM, sorry - in meeting right now -- will respond in a bit15:08
openstackgerritDave Wilde (d34dh0r53) proposed openstack/tripleo-heat-templates stable/train: Create DNS entries in IPA for openstack services  https://review.opendev.org/71819115:11
*** holser has quit IRC15:12
sshnaidmmwhahaha, wrt https://etherpad.openstack.org/p/tripleo-operator-ansible - so, do we create operators for "plan" and related commands? Or just leaving them behind15:13
mwhahahai would create them15:14
mwhahahabecause they are still things and used in previous versions15:14
sshnaidmok15:16
*** holser has joined #tripleo15:16
*** bogdando has quit IRC15:18
*** holser has quit IRC15:18
sshnaidmcloudnull, I suspect the problem is not with string/dictionary pass, since it works fine in multinode job15:19
sshnaidmcloudnull, maybe these multiline private keys..15:19
*** holser has joined #tripleo15:20
openstackgerritPiotr Kopec proposed openstack/tripleo-heat-templates master: Add parameters for vPMEM features  https://review.opendev.org/71765115:21
mariosmwhahaha: sshnaidm: panda: updated when you have time thanks https://review.opendev.org/#/c/716639/15:25
EmilienMade_lee: for you (for once)15:25
EmilienMhttps://bugs.launchpad.net/tripleo/+bug/187166315:25
openstackLaunchpad bug 1871663 in tripleo "/etc/pki/tls/private/overcloud_endpoint.pem is generated at each undercloud/standalone deploy" [High,Triaged]15:25
openstackgerritMerged openstack/tripleo-ansible stable/train: Relax get_failed_containers() filter  https://review.opendev.org/71828115:27
openstackgerritKevin Carter (cloudnull) proposed openstack/tripleo-ansible master: WIP Create derived parameters playbook, modules, and role  https://review.opendev.org/71818415:27
cloudnullsshnaidm those keys should be ok?15:27
cloudnullassuming they're  encapsulated in a dictionary ?15:28
cloudnullmaybe we do the inverse of your review and dump to yaml, which would allow the playbook to set the data accordingly ?15:29
openstackgerritMichele Baldessari proposed openstack/tripleo-heat-templates master: Move the haproxy iptables rules creation to host_prep_tasks  https://review.opendev.org/71815915:29
mwhahahabandini: -^ thought we managed those via anasible now?15:31
openstackgerritMerged openstack/tripleo-ansible stable/train: tripleo_container_rm: fix check mode  https://review.opendev.org/71828215:31
openstackgerritMerged openstack/tripleo-ansible stable/train: tripleo_container_manage: add check mode testing  https://review.opendev.org/71828315:31
openstackgerritMerged openstack/tripleo-ansible stable/train: container_puppet_config: don't show a warning if config_hash is empty  https://review.opendev.org/71828515:31
* mwhahaha is concerned15:32
bandinimwhahaha: so I tested it on train where we did not do the ansible move yet, but the code was there in master15:32
bandinimwhahaha: but yeah master then needs figuring out15:32
bandinilol at my english15:33
bandinimwhahaha: I meant that to say is that haproxy puppet firewall snippet is there in master as well15:34
*** lmiccini has quit IRC15:34
mwhahahayea i noticed it's still there, i blame cloudnull15:34
openstackgerritEmilien Macchi proposed openstack/tripleo-heat-templates master: Deprecate KeepalivedRestart  https://review.opendev.org/71849215:34
EmilienMhjensas: ^ fyi15:34
bandinibah i cannot talk today15:34
bandinilet me check master15:35
cloudnullmwhahaha ¯\_(ツ)_/¯15:36
* mwhahaha is surprised it works15:37
bandiniyeah i wonder if composable ha actually works on master15:37
mwhahahamagic15:37
EmilienMbeagles: have you noticed that the neutron containers aren't idempotent between deploys on the undercloud?15:39
EmilienMbeagles: dhcp/l3/ovs agents only15:39
cloudnullI guess HAP doesn't use the same firewall_rules as all other services ?15:40
mwhahahacloudnull: haproxy needs special care because when it's moved, it needs all the service rules15:40
mwhahahaso it's an oddball15:40
cloudnullah15:40
bandinicloudnull: so haproxy is a bit special because it adds the rules for the stanzas it has configured15:40
bandinicloudnull: see https://github.com/openstack/puppet-tripleo/blob/master/manifests/haproxy/endpoint.pp#L27415:41
EmilienMcan someone review https://review.opendev.org/#/c/716658/ please15:42
cloudnullbandini mwhahaha, I wonder how we express that in tripleo::core::firewall_rules: or firewall_rules or if we could just call the firewall role (or the ansible iptables module) directly?15:44
openstackgerritMichael Johnson proposed openstack/tripleo-heat-templates stable/train: Revert "Allow disabling the octavia provider"  https://review.opendev.org/71849415:44
openstackgerritMichael Johnson proposed openstack/tripleo-heat-templates master: Revert "Allow disabling the ovn provider"  https://review.opendev.org/71849515:45
mwhahahacloudnull: so we could export the rules to the haproxy service15:46
mwhahahausing heat magic15:46
mwhahahasimilar to what we do with the database bits15:46
openstackgerritSagi Shnaidman proposed openstack/python-tripleoclient master: Pass params to tripleo_plan_parameters_update as yaml_dump  https://review.opendev.org/71786515:47
sshnaidmcloudnull, like that ^^15:47
openstackgerritTakashi Kajinami proposed openstack/tripleo-heat-templates master: Disable notification from services by default  https://review.opendev.org/71539415:47
sshnaidmcloudnull, maybe need to "| from_yaml" to task..15:47
*** dtantsur|brb is now known as dtantsur15:48
openstackgerritTakashi Kajinami proposed openstack/tripleo-heat-templates master: Disable notification from services by default  https://review.opendev.org/71539415:53
openstackgerritTakashi Kajinami proposed openstack/instack-undercloud stable/queens: Add missing README.rst in elements  https://review.opendev.org/71850415:56
cloudnullsshnaidm ++15:56
cloudnullthat should work, I think?15:56
cloudnullthe task in the playbook will set that option as a fact, which "should"-tm, convert it15:56
sshnaidmcloudnull, let's see, I'll spawn a reproducer to play with this..15:57
cloudnull++15:59
*** xek__ has joined #tripleo16:00
*** hoonetorg has joined #tripleo16:01
*** xek_ has quit IRC16:02
*** lucasagomes has quit IRC16:03
*** sshnaidm is now known as sshnaidm|afk16:04
*** jcoufal has joined #tripleo16:04
*** xek__ has quit IRC16:05
*** Abdallahyas has quit IRC16:07
*** cylopez has quit IRC16:08
*** marios has quit IRC16:09
bandinimwhahaha: if my review passes CI and reviews and whatnot I'll still merge it while we investigate the composable control plane firewall ansible stuff, yes? (since i need to backport it to train)16:10
mwhahahayes16:10
bandiniok16:10
openstackgerritMerged openstack/tripleo-operator-ansible master: Allows to get the standalone deploy reproducer script  https://review.opendev.org/71842116:15
*** rpittau is now known as rpittau|afk16:23
*** psachin has quit IRC16:24
*** jfrancoa has quit IRC16:26
*** jfrancoa has joined #tripleo16:26
openstackgerritDamien Ciabrini proposed openstack/tripleo-common master: mariadb: add pynacl dependency to support ed25519  https://review.opendev.org/71845316:28
*** hkominos has joined #tripleo16:28
*** dpawlik has quit IRC16:29
hkominosHi guys, During an overcloud upgrade from Rocky -->Stein I stumbled upon an  issue (tackValidationFailed: The Resource Type (OS::TripleO::Services::GlanceRegistry) could not be found). I wonder if this is related with the fact that I a deploying from the default location of the heat templates (/usr/share) which has changed a bit after moving to the16:30
hkominosnew binary ??16:30
hkominosOr could there be another reason ?? (btw the endpoint is working fine)16:31
*** jpich has quit IRC16:31
*** jcapitao has quit IRC16:32
*** evrardjp has quit IRC16:37
*** evrardjp has joined #tripleo16:37
mwhahahahkominos: your role file needs ot be updated to remove that service16:42
hkominosmwhahaha thx. I Assume there is some commit that I have missed that defines that the service has moved ?16:46
hkominosor something ?16:46
mwhahahathe glance registry hasn't been a thing since pike16:46
mwhahahaso we finally dropped it in stein16:46
hkominosok thx !16:46
mwhahahawe kept the heat service definition around for upgrades, but eventually it has to be removed16:47
mwhahahaso you need to make sure your roles_data.yaml has been updated with rocky services16:47
hkominosperfect .Thx for the help!16:50
*** xek__ has joined #tripleo16:53
*** jmasud has quit IRC16:54
*** jmasud has joined #tripleo16:56
*** xek__ is now known as xek16:58
*** derekh has quit IRC16:59
openstackgerritMerged openstack/tripleo-operator-ansible master: Fix task names  https://review.opendev.org/71820417:01
openstackgerritMerged openstack/tripleo-operator-ansible master: Fix debug var  https://review.opendev.org/71819817:05
openstackgerritMerged openstack/puppet-tripleo master: Fix broken unit tests  https://review.opendev.org/71827717:05
openstackgerritMerged openstack/instack-undercloud stable/rocky: Add missing README.rst in elements  https://review.opendev.org/71829517:06
*** dtantsur is now known as dtantsur|afk17:07
*** amoralej is now known as amoralej|off17:14
openstackgerritLukas Bezdicka proposed openstack/tripleo-heat-templates stable/train: [Q->T] Introduce Queens to Train Upgrade  https://review.opendev.org/71348217:18
ade_leemwhahaha, EmilienM https://review.opendev.org/#/c/706564/ please17:24
ade_leemwhahaha, EmilienM cloudnull hjensas https://review.opendev.org/#/c/716784/  please17:26
beaglesEmilienM: there was something that we changed recently to do with deploy identifiers.. but these were/should've only been added to containers that performed some additional configuration steps17:27
*** suuuper has quit IRC17:29
*** jcoufal has quit IRC17:34
lbragstadEmilienM a few of us are working on standing up a new environment (without tls) to see if we can recreate the connection issue during scale down17:41
lbragstadthanks hjensas cloudnull17:42
*** gfidente is now known as gfidente|afk17:49
openstackgerritGiulio Fidente proposed openstack/tripleo-ansible master: Add admin user to mistral group  https://review.opendev.org/71846917:52
*** xek_ has joined #tripleo17:56
*** xek has quit IRC17:58
*** gfidente|afk is now known as gfidente18:10
*** holser__ has joined #tripleo18:16
openstackgerritKevin Carter (cloudnull) proposed openstack/tripleo-ansible master: WIP Create derived parameters playbook, modules, and role  https://review.opendev.org/71818418:16
*** holser has quit IRC18:17
*** hkominos has quit IRC18:20
*** mrebisz has quit IRC18:30
owalshzzzeek: hey, just spotted this https://review.opendev.org/672173 ...18:35
zzzeekowalsh: yes18:36
owalshzzzeek: missed a few places for nova e.g https://github.com/openstack/tripleo-heat-templates/blob/master/deployment/nova/nova-api-container-puppet.yaml#L30518:37
*** rlandy|pto is now known as rlandy18:37
owalshand this one, which might be a problem - https://github.com/openstack/tripleo-heat-templates/blob/master/deployment/nova/nova-api-container-puppet.yaml#L36918:38
*** gfidente is now known as gfidente|afk18:38
owalshwe have logic to update the database/mq uri for the default cell, but not cell018:39
zzzeekowalsh: is it possible that is some newer config stuff or was it there when I did this?  i thought I tested ?18:39
owalshzzzeek: there since queens I suspect18:40
*** mcornea has joined #tripleo18:41
owalshzzzeek: i doubt metrics for the cell0 db are interesting anyway18:41
owalshzzzeek: ah, no actually it will work... it's templated now https://review.opendev.org/61140418:44
zzzeekwe wnat metrics for every DB connection since number of db connecvtions is the main thing we want to look at18:45
owalshzzzeek: since https://review.opendev.org/611404  we set those uri to (literally) '{scheme}://{username}:{password}@{hostname}/nova_cell0?{query}', nova substitutes the values from nova.conf18:46
owalshzzzeek: so we should get the collectd args from {query}18:48
openstackgerritRonelle Landy proposed openstack/tripleo-quickstart-extras master: DNM: Testing ipa multinode  https://review.opendev.org/70637318:48
zzzeekowalsh: yes that was the idea IIRC18:48
owalshzzzeek: ignore me, when I looked at it I though the substitution was happening in heat18:49
owalshthought18:49
zzzeekowalsh: i've never even seen that file before :)18:49
owalshzzzeek: I have, my name on the patch :-)18:53
*** dciabrin_ has joined #tripleo18:57
openstackgerritKevin Carter (cloudnull) proposed openstack/python-tripleoclient master: Expose --limit, --skip-tags, and --tags on the CLI.  https://review.opendev.org/69242118:59
*** slaweq_ has joined #tripleo18:59
*** dciabrin has quit IRC18:59
*** slaweq has quit IRC19:00
*** bandini has quit IRC19:09
*** dsneddon has joined #tripleo19:09
*** maciejjozefczyk has quit IRC19:24
openstackgerritKevin Carter (cloudnull) proposed openstack/python-tripleoclient master: Expose --limit, --skip-tags, and --tags on the CLI.  https://review.opendev.org/69242119:24
openstackgerritFlavio Fernandes proposed openstack/tripleo-heat-templates stable/train: Add Octavia OVN Provider configuration  https://review.opendev.org/71491619:26
*** jfrancoa has quit IRC19:27
weshay|ruckhrm.. what creates /var/lib/certmonger/local/creds https://opendev.org/openstack/puppet-tripleo/src/branch/master/manifests/certmonger/ca/local.pp#L2819:29
openstackgerritMerged openstack/tripleo-heat-templates master: [OVN] SRIOV with native OVN DHCP server  https://review.opendev.org/71043719:30
openstackgerritMerged openstack/python-tripleoclient master: Add releasenote for removal of overcloud remote execute command  https://review.opendev.org/71830819:30
weshay|ruckade_lee, scenario001 is now fixed19:30
ade_leeweshay|ruck, yay!19:30
ade_leeweshay|ruck, /me looking to recheck stuff ..19:30
ade_leeweshay|ruck, thats featureset001 in rdo third party ci check?19:32
weshay|ruckno.. standalone scenario0119:33
weshay|ruckfs001 in third party has a known bug and a patch is up19:33
hjensasstevebaker: dtantsur|afk's idea about an introspection rule sounds promising for https://bugs.launchpad.net/tripleo/+bug/187052919:33
openstackLaunchpad bug 1870529 in tripleo "node provisioning with ansible fails" [High,Triaged] - Assigned to Steve Baker (steve-stevebaker)19:34
hjensasstevebaker: did you start working on that alread? If not I can look into the introspection rule approach.19:34
*** alexmcleod has quit IRC19:34
weshay|ruckade_lee, seen any issues on centos8 w/ /var/lib/certmonger/local/creds not being created?19:35
ade_leeweshay|ruck, that sounds sorta familiar-ish -- I think what is needed there is a certmonger restart19:36
stevebakerhjensas: I haven't started yet, do you want to take it?19:36
weshay|ruckade_lee, this is the bit failing https://opendev.org/openstack/puppet-tripleo/src/branch/master/manifests/certmonger/ca/local.pp#L2919:37
weshay|ruckon ca_pkcs12 missing19:37
hjensasstevebaker: yes, I was'nt aware of introspection rules. but there is an in-net condition. We might actually be able to set the proper physnet for a deployment on routed networks.19:37
weshay|ruckade_lee, works fine in centos-7 train.. just not ussuri centos-819:38
hjensasstevebaker: I will assign it to myself and get going asap.19:38
stevebakerhjensas: sounds good. And we can assume that if they don't set physical_network in instackenv then they must run introspection?19:38
hjensasstevebaker: if they have a 'mac'19:39
stevebakeryep19:39
weshay|ruckade_lee, don't see a bug..19:39
weshay|ruckso I'll put details there19:40
hjensasstevebaker: field in instackenv.json it will default to ctlplane. The issue here is no mac or port was provided in instackenv.json, so introspection is used to "discover" the ports.19:40
stevebakeroohh19:40
hjensasstevebaker: the cool thing with these introspection rules is that we can program them via the API, so based on the config in undercloud.conf we should be able to specify the correct phys-net based on the network. (I hope, only had a 2 min glance at the docs so far. but promising. :)19:42
stevebakernice19:43
openstackgerritKevin Carter (cloudnull) proposed openstack/python-tripleoclient master: Expose --limit, --skip-tags, and --tags on the CLI.  https://review.opendev.org/69242119:43
*** Trevor_V has joined #tripleo19:48
*** TrevorV has quit IRC19:51
ade_leeEmilienM, so yeah - looking at puppet-certmonger, it seems that its doing a resubmit if the cert is already there - which will re-issue the cert.20:02
EmilienMade_lee: that's weird, we should fix that20:02
ade_leeEmilienM, yeah - looking to see how we call that -- on the other hand though, there is maybe a bug in that haproxy should reload if the cert has changed20:03
ade_leeweshay|ruck, looking -- the issue sounds familiar - but all the certmonger issues blend together after awhile :/20:07
*** ccamacho has quit IRC20:08
openstackgerritMerged openstack/tripleo-heat-templates master: Properly place undercloud hosts record upgrade task  https://review.opendev.org/71787220:10
openstackgerritMerged openstack/tripleo-upgrade master: Switch install-docker with ensure-docker  https://review.opendev.org/71782820:10
openstackgerritEmilien Macchi proposed openstack/tripleo-ansible master: WIP - tripleo_container_manage: optimize systemd services/healthcheck bootstrap  https://review.opendev.org/71854520:11
openstackgerritMerged openstack/tripleo-upgrade stable/train: Get rid of the old container images environment file.  https://review.opendev.org/71769820:14
openstackgerritEmilien Macchi proposed openstack/tripleo-ansible master: WIP - tripleo_container_manage: optimize systemd services/healthcheck bootstrap  https://review.opendev.org/71854520:15
openstackgerritMerged openstack/tripleo-upgrade stable/train: Stop assuming a specific inventory structure  https://review.opendev.org/71769520:16
openstackgerritMerged openstack/tripleo-upgrade master: Stop assuming a specific inventory structure  https://review.opendev.org/71549120:16
openstackgerritMerged openstack/tripleo-common master: Fix multistack inventory  https://review.opendev.org/71375020:17
openstackgerritMerged openstack/tripleo-validations master: Fix multistack inventory  https://review.opendev.org/71391920:17
openstackgerritMerged openstack/tripleo-heat-templates master: Exclude /var/lib/ironic/* from container-puppet.sh rsync  https://review.opendev.org/71665820:17
openstackgerritEmilien Macchi proposed openstack/tripleo-heat-templates stable/train: Exclude /var/lib/ironic/* from container-puppet.sh rsync  https://review.opendev.org/71854820:19
openstackgerritEmilien Macchi proposed openstack/tripleo-ansible master: WIP - tripleo_container_manage: optimize systemd services/healthcheck bootstrap  https://review.opendev.org/71854520:23
ade_leeweshay|ruck, the creds thing is a little odd -- do you have a reproducer system?20:35
ade_leeweshay|ruck, and is it intermittent?20:36
weshay|ruckade_lee, I had two.. I think I have one..20:36
ade_leeweshay|ruck, wondering if its a timing thing ..20:36
weshay|ruckya.. I have one.. I can tmate you in..20:37
ade_leecool20:37
*** xek_ has quit IRC20:44
*** rlandy has quit IRC20:46
*** pkopec has quit IRC20:46
openstackgerritKevin Carter (cloudnull) proposed openstack/tripleo-ansible master: WIP Create derived parameters playbook, modules, and role  https://review.opendev.org/71818420:47
*** jmasud has quit IRC20:54
openstackgerritOliver Walsh proposed openstack/tripleo-heat-templates master: WIP: refactor nova db config  https://review.opendev.org/71855220:54
*** jmasud has joined #tripleo20:56
openstackgerritRajini Karthik proposed openstack/tripleo-heat-templates master: Support for PowerMax Cinder Backend  https://review.opendev.org/71253321:18
openstackgerritOliver Walsh proposed openstack/tripleo-common stable/train: Add tripleo_role_networks to inventory role group vars  https://review.opendev.org/71855521:19
openstackgerritOliver Walsh proposed openstack/tripleo-common stable/train: Fix multistack inventory  https://review.opendev.org/71855621:19
openstackgerritOliver Walsh proposed openstack/tripleo-validations stable/train: Fix multistack inventory  https://review.opendev.org/71855721:20
*** jtomasek has quit IRC21:25
*** TrevorV has joined #tripleo21:29
openstackgerritKevin Carter (cloudnull) proposed openstack/tripleo-ansible master: WIP Create derived parameters playbook, modules, and role  https://review.opendev.org/71818421:31
*** Trevor__V has joined #tripleo21:31
*** Trevor_V has quit IRC21:32
*** TrevorV has quit IRC21:34
*** slaweq_ has quit IRC21:38
*** jdandrea has quit IRC21:46
*** jdandrea has joined #tripleo21:47
ade_leeEmilienM, mwhahaha , cloudnull , hjensas https://review.opendev.org/#/c/706564/ please21:49
EmilienMade_lee: why do we delegate on undercloud?21:51
EmilienMit should be localhost no?21:51
EmilienMwe shouldn't hardcode undercloud IMO21:51
ade_leeEmilienM, ack -- I'll change it21:52
openstackgerritAde Lee proposed openstack/tripleo-heat-templates master: Added scale_tasks to handle cleanup on scale down of nodes  https://review.opendev.org/70656421:56
ade_leeEmilienM, ^^21:56
*** mcornea has quit IRC21:58
EmilienMade_lee: +221:58
ade_leeEmilienM, thanks21:59
ade_leehjensas, I think my update squashed your +2/W ..21:59
ade_leehjensas, weshay|ruck thanks!22:00
openstackgerritAlex Schultz proposed openstack/tripleo-heat-templates master: Use kolla tools for memecached configuration  https://review.opendev.org/71856522:08
*** iurygregory has quit IRC22:09
*** slaweq_ has joined #tripleo22:13
openstackgerritAlex Schultz proposed openstack/tripleo-heat-templates master: Use kolla tools for memecached configuration  https://review.opendev.org/71856522:16
*** Trevor__V has quit IRC22:16
*** TrevorV has joined #tripleo22:17
openstackgerritAlex Schultz proposed openstack/tripleo-heat-templates master: Use kolla tools for memecached configuration  https://review.opendev.org/71856522:26
openstackgerritLuke Short proposed openstack/python-tripleoclient stable/train: Expose --limit, --skip-tags, and --tags on the CLI.  https://review.opendev.org/71665422:30
openstackgerritEmilien Macchi proposed openstack/tripleo-ansible master: tripleo_container_manage: optimize systemd services/healthcheck bootstrap  https://review.opendev.org/71854522:31
*** rcernin has joined #tripleo22:32
*** holser__ has quit IRC22:34
*** TrevorV has quit IRC22:50
*** saneax has joined #tripleo23:01
openstackgerritAlex Schultz proposed openstack/tripleo-heat-templates master: Use kolla tools for memcached configuration  https://review.opendev.org/71856523:09
*** threestrands has joined #tripleo23:10
*** sanjayu_ has joined #tripleo23:11
*** saneax has quit IRC23:14
*** tosky has quit IRC23:20
openstackgerritMerged openstack/tripleo-heat-templates master: Skip both tenant and management networks when generating certs  https://review.opendev.org/69684223:24
openstackgerritLuke Short proposed openstack/python-tripleoclient master: Expose --limit, --skip-tags, and --tags on the CLI.  https://review.opendev.org/69242123:37
openstackgerritTakashi Kajinami proposed openstack/puppet-tripleo master: Add missing unit tests for placement manifests.  https://review.opendev.org/71857423:39
openstackgerritTakashi Kajinami proposed openstack/puppet-tripleo master: Add missing unit tests for placement manifests  https://review.opendev.org/71857423:40
openstackgerritTakashi Kajinami proposed openstack/puppet-tripleo master: Add missing unit tests for placement manifests  https://review.opendev.org/71857423:42
openstackgerritTakashi Kajinami proposed openstack/puppet-tripleo master: Fix incorrect spec file name  https://review.opendev.org/71857523:42
openstackgerritDan Sneddon proposed openstack/os-net-config stable/train: Run ifup on a bond when a slave interface is restarted  https://review.opendev.org/71857623:45
openstackgerritDan Sneddon proposed openstack/os-net-config stable/stein: Run ifup on a bond when a slave interface is restarted  https://review.opendev.org/71857723:49
openstackgerritDan Sneddon proposed openstack/os-net-config stable/rocky: Run ifup on a bond when a slave interface is restarted  https://review.opendev.org/71857823:50
openstackgerritAlex Schultz proposed openstack/tripleo-heat-templates master: Use kolla tools for memcached configuration  https://review.opendev.org/71856523:50
openstackgerritDan Sneddon proposed openstack/os-net-config stable/queens: Run ifup on a bond when a slave interface is restarted  https://review.opendev.org/71857923:50

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