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

Fix spelling #189

Merged
merged 3 commits into from
Nov 11, 2021
Merged

Fix spelling #189

merged 3 commits into from
Nov 11, 2021

Conversation

ecl996
Copy link
Contributor

@ecl996 ecl996 commented Nov 10, 2021

What this PR does / why we need it:

Minor spelling mistake in the file

@banzo banzo requested a review from zakaria2905 November 10, 2021 21:55
values.yaml Outdated
@@ -69,7 +69,7 @@ sts:

properties:
# use externalSecure for when inbound SSL is provided by nginx-ingress or other external mechanism
sensitiveKey: changeMechangeMe # Must to have minnimal 12 length key
sensitiveKey: changeMechangeMe # Must to have minimal 12 length key
alogorithm: NIFI_PBKDF2_AES_GCM_256
Copy link
Contributor

Choose a reason for hiding this comment

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

there is one in alogorithm too I am guessing

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ah yes! Added to the PR

Copy link
Contributor

@zakaria2905 zakaria2905 left a comment

Choose a reason for hiding this comment

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

Could you please update the chart version to 1.0.2 in the Chart.yaml file

@ecl996
Copy link
Contributor Author

ecl996 commented Nov 11, 2021

Could you please update the chart version to 1.0.2 in the Chart.yaml file

Yup, all set!

Copy link
Contributor

@zakaria2905 zakaria2905 left a comment

Choose a reason for hiding this comment

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

Your changes have been accepted and will be merged, thanks.

@zakaria2905 zakaria2905 merged commit 4247cbf into cetic:master Nov 11, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants