Skip to content

update to github.com/sirupsen/logrus v1.0.0 [carry #1136]#1234

Merged
stevvooe merged 2 commits intocontainerd:masterfrom
dmcgowan:update-logrus
Jul 21, 2017
Merged

update to github.com/sirupsen/logrus v1.0.0 [carry #1136]#1234
stevvooe merged 2 commits intocontainerd:masterfrom
dmcgowan:update-logrus

Conversation

@dmcgowan
Copy link
Member

[Carry #1136 for upstream changes]

Pending microsoft/hcsshim#136

Fix some dependencies by updating casing to github.com/sirupsen/logrus (The project has moved.)

go test still fails with plugin/context.go:8:2: case-insensitive import collision. Likely need to fix the logrus casing in containerd's dependencies' code as well.

Andrew Pennebaker and others added 2 commits July 21, 2017 15:39
Signed-off-by: Andrew Pennebaker <apennebaker@datapipe.com>
Signed-off-by: Derek McGowan <derek@mcgstyle.net>
@codecov-io
Copy link

Codecov Report

Merging #1234 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1234   +/-   ##
=======================================
  Coverage   27.91%   27.91%           
=======================================
  Files          28       28           
  Lines        2826     2826           
=======================================
  Hits          789      789           
  Misses       1888     1888           
  Partials      149      149
Impacted Files Coverage Δ
fs/diff.go 41.43% <ø> (ø) ⬆️
snapshot/btrfs/btrfs.go 48.26% <ø> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update dd7642f...1491293. Read the comment docs.

@stevvooe
Copy link
Member

LGTM

@stevvooe stevvooe merged commit 4118a25 into containerd:master Jul 21, 2017
@dmcgowan dmcgowan deleted the update-logrus branch September 10, 2019 17:44
ianlewis pushed a commit to ianlewis/containerd that referenced this pull request Dec 8, 2020
…to-fix-race

Update containerd to fix panic caused by race condition.
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