Skip to content

Use patched version of op-rs that hopefully addresses SUP-148. #717

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

Merged
merged 8 commits into from
Nov 25, 2024
Merged

Use patched version of op-rs that hopefully addresses SUP-148. #717

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
7953a0f
Use patched version of op-rs that hopefully addresses SUP-148.
soenkeliebau Nov 21, 2024
9f69785
Updated cargo.lock to include new commits in op-rs pr (should be no s…
soenkeliebau Nov 22, 2024
0b9090a
Update revision of PR branch.
soenkeliebau Nov 22, 2024
10d5958
Update revision of PR branch.
soenkeliebau Nov 22, 2024
85465e7
Update to use released version of operator-rs and add Changelog entry.
soenkeliebau Nov 23, 2024
3ce5470
Change test condition to look for 'succeeded' instead of 'ready', as …
soenkeliebau Nov 24, 2024
7154486
Merge branch 'main' into fix/SUP-148
siegfriedweber Nov 25, 2024
e4a8e4e
Run pre-commit
siegfriedweber Nov 25, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 2 additions & 0 deletions CHANGELOG.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -31,6 +31,7 @@ All notable changes to this project will be documented in this file.
- Failing to parse one `NifiCluster`/`AuthenticationClass` should no longer cause the whole operator to stop functioning ([#662]).
- NiFi will now use the JDK trust store when an OIDC provider uses WebPKI as CA ([#686], [#698]).
- Fix OIDC endpoint construction in case the `rootPath` does not have a trailing slash ([#718]).
- BREAKING: Use distinct ServiceAccounts for the Stacklets, so that multiple Stacklets can be deployed in one namespace. Existing Stacklets will use the newly created ServiceAccounts after restart ([#717]).

### Removed

Expand All @@ -51,6 +52,7 @@ All notable changes to this project will be documented in this file.
[#698]: https://github.com/stackabletech/nifi-operator/pull/698
[#702]: https://github.com/stackabletech/nifi-operator/pull/702
[#708]: https://github.com/stackabletech/nifi-operator/pull/708
[#717]: https://github.com/stackabletech/nifi-operator/pull/717
[#718]: https://github.com/stackabletech/nifi-operator/pull/718

## [24.7.0] - 2024-07-24
Expand Down
2 changes: 1 addition & 1 deletion tests/templates/kuttl/oidc/12_nifi.yaml.j2
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ spec:
clusterConfig:
authentication:
- authenticationClass: nifi-oidc-auth-class-$NAMESPACE
oidc:
oidc:
clientCredentialsSecret: nifi-oidc-client
sensitiveProperties:
keySecret: nifi-sensitive-property-key
Expand Down
2 changes: 1 addition & 1 deletion tests/templates/kuttl/oidc/20-assert.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -8,4 +8,4 @@ kind: Job
metadata:
name: oidc-login-test
status:
ready: 1 # wait for the test job to start before streaming its logs in the next test step
succeeded: 1 # wait for the test job to start before streaming its logs in the next test step
Loading