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 nerdctl stats output #1298

Merged
merged 1 commit into from
Aug 16, 2022

Conversation

fahedouch
Copy link
Member

@fahedouch fahedouch commented Aug 4, 2022

thanks to @liubin to detect the issue (#1278 (comment))

fix stats output and ensure error display (#1278)

Signed-off-by: fahed dorgaa [email protected]

@fahedouch fahedouch force-pushed the fix-nerdctl-stats-output branch 2 times, most recently from 47b1280 to ac9e56a Compare August 4, 2022 16:18
Copy link
Member

@junnplus junnplus left a comment

Choose a reason for hiding this comment

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

Thanks, LGTM

@fahedouch fahedouch force-pushed the fix-nerdctl-stats-output branch 2 times, most recently from b0a7aa0 to c8f715e Compare August 9, 2022 17:47
Signed-off-by: fahed dorgaa <[email protected]>
@fahedouch fahedouch force-pushed the fix-nerdctl-stats-output branch from c8f715e to b71febc Compare August 9, 2022 17:47
@AkihiroSuda AkihiroSuda added this to the v0.22.3 milestone Aug 15, 2022
Copy link
Member

@AkihiroSuda AkihiroSuda left a comment

Choose a reason for hiding this comment

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

Thanks

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