-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
Cut release 0.29.0 #3175
Cut release 0.29.0 #3175
Conversation
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: balopat The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
CHANGELOG.md
Outdated
* Change default docker storage driver to overlay2 [#3121](https://github.com/kubernetes/minikube/pull/3121) | ||
* Add env variable for default ES_JAVA_OPTS [#3086](https://github.com/kubernetes/minikube/pull/3086) | ||
* fix(cli): `minikube start --mount --mountsting` without write permission [#2671](https://github.com/kubernetes/minikube/pull/2671) | ||
* Add workaround for dnsmasq and hyperkit users to drivers readme [#3003](https://github.com/kubernetes/minikube/pull/3003) |
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.
This is just a README update, and not related to the release.
CHANGELOG.md
Outdated
* Fix vmwarefusion driver [#3029](https://github.com/kubernetes/minikube/pull/3029) | ||
* Make CoreDNS default addon [#3072](https://github.com/kubernetes/minikube/pull/3072) | ||
* Update CoreDNS deployment [#3073](https://github.com/kubernetes/minikube/pull/3073) | ||
* readme: update the script to match executable path [#3051](https://github.com/kubernetes/minikube/pull/3051) |
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.
This too.
CHANGELOG.md
Outdated
- Joao Carlos | ||
- Kumbirai Tanekha | ||
- Matt Rickard | ||
- minikube-bot |
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.
Maybe not?
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.
lol
pkg/minikube/cluster/cluster.go
Outdated
@@ -217,7 +217,7 @@ func createHost(api libmachine.API, config cfg.MachineConfig) (*host.Host, error | |||
|
|||
if config.VMDriver != "none" { | |||
if err := config.Downloader.CacheMinikubeISOFromURL(config.MinikubeISO); err != nil { | |||
return nil, errors.Wrap(err, "Error attempting to cache minikube ISO from URL") | |||
return nil, fmt.Errorf("error attempting to cache minikube ISO from URL (%s): %s", config.MinikubeISO, err) |
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.
Unrelated fix?
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.
okay...I'll remove them...
@@ -47,7 +47,7 @@ func testClusterStatus(t *testing.T) { | |||
} | |||
if status != api.ConditionTrue { | |||
err := fmt.Errorf("Component %s is not Healthy! Status: %s", i.GetName(), status) | |||
t.Log("Retrying, %s", err) | |||
t.Logf("Retrying, %s", err) |
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.
Unrelated fix here too.
0.29.0 Release * tag 'v0.29.0': (77 commits) Cut release 0.29.0 (kubernetes#3175) turning error into a warning log message Bump everything to 1.11. Update docs. Update go-ansiterm. Issue kubernetes#3037 change dependency management to dep inlining deleteNetwork into if statement update dashboard version Fix typo: suppports -> supports adding tstromberg kvm2 driver: fixed documentation link in error text kvm2 driver: be more robust with creating of the network, don't delete the network if it is in use kvm2 driver: generate separate random MACs for default and private network NICs kvm2 driver: separated activating and creating of networks kvm2 driver: fixed private network creation and setup fix: --format outputs any string, --https only subsitute http URL scheme Stay in same line Change default docker storage driver to overlay2 Simplifies Linux installation instructions by using install(1) update ...
No description provided.