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
gha: fix conformance-ginkgo base branch retrieval #26085
Conversation
e6f367a
to
c1b6017
Compare
Let's make sure that we use the correct URL to retrieve the PR information also when the workflow is triggered by a pull_request event. Signed-off-by: Marco Iorio <marco.iorio@isovalent.com>
Retrieving the base branch from github.base_branch does not work when the action is triggered by a comment, because that field is not populated. Let's retrieve it from the json information about the PR. Signed-off-by: Marco Iorio <marco.iorio@isovalent.com>
c6d5c01
to
d032acd
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me, thanks for fixing this!
The matrix got generated correctly: https://github.com/cilium/cilium/actions/runs/5224161919/jobs/9432015821?pr=26085. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Retrieving the base branch from
github.base_branch
does not work when the action is triggered by a comment, because that field is not populated. Let's retrieve it from the json information about the PR. Additionally, let's make sure that we use the correct URL to retrieve the PR information also when the workflow is triggered by a pull_request event.Link to running workflow: https://github.com/cilium/cilium/actions/runs/5224161919/jobs/9432015821?pr=26085