[Draft] feat: add E2E tests for --enable-cross-namespace flag#175
[Draft] feat: add E2E tests for --enable-cross-namespace flag#175sapphirew wants to merge 1 commit into
Conversation
…space Regenerate iam-controller using updated code-generator and runtime that introduce the unified --enable-cross-namespace flag with phased rollout. Generated code changes: - Renames EnableCrossNamespaceReferences to EnableCrossNamespace - Updates ValidateCrossNamespaceReference to return (string, bool, error) - Adds --enable-cross-namespace flag to Helm deployment template - Adds enableCrossNamespace: true to Helm values (Phase 1 default) E2E tests cover three scenarios: - flag=true (Phase 1 default): cross-namespace ref succeeds with ACK.CrossNamespaceDeprecation condition as deprecation warning - flag=false: cross-namespace ref rejected with ACK.Terminal condition containing the --enable-cross-namespace flag name - same-namespace: ref always succeeds regardless of flag value with no deprecation condition present
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: sapphirew The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
|
@sapphirew: The following tests failed, say
Full PR test history. Your PR dashboard. DetailsInstructions 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. |
|
Doc changes is added - aws-controllers-k8s/docs#37 |
Issue #, if available:
Description of changes:
Regenerate iam-controller using updated code-generator and runtime that introduce the unified --enable-cross-namespace flag with phased rollout.
Generated code changes:
E2E tests cover three scenarios:
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.