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

gekios commented 4 years ago

Swapped the order of 'ceph'reset' and 'osd.remove' in order to provide a fix for bcs #1150168

Signed-off-by: Georgios Kyratsas gkyratsas@suse.com

Fixes # ceph.purge

Description: I have deployed a basic test for ceph.purge in order to have it in CI where purge was failing every single time (reported in bcs #1150168). So I worked with @swiftgist and he helped me find the issue.

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 88f10d236adf3b7f5f5f2c181305c9ed798bbfc0 is OK for suite deepsea:tier2. Check tests results in the Jenkins job: http://ci.ses.suse.de:8080/job/pr-deepsea/363/

jschmid1 commented 4 years ago

this should probably also end up in SES6. Please create a backport of this @gekios

gekios commented 4 years ago

Sure I will I just thought it would be better to first test/merge it on master.

gekios commented 4 years ago

I opened https://github.com/SUSE/DeepSea/pull/1796 to backport this to SES6.

SUSE / DeepSea

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