Project

General

Profile

Actions

Bug #10034

closed

Teuthology: Calamari_setup cleanup task error

Added by Anonymous over 9 years ago. Updated over 9 years ago.

Status:
Closed
Priority:
Normal
Assignee:
-
Category:
-
% Done:

100%

Source:
Q/A
Tags:
Backport:
Regression:
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Crash signature (v1):
Crash signature (v2):

Description

After running calamari_setup, the following was observed.

[ubuntu@vpm164 ~]$ rpm -V calamari-server
.....UG.. /opt/calamari/webapp/calamari
.....UG.. /opt/calamari/webapp/calamari/.gitignore
.....UG.. /opt/calamari/webapp/calamari/manage.py
missing /var/lib/calamari
.....UG.. /var/lib/graphite
.....UG.. /var/lib/graphite/log
.....UG.. /var/lib/graphite/log/webapp
.....UG.. /var/lib/graphite/whisper
missing /var/log/calamari

I suppose we should get rid of all the calamari server stuff.


Subtasks 1 (0 open1 closed)

Feature #10181: teuthology-nuke: Facility to clean up after specific ceph-qa-suite tasks.Duplicate11/24/2014

Actions
Actions #1

Updated by Anonymous over 9 years ago

  • Project changed from 16 to teuthology

This is in the teuthology setup task.

Actions #2

Updated by Anonymous over 9 years ago

We probably should do the same thing that the calamari processes do. In this case, the teuthology task may be doing too much and it should leave the /var/lib and /var/log files alone.

Teuthology-nuke should probably be updated to remove all this stuff.

Actions #3

Updated by Anonymous over 9 years ago

Redmine Feature #10181 may affect this task.

Actions #4

Updated by Anonymous over 9 years ago

I think that nuke should remove the webapp/calamari and graphite packages. I was originally wondering how much we should remove, but since this will be run on testing machines, it's probably okay to remove these files if they are there.

Part of the reason why this bug was originally opened was due to confusion when testing teuthology calamari_setup. It may be reasonable to close this bug and implement all the changes in #10181

Actions #5

Updated by Anonymous over 9 years ago

  • Status changed from New to Duplicate

Duplicate of #10181

Actions #6

Updated by Anonymous over 9 years ago

def remove_installed_packages(ctx):

    dpkg_configure(ctx)
    config = {'project': 'ceph'}
    install_task.remove_packages(
        ctx,
        config,
        {"deb": install_task.deb_packages['ceph'] + ['salt-common', 'salt-minion'],
         "rpm": install_task.rpm_packages['ceph']})
    install_task.remove_sources(ctx, config)
    install_task.purge_data(ctx)

Question 1:
Should salt-common and salt-minion also be added for rpm?

Question 2:
Should calamari-server also be added to both of these packages?

Actions #7

Updated by Anonymous over 9 years ago

  • Status changed from Duplicate to In Progress
Actions #8

Updated by Anonymous over 9 years ago

  • Status changed from In Progress to Fix Under Review
Actions #9

Updated by Anonymous over 9 years ago

Change: f493c0b4ab2dec6b9a6ba5e30fa756bf8363eb28

PR: 372

wip-10034-wusui

Actions #10

Updated by Anonymous over 9 years ago

  • Status changed from Fix Under Review to Closed
Actions

Also available in: Atom PDF