-
Notifications
You must be signed in to change notification settings - Fork 2k
Conversation
This diff looks much easier to review: https://github.com/ehazlett/machine/compare/driver-refactor...ehazlett:x509-check-ip-san?w=1 I should have some more time to take a look tomorrow. |
@md5 👍 thx! |
I actually just amended that URL to add the magical |
c606bce
to
4d19511
Compare
4d19511
to
26d5965
Compare
After auto regenerate certs, the config isn't changed. see this trace...
|
If a swarm machine changes its IP, all swarm clients must also be changed there config. |
@rossbachp thanks. i am trying with elastic ips to see if i can re-create. |
26d5965
to
ed7660b
Compare
@rossbachp yes this is a bug in the IP persistence. Addressing. Thanks! |
Nice, I hope you can fix it:) |
@rossbachp ok a fix has been pushed to the branch. would you mind testing? thanks! |
Works for me:) Many Thanks! |
Works for me too. I think. :) |
Signed-off-by: Evan Hazlett <[email protected]>
Signed-off-by: Evan Hazlett <[email protected]>
Signed-off-by: Evan Hazlett <[email protected]>
85866f2
to
da3da50
Compare
I detect another changed IP problem, after I restart my swarm ec2 cluster today. The master use the old ip's from the swarm machines
I analyze the problem: The swarm agent are join with the old ip
After the IP from swarm machine changed, the implementation must reconfigure the swarm agent, remove the old container and start a new one. Sorry, that I don't detect the problem yesterday! |
Ok thanks for reporting. I think this is a separate issue -- as it's not related specifically to the TLS issue but more of a side effect of the IPs changing (they are related though). Would you mind filing a separate issue? Thanks! |
Yes, I create issue #806. |
@rossbachp thanks! |
This depends on #756.
This will check the remote host when using
config
andenv
commands to see if the remote certificate is valid. If not, it will automatically regenerate new certs and re-configure the engine to use them.This also implements the
regenerate-certs
command from #702 against the new driver refactor.Fixes #531