Download and install hpacucli with package on debian instead of convert it from rpm
tests: Adding rights on config/hw

This is required to get the .hw upload working.
common: Cleaning mount point if fatal_error occurs

When getting a fatal_error we shall clean all existing mount points.
Let's create a specific function to handle this and call it from
fatal_error() but also from cleanup()
packages: Using RELEASE if DIST isn't setup

When calling packages functions for the upgrade script, DIST is not
setup while RELEASE is. Let's use RELEASE as a workaround in such
configuration.
Merge remote-tracking branch 'dsavineau/hpacucli_remove_alien'
upgrade-from: Using update_repositories

It's better using functions from packages instead of doing it manually
*.install: Setup update repo once for all

Many roles where defining update & security repositories while it was
already done by base.install. Let's remove all this code duplication
mysql-upgrade: Removing update repository definition

Shall have been part of commit c07282c5cb68aeacb6bc4f99f101ed368b4aea43
upgrade-from.d: Updating scheme to uses packages

Since we do have packages functions, let's use them in upgrade-from.d.
That make the code easier to read & much more robust.
packages: Using -y on upgrade

We got used to use -y on some scripts so let's use it also in the
generic function.
common: Print command & return code if trap

If we enter a trap, let's print the faulty command and its return code.
Very useful for debugging
common: Adding clear-trap()

When running upgrade-from, we need to disable the trap unless it occurs
at the end of the script.
haproxy: Removing repositories definition

That's already done in base.install
{haproxy,galera}.install: escape do_chroot args
get_mirror($dist): get best mirror for dist

returns REPOSITORY if defined, otherwise try to guess
a proper mirror.

Conflicts:
	build/base.install
openstack-common: escaping quotes for do_chroot

Required to insure that all arguments are well parsed
ceph: fix apt-key for ceph
common: Don't be verbose is ret=0

If ret=0, that's a normal exit case, there is no need to print debbuging
messages.
packages: fix get_mirror function (put case in else statement)