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] Feature request: when uploading a rootfs image, don't allow it to override an existing upload (unless the user provides some command-line flag) #41590

Closed
DilumAluthge opened this issue Jul 15, 2021 · 1 comment · Fixed by #41591
Assignees
Labels
ci Continuous integration

Comments

@DilumAluthge
Copy link
Member

No description provided.

@DilumAluthge DilumAluthge added the ci Continuous integration label Jul 15, 2021
@DilumAluthge
Copy link
Member Author

DilumAluthge commented Jul 15, 2021

I'm envisioning something like this.

If the user does this:

julia llvm-passes.jl v1

Then the following happens:

  • If the v1 release does not exist, then the v1 is created, and the llvm-passes.tar.gz asset is uploaded to the v1 release.
  • If the v1 release exists, but the v1 release does not already have an asset by the name of llvm-passes.tar.gz, then the llvm-passes.tar.gz asset is uploaded to the v1 release.
  • If the v1 release exists, and the v1 release already has an asset by the name of llvm-passes.tar.gz, then we throw an error and we do not overwrite the old llvm-passes.tar.gz asset.

In contrast, if the user does this:

julia llvm-passes.jl v1 --overwrite

Then the following happens:

  • If the v1 release does not exist, then the v1 is created, and the llvm-passes.tar.gz asset is uploaded to the v1 release.
  • If the v1 release exists, but the v1 release does not already have an asset by the name of llvm-passes.tar.gz, then the llvm-passes.tar.gz asset is uploaded to the v1 release.
  • If the v1 release exists, and the v1 release already has an asset by the name of llvm-passes.tar.gz, then we overwrite the old llvm-passes.tar.gz asset with the new llvm-passes.tar.gz asset.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci Continuous integration
Projects
None yet
2 participants