Skip to content

Complete Overhaul of authorization CRD parts #167

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
3 tasks done
fhennig opened this issue Mar 14, 2022 · 0 comments
Closed
3 tasks done

Complete Overhaul of authorization CRD parts #167

fhennig opened this issue Mar 14, 2022 · 0 comments
Assignees
@maltesander

Comments

@fhennig
Copy link
Contributor

fhennig commented Mar 14, 2022 •
edited by maltesander
Loading

From the CRD Review

This is done when:

  • the CRD has been changed to use the OpaConfig struct from the operator framework (operator-rs#372), the CRD change should be in line with other operators!
  • Remove the self created authorization config map and deploy the rules via example configmap as in the other operators
  • documentation, examples, integration tests have been updated if needed Integration tests added for OPA integration
@lfrancke lfrancke changed the title Complete Overhaul of authorization Complete Overhaul of authorization CRD parts Apr 7, 2022
@maltesander maltesander self-assigned this Apr 8, 2022
@maltesander maltesander mentioned this issue Apr 8, 2022
Closed
6 tasks
bors bot pushed a commit that referenced this issue Apr 13, 2022
@maltesander
Authorization overhaul (#186)
ee23658 
## Description

- removed authorization module
- switched to `OpaConfig` from operator-rs

closes #167



Co-authored-by: Malte Sander <malte.sander.it@gmail.com>
Co-authored-by: Felix Hennig <felix.hennig@stackable.de>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@maltesander maltesander

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[Merged by Bors] - Authorization overhaul stackabletech/trino-operator
3 participants
@lfrancke @fhennig @maltesander