Skip to content

Conversation

scme0
Copy link
Contributor

@scme0 scme0 commented Oct 10, 2025

What

I realised it might be useful to see when exactly these failures occurred so I added them as addition properties on the Warnings and Failure properties.

Related PRs for other repos:

@scme0 scme0 changed the title feat: add warning / failure timestamps feat: add timestamps to failure and warnings Oct 10, 2025
@scme0 scme0 changed the title feat: add timestamps to failure and warnings feat: runtime helm: add timestamps to failure and warnings Oct 10, 2025
@scme0 scme0 force-pushed the feat/add-error-timestamps branch from 96af94c to 5936439 Compare October 12, 2025 07:08
@codefresh-v2-pipelines
Copy link
Contributor

Bumps gitops-operator to version 6881890.

@scme0
Copy link
Contributor Author

scme0 commented Oct 12, 2025

/e2e

@scme0 scme0 marked this pull request as ready for review October 12, 2025 11:18
@scme0 scme0 requested review from a team as code owners October 12, 2025 11:18
registry: quay.io
repository: codefresh/codefresh-gitops-operator
tag: 4d0942a
tag: "6881890"
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@codefresh-io/devops I just realised the automation wrapped the new tag in quotes when it bumped the operator version. Is that what we want?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Example previous PR didn't wrap the tag in quotes: #791

Copy link
Collaborator

Choose a reason for hiding this comment

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

Shouldn't be an issue

@scme0 scme0 merged commit 4216bb7 into main Oct 12, 2025
4 checks passed
@scme0 scme0 deleted the feat/add-error-timestamps branch October 12, 2025 12:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants