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

Launch Template and Launch Template Version resources added #1041

Merged
merged 2 commits into from
Dec 31, 2021
Merged

Launch Template and Launch Template Version resources added #1041

merged 2 commits into from
Dec 31, 2021

Conversation

tanujd11
Copy link

@tanujd11 tanujd11 commented Dec 31, 2021

Signed-off-by: Tanuj Dwivedi tanuj.dwivedi@D11-tanuj.dwivedi

Description of your changes

The Following PR adds LaunchTemplate and LaunchTemplate Version resource to the AWS crossplane provider

Fixes #338

I have:

  • Read and followed Crossplane's contribution process.
  • Run make reviewable test to ensure this PR is ready for review.

How has this code been tested

The code has been tested with manual Creation and Deletion of the example resources.

Signed-off-by: Tanuj Dwivedi <tanuj.dwivedi@D11-tanuj.dwivedi>
@tanujd11 tanujd11 changed the title Launch Template and Luanch Template Version resources added Launch Template and Launch Template Version resources added Dec 31, 2021
… set correct examples

Signed-off-by: haarchri <chhaar30@googlemail.com>
Copy link
Member

@haarchri haarchri left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@tanujd11 good first contribution to provider-aws - LGTM

tested Create & Delete for LaunchTemplate and LaunchTemplateVersion

  • added Referencers for EKS-NodeGroup - also tested EKS-NodeGroup wird LaunchTemplate
  • removed DryRun & ClientToken via Code-Generator from LaunchTemplate & LaunchTemplateVersion
  • adjust code for GenerateDeleteLaunchTemplateVersionInput for removed DryRun & external-name
launchtemplate.ec2.aws.crossplane.io/test-crossplane-obj   True    True     test-crossplane-obj
launchtemplateversion.ec2.aws.crossplane.io/test-crossplane-v3   True    True     2

@haarchri
Copy link
Member

haarchri commented Dec 31, 2021

we are unable to create referencers for LaunchTemplate / LaunchTemplateVersion for EC2-Instance because of Go ImportCycle Issues - i will open an Issue in general for this

Bildschirmfoto 2021-12-31 um 10 44 45

@haarchri haarchri merged commit 3b88a3d into crossplane-contrib:master Dec 31, 2021
@tanujd11 tanujd11 mentioned this pull request Aug 22, 2022
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support for Launch Templates
2 participants