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

Proxies deflector cycle call to make it available on every node. #2448

Merged
merged 4 commits into from Jul 4, 2016

Conversation

Projects
None yet
2 participants
@dennisoelkers
Member

dennisoelkers commented Jul 4, 2016

This PR adds a new cluster resource that finds the master node and
triggers a deflector cycle. This is the last request that needs to be
directed against a master node and after this change, all nodes in a
cluster can be interchangeable called for every request.

dennisoelkers added some commits Jul 4, 2016

Proxies deflector cycle call to make it available on every node.
This PR adds a new cluster resource that finds the master node and
triggers a deflector cycle. This is the last request that needs to be
directed against a master node and after this change, all nodes in a
cluster can be interchangeable called for every request.

@dennisoelkers dennisoelkers added this to the 2.1.0 milestone Jul 4, 2016

@bernd bernd self-assigned this Jul 4, 2016

@@ -43,7 +43,7 @@ const ApiRoutes = {
updatePositions: (dashboardId) => { return { url: `/dashboards/${dashboardId}/positions` }; },
},
DeflectorApiController: {
cycle: () => { return { url: '/system/deflector/cycle' }; },
cycle: () => { return { url: '/cluster/system/deflector/cycle' }; },

This comment has been minimized.

@bernd

bernd Jul 4, 2016

Member

The URL is wrong, this should be /cluster/deflector/cycle.

@bernd

This comment has been minimized.

Member

bernd commented Jul 4, 2016

LGTM 👍

@bernd bernd merged commit 27a07d6 into master Jul 4, 2016

4 checks passed

ci-server-integration Jenkins build graylog2-server-integration-pr 1060 has succeeded
Details
ci-web-linter Jenkins build graylog-pr-linter-check 546 has succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details

@bernd bernd deleted the cluster-deflector-cycling branch Jul 4, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment