-
Notifications
You must be signed in to change notification settings - Fork 345
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
Deploy production ready self provisioned ES by default #585
Conversation
Signed-off-by: Pavol Loffay <ploffay@redhat.com>
@kevinearls @jkandasa heads up this PR can break your internal tests as it changes the ES deployment If you previously were not specifying the number of nodes and resources. |
Signed-off-by: Pavol Loffay <ploffay@redhat.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Only minor comment.
pkg/storage/elasticsearch.go
Outdated
@@ -130,6 +130,11 @@ func (ed *ElasticsearchDeployment) InjectSecretsConfiguration(p *corev1.PodSpec) | |||
// Elasticsearch returns an ES CR for the deployment | |||
func (ed *ElasticsearchDeployment) Elasticsearch() *esv1.Elasticsearch { | |||
uuid := strings.Replace(util.DNSName(ed.Jaeger.Namespace+ed.Jaeger.Name), "-", "", -1) | |||
//res := ed.Jaeger.Spec.Storage.Elasticsearch.Resources |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
left over?
Signed-off-by: Pavol Loffay <ploffay@redhat.com>
Codecov Report
@@ Coverage Diff @@
## master #585 +/- ##
=========================================
Coverage ? 91.24%
=========================================
Files ? 73
Lines ? 3667
Branches ? 0
=========================================
Hits ? 3346
Misses ? 228
Partials ? 93
Continue to review full report at Codecov.
|
1 similar comment
Codecov Report
@@ Coverage Diff @@
## master #585 +/- ##
=========================================
Coverage ? 91.24%
=========================================
Files ? 73
Lines ? 3667
Branches ? 0
=========================================
Hits ? 3346
Misses ? 228
Partials ? 93
Continue to review full report at Codecov.
|
Resolves #560
By default deploy 3 nodes and set resources to 16Gb and 1 CPU.
Signed-off-by: Pavol Loffay ploffay@redhat.com