Skip to content

Conversation

@fabian4
Copy link
Contributor

@fabian4 fabian4 commented Oct 29, 2025

Proposed changes

Allow disabling cert-generator with a Helm value

Closes #4173

Checklist

Before creating a PR, run through this checklist and mark each as complete.

  • I have read the CONTRIBUTING doc
  • I have added tests that prove my fix is effective or that my feature works
  • I have checked that all unit tests pass after adding my changes
  • I have updated necessary documentation
  • I have rebased my branch onto main
  • I will ensure my PR is targeting the main branch and pulling from my branch from my own fork

Release notes

If this PR introduces a change that affects users and needs to be mentioned in the release notes,
please add a brief note that summarizes the change.

Allow disabling cert-generator with a Helm value

@nginx-bot
Copy link

nginx-bot bot commented Oct 29, 2025

Hi @fabian4!

Thanks for opening this pull request!
Be sure to check out our Contributing Guidelines while you wait for someone on the team to review this.

@nginx-bot nginx-bot bot added the community label Oct 29, 2025
@github-actions github-actions bot added the helm-chart Relates to helm chart label Oct 29, 2025
@fabian4 fabian4 force-pushed the allow_disabling_cert_generator branch from a054b7b to 56a2aaf Compare October 29, 2025 12:41
@github-actions github-actions bot added the documentation Improvements or additions to documentation label Oct 29, 2025
@fabian4 fabian4 force-pushed the allow_disabling_cert_generator branch from 56a2aaf to b199680 Compare October 30, 2025 14:28
@fabian4 fabian4 requested a review from sjberman October 30, 2025 14:30
@fabian4 fabian4 force-pushed the allow_disabling_cert_generator branch from b199680 to 2776315 Compare October 31, 2025 01:05
@fabian4 fabian4 requested a review from sjberman October 31, 2025 01:06
@fabian4 fabian4 force-pushed the allow_disabling_cert_generator branch from 2776315 to 3906aba Compare November 3, 2025 13:35
Copy link
Contributor

@salonichf5 salonichf5 left a comment

Choose a reason for hiding this comment

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

lgtm after Saylor's last edit recommended.

@fabian4 fabian4 force-pushed the allow_disabling_cert_generator branch from e696a0d to cf12a23 Compare November 4, 2025 13:28
Problem: Need to allow disabling cert-generator with a Helm value.

Solution: Allow disabling cert-generator with a Helm value.

Testing: Allow disabling cert-generator with a Helm value.
@ciarams87 ciarams87 force-pushed the allow_disabling_cert_generator branch from cf12a23 to 731b51f Compare November 5, 2025 09:50
@ciarams87 ciarams87 enabled auto-merge (squash) November 5, 2025 13:44
@ciarams87 ciarams87 merged commit 97ebb04 into nginx:main Nov 5, 2025
101 of 104 checks passed
@github-project-automation github-project-automation bot moved this from 🆕 New to ✅ Done in NGINX Gateway Fabric Nov 5, 2025
@fabian4 fabian4 deleted the allow_disabling_cert_generator branch November 6, 2025 00:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

community documentation Improvements or additions to documentation helm-chart Relates to helm chart release-notes

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Allow disabling cert-generator with a Helm value

5 participants