Fix log when spak depenencies are used with unsupported storage

[pavolloffay]

Resolves #157

Signed-off-by: Pavol Loffay ploffay@redhat.com

[jpkrohling]

This change is 

[codecov]

Codecov Report

Merging #161 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##           master     #161   +/-   ##
=======================================
  Coverage   96.36%   96.36%           
=======================================
  Files          30       30           
  Lines        1430     1430           
=======================================
  Hits         1378     1378           
  Misses         40       40           
  Partials       12       12

Impacted Files	Coverage Δ
pkg/strategy/production.go	100% <100%> (ø)	⬆️
pkg/strategy/all-in-one.go	100% <100%> (ø)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9adb6a2...f49471f. Read the comment docs.

[jpkrohling]

Reviewed 2 of 2 files at r1.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @pavolloffay and @objectiser)

---

pkg/strategy/all-in-one.go, line 86 at r1 (raw file):

			os = append(os, cronjob.CreateSparkDependencies(c.jaeger))
		} else {
			logrus.WithField("type", c.jaeger.Spec.Storage.Type).Warn("Spark dependencies is enabled for unsupported storage")

Suggestion: "Skipping spark dependencies job due to unsupported storage."

[pavolloffay]

Changed, I will merge this as there will be conflicts in #162 and I won't be available. I will fix any outstanding issues in follow pr.

[jpkrohling]

Reviewed 2 of 2 files at r2.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @objectiser)