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
Update docs for 1.7.1 #3810
Update docs for 1.7.1 #3810
Conversation
docs/examples/basic-requirements.md
Outdated
@@ -14,7 +14,7 @@ As root (either ssh directly to root, local root console, or by using "sudo su - | |||
```bash | |||
cd ~ | |||
curl -LO https://storage.googleapis.com/kubernetes-release/release/$(curl -s https://storage.googleapis.com/kubernetes-release/release/stable.txt)/bin/linux/amd64/kubectl | |||
curl -LO https://github.com/kubernetes/kops/releases/download/1.7.0/kops-linux-amd64 | |||
curl -LO https://github.com/kubernetes/kops/releases/download/1.7.1/kops-linux-amd64 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we do that curl call for the latest release? I know there is a curl command for that.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@chrislovecnm I'll look at that, although "latest" right now is technically Fixed, thanks @justinsb.1.8.0-beta.1
, should that be moved to pre-release
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also @chrislovecnm google seems to say it's only possible if we do something like this
curl -LO https://github.com/kubernetes/kops/releases/download/$(curl -s https://api.github.com/repos/kubernetes/kops/releases/latest | grep tag_name | cut -d '"' -f 4)/kops-linux-amd64
Let me know if that's acceptable!
72afce9
to
a6e7f97
Compare
Updated and cleaned up for consistency. As a side note, I'm adding |
/lgtm For the brew - that sounds good. I think the big problem we had in the past was we were maybe pulling from head, or users were choosing to do so. I like the idea of giving a --devel flag to give a middle-ground :-) (And we're not building pulling from master by default, are we?) |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: justinsb The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these OWNERS Files:
You can indicate your approval by writing |
Automatic merge from submit-queue. |
@justinsb Thanks! Yep, that happened to me in the past. Nope, we link to the tagged source in brew, then it's bottled up for actual releases to make installs faster. |
No description provided.