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

fix: Add idle_cycles_burned_per_day field #3943

Merged
merged 2 commits into from
Oct 8, 2024
Merged

Conversation

berestovskyy
Copy link
Contributor

The field was missed in the canister status output.

The field was missed in the canister status output.
@berestovskyy berestovskyy requested a review from a team as a code owner October 7, 2024 11:52
@ByronBecker
Copy link

@berestovskyy just to clarify, I believe that idle_cycles_burned_per_day value already includes the compute_allocation additions in it?

So if I update my canister to have a 1% compute allocation (from zero), the idle_cycles_burned_per_day seems to increase by 864 billion (per percentage point increase).

Is this correct?

@berestovskyy
Copy link
Contributor Author

Yes, this field is used internally and returned in canister_status, but it's not currently visible in dfx for some reason. By increasing the compute allocation by 1%, we should see the idle_cycles_burned_per_day field increase by 10M cycles multiplied by 86,4K seconds (one day), which equals 864 billion cycles.

@lwshang lwshang enabled auto-merge (squash) October 8, 2024 19:28
@lwshang lwshang merged commit 7618065 into master Oct 8, 2024
296 checks passed
@lwshang lwshang deleted the andriy/add-cycles-burned branch October 8, 2024 20:23
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.

3 participants