| 2020-09-21 15:54:00 |
Corey Bryant |
bug |
|
|
added bug |
| 2020-09-21 15:54:47 |
Corey Bryant |
bug task added |
|
neutron |
|
| 2020-09-21 15:55:37 |
Corey Bryant |
description |
Presumably the default of keepalived_us_no_track default=True breaks bionic deploys and the patch [1] has been backported to several bionic-based stable branches; it's been backported back to stable/rocky.
[1] https://review.opendev.org/#/c/747862/1
This doesn't seem like a good backport to stable branches without having some logic in it that detects the version of keepalived and sets the default appropriately based on that version.
More discussion with slaweq from #openstack-neutron:
coreycb> Corey Bryant slaweq: about this backport, will it break deployments that are on bionic? https://review.opendev.org/#/c/747857/1
10:40 AM I don't think there are any stein deployments on focal
11:06 AM
<slaweq> Sławek Kapłoński coreycb: what keepalived version is available on Bionic?
11:06 AM coreycb: ahh, ok it's 1.x
11:06 AM
<coreycb> Corey Bryant slaweq: correct 1.x
11:06 AM
<slaweq> Sławek Kapłoński so it should be set to False on Bionic
11:10 AM → armax joined ⇐ priteau quit
11:12 AM
<coreycb> Corey Bryant slaweq: it would be really nice to if we could detect the version and set it appropriately |
Presumably the default of keepalived_us_no_track default=True breaks bionic deploys and the patch [1] has been backported to several bionic-based stable branches with a conflicting default config value; it's been backported back to stable/rocky.
[1] https://review.opendev.org/#/c/747862/1
This doesn't seem like a good backport to stable branches without having some logic in it that detects the version of keepalived and sets the default appropriately based on that version.
More discussion with slaweq from #openstack-neutron:
coreycb> Corey Bryant slaweq: about this backport, will it break deployments that are on bionic? https://review.opendev.org/#/c/747857/1
10:40 AM I don't think there are any stein deployments on focal
11:06 AM
<slaweq> Sławek Kapłoński coreycb: what keepalived version is available on Bionic?
11:06 AM coreycb: ahh, ok it's 1.x
11:06 AM
<coreycb> Corey Bryant slaweq: correct 1.x
11:06 AM
<slaweq> Sławek Kapłoński so it should be set to False on Bionic
11:10 AM → armax joined ⇐ priteau quit
11:12 AM
<coreycb> Corey Bryant slaweq: it would be really nice to if we could detect the version and set it appropriately |
|
| 2020-09-21 15:58:58 |
Corey Bryant |
description |
Presumably the default of keepalived_us_no_track default=True breaks bionic deploys and the patch [1] has been backported to several bionic-based stable branches with a conflicting default config value; it's been backported back to stable/rocky.
[1] https://review.opendev.org/#/c/747862/1
This doesn't seem like a good backport to stable branches without having some logic in it that detects the version of keepalived and sets the default appropriately based on that version.
More discussion with slaweq from #openstack-neutron:
coreycb> Corey Bryant slaweq: about this backport, will it break deployments that are on bionic? https://review.opendev.org/#/c/747857/1
10:40 AM I don't think there are any stein deployments on focal
11:06 AM
<slaweq> Sławek Kapłoński coreycb: what keepalived version is available on Bionic?
11:06 AM coreycb: ahh, ok it's 1.x
11:06 AM
<coreycb> Corey Bryant slaweq: correct 1.x
11:06 AM
<slaweq> Sławek Kapłoński so it should be set to False on Bionic
11:10 AM → armax joined ⇐ priteau quit
11:12 AM
<coreycb> Corey Bryant slaweq: it would be really nice to if we could detect the version and set it appropriately |
Presumably the default of keepalived_us_no_track default=True breaks bionic deploys and the patch [1] has been backported to several bionic-based stable branches with a conflicting default config value; it's been backported back to stable/rocky.
[1] https://review.opendev.org/#/c/747862/1
This doesn't seem like a good backport to stable branches without having some logic in it that detects the version of keepalived and sets the default appropriately based on that version.
More discussion with slaweq from #openstack-neutron:
coreycb> Corey Bryant slaweq: about this backport, will it break deployments that are on bionic? https://review.opendev.org/#/c/747857/1
10:40 AM I don't think there are any stein deployments on focal
11:06 AM
<slaweq> Sławek Kapłoński coreycb: what keepalived version is available on Bionic?
11:06 AM coreycb: ahh, ok it's 1.x
11:06 AM
<coreycb> Corey Bryant slaweq: correct 1.x
11:06 AM
<slaweq> Sławek Kapłoński so it should be set to False on Bionic
11:10 AM → armax joined ⇐ priteau quit
11:12 AM
<coreycb> Corey Bryant slaweq: it would be really nice to if we could detect the version and set it appropriately
Can we revert this for all releases prior to Ussuri?
For Ubuntu, Ussuri is available by default with Focal, but we also backport Ussuri to bionic so we would have to change the default for keepalived_us_no_track default to False for bionic. |
|
| 2020-09-21 15:59:51 |
Corey Bryant |
summary |
keepalived_us_no_track default=True breaks bionic deploys |
keepalived_use_no_track default=True breaks bionic deploys |
|
| 2020-09-21 16:00:05 |
Corey Bryant |
description |
Presumably the default of keepalived_us_no_track default=True breaks bionic deploys and the patch [1] has been backported to several bionic-based stable branches with a conflicting default config value; it's been backported back to stable/rocky.
[1] https://review.opendev.org/#/c/747862/1
This doesn't seem like a good backport to stable branches without having some logic in it that detects the version of keepalived and sets the default appropriately based on that version.
More discussion with slaweq from #openstack-neutron:
coreycb> Corey Bryant slaweq: about this backport, will it break deployments that are on bionic? https://review.opendev.org/#/c/747857/1
10:40 AM I don't think there are any stein deployments on focal
11:06 AM
<slaweq> Sławek Kapłoński coreycb: what keepalived version is available on Bionic?
11:06 AM coreycb: ahh, ok it's 1.x
11:06 AM
<coreycb> Corey Bryant slaweq: correct 1.x
11:06 AM
<slaweq> Sławek Kapłoński so it should be set to False on Bionic
11:10 AM → armax joined ⇐ priteau quit
11:12 AM
<coreycb> Corey Bryant slaweq: it would be really nice to if we could detect the version and set it appropriately
Can we revert this for all releases prior to Ussuri?
For Ubuntu, Ussuri is available by default with Focal, but we also backport Ussuri to bionic so we would have to change the default for keepalived_us_no_track default to False for bionic. |
Presumably the default of keepalived_use_no_track default=True breaks bionic deploys and the patch [1] has been backported to several bionic-based stable branches with a conflicting default config value; it's been backported back to stable/rocky.
[1] https://review.opendev.org/#/c/747862/1
This doesn't seem like a good backport to stable branches without having some logic in it that detects the version of keepalived and sets the default appropriately based on that version.
More discussion with slaweq from #openstack-neutron:
coreycb> Corey Bryant slaweq: about this backport, will it break deployments that are on bionic? https://review.opendev.org/#/c/747857/1
10:40 AM I don't think there are any stein deployments on focal
11:06 AM
<slaweq> Sławek Kapłoński coreycb: what keepalived version is available on Bionic?
11:06 AM coreycb: ahh, ok it's 1.x
11:06 AM
<coreycb> Corey Bryant slaweq: correct 1.x
11:06 AM
<slaweq> Sławek Kapłoński so it should be set to False on Bionic
11:10 AM → armax joined ⇐ priteau quit
11:12 AM
<coreycb> Corey Bryant slaweq: it would be really nice to if we could detect the version and set it appropriately
Can we revert this for all releases prior to Ussuri?
For Ubuntu, Ussuri is available by default with Focal, but we also backport Ussuri to bionic so we would have to change the default for keepalived_use_no_track default to False for bionic. |
|
| 2020-09-21 21:48:00 |
Brian Haley |
bug |
|
|
added subscriber Brian Haley |
| 2020-09-21 21:48:03 |
Brian Haley |
neutron: importance |
Undecided |
High |
|
| 2020-09-21 21:48:40 |
Brian Haley |
neutron: status |
New |
Confirmed |
|
| 2020-09-22 03:00:12 |
Dominique Poulain |
bug |
|
|
added subscriber Dominique Poulain |
| 2020-09-22 07:29:07 |
Launchpad Janitor |
neutron (Ubuntu): status |
New |
Confirmed |
|
| 2020-09-29 12:32:53 |
Slawek Kaplonski |
tags |
|
l3-ha |
|
| 2020-09-29 14:51:13 |
Bernard Cafarelli |
bug |
|
|
added subscriber Bernard Cafarelli |
| 2020-10-12 16:36:47 |
OpenStack Infra |
neutron: status |
Confirmed |
In Progress |
|
| 2020-10-12 16:36:47 |
OpenStack Infra |
neutron: assignee |
|
Lajos Katona (lajos-katona) |
|
| 2020-10-14 13:54:11 |
Chris MacNaughton |
bug task added |
|
cloud-archive |
|
| 2020-10-14 13:54:22 |
Chris MacNaughton |
nominated for series |
|
cloud-archive/train |
|
| 2020-10-14 13:54:22 |
Chris MacNaughton |
bug task added |
|
cloud-archive/train |
|
| 2020-10-14 13:54:22 |
Chris MacNaughton |
nominated for series |
|
cloud-archive/stein |
|
| 2020-10-14 13:54:22 |
Chris MacNaughton |
bug task added |
|
cloud-archive/stein |
|
| 2020-10-14 13:54:22 |
Chris MacNaughton |
nominated for series |
|
cloud-archive/ussuri |
|
| 2020-10-14 13:54:22 |
Chris MacNaughton |
bug task added |
|
cloud-archive/ussuri |
|
| 2020-10-14 13:54:32 |
Chris MacNaughton |
cloud-archive: status |
New |
Invalid |
|
| 2020-10-14 14:10:19 |
Chris MacNaughton |
description |
Presumably the default of keepalived_use_no_track default=True breaks bionic deploys and the patch [1] has been backported to several bionic-based stable branches with a conflicting default config value; it's been backported back to stable/rocky.
[1] https://review.opendev.org/#/c/747862/1
This doesn't seem like a good backport to stable branches without having some logic in it that detects the version of keepalived and sets the default appropriately based on that version.
More discussion with slaweq from #openstack-neutron:
coreycb> Corey Bryant slaweq: about this backport, will it break deployments that are on bionic? https://review.opendev.org/#/c/747857/1
10:40 AM I don't think there are any stein deployments on focal
11:06 AM
<slaweq> Sławek Kapłoński coreycb: what keepalived version is available on Bionic?
11:06 AM coreycb: ahh, ok it's 1.x
11:06 AM
<coreycb> Corey Bryant slaweq: correct 1.x
11:06 AM
<slaweq> Sławek Kapłoński so it should be set to False on Bionic
11:10 AM → armax joined ⇐ priteau quit
11:12 AM
<coreycb> Corey Bryant slaweq: it would be really nice to if we could detect the version and set it appropriately
Can we revert this for all releases prior to Ussuri?
For Ubuntu, Ussuri is available by default with Focal, but we also backport Ussuri to bionic so we would have to change the default for keepalived_use_no_track default to False for bionic. |
[Impact]
In the latest Neutron stable release, a new config option was introduced to control using no_track in Keepalived. This config options defaults to true which is broken on releases before Focal.
The proposed fix for this in earlier releases is to change the default to False.
[Test Case]
TBD
[Regression Potential]
The regression potential is low as the fix is changing a configuration default.
--------------------------------------------------------------------------
Presumably the default of keepalived_use_no_track default=True breaks bionic deploys and the patch [1] has been backported to several bionic-based stable branches with a conflicting default config value; it's been backported back to stable/rocky.
[1] https://review.opendev.org/#/c/747862/1
This doesn't seem like a good backport to stable branches without having some logic in it that detects the version of keepalived and sets the default appropriately based on that version.
More discussion with slaweq from #openstack-neutron:
coreycb> Corey Bryant slaweq: about this backport, will it break deployments that are on bionic? https://review.opendev.org/#/c/747857/1
10:40 AM I don't think there are any stein deployments on focal
11:06 AM
<slaweq> Sławek Kapłoński coreycb: what keepalived version is available on Bionic?
11:06 AM coreycb: ahh, ok it's 1.x
11:06 AM
<coreycb> Corey Bryant slaweq: correct 1.x
11:06 AM
<slaweq> Sławek Kapłoński so it should be set to False on Bionic
11:10 AM → armax joined ⇐ priteau quit
11:12 AM
<coreycb> Corey Bryant slaweq: it would be really nice to if we could detect the version and set it appropriately
Can we revert this for all releases prior to Ussuri?
For Ubuntu, Ussuri is available by default with Focal, but we also backport Ussuri to bionic so we would have to change the default for keepalived_use_no_track default to False for bionic. |
|
| 2020-10-29 12:25:29 |
Christian Rohmann |
bug |
|
|
added subscriber Christian Rohmann |
| 2020-11-05 18:52:22 |
OpenStack Infra |
neutron: status |
In Progress |
Fix Released |
|
| 2020-11-27 15:04:48 |
Bernard Cafarelli |
tags |
l3-ha |
l3-ha neutron-proactive-backport-potential |
|
| 2020-12-11 11:01:12 |
Slawek Kaplonski |
tags |
l3-ha neutron-proactive-backport-potential |
l3-ha |
|
| 2021-06-29 11:06:27 |
OpenStack Infra |
cloud-archive/ussuri: status |
New |
In Progress |
|
