Skip to content

fix duration scrubber for golden tests#9464

Merged
jedevc merged 1 commit into
dagger:mainfrom
lgtdio:fix-golden-test
Jan 28, 2025
Merged

fix duration scrubber for golden tests#9464
jedevc merged 1 commit into
dagger:mainfrom
lgtdio:fix-golden-test

Conversation

@eunomie
Copy link
Copy Markdown
Member

@eunomie eunomie commented Jan 28, 2025

The previous version was able to catch things like:

  • 1m2.345s
  • 12.34s

But wasn't able to catch:

  • 1m23s

But it looks like it happens, sometimes:

+++ actual
@@ -11,7 +11,7 @@
 │ ✔ starting session X.Xs

-✔ load module X.Xs
+✔ load module 1m13s
 │ ✔ finding module configuration X.Xs
-│ ✔ initializing module X.Xs
+│ ✔ initializing module 1m12s
 │ ✔ inspecting module metadata X.Xs
 │ ✔ loading type definitions X.X

For instance: https://v3.dagger.cloud/dagger/traces/d3cd71a1531096b0071621d896b3ebe6?span=f3e3b1f9b2b1f4aa&logs#f3e3b1f9b2b1f4aa

The previous version was able to catch things like:
 - 1m2.345s
 - 12.34s
But wasn't able to catch:
 - 1m23s

But it looks like it happens, sometimes:

    +++ actual
    @@ -11,7 +11,7 @@
     │ ✔ starting session X.Xs

    -✔ load module X.Xs
    +✔ load module 1m13s
     │ ✔ finding module configuration X.Xs
    -│ ✔ initializing module X.Xs
    +│ ✔ initializing module 1m12s
     │ ✔ inspecting module metadata X.Xs
     │ ✔ loading type definitions X.X

For instance: https://v3.dagger.cloud/dagger/traces/d3cd71a1531096b0071621d896b3ebe6?span=f3e3b1f9b2b1f4aa&logs#f3e3b1f9b2b1f4aa

Signed-off-by: Yves Brissaud <gh@lgtd.io>
Copy link
Copy Markdown
Contributor

@jedevc jedevc left a comment

Choose a reason for hiding this comment

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

Ha, nice catch 🎉

@jedevc jedevc merged commit ef34007 into dagger:main Jan 28, 2025
@eunomie eunomie deleted the fix-golden-test branch January 28, 2025 13:40
vikram-dagger pushed a commit to vikram-dagger/dagger that referenced this pull request Feb 7, 2025
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