-
Notifications
You must be signed in to change notification settings - Fork 50
Closed
Labels
feat/extensionsLabel created for organizing the work related to extensions. Felt pretty, might delete later.Label created for organizing the work related to extensions. Felt pretty, might delete later.kind/enhancementNew feature or requestNew feature or request
Description
Objective
Add upstream registration storage to RegisteredDataStore in internal/extension/registry.go to support the RegisterUpstreamMethod feature.
Details
Add a new registeredUpstreams map alongside the existing dataProviders and subscriptions maps. Introduce the following types:
type RegisteredUpstreamKey struct {
Policy ResourceID
URL string
}
type RegisteredUpstreamEntry struct {
URL string
ClusterName string // generated: ext-{host}-{port}
TargetRef TargetRef // extracted from policy at registration time
FailureMode string
Timeout string
}
type TargetRef struct {
Group string
Kind string
Name string
Namespace string
}Extend ClearPolicyData to also clear registered upstreams for the deleted policy.
Checklist
- Add
registeredUpstreamsmap toRegisteredDataStore - Add
RegisteredUpstreamKey,RegisteredUpstreamEntry,TargetReftypes - Extend
ClearPolicyDatato clear upstreams by policy - Unit tests for CRUD operations and cleanup
Notes
The PR description should include instructions for reviewing and manually confirming the solution functions locally.
References
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
feat/extensionsLabel created for organizing the work related to extensions. Felt pretty, might delete later.Label created for organizing the work related to extensions. Felt pretty, might delete later.kind/enhancementNew feature or requestNew feature or request
Type
Projects
Status
Done