always raise exit after handling an exception

Signed-off-by: Alfredo Deza <alfredo.deza@inktank.com>

revert scripts/ceph-deploy

Signed-off-by: Alfredo Deza <alfredo.deza@inktank.com>

rename main to _main to avoid fixing scripts

Signed-off-by: Alfredo Deza <alfredo.deza@inktank.com>

Merge pull request #231 from alfredodeza/7594

add a finally block to close stderr/stdout

do not if/else format_exc() results

Signed-off-by: Alfredo Deza <alfredo.deza@inktank.com>

move the stderr/stdout closing to the ceph-deploy executable

Signed-off-by: Alfredo Deza <alfredo.deza@inktank.com>

override the safe default for cli.main()

Signed-off-by: Alfredo Deza <alfredo.deza@inktank.com>

make sure we can still exit with proper exit codes

Signed-off-by: Alfredo Deza <alfredo.deza@inktank.com>

only close stderr if we are not testing

Signed-off-by: Alfredo Deza <alfredo.deza@inktank.com>

ensure CEPH_DEPLOY_TEST is set in tox.ini

Signed-off-by: Alfredo Deza <alfredo.deza@inktank.com>

add a finally block to close stderr/stdout

Signed-off-by: Alfredo Deza <alfredo.deza@inktank.com>

1.5.11

update the changelog

Signed-off-by: Alfredo Deza <alfredo.deza@inktank.com>

bump to 1.5.11

Signed-off-by: Alfredo Deza <alfredo.deza@inktank.com>

remove some debugging print statements

Signed-off-by: Alfredo Deza <alfredo.deza@inktank.com>

Updates to the install.py rpm_dist() and repository_url_part()

to remove type casting and using the native ints in the normalized_release
object.

Signed-off-by: Tom Walsh <tom.walsh@expresshosting.net>
Closes #230

Modified the rpm_dist() and repository_url_part()

So they can use the newly defined distro.normalized_release object to determine
the correct OS version and return the information based on that. This should
work for all releases from RHEL, CentOS, and Scientific Linux 6 and greater.

Changed the test_centos.py file to use pytest fixtures to test multiple input
types for the various OSes we are testing for.

Signed-off-by: Tom Walsh <tom.walsh@expresshosting.net>

Modified the tests to reflect the desired output of normalize_release()

Added normalize_release() to the centos object so that the tests can properly execute.

Added tests for the normalize_release function.

Signed-off-by: Tom Walsh <tom.walsh@expresshosting.net>

Modified the system to handle subversion numbers in the release number. For instance in production our server was reporting a release version of 7.0.1406. This is not a valid float() so the system was returning an el6 value. I have added additional logic to find the first . (dot) in the string and convert that to a float value.

I have also added a test to look for and test this corner case.

Added new line at the end of the file.

Signed-off-by: Tom Walsh <tom.walsh@expresshosting.net>

Reworked the release detection to normalize the value of distro.release as an integer and use that as the basis of determining the correct value of rpm_dist() and repository_url_part(). This should work for all versions of RH, CentOS, and SL going forward.

Signed-off-by: Tom Walsh <tom.walsh@expresshosting.net>

Fine tuned the tests to catch some more corner cases.

Adjusted the logic in install.py to match what was already there.

Signed-off-by: Tom Walsh <tom.walsh@expresshosting.net>

Added checks to the CentOS install scripts to properly detect CentOS 7 and set the correct repo information.

Added tests to confirm that the changes to CentOS detection are working correctly.

Signed-off-by: Tom Walsh <tom.walsh@expresshosting.net>