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

ci: only add /test .. comment if the branch for the PR matches #4070

Merged
merged 1 commit into from
Aug 22, 2023

Conversation

nixpanic
Copy link
Member

By adding an if-statement for each step of the matrix job, only those steps are executed where the base ref of the PR matches the branch in the matrix parameters.


Show available bot commands

These commands are normally not required, but in case of issues, leave any of
the following bot commands in an otherwise empty comment in this PR:

  • /retest ci/centos/<job-name>: retest the <job-name> after unrelated
    failure (please report the failure too!)

By adding an if-statement for each step of the matrix job, only those
steps are executed where the base ref of the PR matches the branch in
the matrix parameters.

Signed-off-by: Niels de Vos <[email protected]>
@nixpanic nixpanic added ci/skip/e2e skip running e2e CI jobs ci/skip/multi-arch-build skip building on multiple architectures labels Aug 22, 2023
@mergify mergify bot added the component/testing Additional test cases or CI work label Aug 22, 2023
@nixpanic
Copy link
Member Author

/test ci/centos/k8s-e2e-external-storage

@nixpanic
Copy link
Member Author

/test ci/centos/mini-e2e-helm

@nixpanic
Copy link
Member Author

/test ci/centos/mini-e2e

@nixpanic
Copy link
Member Author

/test ci/centos/upgrade-tests

@Madhu-1 Madhu-1 requested a review from a team August 22, 2023 11:23
@mergify mergify bot merged commit ee843e6 into ceph:devel Aug 22, 2023
14 checks passed
@Madhu-1
Copy link
Collaborator

Madhu-1 commented Aug 22, 2023

@nixpanic we need to adjust the mergify rules as per the new change. for now i started some tests manually in some PR as mergify still needs all checks to be passed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci/skip/e2e skip running e2e CI jobs ci/skip/multi-arch-build skip building on multiple architectures component/testing Additional test cases or CI work
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants