-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Have upgrade functionality (as tool or server side) #28
Comments
From @roberthbailey on February 7, 2018 6:36 Alternatively, implement server side upgrades as part of the MachineDeployment resource type. |
/assign @k4leung4 |
@k4leung4 - do machine deployments meet the exit criteria of this issue (for nodes)? I think they do, just wanting a second opinion. If so, then we should either re-scope this to control plane upgrades or close it and spawn a separate issue for control plane upgrades. |
/close |
@roberthbailey: Closing this issue. 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/test-infra repository. |
Adds some tests and basic controller logic
…s-sigs#71) * Move the MachineRef property into the ProviderConfig * Move the `MachineRef` property from `ProviderStatus` to `ProviderConfig` object * Upon creation of the `VirtualMachine` the `MachineRef` will be populated inside the spec * Update the glog logs to use higher verbosity for improved readability of logs Resolves kubernetes-sigs#82 Change-Id: I4ef1c5dec11d0c06dd777daae6500f4fe6f13241 * Add a new machinespec for defining the instance on vsphere With the new structure we should be able to: * Support defining multiple nics * Support providing dhcp/static IP configuration for each nic * Support customizing multiple disks if present * Adds a optional flag called "vsphereCloudInit" to indicate if the template used has the vsphere datasource for cloud-init present and enabled Note: This change does not change the existing behavior of using the MachineVariables string map. This will be handled in the subsequent patches. Partially Resolves kubernetes-sigs#24 Partially Resolves kubernetes-sigs#28 Change-Id: Ia49b8ef9984cf6b82f4885179491ac3b3cc6f369
…on vsphere (kubernetes-sigs#72) The code now acts accordingly based on the availability of the new cloud-init datasource "VMwareGuestInfo" inside the template used. This availability can be set by the users in the spec using the flag "vsphereCloudInit" set to true. For usual upstream version of ubuntu cloud image that has been used so far, this flag need not be set at all and the controller will utilize the OVF properties available on the ubuntu cloud image to inject the cloud-init data. Note: The new static IP support is only available with the templates that have the vsphere cloud-init datasource. DHCP based IPs will continue to work as they have in the past. Going forward we prefer users creating a custom ubuntu cloud image that also has this new cloud-init datasource pre-installed. As that will give them the ability to use static IPs as well if they want. See https://github.com/akutz/cloud-init-vmware-guestinfo for more info on the new cloud-init datasource * Adds support for multiple nics * Adds support for static IP Resolves kubernetes-sigs#24 Resolves kubernetes-sigs#28 Change-Id: Id2dee12b382d57b6591f840dadd77a1c287667c4
…yomachine Defer patch byomachine
✨ Adding support for multiple registry mirrors in bottlerocket
From @rsdcastro on January 29, 2018 22:39
Copied from original issue: kubernetes-retired/kube-deploy#537
The text was updated successfully, but these errors were encountered: