-
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
Add volumes and volume-mounts for spark dependencies #1359
Conversation
Signed-off-by: Kevin Earls <kearls@redhat.com>
Codecov Report
@@ Coverage Diff @@
## master #1359 +/- ##
=======================================
Coverage 86.21% 86.22%
=======================================
Files 90 90
Lines 5144 5146 +2
=======================================
+ Hits 4435 4437 +2
Misses 539 539
Partials 170 170
Continue to review full report at Codecov.
|
@rubenvp8510 @objectiser @jpkrohling Please review |
LGTM |
pkg/cronjob/spark_dependencies.go
Outdated
}, | ||
}, | ||
RestartPolicy: corev1.RestartPolicyNever, | ||
Affinity: commonSpec.Affinity, | ||
Tolerations: commonSpec.Tolerations, | ||
SecurityContext: commonSpec.SecurityContext, | ||
ServiceAccountName: account.JaegerServiceAccountFor(jaeger, account.DependenciesComponent), | ||
Volumes: jaeger.Spec.Storage.Dependencies.JaegerCommonSpec.Volumes, |
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.
nit: Seems like formatting issue - may be worth running make format
to see?
Signed-off-by: Kevin Earls <kearls@redhat.com>
Signed-off-by: Kevin Earls kearls@redhat.com
This fixes #1332 Note however that TLS access for the spark dependencies job is not fully supported (see issue #294 ) in general but will work for the instance described in this issue.