Skip to content
This repository has been archived by the owner on Dec 4, 2018. It is now read-only.

Neutron ha tool timeout #272

Merged

Conversation

jsuchome
Copy link
Member

@jsuchome jsuchome commented Apr 4, 2016

increase neutron-ha-tool start timeout to 120s

Backport of crowbar/crowbar-openstack#344 and crowbar/crowbar-openstack#347

Adam Spiers and others added 2 commits April 4, 2016 09:45
SUSE-Cloud/cookbook-openstack-network#1
adds retry logic to neutron-ha-tool so that it will not fail
immediately if there are problems talking to the neutron-server
API.  Therefore we need to give neutron-ha-tool enough time
to have a decent chance of succeeding if the neutron-server
still needs a while to recover after some kind of failover.

https://bugzilla.suse.com/show_bug.cgi?id=965886
(cherry picked from commit 55504e2e8ccfb23abb8a6b5b31e4ff7ef99e911b)
Set the start timeout only for neutron-ha-tool resource,
not for neutron agent services.
This fixes change introduced by commit 55504e2e8ccfb23abb8a6b5b31e4ff7ef99e911b,
where the 120s value was incorrectly set as "start interval"
instead of "start timeout".

(cherry picked from commit ec264aa9b22dc919bb6d588e2dce13074f1c656b)
@AbelNavarro
Copy link

+1

@aspiers
Copy link
Member

aspiers commented Apr 5, 2016

+1 but what is up with this gate failure? Seems to be a common problem now, and I don't see anything on the Jenkins Green board for it.

@aspiers
Copy link
Member

aspiers commented Apr 5, 2016

Retriggered mkcloud rebuild to try to clear CI gate.

@AbelNavarro
Copy link

Failed CI looks unrelated to change. Retriggering.
Trello card for the jenkins failure.

@AbelNavarro
Copy link

CI still failing at the same point.

@aspiers
Copy link
Member

aspiers commented Apr 7, 2016

@AbelNavarro
Copy link

Retriggered CI. This time will investigate in the jenkins slave.

@aspiers
Copy link
Member

aspiers commented Apr 7, 2016

Gate will be fixed by SUSE-Cloud/cct#53

@aspiers aspiers merged commit ef116d5 into crowbar:release/tex/master Apr 7, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants