Skip to content
This repository has been archived by the owner on Jul 27, 2023. It is now read-only.

flannel: start nginx-consul post docker restart #1394

Merged
merged 1 commit into from
May 2, 2016
Merged

flannel: start nginx-consul post docker restart #1394

merged 1 commit into from
May 2, 2016

Conversation

ryane
Copy link
Contributor

@ryane ryane commented Apr 28, 2016

  • Installs cleanly on a fresh build of most recent master branch
  • Upgrades cleanly from the most recent release
  • Updates documentation relevant to the changes

fixes #1391
fixes #1346

@ryane ryane added this to the 1.1 milestone Apr 28, 2016
@ryane ryane mentioned this pull request Apr 28, 2016
Closed
@BrianHicks
Copy link
Contributor

Shouldn't this start logstash too?

@ryane
Copy link
Contributor Author

ryane commented Apr 28, 2016

are you seeing a problem with logstash? it is not installed by default anymore so it won't necessarily be there when this role is installed.

@BrianHicks
Copy link
Contributor

OK, works for me!

@stevendborrelli stevendborrelli merged commit b809fe8 into master May 2, 2016
@stevendborrelli stevendborrelli deleted the fix/k8s-nginx-consul branch May 2, 2016 19:13
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
bug core/kubernetes readytotest
Projects
None yet
Milestone
1.1
Development

Successfully merging this pull request may close these issues.

nginx-consul restart issues on k8s workers nginx-consul not running on kube workers
3 participants
@ryane @BrianHicks @stevendborrelli