fix(deploy): rename hardened Kubernetes manifest to wardnet.yaml - #144
fix(deploy): rename hardened Kubernetes manifest to wardnet.yaml#144seonghobae wants to merge 20 commits into
Conversation
|
Note Reviews pausedIt looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the Use the following commands to manage reviews:
Use the checkboxes below for quick actions:
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Team Run ID: 📒 Files selected for processing (1)
Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review. 📝 WalkthroughWalkthroughKubernetes 매니페스트를 ChangesKubernetes 매니페스트 경로 마이그레이션
Estimated code review effort: 3 (Moderate) | ~20 minutes Merge Risk: ⚪ Minimal · up to This change completes the Wardnet manifest-path migration while retaining controlled migration and rollback references. No concrete current-head merge-blocking risk remains. 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✨ Finishing Touches 💡 1📝 Generate docstrings 💡
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Noema LLM review
The PR correctly renames the hardened Kubernetes manifest from deploy/kubernetes/waf-ids-ai-soc.yaml to deploy/kubernetes/wardnet.yaml and updates all corresponding references in documentation, agent guides, and source code. Crucially, it introduces a structural regression contract in tests/kubernetes_manifest_path.rs to prevent the accidental reintroduction of the legacy path in operational contexts, while allowing it for explicit migration history. It also adds tests/documentation_landing.rs to ensure GitHub Pages links resolve correctly, addressing a previously identified 404 risk.
Reviewed changed lines
AGENTS.md:20 (RIGHT): Correctly updates the remediation path for k8s/IaC misconfigs to the new filename.tests/deployment_manifest.rs:651 (RIGHT): Adds a concrete test to ensure the legacy manifest file is physically absent from the distributable tree.tests/kubernetes_manifest_path.rs:78 (RIGHT): Implements a granular allowlist for legacy paths, distinguishing between dangerous operational references (e.g., kubectl apply) and safe historical documentation.tests/documentation_landing.rs:63 (RIGHT): Implements a repository-wide link validator for the landing page to prevent broken links to the main branch.
Adversarial validation
tests/kubernetes_manifest_path.rs:145 (RIGHT)falsified: The stale-reference scanner can be bypassed by placing the legacy path in a non-text file. — The scanner explicitly iterates over TEXT_EXTENSIONS; however, the legacy path is a source-tree reference typically found in docs/scripts, which are covered.tests/kubernetes_manifest_path.rs:102 (RIGHT)falsified: The scanner fails if the test file itself contains the legacy path literal as a negative test case. — The test uses runtime fragment concatenation (lines 138-140) to avoid literal matches, ensuring the test file remains subject to the scan without triggering it.- Residual risk: low
Findings
- No blocking findings.
- Result: APPROVE
- Head SHA:
9616b94ac1ecf70038071a8c9395348694e6312c - Reviewer credential:
noema-review-github-app-refresh - Actor:
cwl-noema-review[bot]
Fixes #75 only after this exact bounded slice reaches protected
main.Buyer / operator outcome
This branch is the foundation-first repository-path migration for the hardened Kubernetes asset plus its public/operator documentation landing:
deploy/kubernetes/wardnet.yamlwithout renaming live Kubernetes resource identities;docs/index.mdlanding and repository-link validation;The latest pre-restack source repair,
8d656d313c18c2b304d3e1a563317a967c23e250, is valid intervening delta rather than a race: it only rewrites one iterator closure intests/kubernetes_manifest_path.rsto satisfy the Rust 1.98 Clippy contract without changing the asserted path semantics.Fresh protected-main integration — 2026-09-05 KST
Protected/default
mainis exact5829a0f08d78de464dd24393ce5d0f25fba9d126after #159. The previous candidate was 19 protected commits behind. Fresh compare showed the protected-only delta was the six-file workflow-control foundation from #159;AGENTS.mdwas the only semantic overlap.The non-force three-way repair keeps #159's canonical Actions ownership/concurrency/explicit-runner guidance and applies only this lane's Kubernetes source-path substitution inside that file. Protected CI/Fuzz/Scorecard and workflow queue/runner contract files were adopted exactly from protected main. Merge commit
04692b24c179d1c067113f83c52a4ad1b4af7c5apreserves exact8d656d3...and exact protected main as parents; no force push or destructive rebase was used.Fresh compare now reports
behind_by=0. The protected-main-relative effective delta is 13 paths: AGENTS/CHANGELOG/CLAUDE/README, one core documentation string, the manifest rename, buyer/deployment/doctoring docs,docs/index.md, and three path/deployment/documentation regressions.Stack / single-writer decision
This narrower path-only migration is the prerequisite for broad rename PR #114. #114 also renames the same Kubernetes asset and many surrounding product/runtime identities, so it must remain Draft and later non-force adopt the protected result of this PR rather than racing two overlapping rename authorities. If this slice is replaced, the successor must prove complete transfer of every unique path/documentation/test delta before #144 can be retired.
Exact-head gate
Exact current head is
04692b24c179d1c067113f83c52a4ad1b4af7c5a. Fresh runs are non-passing/queued: Security Scan33908029943, CodeQL PR33908029896, SAST Semgrep33908029998, CI33908029976, Fuzz33908029968. All predecessor workflow conclusions are historical after ancestry movement.Keep Draft until this unchanged head has terminal repository/security/coverage/package/SBOM/provenance/central-review/thread/governance evidence. No self/model approval, routine administrator bypass, force push, gate weakening, no-op redispatch commit, or predecessor-evidence reuse.