Skip to content

[release/1.4 backport] snapshot/devmapper: log exported methods correctly#5765

Merged
dmcgowan merged 1 commit intocontainerd:release/1.4from
thaJeztah:1.4_backport_dm_log
Jul 21, 2021
Merged

[release/1.4 backport] snapshot/devmapper: log exported methods correctly#5765
dmcgowan merged 1 commit intocontainerd:release/1.4from
thaJeztah:1.4_backport_dm_log

Conversation

@thaJeztah
Copy link
Copy Markdown
Member

backport of #5655

  • View was somehow logging itself as "prepare"
  • Cleanup should have its debug log as like other exported methods

- View was somehow logging itself as "prepare"
- Cleanup should have its debug log as like other exported methods

Signed-off-by: Kazuyoshi Kato <katokazu@amazon.com>
(cherry picked from commit 03ee450)
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@thaJeztah
Copy link
Copy Markdown
Member Author

@kzys @AkihiroSuda PTAL

@AkihiroSuda I saw you marked this for backporting; a bit unsure if this was needed for 1.4 as well (seems rather minimal, although overall looks "safe")

@theopenlab-ci
Copy link
Copy Markdown

theopenlab-ci Bot commented Jul 20, 2021

Build succeeded.

Copy link
Copy Markdown
Member

@dmcgowan dmcgowan left a comment

Choose a reason for hiding this comment

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

LGTM

@dmcgowan dmcgowan merged commit 11996c1 into containerd:release/1.4 Jul 21, 2021
@thaJeztah thaJeztah deleted the 1.4_backport_dm_log branch July 21, 2021 22:50
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