Skip to content

tests/integration: print times for each step and flush to see it in CI#1547

Merged
kkysen merged 7 commits intomasterfrom
kkysen/tests-integration-misc
Jan 14, 2026
Merged

tests/integration: print times for each step and flush to see it in CI#1547
kkysen merged 7 commits intomasterfrom
kkysen/tests-integration-misc

Conversation

@kkysen
Copy link
Contributor

@kkysen kkysen commented Jan 14, 2026

@fw-immunant, this should hopefully fix why it wasn't printing each step until everything was finished. I also added times for each step (in addition to the total times for each project that we print at the end).

@kkysen kkysen requested a review from fw-immunant January 14, 2026 21:59
Copy link
Contributor

@fw-immunant fw-immunant left a comment

Choose a reason for hiding this comment

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

Code looks good, and seems to work in CI. That said, I don't know if additional explicit timestamps are really necessary given that GH Actions tracks them separately, but it's not a problem to have them.

Copy link
Contributor Author

@kkysen kkysen left a comment

Choose a reason for hiding this comment

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

That said, I don't know if additional explicit timestamps are really necessary given that GH Actions tracks them separately, but it's not a problem to have them.

Wait where does it track them?

@kkysen kkysen merged commit 520d67f into master Jan 14, 2026
11 checks passed
@kkysen kkysen deleted the kkysen/tests-integration-misc branch January 14, 2026 22:55
@fw-immunant
Copy link
Contributor

That said, I don't know if additional explicit timestamps are really necessary given that GH Actions tracks them separately, but it's not a problem to have them.

Wait where does it track them?

Click "Show timestamps" under the gear icon on the top-right of a CI job log.

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