Skip to content
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

NodeJS instrumentation featuregates into cli #2834

Merged

Conversation

yuriolisa
Copy link
Contributor

Description:

Change NodeJS instrumentation feature gates into CLI flags
Link to tracking Issue(s):

Testing:

Documentation:

@yuriolisa yuriolisa marked this pull request as ready for review April 26, 2024 10:29
@yuriolisa yuriolisa requested a review from a team April 26, 2024 10:29
Signed-off-by: Yuri Sa <[email protected]>
Signed-off-by: Yuri Sa <[email protected]>
Signed-off-by: Yuri Sa <[email protected]>
@pavolloffay
Copy link
Member

CI failed

Signed-off-by: Yuri Sa <[email protected]>
Signed-off-by: Yuri Sa <[email protected]>
Signed-off-by: Yuri Sa <[email protected]>
Signed-off-by: Yuri Sa <[email protected]>
Signed-off-by: Yuri Sa <[email protected]>
@swiatekm
Copy link
Contributor

Can you also delete the feature flag? LGTM otherwise.

@yuriolisa
Copy link
Contributor Author

Can you also delete the feature flag? LGTM otherwise.

I've removed it. Could you please review it?

Copy link
Contributor

@swiatekm swiatekm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@TylerHelmuth TylerHelmuth added the ready-to-merge Code review completed; ready to merge by maintainers label Apr 29, 2024
@pavolloffay pavolloffay merged commit 48bc207 into open-telemetry:main Apr 30, 2024
32 checks passed
ItielOlenick pushed a commit to ItielOlenick/opentelemetry-operator that referenced this pull request May 1, 2024
* NodeJS instrumentation featuregates into cli

Signed-off-by: Yuri Sa <[email protected]>

* Added complement to featuregate

Signed-off-by: Yuri Sa <[email protected]>

* Fixed Linters

Signed-off-by: Yuri Sa <[email protected]>

* Fixed Linters

Signed-off-by: Yuri Sa <[email protected]>

* Added e2e parameters

Signed-off-by: Yuri Sa <[email protected]>

* Fixed e2e test

Signed-off-by: Yuri Sa <[email protected]>

* Fixed e2e test

Signed-off-by: Yuri Sa <[email protected]>

* Fixed e2e test

Signed-off-by: Yuri Sa <[email protected]>

* Fixed e2e test

Signed-off-by: Yuri Sa <[email protected]>

* Removed feature flags

Signed-off-by: Yuri Sa <[email protected]>

* Removed feature flags

Signed-off-by: Yuri Sa <[email protected]>

---------

Signed-off-by: Yuri Sa <[email protected]>
ItielOlenick pushed a commit to ItielOlenick/opentelemetry-operator that referenced this pull request May 1, 2024
* NodeJS instrumentation featuregates into cli

Signed-off-by: Yuri Sa <[email protected]>

* Added complement to featuregate

Signed-off-by: Yuri Sa <[email protected]>

* Fixed Linters

Signed-off-by: Yuri Sa <[email protected]>

* Fixed Linters

Signed-off-by: Yuri Sa <[email protected]>

* Added e2e parameters

Signed-off-by: Yuri Sa <[email protected]>

* Fixed e2e test

Signed-off-by: Yuri Sa <[email protected]>

* Fixed e2e test

Signed-off-by: Yuri Sa <[email protected]>

* Fixed e2e test

Signed-off-by: Yuri Sa <[email protected]>

* Fixed e2e test

Signed-off-by: Yuri Sa <[email protected]>

* Removed feature flags

Signed-off-by: Yuri Sa <[email protected]>

* Removed feature flags

Signed-off-by: Yuri Sa <[email protected]>

---------

Signed-off-by: Yuri Sa <[email protected]>
rubenvp8510 pushed a commit to rubenvp8510/opentelemetry-operator that referenced this pull request May 7, 2024
* NodeJS instrumentation featuregates into cli

Signed-off-by: Yuri Sa <[email protected]>

* Added complement to featuregate

Signed-off-by: Yuri Sa <[email protected]>

* Fixed Linters

Signed-off-by: Yuri Sa <[email protected]>

* Fixed Linters

Signed-off-by: Yuri Sa <[email protected]>

* Added e2e parameters

Signed-off-by: Yuri Sa <[email protected]>

* Fixed e2e test

Signed-off-by: Yuri Sa <[email protected]>

* Fixed e2e test

Signed-off-by: Yuri Sa <[email protected]>

* Fixed e2e test

Signed-off-by: Yuri Sa <[email protected]>

* Fixed e2e test

Signed-off-by: Yuri Sa <[email protected]>

* Removed feature flags

Signed-off-by: Yuri Sa <[email protected]>

* Removed feature flags

Signed-off-by: Yuri Sa <[email protected]>

---------

Signed-off-by: Yuri Sa <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-to-merge Code review completed; ready to merge by maintainers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Change instrumentation feature gates into normal command-line flags: operator.autoinstrumentation.nodejs
4 participants