Skip to content

Commit a687416

Browse files
committed
fix(build): include Dockerfile and build scripts in change detection
The supervisor committed-tree fingerprint was missing deploy/docker/Dockerfile.images, and neither gateway nor supervisor included tasks/scripts/docker-build-image.sh. Changes to these files (e.g. from rebasing main) would not trigger a rebuild. Align the git ls-tree paths with the matches_* functions so committed and uncommitted changes are detected consistently.
1 parent edd8fea commit a687416

File tree

1 file changed

+9
-6
lines changed

1 file changed

+9
-6
lines changed

tasks/scripts/cluster-deploy-fast.sh

Lines changed: 9 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -149,13 +149,13 @@ matches_gateway() {
149149
Cargo.toml|Cargo.lock|proto/*|deploy/docker/cross-build.sh)
150150
return 0
151151
;;
152-
crates/openshell-core/*|crates/openshell-policy/*|crates/openshell-providers/*)
152+
deploy/docker/Dockerfile.images|tasks/scripts/docker-build-image.sh)
153153
return 0
154154
;;
155-
crates/openshell-router/*)
155+
crates/openshell-core/*|crates/openshell-policy/*|crates/openshell-providers/*)
156156
return 0
157157
;;
158-
crates/openshell-server/*|deploy/docker/Dockerfile.images)
158+
crates/openshell-router/*|crates/openshell-server/*)
159159
return 0
160160
;;
161161
*)
@@ -170,10 +170,13 @@ matches_supervisor() {
170170
Cargo.toml|Cargo.lock|proto/*|deploy/docker/cross-build.sh)
171171
return 0
172172
;;
173+
deploy/docker/Dockerfile.images|tasks/scripts/docker-build-image.sh)
174+
return 0
175+
;;
173176
crates/openshell-core/*|crates/openshell-policy/*|crates/openshell-router/*)
174177
return 0
175178
;;
176-
crates/openshell-sandbox/*|deploy/docker/Dockerfile.images)
179+
crates/openshell-sandbox/*)
177180
return 0
178181
;;
179182
*)
@@ -206,10 +209,10 @@ compute_fingerprint() {
206209
local committed_trees=""
207210
case "${component}" in
208211
gateway)
209-
committed_trees=$(git ls-tree HEAD Cargo.toml Cargo.lock proto/ deploy/docker/cross-build.sh crates/openshell-core/ crates/openshell-policy/ crates/openshell-providers/ crates/openshell-router/ crates/openshell-server/ deploy/docker/Dockerfile.images 2>/dev/null || true)
212+
committed_trees=$(git ls-tree HEAD Cargo.toml Cargo.lock proto/ deploy/docker/cross-build.sh deploy/docker/Dockerfile.images tasks/scripts/docker-build-image.sh crates/openshell-core/ crates/openshell-policy/ crates/openshell-providers/ crates/openshell-router/ crates/openshell-server/ 2>/dev/null || true)
210213
;;
211214
supervisor)
212-
committed_trees=$(git ls-tree HEAD Cargo.toml Cargo.lock proto/ deploy/docker/cross-build.sh crates/openshell-core/ crates/openshell-policy/ crates/openshell-router/ crates/openshell-sandbox/ 2>/dev/null || true)
215+
committed_trees=$(git ls-tree HEAD Cargo.toml Cargo.lock proto/ deploy/docker/cross-build.sh deploy/docker/Dockerfile.images tasks/scripts/docker-build-image.sh crates/openshell-core/ crates/openshell-policy/ crates/openshell-router/ crates/openshell-sandbox/ 2>/dev/null || true)
213216
;;
214217
helm)
215218
committed_trees=$(git ls-tree HEAD deploy/helm/openshell/ 2>/dev/null || true)

0 commit comments

Comments
 (0)