openstackgerrit | Merged openstack/tripleo-common stable/train: Don't build image upload tasks when dry_run=True https://review.opendev.org/745999 | 00:00 |
---|---|---|
*** alexmcleod has quit IRC | 01:24 | |
*** alexmcleod has joined #tripleo | 01:25 | |
openstackgerrit | Merged openstack/tripleo-common stable/ussuri: Remove check for ./puppet/services when filtering services https://review.opendev.org/746431 | 01:37 |
openstackgerrit | wes hayutin proposed openstack/tripleo-ci master: DNM, debug container build role path https://review.opendev.org/746624 | 01:38 |
openstackgerrit | Merged openstack/tripleo-heat-templates master: Do not hard-code vars_from https://review.opendev.org/746125 | 01:54 |
openstackgerrit | Merged openstack/tripleo-validations stable/ussuri: Updated "overcloud" to "allovercloud" group https://review.opendev.org/745787 | 01:55 |
openstackgerrit | Merged openstack/tripleo-heat-templates stable/train: Fix delegation with FreeIPA cleanup https://review.opendev.org/745956 | 02:28 |
openstackgerrit | Merged openstack/tripleo-heat-templates master: Cleanup parameters for keystone::bootstrap https://review.opendev.org/738382 | 02:28 |
openstackgerrit | Merged openstack/tripleo-heat-templates master: Consolidate the barbican-api client configurations https://review.opendev.org/744747 | 02:28 |
openstackgerrit | Merged openstack/puppet-tripleo master: Cleanup manila netapp deprecated option usage https://review.opendev.org/745758 | 02:28 |
openstackgerrit | Merged openstack/puppet-tripleo master: Allow not setting hiera cephfs_enable_snapshots https://review.opendev.org/745951 | 02:29 |
openstackgerrit | Dan Sneddon proposed openstack/tripleo-heat-templates master: Create external bridge on Compute nodes by default for OVN with DVR https://review.opendev.org/744791 | 03:00 |
*** skramaja has joined #tripleo | 03:24 | |
*** zbr has quit IRC | 03:25 | |
*** zbr has joined #tripleo | 03:26 | |
*** psachin has joined #tripleo | 03:38 | |
*** bandini has joined #tripleo | 03:50 | |
*** ysandeep|away is now known as ysandeep | 04:21 | |
*** raukadah is now known as chkumar|rover | 04:25 | |
*** ratailor has joined #tripleo | 04:30 | |
*** ramishra has quit IRC | 04:32 | |
*** evrardjp has quit IRC | 04:33 | |
*** evrardjp has joined #tripleo | 04:33 | |
*** ramishra has joined #tripleo | 04:34 | |
openstackgerrit | Ian Wienand proposed openstack/diskimage-builder master: Makes EFI images bootable by bios https://review.opendev.org/743243 | 04:41 |
openstackgerrit | Matthew Thode proposed openstack/diskimage-builder master: update gentoo to allow building arm64 images https://review.opendev.org/746000 | 04:50 |
*** abdysn has joined #tripleo | 05:11 | |
*** ykarel_ has joined #tripleo | 05:16 | |
*** ykarel_ is now known as ykarel | 05:16 | |
*** waleedm has joined #tripleo | 05:18 | |
openstackgerrit | Merged openstack/puppet-tripleo master: Avoid direct reference of hieradata https://review.opendev.org/746243 | 05:21 |
openstackgerrit | Merged openstack/puppet-tripleo stable/stein: [stable only] Fix duplicated test case for nova::api https://review.opendev.org/746244 | 05:21 |
openstackgerrit | Zhu Sheng Li proposed openstack/tripleo-common stable/train: Ensure image files are readable to apache https://review.opendev.org/746133 | 05:23 |
*** dmacpher_ has quit IRC | 05:25 | |
*** lmiccini has joined #tripleo | 05:32 | |
openstackgerrit | Merged openstack/diskimage-builder master: Handle NetworkManager for dhcp-all-interfaces https://review.opendev.org/745698 | 05:35 |
*** saneax has joined #tripleo | 05:39 | |
*** marios has joined #tripleo | 05:52 | |
*** marios is now known as marios|ruck | 05:52 | |
openstackgerrit | Chandan Kumar (raukadah) proposed openstack/tripleo-ci master: [DNM] pprint https://review.opendev.org/746633 | 05:59 |
openstackgerrit | Cédric Jeanneret (Tengu) proposed openstack/tripleo-validations stable/ussuri: Allows to use custom role name for validations https://review.opendev.org/746635 | 06:03 |
openstackgerrit | Merged openstack/tripleo-heat-templates stable/ussuri: Update, avoid task skipping by directly importing step file. https://review.opendev.org/746490 | 06:07 |
openstackgerrit | Merged openstack/tripleo-heat-templates stable/train: Add support for Gnocchi NFS Backend https://review.opendev.org/745992 | 06:07 |
openstackgerrit | Merged openstack/tripleo-heat-templates stable/train: Add PermitRootLogin option in sshd_config before leapp. https://review.opendev.org/746033 | 06:07 |
openstackgerrit | Merged openstack/tripleo-heat-templates stable/train: pcmk_remote FFU support for Instance HA https://review.opendev.org/746150 | 06:07 |
openstackgerrit | Merged openstack/puppet-tripleo stable/train: Use pcs 0.9 style authkey/remotes when doing an upgrade https://review.opendev.org/746152 | 06:07 |
openstackgerrit | Merged openstack/tripleo-heat-templates master: [ovn]: Enable port forwarding in neutron service plugins https://review.opendev.org/745756 | 06:07 |
openstackgerrit | Merged openstack/tripleo-heat-templates master: Use tripleo_network_config https://review.opendev.org/744261 | 06:07 |
openstackgerrit | Merged openstack/diskimage-builder master: source-repositories: git is a build-only dependency https://review.opendev.org/745678 | 06:07 |
openstackgerrit | Cédric Jeanneret (Tengu) proposed openstack/tripleo-ansible stable/train: Ensure those two tasks are compatible with t-h-t import https://review.opendev.org/746637 | 06:09 |
openstackgerrit | Chandan Kumar (raukadah) proposed openstack/tripleo-ci master: Remove pprint from test-requirements.txt https://review.opendev.org/746633 | 06:09 |
*** psahoo has joined #tripleo | 06:13 | |
openstackgerrit | Cédric Jeanneret (Tengu) proposed openstack/tripleo-heat-templates stable/ussuri: Do not hard-code vars_from https://review.opendev.org/746638 | 06:16 |
openstackgerrit | Sandeep Yadav proposed openstack/tripleo-quickstart-extras master: Generalize vars in template file in build-test-packages role https://review.opendev.org/745819 | 06:16 |
openstackgerrit | Sandeep Yadav proposed openstack/tripleo-quickstart-extras master: Generalize vars in template file in build-test-packages role https://review.opendev.org/745819 | 06:16 |
openstackgerrit | Cédric Jeanneret (Tengu) proposed openstack/tripleo-heat-templates stable/train: Do not hard-code vars_from https://review.opendev.org/746639 | 06:17 |
*** psahoo has quit IRC | 06:23 | |
*** ysandeep is now known as ysandeep|brb | 06:28 | |
*** psahoo has joined #tripleo | 06:38 | |
*** maciejjozefczyk has joined #tripleo | 06:46 | |
*** ysandeep|brb is now known as ysandeep | 06:48 | |
openstackgerrit | Merged openstack/tripleo-ansible master: Add ansible module for container image prepare https://review.opendev.org/745510 | 06:52 |
openstackgerrit | Marios Andreou proposed openstack/tripleo-heat-templates stable/ussuri: DNM Test centos8 upgrade jobs ussuri https://review.opendev.org/739457 | 06:57 |
*** rcernin has quit IRC | 07:02 | |
openstackgerrit | Jean Pierre Roquesalane proposed openstack/tripleo-specs master: PowerFlex integration into Tripleo https://review.opendev.org/724301 | 07:03 |
*** rcernin has joined #tripleo | 07:03 | |
*** maciejjozefczyk has quit IRC | 07:05 | |
*** slaweq has joined #tripleo | 07:06 | |
*** slaweq has quit IRC | 07:11 | |
*** bshephar has quit IRC | 07:16 | |
*** slaweq has joined #tripleo | 07:16 | |
openstackgerrit | yatin proposed openstack/tripleo-quickstart-extras master: [DNM] Debug issue with ansible-2.9.12 https://review.opendev.org/746644 | 07:18 |
*** maciejjozefczyk has joined #tripleo | 07:24 | |
openstackgerrit | Sandeep Yadav proposed openstack/tripleo-quickstart-extras master: Generalize vars in template file in build-test-packages role https://review.opendev.org/745819 | 07:27 |
openstackgerrit | Arx Cruz proposed openstack/tripleo-quickstart-extras master: Os_tempest everywhere https://review.opendev.org/746509 | 07:28 |
*** tosky has joined #tripleo | 07:40 | |
openstackgerrit | Merged openstack/diskimage-builder master: Fedora 32 support https://review.opendev.org/737217 | 07:43 |
*** bogdando has joined #tripleo | 07:47 | |
*** apetrich has joined #tripleo | 07:48 | |
*** xek has joined #tripleo | 07:51 | |
*** gfidente has joined #tripleo | 07:51 | |
*** bogdando_ has joined #tripleo | 07:52 | |
openstackgerrit | Chandan Kumar (raukadah) proposed openstack/tripleo-ci master: Add container excludes for rhos-16.2 builds https://review.opendev.org/744166 | 07:53 |
*** bogdando has quit IRC | 07:54 | |
*** bogdando_ is now known as bogdando | 07:54 | |
*** jpena|off is now known as jpena | 07:55 | |
openstackgerrit | Merged openstack/diskimage-builder master: Makes EFI images bootable by bios https://review.opendev.org/743243 | 07:59 |
*** jpich has joined #tripleo | 07:59 | |
openstackgerrit | Michele Baldessari proposed openstack/tripleo-heat-templates master: WIP Fix up ovn-dbs update tasks https://review.opendev.org/746651 | 07:59 |
openstackgerrit | Merged openstack/diskimage-builder master: Update name of ipa job https://review.opendev.org/743042 | 08:04 |
openstackgerrit | Michele Baldessari proposed openstack/puppet-tripleo stable/ussuri: Make sure python3-novaclient is installed before creating fence_compute https://review.opendev.org/746653 | 08:06 |
openstackgerrit | Merged openstack/python-tripleoclient master: Remove playbook param from tripleo_validator.py https://review.opendev.org/746498 | 08:09 |
openstackgerrit | Michele Baldessari proposed openstack/puppet-tripleo stable/train: Make sure python3-novaclient is installed before creating fence_compute https://review.opendev.org/746654 | 08:09 |
*** lucasagomes has joined #tripleo | 08:12 | |
openstackgerrit | Merged openstack/diskimage-builder master: Remove glance-registry https://review.opendev.org/739796 | 08:12 |
openstackgerrit | Harald Jensås proposed openstack/python-tripleoclient master: Limit ip_version when resolving public_host and admin_host https://review.opendev.org/746656 | 08:13 |
*** bogdando_ has joined #tripleo | 08:18 | |
*** bogdando has quit IRC | 08:19 | |
*** bogdando_ is now known as bogdando | 08:19 | |
*** gfidente has quit IRC | 08:28 | |
*** gfidente has joined #tripleo | 08:29 | |
openstackgerrit | Michele Baldessari proposed openstack/tripleo-heat-templates master: WIP test concurrent restart pcs fix https://review.opendev.org/746660 | 08:29 |
*** rcernin has quit IRC | 08:34 | |
openstackgerrit | Sagi Shnaidman proposed openstack/tripleo-ci master: DNM dont hide logs - containers https://review.opendev.org/746661 | 08:39 |
openstackgerrit | Michele Baldessari proposed openstack/tripleo-heat-templates stable/train: WIP test concurrent restart pcs fix https://review.opendev.org/746662 | 08:46 |
openstackgerrit | Michele Baldessari proposed openstack/tripleo-heat-templates stable/queens: WIP test concurrent restart pcs fix https://review.opendev.org/746665 | 08:56 |
*** skramaja_ has joined #tripleo | 09:28 | |
*** skramaja has quit IRC | 09:29 | |
*** skramaja_ is now known as skramaja | 09:37 | |
openstackgerrit | Harald Jensås proposed openstack/tripleo-heat-templates master: Fix enporint map tls port for zaqar wss https://review.opendev.org/746671 | 09:40 |
openstackgerrit | Harald Jensås proposed openstack/python-tripleoclient master: Allow same addr for undercloud public and admin endpoint https://review.opendev.org/746672 | 09:42 |
openstackgerrit | Daniel Bengtsson proposed openstack/tripleo-heat-templates stable/train: Update, avoid task skipping by directly importing step file. https://review.opendev.org/746676 | 09:56 |
*** tkajinam has quit IRC | 09:58 | |
openstackgerrit | Bogdan Dobrelya (bogdando) proposed openstack/paunch stable/ussuri: [USSURY-Only] Log benign command errors as warns https://review.opendev.org/746677 | 10:07 |
openstackgerrit | Harald Jensås proposed openstack/tripleo-heat-templates master: Fix endpoint map tls - zaqar wss port https://review.opendev.org/746671 | 10:37 |
*** shyamb has joined #tripleo | 11:09 | |
openstackgerrit | Arx Cruz proposed openstack/tripleo-quickstart-extras master: Os_tempest everywhere https://review.opendev.org/746509 | 11:09 |
*** dciabrin has quit IRC | 11:12 | |
openstackgerrit | Tobias Urdin proposed openstack/puppet-tripleo master: Use keystone::cache https://review.opendev.org/746686 | 11:19 |
shyamb | Hi | 11:19 |
shyamb | We want to achieve following goals. | 11:19 |
shyamb | 1. Need to register triliovault datamover api service in keystone | 11:19 |
shyamb | 2. Need to create a database user, set permissions and create database for triliovault datamover api service | 11:19 |
shyamb | 3. Need to create rabbitmq user for triliovault datamover api service. | 11:19 |
shyamb | This is for triliovault service | 11:20 |
shyamb | Can you please point me to document or code to refer? | 11:20 |
openstackgerrit | Tobias Urdin proposed openstack/tripleo-heat-templates master: Set keystone::cache parameters https://review.opendev.org/746687 | 11:20 |
*** dciabrin has joined #tripleo | 11:21 | |
shyamb | owalsh: I am reffering to openstack-tripleo-heat-templates and ansible | 11:21 |
shyamb | Tengu: ykarel: jaosorior: | 11:21 |
openstackgerrit | Merged openstack/diskimage-builder master: Do not install python2 packages in ubuntu focal https://review.opendev.org/745665 | 11:28 |
*** jpena is now known as jpena|lunch | 11:34 | |
*** dciabrin has quit IRC | 11:34 | |
openstackgerrit | Merged openstack/tripleo-heat-templates master: Revert "Workaround rhel8,0 iptables bug causing neutron_ovs_agent to restart continuously" https://review.opendev.org/746216 | 11:39 |
*** dciabrin has joined #tripleo | 11:41 | |
*** maciejjozefczyk has quit IRC | 11:45 | |
*** maciejjozefczyk has joined #tripleo | 11:46 | |
*** raildo has joined #tripleo | 11:48 | |
*** yurii has joined #tripleo | 11:52 | |
*** yurii is now known as yprokule | 11:53 | |
openstackgerrit | Sagi Shnaidman proposed openstack/tripleo-ci master: DNM dont hide logs - containers https://review.opendev.org/746661 | 11:55 |
*** rlandy has joined #tripleo | 12:00 | |
*** rfolco has joined #tripleo | 12:04 | |
Tengu | marios|ruck: hello there! already seen this in stable/ussuri? ERROR: Could not find a version that satisfies the requirement python-glanceclient===3.1.2 | 12:11 |
Tengu | marios|ruck: example job: https://review.opendev.org/#/c/746635/1 | 12:12 |
marios|ruck | Tengu: sounds new thanks for pointer will check | 12:12 |
Tengu | https://review.opendev.org/#/c/746638/1 another one | 12:12 |
Tengu | marios|ruck: np :) | 12:12 |
marios|ruck | Tengu: doesn't seem to be a widespread thing yet but maybe 'incoming' https://zuul.opendev.org/t/openstack/builds?job_name=openstack-tox-pep8# | 12:14 |
marios|ruck | chkumar|rover: ^^ | 12:14 |
openstackgerrit | Emilien Macchi proposed openstack/tripleo-ansible stable/train: Create a role for network config https://review.opendev.org/746600 | 12:14 |
Tengu | marios|ruck: or maybe some mirroring issue? I didn't trigger a recheck on those jobs yet | 12:14 |
chkumar|rover | checking | 12:15 |
marios|ruck | Tengu: noted for now but thanks for heads up we'll see https://hackmd.io/QnprH9-yRTi6uWlEfaahoQ?both#Tue-18-Aug | 12:16 |
Tengu | 'k. lemme do a recheck on both jobs. | 12:17 |
openstackgerrit | wes hayutin proposed openstack/tripleo-ci master: DNM, wes Separate required-projects and roles from base jobs https://review.opendev.org/746695 | 12:17 |
chkumar|rover | Tengu: https://opendev.org/openstack/requirements/commit/0cf95fef150ee7aa73bed3541d50a9707a3313f2 | 12:17 |
chkumar|rover | has updated glance version | 12:17 |
openstackgerrit | Emilien Macchi proposed openstack/tripleo-common stable/train: tcib: move cradox to gnocchi-base & enable ci on scenario001 https://review.opendev.org/741228 | 12:18 |
Tengu | humpf. | 12:18 |
openstackgerrit | Emilien Macchi proposed openstack/tripleo-common stable/train: container-images: add librados2 to gnocchi-base https://review.opendev.org/743741 | 12:18 |
openstackgerrit | Emilien Macchi proposed openstack/tripleo-common stable/train: TCIB: add python3-pynacl to mariadb image https://review.opendev.org/741331 | 12:18 |
openstackgerrit | Emilien Macchi proposed openstack/tripleo-common stable/train: TCIB: add etcd user to the kolla group https://review.opendev.org/741332 | 12:18 |
openstackgerrit | Emilien Macchi proposed openstack/tripleo-common stable/train: First pass at script cleanup https://review.opendev.org/746543 | 12:18 |
*** zzzeek has quit IRC | 12:19 | |
*** zzzeek has joined #tripleo | 12:23 | |
Tengu | chkumar|rover: so apparently there isn't any 3.1.2... 3.1.1 and 3.2.0 are available, according to the error message | 12:25 |
*** shyam89 has joined #tripleo | 12:27 | |
*** shyamb has quit IRC | 12:30 | |
*** ysandeep is now known as ysandeep|brb | 12:30 | |
openstackgerrit | Emilien Macchi proposed openstack/tripleo-ansible master: Create tripleo_container_config_scripts module https://review.opendev.org/746218 | 12:32 |
openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates master: Create container config scripts with a new module https://review.opendev.org/746219 | 12:33 |
chkumar|rover | Tengu: https://pypi.org/project/python-glanceclient/3.1.2/ | 12:33 |
Tengu | chkumar|rover: so mirroring issue? would be weird. | 12:33 |
openstackgerrit | yatin proposed openstack/tripleo-heat-templates master: Revert "Disable Designate service for scenario 03" https://review.opendev.org/737860 | 12:34 |
chkumar|rover | Tengu: yes might be , if not then I will try to reproduce locally | 12:34 |
Tengu | chkumar|rover: [...], 3.0.0, 3.1.0, 3.1.1, 3.2.0) | 12:34 |
Tengu | that's what it sees here: https://24e738e84b2178e7adaa-7d59354a7b453d58914b817150958c54.ssl.cf2.rackcdn.com/746635/1/check/openstack-tox-pep8/9204071/job-output.txt | 12:34 |
Tengu | and it's using https://mirror.mtl01.inap.opendev.org/ | 12:34 |
openstackgerrit | Emilien Macchi proposed openstack/tripleo-ansible master: tripleo_container_configs module https://review.opendev.org/746214 | 12:35 |
openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates master: Create Container configs with a new module https://review.opendev.org/746215 | 12:35 |
openstackgerrit | Emilien Macchi proposed openstack/python-tripleoclient stable/train: Use user set password values in overcloud export https://review.opendev.org/746160 | 12:36 |
openstackgerrit | Emilien Macchi proposed openstack/tripleo-heat-templates stable/train: Run external_deploy_tasks per step for each role https://review.opendev.org/745965 | 12:36 |
ykarel | glanceclient was hitting yesterday in master as well | 12:36 |
openstackgerrit | Takashi Kajinami proposed openstack/python-tripleoclient master: Allow duplicated IPs for undercloud https://review.opendev.org/746698 | 12:36 |
Tengu | ykarel: ah. meh | 12:37 |
EmilienM | chkumar|rover: hey, are we aware that pretty much all molecule jobs fail on train? | 12:37 |
EmilienM | https://407dc6f5db009aad5c77-3bbb48847b672b5467267c1efe6f2622.ssl.cf2.rackcdn.com/745680/3/check/tripleo-ansible-centos-7-molecule-tripleo-keystone-resources/141261c/job-output.txt | 12:37 |
EmilienM | "Failed building wheel for psutil" | 12:37 |
openstackgerrit | Takashi Kajinami proposed openstack/python-tripleoclient master: Allow usage of duplicated IPs for undercloud config https://review.opendev.org/746698 | 12:38 |
chkumar|rover | EmilienM: not seen this one | 12:38 |
openstackgerrit | Emilien Macchi proposed openstack/tripleo-common stable/train: Relax filter_images_with_labels() and re-add kolla_version https://review.opendev.org/743260 | 12:39 |
Tengu | ykarel: hey, btw - what's the status of podman integration? | 12:39 |
*** jpena|lunch is now known as jpena | 12:40 | |
ykarel | Tengu, i tested it https://review.rdoproject.org/r/#/c/28958/ and result looks good | 12:40 |
ykarel | podman 1.6.4 from extras | 12:40 |
rlandy | bandini: hi ... wrt https://bugzilla.redhat.com/show_bug.cgi?id=1869379 - which additional logs are you looking for? | 12:41 |
openstack | bugzilla.redhat.com bug 1869379 in rabbitmq-server "[OSP 16.2] Containers fail to start for step 2 - mysql_init_bundle error" [Unspecified,New] - Assigned to plemenko | 12:41 |
rlandy | https://sf.hosted.upshift.rdu2.redhat.com/logs/75/208575/27/check/periodic-tripleo-ci-rhel-8-standalone-rhos-16-2/ce39aef/logs/ | 12:41 |
rlandy | over what is available here .... we could hold a node if that helps | 12:42 |
*** shyam89 has quit IRC | 12:42 | |
Tengu | ykarel: great! | 12:42 |
*** ysandeep|brb is now known as ysandeep | 12:44 | |
cloudnull | mornings | 12:44 |
chkumar|rover | EmilienM: Do we want to keep c7 based molecule job for train or move to c8? | 12:45 |
EmilienM | chkumar|rover: no opinion here | 12:45 |
EmilienM | chkumar|rover: i would say yes, but this will be a bit of work, as you know the molecule configs aren't the same on c7 & c8 | 12:50 |
chkumar|rover | EmilienM: fix is coming | 12:50 |
*** Goneri has joined #tripleo | 12:50 | |
EmilienM | if the CI team has time for it, yes go for it. Otherwise i'm sure there are more important items | 12:50 |
openstackgerrit | Chandan Kumar (raukadah) proposed openstack/tripleo-ansible stable/train: Install pytest-xdist<=2.0.0 for c7 jobs https://review.opendev.org/746706 | 12:55 |
chkumar|rover | EmilienM: ^^ | 12:55 |
chkumar|rover | EmilienM: I will add it to coming sprint | 12:55 |
EmilienM | chkumar|rover: thanks! | 12:55 |
EmilienM | chkumar|rover++ | 12:55 |
*** waleedm has quit IRC | 12:56 | |
ykarel | mwhahaha, EmilienM, for that ansible-2.9.12 and scenario004, i ran with no_log: false and get https://logserver.rdoproject.org/96/28996/1/check/rdoinfo-tripleo-master-testing-centos-8-scenario004-standalone/ff69410/logs/undercloud/home/zuul/standalone_deploy.log.txt.gz | 12:56 |
bandini | rlandy: o/ so can we enable debug in CI there, like in here https://review.opendev.org/#/c/728946/ (i.e. ConfigDebug: True in the params) ? | 12:56 |
openstackgerrit | Emilien Macchi proposed openstack/tripleo-ansible stable/train: Create a role for network config https://review.opendev.org/746600 | 12:56 |
ykarel | don't know if it's related to ansible-2.9.12, i also seen a green run with 2.9.12 | 12:56 |
bandini | rlandy: also holding a node would be perfect I think | 12:56 |
*** psachin has quit IRC | 12:57 | |
openstackgerrit | Emilien Macchi proposed openstack/tripleo-ansible stable/train: Use label rather than no_log https://review.opendev.org/745680 | 12:57 |
openstackgerrit | Emilien Macchi proposed openstack/tripleo-ansible stable/train: Switch podman actions to use label https://review.opendev.org/745681 | 12:57 |
EmilienM | chkumar|rover: ^ testing it live | 12:57 |
chkumar|rover | EmilienM: sure | 12:57 |
mwhahaha | ykarel: i don't think it's ansible related, looks podman related | 12:58 |
ykarel | mwhahaha, hmm looks like, have you seen similar issue before in other patches? | 12:58 |
mwhahaha | 2020-08-18 11:46:02.420548 | fa163edf-41e0-afb5-8c72-00000000264d | FATAL | Gather podman infos | standalone | error={"changed": false, "msg": "Unable to gather info for ['194b8245c0f1', 'f74beeadca4e', '9bafa38fa6ee', ...snip...'70c56d5caf87', '1f16f3e22482']: Error: error looking up container \"194b8245c0f1\": no container with name or ID 194b8245c0f1 found: no such container\n"} | 12:58 |
* mwhahaha blames EmilienM | 12:58 | |
openstackgerrit | Kevin Carter (cloudnull) proposed openstack/tripleo-common master: Update kolla sudoers to tripleo sudoers https://review.opendev.org/737854 | 12:59 |
EmilienM | looking | 13:00 |
Tengu | owalsh: hello there! apparently your latest change in the inventory thing (https://review.opendev.org/#/q/I28628100e1300e789114af420bf3ea0dfe9c0d70 ) was backported to stable/train and I suspect it is the cause of https://bugs.launchpad.net/tripleo/+bug/1892008 | 13:01 |
openstack | Launchpad bug 1892008 in tripleo "periodic OVB train centos 7 failing overcloud configuration (config download) Invalid cross-device link" [Critical,Triaged] | 13:01 |
Tengu | at least, this trace from mistral executor log seems to match that change: http://paste.openstack.org/show/796909/ | 13:02 |
EmilienM | mwhahaha, ykarel : looks like podman fun again. | 13:02 |
owalsh | Tengu: hrmm, probably /tmp on a different filesystem | 13:02 |
owalsh | Tengu: should be a simple fix | 13:03 |
EmilienM | mwhahaha, ykarel : I'll take a look | 13:03 |
Tengu | owalsh: care to have a look? | 13:03 |
owalsh | Tengu: sure | 13:03 |
ykarel | EmilienM, ack | 13:03 |
Tengu | owalsh: great, thanks! Added you to the related CIX card then :) | 13:03 |
rlandy | bandini: getting node held - pls send your keys so we can add them | 13:04 |
EmilienM | mwhahaha, ykarel this is super weird | 13:04 |
EmilienM | Aug 18 11:46:01 standalone.localdomain podman[122829]: 2020-08-18 11:46:01.54235855 +0000 UTC m=+0.159055638 container create 194b8245c0f17a44170e5a483c47b558490185d4c8895100caa8a968d3fb117f (image=192.168.24.1:8787/prom/alertmanager:v0.16.2, name=alertmanager) | 13:04 |
rlandy | I'll kick a debug run afterwards | 13:04 |
bandini | rlandy: awesome. thanks. https://github.com/mbaldessari.keys | 13:04 |
EmilienM | somehow a container for alertmanager was started | 13:04 |
EmilienM | and removed right away | 13:05 |
EmilienM | oh that's OS::TripleO::Services::CephGrafana | 13:05 |
ykarel | EmilienM, in an earlier run in same patch alertmanager failed | 13:06 |
ykarel | https://logserver.rdoproject.org/96/28996/1/check/rdoinfo-tripleo-master-testing-centos-8-scenario004-standalone/e13825e/logs/undercloud/home/zuul/standalone_deploy.log.txt.gz | 13:07 |
openstackgerrit | Zhu Sheng Li proposed openstack/tripleo-common stable/train: Ensure image files are readable to apache https://review.opendev.org/746133 | 13:09 |
openstackgerrit | Bogdan Dobrelya (bogdando) proposed openstack/paunch stable/train: Log benign command errors as warns https://review.opendev.org/746713 | 13:12 |
EmilienM | ykarel: yeah i'm investigating | 13:13 |
*** marios|ruck has quit IRC | 13:15 | |
openstackgerrit | Chandan Kumar (raukadah) proposed openstack/tripleo-ansible stable/train: Install pytest-xdist<2.0.0 for c7 jobs https://review.opendev.org/746706 | 13:16 |
chkumar|rover | Sorry little typo | 13:16 |
chkumar|rover | I will recheck the patches | 13:16 |
openstackgerrit | Emilien Macchi proposed openstack/tripleo-ansible stable/train: Use label rather than no_log https://review.opendev.org/745680 | 13:17 |
openstackgerrit | Emilien Macchi proposed openstack/tripleo-ansible stable/train: Create a role for network config https://review.opendev.org/746600 | 13:17 |
openstackgerrit | Emilien Macchi proposed openstack/tripleo-ansible stable/train: Switch podman actions to use label https://review.opendev.org/745681 | 13:17 |
*** tmazur has joined #tripleo | 13:21 | |
*** marios has joined #tripleo | 13:22 | |
*** marios is now known as marios|ruck | 13:22 | |
openstackgerrit | Chandan Kumar (raukadah) proposed openstack/tripleo-ansible stable/train: Install pytest-xdist<=2.0.0 for c7 jobs https://review.opendev.org/746706 | 13:28 |
*** ekultails has joined #tripleo | 13:29 | |
chkumar|rover | EmilienM: let's wait till it finishes | 13:29 |
chkumar|rover | there are other issues there | 13:30 |
chkumar|rover | *zuul gives +1 | 13:30 |
openstackgerrit | Michele Baldessari proposed openstack/tripleo-heat-templates master: Fix up ovn-dbs update tasks https://review.opendev.org/746651 | 13:30 |
*** mburned is now known as mburned_out | 13:33 | |
openstackgerrit | Sagi Shnaidman proposed openstack/tripleo-quickstart-extras master: WIP run new job https://review.opendev.org/746719 | 13:33 |
EmilienM | ykarel: i had to step away, let me keep debugging now | 13:37 |
ykarel | EmilienM, seems like it's issue due to ansible-2.9.12 only | 13:38 |
ykarel | EmilienM, mode: is not specified https://github.com/ceph/ceph-ansible/blob/622d9feae924b216d02fc10a90c5a2089ab98794/roles/ceph-prometheus/tasks/main.yml#L45 | 13:39 |
ykarel | so permission now set to 600 | 13:39 |
ykarel | so ceph-ansible needs to be adopted to support ansible-2.9.12 | 13:40 |
ykarel | fmount, fultonj gfidente ^ | 13:41 |
EmilienM | ykarel: could you file a github issue maybe? | 13:41 |
ykarel | EmilienM, sure can do | 13:42 |
EmilienM | ykarel++ | 13:42 |
openstackgerrit | Chandan Kumar (raukadah) proposed openstack/tripleo-ansible stable/train: Install pytest-xdist<2.0.0 for c7 jobs https://review.opendev.org/746706 | 13:42 |
fultonj | ykarel: are you able to open a ceph-ansible issue on this? | 13:48 |
fultonj | https://github.com/ceph/ceph-ansible/issues | 13:48 |
ykarel | fultonj, on it | 13:48 |
fultonj | thanks ykarel, i think i understand but if you open it i know i won't miss anything in translation | 13:48 |
openstackgerrit | Oliver Walsh proposed openstack/tripleo-common master: Ensure atomic inventory file rename runs on the same mountpoint https://review.opendev.org/746724 | 13:49 |
owalsh | Tengu: FYI ^ | 13:49 |
*** tkajinam has joined #tripleo | 13:50 | |
Tengu | owalsh: ah, great! | 13:50 |
Tengu | owalsh: btw, you might want to have a look at the thread I opened on the ML about the inventory script - would love to get your point of view since you're working on it :) | 13:51 |
owalsh | Tengu: yea, +1 on moving it | 13:51 |
Tengu | owalsh: and about "dropping it and integrating it in the tripleoclient" directly? | 13:52 |
*** lmiccini has quit IRC | 13:52 | |
owalsh | Tengu: yea, eventually | 13:53 |
*** lmiccini has joined #tripleo | 13:53 | |
Tengu | owalsh: would it cause any issue with the Edge usecase? | 13:53 |
owalsh | Tengu: don't see why it would | 13:57 |
*** dmacpher has joined #tripleo | 13:59 | |
*** ntt_ has joined #tripleo | 13:59 | |
Tengu | owalsh: ok. I'll check for that change instead of "just" moving it around. ramishra argument is valid imho, and integrating it withing tripleoclient is probably the best thing to do. | 13:59 |
*** TrevorV has joined #tripleo | 14:01 | |
openstackgerrit | Alex Schultz proposed openstack/tripleo-ansible master: WIP - container image vars https://review.opendev.org/746727 | 14:02 |
openstackgerrit | Harald Jensås proposed openstack/tripleo-heat-templates master: Fix endpoint map tls - zaqar wss port https://review.opendev.org/746671 | 14:03 |
*** TrevorV has quit IRC | 14:03 | |
*** abdysn has quit IRC | 14:04 | |
openstackgerrit | Alex Schultz proposed openstack/tripleo-heat-templates master: WIP - Let ansible provide container images by default https://review.opendev.org/746541 | 14:04 |
*** ntt_ has quit IRC | 14:04 | |
openstackgerrit | Chandan Kumar (raukadah) proposed openstack/tripleo-ansible stable/train: Install pytest-xdist<2.0.0 for c7 jobs https://review.opendev.org/746706 | 14:05 |
ykarel | mwhahaha, EmilienM, fultonj reported https://github.com/ceph/ceph-ansible/issues/5677 | 14:08 |
mwhahaha | oh fallout from the permissions change | 14:08 |
mwhahaha | did they not revert that yet? | 14:08 |
EmilienM | ykarel: perfectly described, thanks | 14:08 |
mwhahaha | https://github.com/ansible/ansible/issues/71200 | 14:09 |
mwhahaha | they reverted it in devel | 14:09 |
mwhahaha | i guess they haven't reverted it in 2.9.x | 14:09 |
mwhahaha | we know 2.9.12 is bad | 14:11 |
fultonj | if ansible is going to issue 2.9.13 with revert does ceph-ansible still need fix? | 14:11 |
*** ykarel_ has joined #tripleo | 14:11 | |
fultonj | or we can just add a 'mode: 666' regardless | 14:11 |
mwhahaha | it would be a good idea to have it but the fix is to not use 2.9.12 | 14:12 |
mwhahaha | though we don't get such luxuries downstream | 14:12 |
fultonj | so inject the number of the beast to make it go away? | 14:12 |
fultonj | ;) | 14:13 |
* ykarel_ got disconnected | 14:13 | |
mwhahaha | ykarel_: 2.9.12 is bad because of ansible issue 71200 | 14:13 |
ykarel_ | mwhahaha, ohhk looking | 14:13 |
*** ykarel has quit IRC | 14:14 | |
*** TrevorV has joined #tripleo | 14:14 | |
ykarel_ | mwhahaha, okk so it's reverted in stable-2.9 too | 14:15 |
ykarel_ | so next release would not see this ceph issue | 14:15 |
ykarel_ | but anyway explicit mode can be set for those tasks | 14:15 |
*** ykarel_ is now known as ykarel | 14:16 | |
mwhahaha | yea | 14:16 |
fultonj | https://github.com/ceph/ceph-ansible/blob/622d9feae924b216d02fc10a90c5a2089ab98794/roles/ceph-prometheus/tasks/main.yml#L45 | 14:16 |
fultonj | and ? | 14:16 |
mwhahaha | it's a good idea to set mode, but we're sure tripleo will get broken a bunch too | 14:16 |
fultonj | ah i see | 14:16 |
mwhahaha | we're missing mode in a lot of places | 14:16 |
EmilienM | we should get a list and fix it | 14:17 |
ykarel | fultonj, other is https://github.com/ceph/ceph-ansible/blob/622d9feae924b216d02fc10a90c5a2089ab98794/roles/ceph-prometheus/tasks/main.yml#L15 but there issue was not seen as owner: is set and container running with same user | 14:17 |
ykarel | so it worked fine | 14:17 |
ykarel | mwhahaha, atleast in CI it was catched only on scenario004 | 14:18 |
ykarel | rest other jobs passed | 14:18 |
mwhahaha | magic | 14:18 |
ykarel | :) | 14:19 |
openstackgerrit | Alex Schultz proposed openstack/python-tripleoclient master: Stop running container image prepare upfront https://review.opendev.org/746733 | 14:20 |
*** psahoo has quit IRC | 14:23 | |
*** ykarel is now known as ykarel|away | 14:25 | |
openstackgerrit | Alex Schultz proposed openstack/python-tripleoclient master: Stop running container image prepare upfront https://review.opendev.org/746733 | 14:28 |
*** hjensas is now known as hjensas|afk | 14:29 | |
openstackgerrit | Chandan Kumar (raukadah) proposed openstack/tripleo-ansible stable/train: Install pytest-xdist<2.0.0 for c7 jobs https://review.opendev.org/746706 | 14:30 |
openstackgerrit | Ronelle Landy proposed openstack/tripleo-ci master: Separate required-projects and roles from base jobs https://review.opendev.org/744562 | 14:34 |
EmilienM | dciabrin: bandini: I'm not sure you saw my comment on https://review.opendev.org/#/c/741231/ | 14:37 |
EmilienM | i'm not a fan of installing rpms with puppet, we have host prep tasks for that now | 14:37 |
*** ykarel|away has quit IRC | 14:37 | |
mwhahaha | oh it also don't work | 14:37 |
*** psahoo has joined #tripleo | 14:38 | |
mwhahaha | unless the tripleo packages var is set | 14:38 |
openstackgerrit | Rafael Folco proposed openstack/tripleo-ci master: WIP: Enable py3 on dib release 7 https://review.opendev.org/733659 | 14:38 |
dciabrin | EmilienM, i didn't think about that | 14:39 |
EmilienM | we should reduce our usage of puppet-tripleo :/ | 14:39 |
bandini | well we do it in so many other places. plus we tested this with IHA FFU (not sure when enablepackageinstall is set to false) | 14:40 |
mwhahaha | ffu probably has it set to true | 14:40 |
mwhahaha | but not certain | 14:40 |
mwhahaha | it's not a cureall for missing packages | 14:40 |
EmilienM | dciabrin, bandini : I would do it in THT tbh, and we have EnableFencing so we can easily enable the task if EnableFencing is true | 14:42 |
EmilienM | the reason is to try to not add more bits in puppet-tripleo | 14:42 |
EmilienM | Ansible is now our major framework | 14:42 |
bandini | we only want that rpm when IHA is a thing not when fencing is configured | 14:43 |
EmilienM | so use EnableInstanceHA in THT? | 14:43 |
*** ratailor has quit IRC | 14:45 | |
EmilienM | bandini: are the backports urgent? If yes, i'm fine landing them | 14:45 |
EmilienM | and we fix later in THT | 14:45 |
bandini | EmilienM: yeah let's do that and I'll start a cleanup in the meantime | 14:46 |
EmilienM | cool | 14:46 |
openstackgerrit | Sagi Shnaidman proposed openstack/tripleo-quickstart master: Fix python name for collection package in reqs https://review.opendev.org/746742 | 14:46 |
EmilienM | bandini: conditionalizing it in ansible shouldn't be that messy (tm), if we use EnableInstanceHA | 14:47 |
bandini | oh the jinx! | 14:48 |
openstackgerrit | Merged openstack/tripleo-ansible stable/ussuri: Create a role for network config https://review.opendev.org/746597 | 14:48 |
*** psahoo has quit IRC | 14:53 | |
openstackgerrit | Chandan Kumar (raukadah) proposed openstack/tripleo-ansible stable/train: Install pytest-xdist<2.0.0 for c7 jobs https://review.opendev.org/746706 | 14:54 |
*** chkumar|rover is now known as raukadah | 14:57 | |
*** gfidente has quit IRC | 14:57 | |
*** gfidente has joined #tripleo | 15:01 | |
openstackgerrit | Michele Baldessari proposed openstack/tripleo-heat-templates master: WIP python3-novaclient installation via THT https://review.opendev.org/746745 | 15:03 |
tkajinam | weshay|ruck, May I ask your thought about this when you have time ? https://review.opendev.org/#/c/741347/ | 15:05 |
*** gfidente has quit IRC | 15:05 | |
openstackgerrit | Merged openstack/puppet-tripleo master: Fix typos, extreamio and xtreamio to xtremio https://review.opendev.org/745993 | 15:19 |
openstackgerrit | Merged openstack/tripleo-quickstart-extras master: Pass proper VIPs on Standalone https://review.opendev.org/725782 | 15:19 |
openstackgerrit | Merged openstack/tripleo-upgrade master: Iterate over all templates which need NW template migration. https://review.opendev.org/746507 | 15:20 |
openstackgerrit | Sagi Shnaidman proposed openstack/tripleo-heat-templates master: DNM test var https://review.opendev.org/746749 | 15:21 |
*** skramaja has quit IRC | 15:22 | |
weshay|ruck | tkajinam, have to think about that | 15:23 |
*** trozet_ has joined #tripleo | 15:25 | |
openstackgerrit | Bogdan Dobrelya (bogdando) proposed openstack/ansible-role-collect-logs master: Use root when listing installed packages https://review.opendev.org/746750 | 15:25 |
tkajinam | weshay|ruck, yeah... I'd appreciate your attention about this because the change might require agreement as a team | 15:26 |
tkajinam | though this is reverting something not yet agreed :-P | 15:26 |
EmilienM | tkajinam: so Sahara isn't being used in TripleO commmunity so far | 15:26 |
EmilienM | we have no bug report, no report of any usage and no downstream customers; we have a long term effort to reduce the # of things we support because we have less people to maintain these things | 15:27 |
EmilienM | (sahara is an example but other unused services are in the same boat) | 15:27 |
EmilienM | i mean i wish we could keep it but this isn't just aligned with our roadmap to remove things that we don't support anymore | 15:28 |
tkajinam | EmilienM, yeah I agreed with you and that's what I thought when I made the deprecation patch initially. | 15:32 |
mwhahaha | we're deprecating mistral/zaqar for same reasons | 15:32 |
* mwhahaha shrugs and moves on | 15:32 | |
EmilienM | we have nothing against sahara, zaqar or mistral | 15:32 |
EmilienM | these are great project, great people | 15:32 |
EmilienM | we just have nobody to maintain it | 15:33 |
*** bandini has quit IRC | 15:34 | |
tkajinam | The point is that actually there are actually some people interested in maintaining Sahara in this case | 15:34 |
openstackgerrit | Chandan Kumar (raukadah) proposed openstack/tripleo-ansible stable/train: Install pytest-xdist<2.0.0 for c7 jobs https://review.opendev.org/746706 | 15:34 |
openstackgerrit | Merged openstack/tripleo-ci master: Remove pprint from test-requirements.txt https://review.opendev.org/746633 | 15:35 |
tkajinam | and I saw some negative feedback from Sahara cores about this directions | 15:35 |
mwhahaha | folks who haven't committed anything to tripleo or puppet-sahara in at least a year | 15:37 |
mwhahaha | not exactly folks who are actually part of this community | 15:37 |
openstackgerrit | Merged openstack/tripleo-ansible master: Wait for node to be unlocked before provide https://review.opendev.org/746234 | 15:38 |
tkajinam | I won't like saying this here, but actually these feedbacks comes from RH guys | 15:39 |
EmilienM | https://github.com/openstack/tripleo-heat-templates/commits/fffdcf0f3059a4f1146ec533f51a65442a105092/deployment/sahara | 15:39 |
mwhahaha | tbh that means nothing | 15:39 |
EmilienM | there is nobody from the Sahara team who maintained the THT part of Sahara in tripleo | 15:39 |
tosky | EmilienM: that's not true | 15:39 |
tosky | again, lies | 15:39 |
EmilienM | well you have one patch in February 2019 | 15:39 |
tosky | I contributed the ffu part long time ago, and there was simply no one asking about changes around | 15:40 |
tosky | because no many changes are really needed | 15:40 |
mwhahaha | being asked for changes isn't participating in a community | 15:40 |
mwhahaha | be proactive not reactive | 15:40 |
tosky | that depends | 15:40 |
tosky | there haven't been need for changes in puppet-sahara lately | 15:40 |
tosky | and neither in tripleo, the structure is stable as it is | 15:41 |
EmilienM | you know it would be extremely simple to host the heat templates to deploy sahara out of tree if you really wanted | 15:41 |
weshay|ruck | tosky, easy brotha | 15:41 |
tosky | why not keep them there, if they ain't broke | 15:41 |
tosky | weshay|ruck: don't easy me | 15:42 |
mwhahaha | extra dependencies and overhead that we carry for a service used by who? | 15:42 |
EmilienM | we have a long term goal to remove unused services/code in tripleo | 15:42 |
tosky | mwhahaha: have you asked all RDO users? | 15:42 |
mwhahaha | we aren't dropping RDO support | 15:42 |
mwhahaha | just tripleo | 15:42 |
*** ramishra has quit IRC | 15:42 | |
mwhahaha | puppet-sahara exists | 15:42 |
tosky | well | 15:42 |
mwhahaha | packaging exists | 15:42 |
mwhahaha | cern can still use it if they want to | 15:42 |
mwhahaha | it's a tripleo decisiion not an RDO one | 15:43 |
EmilienM | yeah and cern doesn't use tripleo | 15:43 |
mwhahaha | tripleo != RDO | 15:43 |
tosky | that's not making things easier in terms of containers, especially now that it's diverging from kolla | 15:43 |
mwhahaha | right so we've decided to actually be opinionated and not support the world | 15:43 |
mwhahaha | because it's been painful for years because of this | 15:43 |
EmilienM | so we can focus on what matters to us the most | 15:43 |
tosky | mwhahaha: I never implied that RDO and tripleo are the same, I know the difference | 15:43 |
EmilienM | which is, taking care of the majority of our users | 15:43 |
mwhahaha | tosky: then don't say we need to ask RDO users | 15:43 |
tosky | how were sahara bits painful to maintain? | 15:44 |
EmilienM | solve scalability issues, make deploy/upgrades faster/easier, etc | 15:44 |
mwhahaha | that has nothing to do with this discussion | 15:44 |
* weshay|ruck ends the discussion... | 15:44 | |
mwhahaha | tosky: we have to mainatin container builds bits, drop depts | 15:44 |
tosky | mwhahaha: how do you measure tripleo users otherwise? | 15:44 |
weshay|ruck | the answer is take it out of tree | 15:44 |
EmilienM | btw we removed openshift support, kubernetes, skydive, soon zaqar and heat | 15:44 |
tosky | mwhahaha: do you have tripleo users (not Red Hat internal, we are talking here in the community)? | 15:44 |
EmilienM | all for the same reason: not used, no need to keep them, low maintenance cost | 15:44 |
tosky | EmilienM: heat as service used internally, or heat as service deployed on the overcloud? | 15:45 |
EmilienM | zaqar & mistral (not heat sorry) | 15:45 |
mwhahaha | heat is supported in the overcloud | 15:45 |
tosky | at least mistral was internally used | 15:45 |
mwhahaha | mistral/zaqar were not | 15:45 |
tosky | and sahara is an overcloud service, not an undercloud one | 15:45 |
tosky | so the removal of mistral and zaqar is not the same story | 15:46 |
mwhahaha | please go find an actual user who wants to keep this | 15:46 |
tosky | weshay|ruck: and for containers building? | 15:46 |
weshay|ruck | tosky, you are welcome to build them :) | 15:46 |
EmilienM | tosky: building containers is way simpler than before | 15:47 |
tosky | weshay|ruck: is there actual documentation to fully maintain a service out of tree with full CI? | 15:47 |
weshay|ruck | tosky, we'll give you access to the same registry if you like | 15:47 |
EmilienM | yes | 15:47 |
weshay|ruck | ipa is a good example | 15:47 |
EmilienM | let me pull it | 15:47 |
tosky | and is it a the point where the various interface involved are kind of guaranteed to be, if not stable, at least not changing every two weeks? | 15:48 |
EmilienM | to the same point where we had to update sahara templates to keep up with the new interface | 15:48 |
weshay|ruck | tosky, you can still run a tripleo ci job against it.. that's not the same as tripleo supporting it | 15:49 |
EmilienM | tosky: https://docs.openstack.org/project-deploy-guide/tripleo-docs/latest/deployment/3rd_party.html#adding-layers-to-existing-containers-using-tripleo-tooling | 15:49 |
tosky | EmilienM: is it stable or not? | 15:51 |
tosky | because I could ask whether the cinder, swift, glance, designate cores did update the respective templates, or whether you did the work | 15:51 |
EmilienM | tosky: yes and we backported it down to train | 15:52 |
tosky | oki | 15:52 |
EmilienM | tosky: yes we have a bunch of folks here maintaining their templates and working on something else | 15:52 |
EmilienM | tosky: the ceph folks, the cinder folks, octavia, neutron, name it | 15:52 |
EmilienM | most of them are tripleo core | 15:52 |
openstackgerrit | Emilien Macchi proposed openstack/tripleo-docs master: Update the doc about 3rd party with recent changes https://review.opendev.org/746752 | 15:53 |
mwhahaha | we have people who participate and fix other things and not necessarily just their own stuff | 15:53 |
tosky | I've seen you are not part of the community if you are not activily involved: I'm always here and I may have payed more attention to the changes, | 15:53 |
tosky | but given the amount of time sahara stuff requires changes (not too many) I still argue it's not you don't know me and you couldn't have nudged me her | 15:54 |
*** lmiccini has quit IRC | 15:54 | |
tosky | that said: you mentioned ipa as an example of containers - the documentation above refers to a modification to cinder container, I guess you can use something more base if you want to implement a full service from scratch? | 15:54 |
mwhahaha | you can fully out of tree a service (3rd parties do) | 15:55 |
mwhahaha | so it's about using the established service stuff | 15:55 |
mwhahaha | you could likely take teh THT code and forklift it into another package | 15:55 |
raukadah | cloudnull: mwhahaha EmilienM https://review.opendev.org/#/c/746706/ molecule job now fixed | 15:55 |
mwhahaha | then it's just a matter of wiring up the heat stuff which hasn't changed in years | 15:56 |
cloudnull | raukadah looking | 15:56 |
weshay|ruck | really? /me looks | 15:56 |
*** marios|ruck has quit IRC | 15:56 | |
weshay|ruck | cloudnull, is too slow | 15:56 |
*** ramishra has joined #tripleo | 15:56 | |
mwhahaha | but part of this is be proactive, not reactive. We shouldn't have to tell you about changes | 15:56 |
cloudnull | haha | 15:56 |
openstackgerrit | Emilien Macchi proposed openstack/tripleo-ansible stable/train: Use label rather than no_log https://review.opendev.org/745680 | 15:56 |
weshay|ruck | cloudnull, those waffles are slowing you down man... you lost a step | 15:56 |
openstackgerrit | Emilien Macchi proposed openstack/tripleo-ansible stable/train: Switch podman actions to use label https://review.opendev.org/745681 | 15:56 |
mwhahaha | to the framework, processes, etc. that's part of being a part of the community | 15:56 |
openstackgerrit | Emilien Macchi proposed openstack/tripleo-ansible stable/train: Create a role for network config https://review.opendev.org/746600 | 15:57 |
EmilienM | raukadah: thanks a lot | 15:57 |
tosky | mwhahaha: it's about a matter of times - when you expect almost no changes going on, it's easy to forget, but it doesn't mean you are not around | 15:57 |
cloudnull | weshay|ruck those waffles were legit ! | 15:57 |
weshay|ruck | :) | 15:57 |
cloudnull | double waffle maker for the win! | 15:57 |
mwhahaha | tosky: idling in the channel != participating. proactive vs reactive | 15:58 |
tosky | mwhahaha: what did I say about "expected amount of changes" | 15:58 |
tosky | mwhahaha: anyway, I understand all the out of tree thing, I was looking for more documentation from a real example, in addition to the documentation | 15:58 |
mwhahaha | we work to keep interfaces stable as much as humanly possible between versions | 15:58 |
mwhahaha | but if you are finding they are breaking then communication (aka participation) is a two way street. | 15:59 |
EmilienM | these things should not be taken personally, it's a fun task to remove stuffs from tripleo but we're making decisions to reduce our cost of maintenance for the long term | 16:00 |
EmilienM | it's not a fun task I meant | 16:00 |
tosky | you know, when a certain project for notifying people about breakages and changes (you can't act you don't know something is changing) I was the one who spent a lot of time working on it internally | 16:00 |
mwhahaha | internally | 16:00 |
mwhahaha | that's the problem there | 16:00 |
tosky | about external changes | 16:00 |
tosky | about external, tripleo, breakages | 16:01 |
bogdando | weshay|ruck EmilienM: WDYT of writing a blog post for an example out of tree component e2e, with 3rd party CI added? | 16:01 |
mwhahaha | visibility is key | 16:01 |
tosky | *external* changes, again | 16:01 |
bogdando | I think that would keep answering the concern for long time | 16:01 |
bogdando | the gift that keeps giving :) | 16:01 |
tosky | bogdando: why a blog post if it can be part of the official documentation? | 16:01 |
EmilienM | bogdando: it was on my list before my PTO | 16:01 |
EmilienM | but the doc should be enough I think | 16:01 |
bogdando | tosky: that would be more of a blog | 16:01 |
EmilienM | either way, i think we already have the content | 16:02 |
*** lucasagomes has quit IRC | 16:02 | |
bogdando | since a lot of devops things and should be a guide, not a tech doc (IMHO) | 16:02 |
bogdando | EmilienM: great! | 16:02 |
openstackgerrit | Jose Luis Franco proposed openstack/tripleo-ansible master: Add role tripleo_stream_enable. https://review.opendev.org/745177 | 16:02 |
EmilienM | we have a whole guide about composable services: https://docs.openstack.org/tripleo-docs/latest/developer/tht_walkthrough/tht_walkthrough.html | 16:02 |
bogdando | s/would/should | 16:03 |
EmilienM | (which could use some love, a lot of things are outdated) | 16:03 |
bogdando | yea | 16:03 |
tosky | EmilienM: but doesn't it describe on how to add things inside the central THT repository? | 16:03 |
openstackgerrit | Jose Luis Franco proposed openstack/tripleo-ansible master: Add role tripleo_stream_enable. https://review.opendev.org/745177 | 16:04 |
EmilienM | I agree there is currently no step-by-step tutorial to how to create a full external repo for a third party service | 16:04 |
EmilienM | and this should be done I guess | 16:04 |
tosky | that would be very appreciated | 16:04 |
*** ysandeep is now known as ysandeep|away | 16:05 | |
EmilienM | tosky: Dell is a great example on that field, https://github.com/dell/tripleo-powerflex | 16:05 |
mwhahaha | https://github.com/dell/tripleo-powerflex/blob/master/overcloud-resource-registry-puppet.j2.yaml makes me concerned however | 16:06 |
weshay|ruck | bogdando, that would be very helpful :) | 16:06 |
weshay|ruck | bogdando, it can be first class CI upstream CI | 16:07 |
EmilienM | mwhahaha: yeah I didn't see it before. I'll talk to JP | 16:07 |
mwhahaha | that shouldn't be required anymore (though we should test that better) | 16:07 |
weshay|ruck | bogdando, just because it's third party to tripleo doesn't mean we can't run an upstream ci job | 16:07 |
*** bogdando has quit IRC | 16:07 | |
EmilienM | mwhahaha: right, i think they could just add the services in the environment files under "resource_registry:" | 16:07 |
mwhahaha | there was a bug with that being incorrectly relative or something, though it should work | 16:08 |
*** jpich has quit IRC | 16:09 | |
*** yprokule has left #tripleo | 16:11 | |
*** jpena is now known as jpena|off | 16:11 | |
openstackgerrit | Merged openstack/tripleo-ansible stable/train: Install pytest-xdist<2.0.0 for c7 jobs https://review.opendev.org/746706 | 16:11 |
openstackgerrit | Sagi Shnaidman proposed openstack/tripleo-quickstart-extras master: WIP run new job https://review.opendev.org/746719 | 16:13 |
EmilienM | mwhahaha: https://github.com/dell/tripleo-powerflex/pull/2 | 16:14 |
EmilienM | tosky: if nobody does it before, I'll start looking at a third party guide when I have spare time | 16:16 |
tosky | EmilienM: thanks | 16:16 |
jeremyfreudberg | EmilienM:thanks, i appreciate that too | 16:16 |
weshay|ruck | how are tripleo-validations back in the tripleo-gate | 16:16 |
*** hjensas|afk is now known as hjensas | 16:17 | |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/tripleo-upgrade stable/ussuri: Iterate over all templates which need NW template migration. https://review.opendev.org/746757 | 16:17 |
openstackgerrit | Jesse Pretorius (odyssey4me) proposed openstack/tripleo-upgrade stable/train: Iterate over all templates which need NW template migration. https://review.opendev.org/746166 | 16:17 |
openstackgerrit | Michele Baldessari proposed openstack/tripleo-heat-templates stable/queens: WIP test concurrent restart pcs fix https://review.opendev.org/746665 | 16:20 |
openstackgerrit | Michele Baldessari proposed openstack/tripleo-heat-templates stable/train: WIP test concurrent restart pcs fix https://review.opendev.org/746662 | 16:21 |
EmilienM | tosky: https://bugs.launchpad.net/tripleo/+bug/1892072 feel free to comment | 16:29 |
openstack | Launchpad bug 1892072 in tripleo "Missing manual for end-to-end third party integration" [Undecided,New] | 16:29 |
EmilienM | i need to gather volunteers to write it | 16:30 |
tosky | EmilienM: thanks, I'm going to subscribe to it for a start | 16:30 |
openstackgerrit | Merged openstack/tripleo-ci master: Add container excludes for rhos-16.2 builds https://review.opendev.org/744166 | 16:31 |
openstackgerrit | Rafael Folco proposed openstack/tripleo-ci master: WIP: Enable py3 on dib release 7 https://review.opendev.org/733659 | 16:46 |
*** cfontain has joined #tripleo | 16:47 | |
*** cfontain has quit IRC | 17:02 | |
*** cfontain has joined #tripleo | 17:02 | |
openstackgerrit | Alex Schultz proposed openstack/tripleo-heat-templates master: WIP - Let ansible provide container images by default https://review.opendev.org/746541 | 17:03 |
openstackgerrit | Ronelle Landy proposed openstack/tripleo-quickstart-extras master: Add else clause to tripleo_deploy_control_virtual_ip https://review.opendev.org/746772 | 17:08 |
rlandy | ade_lee: ^^ hey - can you review this | 17:11 |
rlandy | the logic here is interesting .... | 17:11 |
rlandy | we added standalone_control_virtual_ip for the ipa tests | 17:12 |
rlandy | standalone_ha_control_virtual_ip could be picked | 17:12 |
rlandy | thinking about switching the logic there | 17:12 |
rlandy | the way it is now, break releases train and earlier | 17:12 |
rlandy | weshay|ruck: ^^ fyi | 17:13 |
openstackgerrit | Alex Schultz proposed openstack/tripleo-heat-templates master: WIP - Let ansible provide container images by default https://review.opendev.org/746541 | 17:23 |
ade_lee | rlandy, looking | 17:26 |
rlandy | switching logic there | 17:26 |
ade_lee | hjensas, looks like good set of patches on the dns-ip-resolving issue | 17:26 |
*** tmazur has quit IRC | 17:28 | |
openstackgerrit | Ronelle Landy proposed openstack/tripleo-quickstart-extras master: Switch logic on when to use control_virtual_ip https://review.opendev.org/746772 | 17:28 |
rlandy | ade_lee: ^^ a second option | 17:28 |
openstackgerrit | Alex Schultz proposed openstack/tripleo-heat-templates master: WIP - Let ansible provide container images by default https://review.opendev.org/746541 | 17:36 |
openstackgerrit | Sagi Shnaidman proposed openstack/tripleo-quickstart-extras master: WIP run new job https://review.opendev.org/746719 | 17:37 |
openstackgerrit | Ronelle Landy proposed openstack/tripleo-quickstart-extras master: Switch logic on when to use control_virtual_ip https://review.opendev.org/746772 | 17:38 |
openstackgerrit | Sagi Shnaidman proposed openstack/tripleo-quickstart-extras master: WIP run new job https://review.opendev.org/746719 | 17:40 |
*** ekultails has quit IRC | 18:10 | |
*** ekultails has joined #tripleo | 18:12 | |
openstackgerrit | Rafael Folco proposed openstack/tripleo-ci master: WIP: Enable py3 on dib release 7 https://review.opendev.org/733659 | 18:16 |
openstackgerrit | Merged openstack/python-tripleoclient master: Limit ip_version when resolving public_host and admin_host https://review.opendev.org/746656 | 18:18 |
openstackgerrit | Sagi Shnaidman proposed openstack/tripleo-quickstart-extras master: WIP run new job https://review.opendev.org/746719 | 18:21 |
openstackgerrit | Merged openstack/paunch stable/ussuri: [USSURY-Only] Log benign command errors as warns https://review.opendev.org/746677 | 18:23 |
openstackgerrit | wes hayutin proposed openstack/tripleo-ci master: DNM, debug container build role path https://review.opendev.org/746624 | 18:38 |
openstackgerrit | wes hayutin proposed openstack/tripleo-ci master: DNM, wes Separate required-projects and roles from base jobs https://review.opendev.org/746695 | 18:38 |
openstackgerrit | Emilien Macchi proposed openstack/python-tripleoclient stable/ussuri: Limit ip_version when resolving public_host and admin_host https://review.opendev.org/746785 | 18:40 |
openstackgerrit | Emilien Macchi proposed openstack/python-tripleoclient stable/train: Limit ip_version when resolving public_host and admin_host https://review.opendev.org/746786 | 18:40 |
*** saneax has quit IRC | 18:57 | |
openstackgerrit | wes hayutin proposed openstack/tripleo-ci master: DNM, debug container build role path https://review.opendev.org/746624 | 19:08 |
openstackgerrit | wes hayutin proposed openstack/tripleo-ci master: update the container build ansible role path https://review.opendev.org/746624 | 19:08 |
*** dprince has joined #tripleo | 19:11 | |
openstackgerrit | wes hayutin proposed openstack/tripleo-ci master: update the container build ansible role path https://review.opendev.org/746624 | 19:15 |
openstackgerrit | wes hayutin proposed openstack/tripleo-ci master: update the container build ansible role path https://review.opendev.org/746624 | 19:24 |
ade_lee | rlandy, sorry - got distracted -- did you post two different reviews? | 19:31 |
openstackgerrit | wes hayutin proposed openstack/tripleo-ci master: update the container build ansible role path https://review.opendev.org/746624 | 19:32 |
openstackgerrit | wes hayutin proposed openstack/tripleo-ci master: Separate required-projects and roles from base jobs https://review.opendev.org/744562 | 19:32 |
ade_lee | rlandy, ah - different patchset -- much more readable | 19:33 |
*** dprince has quit IRC | 19:36 | |
ade_lee | rlandy, what is the value of tripleo_deploy_ha_control_plane ? | 19:36 |
rlandy | ade_lee: nothing in train or earlier | 19:37 |
rlandy | defaulted in ussuri onwards | 19:37 |
ade_lee | rlandy, to what -- a value or true/false? | 19:37 |
rlandy | ade_lee: https://github.com/openstack/tripleo-quickstart-extras/commit/63c1365d94b2246a0be8580026aff001e6d8fce8 | 19:37 |
rlandy | standalone_ha_control_virtual_ip: 192.168.24.3 | 19:38 |
rlandy | tripleo_deploy_ha_control_plane: "{{ release not in ['newton', 'ocata', 'pike', 'queens', 'rocky', 'stein', 'train'] }}" | 19:38 |
ade_lee | rlandy, so true/false | 19:38 |
rlandy | undefined | 19:38 |
rlandy | yeah | 19:39 |
ade_lee | rlandy, so you've replaced a true/false check with a defined/undefined check? | 19:39 |
rlandy | actually you're right | 19:39 |
rlandy | should true/false | 19:40 |
ade_lee | ok cool | 19:40 |
rlandy | 'let me fix that | 19:40 |
rlandy | ade_lee: this is no good | 19:41 |
rlandy | means that releases after train will always get ha | 19:42 |
openstackgerrit | Alex Schultz proposed openstack/tripleo-heat-templates master: WIP - Let ansible provide container images by default https://review.opendev.org/746541 | 19:42 |
openstackgerrit | Ronelle Landy proposed openstack/tripleo-quickstart-extras master: Switch logic on when to use control_virtual_ip https://review.opendev.org/746772 | 19:52 |
rlandy | ade_lee: ^^ how about that? | 19:53 |
ade_lee | rlandy, ack -- meeting right now - but will look shortly | 19:55 |
*** rlandy is now known as rlandy|mtg | 19:59 | |
*** zbitter has joined #tripleo | 20:04 | |
*** zaneb has quit IRC | 20:04 | |
openstackgerrit | Merged openstack/tripleo-validations master: Correct role name in generated playbook https://review.opendev.org/744886 | 20:04 |
openstackgerrit | Merged openstack/tripleo-upgrade stable/ussuri: Iterate over all templates which need NW template migration. https://review.opendev.org/746757 | 20:04 |
mwhahaha | cloudnull: do you know how to set facts as a result of a module execution? | 20:06 |
mwhahaha | aka do magic | 20:06 |
cloudnull | like to hand off to another job? | 20:07 |
mwhahaha | run module -> return set's facts | 20:07 |
mwhahaha | result['ansible_facts'] doesn't seem to set the facts | 20:07 |
cloudnull | oh, im stupid, i read that as molecule (had molecule on the brain) | 20:08 |
cloudnull | I think you have to return some json bits, and then run a set fact from that that return using a register. | 20:08 |
* cloudnull idk if you can set fact in module | 20:08 | |
mwhahaha | the strategy is where it's actually handled | 20:09 |
mwhahaha | it's spaghetti | 20:09 |
mwhahaha | might be because it's not set facts everywhere because it's delegated | 20:09 |
cloudnull | https://media1.giphy.com/media/etrzgFf1vabpc9F8O2/giphy.gif | 20:10 |
mwhahaha | https://github.com/ansible/ansible/blob/devel/lib/ansible/plugins/strategy/__init__.py#L668-L695 | 20:10 |
mwhahaha | i need teh var to go global :/ | 20:11 |
mwhahaha | conext: container-image-prepare runs on undercloud delgated to localhost | 20:11 |
mwhahaha | but the vars need to be used by all the hosts | 20:11 |
mwhahaha | so i think using ansible_facts in the return sets it on the undercloud only | 20:11 |
*** hjensas has quit IRC | 20:11 | |
cloudnull | https://github.com/ansible/ansible/blob/devel/lib/ansible/plugins/strategy/__init__.py#L693 | 20:12 |
cloudnull | yuk , it looks like you can set a single fact you have reset all the facts? | 20:12 |
cloudnull | s/can/can't/ | 20:13 |
mwhahaha | maybe | 20:13 |
cloudnull | im not quite sure what set_nonpersistent_facts does | 20:14 |
mwhahaha | feels like i need to do some awful var file/loading things | 20:14 |
mwhahaha | probably related to fact caching | 20:14 |
cloudnull | however, if it works like it would in my mind maybe you can set a single fact their thats only available at runtime? | 20:14 |
cloudnull | **there. | 20:15 |
mwhahaha | meh | 20:15 |
mwhahaha | i don't think i can do what i want because the external deploy tasks are invoked against hosts: Undercloud and the data needs to be used in a different play that's hosts: Overcloud | 20:16 |
cloudnull | we could set it as a local fact ? | 20:16 |
cloudnull | and change the lookup to use local facts? | 20:17 |
mwhahaha | it's the wrong host tho | 20:17 |
mwhahaha | ehh i don't want to use lookup | 20:17 |
cloudnull | ++ that could get ugly quick | 20:17 |
mwhahaha | kinda makes me thing switching container image prepare to an ansible module isn't so grand | 20:18 |
cloudnull | can we set the facts as group_vars ? | 20:18 |
cloudnull | or are they host specific ? | 20:19 |
mwhahaha | i mean we can do group vars | 20:19 |
mwhahaha | but i don't know how to set that from an ansible module | 20:19 |
cloudnull | ++ I was just thinking the same ... | 20:19 |
cloudnull | even if we drop them as a file it would take effect until ansible was stopped and restarted. | 20:20 |
mwhahaha | we could slurp/load_vars | 20:20 |
mwhahaha | but meh | 20:20 |
cloudnull | we could write it out as a json file, slurp file, and set_fact result | from_json ? | 20:21 |
cloudnull | :\ | 20:21 |
mwhahaha | the problem is the set fact across plays | 20:21 |
openstackgerrit | Rafael Folco proposed openstack/tripleo-ci master: WIP: Enable py3 on dib release 7 https://review.opendev.org/733659 | 20:21 |
cloudnull | yup | 20:21 |
cloudnull | local facts would be available across plays | 20:22 |
cloudnull | and are dynamic-ish | 20:22 |
cloudnull | but the lookup is miserable | 20:23 |
cloudnull | https://docs.ansible.com/ansible/latest/user_guide/playbooks_variables.html#local-facts-facts-d | 20:24 |
mwhahaha | set_facts aren't available cross plays are they? | 20:26 |
* mwhahaha sighs at scope | 20:26 | |
cloudnull | no. | 20:27 |
cloudnull | unless caching is availble | 20:27 |
cloudnull | then the cached fact is set until the timeout is met | 20:27 |
*** rlandy|mtg is now known as rlandy | 20:34 | |
mwhahaha | meh this effort seems like it might make things more complicated | 20:34 |
* mwhahaha switches to do something else | 20:34 | |
*** Tengu has quit IRC | 20:36 | |
*** ekultails has quit IRC | 20:37 | |
*** Tengu has joined #tripleo | 20:38 | |
*** ekultails has joined #tripleo | 20:38 | |
*** jfrancoa has quit IRC | 20:48 | |
*** ramishra has quit IRC | 20:56 | |
openstackgerrit | Luke Short proposed openstack/tripleo-heat-templates master: [WIP] Expose the Apache WSGI timeout https://review.opendev.org/738925 | 21:01 |
*** TrevorV has quit IRC | 21:02 | |
EmilienM | can someone review https://review.opendev.org/#/c/746589/ please? | 21:04 |
ekultails | EmilienM: I don't have almighty +workflow power but it LGTM +1 | 21:07 |
*** hjensas has joined #tripleo | 21:07 | |
EmilienM | thanks | 21:10 |
*** raildo has quit IRC | 21:14 | |
EmilienM | mrunge: FYI i filed https://bugs.launchpad.net/tripleo/+bug/1892097 | 21:14 |
openstack | Launchpad bug 1892097 in tripleo "collectd container fails to start: Error: Parsing the config file failed" [Medium,Triaged] | 21:14 |
*** rcernin has joined #tripleo | 21:19 | |
openstackgerrit | Kevin Carter (cloudnull) proposed openstack/python-tripleoclient stable/ussuri: overcloud_cell: fix location of config-download https://review.opendev.org/746801 | 21:20 |
openstackgerrit | Kevin Carter (cloudnull) proposed openstack/python-tripleoclient stable/train: Add tripleo container image hotfix command https://review.opendev.org/741237 | 21:23 |
openstackgerrit | Merged openstack/tripleo-docs master: Introduce a manual about TripleO core reviewers https://review.opendev.org/743999 | 21:24 |
*** xek has quit IRC | 21:26 | |
openstackgerrit | Merged openstack/tripleo-docs master: Update the doc about 3rd party with recent changes https://review.opendev.org/746752 | 21:30 |
*** rcernin has quit IRC | 21:35 | |
*** rcernin has joined #tripleo | 22:09 | |
*** rcernin has quit IRC | 22:09 | |
*** tosky has quit IRC | 22:18 | |
*** rlandy is now known as rlandy|bbl | 22:22 | |
*** rcernin has joined #tripleo | 22:33 | |
*** sri_ has quit IRC | 22:42 | |
*** csatari has quit IRC | 22:43 | |
*** rpioso has quit IRC | 22:43 | |
*** sri_ has joined #tripleo | 22:51 | |
*** rpioso has joined #tripleo | 22:54 | |
*** csatari has joined #tripleo | 22:55 | |
*** marbindrakon has joined #tripleo | 23:21 | |
*** slaweq has quit IRC | 23:33 | |
*** hjensas has quit IRC | 23:33 | |
openstackgerrit | Merged openstack/tripleo-ansible master: Create tripleo_container_config_scripts module https://review.opendev.org/746218 | 23:37 |
openstackgerrit | Takashi Kajinami proposed openstack/puppet-tripleo master: Add unit test cases for tripleo::profile::base::swift https://review.opendev.org/746819 | 23:51 |
openstackgerrit | Takashi Kajinami proposed openstack/puppet-tripleo master: Add unit test cases for tripleo::profile::base::swift https://review.opendev.org/746819 | 23:52 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!