Skip to content

feat: add allowList to policies for jobs authz - #2931

Open
minottic wants to merge 1 commit into
partial_updatefrom
policy_permissions
Open

feat: add allowList to policies for jobs authz#2931
minottic wants to merge 1 commit into
partial_updatefrom
policy_permissions

Conversation

@minottic

@minottic minottic commented Sep 2, 2026

Copy link
Copy Markdown
Member

Description

Add policy based authorisation to jobs

Motivation

Some jobs are restricted to single users per ownergroup, for example deletion ones are restricted to a very limited pool of users not contained in the dataset

Tests included

  • Included for each change/fix?
  • Passing?

Documentation

  • swagger documentation updated (required for API changes)
  • official documentation updated

official documentation info

Summary by Sourcery

Enable allow-list policy authorization for creating jobs across datasets.

New Features:

  • Add policy-based job creation authorization that matches users by email or group against per-job allow lists on dataset owner policies.

Enhancements:

  • Require all datasets in a multi-dataset job request to authorize the requesting user through their linked policies, while preserving administrator access.

Documentation:

  • Document the new #datasetPolicyList authorization mode and allowed-list policy configuration in the example job configuration and API schema.

Tests:

  • Add end-to-end coverage for email and group authorization, denied access, missing or empty policies, mixed dataset requests, unauthenticated requests, and administrator access.

@minottic
minottic requested a review from a team as a code owner September 2, 2026 20:29

@sourcery-ai sourcery-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry @minottic, you've used your own review budget of 250,000 diff characters for the last 7 days.

You can request another review in 15 hours and 44 minutes by commenting @sourcery-ai review. Upgrade to get a review now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant