Skip to content

Comments

Move Policy IR to Emitter IR and Align Conversion with Upstream#84

Merged
danehans merged 4 commits intokgateway-dev:mainfrom
danehans:issue_62
Feb 9, 2026
Merged

Move Policy IR to Emitter IR and Align Conversion with Upstream#84
danehans merged 4 commits intokgateway-dev:mainfrom
danehans:issue_62

Conversation

@danehans
Copy link
Collaborator

@danehans danehans commented Feb 6, 2026

What type of PR is this?

/kind cleanup

What this PR does / why we need it:

  • d33a4cd: added emitter-side ingress-nginx IR types/policy aliases and populated ingress-nginx policy data on emitter HTTPRoute IR.
  • d33a4cd: updated agentgateway and kgateway emitters to consume emitter IR policy types instead of provider IR policy types.
  • d33a4cd: added conversion test coverage and updated ingress-nginx/emitter README docs for the IR ownership shift.

Which issue(s) this PR fixes:

Fixes #62

Does this PR introduce a user-facing change?:

NONE

Add emitter_intermediate ingress-nginx policy types and conversion from provider IR.
Update agentgateway and kgateway emitters to consume emitter IR policy types.
Add conversion tests and refresh emitter/provider README guidance.

Signed-off-by: Daneyon Hansen <[email protected]>
- Move ProviderIR -> EmitterIR conversion into provider_intermediate
to match upstream repository structure.
- Remove emitter_intermediate conversion entrypoint.
- Preserve downstream ingress-nginx policy projection into emitter IR.
- Update ingress-nginx provider to call providerir.ToEmitterIR
- Keep conversion coverage in provider_intermediate tests.

Signed-off-by: Daneyon Hansen <[email protected]>
- Remove the standalone emitter_intermediate/ingressnginx.go file.
- Colocate ingress-nginx emitter IR types in emitter_intermediate
and intermediate_representation.go to better match upstream IR layout
conventions.

Signed-off-by: Daneyon Hansen <[email protected]>
- Remove the emitter HTTPRoute IngressNginx wrapper.
- Project ingress-derived feature fields directly on HTTPRouteContext
using provider-neutral type names.
- Restructure provider_intermediate conversion to match upstream shape.
- Move downstream ingress feature projection into a dedicated helper file
for easier future rebases.

Signed-off-by: Daneyon Hansen <[email protected]>
@danehans danehans merged commit f2be47a into kgateway-dev:main Feb 9, 2026
3 checks passed
@danehans danehans deleted the issue_62 branch February 9, 2026 17:10
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.

Move Provider IR Fields to Emitter IR

1 participant