Bug #9975
closedmachines are leaving behind a ~/.cephdeployconf with custom repo data
0%
Description
ceph-deploy has a mechanism to create custom repositories based on configuration files
that can be saved in $HOME/.cephdeployconf or in the current working directory.
Specifically for ICE, we use this one feature to point to where repositories and the release.asc files are.
If we are no longer testing/using ICE and just doing a teuthology run, that file should be removed to avoid failures like:
2014-10-30T14:26:27.080 INFO:teuthology.orchestra.run.plana58.stderr:[plana58][INFO ] Running command: sudo wget -O release.asc http://plana58.front.sepia.ceph.com/static/ceph/0.80.7/release.asc 2014-10-30T14:26:27.096 INFO:teuthology.orchestra.run.plana58.stderr:[plana58][WARNING] --2014-10-30 14:26:27-- http://plana58.front.sepia.ceph.com/static/ceph/0.80.7/release.asc 2014-10-30T14:26:27.097 INFO:teuthology.orchestra.run.plana58.stderr:[plana58][WARNING] Resolving plana58.front.sepia.ceph.com (plana58.front.sepia.ceph.com)... 10.214.132.20 2014-10-30T14:26:27.098 INFO:teuthology.orchestra.run.plana58.stderr:[plana58][WARNING] Connecting to plana58.front.sepia.ceph.com (plana58.front.sepia.ceph.com)|10.214.132.20|:80... failed: Connection refused. 2014-10-30T14:26:27.098 INFO:teuthology.orchestra.run.plana58.stderr:[plana58][WARNING] command returned non-zero exit status: 4 2014-10-30T14:26:27.100 INFO:teuthology.orchestra.run.plana58.stderr:[plana58][INFO ] Running command: sudo apt-key add release.asc 2014-10-30T14:26:27.135 INFO:teuthology.orchestra.run.plana58.stderr:[plana58][WARNING] gpg: no valid OpenPGP data found. 2014-10-30T14:26:27.136 INFO:teuthology.orchestra.run.plana58.stderr:[plana58][ERROR ] RuntimeError: command returned non-zero exit status: 2 2014-10-30T14:26:27.136 INFO:teuthology.orchestra.run.plana58.stderr:[ceph_deploy][ERROR ] RuntimeError: Failed to execute command: apt-key add release.asc
The above looks like a normal network error, but it isn't: the location of the release.asc file is incorrect (although in the usual spot for ICE repos).
A bit above those log lines ceph-deploy does mention that has found a valid cephdeploy.conf file with a valid custom repository that will be used:
2014-10-30T14:26:26.949 INFO:teuthology.orchestra.run.plana58.stderr:[ceph_deploy.conf][DEBUG ] found configuration file at: /home/ubuntu/.cephdeploy.conf 2014-10-30T14:26:26.950 INFO:teuthology.orchestra.run.plana58.stderr:[ceph_deploy.cli][INFO ] Invoked (1.5.19): ./ceph-deploy install --dev=dumpling plana58 burnupi14 plana78 2014-10-30T14:26:26.950 INFO:teuthology.orchestra.run.plana58.stderr:[ceph_deploy.install][DEBUG ] Installing dev version dumpling on cluster ceph hosts plana58 burnupi14 plana78 2014-10-30T14:26:26.950 INFO:teuthology.orchestra.run.plana58.stderr:[ceph_deploy.install][DEBUG ] Detecting platform for host plana58 ... 2014-10-30T14:26:27.014 INFO:teuthology.orchestra.run.plana58.stderr:[plana58][DEBUG ] connection detected need for sudo 2014-10-30T14:26:27.047 INFO:teuthology.orchestra.run.plana58.stderr:[plana58][DEBUG ] connected to host: plana58 2014-10-30T14:26:27.048 INFO:teuthology.orchestra.run.plana58.stderr:[plana58][DEBUG ] detect platform information from remote host 2014-10-30T14:26:27.075 INFO:teuthology.orchestra.run.plana58.stderr:[plana58][DEBUG ] detect machine type 2014-10-30T14:26:27.078 INFO:teuthology.orchestra.run.plana58.stderr:[ceph_deploy.install][INFO ] Distro info: Ubuntu 12.04 precise 2014-10-30T14:26:27.078 INFO:teuthology.orchestra.run.plana58.stderr:[plana58][INFO ] installing ceph on plana58 2014-10-30T14:26:27.078 INFO:teuthology.orchestra.run.plana58.stderr:[ceph_deploy.install][INFO ] detected valid custom repositories from config file 2014-10-30T14:26:27.078 INFO:teuthology.orchestra.run.plana58.stderr:[ceph_deploy.install][INFO ] will use default repository: ceph
The solution is for the cleanup/purging state (I think there is one?) for machines should also remove that file, or at the very least ensure that
file is not there when running a test.
Updated by Sage Weil over 9 years ago
nuke.py should be updated to clean this up too
Updated by Alfredo Deza over 9 years ago
- Status changed from 12 to Fix Under Review
- Assignee set to Alfredo Deza
Pull request opened https://github.com/ceph/teuthology/pull/353
Updated by Alfredo Deza over 9 years ago
- Status changed from Fix Under Review to Resolved
merged commit 739a881 into master