Skip to content

fix(controller): Fix infra registry check#171

Merged
danielpanzella merged 1 commit intov2from
cabernathy/conditional_gateway_crds
Apr 29, 2026
Merged

fix(controller): Fix infra registry check#171
danielpanzella merged 1 commit intov2from
cabernathy/conditional_gateway_crds

Conversation

@casey-coreweave
Copy link
Copy Markdown
Contributor

@casey-coreweave casey-coreweave commented Apr 29, 2026

Bugfix: fix IsServerResource comparison

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 29, 2026

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: d8907ea2-4083-477f-be62-0293292859d8

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch cabernathy/conditional_gateway_crds

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@casey-coreweave casey-coreweave force-pushed the cabernathy/conditional_gateway_crds branch from c783278 to 1a459f1 Compare April 29, 2026 16:12
@casey-coreweave casey-coreweave changed the title fix(ingress): Conditional support for GatewayAPI fix(controller): Fix infra registry check Apr 29, 2026
@danielpanzella danielpanzella merged commit a795aaa into v2 Apr 29, 2026
6 of 7 checks passed
@danielpanzella danielpanzella deleted the cabernathy/conditional_gateway_crds branch April 29, 2026 16:19
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.

2 participants