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 assets release #3178

Merged
merged 1 commit into from
Apr 30, 2023
Merged

fix assets release #3178

merged 1 commit into from
Apr 30, 2023

Conversation

deitch
Copy link
Contributor

@deitch deitch commented Apr 30, 2023

The assets release workflow was breaking, see https://github.com/lf-edge/eve/actions/runs/4840159952

The cause was that the checkout action was happening after some of the assets were created. checkout can wipe out the contents of the current directory.

This does several things:

  1. Moves checkout to the first step
  2. Removes the rm -rf assets && mkdir -p assets to its own step, so it is very clear when and where it is happening
  3. Adds assets/ to .gitignore, so we do not have any issues with an apparently dirty git repo after checkout and then creating things in assets.
  4. Uses the existing checkout in the step Pull the EVE release from DockerHUB or build it rather than trying to check out again
  5. Ensures that the one step that did a cd assets (Pull the EVE release from DockerHUB or build it) works out of root and explicitly paths things to assets/

Signed-off-by: Avi Deitcher <avi@deitcher.net>
Copy link
Contributor

@eriknordmark eriknordmark left a comment

Choose a reason for hiding this comment

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

LGTM

Is there a way to test the assets and publish workflow, which only run on master and release branches, short of merging the changes?

@deitch
Copy link
Contributor Author

deitch commented Apr 30, 2023

Nope. I've wanted that for a long time. I've even asked contacts at GitHub over the years. No such luck.

@eriknordmark eriknordmark merged commit 1ed3eec into lf-edge:master Apr 30, 2023
@deitch deitch deleted the fix-assets branch May 1, 2023 07:24
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.

2 participants