opentelemetry-operator icon indicating copy to clipboard operation
opentelemetry-operator copied to clipboard

feat(collector): adds spec.hostPID for all collector modes

Open olandr opened this issue 4 months ago • 3 comments

Description: At the moment it is not possible to set the spec.hostPID field for the collector (regardless of type). This change adds a featuregate operator.security.hostpid that allows a user to configure the spec.hostPID field for the collector.

Inspiration for the code follows from the functionality of the hostNetwork boolean field Link to tracking Issue(s): 4214

  • Resolves: #4214

Testing: <Describe what testing was performed and which tests were added.>

  • adds e2e tests for all types
  • adds unit tests for configuring the hostPID depending on featureflag activation.

Documentation: <Describe the documentation added.> Autogenerated docs from the type defs.


on-behalf-of: @SAP Simon Olander ([email protected])

olandr avatar Aug 13 '25 10:08 olandr

CLA Signed

The committers listed above are authorized under a signed CLA.

  • :white_check_mark: login: olandr / name: Simon Olander (0bfed4f9b0f219a71d6710f02c5585cdf728d7ca, 14dd1afc3ae511ce5dea658f374469ca7e43c641, 1c81f16f48a094673f5436f4eb66e6ce31c78df0, 6c70c68e4bb8fd6871c38b2aa9c0f6bf71d883c1, 738db3503cb6460658291d1ff4ce810c8c764ca2, 7f8ddbd065047513a8eeed6ac84cddf8e8179a48, 9434f286f5bcfd23eee324fb860f9595925c5fff, a46e8d4455b2bd082ef2de6fe17039bd811e41c7, c8f8b1671e9302132b7382b7c019a02f4f556ef3, dbc9b33a8915bd06e41e26e870f8479ee2d538a5)

E2E Test Results

 33 files   - 1  223 suites   - 4   1h 58m 18s ⏱️ - 1m 35s  90 tests ±0   89 ✅  - 1  0 💤 ±0  1 ❌ +1  227 runs   - 4  225 ✅  - 6  0 💤 ±0  2 ❌ +2 

For more details on these failures, see this check.

Results for commit 9434f286. ± Comparison against base commit 12ec60fd.

:recycle: This comment has been updated with latest results.

github-actions[bot] avatar Oct 07 '25 11:10 github-actions[bot]

I just added a case in the issue to reproduce the error. Which this change resolves.

olandr avatar Oct 10 '25 09:10 olandr

@olandr you need to run make update to make the CI pass.

swiatekm avatar Dec 16 '25 11:12 swiatekm