Skip to content

[v0.2.x] Update logrus to v1.0.1#1252

Merged
crosbymichael merged 2 commits intocontainerd:v0.2.xfrom
dmcgowan:0.2.x-update-logrus
Aug 1, 2017
Merged

[v0.2.x] Update logrus to v1.0.1#1252
crosbymichael merged 2 commits intocontainerd:v0.2.xfrom
dmcgowan:0.2.x-update-logrus

Conversation

@dmcgowan
Copy link
Member

@dmcgowan dmcgowan commented Jul 26, 2017

Fix case-sensitivity issue in containerd files for importers

Fix case-sensitivity issue in containerd files

Signed-off-by: Derek McGowan <derek@mcgstyle.net>
@dmcgowan
Copy link
Member Author

Still needs some of the dependencies updated

Ensure vendors import correct logrus.
go-connections and go-ansiterm are already updated upstream.
docker/docker is in progress.
Changing hash in this change was avoided to make update smaller.

Signed-off-by: Derek McGowan <derek@mcgstyle.net>
@dmcgowan
Copy link
Member Author

Added a change which does a dirty vendor for 3 of the dependencies. 2 of those are already fixed upstream and any future vendor updates should pick up the change. The docker/docker change is a work in progress and needs this change in order to be completed properly. 🐔 🥚

@vdemeester
Copy link
Contributor

@crosbymichael
Copy link
Member

LGTM

@crosbymichael crosbymichael merged commit f6d7c16 into containerd:v0.2.x Aug 1, 2017
@dmcgowan dmcgowan deleted the 0.2.x-update-logrus branch September 10, 2019 17:45
ianlewis pushed a commit to ianlewis/containerd that referenced this pull request Dec 8, 2020
Update cri-tools to fix all image reference test failure.
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