Skip to content
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

Bundle Builder v2 #578

Merged
merged 62 commits into from
Jul 31, 2024
Merged

Bundle Builder v2 #578

merged 62 commits into from
Jul 31, 2024

Conversation

nightkr
Copy link
Member

@nightkr nightkr commented May 20, 2024 •
edited
Loading

Description

Fixes #577

Tasks
Beta Give feedback

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

Author
Beta Give feedback

Reviewer
Beta Give feedback

Acceptance
Beta Give feedback

nightkr and others added 30 commits April 12, 2023 11:31
@nightkr
Add group fetcher container
455bdc4
@nightkr
Spike group fetcher functionality
63ee375
@nightkr
Rename group fetcher to user info fetcher
bca2d62
@nightkr
Generalize enrichment endpoints to have room for arbitrary user info
9c1098c
@nightkr
Expose user roles
8b28fd4
@nightkr
Make UIF configurable
a5bbe88
@nightkr
Make UIF credentials configurable too
08823b7
@nightkr
OPA 0.45 -> 0.51
412e542
@nightkr
Broke out UIF into its own crate
22507d0
@nightkr
UIF error handling
cffa188
@nightkr
Ingest custom attributes
c74b6d9
@nightkr
Shut down on SIGTERM
1ef78e8
@nightkr
Split keycloak backend out into separate module
67e411f
@nightkr
Fix UIF startup when using none backend
eb134b7
@nightkr
Rename GroupMembershipRequest to UserInfoRequest
fb6da54
@nightkr
Cache fetched UserInfo
f298e0b
@nightkr
Merge branch 'main' into spike/user-info-fetcher
b1644e4
@nightkr
Switch UIF to also use workspace dependencies
3d160ca
@nightkr
Configurable UIF cache ttl
a4f495d
@nightkr
UIF crate metadata
57b45c7
@nightkr
Move userInfo rule into helm chart
20e585c
@nightkr
Turn UIF custom attributes into a multidict
05df3cb
@nightkr
UIF smoke test
2c511e4
@nightkr
Remove rules from Helm chart until we have a better way to deploy them
23a55c6
@nightkr
Lint-b-gone
8c45dc1
@nightkr
Update CRD
0cdbfd3
@nightkr
UIF readme
fc7de76
@nightkr
SNAFU error for UIF config
0a684d9
@nightkr
Revert Cmd wrapper enum
3aceb68
@nightkr
docs
65934ee
@nightkr nightkr mentioned this pull request May 29, 2024
Merged
@nightkr nightkr mentioned this pull request Jun 26, 2024
Merged
@nightkr nightkr mentioned this pull request Jul 29, 2024
Open
@nightkr nightkr requested a review from a team July 29, 2024 14:19
sbernauer
sbernauer requested changes Jul 31, 2024
View reviewed changes
Copy link
Member

@sbernauer sbernauer left a comment

Choose a reason for hiding this comment

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

Code looks good

CHANGELOG.md Outdated Show resolved Hide resolved
rust/operator-binary/src/controller.rs Show resolved Hide resolved
sbernauer
sbernauer approved these changes Jul 31, 2024
View reviewed changes
Copy link
Member

@sbernauer sbernauer left a comment

Choose a reason for hiding this comment

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

Thanks, LGTM!

@nightkr nightkr enabled auto-merge July 31, 2024 11:57
@nightkr nightkr disabled auto-merge July 31, 2024 12:01
@nightkr nightkr added this pull request to the merge queue Jul 31, 2024
@nightkr nightkr mentioned this pull request Jul 31, 2024
Merged
Merged via the queue into main with commit f6c0767 Jul 31, 2024
31 checks passed
@nightkr nightkr deleted the spike/bundle-builder-v2 branch July 31, 2024 12:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sbernauer sbernauer sbernauer approved these changes

@maltesander maltesander Awaiting requested review from maltesander

@siegfriedweber siegfriedweber Awaiting requested review from siegfriedweber

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Migrate to v2 bundle builder
4 participants
@nightkr @sbernauer @maltesander @siegfriedweber