Skip to content

feat: ic-management-canister-types 0.7.1 with missing log_memory_store_size#694

Merged
lwshang merged 1 commit into
mainfrom
lwshang/mgmt-types_0.7.1
Mar 3, 2026
Merged

feat: ic-management-canister-types 0.7.1 with missing log_memory_store_size#694
lwshang merged 1 commit into
mainfrom
lwshang/mgmt-types_0.7.1

Conversation

@lwshang
Copy link
Copy Markdown
Contributor

@lwshang lwshang commented Mar 3, 2026

Description

Release 0.7.1 supersedes 0.7.0 (yanked) by adding the missing log_memory_store_size field to MemoryMetrics. Also adds e2e test assertions for memory_metrics and query_stats fields in canister_status_result.

dfinity/portal#6106 will be updated to include spec for it.

How Has This Been Tested?

Added e2e test assertions for memory_metrics and query_stats fields in canister_status_result.

Checklist:

  • The title of this PR complies with Conventional Commits.
  • I have edited the CHANGELOG accordingly.
  • I have made corresponding changes to the documentation.

…re_size`

Release 0.7.1 supersedes 0.7.0 (yanked) by adding the missing
`log_memory_store_size` field to `MemoryMetrics`. Also adds e2e test
assertions for `memory_metrics` and `query_stats` fields in
`canister_status_result`.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Copy link
Copy Markdown
Contributor

@maksymar maksymar left a comment

Choose a reason for hiding this comment

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

LGTM adding log_memory_store_size to memory_metrics.

@lwshang lwshang marked this pull request as ready for review March 3, 2026 15:14
@lwshang lwshang requested a review from a team as a code owner March 3, 2026 15:14
@lwshang lwshang merged commit c78c1fa into main Mar 3, 2026
16 of 17 checks passed
@lwshang lwshang deleted the lwshang/mgmt-types_0.7.1 branch March 3, 2026 15:14
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