Update AWSResourceManagerFactory to include roleARN paramter
#156
Conversation
This patch modifies the `AWSResourceManageFactory` intgerface and its implementation to include a `roleARN` parameter in the `ManagerFor` function. The addition of `roleARN` is necessary to support fine-grained access control and multi-tenant scenarios where different IAM roles may be used within the same AWS account and region. This change corresponds to the updates in code-generaotr repository.
|
@a-hilaly: The following tests failed, say
Full PR test history. Your PR dashboard. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here. |
|
Tests are failling since this change also requires code-gen changes |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: a-hilaly, TiberiuGC The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
This patch modifies the
AWSResourceManageFactoryintgerface andits implementation to include a
roleARNparameter in theManagerForfunction.The addition of
roleARNis necessary to support fine-grainedaccess control and multi-tenant scenarios where different IAM roles
may be used within the same AWS account and region. This change
corresponds to the updates in code-generaotr repository.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.