Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[tmp] fix to run the ci against #1748

Closed
wants to merge 1 commit into from
Closed

[tmp] fix to run the ci against #1748

wants to merge 1 commit into from

Conversation

jschmid1
Copy link
Contributor

Signed-off-by: Joshua Schmid jschmid@suse.de

see #1747 for more infos


Checklist:

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

Signed-off-by: Joshua Schmid <jschmid@suse.de>
@jschmid1 jschmid1 added the DNM label Sep 16, 2019
@jschmid1
Copy link
Contributor Author

@susebot run teuthology

@susebot
Copy link
Collaborator

susebot commented Sep 16, 2019

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

@jschmid1
Copy link
Contributor Author

That seem to fix the issues we've seen in the disk replacement and rebuild tests. I'll come up with a proper fix that's also backwards compatible

@jan--f
Copy link
Contributor

jan--f commented Sep 17, 2019

hmm I think this might only mask the issue. ceph-volume in 14.2.3 actually fails to zap disks when run through salt. will test a fix today.

@jschmid1
Copy link
Contributor Author

skipped 14.2.3 -> now using 14.2.4 which fixed the issues @jan--f described. closing

@jschmid1 jschmid1 closed this Sep 18, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants