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

ceph-ansible suite in upstream point release validation process. #1938

Closed
yuriw opened this issue Sep 21, 2017 · 3 comments
Closed

ceph-ansible suite in upstream point release validation process. #1938

yuriw opened this issue Sep 21, 2017 · 3 comments
Assignees

Comments

@yuriw
Copy link

yuriw commented Sep 21, 2017

Awhile ago we included ceph-ansible suite to be part of any point release validation process.
Which means that during such validations ceph-ansible suite expected to pass.

For example for jewel v10.2.8 ceph-ansible suite passed as expected http://tracker.ceph.com/issues/19538#note-89

However, if you look at nighties runs now, this suite http://pulpito.ceph.com/?suite=ceph-ansible it's solidly red.

It raised several questions:

  1. Do we want to keep running this suite as part of nighties and point releases validations?
  2. Who is "babysitting" ceph-ansible suite on ongoing basis? See ref: http://tracker.ceph.com/projects/ceph-releases/wiki/HOWTO_monitor_the_automated_tests_AKA_nightlies
  3. If the answer to question 1 is YES , how do we make sure that ceph-ansible is using either stable or backwards compatible versions of ansible so tests don't fail due to environmental issues?
  4. If the answer to question 1 is NO, what alternative process do we use to validated ceph-ansible for releases if any?
@vasukulkarni
Copy link
Contributor

@yuriw

for 1.) Yes we have to keep running this, the reason we had so many issues was fast moving changes in ceph-ansible and trying to keep it up for master branch and then eventually we have to backport all those to luminous + jewel.
for jewel atleast we can try to fix the ceph-ansible branch to stable since we dont expect to carry over many of those master changes and the current test exists in jewel branch should be sufficient.

for 2.) we have had green runs in master for some time but I guess the purge cluster issue for which the PR exists is causing the tests to fail(ceph/teuthology#1112), For backports I am not really sure, probably you can take cherrypick those commits or add some more labels to make sure the backport team notices them.

  1. Both @andrewschoen and I agree we have to use latest 'master' branch for testing luminous/master of ceph, we can use stable branch for 'jewel'

@yuriw
Copy link
Author

yuriw commented Sep 22, 2017

One of the issues we need to agree on - how do we backport ceph-ansible related PRs fro example for jewel v10.2.10

I have a thoery why ceph-ansible issues are missed from backporting process, see http://tracker.ceph.com/projects/ceph-releases/wiki/HOWTO_start_backporting_a_stable_release
Suspect it's because we have no issues in the tracker.

What do you think @vasukulkarni @smithfarm @theanalyst @andrewschoen ?

@leseb leseb removed the question label Oct 11, 2017
@guits
Copy link
Collaborator

guits commented May 10, 2019

Closed due to inactivity, feel free to re-open if needed. Thanks!

@guits guits closed this as completed May 10, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

6 participants