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

charts/karavi-observability: PR for Observability Patch v1.1.1 #112

Merged
merged 8 commits into from
May 24, 2022

Conversation

sharmilarama
Copy link
Collaborator

@sharmilarama sharmilarama commented May 6, 2022

Is this a new chart?

No

What this PR does / why we need it:

Which issue(s) is this PR associated with:

Special notes for your reviewer:

Checklist:

[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

  • Chart Version bumped
  • Variables are documented in the chart README.md
  • Title of the PR starts with the chart name (e.g. [charts_dir/mychartname]) if applicable

atye and others added 5 commits January 27, 2022 13:29
* enable auth sidecar

* fix manifest

* finish helm deployment of auth sidecar

* update version

* fix linting

* fix chart version

* add release branch to helm validations
… for observability helm chart (#106)

* updated collecter config to use proper tls key

* fixed otel collecter endpoint

* updated cert-manager chart version

* updatec otel-collecter image version
@sharmilarama sharmilarama changed the title <DO NOT MERGE TILL RELEASE> PR for Observability Patch v1.1.1 PR for Observability Patch v1.1.1 May 24, 2022
@sharmilarama sharmilarama changed the title PR for Observability Patch v1.1.1 charts/karavi-observability: PR for Observability Patch v1.1.1 May 24, 2022
hoppea2
hoppea2 previously approved these changes May 24, 2022
EvgenyUglov
EvgenyUglov previously approved these changes May 24, 2022
@sharmilarama sharmilarama dismissed stale reviews from EvgenyUglov and hoppea2 via 194a990 May 24, 2022 19:41
@sharmilarama sharmilarama merged commit a8c6c7c into main May 24, 2022
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.

7 participants