Skip to content

Update AwsAuthManager to support multi-team #65371 - #65393

Merged
vincbeck merged 3 commits into
apache:mainfrom
haseebmalik18:aws-auth-manager-multi-team
Apr 21, 2026
Merged

Update AwsAuthManager to support multi-team #65371#65393
vincbeck merged 3 commits into
apache:mainfrom
haseebmalik18:aws-auth-manager-multi-team

Conversation

@haseebmalik18

Copy link
Copy Markdown
Contributor

AwsAuthManager currently ignores team_name entirely and is_authorized_team raises NotImplementedError, which blocks multi-team for AWS deployments.

Following the KeycloakAuthManager as reference, this PR:

  • Extracts team_name from resource details in is_authorized_* methods
  • Implements is_authorized_team with a new TEAM AVP entity
  • Passes team_name through filter_authorized_* methods to the AVP facade
  • Enhances AVP context with team_name for Cedar policy evaluation
  • Updates the Cedar schema with Team entity type and team_name context on team-scoped actions

closes: #65371

@boring-cyborg boring-cyborg Bot added area:providers provider:amazon AWS/Amazon - related issues labels Apr 16, 2026
@haseebmalik18 haseebmalik18 reopened this Apr 16, 2026
@haseebmalik18
haseebmalik18 force-pushed the aws-auth-manager-multi-team branch from d74586a to c93ed5d Compare April 17, 2026 00:42
@haseebmalik18
haseebmalik18 force-pushed the aws-auth-manager-multi-team branch from 5d488c6 to e2e2b32 Compare April 17, 2026 20:17
@vincbeck

Copy link
Copy Markdown
Contributor

Did you actually tested it with your AWS account?

@haseebmalik18

haseebmalik18 commented Apr 21, 2026

Copy link
Copy Markdown
Contributor Author

Hey, I tested this against a real Amazon Verified Permissions policy store and everything works as expected. The updated schema uploads fine, Cedar policies can use context.team_name to scope access by team, and the new Team entity type works for membership checks. Verified all the ALLOW/DENY scenarios through the Test Bench

@vincbeck

Copy link
Copy Markdown
Contributor

Thank you!

@vincbeck
vincbeck merged commit 551870b into apache:main Apr 21, 2026
98 checks passed
Cloud-Architect-Emma pushed a commit to Cloud-Architect-Emma/airflow that referenced this pull request Apr 30, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:providers provider:amazon AWS/Amazon - related issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Update AwsAuthManager to support multi-team

2 participants