Skip to content
This repository has been archived by the owner on Jun 20, 2024. It is now read-only.

Change log level of "Vetoed installation of hairpin flow" to WARNING #2808

Closed
brb opened this issue Feb 22, 2017 · 1 comment
Closed

Change log level of "Vetoed installation of hairpin flow" to WARNING #2808

brb opened this issue Feb 22, 2017 · 1 comment
Assignees
@bboreham
Labels
chore [component/router]
Milestone
1.9.5

Comments

@brb
Copy link
Contributor

brb commented Feb 22, 2017

It seems that "Vetoed installation of hairpin flow" happens not only in harmful cases, e.g. kontena/kontena#1834 (comment) . Logging it as an error might confuse end users.
@bboreham bboreham changed the title Change a log level of "Vetoed installation of hairpin flow" to WARNING Change log level of "Vetoed installation of hairpin flow" to WARNING Mar 8, 2017
@brb
Copy link
Contributor Author

brb commented Apr 3, 2017

Also, the current DetectHairpin failed: Hairpin mode enabled on "vethwe-bridge" error msg could be improved to warn a user about possible connectivity issues.

@bboreham bboreham self-assigned this May 3, 2017
@bboreham bboreham mentioned this issue May 3, 2017
Merged
marccarre added a commit that referenced this issue May 3, 2017
@marccarre
Merge remote-tracking branch 'origin/1.9'
507e734 
- PR #2926 - Branch issues/2808-improve-hairpin-messages - Fixes #2808.
- PR #2907 - Branch issues/2771-checkpoint-net-modes - Fixes #2771.
@bboreham bboreham added this to the 1.9.5 milestone May 4, 2017
@bboreham bboreham mentioned this issue Mar 1, 2019
Open
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@bboreham bboreham

Labels
chore [component/router]
Projects
None yet
Milestone
1.9.5
Development

No branches or pull requests
2 participants
@brb @bboreham