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

[productcatalog] - generate error on feature flag #245

Merged
merged 13 commits into from
Aug 17, 2022

Conversation

puckpuck
Copy link
Contributor

@puckpuck puckpuck commented Aug 1, 2022

Changes

Generates an error on a specific product when a feature flag is enabled.
related issue: #162

Currently, this PR will not work since we have an issue that needs resolving with the feature flag service. Once that is resolved, we can finalize this PR.

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

I will update the changelog after we resolve feature flag service communication issues

  • Appropriate CHANGELOG.md updated for non-trivial changes

@puckpuck puckpuck marked this pull request as ready for review August 9, 2022 02:20
@puckpuck puckpuck requested a review from a team August 9, 2022 02:20
@puckpuck
Copy link
Contributor Author

puckpuck commented Aug 9, 2022

After we merge #271 I'll update this branch from the base branch.

Copy link
Member

@austinlparker austinlparker left a comment

Choose a reason for hiding this comment

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

lgtm

@cartersocha
Copy link
Contributor

Can we have some user documentation for enabling this or will it be automatically set up soon after we merge this pr in a subsequent pr?

@puckpuck
Copy link
Contributor Author

Can we have some user documentation for enabling this or will it be automatically set up soon after we merge this pr in a subsequent pr?

That will come in a subsequent PR which also adds the actual feature flags to the database when the feature flag service starts.

@cartersocha cartersocha merged commit 6cd2272 into open-telemetry:main Aug 17, 2022
@puckpuck puckpuck deleted the puckpuck.featurebreak branch August 18, 2022 00:35
jmichalak9 pushed a commit to jmichalak9/opentelemetry-demo that referenced this pull request Mar 22, 2024
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