launchrctl / launchr

Launchr is a CLI action runner that executes actions inside short-lived local containers.
Apache License 2.0
10 stars 2 forks source link

'--remove-image' is overriding error code of action #31

Closed davidferlay closed 6 months ago

davidferlay commented 7 months ago

Observed

➜ launchrctl platform:sync dev --username "XXX" --password "XXX" --vaultpass "XXX" --override xxxxxx; echo $?
Image "platform-actions-sync:latest" exists locally                  
OVERRIDDEN_COMPARISON_REF has been set: xxxxxx
Repository name: ski-platform
Last bump commit identified: xxxxxx
Attempting to download artifact: https://repositories.skilld.cloud/repository/ski-platform-artifacts/ski-platform-xxxxxx-plasma-src.tar.gz
ERROR: Artifact was not found
1

➜ launchrctl platform:sync dev --username "XXX" --password "XXX" --vaultpass "XXX" --override xxxxxx --remove-image; echo $?
Image "platform-actions-sync:latest" exists locally
OVERRIDDEN_COMPARISON_REF has been set: xxxxxx
Repository name: ski-platform
Last bump commit identified: xxxxxx
Attempting to download artifact: https://repositories.skilld.cloud/repository/ski-platform-artifacts/ski-platform-xxxxxx-plasma-src.tar.gz
ERROR: Artifact was not found
Image "platform-actions-sync:latest" was successfully removed
0

Expected

➜ launchrctl platform:sync dev --username "XXX" --password "XXX" --vaultpass "XXX" --override xxxxxx --remove-image; echo $?
Image "platform-actions-sync:latest" exists locally
OVERRIDDEN_COMPARISON_REF has been set: xxxxxx
Repository name: ski-platform
Last bump commit identified: xxxxxx
Attempting to download artifact: https://repositories.skilld.cloud/repository/ski-platform-artifacts/ski-platform-xxxxxx-plasma-src.tar.gz
ERROR: Artifact was not found
Image "platform-actions-sync:latest" was successfully removed
1
lexbritvin commented 6 months ago

Fixed and merged to the main branch