Skip to content

Commit 957d79a

Browse files
chore: Cleanup after successful update (#2758)
remove pkg file after successful update Co-authored-by: Ian Adams <ian.adams@observiq.com>
1 parent 7477a09 commit 957d79a

1 file changed

Lines changed: 6 additions & 0 deletions

File tree

opamp/observiq/observiq_client.go

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -22,6 +22,7 @@ import (
2222
"fmt"
2323
"net/http"
2424
"net/url"
25+
"os"
2526
"slices"
2627
"sync"
2728
"time"
@@ -739,6 +740,11 @@ func (c *Client) finishPackageInstall(pkgStatuses *protobufs.PackageStatuses) {
739740
if err := c.opampClient.SetPackageStatuses(pkgStatuses); err != nil {
740741
c.logger.Error("OpAMP client failed to set package statuses", zap.Error(err))
741742
}
743+
744+
// safely remove the package status artifact now that we've successfully updated
745+
if err := os.Remove(packagestate.DefaultFileName); err != nil {
746+
c.logger.Warn("Failed to remove package status artifact after successful update", zap.Error(err))
747+
}
742748
}
743749

744750
// getVerifiedPackageStatuses returns the last available PackagesStatuses info only if

0 commit comments

Comments
 (0)