| opendevreview | James Denton proposed openstack/openstack-ansible master: Implement OVN inventory changes and deploy by default https://review.opendev.org/c/openstack/openstack-ansible/+/862924 | 01:15 |
|---|---|---|
| spatel | noonedeadpunk jrosser what is going on here - https://paste.opendev.org/show/bcJBKFO0A7RqikDs6Idc/ | 01:55 |
| spatel | is this something new introduced recently? | 02:07 |
| spatel | In document this step is missing - openstack-ansible certificate-ssh-authority.yml | 02:12 |
| spatel | After digging its part of setup-hosts.yml then why it didn't run ? - https://opendev.org/openstack/openstack-ansible/commit/6ae7806fdd91a082c01626b68009d51fc03b8484 | 02:15 |
| spatel | very odd.. | 02:15 |
| noonedeadpunk | spatel: I bet that is covered in release notes fwiw | 08:36 |
| noonedeadpunk | as well as in setup-infrastructure | 08:37 |
| noonedeadpunk | it seems you haven't ran certificate-ssh-authority.yml playbook when upgrading | 08:37 |
| opendevreview | Jonathan Rosser proposed openstack/openstack-ansible stable/yoga: Update upgrade instructions for Yoga release https://review.opendev.org/c/openstack/openstack-ansible/+/866460 | 08:40 |
| jrosser | noonedeadpunk: ^ i think we missed this backport for the major upgrade instructions in Y? | 08:40 |
| jrosser | though really there is no reason it would not run automatically as part of setup-hosts, there is no conditional to stop it | 08:41 |
| noonedeadpunk | oh, well, makes sense then | 08:41 |
| jrosser | but use of --limit might get in the way of course | 08:41 |
| jrosser | also we must take this out of the Zed upgrade instructions perhaps? | 08:42 |
| jrosser | and also according to the notes we can remove this now https://github.com/openstack/openstack-ansible-repo_server/blob/master/tasks/main.yml#L32-L35 | 08:44 |
| noonedeadpunk | With upgrade this is tough for me a bit to understand when we can/should drop upgrade steps | 08:46 |
| noonedeadpunk | as we need to support upgrade path from Yoga to Antelope | 08:46 |
| noonedeadpunk | But this step is from X to Y, so likely we can drop it now.... | 08:46 |
| noonedeadpunk | Though I'm trying to fix ovn default deployment now, as some logic looks odd now | 08:47 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible master: Implement OVN inventory changes and deploy by default https://review.opendev.org/c/openstack/openstack-ansible/+/862924 | 08:51 |
| noonedeadpunk | btw I've found couple of issues in our gate scripts that really slow down things. I guess we can save around 5 mins overall if fix them. Most valuable is https://review.opendev.org/c/openstack/openstack-ansible/+/866374 and https://review.opendev.org/c/openstack/openstack-ansible/+/866372 | 08:52 |
| noonedeadpunk | (they're all failing now due to ovn) | 08:53 |
| noonedeadpunk | also for ovn - we must check on https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/865973 status as well before landing to integrated repo. | 08:54 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible master: Implement OVN inventory changes and deploy by default https://review.opendev.org/c/openstack/openstack-ansible/+/862924 | 12:33 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible master: Implement OVN inventory changes and deploy by default https://review.opendev.org/c/openstack/openstack-ansible/+/862924 | 12:50 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_neutron master: Add lxb jobs https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/865973 | 12:51 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible master: Implement OVN inventory changes and deploy by default https://review.opendev.org/c/openstack/openstack-ansible/+/862924 | 19:07 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/openstack-ansible-os_neutron master: Do not install neutron venv if not needed. https://review.opendev.org/c/openstack/openstack-ansible-os_neutron/+/863546 | 19:14 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!