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

doc: remove /retest all command for Jenkins jobs #3957

Merged
merged 1 commit into from
Jul 4, 2023

Conversation

nixpanic
Copy link
Member

@nixpanic nixpanic commented Jun 29, 2023

/retest all causes a spike in resource consumption in Jenkins and the OpenShift cluster kills the Pod. That means tests are not fully running yet, and results never arrive back in the PR. Instead of /retest all, the ok-to-test label can be used to trigger required tests with a slight delay between each command.

Depends-on: #3965


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!)
  • /retest all: run this in case the CentOS CI failed to start/report any test
    progress or results

@mergify mergify bot added ci/skip/e2e skip running e2e CI jobs ci/skip/multi-arch-build skip building on multiple architectures component/docs Issues and PRs related to documentation labels Jun 29, 2023
@Madhu-1 Madhu-1 requested a review from a team June 29, 2023 11:09
@nixpanic
Copy link
Member Author

nixpanic commented Jul 3, 2023

@Mergifyio queue

@mergify
Copy link
Contributor

mergify bot commented Jul 3, 2023

queue

🛑 The pull request has been removed from the queue default

The queue conditions cannot be satisfied due to failing checks.

You can take a look at Queue: Embarked in merge train 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.

@nixpanic nixpanic force-pushed the doc/jobs/remove-retest-all branch from d397025 to ba733b1 Compare July 3, 2023 15:39
@mergify mergify bot added the ok-to-test Label to trigger E2E tests label Jul 3, 2023
@ceph-csi-bot
Copy link
Collaborator

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

@ceph-csi-bot
Copy link
Collaborator

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

@ceph-csi-bot
Copy link
Collaborator

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

@ceph-csi-bot
Copy link
Collaborator

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

@ceph-csi-bot
Copy link
Collaborator

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

@ceph-csi-bot
Copy link
Collaborator

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

@ceph-csi-bot
Copy link
Collaborator

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

@ceph-csi-bot
Copy link
Collaborator

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

@ceph-csi-bot
Copy link
Collaborator

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

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/upgrade-tests-cephfs

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/upgrade-tests-rbd

@ceph-csi-bot ceph-csi-bot removed the ok-to-test Label to trigger E2E tests label Jul 3, 2023
@nixpanic
Copy link
Member Author

nixpanic commented Jul 3, 2023

@Mergifyio queue

Oh, I thought this was a PR for the ci/centos branch...

@nixpanic
Copy link
Member Author

nixpanic commented Jul 4, 2023

@Mergifyio rebase

@mergify
Copy link
Contributor

mergify bot commented Jul 4, 2023

rebase

✅ Nothing to do for rebase action

@nixpanic
Copy link
Member Author

nixpanic commented Jul 4, 2023

Needs rebasing once #3965 is merged.

@nixpanic
Copy link
Member Author

nixpanic commented Jul 4, 2023

@Mergifyio rebase

`/retest all` causes a spike in resource consumption in Jenkins and the
OpenShift cluster kills the Pod. That means tests are not fully running
yet, and results never arrive back in the PR. Instead of `/retest all`,
the `ok-to-test` label can be used to trigger required tests with a
slight delay between each command.

Signed-off-by: Niels de Vos <ndevos@ibm.com>
@mergify
Copy link
Contributor

mergify bot commented Jul 4, 2023

rebase

✅ Branch has been successfully rebased

@nixpanic nixpanic force-pushed the doc/jobs/remove-retest-all branch from ba733b1 to 755b607 Compare July 4, 2023 11:29
@nixpanic
Copy link
Member Author

nixpanic commented Jul 4, 2023

@Mergifyio queue

@mergify
Copy link
Contributor

mergify bot commented Jul 4, 2023

queue

✅ The pull request has been merged automatically

The pull request has been merged automatically at 489d0ff

@mergify mergify bot added the ok-to-test Label to trigger E2E tests label Jul 4, 2023
@ceph-csi-bot
Copy link
Collaborator

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

@ceph-csi-bot
Copy link
Collaborator

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

@ceph-csi-bot
Copy link
Collaborator

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

@ceph-csi-bot
Copy link
Collaborator

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

@ceph-csi-bot
Copy link
Collaborator

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

@ceph-csi-bot
Copy link
Collaborator

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

@ceph-csi-bot
Copy link
Collaborator

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

@ceph-csi-bot
Copy link
Collaborator

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

@ceph-csi-bot
Copy link
Collaborator

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

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/upgrade-tests-cephfs

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/upgrade-tests-rbd

@ceph-csi-bot ceph-csi-bot removed the ok-to-test Label to trigger E2E tests label Jul 4, 2023
@mergify mergify bot merged commit 489d0ff into ceph:devel Jul 4, 2023
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/docs Issues and PRs related to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants