In case /var/lib/nova/instances resides on NFS we have seen migrations
failing with 'Failed to get "write" lock - Is another process using the
image' errors.
This has been tracked down to grace/lease timeouts not having expired
before attempting the migration/evacuate, so in this cases it might be
desirable to delay the nova evacuate call to give the storage time to
release the locks.
Change-Id: Ie2fe784202d754eda38092479b1ab3ff4d02136a
Resolves: rhbz#1740069
In all the chaos of complicated patch unentanglement and rebasing,
I forgot to actually even syntax-check the thing :-(
Change-Id: Ib0a31efe2ff75fc55cf67d4de73e74ebafb219b0
Apache-2.0 is the recommended license for OpenStack Big Tent
projects (see https://governance.openstack.org/reference/licensing.html)
and this simplifies the licensing of the overall git repo
quite a bit by removing an exception clause.
Change-Id: I827eb91fd18ced1848439d573cfe6df16ed27748
Closes-Bug: #1564844
Make it clearer what the risks of not using shared storage are.
Information is based on:
http://docs.openstack.org/user-guide-admin/cli_nova_evacuate.html
which says "The command rebuilds the instance from the original image or
volume" but later says that "To preserve the user disk data on the
evacuated server, deploy Compute with a shared file system" and then use
--on-shared-storage.
Change-Id: I09600414eb0d7fff1cf301b11b3fa9a76fc08c77
This commit fixes bashate E002 (indents are only spaces, and not hard
tabs) violations
Partial-Bug: #1550203
Change-Id: I7d156d47023781be74e6fa8daef6ffc311b55d9d
Signed-off-by: Norbert Illes <norbert.e.illes@ericsson.com>
These can be quite useful in some setups.
This depends on https://github.com/ClusterLabs/fence-agents/pull/37
Change-Id: I2cfef0a4bf7f94f74041c8fee236788c7a110cc5
Signed-off-by: Vincent Untz <vuntz@suse.com>
When no evacuation has been done yet, we're spamming syslog with:
Could not query value of evacuate: attribute does not exist
So let's just filter this out, since it's known to be expected on
initial setup.
As this requires a bashism, also move the script to use bash.
Change-Id: I3351919febc0ef0101e4a08ce6eb412e3c7cfc76
This commit removes hopefully all trailing whitespaces from all files in
the repository.
Change-Id: I248160ee767e13c34c1cd19745c5496a4bcbba0b
Signed-off-by: Norbert Illes <norbert.e.illes@ericsson.com>
attrd_updater deprecated usage of -v a long time ago in favour of -U:
927660ba0c
so change the agent to use the documented option.
Change-Id: I4e19ef3e22c822bc7a6da98f684d67988367ef25
Signed-off-by: Adam Spiers <aspiers@suse.com>
attrd_updater -D doesn't remove attributes; it just sets them to the
empty string. Make sure this is handled correctly in case anyone
invokes attrd_updater manually in order to mark a host as no longer
needing evacuation.
Change-Id: I0c05e92c706555459adff73dbc46891fe72d54b9
Signed-off-by: Adam Spiers <aspiers@suse.com>
OCF RAs are not meant to be run directly, so for consistency with the
other RAs in this repo, remove the executable bits from these two files.
Change-Id: I35955089cbff0a73f3cd48fb1febfc05bc479dc1
Signed-off-by: Adam Spiers <aspiers@suse.com>
This requires the same fix to be applied to fence_compute:
https://github.com/ClusterLabs/fence-agents/pull/27
and is a more complete version of
33154a6680
which only patched NovaEvacuate and not NovaCompute.
Signed-off-by: Adam Spiers <aspiers@suse.com>
Change-Id: Iea778d6455341275303e592a09bca7d6db120950