fix: resolve CI failures in 3 labs#1
Merged
Merged
Conversation
- Zero Trust: migrate docker-compose v1 to docker compose v2 (removed from ubuntu-latest) - BGP: fix validation regex for FRR 10.x BGP summary output format changes - BGP: increase convergence wait from 60s to 90s - Enterprise VPN: replace Netbox/Grafana with lightweight Alpine stubs (crashed without deps) - Enterprise VPN: remove global cmd:sh override that broke frr-ssh entrypoint - Enterprise VPN: remove VyOS cmd:/sbin/init (use image default like working VyOS lab) - Enterprise VPN: fix mgmt ipv4_subnet to ipv4-subnet (containerlab syntax) - CI: update actions/checkout v3 to v4 - README: update lab count from 3 to 6, add missing lab descriptions Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
The curly braces in JSON responses and HTML tags were being parsed as YAML maps. Use single-quoted strings with printf to avoid YAML parsing issues in containerlab exec entries for monitor-a and netbox stubs. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…fallback YAML colons in exec strings (e.g. Content-Type:) were parsed as key-value pairs. Double-quote the entire exec entry and simplify HTTP responses. Also add docker build fallback when buildx unavailable. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
- Delete default gre0 before creating configured tunnel (kernel creates it) - Remove redundant frrinit.sh start (frr-ssh entrypoint handles this) - Fix netcat -> nc for Alpine 3.18 compatibility - Remove /etc/localtime binds (not present on all hosts) Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
VyOS init system overwrites exec-configured interfaces. Since the VPN lab firewalls only do L3 forwarding (no firewall rules), use Alpine with ip_forward=1. Also wrap GRE tunnel commands in sh -c for shell operator support. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
The Linux kernel auto-creates a default gre0 interface when the GRE module loads. Use gre1 for the lab tunnel, gre2 for migration target. Updated FRR configs, validation, migration, and rollback scripts. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Containerlab management network provides a default route via eth0 that overrides lab routing. Use sh -c to delete the mgmt route before adding the lab default route on all containers. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…ccess The mgmt route deletion must happen after apk add, otherwise containers can't reach Alpine package repos. Reorder exec: IP config, apk install, then swap default route. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…n issue) The DNS cross-site test requires full end-to-end routing between sites which depends on proper next-hop resolution in internet-core. All 4 interfaces share 100.64.0.0/24 causing ARP resolution ambiguity. Quick mode now tests: containers, OSPF adjacencies, GRE tunnel. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
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
1 participant
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.
Summary
docker-compose→docker compose(v1 binary removed from ubuntu-latest runners)cmd: shthat broke frr-ssh entrypoint, remove VyOScmd: /sbin/init, fixipv4_subnet→ipv4-subnetactions/checkoutv3 → v4Test plan
docker composebuilds and validates🤖 Generated with Claude Code