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

Add initial Zuul jobs #102

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Conversation

mnaser
Copy link

@mnaser mnaser commented Jul 10, 2024

No description provided.

@EmilienM
Copy link
Contributor

I could not join the synchronous discussion around that but I just saw this and discussed on Slack with @mnaser . I'm not in favour of adding Zuul to the CI here.
It might be useful for cross repo checks, but the only repo that we would cross-test is Gophercloud. I don't think we'll ever bring back Zuul to Gophercloud CI. We left it years ago (it was OpenLab) because it was always broken and very slow (one hour per job) and now we use Gihut Action, and it's very fast (jobs take no more than 15 minutes total and there are almost no flakes).

So for what my vote is worth, I'm not sure about this PR and I want to know what the plan is on adding Zuul back to the Gophercloud/ORC eco system.

@mnaser
Copy link
Author

mnaser commented Jul 10, 2024

It might be useful for cross repo checks, but the only repo that we would cross-test is Gophercloud. I don't think we'll ever bring back Zuul to Gophercloud CI. We left it years ago (it was OpenLab) because it was always broken and very slow (one hour per job) and now we use Gihut Action, and it's very fast (jobs take no more than 15 minutes total and there are almost no flakes).

FTR, we don't need to install Zuul in Gophercloud to do cross-repo checks. It can work without it installed there.

@pierreprinetti
Copy link
Member

I think we have nothing to lose in investigating the Zuul option and what it can bring to the project. The point here is not to immediately replace the existing CI, which I think is well-dimensioned for ORC at this stage, but rather to enable new ways of testing, for example, the compatibility with other tools compared to Devstack and Kind.

@mnaser
Copy link
Author

mnaser commented Jul 10, 2024

Alright, if that's the case, I'll look at adding the DevStack jobs next as well and the rest.

@mnaser
Copy link
Author

mnaser commented Jul 17, 2024

Did a bit of checking, we'll have to leverage devstack-tox-functional-consumer .. but without tox, but we can leverage the roles easily.

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

Successfully merging this pull request may close these issues.

3 participants