Add ability to indicate PriorityClass for collector and query

[majidazimi]

Note:

it seems make generate introduced some unwanted changes in deploy/jaegertracing.io_jaegers_crd.yaml. Not sure whether these changes are required.

[codecov]

Codecov Report

Merging #1413 (55aa8e2) into master (d8fbe6c) will increase coverage by 0.02%.
The diff coverage is 100.00%.

❗ Current head 55aa8e2 differs from pull request most recent head 71c9461. Consider uploading reports for the commit 71c9461 to get more accurate results

@@            Coverage Diff             @@
##           master    #1413      +/-   ##
==========================================
+ Coverage   86.71%   86.73%   +0.02%     
==========================================
  Files          90       90              
  Lines        5021     5029       +8     
==========================================
+ Hits         4354     4362       +8     
  Misses        513      513              
  Partials      154      154              

Impacted Files	Coverage Δ
pkg/apis/jaegertracing/v1/jaeger_types.go	100.00% <ø> (ø)
pkg/deployment/collector.go	100.00% <100.00%> (ø)
pkg/deployment/query.go	100.00% <100.00%> (ø)

---

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 d8fbe6c...71c9461. Read the comment docs.

[jpkrohling]

it seems make generate introduced some unwanted changes in deploy/jaegertracing.io_jaegers_crd.yaml. Not sure whether these changes are required.

That does look odd. Looks like we generated it with v0.3.0 of the controller-gen, but we are installing an earlier version:

jaeger-operator/Makefile

Line 396 in d8fbe6c

sigs.k8s.io/controller-tools/cmd/[email protected] \

Could you try removing the controller-gen and installing it using the following command (if you are using Go 1.16 or higher)?

go install sigs.k8s.io/controller-tools/cmd/[email protected]

[jpkrohling]

Looks great! Thanks for the contribution!

[majidazimi]

So using [email protected] passes all e2e tests.

[jpkrohling]

cc @rubenvp8510 , this needs to be added to the changelog for 1.22