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

Make retry configurable #161

Merged
merged 1 commit into from
Oct 10, 2017
Merged

Make retry configurable #161

merged 1 commit into from
Oct 10, 2017

Conversation

bdangit
Copy link
Owner

@bdangit bdangit commented Oct 10, 2017

Signed-off-by: Ben Dang [email protected]

@bdangit
Copy link
Owner Author

bdangit commented Oct 10, 2017

I'm sure this change is benign, but running through a few TK setups.

This will also address #150

- Removed check for port 8083 since in v1.3 the web ui is now
provided by telegraf

Signed-off-by: Ben Dang <[email protected]>
@bdangit bdangit changed the title [DNM] Make retry configurable Make retry configurable Oct 10, 2017
@bdangit
Copy link
Owner Author

bdangit commented Oct 10, 2017

ready for review

@bdangit
Copy link
Owner Author

bdangit commented Oct 10, 2017

#149 points out that v1.3+ 8083 webui has pushed off to Chronograph

Copy link
Collaborator

@fishnix fishnix left a comment

Choose a reason for hiding this comment

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

Looks good 👍

@bdangit bdangit merged commit 70e4069 into master Oct 10, 2017
@bdangit bdangit deleted the retry-limit branch October 10, 2017 17:37
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