Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Kubernetes provider: traefik.frontend.rule.type logging #668

Merged
merged 1 commit into from Sep 13, 2016
Merged

Kubernetes provider: traefik.frontend.rule.type logging #668

merged 1 commit into from Sep 13, 2016

Conversation

yvespp
Copy link
Contributor

@yvespp yvespp commented Sep 12, 2016

Don't log a warning if traefik.frontend.rule.type is empty, log namespace and ingress if invalide.

Removes the spamming of time="2016-08-18T18:37:14+02:00" level=warning msg="Unknown RuleType ``, falling back to \PathPrefix"` for every rule.

Example: https://gist.githubusercontent.com/yvespp/3e7aac2990d1124a0bd631326eaa2ff7/raw/a10c28cd0639434ab9b30dec84a7583525d6dc64/traefik.log4

…e is empty, log namespace and ingress if invalide.
@yvespp
Copy link
Contributor Author

yvespp commented Sep 12, 2016

Test failure is in test-integration https_test.go: https://travis-ci.org/containous/traefik/jobs/159399215#L2083

I don't know what causes this an can't reproduce it localy with docker 1.12. Ideas?

@errm
Copy link
Contributor

errm commented Sep 12, 2016

Looks like test flake to me, trying a rerun on travis..

@errm
Copy link
Contributor

errm commented Sep 12, 2016

LGTM ping @containous/traefik

@vdemeester
Copy link
Contributor

LGTM 🐸

@vdemeester vdemeester merged commit 9a2b7cf into traefik:master Sep 13, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants