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.

Sign up for GitHub

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

Jump to bottom

ci: update mergify rules for K8s v1.32 #5050

Merged
merged 2 commits into from
Jan 6, 2025
Merged

ci: update mergify rules for K8s v1.32 #5050

merged 2 commits into from
Jan 6, 2025

Conversation

iPraveenParihar
Copy link
Contributor

@iPraveenParihar iPraveenParihar commented Jan 6, 2025 •
edited by nixpanic
Loading

Describe what this PR does

ci: update mergify rules for K8s v1.32

Related issues

Updates: #5015

Checklist:

  • Commit Message Formatting: Commit titles and messages follow guidelines in the developer guide.
  • Reviewed the developer guide on Submitting a Pull Request
  • Pending release notes updated with breaking and/or notable changes for the next major release.
  • Documentation has been updated, if necessary.
  • Unit tests have been added, if necessary.
  • Integration tests have been added, if necessary.
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!)

@iPraveenParihar iPraveenParihar added the ci/skip/multi-arch-build skip building on multiple architectures label Jan 6, 2025
@iPraveenParihar iPraveenParihar self-assigned this Jan 6, 2025
@iPraveenParihar iPraveenParihar mentioned this pull request Jan 6, 2025
Open
8 tasks
@mergify mergify bot added component/testing Additional test cases or CI work ci/skip/e2e skip running e2e CI jobs Repo activity Process/activites on ceph-csi repo labels Jan 6, 2025
@iPraveenParihar iPraveenParihar marked this pull request as ready for review January 6, 2025 08:50
nixpanic
nixpanic reviewed Jan 6, 2025
View reviewed changes
Copy link
Member

@nixpanic nixpanic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Kubernetes 1.29 is only supported for 2 more months. I think we can drop that version from the devel branch now too. It'll save quite some CI load.

nixpanic
nixpanic reviewed Jan 6, 2025
View reviewed changes
.github/workflows/pull-request-commentor.yaml Outdated
@@ -19,7 +19,7 @@ jobs:
strategy:
matrix:
branch: [release-v3.12, release-v3.13, devel]
k8s: ["1.29", "1.30", "1.31", "1.32"]
k8s: ["1.30", "1.31", "1.32"]
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

1.29 is still needed for the current release-v3.x branches.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ahh.. Yea
thanks!

@nixpanic nixpanic requested a review from a team January 6, 2025 10:46
@nixpanic
Copy link
Member

nixpanic commented Jan 6, 2025

@Mergifyio queue

@mergify Mergify
Copy link
Contributor

mergify bot commented Jan 6, 2025 •
edited
Loading

queue

🛑 The pull request has been removed from the queue default

The merge conditions cannot be satisfied due to failing checks.

You can take a look at Queue: Embarked in merge queue check runs for more details.

In case of a failure due to a flaky test, you should first retrigger the CI.
Then, re-embark the pull request into the merge queue by posting the comment
@mergifyio refresh on the pull request.

@mergify mergify bot added the ok-to-test Label to trigger E2E tests label Jan 6, 2025
@ceph-csi-bot
Copy link
Collaborator

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

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/mini-e2e-helm/k8s-1.30

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/upgrade-tests-cephfs

@ceph-csi-bot
Copy link
Collaborator

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

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/mini-e2e/k8s-1.30

@ceph-csi-bot
Copy link
Collaborator

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

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/upgrade-tests-rbd

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/mini-e2e-helm/k8s-1.29

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/mini-e2e-helm/k8s-1.31

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/mini-e2e/k8s-1.29

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/mini-e2e/k8s-1.31

@ceph-csi-bot ceph-csi-bot removed the ok-to-test Label to trigger E2E tests label Jan 6, 2025
@mergify Mergify
Copy link
Contributor

mergify bot commented Jan 6, 2025

This pull request has been removed from the queue for the following reason: checks failed.

The merge conditions cannot be satisfied due to failing checks:

You should look at the reason for the failure and decide if the pull request needs to be fixed or if you want to requeue it.

If you want to requeue this pull request, you need to post a comment with the text: @mergifyio requeue

@nixpanic
Copy link
Member

nixpanic commented Jan 6, 2025

/retest ci/centos/mini-e2e-helm/k8s-1.29

@nixpanic
Copy link
Member

nixpanic commented Jan 6, 2025

@Mergifyio requeue

ci/centos/mini-e2e-helm/k8s-1.29 failed to clone the repository from GitHub

@mergify Mergify
Copy link
Contributor

mergify bot commented Jan 6, 2025

requeue

✅ The queue state of this pull request has been cleaned. It can be re-embarked automatically

@mergify mergify bot merged commit 81d4f61 into ceph:devel Jan 6, 2025
38 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Madhu-1 Madhu-1 Madhu-1 approved these changes

@nixpanic nixpanic nixpanic approved these changes

Assignees

@iPraveenParihar iPraveenParihar

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 Repo activity Process/activites on ceph-csi repo
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@iPraveenParihar @nixpanic @ceph-csi-bot @Madhu-1