fix: propagate attachmentEnvDrifted error instead of masking as skip

Previously, env-drift probe failures returned (true, nil), silently
treating errors as 'up to date' and skipping attachment updates.

1 file · +1 -2

M internal/usecase/container/service.go +1 -2


@@ -2172,8 +2172,7 @@ func (s Service) handleRunningAttachment(ctx context.Context, existing domain.
2172	2172		// Check env drift
2173	2173		envDrifted, err := s.attachmentEnvDrifted(ctx, existing, containerName, serviceImage)
2174	2174		if err != nil {
2175		-	log.WrapErr(err, "failed to check attachment env drift")
2176		-	return true, nil
	2175	+	return false, log.WrapErr(err, "failed to check attachment env drift")
2177	2176		}
2178	2177
2179	2178		if !envDrifted && !imageDrifted {


@@ -2172,8 +2172,7 @@ func (s Service) handleRunningAttachment(ctx context.Context, existing domain.
2172		// Check env drift	2172		// Check env drift
2173		envDrifted, err := s.attachmentEnvDrifted(ctx, existing, containerName, serviceImage)	2173		envDrifted, err := s.attachmentEnvDrifted(ctx, existing, containerName, serviceImage)
2174		if err != nil {	2174		if err != nil {
2175	-	log.WrapErr(err, "failed to check attachment env drift")	2175	+	return false, log.WrapErr(err, "failed to check attachment env drift")
2176	-	return true, nil
2177		}	2176		}
2178			2177
2179		if !envDrifted && !imageDrifted {	2178		if !envDrifted && !imageDrifted {