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

minikube_latest_amd64.deb accidentally included in Github release assets #9341

Closed
thetillhoff opened this issue Sep 28, 2020 · 7 comments · Fixed by #9351
Closed

minikube_latest_amd64.deb accidentally included in Github release assets #9341

thetillhoff opened this issue Sep 28, 2020 · 7 comments · Fixed by #9351
Labels
area/build-release good first issue Denotes an issue ready for a new contributor, according to the "help wanted" guidelines. help wanted Denotes an issue that needs help from a contributor. Must meet "help wanted" guidelines. kind/bug Categorizes issue or PR as related to a bug. priority/backlog Higher priority than priority/awaiting-more-evidence.

Comments

@thetillhoff
Copy link
Contributor

thetillhoff commented Sep 28, 2020

When checking through the release assets of the last release (https://github.com/kubernetes/minikube/releases/tag/v1.13.1), I found that there are two .deb files for the same thing:

  • minikube_latest_amd64.deb
  • minikube_1.13.1-0_amd64.deb

IMO the latest one shouldn't be there, since the path to it is still versioned as https://github.com/kubernetes/minikube/releases/download/v1.13.1/minikube_latest_amd64.deb.

Since I haven't found the corresponding pipeline so far, maybe you could/should look into this.

@afbjorklund
Copy link
Collaborator

It’s just to make the URL in the documentation ”static” (latest)
It’s the exact same content as in the “normal” packages (which in turn is identical to the uncompressed binary)

@afbjorklund
Copy link
Collaborator

See #7971

@tstromberg tstromberg changed the title minikube_latest_amd64.deb included in every release minikube_latest_amd64.deb accidentally included in Github release assets Sep 28, 2020
@tstromberg
Copy link
Contributor

This is a bug. The latest artifact is indeed intended for static URL's, but it shouldn't be part of the GitHub release. These unversioned files should get filtered out from here:

for file in out/minikube[_-]* out/docker-machine-*; do

@tstromberg tstromberg added area/build-release kind/bug Categorizes issue or PR as related to a bug. priority/backlog Higher priority than priority/awaiting-more-evidence. good first issue Denotes an issue ready for a new contributor, according to the "help wanted" guidelines. help wanted Denotes an issue that needs help from a contributor. Must meet "help wanted" guidelines. labels Sep 28, 2020
@afbjorklund
Copy link
Collaborator

Right, the docs* says to get it from google and not github...

* https://minikube.sigs.k8s.io/docs/start/

@thetillhoff
Copy link
Contributor Author

I would love to work on this. Will start tomorrow.

@thetillhoff
Copy link
Contributor Author

Short note: minikube-latest.x86_64.rpm is also excluded now.

@thetillhoff
Copy link
Contributor Author

This is a bug. The latest artifact is indeed intended for static URL's, but it shouldn't be part of the GitHub release. These unversioned files should get filtered out from here:

for file in out/minikube[_-]* out/docker-machine-*; do

@tstromberg Thank you for pointing there, that saved me a lot of time 🥇

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/build-release good first issue Denotes an issue ready for a new contributor, according to the "help wanted" guidelines. help wanted Denotes an issue that needs help from a contributor. Must meet "help wanted" guidelines. kind/bug Categorizes issue or PR as related to a bug. priority/backlog Higher priority than priority/awaiting-more-evidence.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants