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

CI: Add test for action.yaml to prevent breakage in main #153

Open
joestringer opened this issue Jan 25, 2024 · 1 comment
Open

CI: Add test for action.yaml to prevent breakage in main #153

joestringer opened this issue Jan 25, 2024 · 1 comment
Labels
enhancement New feature or request

Comments

@joestringer
Copy link
Member

joestringer commented Jan 25, 2024

PR #133 (commit 218de8b) broke the way that action.yaml runs VMs due to the new pull --cache feature. To discover that this was broken, we had to merge the PR, release a new version of LVH, and update cilium/cilium to invoke the new action.yaml. Furthermore, in order to test the fix, it seems like we also need to merge the PR (not yet knowing if it fixes everything), release a new LVH, and update cilium/cilium to invoke the new action.yaml.

It'd be an easier developer experience with lower likelihood to break things if we could push a branch to this repo with a PR, then run the action.yaml + proposed code in the branch before merging. Note that if we implement this, we should be careful not to expose the ability to write untrusted images to official LVH images if a drive-by contributor submits a PR from a fork.

@joestringer joestringer added the enhancement New feature or request label Jan 25, 2024
@joestringer
Copy link
Member Author

joestringer commented Jan 25, 2024

For reference, I was able to test a new version of the LVH Go code (NB: not the action.yaml code) by pushing a PR to cilium/cilium with extra steps to manually pull & build a target version of LVH. This did not require merging the PR or releasing a new version of LVH. Perhaps we could make a similar action in this repo based on this idea. See this link:

https://github.com/cilium/cilium/pull/30447/files

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant