Skip to content

[grafana] fix: lookup volumename on pvc #3041

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

Merged
merged 3 commits into from
Apr 18, 2024
Merged

Conversation

guilhem
Copy link
Contributor

@guilhem guilhem commented Mar 25, 2024

This fix upgrade problem where helm try to override immutable field

@CLAassistant
Copy link

CLAassistant commented Mar 25, 2024

CLA assistant check
All committers have signed the CLA.

@zanhsieh zanhsieh changed the title fix: lookup volumename on pvc [grafana] fix: lookup volumename on pvc Mar 31, 2024
Copy link
Contributor

@zanhsieh zanhsieh left a comment

Choose a reason for hiding this comment

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

@guilhem
Can you bump the version in Chart.yaml to latest and sign DCO please?

@guilhem guilhem force-pushed the pvcvolumename branch 2 times, most recently from f27b35f to e51fa57 Compare March 31, 2024 10:10
@guilhem
Copy link
Contributor Author

guilhem commented Mar 31, 2024

@zanhsieh rebased and sign-off :)

@guilhem guilhem force-pushed the pvcvolumename branch 3 times, most recently from df5599a to fd3ede9 Compare April 17, 2024 16:07
@Xtigyro
Copy link
Collaborator

Xtigyro commented Apr 17, 2024

@guilhem Could you please merge the latest changes from main?

guilhem added 2 commits April 17, 2024 19:02
This fix upgrade problem where helm try to override immutable field

Signed-off-by: Guilhem Lettron <guilhem@barpilot.io>
Signed-off-by: Guilhem Lettron <guilhem@barpilot.io>
@guilhem
Copy link
Contributor Author

guilhem commented Apr 17, 2024

@guilhem Could you please merge the latest changes from main?

@Xtigyro should be ok

@Xtigyro Xtigyro merged commit ceb3879 into grafana:main Apr 18, 2024
6 checks passed
szinn referenced this pull request in szinn/k8s-homelab Apr 18, 2024
This PR contains the following updates:

| Package | Update | Change |
|---|---|---|
| [grafana](https://grafana.com)
([source](https://togithub.com/grafana/helm-charts)) | patch | `7.3.8`
-> `7.3.9` |

---

### Release Notes

<details>
<summary>grafana/helm-charts (grafana)</summary>

###
[`v7.3.9`](https://togithub.com/grafana/helm-charts/releases/tag/grafana-7.3.9)

[Compare
Source](https://togithub.com/grafana/helm-charts/compare/grafana-7.3.8...grafana-7.3.9)

The leading tool for querying and visualizing time series and metrics.

#### What's Changed

- \[grafana] fix: lookup volumename on pvc by
[@&#8203;guilhem](https://togithub.com/guilhem) in
[https://github.com/grafana/helm-charts/pull/3041](https://togithub.com/grafana/helm-charts/pull/3041)

#### New Contributors

- [@&#8203;guilhem](https://togithub.com/guilhem) made their first
contribution in
[https://github.com/grafana/helm-charts/pull/3041](https://togithub.com/grafana/helm-charts/pull/3041)

**Full Changelog**:
grafana/helm-charts@grafana-sampling-0.1.1...grafana-7.3.9

</details>

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy4zMDQuMCIsInVwZGF0ZWRJblZlciI6IjM3LjMwNC4wIiwidGFyZ2V0QnJhbmNoIjoibWFpbiIsImxhYmVscyI6WyJyZW5vdmF0ZS9oZWxtIiwidHlwZS9wYXRjaCJdfQ==-->

Co-authored-by: repo-jeeves[bot] <106431701+repo-jeeves[bot]@users.noreply.github.com>
kireque referenced this pull request in kireque/home-ops Apr 18, 2024
This PR contains the following updates:

| Package | Update | Change |
|---|---|---|
| [grafana](https://grafana.com)
([source](https://togithub.com/grafana/helm-charts)) | patch | `7.3.8`
-> `7.3.9` |

---

### Release Notes

<details>
<summary>grafana/helm-charts (grafana)</summary>

###
[`v7.3.9`](https://togithub.com/grafana/helm-charts/releases/tag/grafana-7.3.9)

[Compare
Source](https://togithub.com/grafana/helm-charts/compare/grafana-7.3.8...grafana-7.3.9)

The leading tool for querying and visualizing time series and metrics.

#### What's Changed

- \[grafana] fix: lookup volumename on pvc by
[@&#8203;guilhem](https://togithub.com/guilhem) in
[https://github.com/grafana/helm-charts/pull/3041](https://togithub.com/grafana/helm-charts/pull/3041)

#### New Contributors

- [@&#8203;guilhem](https://togithub.com/guilhem) made their first
contribution in
[https://github.com/grafana/helm-charts/pull/3041](https://togithub.com/grafana/helm-charts/pull/3041)

**Full Changelog**:
grafana/helm-charts@grafana-sampling-0.1.1...grafana-7.3.9

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined),
Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you
are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR has been generated by [Renovate
Bot](https://togithub.com/renovatebot/renovate).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy4zMDMuNCIsInVwZGF0ZWRJblZlciI6IjM3LjMwMy40IiwidGFyZ2V0QnJhbmNoIjoibWFpbiIsImxhYmVscyI6WyJyZW5vdmF0ZS9oZWxtIiwidHlwZS9wYXRjaCJdfQ==-->

Co-authored-by: kireque-bot[bot] <143391978+kireque-bot[bot]@users.noreply.github.com>
lumiere-bot bot referenced this pull request in coolguy1771/home-ops Apr 19, 2024
This PR contains the following updates:

| Package | Update | Change |
|---|---|---|
| [grafana](https://grafana.com)
([source](https://togithub.com/grafana/helm-charts)) | patch | `7.3.8`
-> `7.3.9` |

---

> [!WARNING]
> Some dependencies could not be looked up. Check the Dependency
Dashboard for more information.

---

### Release Notes

<details>
<summary>grafana/helm-charts (grafana)</summary>

###
[`v7.3.9`](https://togithub.com/grafana/helm-charts/releases/tag/grafana-7.3.9)

[Compare
Source](https://togithub.com/grafana/helm-charts/compare/grafana-7.3.8...grafana-7.3.9)

The leading tool for querying and visualizing time series and metrics.

#### What's Changed

- \[grafana] fix: lookup volumename on pvc by
[@&#8203;guilhem](https://togithub.com/guilhem) in
[https://github.com/grafana/helm-charts/pull/3041](https://togithub.com/grafana/helm-charts/pull/3041)

#### New Contributors

- [@&#8203;guilhem](https://togithub.com/guilhem) made their first
contribution in
[https://github.com/grafana/helm-charts/pull/3041](https://togithub.com/grafana/helm-charts/pull/3041)

**Full Changelog**:
grafana/helm-charts@grafana-sampling-0.1.1...grafana-7.3.9

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined),
Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you
are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR has been generated by [Renovate
Bot](https://togithub.com/renovatebot/renovate).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy4zMDMuNCIsInVwZGF0ZWRJblZlciI6IjM3LjMwMy40IiwidGFyZ2V0QnJhbmNoIjoibWFpbiIsImxhYmVscyI6WyJyZW5vdmF0ZS9oZWxtIiwidHlwZS9wYXRjaCJdfQ==-->

Co-authored-by: lumiere-bot[bot] <98047013+lumiere-bot[bot]@users.noreply.github.com>
lambchop4prez referenced this pull request in lambchop4prez/network Apr 24, 2024
[![Mend
Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Update | Change |
|---|---|---|
| [grafana](https://grafana.com)
([source](https://togithub.com/grafana/helm-charts)) | patch | `7.3.8`
-> `7.3.9` |

---

> [!WARNING]
> Some dependencies could not be looked up. Check the Dependency
Dashboard for more information.

---

### Release Notes

<details>
<summary>grafana/helm-charts (grafana)</summary>

###
[`v7.3.9`](https://togithub.com/grafana/helm-charts/releases/tag/grafana-7.3.9)

[Compare
Source](https://togithub.com/grafana/helm-charts/compare/grafana-7.3.8...grafana-7.3.9)

The leading tool for querying and visualizing time series and metrics.

#### What's Changed

- \[grafana] fix: lookup volumename on pvc by
[@&#8203;guilhem](https://togithub.com/guilhem) in
[https://github.com/grafana/helm-charts/pull/3041](https://togithub.com/grafana/helm-charts/pull/3041)

#### New Contributors

- [@&#8203;guilhem](https://togithub.com/guilhem) made their first
contribution in
[https://github.com/grafana/helm-charts/pull/3041](https://togithub.com/grafana/helm-charts/pull/3041)

**Full Changelog**:
grafana/helm-charts@grafana-sampling-0.1.1...grafana-7.3.9

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined),
Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you
are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR has been generated by [Mend
Renovate](https://www.mend.io/free-developer-tools/renovate/). View
repository job log
[here](https://developer.mend.io/github/lambchop4prez/network).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy4zMDEuNCIsInVwZGF0ZWRJblZlciI6IjM3LjMwMS40IiwidGFyZ2V0QnJhbmNoIjoibWFpbiIsImxhYmVscyI6W119-->
mjnagel referenced this pull request in defenseunicorns/uds-core Apr 26, 2024
[![Mend
Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Update | Change |
|---|---|---|
| [grafana](https://grafana.com)
([source](https://togithub.com/grafana/helm-charts)) | patch | `7.3.8`
-> `7.3.9` |

---

### Release Notes

<details>
<summary>grafana/helm-charts (grafana)</summary>

###
[`v7.3.9`](https://togithub.com/grafana/helm-charts/releases/tag/grafana-7.3.9)

[Compare
Source](https://togithub.com/grafana/helm-charts/compare/grafana-7.3.8...grafana-7.3.9)

The leading tool for querying and visualizing time series and metrics.

#### What's Changed

- \[grafana] fix: lookup volumename on pvc by
[@&#8203;guilhem](https://togithub.com/guilhem) in
[https://github.com/grafana/helm-charts/pull/3041](https://togithub.com/grafana/helm-charts/pull/3041)

#### New Contributors

- [@&#8203;guilhem](https://togithub.com/guilhem) made their first
contribution in
[https://github.com/grafana/helm-charts/pull/3041](https://togithub.com/grafana/helm-charts/pull/3041)

**Full Changelog**:
grafana/helm-charts@grafana-sampling-0.1.1...grafana-7.3.9

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined),
Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you
are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR has been generated by [Mend
Renovate](https://www.mend.io/free-developer-tools/renovate/). View
repository job log
[here](https://developer.mend.io/github/defenseunicorns/uds-core).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy4zMDEuNCIsInVwZGF0ZWRJblZlciI6IjM3LjMyMS4yIiwidGFyZ2V0QnJhbmNoIjoibWFpbiIsImxhYmVscyI6W119-->

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
rjferguson21 referenced this pull request in defenseunicorns/uds-core Jul 11, 2024
[![Mend
Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Update | Change |
|---|---|---|
| [grafana](https://grafana.com)
([source](https://togithub.com/grafana/helm-charts)) | patch | `7.3.8`
-> `7.3.9` |

---

### Release Notes

<details>
<summary>grafana/helm-charts (grafana)</summary>

###
[`v7.3.9`](https://togithub.com/grafana/helm-charts/releases/tag/grafana-7.3.9)

[Compare
Source](https://togithub.com/grafana/helm-charts/compare/grafana-7.3.8...grafana-7.3.9)

The leading tool for querying and visualizing time series and metrics.

#### What's Changed

- \[grafana] fix: lookup volumename on pvc by
[@&#8203;guilhem](https://togithub.com/guilhem) in
[https://github.com/grafana/helm-charts/pull/3041](https://togithub.com/grafana/helm-charts/pull/3041)

#### New Contributors

- [@&#8203;guilhem](https://togithub.com/guilhem) made their first
contribution in
[https://github.com/grafana/helm-charts/pull/3041](https://togithub.com/grafana/helm-charts/pull/3041)

**Full Changelog**:
grafana/helm-charts@grafana-sampling-0.1.1...grafana-7.3.9

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined),
Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you
are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR has been generated by [Mend
Renovate](https://www.mend.io/free-developer-tools/renovate/). View
repository job log
[here](https://developer.mend.io/github/defenseunicorns/uds-core).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy4zMDEuNCIsInVwZGF0ZWRJblZlciI6IjM3LjMyMS4yIiwidGFyZ2V0QnJhbmNoIjoibWFpbiIsImxhYmVscyI6W119-->

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
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.

4 participants