Backport: srv/salt/ceph/purge: Changed order of runner calls

SUSE / DeepSea

A collection of Salt files for deploying, managing and automating Ceph.

GNU General Public License v3.0

161 stars 75 forks source link

Backport: srv/salt/ceph/purge: Changed order of runner calls #1796

Closed gekios closed 4 years ago

gekios commented 4 years ago

Fixes #

Description: Backport PR #1793 to SES6 branch in order to fix the bug bcs #1150168 regarding ceph.purge.

Checklist:

[ ] Added unittests and or functional tests
[ ] Adapted documentation
[x] Referenced issues or internal bugtracker
[ ] Ran integration tests successfully (trigger with "@susebot run teuthology" in a GitHub comment; see the wiki for more information)

gekios commented 4 years ago

@susebot run teuthology

susebot commented 4 years ago

Commit f553f6e003428910b4072506c116095e6b2e82bb is NOT OK for suite deepsea:tier2. Check tests results in the Jenkins job: http://ci.ses.suse.de:8080/job/pr-deepsea/368/

gekios commented 4 years ago

Failure shouldn't be related to this PR:


  target192168000034.teuthology:
    install ceph-iscsi dependencies: An error was encountered while installing package(s): Zypper command failure: Running scope as unit: run-r3eda6676e1074dfa858099ce2c4cfa05.scope
    enable tcmu-runner: The named service tcmu-runner is not available
    install ceph-iscsi: An error was encountered while installing package(s): Zypper command failure: Running scope as unit: run-rf242a89c13384720aa5666a38d526043.scope
    enable rbd-target-api: The named service rbd-target-api is not available```

jschmid1 commented 4 years ago

that's actually a failure that should be analyzed by the CI team.. tcmu-runner/ceph-iscsi should be present on the repos.

jschmid1 commented 4 years ago

This however won't block the merge.

gekios commented 4 years ago

Yep actually I am checking it cause I see that both the PR in master that passed and this one use the same repos so it's weird why it fails...