feat(endpoint-exposer): fetch_httproute, clean_httproute_rules y nuevos specs#16
Open
sebastiancorrea81 wants to merge 20 commits into
Open
feat(endpoint-exposer): fetch_httproute, clean_httproute_rules y nuevos specs#16sebastiancorrea81 wants to merge 20 commits into
sebastiancorrea81 wants to merge 20 commits into
Conversation
feat: implement external dns and httproute feat: change logic to get specification id feat: add switch for different dns types feat: add switch for different dns types feat: add endpoint exposer service specs feat: add domain editable feat: add domain editable feat: implement new route schema feat: implement new route schema feat: implement new route schema feat: implement new route schema feat: implement new route schema feat: implement new route schema fix: get scope fix: get scope fix: path fix: get httproute if exists fix: get httproute if exists fix: get httproute rules feat: implement echo for link and unlink feat: implement echo for link and unlink feat: implement echo for link and unlink feat: implement echo for link and unlink feat: implement echo for link and unlink chore: change service slug feat: update flow feat: update flow feat: update flow feat: update flow feat: update flow feat: update flow feat: update flow feat: update flow feat: update flow feat: accept prefix and reg ex
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Resumen
Este branch agrega scripts e infraestructura base que faltan en
main:scripts/istio/fetch_httproute— en lugar de construir el HTTPRoute desde cero siempre, primero busca el existente en K8s y lo reutiliza. Hace el update idempotente.scripts/istio/clean_httproute_rules— resetea el HTTPRoute a la regla fallbackresponse-404antes de reprocesar todas las rutas, evitando reglas huérfanas.templates/istio/empty.yaml.tpl— template alternativo para crear un HTTPRoute vacío de base.specs/service-spec.json.tplyspecs/actions/read.json.tpl— spec del servicio que actualmente solo vive eninstall/.entrypoint,build_context,build_rule,build_ingress_with_rule,process_routes,apply,container-scope-override.Cambio de paradigma vs main
En
mainel HTTPRoute siempre se construye desde cero. Este branch introduce el patrón fetch → clean → rebuild, que es más seguro para updates.Test plan
fetch_httproutefunciona cuando el HTTPRoute ya existe en el clusterfetch_httproutecrea desde template cuando no existeclean_httproute_rulesdeja solo la regla fallback