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

AWS Go SDK v1 connection method for ACK-generated resources #401

Merged
merged 2 commits into from
Nov 12, 2020

Conversation

muvaf
Copy link
Member

@muvaf muvaf commented Oct 26, 2020

Description of your changes

We will be using ACK code generation pipeline to add new resources to provider-aws but ACK and generated code works with AWS Go SDK v1 whereas we use v2. One place they differ is how they authorize. This PR adds a function specific to v1.

Fixes #

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

Not tested yet with an actual controller.

@prasek prasek mentioned this pull request Nov 3, 2020
26 tasks
@muvaf muvaf marked this pull request as ready for review November 5, 2020 13:15
…es necessary authorization methods that use v1 SDK.

Signed-off-by: Muvaffak Onus <onus.muvaffak@gmail.com>
Copy link
Member

@hasheddan hasheddan left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks @muvaf! One comment on your TODO that looks like it has already been implemented.

…S Go SDK v1

Signed-off-by: Muvaffak Onus <onus.muvaffak@gmail.com>
@muvaf muvaf merged commit 0f9e291 into crossplane-contrib:master Nov 12, 2020
@muvaf muvaf deleted the ack-v1-connect branch November 12, 2020 17:53
wolffbe pushed a commit to wolffbe/provider-aws that referenced this pull request Feb 12, 2021
AWS Go SDK v1 connection method for ACK-generated resources
namku pushed a commit to namku/provider-aws that referenced this pull request Mar 9, 2021
AWS Go SDK v1 connection method for ACK-generated resources
tektondeploy pushed a commit to gtn3010/provider-aws that referenced this pull request Mar 12, 2024
[provider-aws]: Update the authentication guide
tektondeploy pushed a commit to gtn3010/provider-aws that referenced this pull request Mar 12, 2024
…l-networkmanager

Moving networkfirewall(2), networkmanager(9) resources to v1beta1 version
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.

2 participants