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

Machine Update: Stage 1: Introduce InitializeMachine to Driver interface #840

Closed
unmarshall opened this issue Aug 8, 2023 · 4 comments
Closed
Assignees
Labels
area/control-plane Control plane related kind/enhancement Enhancement, improvement, extension priority/3 Priority (lower number equals higher priority) status/closed Issue is closed (either delivered or triaged)

Comments

@unmarshall
Copy link
Contributor

unmarshall commented Aug 8, 2023

How to categorize this issue?

/area control-plane
/kind enhancement
/priority 3

What would you like to be added:
Currently Driver implemented by all providers does not have a way to initialize a machine after creation. We now have two use cases for which InitializeMachine is now a requirement:

  1. Add Ipv6AddressCount and Ipv6PrefixCount to enable the assignment of an ipv6 address and an ipv6 prefix to instances. machine-controller-manager-provider-aws#128 - assigns IPV6 addresses to the instance post creation of the instance.
  2. Disable SrcAndDestCheck on NAT Instances machine-controller-manager-provider-aws#39 - enable/disable source/destination check on an instance.

CreateMachine should be an atomic operation and should only do one thing - create a new machine. Issue#comment highlights this requirement. We have already broken the contract of CreateMachine once (see 2 above) and as a consequence also broken the contract of GetMachineStatus (see 2 above).

Why is this needed:
There is a need to introduce a new method InitializeMachine which only does a one-time update of an existing instance. Changes that are done in (1) and (2) above should be moved to this new method.

@himanshu-kun
Copy link
Contributor

There is already an issue open for this #635 , shall we close this one and update the other, as it has more links and context attached to it ?

@himanshu-kun
Copy link
Contributor

The arguments for UpdateMachine() depends on the proposal for #750

@himanshu-kun himanshu-kun assigned elankath and himanshu-kun and unassigned elankath Oct 5, 2023
@elankath elankath changed the title Introduce UpdateMachine to Driver interface Machine Update: Stage 2: Introduce UpdateMachine to Driver interface Nov 16, 2023
@elankath elankath changed the title Machine Update: Stage 2: Introduce UpdateMachine to Driver interface Machine Update: Stage 1: Introduce InitializeMachine to Driver interface Feb 6, 2024
@elankath
Copy link
Contributor

elankath commented Feb 6, 2024

This should be closed as there is already a separate issue #871.

The UpdateMachine machine meant for Stage-2 is at: #635

@rishabh-11
Copy link
Contributor

/close

@gardener-robot gardener-robot added the status/closed Issue is closed (either delivered or triaged) label Mar 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/control-plane Control plane related kind/enhancement Enhancement, improvement, extension priority/3 Priority (lower number equals higher priority) status/closed Issue is closed (either delivered or triaged)
Projects
None yet
Development

No branches or pull requests

5 participants