| opendevreview | Merged openstack/tripleo-ci master: Remove EPEL from test images https://review.opendev.org/c/openstack/tripleo-ci/+/797528 | 00:05 |
|---|---|---|
| opendevreview | Brent Eagles proposed openstack/tripleo-heat-templates master: Enable octavia driver agent puppet https://review.opendev.org/c/openstack/tripleo-heat-templates/+/793422 | 00:10 |
| opendevreview | Ian Wienand proposed openstack/diskimage-builder master: cache-url : turn down verbose curl https://review.opendev.org/c/openstack/diskimage-builder/+/797791 | 00:11 |
| *** ysandeep|out is now known as ysandeep | 00:32 | |
| opendevreview | John Fulton proposed openstack/tripleo-ansible master: Add deployed_ceph feature https://review.opendev.org/c/openstack/tripleo-ansible/+/797538 | 00:59 |
| opendevreview | Merged openstack/tripleo-validations stable/victoria: Trigger molecule tests when modifying CI scripts https://review.opendev.org/c/openstack/tripleo-validations/+/795944 | 01:30 |
| opendevreview | Steve Baker proposed openstack/tripleo-ci master: Handle overcloud-hardened-uefi-full in oooci-build-images https://review.opendev.org/c/openstack/tripleo-ci/+/797360 | 01:31 |
| opendevreview | Ian Wienand proposed openstack/diskimage-builder master: Revert "Install rng-tools in Red Hat family distro images" https://review.opendev.org/c/openstack/diskimage-builder/+/797794 | 01:32 |
| opendevreview | Merged openstack/tripleo-heat-templates stable/train: Limit collectds memory usage https://review.opendev.org/c/openstack/tripleo-heat-templates/+/797704 | 01:52 |
| *** rlandy|bbl is now known as rlandy | 02:07 | |
| *** ykarel|away is now known as ykarel | 04:31 | |
| *** marios is now known as marios|ruck | 05:01 | |
| opendevreview | Gael Chamoulaud proposed openstack/validations-common master: [check_disk_space] Add role documentation in sphinx https://review.opendev.org/c/openstack/validations-common/+/797634 | 05:07 |
| opendevreview | Gael Chamoulaud proposed openstack/validations-common master: [check_latest_packages_version] Add role documentation in sphinx https://review.opendev.org/c/openstack/validations-common/+/797635 | 05:07 |
| opendevreview | Gael Chamoulaud proposed openstack/validations-common master: [check_ram] Remove README.md file https://review.opendev.org/c/openstack/validations-common/+/797666 | 05:07 |
| opendevreview | Gael Chamoulaud proposed openstack/validations-common master: [check_selinux_mode] Add role documentation in sphinx https://review.opendev.org/c/openstack/validations-common/+/797667 | 05:07 |
| opendevreview | Gael Chamoulaud proposed openstack/validations-common master: [dns] Add role documentation in sphinx https://review.opendev.org/c/openstack/validations-common/+/797801 | 05:07 |
| opendevreview | Gael Chamoulaud proposed openstack/validations-common master: [no_op] Add role documentation in sphinx https://review.opendev.org/c/openstack/validations-common/+/797802 | 05:07 |
| *** bhagyashris_ is now known as bhagyashris | 05:13 | |
| dpawlik | weshay|ruck: checking | 06:08 |
| shyamb | Hi ykarel: For tripleo train centos cloud, after a successful deployment, If I run deploy command again, it fails with rabbitmq pacemaker resource going down. | 06:09 |
| shyamb | http://paste.openstack.org/show/806903/ | 06:09 |
| shyamb | tripleo train centos8. | 06:09 |
| shyamb | First time, cloud is deployed with tripleo quickstart. | 06:10 |
| *** pojadhav- is now known as pojadhav | 06:16 | |
| opendevreview | Francesco Pantano proposed openstack/tripleo-quickstart master: Remove from master/wallaby the ceph-ansible references https://review.opendev.org/c/openstack/tripleo-quickstart/+/797805 | 06:31 |
| shyamb | rabbitmq bundle has not restarted from 2 days and pacemaker rabbitmq resource is in 'STOPPED' state. | 06:36 |
| chandankumar | marios|ruck: hello, new tripleo-operator-ansible release tarballs are available https://tarballs.opendev.org/openstack/tripleo-operator-ansible/ | 06:58 |
| chandankumar | thanks for all the help. | 06:59 |
| dpawlik | weshay|ruck: finally I download the logs. I don't see vmx flag in cpu info 'logs/undercloud/var/log/extra/cpuinfo.txt' | 07:01 |
| dpawlik | weshay|ruck: also kvm is not set in lsmod | 07:02 |
| marios|ruck | nice one chandankumar good job getting that sorted out | 07:02 |
| ykarel | shyamb, before redeployment, deployment was working fine? | 07:13 |
| shyamb | ykarel: Yes, deployment was successful. | 07:14 |
| ykarel | and rabbitmq was working fine? | 07:14 |
| ykarel | wrt. rabbitmq bundle has not restarted from 2 days, redeployment ran in these 2 days, or it ran before 2 days | 07:16 |
| opendevreview | Jiri Podivin proposed openstack/validations-common master: DNM potentialfix https://review.opendev.org/c/openstack/validations-common/+/797603 | 07:16 |
| anbanerj|rover | stevebaker, thanks for the patch. I abandoned https://review.opendev.org/c/openstack/tripleo-ci/+/797787 | 07:18 |
| shyamb | ykarel: I ran redeployment 2-3 times in these two days. | 07:20 |
| shyamb | ykarel: rabbitmq_wait_bundle is exiting with 1: http://paste.openstack.org/show/806904/ | 07:21 |
| *** rpittau|afk is now known as rpittau | 07:21 | |
| opendevreview | Jiri Podivin proposed openstack/validations-common master: DNM potentialfix https://review.opendev.org/c/openstack/validations-common/+/797603 | 07:21 |
| ykarel | shyamb, yes as per logs rabbitmq looks down | 07:26 |
| ykarel | /var/log/containers/stdouts/rabbitmq_wait_bundle.log should have logs | 07:27 |
| ykarel | but no idea if just redeployment could cause it | 07:27 |
| ykarel | bandini may have more idea over this | 07:27 |
| shyamb | ykarel: Yes, I looked into that log file. | 07:32 |
| shyamb | But not getting clear idea. | 07:32 |
| shyamb | I have rebooted the controller node and trying redeployment. | 07:32 |
| shyamb | Let's see. | 07:32 |
| shyamb | ykarel: If I get it reproduced again, will tag bandini | 07:36 |
| ykarel | ack | 07:36 |
| *** jpena|off is now known as jpena | 07:36 | |
| shyamb | ykarel: Just redeployment is restarting haproxy services | 07:37 |
| shyamb | I don't think it's required, but maybe a bug. | 07:38 |
| ykarel | me unsure about it if restart is expected or not in redeployment | 07:39 |
| bandini | an haproxy restart on redeployment without any config changes is a bug fwiw | 07:39 |
| shyamb | ykarel: bandini: yes | 07:39 |
| shyamb | I also think so. | 07:40 |
| opendevreview | Jiri Podivin proposed openstack/tripleo-validations stable/train: DNM - test for potential job fix/info gathering. https://review.opendev.org/c/openstack/tripleo-validations/+/797442 | 07:51 |
| opendevreview | Jiri Podivin proposed openstack/python-tripleoclient master: Preventing infinite loop behavior in export https://review.opendev.org/c/openstack/python-tripleoclient/+/797624 | 08:02 |
| opendevreview | Bogdan Dobrelya proposed openstack/tripleo-common stable/wallaby: Make sure Heat Ephemeral has images prepared https://review.opendev.org/c/openstack/tripleo-common/+/797764 | 08:14 |
| opendevreview | Bogdan Dobrelya proposed openstack/tripleo-heat-templates stable/wallaby: Add Ephemeral Heat service https://review.opendev.org/c/openstack/tripleo-heat-templates/+/797765 | 08:14 |
| *** amoralej|off is now known as amoralej | 08:20 | |
| *** ykarel is now known as ykarel|lunch | 08:34 | |
| opendevreview | Jiri Podivin proposed openstack/python-tripleoclient master: Moving the 'export_data' dict into constants https://review.opendev.org/c/openstack/python-tripleoclient/+/797816 | 08:35 |
| *** sshnaidm|afk is now known as sshnaidm | 08:45 | |
| opendevreview | Jiri Podivin proposed openstack/python-tripleoclient master: Moving the 'export_data' dict into constants https://review.opendev.org/c/openstack/python-tripleoclient/+/797816 | 08:47 |
| opendevreview | Shnaidman Sagi (Sergey) proposed openstack/tripleo-quickstart-extras master: DNM try ansible 2.11 https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/797823 | 09:04 |
| opendevreview | Jose Luis Franco proposed openstack/tripleo-upgrade stable/train: [Train-Only] Move the rename-nics after upgrade prepare. https://review.opendev.org/c/openstack/tripleo-upgrade/+/797824 | 09:05 |
| opendevreview | Carlos Gonçalves proposed openstack/tripleo-ansible master: WIP DNM/DNR OVN BGP Agent support https://review.opendev.org/c/openstack/tripleo-ansible/+/785664 | 09:06 |
| shyamb | bandini: Hi | 09:07 |
| shyamb | rabbitmq issue is reproduced. | 09:07 |
| shyamb | With no change in deploy command, tried redeployment. rabbitmq pacemaker service is in STOPPED state | 09:08 |
| shyamb | but rabbitmq-bundle-podman-0 container has not restarted. | 09:09 |
| shyamb | It's up and running | 09:09 |
| shyamb | http://paste.openstack.org/show/806904/ | 09:10 |
| shyamb | Should I log a bug? | 09:12 |
| opendevreview | Pooja Jadhav proposed openstack/tripleo-ci master: Adds master upgrades/updates jobs to periodic https://review.opendev.org/c/openstack/tripleo-ci/+/797858 | 09:23 |
| shyamb | bandini: Here is the bug: https://bugs.launchpad.net/tripleo/+bug/1933446 | 09:24 |
| opendevreview | mbu proposed openstack/tripleo-validations stable/train: Override log directory for CI train jobs https://review.opendev.org/c/openstack/tripleo-validations/+/797859 | 09:33 |
| opendevreview | mbu proposed openstack/validations-common master: Expose VALIDATIONS_LOG_DIR to the run command https://review.opendev.org/c/openstack/validations-common/+/797860 | 09:34 |
| opendevreview | Pooja Jadhav proposed openstack/tripleo-ci master: Adds wallaby upgrades/updates jobs to periodic https://review.opendev.org/c/openstack/tripleo-ci/+/796771 | 09:43 |
| opendevreview | Pooja Jadhav proposed openstack/tripleo-ci master: Adds master upgrades/updates jobs to periodic https://review.opendev.org/c/openstack/tripleo-ci/+/797858 | 09:43 |
| *** marios is now known as marios|ruck | 09:48 | |
| *** bhagyashris_ is now known as bhagyashris | 09:48 | |
| opendevreview | Shnaidman Sagi (Sergey) proposed openstack/tripleo-quickstart-extras master: DNM try ansible 2.11 https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/797823 | 09:48 |
| opendevreview | Jiri Podivin proposed openstack/python-tripleoclient master: Overriding the log path argument for tripleoclient cli https://review.opendev.org/c/openstack/python-tripleoclient/+/794577 | 09:51 |
| opendevreview | Jiri Podivin proposed openstack/python-tripleoclient master: Overriding the log path argument for tripleoclient cli https://review.opendev.org/c/openstack/python-tripleoclient/+/794577 | 09:53 |
| opendevreview | Matthias Runge proposed openstack/tripleo-puppet-elements stable/wallaby: Remove pankoclient https://review.opendev.org/c/openstack/tripleo-puppet-elements/+/797766 | 09:59 |
| opendevreview | Gael Chamoulaud proposed openstack/validations-common master: [ntp] Add role documentation in sphinx https://review.opendev.org/c/openstack/validations-common/+/797861 | 09:59 |
| opendevreview | Gael Chamoulaud proposed openstack/validations-common master: [service_status] Add role documentation in sphinx https://review.opendev.org/c/openstack/validations-common/+/797862 | 09:59 |
| opendevreview | Gael Chamoulaud proposed openstack/validations-common master: [validate_selinux] Add role documentation in sphinx https://review.opendev.org/c/openstack/validations-common/+/797863 | 09:59 |
| opendevreview | Matthias Runge proposed openstack/tripleo-puppet-elements stable/wallaby: Remove leftover of pankoclient https://review.opendev.org/c/openstack/tripleo-puppet-elements/+/797767 | 10:00 |
| opendevreview | Pooja Jadhav proposed openstack/tripleo-ci master: Move content providers to content-provider template and fix branches https://review.opendev.org/c/openstack/tripleo-ci/+/793144 | 10:07 |
| opendevreview | Pooja Jadhav proposed openstack/tripleo-ci master: Adds wallaby undercloud-upgrade, minor update jobs and template https://review.opendev.org/c/openstack/tripleo-ci/+/793120 | 10:07 |
| opendevreview | Pooja Jadhav proposed openstack/tripleo-ci master: Adds wallaby upgrades/updates jobs to periodic https://review.opendev.org/c/openstack/tripleo-ci/+/796771 | 10:07 |
| opendevreview | Pooja Jadhav proposed openstack/tripleo-ci master: Adds master upgrades/updates jobs to periodic https://review.opendev.org/c/openstack/tripleo-ci/+/797865 | 10:07 |
| opendevreview | Gael Chamoulaud proposed openstack/validations-common master: [xfs_check_ftype] Add role documentation in sphinx https://review.opendev.org/c/openstack/validations-common/+/797866 | 10:08 |
| opendevreview | Francesco Pantano proposed openstack/python-tripleoclient master: Fail the overcloud deploy is ceph-ansible.yaml is included https://review.opendev.org/c/openstack/python-tripleoclient/+/797867 | 10:10 |
| *** ykarel|lunch is now known as ykarel | 10:21 | |
| opendevreview | Sorin Sbârnea proposed openstack/tripleo-ci master: Enable check-requirements https://review.opendev.org/c/openstack/tripleo-ci/+/797871 | 10:35 |
| opendevreview | Sorin Sbârnea proposed openstack/tripleo-ci master: Enable check-requirements https://review.opendev.org/c/openstack/tripleo-ci/+/797871 | 10:37 |
| opendevreview | Christian Schwede proposed openstack/tripleo-heat-templates stable/train: Do not run Swift rsync container in single replica mode https://review.opendev.org/c/openstack/tripleo-heat-templates/+/797768 | 10:53 |
| opendevreview | Jad Haj Yahya proposed openstack/tripleo-operator-ansible master: Fixing tripleo_validator_show_parameter role https://review.opendev.org/c/openstack/tripleo-operator-ansible/+/797873 | 10:54 |
| opendevreview | Francesco Pantano proposed openstack/tripleo-heat-templates master: DNM - Test Pending Ceph Pacific daemons in CI https://review.opendev.org/c/openstack/tripleo-heat-templates/+/778915 | 10:54 |
| opendevreview | Francesco Pantano proposed openstack/python-tripleoclient master: Fail the overcloud deploy is ceph-ansible.yaml is included https://review.opendev.org/c/openstack/python-tripleoclient/+/797867 | 10:57 |
| opendevreview | Marios Andreou proposed openstack/tripleo-ci master: Revert "Set T->U undercloud upgrade to non-voting" https://review.opendev.org/c/openstack/tripleo-ci/+/797769 | 11:03 |
| opendevreview | Jesse Pretorius proposed openstack/tripleo-heat-templates stable/wallaby: Do not run Swift rsync container in single replica mode https://review.opendev.org/c/openstack/tripleo-heat-templates/+/797770 | 11:03 |
| opendevreview | Jesse Pretorius proposed openstack/tripleo-heat-templates stable/victoria: Do not run Swift rsync container in single replica mode https://review.opendev.org/c/openstack/tripleo-heat-templates/+/797771 | 11:03 |
| opendevreview | Jesse Pretorius proposed openstack/tripleo-heat-templates stable/ussuri: Do not run Swift rsync container in single replica mode https://review.opendev.org/c/openstack/tripleo-heat-templates/+/797772 | 11:04 |
| opendevreview | Sorin Sbârnea proposed openstack/tripleo-ci master: POC: Enable check-requirements https://review.opendev.org/c/openstack/tripleo-ci/+/797871 | 11:07 |
| opendevreview | Pooja Jadhav proposed openstack/tripleo-ci master: Adds master upgrades/updates jobs to periodic https://review.opendev.org/c/openstack/tripleo-ci/+/797865 | 11:18 |
| opendevreview | Jiri Podivin proposed openstack/validations-common master: DNM potentialfix https://review.opendev.org/c/openstack/validations-common/+/797603 | 11:31 |
| *** dviroel|out is now known as dviroel | 11:32 | |
| opendevreview | Pooja Jadhav proposed openstack/tripleo-ci master: Adds master upgrades/updates jobs to periodic https://review.opendev.org/c/openstack/tripleo-ci/+/797865 | 11:33 |
| *** chem is now known as Guest421 | 11:38 | |
| *** jpena is now known as jpena|lunch | 11:39 | |
| opendevreview | Jiri Podivin proposed openstack/python-tripleoclient master: Preventing infinite loop behavior in export https://review.opendev.org/c/openstack/python-tripleoclient/+/797624 | 11:48 |
| opendevreview | Sorin Sbârnea proposed openstack/tripleo-ci master: POC: Enable check-requirements https://review.opendev.org/c/openstack/tripleo-ci/+/797871 | 11:49 |
| opendevreview | Gael Chamoulaud proposed openstack/tripleo-validations master: Add zuul.d/molecule.yaml file back in the yamllint ignore list https://review.opendev.org/c/openstack/tripleo-validations/+/797884 | 11:54 |
| opendevreview | Gael Chamoulaud proposed openstack/tripleo-validations master: Sort jobs in molecule.yaml file when adding new role https://review.opendev.org/c/openstack/tripleo-validations/+/797885 | 11:54 |
| opendevreview | Francesco Pantano proposed openstack/python-tripleoclient master: Fail the overcloud deploy is ceph-ansible.yaml is included https://review.opendev.org/c/openstack/python-tripleoclient/+/797867 | 11:55 |
| cloudnull | mornings | 11:57 |
| opendevreview | Jad Haj Yahya proposed openstack/tripleo-operator-ansible master: Fixing tripleo_validator_show_parameter role https://review.opendev.org/c/openstack/tripleo-operator-ansible/+/797873 | 12:02 |
| -opendevstatus- NOTICE: Our Zuul gating CI/CD services will be offline starting around 14:00 UTC (in roughly two hours from now) in order to apply some critical security updates, and is not expected to remain offline for more than 30 minutes. | 12:03 | |
| opendevreview | Jesse Pretorius proposed openstack/tripleo-upgrade master: Remove xinetd when leapp upgrading the Undercloud. https://review.opendev.org/c/openstack/tripleo-upgrade/+/796821 | 12:09 |
| opendevreview | Matthias Runge proposed openstack/tripleo-puppet-elements stable/wallaby: Remove leftover of pankoclient https://review.opendev.org/c/openstack/tripleo-puppet-elements/+/797767 | 12:17 |
| opendevreview | Sorin Sbârnea proposed openstack/tripleo-ci master: Pin test dependencies https://review.opendev.org/c/openstack/tripleo-ci/+/797891 | 12:30 |
| *** amoralej is now known as amoralej|lunch | 12:32 | |
| *** jpena|lunch is now known as jpena | 12:36 | |
| opendevreview | Merged openstack/tripleo-heat-templates stable/wallaby: Add support for keystone_authtoken/memcache_use_advanced_pool https://review.opendev.org/c/openstack/tripleo-heat-templates/+/796248 | 12:39 |
| opendevreview | Merged openstack/tripleo-heat-templates master: Explicit set qemu certificate group ownership https://review.opendev.org/c/openstack/tripleo-heat-templates/+/797627 | 12:40 |
| opendevreview | Merged openstack/tripleo-validations stable/victoria: Quick documentation content reorganization https://review.opendev.org/c/openstack/tripleo-validations/+/795797 | 12:40 |
| opendevreview | Merged openstack/tripleo-validations stable/victoria: Remove workaround for tox-ansible and global molecule config https://review.opendev.org/c/openstack/tripleo-validations/+/795796 | 12:40 |
| opendevreview | Merged openstack/tripleo-validations stable/victoria: Introduce new Ansible validation_init role. https://review.opendev.org/c/openstack/tripleo-validations/+/795945 | 12:40 |
| opendevreview | Merged openstack/tripleo-validations stable/victoria: Add Validation Development Workflow Documentation https://review.opendev.org/c/openstack/tripleo-validations/+/795946 | 12:40 |
| opendevreview | mbu proposed openstack/tripleo-quickstart-extras master: DNM test validation log directory change https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/797895 | 12:51 |
| opendevreview | mbu proposed openstack/tripleo-validations stable/train: Override log directory for CI train jobs https://review.opendev.org/c/openstack/tripleo-validations/+/797859 | 12:52 |
| opendevreview | Sergii Golovatiuk proposed openstack/tripleo-upgrade master: Remove additional dependency from pre-commit https://review.opendev.org/c/openstack/tripleo-upgrade/+/797896 | 12:52 |
| opendevreview | Francesco Pantano proposed openstack/python-tripleoclient master: Fail the overcloud deploy is ceph-ansible.yaml is included https://review.opendev.org/c/openstack/python-tripleoclient/+/797867 | 12:53 |
| opendevreview | Francesco Pantano proposed openstack/python-tripleoclient master: Fail the overcloud deploy is ceph-ansible.yaml is included https://review.opendev.org/c/openstack/python-tripleoclient/+/797867 | 12:56 |
| opendevreview | Francesco Pantano proposed openstack/python-tripleoclient master: Fail the overcloud deploy is ceph-ansible.yaml is included https://review.opendev.org/c/openstack/python-tripleoclient/+/797867 | 12:57 |
| opendevreview | Sergii Golovatiuk proposed openstack/tripleo-upgrade master: Remove unicode prefix https://review.opendev.org/c/openstack/tripleo-upgrade/+/788560 | 13:05 |
| mwhahaha | marios|ruck: plz see the ci results https://review.opendev.org/c/openstack/tripleo-heat-templates/+/797496 | 13:06 |
| mwhahaha | oh nm was looking at the wrong job | 13:07 |
| mwhahaha | fun | 13:07 |
| opendevreview | Shnaidman Sagi (Sergey) proposed openstack/tripleo-quickstart-extras master: DNM try ansible 2.11 https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/797823 | 13:08 |
| mwhahaha | oh i know what it is | 13:10 |
| opendevreview | Alex Schultz proposed openstack/tripleo-heat-templates stable/ussuri: Revert "Always run network config for undercloud" https://review.opendev.org/c/openstack/tripleo-heat-templates/+/797476 | 13:11 |
| opendevreview | Alex Schultz proposed openstack/tripleo-heat-templates stable/ussuri: Always run network config for undercloud https://review.opendev.org/c/openstack/tripleo-heat-templates/+/797496 | 13:12 |
| opendevreview | Sorin Sbârnea proposed openstack/tripleo-quickstart master: WIP: pip-compile https://review.opendev.org/c/openstack/tripleo-quickstart/+/797901 | 13:15 |
| *** amoralej|lunch is now known as amoralej | 13:18 | |
| *** marios|ruck is now known as marios|ruck|call | 13:19 | |
| marios|ruck|call | mwhahaha: sorry in call whill check in a sec | 13:34 |
| mwhahaha | marios|ruck|call: yea no problem, i think i figured it out. | 13:34 |
| opendevreview | Merged openstack/tripleo-validations stable/victoria: Compute TSX validation https://review.opendev.org/c/openstack/tripleo-validations/+/795632 | 13:34 |
| opendevreview | Merged openstack/tripleo-heat-templates stable/wallaby: Introduce nova virtlogd wrapper https://review.opendev.org/c/openstack/tripleo-heat-templates/+/795149 | 13:35 |
| opendevreview | Pooja Jadhav proposed openstack/tripleo-ci master: WIP : Adds master upgrades/updates jobs to periodic https://review.opendev.org/c/openstack/tripleo-ci/+/797865 | 13:35 |
| opendevreview | Shnaidman Sagi (Sergey) proposed openstack/tripleo-quickstart-extras master: Add option to build collection github PRs https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/796522 | 13:40 |
| marios|ruck|call | in case folks missed it 15:00 -opendevstatus:#oooq- NOTICE: Our Zuul gating CI/CD services will be offline starting around 14:00 UTC (in roughly two hours from now) in order to apply some critical security updates, and is not expected to remain offline for more than 30 minutes. | 13:41 |
| marios|ruck|call | that's in like 20 mins? | 13:41 |
| opendevreview | Jad Haj Yahya proposed openstack/tripleo-operator-ansible master: Fixing tripleo_validator_show_parameter role https://review.opendev.org/c/openstack/tripleo-operator-ansible/+/797873 | 13:45 |
| opendevreview | Shnaidman Sagi (Sergey) proposed openstack/tripleo-quickstart-extras master: Add github PR change from commit message https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/797463 | 13:46 |
| opendevreview | Shnaidman Sagi (Sergey) proposed openstack/tripleo-quickstart-extras master: Add github PR change from commit message https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/797463 | 13:46 |
| opendevreview | Shnaidman Sagi (Sergey) proposed openstack/tripleo-quickstart-extras master: DNM test build pr https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/797470 | 13:47 |
| *** marios|ruck|call is now known as marios|ruck | 13:49 | |
| marios|ruck | ack mwhahaha i saw the comments on the review thanks let's see - still worth considering the depends-on for voting but up to you main concern is to not merge that again with the job not fixed | 13:49 |
| opendevreview | Alex Schultz proposed openstack/tripleo-ansible stable/victoria: Fix failures because of Jinja version limit https://review.opendev.org/c/openstack/tripleo-ansible/+/795143 | 13:49 |
| mwhahaha | if we can get away w/o the job merge that'd be great. let's see what CI says | 13:49 |
| opendevreview | Jiri Podivin proposed openstack/validations-libs master: Coverage change job https://review.opendev.org/c/openstack/validations-libs/+/796845 | 13:52 |
| opendevreview | Yariv proposed openstack/tripleo-upgrade master: [FFWD upgrade pre] Add support in cold evacuate https://review.opendev.org/c/openstack/tripleo-upgrade/+/793113 | 13:57 |
| -opendevstatus- NOTICE: Our Zuul gating CI/CD services are being taken offline now in order to apply some critical security updates, and are not expected to remain offline for more than 30 minutes. | 13:58 | |
| *** pojadhav is now known as pojadhav|afk | 14:02 | |
| opendevreview | Harald Jensås proposed openstack/python-tripleoclient master: Implement protected resource types https://review.opendev.org/c/openstack/python-tripleoclient/+/794593 | 14:14 |
| opendevreview | Jiri Podivin proposed openstack/validations-libs master: Coverage change job https://review.opendev.org/c/openstack/validations-libs/+/796845 | 14:22 |
| opendevreview | Francesco Pantano proposed openstack/tripleo-quickstart master: Remove from master/wallaby the ceph-ansible references https://review.opendev.org/c/openstack/tripleo-quickstart/+/797805 | 14:26 |
| opendevreview | Francesco Pantano proposed openstack/tripleo-quickstart master: Remove from master the ceph-ansible references https://review.opendev.org/c/openstack/tripleo-quickstart/+/797805 | 14:27 |
| fultonj | hjensas: http://paste.openstack.org/show/806921/ | 14:28 |
| opendevreview | Harald Jensås proposed openstack/python-tripleoclient master: Don't initialize created_env_files variable https://review.opendev.org/c/openstack/python-tripleoclient/+/797906 | 14:28 |
| fultonj | hjensas: i was thinking of modifying ramishra's ansible module which creaters the inventory so that if groups are defined in roles_data.yaml those groups get created | 14:29 |
| fultonj | hjensas: do you have an opinion on this? | 14:29 |
| fultonj | Tengu: ^ you mentioned the future of tripleo-ansible-inventory at the PTG, what do you think | 14:30 |
| * Tengu reading | 14:30 | |
| opendevreview | Takashi Kajinami proposed openstack/puppet-tripleo master: Example commit for review 796832 https://review.opendev.org/c/openstack/puppet-tripleo/+/797907 | 14:31 |
| Tengu | fultonj: so basing part of the inventory generation on on-disk data? That's moving in the right direction imho :) | 14:31 |
| opendevreview | Francesco Pantano proposed openstack/tripleo-ansible master: Add tripleo_ceph_distribute_keys role https://review.opendev.org/c/openstack/tripleo-ansible/+/797511 | 14:31 |
| fultonj | Tengu: so roles_data.yaml remains authoritative for what ansible groups might exist | 14:32 |
| Tengu | fultonj: makes sense, since groups ARE actually defined there... | 14:32 |
| fultonj | if i compose roles_data.yaml however I need, then config-download creates corresponding groups | 14:32 |
| hjensas | fultonj: do you mean adding something like "inventory group membership" to role definitions in the roles data used with node provision? | 14:32 |
| fultonj | hjensas: yes | 14:33 |
| hjensas | fultonj: so Controllers will have group membership = ['ceph_mon', 'ceph_mgr'] ? | 14:33 |
| fultonj | hjensas: "inventory group membership" BASED ON role_definitions found in tht's role_data.yaml | 14:33 |
| fultonj | hjensas: yes | 14:33 |
| fultonj | hjensas: and if someone composes the role elsewhere, the inventory will automatically fit what was composed | 14:34 |
| hjensas | the inventory generation stuff has no clue about THT's role data at that point. | 14:34 |
| fultonj | hjensas: right! | 14:34 |
| fultonj | hjensas: but i want it to have an option to use that data | 14:34 |
| * hjensas has a feeling we should merge the roles used with node provision and the one used with THT. | 14:35 | |
| fultonj | hjensas: right, i'm having the same feeling | 14:36 |
| fultonj | but i wanted to check if there was anything thinking anything different | 14:36 |
| fultonj | cloudnull: mwhahaha: ^ | 14:36 |
| hjensas | I won't be against it. There is quite a few overlapping things. Node count, default_route_network, the networks for a role etc is in both files. | 14:37 |
| fultonj | my goal: http://paste.openstack.org/show/806921/ | 14:37 |
| fultonj | openstack overcloud node provision --network-ports | 14:37 |
| fultonj | calls this... | 14:37 |
| fultonj | https://github.com/openstack/tripleo-ansible/blob/master/tripleo_ansible/ansible_plugins/modules/tripleo_generate_ansible_inventory.py#L133 | 14:38 |
| fultonj | which calls... | 14:38 |
| fultonj | https://github.com/openstack/tripleo-common/blob/master/tripleo_common/inventory.py#L282 | 14:38 |
| fultonj | i want to add an option to have it use not HEAT for get_roles_by_service but what will be a part of the heat input, i.e. roles_data.yaml | 14:38 |
| fultonj | right now if you look at the working dir you have TWO inventories! | 14:39 |
| fultonj | which is fine as we're in a state of transition | 14:39 |
| fultonj | but i assume we might converge to one | 14:39 |
| hjensas | fultonj: sounds like we should converge | 14:40 |
| fultonj | ack | 14:40 |
| cloudnull | hjensas fultonj no strong opinions here, if we can converge to a single implementation +1 from me | 14:40 |
| fultonj | cloudnull: hjensas: Tengu: thanks for your input | 14:41 |
| hjensas | the same tripleo_common/inventory.py is used by the "node provison" workflow as the one used with config-download afik. So should just be a matter of aligning the filename it outputs? | 14:41 |
| fultonj | today we have $WORKING_DIR/tripleo-ansible-inventory.yaml and$WORKING_DIR/config-download/overcloud-0/tripleo-ansible-inventory.yaml | 14:43 |
| fultonj | where the 2nd has the groups because HEAT has run | 14:43 |
| fultonj | i was thinking of not breaking the second one | 14:43 |
| fultonj | i was thinking of modifying the first to have groups from roles_data.yaml | 14:44 |
| mwhahaha | it'd be nice if we could get out of the business of shoving ansible vars in the inventory | 14:44 |
| fultonj | mwhahaha: i'm not talking about vars, i'm talking about groups | 14:44 |
| mwhahaha | yea but its part of teh generation | 14:44 |
| mwhahaha | techincally groups are ansible-isms | 14:44 |
| fultonj | yes, but i'm not proposing anything about the vars | 14:45 |
| mwhahaha | yes i know but as we look to de-heating things that's part of this | 14:45 |
| fultonj | e.g. (i'm going to change the oil.... since you're working on the car could you replace the breaks?) | 14:46 |
| fultonj | (hmmm maybe the breaks need to be replaced, but i'm worried about the oil now) | 14:46 |
| mwhahaha | not necessarily but i don't have the full context here | 14:46 |
| fultonj | mwhahaha: sure, i ralize you're not asking me to do the vars part | 14:47 |
| fultonj | mwhahaha: http://paste.openstack.org/show/806921/ | 14:47 |
| fultonj | i wanted to get groups from roles_data.yaml before heat is run | 14:47 |
| mwhahaha | groups are services | 14:47 |
| mwhahaha | so you can't do that at that time | 14:48 |
| fultonj | well.... | 14:48 |
| mwhahaha | techincally ceph_osd and ceph_mon and ceph_mgr won't actually exist for cephadm | 14:48 |
| fultonj | the services which are GOING TO be created are in roles_data.yaml | 14:48 |
| fultonj | i know they don't exist yet | 14:48 |
| mwhahaha | no they aren't | 14:48 |
| mwhahaha | it's the service name | 14:48 |
| fultonj | which have a 1:1 mapping to the groups | 14:49 |
| mwhahaha | so you'd have grok THT to find those | 14:49 |
| mwhahaha | and only if they are enabled | 14:49 |
| fultonj | yes | 14:49 |
| mwhahaha | I would prefer we not use the service name | 14:49 |
| fultonj | mwhahaha: ok... | 14:49 |
| fultonj | cd $WORKING_DIR | 14:50 |
| fultonj | find . -name tripleo-ansible-inventory.yaml | 14:50 |
| fultonj | ./tripleo-ansible-inventory.yaml | 14:50 |
| fultonj | ./config-download/overcloud-0/tripleo-ansible-inventory.yaml | 14:50 |
| mwhahaha | you can propose a patch but you need the environment files in order to do what you're asking which may not be prepared at provision time | 14:50 |
| fultonj | the 2nd has the groups, the first doesn't beecause of heat but what's heat depends on env files and roles_data | 14:50 |
| fultonj | are extra groups which won't be used harmful? | 14:51 |
| mwhahaha | yes | 14:51 |
| mwhahaha | more info in ansible is always harmful | 14:51 |
| mwhahaha | because it's teerrible | 14:51 |
| jbadiapa | ls | 14:52 |
| fultonj | mwhahaha: i can think of other ways to get the groups into the inventory | 14:54 |
| mwhahaha | fultonj: so when we talked about the ceph bits, you need to provide the host/ceph mapping for cephadm | 14:54 |
| mwhahaha | fultonj: i thought that was to live external to THT so the role_data/env files no longer have tha tinfo | 14:54 |
| mwhahaha | what does the input for hosts into cephadm look like? | 14:55 |
| fultonj | that's already sovled | 14:56 |
| fultonj | i ahve a ceph spec file | 14:56 |
| fultonj | i genereate it based on deployed metal and roles_data | 14:56 |
| fultonj | or i can generate it via tripleo-ansible inventory | 14:56 |
| fultonj | we have an ansible module for it | 14:56 |
| opendevreview | Jiri Podivin proposed openstack/validations-libs master: Coverage change job https://review.opendev.org/c/openstack/validations-libs/+/796845 | 14:56 |
| fultonj | ceph_spec_bootstrap | 14:57 |
| mwhahaha | i don't think we should reuse the tripleo-ansible inventory (we actually might want to drop that later for other reasons) | 14:57 |
| fultonj | ansible-doc ceph_spec_bootstrap | 14:57 |
| fultonj | ansible-doc tripleo_generate_ansible_inventory | 14:58 |
| fultonj | is used today for networkv2 | 14:58 |
| fultonj | when you run openstack overcloud node provision --network-ports | 14:58 |
| fultonj | i'm trying to map the provisioned servers to which goes where | 14:58 |
| mwhahaha | yea so i'm not sure tripleo-ansible-inventory will continue, so it'd likely be better not to try and adopt that | 14:58 |
| opendevreview | Francesco Pantano proposed openstack/tripleo-heat-templates master: Remove from master the ceph-ansible references https://review.opendev.org/c/openstack/tripleo-heat-templates/+/797915 | 15:00 |
| fultonj | mwhahaha: but for xena , i'd be surprised if we stopped using tripleo-ansible-inventory | 15:01 |
| mwhahaha | i'm looking to move that for x | 15:01 |
| fultonj | mwhahaha: but i appreicate the advise that it may not always be there | 15:01 |
| fultonj | ah! | 15:01 |
| fultonj | glad i asked :) | 15:01 |
| mwhahaha | or at least change up how/when/what is generated | 15:01 |
| mwhahaha | you can add it, but we'll end up reworking it again :D | 15:02 |
| fultonj | ive never experienced that before in tripleo ever ;) | 15:02 |
| * fultonj joking | 15:02 | |
| mwhahaha | if it's worth doing, it's worth doing evey cycle | 15:02 |
| fultonj | so, i'd like to do the best thing i can do for right now | 15:03 |
| fultonj | so it's consistent with where things are going | 15:03 |
| fultonj | https://specs.openstack.org/openstack/tripleo-specs/specs/wallaby/tripleo-ceph.html#ansible-inventory-and-ansible-user | 15:03 |
| mwhahaha | i never used the inventory generated by the network stuff | 15:03 |
| mwhahaha | and i don't think we use it during the deploy so i'm not certain what actually comsumes it (or why we generate it) | 15:03 |
| fultonj | i had originally proposed `openstack overcloud ceph inventory` | 15:04 |
| fultonj | openstack overcloud node provision --network-ports | 15:04 |
| fultonj | consumes it | 15:04 |
| fultonj | it produces it and then uses it | 15:04 |
| mwhahaha | i see | 15:05 |
| fultonj | mwhahaha: i can make my own inventory for the ceph deploy using input from deployed-server, deployed-network | 15:05 |
| fultonj | i just thought., why create yet another inventory we have 2 right now | 15:05 |
| mwhahaha | what consumes the ceph inventory? | 15:06 |
| opendevreview | Ronelle Landy proposed openstack/tripleo-quickstart master: Exclude ansible-* from ceph-pacific repo https://review.opendev.org/c/openstack/tripleo-quickstart/+/797541 | 15:06 |
| fultonj | https://github.com/openstack/tripleo-ansible/blob/master/tripleo_ansible/playbooks/cli-overcloud-node-provision.yaml#L202-L210 | 15:06 |
| fultonj | by the way | 15:07 |
| fultonj | what will consume the ceph inventory? | 15:07 |
| fultonj | the deployed_ceph feature | 15:07 |
| fultonj | https://review.opendev.org/c/openstack/tripleo-ansible/+/797538/7/tripleo_ansible/playbooks/cli-deployed-ceph.yaml#51 | 15:07 |
| opendevreview | Jiri Podivin proposed openstack/validations-libs master: Coverage change job https://review.opendev.org/c/openstack/validations-libs/+/796845 | 15:09 |
| mwhahaha | fultonj: so it's just used to run the ceph bootstrap on. why do you even need an inventory? | 15:09 |
| fultonj | to install the ssh keys cephadm needs | 15:10 |
| mwhahaha | hmm ok | 15:10 |
| fultonj | 1. to identify the first monitor in cli-deployed-ceph.yaml playbook | 15:10 |
| fultonj | 2. to identify every potential member of a cephadm cluster and give them the right SSH key | 15:10 |
| opendevreview | Francesco Pantano proposed openstack/tripleo-validations master: Remove from master the ceph-ansible references https://review.opendev.org/c/openstack/tripleo-validations/+/797957 | 15:11 |
| fultonj | what's nice is that everything i've been doing so far remains back-compatible with composable roles | 15:11 |
| hjensas | fultonj: so you look for "Ceph" in ServicesDefault in roles data? | 15:12 |
| fultonj | hjensas: yes | 15:12 |
| mwhahaha | roles data location is customizable right? | 15:12 |
| hjensas | fultonj: are'nt those mapped to OS::Heat::None? So we actually need to see if the environment/enable-ceph.yaml is included? | 15:12 |
| fultonj | mwhahaha: yes | 15:13 |
| opendevreview | Merged openstack/tripleo-operator-ansible master: Removing --validation from role https://review.opendev.org/c/openstack/tripleo-operator-ansible/+/797677 | 15:13 |
| fultonj | hjensas: no | 15:13 |
| opendevreview | Francesco Pantano proposed openstack/tripleo-heat-templates master: Remove from master the ceph-ansible references https://review.opendev.org/c/openstack/tripleo-heat-templates/+/797915 | 15:13 |
| fultonj | roles_data.yaml will tell me what services will be on what role | 15:14 |
| fultonj | ceph_mgr,ceph_mon,ceph_osd,ceph_rgw,ceph_mds,ceph_nfs,ceph_rbdmirror | 15:14 |
| fultonj | ^ will be translated to children in an ansible group | 15:14 |
| mwhahaha | so those go away if there's not service anymore | 15:15 |
| mwhahaha | btw | 15:15 |
| fultonj | "if there's not service anymore" | 15:15 |
| mwhahaha | not a | 15:15 |
| fultonj | we're still going to deploy services on our overcloud | 15:15 |
| mwhahaha | k | 15:15 |
| fultonj | i'm not understanding | 15:15 |
| mwhahaha | it's fine | 15:16 |
| fultonj | "if there's not a service anymore" doesn't make sense to me | 15:16 |
| mwhahaha | i wasn't sure if we were still going to continue the tripleo definitions for those with cephadm | 15:16 |
| fultonj | ah | 15:16 |
| fultonj | yes, they're still there | 15:16 |
| mwhahaha | i thought cephadm might take over all the related items | 15:16 |
| fultonj | it deploys it all | 15:17 |
| mwhahaha | but it seems we're defining it for firewall rules and some metadata bits | 15:17 |
| fultonj | yes | 15:17 |
| mwhahaha | so back to the original issue is that you want to move the roles_datas processing for inventory generation early in the overall process | 15:17 |
| fultonj | yeah, that's one way to solve the problem | 15:18 |
| mwhahaha | if we expect the end user to have their customized roles_data.yaml at provision time, we could stub out all the defined services but it requires processing THT to generate those group names because their output from heat | 15:18 |
| fultonj | there's a notion of a role still in deployed_server | 15:18 |
| hjensas | fultonj: unless the user override it by setting THT parameter {{role.name}}Services in an enviornment file. | 15:18 |
| fultonj | the heat processing is only necessary to filter out waht you don't need | 15:19 |
| fultonj | but filtering prevents making a mess | 15:19 |
| mwhahaha | fultonj: no the service name is an output from the heat service | 15:19 |
| fultonj | [10:51] @<mwhahaha> because it's teerrible | 15:19 |
| mwhahaha | https://opendev.org/openstack/tripleo-heat-templates/src/branch/master/deployment/cephadm/ceph-mon.yaml#L64 | 15:20 |
| mwhahaha | it's an outputs | 15:20 |
| opendevreview | Francesco Pantano proposed openstack/tripleo-ansible master: Remove the ceph-ansible roles https://review.opendev.org/c/openstack/tripleo-ansible/+/797959 | 15:20 |
| fultonj | true | 15:22 |
| fultonj | https://github.com/openstack/tripleo-heat-templates/blob/master/roles_data.yaml#L64 | 15:22 |
| mwhahaha | right, it's not always camel to snake | 15:23 |
| mwhahaha | i think it usually is | 15:23 |
| mwhahaha | i mean you could make that assumption for ceph if you'll always make sure that's true | 15:23 |
| fultonj | to me, it's pretty obvious that OS::TripleO::Services::CephMon under name: Controller means that i create this... | 15:23 |
| fultonj | ceph_mon: | 15:23 |
| fultonj | children: | 15:23 |
| fultonj | Controller: {} | 15:24 |
| fultonj | yeah, always true for ceph | 15:24 |
| fultonj | maybe not true for everyone | 15:24 |
| mwhahaha | yea which is why i'd say it's safer for you to make those assumptions only in your cli commands | 15:24 |
| fultonj | mwhahaha: i can add only the ceph groups | 15:25 |
| hjensas | I'm still worried about using "ServicesDefault" in roles_data.yaml, If I set in parameter_defaults: { ControllerServices: [<a_list_not_containing OS::TripleO::Services::Ceph* services>] } We end up with a mismatch. | 15:26 |
| mwhahaha | we use those vars in CI fwiw | 15:26 |
| mwhahaha | but yea that could be problematic | 15:26 |
| hjensas | yes, and who knows what is used by $'ing customers. | 15:26 |
| mwhahaha | deprecate all the things :D | 15:27 |
| fultonj | https://github.com/openstack/tripleo-ansible/blob/master/tripleo_ansible/roles/tripleo_run_cephadm/tasks/enable_ceph_admin_user.yml | 15:27 |
| fultonj | is why i like to have the groups for cephadm to get the right keys | 15:28 |
| hjensas | :) yeah, I wish we'd just stick whats in roles_data.yaml into THT and have one place with the truth. | 15:28 |
| fultonj | otherwise i can just ask the user to point at the bootstrap node | 15:28 |
| mwhahaha | i think it'd be ok to initially add the service -> groups based on whats in roles data for this workflow | 15:29 |
| mwhahaha | because this is a new workflow i think we can define the limitation of *Services won't be taken into account during ceph provisioning and that the roles_data needs to have the correct values | 15:30 |
| fultonj | https://github.com/openstack/tripleo-common/blob/master/tripleo_common/inventory.py#L282 | 15:30 |
| mwhahaha | we'd still generate an inventory after heat that would have the reduced inventory | 15:30 |
| fultonj | [11:30] @<mwhahaha> we'd still generate an inventory after heat that would have the reduced inventory | 15:31 |
| fultonj | yes | 15:31 |
| fultonj | unused groups would exist because what's in roles_data.yaml may not be all of what's deployed | 15:31 |
| fultonj | i could limit it to just ceph groups if you like | 15:32 |
| * hjensas thinks we should deprecate {% role %}Services parameter in THT, and rename s/ServicesDefault/Services/ in roles_data as well ... | 15:32 | |
| mwhahaha | i'd just do it for all (i'll likely regret thi slater) | 15:32 |
| opendevreview | Jad Haj Yahya proposed openstack/tripleo-operator-ansible master: Fixing tripleo_validator_show_parameter role https://review.opendev.org/c/openstack/tripleo-operator-ansible/+/797873 | 15:33 |
| fultonj | mwhahaha: ok, i'll give it a shot | 15:35 |
| fultonj | i'll add hjensas mwhahaha to the review | 15:35 |
| fultonj | thanks | 15:36 |
| *** ysandeep is now known as ysandeep|out | 15:36 | |
| opendevreview | Marios Andreou proposed openstack/tripleo-ci master: Imports close_bugs.py utility to move stale bugs to Incomplete https://review.opendev.org/c/openstack/tripleo-ci/+/776232 | 15:37 |
| opendevreview | Marios Andreou proposed openstack/tripleo-ci master: Add milestone and close message to bugs in close_bugs.py https://review.opendev.org/c/openstack/tripleo-ci/+/776246 | 15:37 |
| marios|ruck | weshay|ruck: add to your reviews please ? so i don't have to apply the patch each time ;) and it will be easier for next person ^^^ | 15:37 |
| weshay|ruck | k.. thanks | 15:37 |
| fultonj | hmmm | 15:39 |
| * fultonj still worried about the size of the groups list | 15:39 | |
| fultonj | i think i'l have a filter option | 15:40 |
| fultonj | we'll see in the review | 15:40 |
| fultonj | hjensas: a mismatch? | 15:43 |
| fultonj | hjensas: you said... | 15:44 |
| fultonj | "If I set in parameter_defaults: { ControllerServices: [<a_list_not_containing OS::TripleO::Services::Ceph* services>] } We end up with a mismatch." | 15:44 |
| hjensas | fultonj: the "ServicesDefault" in roles_data.yaml can be overriden using THT parameters. | 15:44 |
| fultonj | oh! | 15:44 |
| fultonj | yes | 15:44 |
| fultonj | i see | 15:44 |
| fultonj | so post heat is necessary to correct potential mismatches | 15:45 |
| fultonj | so if i do this thing... | 15:45 |
| fultonj | the groups are not 100% reliable | 15:45 |
| mwhahaha | thats what i was daying about documenting that the *Services won't be supported | 15:45 |
| mwhahaha | for this workflow | 15:45 |
| fultonj | i like the option of including "roles_data.yaml | filter" | 15:47 |
| fultonj | where the user can set a filter of what gets in | 15:47 |
| fultonj | heat is the true filter but for a subset of services it can be useful to get them in advance | 15:47 |
| fultonj | advance --> before heat | 15:48 |
| fultonj | i can do it all have a filter option too and maybe default the filter to ceph | 15:49 |
| mwhahaha | i'd not over think it for now | 15:49 |
| mwhahaha | and just add roles_data.yaml support (overridable via cli param) with camel -> snake | 15:49 |
| mwhahaha | and document that <role>Services won't be taken into consideration for the provisioning workflows | 15:50 |
| fultonj | mwhahaha: ok, thanks | 15:50 |
| mwhahaha | but will be considered @ actual deployment time | 15:50 |
| opendevreview | Jiri Podivin proposed openstack/validations-libs master: Coverage change job https://review.opendev.org/c/openstack/validations-libs/+/796845 | 15:52 |
| fultonj | i expect the "deployed ceph" feature will be ready around M2 | 15:52 |
| fultonj | metal, network, ceph, overcloud | 15:53 |
| *** rpittau is now known as rpittau|afk | 16:09 | |
| opendevreview | Sergii Golovatiuk proposed openstack/tripleo-upgrade master: Do not run job on .pre-commit-config.yaml changes https://review.opendev.org/c/openstack/tripleo-upgrade/+/797969 | 16:12 |
| *** jpena is now known as jpena|lunch | 16:30 | |
| *** jpena|lunch is now known as jpena | 16:30 | |
| opendevreview | Harald Jensås proposed openstack/python-tripleoclient master: Option to disable Node, Net and VIP provisioning https://review.opendev.org/c/openstack/python-tripleoclient/+/797978 | 16:30 |
| opendevreview | Harald Jensås proposed openstack/python-tripleoclient master: Dynamic defaults for roles, networks, vips and bm https://review.opendev.org/c/openstack/python-tripleoclient/+/797979 | 16:30 |
| *** jpena is now known as jpena|off | 16:31 | |
| *** amoralej is now known as amoooralej|off | 16:55 | |
| *** gfidente is now known as gfidente|afk | 16:59 | |
| fultonj | [11:49] @<mwhahaha> and just add roles_data.yaml support (overridable via cli param) with camel -> snake | 17:05 |
| fultonj | mwhahaha: When you say "overridable via cli param" which command do you mean that for? | 17:05 |
| fultonj | `openstack overcloud node provision --network-ports` creates $WORKING_DIR/tripleo-ansible-inventory.yaml | 17:05 |
| fultonj | I could add `openstack overcloud node provision --network-ports --roles-data` | 17:06 |
| fultonj | My workflow doesn't currently use the tripleo-ansible-inventory command, but I'm wondering if that's what you meant. | 17:07 |
| opendevreview | mbu proposed openstack/validations-common master: Expose VALIDATIONS_LOG_DIR to the run command https://review.opendev.org/c/openstack/validations-common/+/797860 | 17:11 |
| opendevreview | mbu proposed openstack/validations-common master: Export VALIDATIONS_LOG_DIR to the run command https://review.opendev.org/c/openstack/validations-common/+/797860 | 17:12 |
| opendevreview | mbu proposed openstack/tripleo-validations stable/train: Enabling indentation rule for yamllint. https://review.opendev.org/c/openstack/tripleo-validations/+/797389 | 17:13 |
| mwhahaha | fultonj: all of them? Any that generate the inventory | 17:26 |
| mwhahaha | If that's the command that currently does it, that one | 17:26 |
| fultonj | mwhahaha: ok, sounds good. | 17:27 |
| mwhahaha | fultonj: so i guess my question is, do you pass the ~/tripleo-ansible-inventory.yaml into the ceph provision command? | 17:40 |
| fultonj | mwhahaha: yes, i'm planning to | 17:41 |
| mwhahaha | k | 17:42 |
| fultonj | mwhahaha: i don't have to, but the ansible that's going to configure ceph needs an inventory | 17:42 |
| fultonj | so the command could create one behind the scenes and use it | 17:42 |
| mwhahaha | then yea, the --roles-data would be an argument to whatever generates that | 17:42 |
| fultonj | right | 17:42 |
| fultonj | and the docs would need to tell the person deploying ceph to use that option | 17:43 |
| opendevreview | Ronelle Landy proposed openstack/tripleo-quickstart master: Exclude ansible-* from ceph-pacific repo https://review.opendev.org/c/openstack/tripleo-quickstart/+/797541 | 18:04 |
| opendevreview | Merged openstack/tripleo-heat-templates master: Include tripleo container service management once https://review.opendev.org/c/openstack/tripleo-heat-templates/+/794334 | 18:25 |
| opendevreview | Merged openstack/tripleo-heat-templates master: Use module for container puppet tasks https://review.opendev.org/c/openstack/tripleo-heat-templates/+/789608 | 18:41 |
| opendevreview | Merged openstack/tripleo-ansible master: Adjust shutdown service management https://review.opendev.org/c/openstack/tripleo-ansible/+/794335 | 18:42 |
| opendevreview | Alan Bishop proposed openstack/tripleo-common master: tcib: clean up cinder packages https://review.opendev.org/c/openstack/tripleo-common/+/796531 | 19:01 |
| opendevreview | Gael Chamoulaud proposed openstack/validations-common master: [check_ram] Remove README.md file https://review.opendev.org/c/openstack/validations-common/+/797666 | 19:55 |
| opendevreview | Gael Chamoulaud proposed openstack/validations-common master: [check_selinux_mode] Add role documentation in sphinx https://review.opendev.org/c/openstack/validations-common/+/797667 | 19:55 |
| opendevreview | Gael Chamoulaud proposed openstack/validations-common master: [dns] Add role documentation in sphinx https://review.opendev.org/c/openstack/validations-common/+/797801 | 19:55 |
| opendevreview | Gael Chamoulaud proposed openstack/validations-common master: [no_op] Add role documentation in sphinx https://review.opendev.org/c/openstack/validations-common/+/797802 | 19:55 |
| opendevreview | Gael Chamoulaud proposed openstack/validations-common master: [ntp] Add role documentation in sphinx https://review.opendev.org/c/openstack/validations-common/+/797861 | 19:55 |
| opendevreview | Gael Chamoulaud proposed openstack/validations-common master: [service_status] Add role documentation in sphinx https://review.opendev.org/c/openstack/validations-common/+/797862 | 19:55 |
| opendevreview | Gael Chamoulaud proposed openstack/validations-common master: [validate_selinux] Add role documentation in sphinx https://review.opendev.org/c/openstack/validations-common/+/797863 | 19:55 |
| opendevreview | Gael Chamoulaud proposed openstack/validations-common master: [xfs_check_ftype] Add role documentation in sphinx https://review.opendev.org/c/openstack/validations-common/+/797866 | 19:55 |
| *** dviroel is now known as dviroel|afk | 20:25 | |
| *** dviroel|afk is now known as dviroel | 20:54 | |
| opendevreview | James Slagle proposed openstack/tripleo-specs master: TripleO.Next - Container Pods Spec https://review.opendev.org/c/openstack/tripleo-specs/+/797676 | 21:16 |
| *** dviroel is now known as dviroel|out | 21:29 | |
| opendevreview | Merged openstack/tripleo-quickstart-extras master: Add missing custom container parameters - overcloud https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/797146 | 21:30 |
| opendevreview | Merged openstack/tripleo-validations master: [validation_init] Remove the creation of a README.md file for a new validation https://review.opendev.org/c/openstack/tripleo-validations/+/795295 | 21:30 |
| opendevreview | Merged openstack/tripleo-operator-ansible master: Fixing tripleo_validator_show_parameter role https://review.opendev.org/c/openstack/tripleo-operator-ansible/+/797873 | 21:31 |
| opendevreview | Merged openstack/tripleo-upgrade master: Skip evacuations for non-compute nodes https://review.opendev.org/c/openstack/tripleo-upgrade/+/797583 | 21:31 |
| opendevreview | Merged openstack/tripleo-upgrade stable/victoria: Skip evacuations for non-compute nodes https://review.opendev.org/c/openstack/tripleo-upgrade/+/797582 | 22:04 |
| opendevreview | Merged openstack/tripleo-upgrade stable/ussuri: Skip evacuations for non-compute nodes https://review.opendev.org/c/openstack/tripleo-upgrade/+/797581 | 22:04 |
| opendevreview | Merged openstack/tripleo-upgrade master: Bump dogpile to >=0.9.2 https://review.opendev.org/c/openstack/tripleo-upgrade/+/797690 | 22:04 |
| opendevreview | Merged openstack/tripleo-ansible stable/wallaby: Container manage module, attempt 2 https://review.opendev.org/c/openstack/tripleo-ansible/+/797530 | 22:04 |
| opendevreview | Steve Baker proposed openstack/tripleo-quickstart master: WIP Switch to overcloud-hardened-uefi-full by default on master https://review.opendev.org/c/openstack/tripleo-quickstart/+/797378 | 23:08 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!