p4u1/abra
0
0
Fork 0
forked from toolshed/abra
Code Pull Requests Activity

fix(app): Do not write recipe version on undeploy

Browse Source
This commit is contained in:
p4u1 2025-03-03 16:54:21 +01:00
parent d09a19a385
commit ba0c0c793d
2 changed files with 3 additions and 15 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
cli/app/undeploy.go
View File

@ -59,16 +59,16 @@ Passing "--prune/-p" does not remove those volumes.`,
chaosVersion = deployMeta.ChaosVersion
}
toWriteVersion := deployMeta.Version
version := deployMeta.Version
if deployMeta.IsChaos {
toWriteVersion = chaosVersion
version = chaosVersion
}
if err := internal.UndeployOverview(
app,
deployMeta.Version,
chaosVersion,
toWriteVersion,
version,
); err != nil {
log.Fatal(err)
}
@ -86,10 +86,6 @@ Passing "--prune/-p" does not remove those volumes.`,
log.Fatal(err)
}
}
if err := app.WriteRecipeVersion(toWriteVersion, false); err != nil {
log.Fatalf("writing recipe version failed: %s", err)
}
},
}

8
tests/integration/app_undeploy_overview.bats
View File

@ -64,10 +64,6 @@ teardown(){
# env version
assert_output --regexp 'CURRENT VERSION.*' + "${latestRelease}"
assert_output --regexp 'NEW VERSION.*' + "${headHash:0:8}"
run grep -q "TYPE=$TEST_RECIPE:${headHash:0:8}" \
"$ABRA_DIR/servers/$TEST_SERVER/$TEST_APP_DOMAIN.env"
assert_success
}
@test "chaos deploy with unstaged commits and undeploy" {
@ -93,10 +89,6 @@ teardown(){
assert_output --regexp 'CURRENT VERSION.*' + "${latestRelease}"
assert_output --regexp 'NEW VERSION.*' + "${headHash:0:8}+U"
run grep -q "TYPE=$TEST_RECIPE:${headHash:0:8}" \
"$ABRA_DIR/servers/$TEST_SERVER/$TEST_APP_DOMAIN.env"
assert_success
run rm -rf "$ABRA_DIR/recipes/$TEST_RECIPE/foo"
assert_not_exists "$ABRA_DIR/recipes/$TEST_RECIPE/foo"
}