Skip to content

ILS-2074 Cache gateway resources only in operator namespace#1063

Merged
Michal-Szczygiel merged 4 commits intodevelopfrom
ILS-2074-gateway-resources-caching
Apr 14, 2026
Merged

ILS-2074 Cache gateway resources only in operator namespace#1063
Michal-Szczygiel merged 4 commits intodevelopfrom
ILS-2074-gateway-resources-caching

Conversation

@Michal-Szczygiel
Copy link
Copy Markdown
Collaborator

Description

Bug fixes:

  • Cache gateway resources only in operator namespace
  • Additionally: skip stale ns cleanup if no operator group CRD-s exist (valid for no OLM installs)

Parent issue

ILS-2074

Type

What types of changes does your code introduce?
Put an x in the boxes that apply

  • Bugfix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Feature change (non-breaking change which modifies existing functionality)
  • Dependency/version upgrade/CVE remediation
  • Velocity-improvement (enhancing testing strategy, tidy code, CICD update)

Extra information

  • I have added tests that prove my fix is effective or that my feature works
  • I have added the necessary documentation (if appropriate)
  • Lint and unit tests pass locally with my changes
  • Tester is needed
  • This change requires a documentation update - create separate issue with input for doc team

Testing

  • Manual tests
  • Automated sert tests:
    Before merging your PR, ensure Jenkins tests pass by following these steps:
  1. Run the tests on the clustered environment (OCP/IKS depending on the content of the pull request) using Jenkins pipelines.
  2. If the build fails due to being unstable, restart it.
  3. If the failure is due to other issues unrelated to the tests themselves, address the underlying problem before proceeding.

Test images for further QA testing (if applicable):

Comment thread main.go Outdated
&corev1.Pod{}: {Label: licensingLabelSelector},
&gatewayv1.Gateway{}: {Namespaces: operatorNamespaceOnly},
&gatewayv1.HTTPRoute{}: {Namespaces: operatorNamespaceOnly},
&gatewayv1.BackendTLSPolicy{}: {Namespaces: operatorNamespaceOnly},
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

will this work if there are no gateway API CRDs on the cluster?
we also probably should consider this switch here: IsGatewayAPI

@Michal-Szczygiel Michal-Szczygiel marked this pull request as ready for review April 13, 2026 08:26
Comment thread main.go Outdated
@Michal-Szczygiel Michal-Szczygiel changed the base branch from develop to master April 13, 2026 10:33
@Michal-Szczygiel Michal-Szczygiel changed the base branch from master to develop April 14, 2026 08:48
@Michal-Szczygiel Michal-Szczygiel merged commit 4a45b7b into develop Apr 14, 2026
2 of 3 checks passed
@Michal-Szczygiel Michal-Szczygiel deleted the ILS-2074-gateway-resources-caching branch April 14, 2026 10:59
Michal-Szczygiel added a commit that referenced this pull request Apr 15, 2026
* Cache gateway resources only in operator ns

* Check for CRD-s before setting cache

* Exist with status = 1 after k8s API check failure

* Refactoring

---------

Co-authored-by: Michal Szczygiel <Michal.Szczygiel@ibm.com>
Michal-Szczygiel added a commit that referenced this pull request Apr 16, 2026
…1066)

* ILS-2074 Cache gateway resources only in operator namespace (#1063)

* Cache gateway resources only in operator ns

* Check for CRD-s before setting cache

* Exist with status = 1 after k8s API check failure

* Refactoring

---------

Co-authored-by: Michal Szczygiel <Michal.Szczygiel@ibm.com>

* bump go to 1.26.2

* Version bump to 4.2.22 - LS only

---------

Co-authored-by: Michal Szczygiel <Michal.Szczygiel@ibm.com>
Co-authored-by: Karol Matoga <Karol.Matoga@ibm.com>
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