Skip to content
This repository has been archived by the owner on Feb 2, 2021. It is now read-only.

Add ECR support #133

Merged
merged 4 commits into from
Oct 28, 2019
Merged

Add ECR support #133

merged 4 commits into from
Oct 28, 2019

Conversation

alexellis
Copy link
Member

Add ECR support

Signed-off-by: Alex Ellis (OpenFaaS Ltd) alexellis2@gmail.com

Description

ECR support is required for some customers and users. This patch
also brings the of-builder over as a templated resource so that
the AWS secret for accessing ECR can be bound when necessary
to the register-image function and to the of-builder itself.

How Has This Been Tested?

Automated e2e testing TBD, upstream feature tested through manual deployment.

Checklist:

I have:

  • checked my changes follow the style of the existing code / OpenFaaS repos
  • updated the documentation and/or roadmap in README.md
  • read the CONTRIBUTION guide
  • signed-off my commits with git commit -s
  • added unit tests

@alexellis alexellis requested a review from martindekov October 10, 2019 15:25
ECR support is required for some customers and users. This patch
also brings the of-builder over as a templated resource so that
the AWS secret for accessing ECR can be bound when necessary
to the register-image function and to the of-builder itself.

Signed-off-by: Alex Ellis (OpenFaaS Ltd) <alexellis2@gmail.com>
A unique name is needed so that it doesn't clash with the secret
created in the openfaas-fn namespace.

Signed-off-by: Alex Ellis (OpenFaaS Ltd) <alexellis2@gmail.com>
@alexellis alexellis force-pushed the openfaasltd/add-ecr branch from 9607641 to 6763fb4 Compare October 28, 2019 09:37
Signed-off-by: Alex Ellis (OpenFaaS Ltd) <alexellis2@gmail.com>
@alexellis alexellis removed the request for review from martindekov October 28, 2019 09:39
Tested e2e with the community cluster.

Signed-off-by: Alex Ellis (OpenFaaS Ltd) <alexellis2@gmail.com>
@alexellis alexellis merged commit a37a6ca into master Oct 28, 2019
@alexellis alexellis deleted the openfaasltd/add-ecr branch October 28, 2019 09:47
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant