Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Negative timings show up when running pulumi destroy with --refresh=true #11188

Closed
justinvp opened this issue Oct 28, 2022 · 0 comments · Fixed by #11631
Closed

Negative timings show up when running pulumi destroy with --refresh=true #11188

justinvp opened this issue Oct 28, 2022 · 0 comments · Fixed by #11631
Assignees
Labels
area/cli UX of using the CLI (args, output, logs) kind/bug Some behavior is incorrect or out of spec resolution/fixed This issue was fixed
Milestone

Comments

@justinvp
Copy link
Member

justinvp commented Oct 28, 2022

What happened?

$ pulumi destroy --skip-preview --refresh=true
Destroying (dev)

View Live: https://app.pulumi.com/justinvp/timings/dev/updates/2

     Type                 Name         Status
 -   pulumi:pulumi:Stack  timings-dev  deleted (-0.00s)

Resources:
    - 1 deleted

Screen Shot 2022-10-28 at 3 24 55 PM

I've also seen other negative numbers, like:

Screen Shot 2022-10-24 at 4 22 03 PM

Steps to reproduce

mkdir timings && cd timings
pulumi new yaml
pulumi up --skip-preview
pulumi destroy --skip-preview --refresh=true

It also happens when the refresh option is set in Pulumi.yaml:

options:
  refresh: always

Expected Behavior

Expected non-negative numbers and -0.00 should just be 0 (w/out the sign and decimal).

Actual Behavior

Actual "deleted (-0.00s)"

Contributing

Vote on this issue by adding a 👍 reaction.
To contribute a fix for this issue, leave a comment (and link to your pull request, if you've opened one already).

@justinvp justinvp added kind/bug Some behavior is incorrect or out of spec area/cli UX of using the CLI (args, output, logs) needs-triage Needs attention from the triage team and removed needs-triage Needs attention from the triage team labels Oct 28, 2022
@dixler dixler added this to the 0.82 milestone Dec 12, 2022
@bors bors bot closed this as completed in feac805 Dec 13, 2022
@pulumi-bot pulumi-bot added the resolution/fixed This issue was fixed label Dec 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/cli UX of using the CLI (args, output, logs) kind/bug Some behavior is incorrect or out of spec resolution/fixed This issue was fixed
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants