Skip to content
This repository has been archived by the owner on Feb 12, 2024. It is now read-only.

Make compatible with apache/nifi >= 1.19.0 (#313) #314

Merged
merged 3 commits into from
Dec 20, 2023

Conversation

jrebmann
Copy link
Contributor

What this PR does / why we need it:

This pull request allows apache/nifi versions >=0.19.0 to be released with this helm chart.

As described in issue #313, this only required the keystore type to be explicitly set to JKS when calling keytool to be compatible with Java 11.

This change is backward compatible with Java 8.

Which issue this PR fixes

Checklist

  • DCO signed
  • Chart Version bumped

Signed-off-by: Jannik Rebmann <jannik.rebmann@ise.fraunhofer.de>
@jrebmann
Copy link
Contributor Author

jrebmann commented Sep 29, 2023

I just took a look at it. I don't think the build fails because of my change.

The unit test OIDC Logins / OIDC (cluster, Ingress, cert-manager local issuer) fails since commit f296a8e (Jun 15, 23).

Is the problem already known?

@banzo
Copy link
Contributor

banzo commented Nov 6, 2023

yeah, not sure what's wrong.

#319 did not have an issue with the tests, is this PR still relevant?

@jrebmann
Copy link
Contributor Author

jrebmann commented Nov 6, 2023

Absolutely! This PR is essential to use Apache NiFi versions >=1.19.0.

@jrebmann jrebmann mentioned this pull request Nov 23, 2023
@banzo banzo merged commit 89857aa into cetic:master Dec 20, 2023
9 of 10 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[cetic/nifi] Make helm chart compatible with apache/nifi >= 1.19.0
2 participants