Skip to content

add feature flag service to proto file#26

Merged
cartersocha merged 2 commits intoopen-telemetry:mainfrom
tsloughter:flag-service-pb
May 20, 2022
Merged

add feature flag service to proto file#26
cartersocha merged 2 commits intoopen-telemetry:mainfrom
tsloughter:flag-service-pb

Conversation

@tsloughter
Copy link
Copy Markdown
Member

@tsloughter tsloughter commented May 20, 2022

Protos for issue #28

Changes

This adds the feature flag service protos. I figured this was the best place to start with adding this service.

I can open a design discussion issue like the check boxes below say but I thought this might be simply enough to just open a PR. I didn't include anything beyond a feature flag having a name and a bool, to keep it simple. I assume we don't want stuff like variants/experiments/etc, but could easily add them to this PR if they are.


For significant contributions please make sure you have completed the following items:

  • Appropriate CHANGELOG.md updated for non-trivial changes
  • Design discussion issue #

@tsloughter tsloughter requested a review from a team May 20, 2022 20:29
@reyang
Copy link
Copy Markdown
Member

reyang commented May 20, 2022

LGTM with a changelog entry.

@cartersocha
Copy link
Copy Markdown
Contributor

cartersocha commented May 20, 2022

Please open an issue to track the overall service addition too

@tsloughter
Copy link
Copy Markdown
Member Author

k, will do

@tsloughter tsloughter mentioned this pull request May 20, 2022
@tsloughter
Copy link
Copy Markdown
Member Author

Opened #28 and updated the changelog.

Copy link
Copy Markdown
Contributor

@cartersocha cartersocha left a comment

Choose a reason for hiding this comment

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

LGTM w/ changelog update. Do you need an approver or maintainer to merge?

@tsloughter
Copy link
Copy Markdown
Member Author

Yea, I don't have permissions to merge.

@cartersocha cartersocha merged commit ccf594b into open-telemetry:main May 20, 2022
jmichalak9 pushed a commit to jmichalak9/opentelemetry-demo that referenced this pull request Mar 22, 2024
* add feature flag service to proto file

* update changelog for added feature flag service protos
smith pushed a commit to smith/opentelemetry-demo that referenced this pull request Aug 5, 2024
shantanualsi pushed a commit to shantanualsi/opentelemetry-demo that referenced this pull request Mar 4, 2025
…span-profiling-checkoutservice

Enable span profiling for checkoutservice
mohamed3637 added a commit to mohamed3637/opentelemetry-demo that referenced this pull request Oct 7, 2025
* add feature flag service to proto file

* update changelog for added feature flag service protos
damemi pushed a commit to odigos-io/opentelemetry-demo that referenced this pull request Feb 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants