Skip to content

Conversation

@sjawhar
Copy link
Contributor

@sjawhar sjawhar commented Nov 29, 2017

What does this PR do?

The intention is just to bump the version of lego up to aa94fb46, to add support for the GoDaddy DNS provider. But glide and I don't seem to get along very well. It seems to just want to update ALL THE THINGS. I tried just running

./script/glide.sh get github.com/xenolf/lego#aa94fb4696349fd6f35168dabe3ab030ddf3e484

but I'm told "lego is already in glide.yaml, dummy!"

I've seen that some of you have made very small changes to glide.yaml successfully, and I'd love to know how I can do it too!

Motivation

Now that that go-acme/lego#416 has been merged, it would be great it the version of lego could be bumped here in traefik so I don't have to keep making custom builds :)

build.Dockerfile Outdated
Copy link
Contributor Author

@sjawhar sjawhar Nov 29, 2017

Choose a reason for hiding this comment

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

When I built the traefik-dev image from build.Dockerfile, I noticed the DOCKER_VERSION environment variable was set to 17.09.0-ce. Maybe Docker changed this recently? Anyway, the build wouldn't complete because it kept trying to download docker-17.09.0-ce-ce.tgz, which doesn't work so well.

glide.lock Outdated
Copy link
Contributor Author

@sjawhar sjawhar Nov 29, 2017

Choose a reason for hiding this comment

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

This right here and the GoDaddy DNS provider on line 561 are all I really want to change. Any help accomplishing that would be much appreciated. I'm very much a noob at gliding.

@ldez
Copy link
Contributor

ldez commented Nov 29, 2017

@sjawhar Thanks for your interest in Traefik 😃

I will fix your PR.

Copy link
Contributor

@ldez ldez left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Member

@mmatur mmatur left a comment

Choose a reason for hiding this comment

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

LGTM

@sjawhar
Copy link
Contributor Author

sjawhar commented Nov 29, 2017

@ldez Thanks for the assist! So I can be more helpful in the future, what's the right way to update glide.yaml and .lock as minimally as you did?

Copy link
Member

@juliens juliens left a comment

Choose a reason for hiding this comment

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

LGTM

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

Labels

area/acme kind/enhancement a new or improved feature. size/S

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants