Skip to content

Conversation

@haavardeide
Copy link
Contributor

@haavardeide haavardeide commented Feb 11, 2026

@haavardeide haavardeide requested a review from a team as a code owner February 11, 2026 12:25
stekern
stekern previously approved these changes Feb 11, 2026
Copy link
Member

@stekern stekern left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good stuff 👌

Long-term we might want to consider if this is how we want to collect the data, and if this format includes what we want. But to see our progress in the short-term, I like this!

Have you tested this for one of our reference applications?

@haavardeide
Copy link
Contributor Author

Good stuff 👌

Long-term we might want to consider if this is how we want to collect the data, and if this format includes what we want. But to see our progress in the short-term, I like this!

Have you tested this for one of our reference applications?

Not yet since first part was agreeing on format :) The plan is to do that thursday morning

…lies

Add always() to the Datadog metric step so it runs even when
terraform apply fails, allowing failed deployments to be reported.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants