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

Adding a configurable timeout for calls to the loki server. #368

Merged
merged 3 commits into from
Mar 5, 2019

Conversation

slim-bean
Copy link
Collaborator

Also updating promtail test to set sane defaults for all configurations and then override for testing.
Removing .idea folder from .gitignore, we should encourage people (and myself) to use global ignore for IDE configs

Fixes #264

Updating promtail test to set sane defaults for all configurations and then override for testing.
Removing .idea folder from .gitignore, we should encourage people (and myself) to use global ignore for IDE configs
@slim-bean slim-bean requested review from tomwilkie and woodsaj March 4, 2019 16:37
…the cancel rather than having to call it explicitly.

Removed some unnecessary struct initialization.
Copy link
Contributor

@tomwilkie tomwilkie left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM with one nit.

@slim-bean slim-bean merged commit 19260d4 into master Mar 5, 2019
@slim-bean slim-bean deleted the promtail-timeout branch March 5, 2019 21:07
xperimental pushed a commit to xperimental/loki that referenced this pull request Nov 18, 2024
Update from upstream repository
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants