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

Add AWS CLI and environment patch #20

Merged
merged 9 commits into from
Sep 20, 2024

Conversation

clementblaise
Copy link
Contributor

@clementblaise clementblaise commented Aug 27, 2024

Description of your changes

The changes enable using AWS CLI and use in-cluster ProviderConfig to pass the configuration.

  • Added AWS CLI to the build
  • Added valueRef to environment variables parameters
  • Added air.toml for hot reload

Fixes crossplane/crossplane#4141 crossplane/crossplane#5830

I have:

@clementblaise
Copy link
Contributor Author

Updated to follow this proposal, allowing access to any context key or composite value

@humoflife
Copy link
Collaborator

@clementblaise great job. This will be super useful for many Crossplane users.

env.go Show resolved Hide resolved
example/aws/composition.yaml Outdated Show resolved Hide resolved
example/aws/extra-resources.yaml Outdated Show resolved Hide resolved
example/aws/composition.yaml Show resolved Hide resolved
example/aws/extra-resources.yaml Outdated Show resolved Hide resolved
example/aws/composition.yaml Show resolved Hide resolved
Signed-off-by: Clément Blaise <[email protected]>
Signed-off-by: Clément Blaise <[email protected]>
Signed-off-by: Clément Blaise <[email protected]>
Signed-off-by: Clément Blaise <[email protected]>
Signed-off-by: Clément Blaise <[email protected]>
Signed-off-by: Clément Blaise <[email protected]>
example/aws/README.md Outdated Show resolved Hide resolved
example/aws/composition.yaml Show resolved Hide resolved
Signed-off-by: Clément Blaise <[email protected]>
@humoflife humoflife merged commit 1ecbdbd into crossplane-contrib:main Sep 20, 2024
6 checks passed
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 Querying/Filtering for Import and Observe
2 participants