Byi more replicas #23
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
samjsong
reviewed
May 7, 2019
| @@ -155,7 +155,7 @@ spec: | |||
| selector: | |||
| matchLabels: | |||
| k8s-app: fluentd-sumologic | |||
| replicas: 1 | |||
| replicas: 3 | |||
There was a problem hiding this comment.
Is there any data duplication with 3 replicas of Fluentd running? Or when Prometheus writes to the remote write endpoint do the metrics only get sent to one of the 3 replicas?
There was a problem hiding this comment.
No data duplication. the idea is much similar as our receiver nodes. from outside (e.g. Prometheus) there is only single endpoint (which we defined as a kubernetes service). The traffic will be distributed to 3 nodes. We don't have any session or partition concept here so basically it's just randomly pick anyone
There was a problem hiding this comment.
Nice, thanks for confirming
samjsong
approved these changes
May 7, 2019
psaia
pushed a commit
to psaia/sumologic-kubernetes-collection
that referenced
this pull request
May 25, 2021
* start creating test account resources * add sumo env var * move account config into tf config * tf fmt * sign pipeline * specify dag * limit concurrency * update * separate pipelines * chain the pipelines, remove second fmt step * move sumo_env to the drone file * remove vars file * update pipeline * remove experiments
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Tuning fluentd resource to fix Context canceled issue
3replica rather than12Gito256Mi(each container)200mto100m(each container)Monitoring for couple of hours and looks like no more
context cancelederror in Prometheus log nowFYI - definition of Kubernetes resource limitation
@frankreno @ggarg2906sumo