-
Notifications
You must be signed in to change notification settings - Fork 3.5k
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
Support promtail reload config #573
Conversation
bce1070
to
266b291
Compare
7e2094b
to
40dab72
Compare
36e6f00
to
c0380cc
Compare
3675366
to
43ee199
Compare
Signed-off-by: Xiang Dai <764524258@qq.com>
Signed-off-by: Xiang Dai <764524258@qq.com>
Signed-off-by: Xiang Dai <764524258@qq.com>
@daixiang0: Would this also cause promtail to reload TLS client config, even if say |
@slim-bean @cyriltovena hi, mind have a look for this? If meaningful, i would update and fix conflicts. |
This issue has been automatically marked as stale because it has not had any activity in the past 30 days. It will be closed in 7 days if no further activity occurs. Thank you for your contributions. |
@slim-bean @cyriltovena any update? |
This issue has been automatically marked as stale because it has not had any activity in the past 30 days. It will be closed in 7 days if no further activity occurs. Thank you for your contributions. |
@sh0rez @rfratto @slim-bean any input? |
Hi, we have completely refactored and rewritten the configuration framework used in the meantime, so these changes are obsolete ;) In case this issue still needs a fix, please open a new PR :D |
Based on comment
Fix #16
Signed-off-by: Xiang Dai 764524258@qq.com