Skip to content

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

Closed

Description

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.

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Assignees

Labels

area/control-planeControl plane relatedkind/enhancementEnhancement, improvement, extensionpriority/3Priority (lower number equals higher priority)status/closedIssue is closed (either delivered or triaged)

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions