Skip to content

feat(airflow): Add OPA support to Airflow#978

Merged
siegfriedweber merged 16 commits intomainfrom
feat/airflow-opa
Feb 7, 2025
Merged

feat(airflow): Add OPA support to Airflow#978
siegfriedweber merged 16 commits intomainfrom
feat/airflow-opa

Conversation

@siegfriedweber
Copy link
Member

@siegfriedweber siegfriedweber commented Jan 17, 2025 •
edited
Loading

Description

part of stackabletech/airflow-operator#446

Add OPA support to Airflow

Definition of Done Checklist

  • Not all of these items are applicable to all PRs, the author should update this template to only leave the boxes in that are relevant
  • Please make sure all these things are done and tick the boxes
- [x] Changes are OpenShift compatible
- [x] Add an entry to the CHANGELOG.md file
- [x] Integration tests ran successfully
TIP: Running integration tests with a new product image

The image can be built and uploaded to the kind cluster with the following commands:

bake --product <product> --image-version <stackable-image-version>
kind load docker-image <image-tagged-with-the-major-version> --name=<name-of-your-test-cluster>

See the output of bake to retrieve the image tag for <image-tagged-with-the-major-version>.

@siegfriedweber siegfriedweber self-assigned this Jan 17, 2025
@siegfriedweber siegfriedweber mentioned this pull request Jan 17, 2025
Closed
4 tasks
@siegfriedweber siegfriedweber marked this pull request as ready for review January 28, 2025 15:21
@siegfriedweber siegfriedweber requested a review from a team January 28, 2025 15:21
labrenbe
labrenbe approved these changes Feb 6, 2025
View reviewed changes
Copy link
Member

@labrenbe labrenbe left a comment

Choose a reason for hiding this comment

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

lgtm

@siegfriedweber siegfriedweber added this pull request to the merge queue Feb 7, 2025
Merged via the queue into main with commit a58c1e6 Feb 7, 2025
3 checks passed
@siegfriedweber siegfriedweber deleted the feat/airflow-opa branch February 7, 2025 07:14
dervoeti pushed a commit that referenced this pull request Feb 26, 2025
@siegfriedweber @dervoeti
feat(airflow): Add OPA support to Airflow (#978)
520cd12 
* feat: Add initial OpaFabAuthManager

* feat(airflow): Call OPA from FabAuthManager

* feat(airflow): Add cache and configuration options for OPA and implement all is_authorized functions

* fix(airflow): Fix call to OPA

* test(airflow): Test OpaFabAuthManager configuration and caching

* chore: Update changelog

* feat(airflow): Add metric opa_cache_limit_reached

* docs(airflow): Add a README to the opa-auth-manager

* chore(airflow): Reformat Python code

* docs(airflow): Fix markdown linter warning

* chore(airflow): Fix Dockerfile linter warning

* docs(airflow): Fix a typo
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@labrenbe labrenbe labrenbe approved these changes

@Maleware Maleware Awaiting requested review from Maleware

Assignees

@siegfriedweber siegfriedweber

Labels

release/25.3.0

Projects

Stackable Engineering
Archived in project

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@siegfriedweber @labrenbe @lfrancke