Skip to content

Improve information if IsStateDirty() fails#2909

Merged
dperny merged 1 commit into
moby:masterfrom
thaJeztah:dirty_details
Jul 21, 2022
Merged

Improve information if IsStateDirty() fails#2909
dperny merged 1 commit into
moby:masterfrom
thaJeztah:dirty_details

Conversation

@thaJeztah
Copy link
Copy Markdown
Member

this was a commit that was left over in #2837, and thought it could still be useful when debugging problems.

Before:

panic: unexpected field type in StoreSnapshot

After:

panic: unexpected field type in StoreSnapshot: XXX_NoUnkeyedLiteral struct

Before:

    panic: unexpected field type in StoreSnapshot

After:

    panic: unexpected field type in StoreSnapshot: XXX_NoUnkeyedLiteral struct

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@thaJeztah
Copy link
Copy Markdown
Member Author

ping @dani-docker @dperny PTAL

@codecov
Copy link
Copy Markdown

codecov Bot commented Oct 20, 2019

Codecov Report

Merging #2909 into master will decrease coverage by 0.14%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master    #2909      +/-   ##
==========================================
- Coverage   61.62%   61.48%   -0.15%     
==========================================
  Files         139      139              
  Lines       22615    22615              
==========================================
- Hits        13936    13904      -32     
- Misses       7194     7231      +37     
+ Partials     1485     1480       -5

@dani-docker
Copy link
Copy Markdown
Contributor

yes please!
LGTM 👍

@thaJeztah
Copy link
Copy Markdown
Member Author

@dperny PTAL 🤗

@thaJeztah
Copy link
Copy Markdown
Member Author

@dperny @tonistiigi @crazy-max PTAL

Copy link
Copy Markdown
Member

@crazy-max crazy-max left a comment

Choose a reason for hiding this comment

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

LGTM

@dperny dperny merged commit 298f092 into moby:master Jul 21, 2022
@thaJeztah thaJeztah deleted the dirty_details branch July 21, 2022 17:42
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.

4 participants