Skip to content

Commit

Permalink
move delete bool
Browse files Browse the repository at this point in the history
  • Loading branch information
tedim52 committed Nov 30, 2023
1 parent a35fc3e commit ae39458
Showing 1 changed file with 1 addition and 3 deletions.
4 changes: 1 addition & 3 deletions engine/server/engine/enclave_manager/enclave_creator.go
Original file line number Diff line number Diff line change
Expand Up @@ -83,13 +83,12 @@ func (creator *EnclaveCreator) CreateEnclave(
}()

// only create log collector for backend as
var shouldDeleteLogsCollector bool
shouldDeleteLogsCollector := true
if kurtosisBackendType == args.KurtosisBackendType_Docker {
// TODO the logs collector has a random private ip address in the enclave network that must be tracked
if _, err := creator.kurtosisBackend.CreateLogsCollectorForEnclave(setupCtx, enclaveUuid, defaultTcpLogsCollectorPortNum, defaultHttpLogsCollectorPortNum); err != nil {
return nil, stacktrace.Propagate(err, "An error occurred creating the logs collector with TCP port number '%v' and HTTP port number '%v'", defaultTcpLogsCollectorPortNum, defaultHttpLogsCollectorPortNum)
}
shouldDeleteLogsCollector = true
defer func() {
if shouldDeleteLogsCollector {
err = creator.kurtosisBackend.DestroyLogsCollectorForEnclave(teardownCtx, enclaveUuid)
Expand All @@ -113,7 +112,6 @@ func (creator *EnclaveCreator) CreateEnclave(
cloudUserID,
cloudInstanceID,
)

if err != nil {
return nil, stacktrace.Propagate(err, "An error occurred launching the API container")
}
Expand Down

0 comments on commit ae39458

Please sign in to comment.