| opendevreview | Brendan Shephard proposed openstack/python-tripleoclient master: WIP - Move baremetal CLI commands https://review.opendev.org/c/openstack/python-tripleoclient/+/820463 | 00:32 |
|---|---|---|
| opendevreview | Brendan Shephard proposed openstack/python-tripleoclient master: WIP - Move baremetal CLI commands https://review.opendev.org/c/openstack/python-tripleoclient/+/820463 | 01:54 |
| *** rlandy|ruck|bbl is now known as rlandy|ruck | 02:14 | |
| *** rlandy|ruck is now known as rlandy|out | 02:16 | |
| opendevreview | Merged openstack/tripleo-heat-templates stable/wallaby: Add CephAdmVerbose variable https://review.opendev.org/c/openstack/tripleo-heat-templates/+/822411 | 04:28 |
| opendevreview | Merged openstack/puppet-tripleo stable/train: Support configuring extra cinder RBD options https://review.opendev.org/c/openstack/puppet-tripleo/+/822557 | 04:28 |
| opendevreview | Merged openstack/puppet-tripleo master: Add powermax support https://review.opendev.org/c/openstack/puppet-tripleo/+/818119 | 04:28 |
| opendevreview | Merged openstack/python-tripleoclient stable/wallaby: Don't set 'subnet' for ctlplane in node extract https://review.opendev.org/c/openstack/python-tripleoclient/+/822519 | 04:34 |
| opendevreview | yatin proposed openstack/tripleo-quickstart master: [DNM] libvirt multi-nic deploy https://review.opendev.org/c/openstack/tripleo-quickstart/+/822712 | 05:20 |
| *** pojadhav|pto is now known as pojadhav | 05:50 | |
| opendevreview | Merged openstack/tripleo-heat-templates master: Fix IPv6 router on UC re-install https://review.opendev.org/c/openstack/tripleo-heat-templates/+/822460 | 05:56 |
| opendevreview | Merged openstack/tripleo-heat-templates master: Use Python3 yoga unit tests https://review.opendev.org/c/openstack/tripleo-heat-templates/+/821808 | 05:56 |
| opendevreview | Merged openstack/tripleo-puppet-elements stable/wallaby: Only install required/supported fence agents https://review.opendev.org/c/openstack/tripleo-puppet-elements/+/822361 | 05:56 |
| opendevreview | Merged openstack/python-tripleoclient stable/wallaby: Don't run playbooks if role count == 0 https://review.opendev.org/c/openstack/python-tripleoclient/+/822414 | 05:56 |
| opendevreview | mbu proposed openstack/validations-common master: Add feature tests for VF RFE https://review.opendev.org/c/openstack/validations-common/+/821333 | 07:27 |
| opendevreview | Brendan Shephard proposed openstack/python-tripleoclient master: WIP - Move baremetal CLI commands https://review.opendev.org/c/openstack/python-tripleoclient/+/820463 | 07:37 |
| *** pojadhav is now known as pojadhav|brb | 08:05 | |
| opendevreview | Merged openstack/tripleo-heat-templates stable/victoria: Add missing services to ControllerNovaStandalone role https://review.opendev.org/c/openstack/tripleo-heat-templates/+/822392 | 08:14 |
| *** jpena|off is now known as jpena | 08:16 | |
| opendevreview | chandan kumar proposed openstack/tripleo-quickstart-extras master: Use proper http.server command https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/822783 | 08:30 |
| *** pojadhav|brb is now known as pojadhav | 08:51 | |
| opendevreview | Vijayalakshmi proposed openstack/tripleo-heat-templates master: Update dervice_pci script to handle pci address formats https://review.opendev.org/c/openstack/tripleo-heat-templates/+/799318 | 09:07 |
| opendevreview | Merged openstack/tripleo-heat-templates stable/train: Add missing services to ControllerNovaStandalone role https://review.opendev.org/c/openstack/tripleo-heat-templates/+/822395 | 09:10 |
| opendevreview | Merged openstack/tripleo-heat-templates stable/ussuri: Add missing services to ControllerNovaStandalone role https://review.opendev.org/c/openstack/tripleo-heat-templates/+/822410 | 09:10 |
| opendevreview | Merged openstack/tripleo-heat-templates stable/wallaby: Use a designate specific directory for the bind persistent storage https://review.opendev.org/c/openstack/tripleo-heat-templates/+/817957 | 09:10 |
| opendevreview | Merged openstack/tripleo-ci master: Make standalone-ipa job non-voting during debug https://review.opendev.org/c/openstack/tripleo-ci/+/822766 | 09:10 |
| opendevreview | Harald Jensås proposed openstack/tripleo-heat-templates stable/wallaby: Fix IPv6 router on UC re-install https://review.opendev.org/c/openstack/tripleo-heat-templates/+/822687 | 09:11 |
| opendevreview | Harald Jensås proposed openstack/tripleo-heat-templates stable/victoria: Fix IPv6 router on UC re-install https://review.opendev.org/c/openstack/tripleo-heat-templates/+/822688 | 09:12 |
| opendevreview | Harald Jensås proposed openstack/tripleo-heat-templates stable/ussuri: Fix IPv6 router on UC re-install https://review.opendev.org/c/openstack/tripleo-heat-templates/+/822689 | 09:12 |
| opendevreview | Harald Jensås proposed openstack/tripleo-heat-templates stable/train: Fix IPv6 router on UC re-install https://review.opendev.org/c/openstack/tripleo-heat-templates/+/822790 | 09:12 |
| opendevreview | mbu proposed openstack/validations-common master: Add feature tests for VF RFE https://review.opendev.org/c/openstack/validations-common/+/821333 | 09:45 |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates master: Nova: Disable instance usage audit by default https://review.opendev.org/c/openstack/tripleo-heat-templates/+/822789 | 09:58 |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates master: Nova: Disable instance usage audit by default https://review.opendev.org/c/openstack/tripleo-heat-templates/+/822789 | 09:58 |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates master: Nova: Disable instance usage audit by default https://review.opendev.org/c/openstack/tripleo-heat-templates/+/822789 | 09:59 |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates master: Nova: Disable instance usage audit by default https://review.opendev.org/c/openstack/tripleo-heat-templates/+/822789 | 10:01 |
| opendevreview | Merged openstack/tripleo-ansible master: These should be indented four spaces to match the other things in this block https://review.opendev.org/c/openstack/tripleo-ansible/+/820021 | 10:09 |
| *** pojadhav is now known as pojadhav|brb | 10:19 | |
| pdeore | hi all, can I get some reviews on this patch please? https://review.opendev.org/c/openstack/tripleo-heat-templates/+/819679 | 10:57 |
| *** pojadhav|brb is now known as pojadhav | 11:04 | |
| *** rlandy|out is now known as rlandy|ruck | 11:14 | |
| *** holser is now known as holser_ | 11:26 | |
| *** holser_ is now known as holser | 11:26 | |
| opendevreview | Ronelle Landy proposed openstack/tripleo-heat-templates master: Revert "Add chrony waitsync back in" https://review.opendev.org/c/openstack/tripleo-heat-templates/+/822791 | 11:29 |
| opendevreview | Ronelle Landy proposed openstack/tripleo-ci master: Revert "Make standalone-ipa job non-voting during debug" https://review.opendev.org/c/openstack/tripleo-ci/+/822792 | 11:30 |
| *** jcapitao is now known as jcapitao_lunch | 11:41 | |
| opendevreview | Shnaidman Sagi (Sergey) proposed openstack/tripleo-quickstart master: Fix syntax error in collection install https://review.opendev.org/c/openstack/tripleo-quickstart/+/822826 | 12:24 |
| sshnaidm | rlandy|ruck, chkumar|rover fyi ^ | 12:24 |
| sshnaidm | rlandy|ruck, affected gates ^ | 12:25 |
| rlandy|ruck | sshnaidm: for good or bad? | 12:25 |
| opendevreview | Douglas Viroel proposed openstack/tripleo-ci master: [CS9] Add missing CS9 jobs defintion https://review.opendev.org/c/openstack/tripleo-ci/+/822496 | 12:25 |
| rlandy|ruck | ok - ok | 12:25 |
| sshnaidm | rlandy|ruck, galaxy server was down | 12:25 |
| rlandy|ruck | sshnaidm: need to move that to the top of the gate queue? | 12:26 |
| sshnaidm | rlandy|ruck, I see queue is not too high | 12:26 |
| sshnaidm | probably galaxy is back | 12:26 |
| opendevreview | Merged openstack/tripleo-ci-health-queries master: Add patterns for sova - provision, no container https://review.opendev.org/c/openstack/tripleo-ci-health-queries/+/822535 | 12:30 |
| opendevreview | David Hill proposed openstack/tripleo-heat-templates master: Add OVNContainerCpusetCpus https://review.opendev.org/c/openstack/tripleo-heat-templates/+/820535 | 12:37 |
| opendevreview | chandan kumar proposed openstack/tripleo-quickstart-extras master: Use Python3 based http.server command https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/822783 | 12:40 |
| *** holser is now known as holser_ | 12:48 | |
| *** holser_ is now known as holser | 12:48 | |
| *** holser is now known as holser_ | 12:49 | |
| opendevreview | Marios Andreou proposed openstack/tripleo-ci master: Allow hash override for emit releases script for minor update https://review.opendev.org/c/openstack/tripleo-ci/+/822482 | 12:50 |
| *** holser_ is now known as holser | 12:52 | |
| opendevreview | mbu proposed openstack/validations-common master: Add feature tests for VF RFE https://review.opendev.org/c/openstack/validations-common/+/821333 | 12:57 |
| opendevreview | John Fulton proposed openstack/tripleo-ansible master: Tag ceph_spec portions of deployed ceph playbook https://review.opendev.org/c/openstack/tripleo-ansible/+/822829 | 13:05 |
| *** jcapitao_lunch is now known as jcapitao | 13:53 | |
| *** holser is now known as holser_ | 14:16 | |
| opendevreview | John Fulton proposed openstack/python-tripleoclient master: Introduce "openstack overcloud ceph spec" https://review.opendev.org/c/openstack/python-tripleoclient/+/822835 | 14:20 |
| opendevreview | Shnaidman Sagi (Sergey) proposed openstack/tripleo-ci-health-queries master: Add pattern for galaxy server failure https://review.opendev.org/c/openstack/tripleo-ci-health-queries/+/822836 | 14:21 |
| *** dviroel is now known as dviroel|afk | 14:22 | |
| opendevreview | Merged openstack/tripleo-ci master: Set ci_branch for downstream https://review.opendev.org/c/openstack/tripleo-ci/+/822718 | 14:29 |
| *** pojadhav is now known as pojadhav|afk | 14:33 | |
| opendevreview | Merged openstack/tripleo-heat-templates stable/wallaby: Allow configuring secure RBAC in glance https://review.opendev.org/c/openstack/tripleo-heat-templates/+/819679 | 15:18 |
| *** dviroel|afk is now known as dviroel|afk|lunch | 15:29 | |
| opendevreview | Shnaidman Sagi (Sergey) proposed openstack/tripleo-quickstart master: Add messages about location of collections install https://review.opendev.org/c/openstack/tripleo-quickstart/+/822844 | 16:01 |
| *** ykarel is now known as ykarel|away | 16:02 | |
| *** dviroel|afk|lunch is now known as dviroel | 16:10 | |
| opendevreview | Merged openstack/tripleo-ci master: Enable CS9 jobs for all tripleo projects https://review.opendev.org/c/openstack/tripleo-ci/+/822648 | 16:19 |
| opendevreview | yatin proposed openstack/tripleo-quickstart master: [DNM] libvirt multi-nic deploy https://review.opendev.org/c/openstack/tripleo-quickstart/+/822712 | 16:23 |
| opendevreview | John Fulton proposed openstack/tripleo-ansible master: Tag ceph_spec portions of deployed ceph playbook https://review.opendev.org/c/openstack/tripleo-ansible/+/822829 | 16:26 |
| opendevreview | Alex Schultz proposed openstack/tripleo-quickstart-extras master: Drop ipa custom ntp server https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/822566 | 16:32 |
| *** marios is now known as marios|out | 16:35 | |
| rlandy|ruck | mwhahaha: aren't you on PTO? | 16:36 |
| mwhahaha | yes | 16:36 |
| mwhahaha | but i'm terrible at pto | 16:36 |
| rlandy|ruck | your bot submitting code?? | 16:36 |
| rlandy|ruck | mwhahaha: we decided to go with the revert of your master patch | 16:36 |
| rlandy|ruck | so we can re-enable the ipa test for the shutdown | 16:37 |
| mwhahaha | ehhh ok | 16:37 |
| rlandy|ruck | and deal with it when we have people back | 16:37 |
| rlandy|ruck | mwhahaha: we thought you'd be away :) | 16:37 |
| rlandy|ruck | mwhahaha: if you want to take another shot at the ntp change, | 16:37 |
| rlandy|ruck | we can decide at EoD today | 16:37 |
| * rlandy|ruck is out tomorrow | 16:37 | |
| mwhahaha | i think something else is going on because it works for every other job | 16:37 |
| mwhahaha | i think something else is wrong int he ipa job | 16:38 |
| mwhahaha | it's likely dns + ipa + enviornment | 16:38 |
| mwhahaha | because it works in check and fails in gate | 16:38 |
| rlandy|ruck | mwhahaha: we have a failure on kvm job that is run internally | 16:38 |
| mwhahaha | internally the value needs to be set to clock.redhat.com | 16:38 |
| mwhahaha | because they block the externap servers | 16:38 |
| rlandy|ruck | ok - so you'll need the revert on the non-voting ipa to test that | 16:38 |
| rlandy|ruck | akahat|ruck: ^^ fyi | 16:39 |
| mwhahaha | it's fine we can revert it and i'll poke at it next week when i'm bored | 16:39 |
| mwhahaha | should add the ipa job to THT timesync | 16:39 |
| rlandy|ruck | mwhahaha: ack on ^^ | 16:39 |
| mwhahaha | so it'll catch it next time | 16:39 |
| mwhahaha | it'd be nice for log collect to also capture chrony status | 16:40 |
| mwhahaha | cause it doesn't right now either | 16:40 |
| rlandy|ruck | mwhahaha: here are the two revert patches ... you can w+ or abandon when ready | 16:40 |
| rlandy|ruck | https://review.opendev.org/c/openstack/tripleo-ci/+/822792 | 16:41 |
| rlandy|ruck | https://review.opendev.org/c/openstack/tripleo-heat-templates/+/822791 | 16:41 |
| mwhahaha | rlandy|ruck: where do we run random stuff to put into /var/log/extras? | 16:42 |
| rlandy|ruck | we have a complicated situation where we run nexted virt kvm job scenario010 on downstream | 16:42 |
| * rlandy|ruck gets | 16:43 | |
| opendevreview | Merged openstack/tripleo-validations master: fix(ci): Install python3 package first https://review.opendev.org/c/openstack/tripleo-validations/+/822295 | 16:44 |
| rlandy|ruck | https://github.com/openstack/ansible-role-collect-logs/ | 16:44 |
| mwhahaha | i found it in the vars file | 16:44 |
| opendevreview | Alex Schultz proposed openstack/ansible-role-collect-logs master: Add chrony status capture https://review.opendev.org/c/openstack/ansible-role-collect-logs/+/822848 | 16:47 |
| mwhahaha | rlandy|ruck: -^ | 16:47 |
| mwhahaha | wonder if it needs a sudo | 16:49 |
| opendevreview | John Fulton proposed openstack/tripleo-ansible master: Tag ceph_spec portions of deployed ceph playbook https://review.opendev.org/c/openstack/tripleo-ansible/+/822829 | 16:49 |
| mwhahaha | ah no the whole block is become: true | 16:49 |
| mwhahaha | anwyay i'm off, will look into the stuff later | 16:49 |
| rlandy|ruck | k - we can check on the sudo thing | 16:49 |
| mwhahaha | roles/collect_logs/tasks/collect.yml is become: true so it should be fine | 16:50 |
| mwhahaha | i'll check on it tomorrow probably | 16:50 |
| * mwhahaha wanders off | 16:50 | |
| rlandy|ruck | thanks | 16:50 |
| *** holser_ is now known as holser | 17:18 | |
| *** jpena is now known as jpena|off | 17:23 | |
| opendevreview | Douglas Viroel proposed openstack/tripleo-ci master: [CS9] Add missing CS9 jobs defintion https://review.opendev.org/c/openstack/tripleo-ci/+/822496 | 17:28 |
| opendevreview | Douglas Viroel proposed openstack/tripleo-ci master: Enable remaining CS9 jobs in tripleo https://review.opendev.org/c/openstack/tripleo-ci/+/822497 | 17:28 |
| *** akahat|ruck is now known as akahat|out | 17:57 | |
| opendevreview | Shnaidman Sagi (Sergey) proposed openstack/tripleo-quickstart master: Fix reinstall quickstart/extras upon bootstrap https://review.opendev.org/c/openstack/tripleo-quickstart/+/822639 | 18:18 |
| opendevreview | Shnaidman Sagi (Sergey) proposed openstack/tripleo-quickstart master: Fix reinstall quickstart/extras upon bootstrap https://review.opendev.org/c/openstack/tripleo-quickstart/+/822853 | 18:20 |
| opendevreview | Shnaidman Sagi (Sergey) proposed openstack/tripleo-quickstart master: Reinstall quickstart if config dir is overwritten https://review.opendev.org/c/openstack/tripleo-quickstart/+/822853 | 18:48 |
| opendevreview | Shnaidman Sagi (Sergey) proposed openstack/tripleo-quickstart master: Reinstall quickstart if config dir is overwritten https://review.opendev.org/c/openstack/tripleo-quickstart/+/822853 | 19:08 |
| *** rlandy|ruck is now known as rlandy|ruck|brb | 19:21 | |
| opendevreview | Merged openstack/tripleo-ci-health-queries master: Add pattern for galaxy server failure https://review.opendev.org/c/openstack/tripleo-ci-health-queries/+/822836 | 19:30 |
| *** dviroel is now known as dviroel|brb | 19:57 | |
| opendevreview | John Fulton proposed openstack/python-tripleoclient master: Introduce "openstack overcloud ceph spec" https://review.opendev.org/c/openstack/python-tripleoclient/+/822835 | 19:58 |
| *** dviroel|brb is now known as dviroel | 20:20 | |
| *** dviroel is now known as dviroel|out | 20:23 | |
| *** rlandy|ruck|brb is now known as rlandy|ruck | 20:26 | |
| rlandy|ruck | https://code.engineering.redhat.com/gerrit/c/testproject/+/300913 - testing https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/822566 with downstream | 20:36 |
| opendevreview | Alex Schultz proposed openstack/ansible-role-collect-logs master: Add chrony status capture https://review.opendev.org/c/openstack/ansible-role-collect-logs/+/822848 | 20:36 |
| mwhahaha | my assumption is that it fails because we don't specity NtpServer downstream | 20:37 |
| mwhahaha | anyway we'll see | 20:37 |
| opendevreview | Merged openstack/python-tripleoclient master: Don't assume json network config in validations https://review.opendev.org/c/openstack/python-tripleoclient/+/821880 | 20:46 |
| opendevreview | Alex Schultz proposed openstack/tripleo-ci master: Add multinode-ipa to chrony changes https://review.opendev.org/c/openstack/tripleo-ci/+/822857 | 20:50 |
| opendevreview | Alex Schultz proposed openstack/tripleo-heat-templates master: Revert "Add chrony waitsync back in" https://review.opendev.org/c/openstack/tripleo-heat-templates/+/822791 | 20:50 |
| opendevreview | Alex Schultz proposed openstack/tripleo-ci master: Revert "Make standalone-ipa job non-voting during debug" https://review.opendev.org/c/openstack/tripleo-ci/+/822792 | 20:51 |
| opendevreview | Alex Schultz proposed openstack/tripleo-heat-templates stable/wallaby: Add chrony waitsync back in https://review.opendev.org/c/openstack/tripleo-heat-templates/+/822428 | 20:52 |
| opendevreview | Alex Schultz proposed openstack/tripleo-heat-templates stable/wallaby: Add chrony waitsync back in https://review.opendev.org/c/openstack/tripleo-heat-templates/+/822428 | 20:53 |
| rlandy|ruck | https://sf.hosted.upshift.rdu2.redhat.com/logs/13/300913/2/check/periodic-tripleo-ci-centos-8-scenario010-kvm-internal-standalone-master/7173873/logs/undercloud/home/zuul/standalone_deploy.log | 21:35 |
| rlandy|ruck | mwhahaha: so you're right - downstream it fails | 21:35 |
| mwhahaha | Yea cause NtpServer is not being set universally. I didn't see it in the standalone params | 21:40 |
| rlandy|ruck | mwhahaha: suggestion on where to go from here? | 21:41 |
| mwhahaha | i threw up a few test patches that have the chrony logging | 21:43 |
| mwhahaha | but we need to be setting NtpServer | 21:43 |
| mwhahaha | https://sf.hosted.upshift.rdu2.redhat.com/logs/13/300913/2/check/periodic-tripleo-ci-centos-8-scenario010-kvm-internal-standalone-master/7173873/logs/undercloud/home/zuul/standalone_parameters.yaml | 21:44 |
| mwhahaha | NtpServer needs to be set downstream always | 21:44 |
| mwhahaha | upstream we can just leave the defaults | 21:44 |
| rlandy|ruck | ok - https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/822566/4/roles/standalone/tasks/containers.yml | 21:45 |
| rlandy|ruck | instead of removing set in downstream only | 21:45 |
| rlandy|ruck | if job.standalone_ntp_servers is defined | 21:46 |
| mwhahaha | no it needs to be set everywhere | 21:46 |
| mwhahaha | that bit needs to be nuked | 21:46 |
| mwhahaha | those values would be ignored if set in standalone_parameters.yaml | 21:46 |
| mwhahaha | because the file it munges (the ci/environment file) is actually defined before standalone_parameters.yaml | 21:47 |
| mwhahaha | i have no idea why that block exists but it shouldn't | 21:47 |
| mwhahaha | actually i lied | 21:48 |
| mwhahaha | we're setting the ci env after standalone params | 21:48 |
| mwhahaha | https://sf.hosted.upshift.rdu2.redhat.com/logs/13/300913/2/check/periodic-tripleo-ci-centos-8-scenario010-kvm-internal-standalone-master/7173873/logs/undercloud/home/zuul/tripleo_deploy.sh | 21:48 |
| mwhahaha | we shouldn't be | 21:48 |
| rlandy|ruck | we are setting https://code.engineering.redhat.com/gerrit/gitweb?p=openstack/tripleo-ci-internal-config.git;a=blob;f=playbooks/set-internal-vars-standalone.yaml;h=a3409450f5648540f3c757f2a1c80ee0d0f71ac5;hb=HEAD#l28 for rhel-based jobs | 21:49 |
| mwhahaha | NtpServer should be an env config | 21:49 |
| mwhahaha | yea but that's standalone_config | 21:52 |
| mwhahaha | wtf is that | 21:52 |
| mwhahaha | cause that didn't make it into https://sf.hosted.upshift.rdu2.redhat.com/logs/13/300913/2/check/periodic-tripleo-ci-centos-8-scenario010-kvm-internal-standalone-master/7173873/logs/undercloud/home/zuul/standalone_parameters.yaml | 21:53 |
| rlandy|ruck | correct | 21:53 |
| rlandy|ruck | not rhel-based job | 21:53 |
| rlandy|ruck | that is a centos8 based job that gets run in downstream | 21:53 |
| rlandy|ruck | long story | 21:53 |
| mwhahaha | to which i will probably flip many a table | 21:53 |
| rlandy|ruck | we lost psi pubic | 21:53 |
| mwhahaha | (╯°□°)╯︵ ┻━┻ | 21:53 |
| mwhahaha | yea i know | 21:53 |
| mwhahaha | anyway we need to have a bit to set that everywhere and not hack it in | 21:54 |
| mwhahaha | similar to the ovb-common.yaml | 21:54 |
| rlandy|ruck | k | 21:54 |
| mwhahaha | i thought we had one somewhere | 21:54 |
| rlandy|ruck | https://github.com/openstack/tripleo-quickstart-extras/blob/master/roles/standalone/templates/standalone_config.yaml.j2 | 21:56 |
| rlandy|ruck | that's the standalone config we use | 21:56 |
| mwhahaha | yea NtpServer should be set there | 21:56 |
| opendevreview | Merged openstack/tripleo-heat-templates stable/victoria: GaneshaNetwork - fallback -> external -> ctlplane https://review.opendev.org/c/openstack/tripleo-heat-templates/+/822022 | 21:56 |
| mwhahaha | DnsServer too but i think i know why the ipa thing hacks it in there | 21:56 |
| *** sshnaidm is now known as sshnaidm|afk | 21:57 | |
| mwhahaha | https://github.com/redhat-openstack/infrared/blob/fa79d0634534775f890f3ed5a94b45edab99e464/plugins/tripleo-overcloud/vars/deployment/files/composable_roles/network/network-environment.yaml.j2#L29-L31 | 21:57 |
| mwhahaha | that's where it ends up being set in infrared for the overcloud | 21:58 |
| mwhahaha | https://github.com/redhat-openstack/infrared/blob/35481590353ff855e1242e99b14bca81af013fac/plugins/tripleo-undercloud/templates/undercloud.conf.j2#L43-L45 | 21:58 |
| mwhahaha | we conf that in the feature sets but that won't work with upstream/downstream needing to have different values | 21:59 |
| mwhahaha | it probably should be a zuul var with each zuul base setting it differently | 21:59 |
| rlandy|ruck | it we do what you had ... | 22:00 |
| rlandy|ruck | NtpServer: {{ job.standalone_ntp_servers|default("['0.pool.ntp.org', '1.pool.ntp.org', '2.pool.ntp.org', '3.pool.ntp.org']") }} | 22:00 |
| rlandy|ruck | in this file | 22:00 |
| mwhahaha | yea that'd be fine | 22:01 |
| rlandy|ruck | and define standalone_ntp_servers for downstream jobs | 22:01 |
| * rlandy|ruck tries | 22:01 | |
| mwhahaha | we also need to set it for undercloud_ntp_servers too | 22:01 |
| mwhahaha | we should have been hitting this in Queens/early train but I guess we broke something along the way | 22:03 |
| opendevreview | Ronelle Landy proposed openstack/tripleo-quickstart-extras master: Add NtpServer to standalone_config.yaml https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/822858 | 22:07 |
| rlandy|ruck | let's see if this will work for with downstream defined value | 22:07 |
| mwhahaha | it should if the job has a value | 22:08 |
| mwhahaha | anyway i'll check back later/tomorrow | 22:08 |
| mwhahaha | have a good one | 22:08 |
| rlandy|ruck | mwhahaha: thanks - you too | 22:08 |
| opendevreview | Ronelle Landy proposed openstack/tripleo-quickstart-extras master: Add NtpServer to standalone_config.yaml https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/822858 | 22:09 |
| opendevreview | Takashi Kajinami proposed openstack/tripleo-heat-templates master: Reduce frequency of task retries https://review.opendev.org/c/openstack/tripleo-heat-templates/+/822861 | 23:57 |
| rlandy|ruck | mwhahaha: https://code.engineering.redhat.com/gerrit/c/testproject/+/300913 - https://review.opendev.org/c/openstack/tripleo-quickstart-extras/+/822858 will work for internal | 23:57 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!