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

feat(elbv2): Add new Target resource #1402

Merged

Conversation

MisterMX
Copy link
Collaborator

@MisterMX MisterMX commented Aug 4, 2022

Description of your changes

Fixes #1401

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

Manually, using examples.

@MisterMX MisterMX marked this pull request as draft August 4, 2022 15:14
@MisterMX
Copy link
Collaborator Author

MisterMX commented Aug 4, 2022

Waiting for #1403

@MisterMX MisterMX force-pushed the elb-target-group-registration branch from abb5257 to a1aaa12 Compare August 10, 2022 09:55
@MisterMX MisterMX marked this pull request as ready for review August 10, 2022 09:55
@MisterMX MisterMX force-pushed the elb-target-group-registration branch 3 times, most recently from 13481ee to 97e8348 Compare August 10, 2022 10:59
@MisterMX
Copy link
Collaborator Author

Ready for review. @haarchri can you take a look at this?

@MisterMX MisterMX force-pushed the elb-target-group-registration branch from 97e8348 to bf4c806 Compare August 11, 2022 14:04
Signed-off-by: Maximilian Blatt <maximilian.blatt-extern@deutschebahn.com>
(external expert on behalf of DB Netz AG)
@haarchri
Copy link
Member

Can you please rebase ?

@MisterMX MisterMX force-pushed the elb-target-group-registration branch from bf4c806 to 72a6950 Compare August 12, 2022 07:19
@MisterMX
Copy link
Collaborator Author

@haarchri Done

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.

LGTM

@haarchri haarchri merged commit 41987e2 into crossplane-contrib:master Aug 15, 2022
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.

elbv2: Add new resource to allow registering targets
2 participants