Waypoint action sequence numbers are now reported on agent run completion

[HenryEstberg]

Changes proposed in this PR:

When Waypoint agents have completed an action, they will now report the action sequence number like other types of Waypoint actions.

Note

The linter is failing on this PR with errors logged for cloud-vault-secrets commands, the failure is unrelated to changes in this PR. This is due to the update to the hcp-sdk-go version.

How I've tested this PR:

I have manually tested this by creating an agent group and a corresponding action, then running hcp waypoint agent run and launching the action to allow the Waypoint agent to pick it up. I then verified that the correct sequence number was displayed.

How I expect reviewers to test this PR:

Reviewers can test this PR by running an agent action of their own.

% ./hcp waypoint agent run --config=./agent.hcl 
2024-09-23T12:30:37.315-0700 [INFO]  hcp.waypoint.agent.run: Waypoint agent initialized: hcp-org=*** hcp-project=*** waypoint-namespace=*** groups=["sequence-num-testing"]
2024-09-23T12:31:37.924-0700 [INFO]  hcp.waypoint.agent.run: reporting action run starting: action-run-id=fe192bd6-30e7-4037-8734-e7f3ca6c8648 group=sequence-num-testing operation=test-null
2024-09-23T12:31:38.082-0700 [INFO]  hcp.waypoint.agent.run: finished operation: action-run-id=fe192bd6-30e7-4037-8734-e7f3ca6c8648 group=sequence-num-testing operation=test-null
2024-09-23T12:31:38.082-0700 [INFO]  hcp.waypoint.agent.run: reporting action run ended: action-run-id=fe192bd6-30e7-4037-8734-e7f3ca6c8648 group=sequence-num-testing operation=test-null status="output: hello" status-code=0 sequence=2

Checklist:

• Tests added if applicable
• CHANGELOG entry added or label 'pr/no-changelog' added to PR

  Run CHANGELOG_PR=<PR number> make changelog/new-entry for guidance
  in authoring a changelog entry, and commit the resulting file, which should
  have a name matching your PR number. Entries should use imperative present
  tense (e.g. Add support for...)

[briancain]

I think you can do this and we can save ourself a nested if:

if resp != nil && resp.Payload != nil { ... }

[briancain]

We should be more specific here just so it's clear:

Suggested change

	log.Info("reporting action run ended", "status", status, "status-code", statusCode, "sequence", sequenceNum)
	log.Info("reporting action run ended", "status", status, "status-code", statusCode, "action-run-sequence", sequenceNum)