feat: Rename transaction, report status and required approval tag

[armenzg]

There's various features in here:

• Setting different titles depending on the action: save snapshots vs run
  • Before this, they were all named "visual snapshots" even though the top span of each was named correctly
• Report the status of a transaction (instead of currently being always unknown)
  • This can be Ok when there are no errors and InternalError when anything fails
• Tag a transaction as needing approval or not
  • This is useful to determine how many executions required a human to approve the execution

This means that we will be able to do the following in Discover:

• failure_rate() -> A failure is when the execution fails in an unexpected manner
• requiring approval -> The amount of times a developer is unable to merge a PR until approving a check

A failure has a status of internal_error. No tag saying if it requires approval or not.

A transaction with approval required and a transaction with NO approval required. Both of them have a status of Ok.

Here's a query with data base on this branch.

[armenzg]

Hi @JonasBa any initial thoughts?

[armenzg]

I'm merging the PR and I will follow up the comments on a new PR. I want Jonas to have this so we can measure the impact of his changes.