fix(acs-ci): use absolute gcloud path - #218
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Enterprise Run ID: 📒 Files selected for processing (1)
Included review availability: Your plan provides up to 12 included reviews per hour; 6 remain after this review. WalkthroughThe GCS preflight check now invokes ChangesGCS preflight
Priority: ⬇️ Low Estimated code review effort: 1 (Trivial) | ~3 minutes Change: Bug fix Merge Risk: ⚪ Minimal · up to The preflight command now uses the pinned absolute gcloud path while preserving its existing behavior, so it is mergeable. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
OpenShell replaces the sandbox PATH at runtime, so the first gcloud migration could not find the image-installed CLI even though the image contains it. Use the pinned absolute path in the trusted preflight command; it matches the task policy executable rule.
Summary by CodeRabbit