chore: bump duty to v1.0.802 (#1736)

* chore: bump duty to v1.0.802

* fix: don't save agent_id on playbook_run_actions when running on agents

the agent_id is the agent's identity on the upstream.
the agent itself doesn't have itself as an agent in its db

---------

Co-authored-by: moshloop <[email protected]>
Co-authored-by: Aditya Thebe <[email protected]>

Author: 3 people

go.mod

@@ -14,7 +14,7 @@ require (
 	github.com/containrrr/shoutrrr v0.8.0
 	github.com/fergusstrange/embedded-postgres v1.29.0 // indirect
 	github.com/flanksource/commons v1.35.3
-	github.com/flanksource/duty v1.0.800
+	github.com/flanksource/duty v1.0.802
 	github.com/flanksource/gomplate/v3 v3.24.50
 	github.com/flanksource/kopper v1.0.11
 	github.com/gomarkdown/markdown v0.0.0-20240419095408-642f0ee99ae2

go.sum

@@ -443,8 +443,8 @@ github.com/flanksource/artifacts v1.0.14 h1:Vv70bccsae0MwGaf/uSPp34J5V1/PyKfct9z
 github.com/flanksource/artifacts v1.0.14/go.mod h1:qHVCnQu5k50aWNJ5UhpcAKEl7pAzqUrFFKGSm147G70=
 github.com/flanksource/commons v1.35.3 h1:EG46iWodmSQQbXywjvEAgK56ZH26jYtMv0RiPM3eKDE=
 github.com/flanksource/commons v1.35.3/go.mod h1:cLZURmvF0dw3wR5VyPuibRl/7h+NEiyMdo70WhJFB9Y=
-github.com/flanksource/duty v1.0.800 h1:tf9JdPKmpYXHlm/y1BAcnJev1aJfuv+eKLXf0ALuBs4=
-github.com/flanksource/duty v1.0.800/go.mod h1:SUJg5MSfqt0UUM2l2hP/pPAsipS0Px5L8btRD7fLwM8=
+github.com/flanksource/duty v1.0.802 h1:5OLObNalnoWB5wdkzr30DNPHz7IvEZ3UJRodUyX6H8Q=
+github.com/flanksource/duty v1.0.802/go.mod h1:SUJg5MSfqt0UUM2l2hP/pPAsipS0Px5L8btRD7fLwM8=
 github.com/flanksource/gomplate/v3 v3.24.50 h1:z8wKtFqp6yqKf2aiNfKeiG57/JlvYmpGULApQI+4rM4=
 github.com/flanksource/gomplate/v3 v3.24.50/go.mod h1:N8lVd8fnm6gS0l3JWlcJgso2Ol9P2L7CW00gvuIHEk0=
 github.com/flanksource/is-healthy v1.0.57 h1:y9lyxu4GxWcghmT8uWAkg0gaC8bAUdJ4aOUGDpe4oQo=

playbook/jobs.go

@@ -99,8 +99,8 @@ func PullPlaybookAction(ctx job.JobRuntime, upstreamConfig upstream.UpstreamConf
 		Record(time.Duration(time.Since(response.Action.ScheduledTime).Milliseconds()))
 
 	err = ctx.Transaction(func(ctx context.Context, _ trace.Span) error {
-		// Don't save playbook_run_id to avoid foreign key constraint
-		if err := ctx.DB().Omit("playbook_run_id").Save(&response.Action).Error; err != nil {
+		// Don't save playbook_run_id & agent_id to avoid foreign key constraint
+		if err := ctx.DB().Omit("playbook_run_id", "agent_id").Save(&response.Action).Error; err != nil {
 			return oops.Wrap(err)
 		}