-
Notifications
You must be signed in to change notification settings - Fork 5.1k
macOS 13 support removed and updated docs to macOS 14 #21920
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
Conversation
|
@p172913: The label(s) In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: p172913 The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
|
Hi @p172913. Thanks for your PR. I'm waiting for a github.com member to verify that this patch is reasonable to test. If it is, they should reply with Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
|
Can one of the admins verify this patch? |
nirs
left a comment
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.
@p172913 thanks!
| driver: qemu | ||
| cruntime: docker | ||
| os: macos-13 | ||
| os: macos-15 |
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 wrong - macos-15 runniner is M1 which does not support virtualization, so this test will fail. We need to replace macos-13 with macos-15-intel. We don't have macos-14-intel runners.
CHANGELOG.md
Outdated
| * Update Windows installer to create system-wide shortcut [#15405](https://github.com/kubernetes/minikube/pull/15405) | ||
| * Add `--subnet` validation [#15530](https://github.com/kubernetes/minikube/pull/15530) | ||
| * Warn users if using VirtualBox on macOS 13+ [#15624](https://github.com/kubernetes/minikube/pull/15624) | ||
| * Warn users if using VirtualBox on macos 14+ [#15624](https://github.com/kubernetes/minikube/pull/15624) |
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.
We cannot change the changelog, remove this change.
cmd/minikube/cmd/start.go
Outdated
| return | ||
| } | ||
| out.WarningT(`Due to changes in macOS 13+ minikube doesn't currently support VirtualBox. You can use alternative drivers such as 'vfkit', 'qemu', or 'docker'. | ||
| out.WarningT(`Due to changes in macos 14+ minikube doesn't currently support VirtualBox. You can use alternative drivers such as 'vfkit', 'qemu', or 'docker'. |
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.
We don't want to change this warning - it is about a change in macOS 13+ - it is still true today.
pkg/minikube/detect/detect.go
Outdated
| } | ||
|
|
||
| // MacOS13Plus returns if the current machine is running macOS 13+ | ||
| // MacOS13Plus returns if the current machine is running macos 14+ |
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.
You change the comment but the function name is still the same. Consider changing it to MacOS14Plus() and update the version check, it should be:
major >= 14
translations/de.json
Outdated
| "Due to changes in macOS 13+ minikube doesn't currently support VirtualBox. You can use alternative drivers such as 'vfkit', 'qemu', or 'docker'.\n https://minikube.sigs.k8s.io/docs/drivers/vfkit/\n https://minikube.sigs.k8s.io/docs/drivers/qemu/\n https://minikube.sigs.k8s.io/docs/drivers/docker/\n For more details on the issue see: https://github.com/kubernetes/minikube/issues/15274\n": "", | ||
| "Due to changes in macOS 13+ minikube doesn't currently support VirtualBox. You can use alternative drivers such as docker or {{.driver}}.\n https://minikube.sigs.k8s.io/docs/drivers/docker/\n https://minikube.sigs.k8s.io/docs/drivers/{{.driver}}/\n\n For more details on the issue see: https://github.com/kubernetes/minikube/issues/15274\n": "Aufgrund von Änderungen in macOS 13+ unterstützt Minikube derzeit VirtualBox nicht. Sie können alternative Treiber verwenden, wie z.B. Docker oder {{.driver}}.\nhttps://minikube.sigs.k8s.io/docs/drivers/docker/\n https://minikube.sigs.k8s.io/docs/drivers/{{.driver}}/\n\n Weitere Informationen finden sich in folgendem Issue: https://github.com/kubernetes/minikube/issues/15274\n", | ||
| "Due to changes in macos 14+ minikube doesn't currently support VirtualBox. You can use alternative drivers such as 'vfkit', 'qemu', or 'docker'.\n https://minikube.sigs.k8s.io/docs/drivers/vfkit/\n https://minikube.sigs.k8s.io/docs/drivers/qemu/\n https://minikube.sigs.k8s.io/docs/drivers/docker/\n For more details on the issue see: https://github.com/kubernetes/minikube/issues/15274\n": "", | ||
| "Due to changes in macos 14+ minikube doesn't currently support VirtualBox. You can use alternative drivers such as docker or {{.driver}}.\n https://minikube.sigs.k8s.io/docs/drivers/docker/\n https://minikube.sigs.k8s.io/docs/drivers/{{.driver}}/\n\n For more details on the issue see: https://github.com/kubernetes/minikube/issues/15274\n": "Aufgrund von Änderungen in macos 14+ unterstützt Minikube derzeit VirtualBox nicht. Sie können alternative Treiber verwenden, wie z.B. Docker oder {{.driver}}.\nhttps://minikube.sigs.k8s.io/docs/drivers/docker/\n https://minikube.sigs.k8s.io/docs/drivers/{{.driver}}/\n\n Weitere Informationen finden sich in folgendem Issue: https://github.com/kubernetes/minikube/issues/15274\n", |
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.
Remove all the translations changes. These files will be regenerated automatically if related string in the code change.
Update support for macOS 13+ issues changed macos to macOS in translations changed macos to macOS in translations
nirs
left a comment
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.
Please address comments and squash all commit to one.
If you want to split this to multiple commit, you can split to:
- GitHub workflow changes
- Other changes
But this is a very small change and one commit is fine.
| driver: vfkit | ||
| cruntime: docker | ||
| os: macos-13 | ||
| os: macos-15 |
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 cannot work for vfkit for the same reason it can't work for qemu.
| - name: Ensure bootpd is enabled (macos-13) | ||
| if: matrix.os == 'macos-13' | ||
| - name: Ensure bootpd is enabled (macos-15) | ||
| if: matrix.os == 'macos-15' |
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 entire step can be removed, not needed since macOS 15
| kubectl version --client=true | ||
| - name: Install qemu and socket_vmnet (macos) | ||
| if: matrix.os == 'macos-13' && matrix.driver == 'qemu' | ||
| if: matrix.os == 'macos-15' && matrix.driver == 'qemu' |
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.
macos-15-intel
| HOMEBREW=$(which brew) && sudo ${HOMEBREW} services start socket_vmnet | ||
| - name: Install vfkit and vmnet_helper (macos) | ||
| if: matrix.os == 'macos-13' && matrix.driver == 'vfkit' | ||
| if: matrix.os == 'macos-15' && matrix.driver == 'vfkit' |
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.
macos-15-intel
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.
Match the changes to functional tests in this file, replacing macos-13 with macos-15-intel and removing bootpd step.
cmd/minikube/cmd/start.go
Outdated
| return | ||
| } | ||
| out.WarningT(`Due to changes in macOS 13+ minikube doesn't currently support VirtualBox. You can use alternative drivers such as 'vfkit', 'qemu', or 'docker'. | ||
| out.WarningT(`Due to changes in macos 13+ minikube doesn't currently support VirtualBox. You can use alternative drivers such as 'vfkit', 'qemu', or 'docker'. |
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.
Remove this change, the right spelling is macOS.
pkg/minikube/detect/detect.go
Outdated
|
|
||
| // MacOS13Plus returns if the current machine is running macOS 13+ | ||
| func MacOS13Plus() bool { | ||
| // MacOS14Plus returns if the current machine is running macos 14+ |
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.
comment: macOS not macOS
function name: you rename the function, but did not change the callers of this function. Please update the callers, and if this function is not used delete the function.
CHANGELOG.md
Outdated
| * Update Windows installer to create system-wide shortcut [#15405](https://github.com/kubernetes/minikube/pull/15405) | ||
| * Add `--subnet` validation [#15530](https://github.com/kubernetes/minikube/pull/15530) | ||
| * Warn users if using VirtualBox on macOS 13+ [#15624](https://github.com/kubernetes/minikube/pull/15624) | ||
| * Warn users if using VirtualBox on macos 13+ [#15624](https://github.com/kubernetes/minikube/pull/15624) |
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.
remove this change, we don't change the changelog.
|
Squashing commit giving merger issue with others commits. So, created new PR #22027. |
What type of PR is this?
/kind enhancement
What this PR does / why we need it:
macOS 13 support removed and updated macOS 13 to macOS 15 along with updated docs to macOS 14
Which issue(s) this PR fixes:
Fixes #21666
Special notes for your reviewer:
None
Does this PR introduce a user-facing change?
None
Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.:
None