Skip to content

[19.03] vendor: swarmkit 0b8364e7d08aa0e972241eb59ae981a67a587a0e#40831

Merged
AkihiroSuda merged 1 commit intomoby:19.03from
thaJeztah:19.03_bump_swarmkit
Apr 17, 2020
Merged

[19.03] vendor: swarmkit 0b8364e7d08aa0e972241eb59ae981a67a587a0e#40831
AkihiroSuda merged 1 commit intomoby:19.03from
thaJeztah:19.03_bump_swarmkit

Conversation

@thaJeztah
Copy link
Copy Markdown
Member

full diff: moby/swarmkit@062b694...0b8364e

full diff: moby/swarmkit@062b694...0b8364e

- Fix leaking tasks.db

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

@cpuguy83 @SamWhited @dperny PTAL

Copy link
Copy Markdown
Contributor

@SamWhited SamWhited left a comment

Choose a reason for hiding this comment

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

Ran vndr and nothing changes and the commit appears right so LGTM.

Copy link
Copy Markdown
Member

@cpuguy83 cpuguy83 left a comment

Choose a reason for hiding this comment

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

LGTM

@AkihiroSuda AkihiroSuda merged commit 6a0995e into moby:19.03 Apr 17, 2020
@thaJeztah thaJeztah deleted the 19.03_bump_swarmkit branch April 17, 2020 07:56
@thaJeztah thaJeztah mentioned this pull request May 29, 2020
@mellertson
Copy link
Copy Markdown

For the uninitiated, what is LGTM short for?

@thaJeztah
Copy link
Copy Markdown
Member Author

"look good to me"

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants