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

Demo Docs: Update the name of recommendation cache feature flag #4159

Merged
merged 5 commits into from
Mar 16, 2024

Conversation

cthain
Copy link
Contributor

@cthain cthain commented Mar 13, 2024

This PR updates the demo docs to reference the correct name of the feature flag for the recommendation service cache failure. It's not a significant change but brings the docs in line with the flag name in the demo.flagd.json file.

cthain added 2 commits March 13, 2024 13:26
The name of the feature-flag for introducing the memory leak is actually `recommendationServiceCacheFailure`.
@cthain cthain requested review from a team March 13, 2024 20:33
Copy link
Member

@julianocosta89 julianocosta89 left a comment

Choose a reason for hiding this comment

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

Thank you @cthain!

@cthain
Copy link
Contributor Author

cthain commented Mar 13, 2024

No worries. Before this gets merged, do you have any input on my comment about the feature flags when running in kube: open-telemetry/opentelemetry-demo#1343 (comment)? I'm happy to augment this PR with k8s-specific info if you think that would be helpful.

It might also be helpful to harmonize on the flag names, since they are currently named differently in the json file (docker compose) vs the ffspostgres (k8s)

Copy link
Member

@theletterf theletterf left a comment

Choose a reason for hiding this comment

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

Thanks!

Copy link
Contributor

@cartermp cartermp left a comment

Choose a reason for hiding this comment

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

final whitespace changes

content/en/docs/demo/feature-flags.md Outdated Show resolved Hide resolved
@cartermp
Copy link
Contributor

/fix:all

Copy link
Contributor

@cartermp cartermp merged commit 093f127 into open-telemetry:main Mar 16, 2024
5 checks passed
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.

5 participants