Add E2E test suite and CI workflows#8
Open
scotwells wants to merge 6 commits into04-deploymentfrom
Open
Conversation
This was referenced Mar 17, 2026
scotwells
force-pushed
the
05-e2e-and-ci
branch
2 times, most recently
from
f8870e1 to
a91dd8d
Compare
- Add Chainsaw-based E2E test suite with five test scenarios: system-readiness, bgp-peering, session-lifecycle, bgp-advertisement, and bgp-route-policy - Add kind cluster config (kind-config.yaml) with multi-node topology for realistic BGP peering tests - Add chainsaw-config.yaml with test timeouts and parallel execution settings - Add test/e2e/overlay/kustomization.yaml for deploying the controller into the kind cluster during tests - Add test/e2e/fixtures/ with shared baseline BGPConfiguration fixture - Add Taskfile.yml for running the full E2E suite locally - Add .github/workflows/ci.yaml with lint, build, unit test, and E2E stages - Add .github/workflows/docker.yaml for building and pushing the container image to ghcr.io on push to main and on tags Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Remove SessionState, ReceivedPrefixes, AdvertisedPrefixes, LastTransitionTime, and FlapCount from BGPSessionStatus. These operational counters are now exposed exclusively as Prometheus metrics to eliminate the 10-second polling loop that wrote to every BGPSession object cluster-wide. Conditions (Configured and SessionEstablished) remain on the CRD status for kubectl visibility. Update printcolumns to show Configured and Established condition status instead of the removed sessionState and receivedPrefixes fields. Regenerate CRD manifest and update deepcopy accordingly. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Delete status.go and its RunStatusPoller goroutine, which polled GoBGP every 10 seconds and wrote to every BGPSession object in the cluster. This caused multi-writer races in the DaemonSet deployment and generated unnecessary API server load. Replace with inline logic in the SessionReconciler: - After AddPeer/UpdatePeer succeeds, call ListPeer for the specific peer to get its current state in the same reconcile pass. - Set the SessionEstablished condition based on the live GoBGP state. - Emit bgp_session_state, bgp_received_prefixes_total, and bgp_session_flaps_total Prometheus metrics from the reconciler. - Return RequeueAfter: 30s so the condition and metrics are refreshed periodically. Each node only updates sessions it owns (LocalEndpoint match), eliminating the multi-writer race. Move peerStateToString from status.go into session_reconciler.go since it is only needed there now. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Replace assertions on .status.sessionState with assertions on the SessionEstablished condition now that the sessionState field has been removed from BGPSessionStatus. - bgp-peering test: query SessionEstablished condition status instead of sessionState string when counting established sessions. - session-lifecycle test: assert SessionEstablished condition=True instead of sessionState: Established in the chainsaw assert step. Update docs/api/README.md: - Remove sessionState, receivedPrefixes, advertisedPrefixes, lastTransitionTime, and flapCount from the BGPSession status table. - Add a Prometheus metrics table for the operational counters that moved out of the CRD status. - Update the Verifying Configuration section to use the condition JSONPath. - Update the printed columns description (Local, Remote, Configured, Established). Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
These tests still queried .status.sessionState which was removed in the status simplification. Use the SessionEstablished condition instead. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
scotwells
force-pushed
the
05-e2e-and-ci
branch
from
3a349d4 to
56c6936
Compare
Update container image paths from ghcr.io/datum-cloud/bgp to ghcr.io/milo-os/bgp, update doc repo references, and update copyright holder to Datum Technology, Inc. Co-Authored-By: Claude Sonnet 4.6 <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
kind-config.yamlconfiguring a multi-node kind cluster for realistic BGP peering topologychainsaw-config.yamlwith per-test timeouts and parallel execution settingstest/e2e/overlay/kustomization.yamlfor deploying the controller into the kind cluster during test runstest/e2e/fixtures/with a shared baseline BGPConfiguration used across multiple test casestest/e2e/Taskfile.ymlso engineers can run the full E2E suite locally withtask e2e.github/workflows/ci.yamlwith lint, build, unit test, and E2E stages running on every PR.github/workflows/docker.yamlbuilding and pushing the container image toghcr.io/datum-cloud/bgpon push to main and on semver tagsContext
This is part of a stacked PR series:
🤖 Generated with Claude Code