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.

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

Feature/nifinodegroupenabling #439

Merged

Conversation

aswathinj-pai
Copy link
Contributor

Q A
Bug fix? yes
New feature? no
API breaks? no
Deprecations? no
Related tickets fixes #X, partially #Y, mentioned in #Z
License Apache 2.0

What's in this PR?

The current helm chart doesn't handle the replicas attribute in the nifinodegroupautoscaler template in nifi-cluster helm chart. Consequently, it defaults to 0, disabling the HPA created alongside it and preventing autoscaling. Therefore, to enable this feature, we need to manage the replicas attribute in the nifi-node-group-autoscaler.yaml template.

Checklist

  • Implementation tested
  • User guide and development docs updated (if needed)
  • Append changelog with changes

To Do

  • If the PR is not complete but you want to discuss the approach, list what remains to be done here

@mh013370
Copy link
Member

mh013370 commented Sep 5, 2024

@aswathinj-pai : Can you please just update the CHANGELOG with this feature?

@juldrixx juldrixx self-requested a review September 5, 2024 08:37
@aswathinj-pai
Copy link
Contributor Author

Updated the changelog @mh013370

helm/nifi-cluster/values.yaml Outdated Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
@juldrixx
Copy link
Contributor

juldrixx commented Sep 6, 2024

And can you fix the conflicts?

@aswathinj-pai
Copy link
Contributor Author

@juldrixx conflict resolved.

CHANGELOG.md Outdated Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
aswathinj-pai and others added 2 commits September 6, 2024 17:18
Co-authored-by: Juldrixx <[email protected]>
Co-authored-by: Juldrixx <[email protected]>
@juldrixx juldrixx merged commit 216e17b into konpyutaika:master Sep 6, 2024
4 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants