Avoid touching jaeger deps on deployment/ns controller

[rubenvp8510]

Signed-off-by: Ruben Vargas [email protected]

Which problem is this PR solving?

• Fixes Operator tried to delete jaeger-query sidecar #1530

Short description of the changes

• There is a warning for query deployment trying to remove his own Sidecar, I don't want to overcomplicated the injection logic, instead I added a logic at the beginning of deployment and on the namespace controller to just ignore the Jaeger deployments. (label app="jaeger"). Said that I think this deserve a re-factorization, but for now this PR should fix it.

Hope the use of webhook in v2 will simplify a lot of things here.

[codecov]

Codecov Report

Merging #1529 (b19a46b) into master (68cc6ee) will decrease coverage by 0.13%.
The diff coverage is 100.00%.

❗ Current head b19a46b differs from pull request most recent head 924a284. Consider uploading reports for the commit 924a284 to get more accurate results

@@            Coverage Diff             @@
##           master    #1529      +/-   ##
==========================================
- Coverage   87.72%   87.58%   -0.14%     
==========================================
  Files          93       94       +1     
  Lines        5866     5954      +88     
==========================================
+ Hits         5146     5215      +69     
- Misses        553      562       +9     
- Partials      167      177      +10     

Impacted Files	Coverage Δ
pkg/controller/deployment/deployment_controller.go	53.79% <100.00%> (+23.72%)	⬆️
pkg/controller/namespace/namespace_controller.go	39.53% <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 68cc6ee...924a284. Read the comment docs.

[rubenvp8510]

@pavolloffay Could you please review?