Skip to content
This repository has been archived by the owner on Jul 19, 2023. It is now read-only.

Adds tenant overrides in helm #551

Merged
merged 4 commits into from
Mar 2, 2023
Merged

Adds tenant overrides in helm #551

merged 4 commits into from
Mar 2, 2023

Conversation

cyriltovena
Copy link
Collaborator

No description provided.

Copy link
Collaborator

@simonswine simonswine left a comment

Choose a reason for hiding this comment

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

LGTM

@@ -710,6 +710,12 @@ phlare:
regex: Pending|Succeeded|Failed|Completed
action: drop

# Allows to add tenant specific overrides to the default limit configuration.
Copy link
Collaborator

Choose a reason for hiding this comment

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

If you do this, it should show up in the README.md correctly

Suggested change
# Allows to add tenant specific overrides to the default limit configuration.
# -- Allows to add tenant specific overrides to the default limit configuration.

@cyriltovena cyriltovena merged commit 3294e5f into main Mar 2, 2023
@cyriltovena cyriltovena deleted the helm-overrides branch March 2, 2023 07:50
simonswine pushed a commit to simonswine/pyroscope that referenced this pull request Jun 30, 2023
* Adds tenant overrides in helm

* regenerate rendered files

* regenerate rendered files

* fixes linter issues
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants