Skip to content

fix(plugins): collect exposed ingress resources if service-proxy disabled#1912

Merged
IvoGoman merged 3 commits intomainfrom
docs/cluster-disable-service-proxy
Apr 13, 2026
Merged

fix(plugins): collect exposed ingress resources if service-proxy disabled#1912
IvoGoman merged 3 commits intomainfrom
docs/cluster-disable-service-proxy

Conversation

@IvoGoman
Copy link
Copy Markdown
Contributor

Description

If the service-proxy is disabled for a Cluster this should not disable the collection of ingress resources.

This ensures only exposed services are ignored.
If the service list is empty and the service-proxy is disabled the status will reflect this.
This documents the feature of disabling the service proxy for a cluster.

What type of PR is this? (check all applicable)

  • 🍕 Feature
  • 🐛 Bug Fix
  • 📝 Documentation Update
  • 🎨 Style
  • 🧑‍💻 Code Refactor
  • 🔥 Performance Improvements
  • ✅ Test
  • 🤖 Build
  • 🔁 CI
  • 📦 Chore (Release)
  • ⏩ Revert

Related Tickets & Documents

Added tests?

  • 👍 yes
  • 🙅 no, because they aren't needed
  • 🙋 no, because I need help
  • Separate ticket for tests # (issue/pr)

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

Added to documentation?

  • 📜 README.md
  • 🤝 Documentation pages updated
  • 🙅 no documentation needed
  • (if applicable) generated OpenAPI docs for CRD changes

Checklist

  • My code follows the style guidelines of this project
  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • My changes generate no new warnings
  • New and existing unit tests pass locally with my changes

@IvoGoman IvoGoman requested review from a team as code owners April 10, 2026 12:21
@github-actions github-actions bot added size/L documentation Improvements or additions to documentation core-apis labels Apr 10, 2026
@IvoGoman IvoGoman force-pushed the docs/cluster-disable-service-proxy branch from bc89e13 to e9b64e6 Compare April 10, 2026 12:23
@IvoGoman IvoGoman linked an issue Apr 10, 2026 that may be closed by this pull request
3 tasks
@IvoGoman IvoGoman force-pushed the docs/cluster-disable-service-proxy branch from 1f19408 to 5edd766 Compare April 10, 2026 14:18
uwe-mayer
uwe-mayer previously approved these changes Apr 13, 2026
@IvoGoman IvoGoman force-pushed the docs/cluster-disable-service-proxy branch from 5edd766 to 06334ee Compare April 13, 2026 13:03
…bled

If the service-proxy is disabled for a Cluster this should not disable the collection of ingress resources.

This ensures only exposed services are ignored.
If the service list is empty and the service-proxy is disabled the status will reflect this.
This documents the feature of disabling the service proxy for a cluster.
@IvoGoman IvoGoman force-pushed the docs/cluster-disable-service-proxy branch from 06334ee to 7ee8ccb Compare April 13, 2026 13:49
@IvoGoman IvoGoman merged commit e2d49e3 into main Apr 13, 2026
22 checks passed
@IvoGoman IvoGoman deleted the docs/cluster-disable-service-proxy branch April 13, 2026 13:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

core-apis documentation Improvements or additions to documentation size/L

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[FEAT] - Disable service-proxy for a Cluster

3 participants