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: Revert changes to comment-triggered workflows #17173

Merged

Conversation

pchaigno
Copy link
Member

This commit reverts 98e6ea4 ("workflows: Improve the change check for issue_comment triggers").

That issue and its tentative fix are breaking comment-triggered workflows. In the latest version, it was failing because git rev-parse needs the pull request's branch to be fetched to work.

This commit reverts 98e6ea4 ("workflows: Improve the change check for
issue_comment triggers").

That issue and its tentative fix are breaking comment-triggered
workflows. In the latest version, it was failing because 'git rev-parse'
needs the pull request's branch to be fetched to work.

Signed-off-by: Paul Chaignon <paul@cilium.io>
@pchaigno pchaigno added kind/bug/CI This is a bug in the testing code. area/CI Continuous Integration testing issue or flake release-note/ci This PR makes changes to the CI. labels Aug 17, 2021
@pchaigno pchaigno requested review from a team as code owners August 17, 2021 08:51
@pchaigno pchaigno requested a review from tklauser August 17, 2021 08:51
@pchaigno pchaigno requested a review from aanm August 17, 2021 08:51
@aanm aanm merged commit bfb19a5 into cilium:master Aug 17, 2021
@pchaigno pchaigno deleted the revert-comment-triggered-workflow-changes branch August 17, 2021 09:02
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/bug/CI This is a bug in the testing code. 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

3 participants