[SRU] Agent is failing to process HA router if initialize() fails
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Ubuntu Cloud Archive |
Fix Released
|
Undecided
|
Unassigned | ||
Mitaka |
Fix Released
|
Undecided
|
Edward Hope-Morley | ||
Newton |
Fix Released
|
Undecided
|
Edward Hope-Morley | ||
neutron |
Fix Released
|
High
|
venkata anil | ||
neutron (Ubuntu) |
Fix Released
|
Undecided
|
Unassigned | ||
Xenial |
Fix Released
|
Undecided
|
Edward Hope-Morley | ||
Yakkety |
Fix Released
|
Undecided
|
Edward Hope-Morley |
Bug Description
[Impact]
This patch resolves, amongst other things, issues with a create and delete router request race condition when using l3 HA. At the time of backport this patch is already available from Ocata onwards and has been verified as sufficiently minimal and safe for backport to Newton and Mitaka. Essentially the error case is a result of an incorrectly intialised router update action being executed without proper checks and this patch fixes this.
[Test Case]
* Deploy Openstack Mitaka - http://
* Repeatedly create and delete routers in rapid succession and check that the l3 agent does not go into an infinite error loop i.e. run http://
[Regression Potential]
* I do not envisage any regression potential from this patch.
====
When HA router initialize() function fails for some reason(rabbitmq restart or no ha_port), keepalived_manager or KeepalivedInstance won't be configured. In this case, _process_
2017-02-06 18:34:18.539 26120 DEBUG neutron.
3406e1-e7'] execute_
2017-02-06 18:34:18.544 26120 DEBUG neutron.
Command: ['ip', 'netns', 'exec', u'qrouter-
Exit code: 0
execute /usr/lib/
2017-02-06 18:34:18.544 26120 ERROR neutron.
2017-02-06 18:34:18.544 26120 ERROR neutron.
2017-02-06 18:34:18.544 26120 ERROR neutron.
2017-02-06 18:34:18.544 26120 ERROR neutron.
2017-02-06 18:34:18.544 26120 ERROR neutron.
2017-02-06 18:34:18.544 26120 ERROR neutron.
2017-02-06 18:34:18.544 26120 ERROR neutron.
2017-02-06 18:34:18.544 26120 ERROR neutron.
2017-02-06 18:34:18.544 26120 ERROR neutron.
2017-02-06 18:34:18.544 26120 ERROR neutron.
2017-02-06 18:34:18.544 26120 ERROR neutron.
2017-02-06 18:34:18.544 26120 ERROR neutron.
2017-02-06 18:34:18.544 26120 ERROR neutron.
2017-02-06 18:34:18.544 26120 ERROR neutron.
2017-02-06 18:34:18.544 26120 ERROR neutron.
2017-02-06 18:34:18.544 26120 ERROR neutron.
2017-02-06 18:34:18.549 26120 ERROR neutron.
2017-02-06 18:34:18.549 26120 ERROR neutron.
2017-02-06 18:34:18.549 26120 ERROR neutron.
2017-02-06 18:34:18.549 26120 ERROR neutron.
2017-02-06 18:34:18.549 26120 ERROR neutron.
2017-02-06 18:34:18.549 26120 ERROR neutron.
2017-02-06 18:34:18.549 26120 ERROR neutron.
2017-02-06 18:34:18.549 26120 ERROR neutron.
2017-02-06 18:34:18.549 26120 ERROR neutron.
2017-02-06 18:34:18.549 26120 ERROR neutron.
2017-02-06 18:34:18.549 26120 ERROR neutron.
2017-02-06 18:34:18.549 26120 ERROR neutron.
2017-02-06 18:34:18.549 26120 ERROR neutron.
2017-02-06 18:34:18.549 26120 ERROR neutron.
2017-02-06 18:34:18.549 26120 ERROR neutron.
2017-02-06 18:34:18.549 26120 ERROR neutron.
2017-02-06 18:34:18.549 26120 ERROR neutron.
2017-02-06 18:34:18.549 26120 ERROR neutron.
2017-02-06 18:34:18.549 26120 ERROR neutron.
2017-02-06 18:34:18.549 26120 ERROR neutron.
2017-02-06 18:34:18.549 26120 ERROR neutron.
2017-02-06 18:34:18.549 26120 ERROR neutron.
2017-02-06 18:34:18.549 26120 ERROR neutron.
2017-02-06 18:34:18.549 26120 ERROR neutron.
2017-02-06 18:34:18.549 26120 ERROR neutron.
2017-02-06 18:34:18.549 26120 ERROR neutron.
2017-02-06 18:34:18.549 26120 ERROR neutron.
Changed in neutron: | |
assignee: | nobody → venkata anil (anil-venkata) |
tags: | added: l3-ha |
Changed in neutron: | |
importance: | Undecided → Medium |
Changed in neutron: | |
assignee: | venkata anil (anil-venkata) → Brian Haley (brian-haley) |
Changed in neutron: | |
assignee: | Brian Haley (brian-haley) → venkata anil (anil-venkata) |
Changed in neutron (Ubuntu Xenial): | |
assignee: | nobody → Edward Hope-Morley (hopem) |
Changed in neutron (Ubuntu Yakkety): | |
assignee: | nobody → Edward Hope-Morley (hopem) |
Changed in neutron (Ubuntu Xenial): | |
status: | New → In Progress |
Changed in neutron (Ubuntu Yakkety): | |
status: | New → In Progress |
Fix proposed to branch: master /review. openstack. org/431026
Review: https:/