Skip to content

Sirupsen fix

Alessio Caiazza requested to merge sirupsen-fix into master

What does this MR do?

This MR handles github.com/Sirupsen/logrus rename to github.com/sirupsen/logrus (downcase s)

Why was this MR needed?

We are currently stalled in dependency management, one of the reason is that we have some big old dep that still refers to Sirupsen (namely docker and his friends).

We have no plan to upgrade our docker lib at the moment but this is keeping stalled the whole dependency tree.

Changes:

  • github.com/Sirupsen/logrus -> github.com/sirupsen/logrus
  • github.com/minio/minio-go is no longer forked
  • Docker and some deps have ben forked and edited with find ./ -type f -name "*.go" -print0 | xargs -0 sed -i '' 's/Sirupsen\/logrus/sirupsen\/logrus/g' from the old vendored revision
    • github.com/docker/docker
    • github.com/docker/distribution
    • github.com/opencontainers/runc
  • github.com/Azure/go-ansiterm upgraded the the revision fixing Sirupsen case
  • github.com/prometheus/common upgraded to master
  • github.com/vbatts/tar-split provided PR upstream and updated

Are there points in the code the reviewer needs to double check?

Does this MR meet the acceptance criteria?

  • Documentation created/updated
  • Tests
    • Added for this feature/bug
    • All builds are passing
  • Branch has no merge conflicts with master (if you do - rebase it please)

What are the relevant issue numbers?

#2884

Merge request reports