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

Fix GHA CI for PRs from external contributors #2486

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

jepio
Copy link
Member

@jepio jepio commented Nov 28, 2024

Fix GHA CI for PRs from external contributors

GHA CI fails for PRs from external contributors because in devcontainer tests, emerge-gitclone can't fetch the right ref from the upstream repository. We can make this work by:

  1. switching to encoding a full sha in the FLATCAR_VERSION string in GHA
  2. using that full sha with git fetch origin <sha>; git checkout FETCH_HEAD in emerge-gitclone

This PRs does "1." and switches to using a branch of flatcar-dev-util that does "2." (https://github.com/flatcar/flatcar-dev-util/compare/jepio/fetch-head - PR will follow if this works).

How to use

[ describe what reviewers need to do in order to validate this PR ]

Testing done

[Describe the testing you have done before submitting this PR. Please include both the commands you issued as well as the output you got.]

  • Changelog entries added in the respective changelog/ directory (user-facing change, bug fix, security fix, update)
  • Inspected CI output for image differences: /boot and /usr size, packages, list files for any missing binaries, kernel modules, config files, kernel modules, etc.

Pulls in a commit in emerge-gitclone, that adapts emerge-gitclone to work for
commits for PRs from external contributors.

Signed-off-by: Jeremi Piotrowski <[email protected]>
This gives emerge-gitclone the full sha so that it can fetch by hash from the
remote repository, which lets it succeed for PRs from external contributors.

Signed-off-by: Jeremi Piotrowski <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant