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

workflows: add trigger sentence in ci-verifier workflow file #23587

Merged
merged 1 commit into from Feb 6, 2023

Conversation

kaworu
Copy link
Member

@kaworu kaworu commented Feb 6, 2023

Had to look at the workflow file to find which trigger comment I needed to re-run the Datapath BPF Complexity CI job, so document it as we do for other jobs, e.g. 434d9f9.

This helps developers to know which trigger they can use to retest the
Datapath BPF Complexity workflow.

Signed-off-by: Alexandre Perrin <alex@isovalent.com>
@kaworu kaworu added kind/enhancement This would improve or streamline existing functionality. area/CI Continuous Integration testing issue or flake release-note/ci This PR makes changes to the CI. labels Feb 6, 2023
@kaworu kaworu requested review from a team as code owners February 6, 2023 08:17
@kaworu kaworu requested review from aanm and nebril February 6, 2023 08:17
Copy link
Member

@tklauser tklauser left a comment

Choose a reason for hiding this comment

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

Thanks

@tklauser tklauser removed the request for review from nebril February 6, 2023 10:06
@tklauser tklauser added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Feb 6, 2023
Copy link
Member

@pchaigno pchaigno left a comment

Choose a reason for hiding this comment

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

Thanks!

@pchaigno pchaigno merged commit aa51642 into cilium:master Feb 6, 2023
@kaworu kaworu deleted the pr/kaworu/ci-verifier-workflow-doc branch February 7, 2023 09:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/CI Continuous Integration testing issue or flake kind/enhancement This would improve or streamline existing functionality. ready-to-merge This PR has passed all tests and received consensus from code owners to merge. release-note/ci This PR makes changes to the CI.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants