Skip to content

Conversation

ciarams87
Copy link
Contributor

Proposed changes

Problem: Creating the operator requires creating and validating a bundle to be used for certification

Solution: Created the bundle and made the necessary changes to get it to run on a local cluster.

The main changes required were related to RBAC. This included updating the Helm RBAC definition as we were mixing apiGroups in the rules which was causing validation problems in the Operator.
I also updated the operator Dockerfile to use the Helm chart from the root directory to avoid duplication and drift.

Testing: Installed OLM on my local kind cluster and created the catalog using the bundle. Created an instantiation of NGF and deployed a Gateway successfully. Ran the scorecard tests and validated the bundle.

Please focus on (optional): Note that automation will follow in a separate ticket. Scorecard validation is passing, but this is less rigorous that preflight (the RedHat certification tool) so there may be more work here later.

Closes #3908

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.

NONE

@ciarams87 ciarams87 requested a review from a team as a code owner September 25, 2025 13:54
@github-actions github-actions bot added documentation Improvements or additions to documentation enhancement New feature or request helm-chart Relates to helm chart labels Sep 25, 2025
Copy link
Contributor

@tataruty tataruty left a comment

Choose a reason for hiding this comment

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

please check if my changes in yaml linter and pre-checks can be now removed

@ciarams87 ciarams87 merged commit 0a624fd into feat/openshift-support Sep 25, 2025
45 checks passed
@ciarams87 ciarams87 deleted the feat/operator-bundle branch September 25, 2025 18:59
@github-project-automation github-project-automation bot moved this from 🆕 New to ✅ Done in NGINX Gateway Fabric Sep 25, 2025
ciarams87 added a commit that referenced this pull request Oct 2, 2025
Add bundle, update RBAC, use root helm chart
ciarams87 added a commit that referenced this pull request Oct 2, 2025
Add bundle, update RBAC, use root helm chart
ciarams87 added a commit that referenced this pull request Oct 2, 2025
Add bundle, update RBAC, use root helm chart
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation enhancement New feature or request helm-chart Relates to helm chart
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants