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

feat(observability-pipeline): use pvc for cacheing latest config #421

Merged
merged 3 commits into from
Feb 4, 2025

Conversation

VinozzZ
Copy link
Contributor

@VinozzZ VinozzZ commented Jan 24, 2025

Which problem is this PR solving?

  • adds support for using pvc to persist latest config in control plane

Short description of the changes

  • add a new persistentVolumnClaimName in the values.yaml file to allow users to mount a volume
  • pass in the mount path using -config-cache-path flag

How to verify that this has the expected result

@VinozzZ VinozzZ self-assigned this Jan 24, 2025
@VinozzZ VinozzZ force-pushed the yingrong/cache_latest_config branch from d16c04e to 0a60d90 Compare February 3, 2025 21:17
@VinozzZ VinozzZ force-pushed the yingrong/cache_latest_config branch from 0a60d90 to 9ec3a30 Compare February 3, 2025 21:24
@VinozzZ VinozzZ changed the title feat(observability-pipeline): use pvc for cacheing latest config feat(observability-pipeline): use pvc for caching latest config Feb 3, 2025
@VinozzZ VinozzZ changed the title feat(observability-pipeline): use pvc for caching latest config feat(observability-pipeline): use pvc for cacheing latest config Feb 3, 2025
@VinozzZ VinozzZ marked this pull request as ready for review February 4, 2025 16:12
@VinozzZ VinozzZ requested a review from a team as a code owner February 4, 2025 16:12
@VinozzZ VinozzZ requested a review from TylerHelmuth February 4, 2025 16:33
@VinozzZ VinozzZ merged commit 2901c1d into main Feb 4, 2025
3 checks passed
@VinozzZ VinozzZ deleted the yingrong/cache_latest_config branch February 4, 2025 17:11
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.

2 participants