Skip to content

Commit

Permalink
Merge pull request #127 from celo-org/jcortejoso/fullnode-delete-pvc
Browse files Browse the repository at this point in the history
typo in celo-fullnode cj for deleting pvc
  • Loading branch information
jcortejoso committed Aug 25, 2023
2 parents 425b5a1 + 5bc66bf commit 4899420
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 4 deletions.
2 changes: 1 addition & 1 deletion charts/celo-fullnode/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ keywords:
- Validator
- Ethereum
- Proof-of-Stake
version: 0.5.9
version: 0.5.10
dependencies:
- name: common
repository: oci://us-west1-docker.pkg.dev/devopsre/clabs-public-oci
Expand Down
4 changes: 2 additions & 2 deletions charts/celo-fullnode/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@

Helm chart for deploying a Celo fullnode. More info at https://docs.celo.org

![Version: 0.5.9](https://img.shields.io/badge/Version-0.5.9-informational?style=flat-square) ![Type: application](https://img.shields.io/badge/Type-application-informational?style=flat-square) ![AppVersion: 1.6.0](https://img.shields.io/badge/AppVersion-1.6.0-informational?style=flat-square)
![Version: 0.5.10](https://img.shields.io/badge/Version-0.5.10-informational?style=flat-square) ![Type: application](https://img.shields.io/badge/Type-application-informational?style=flat-square) ![AppVersion: 1.6.0](https://img.shields.io/badge/AppVersion-1.6.0-informational?style=flat-square)

- [celo-fullnode](#celo-fullnode)
- [Chart requirements](#chart-requirements)
Expand Down Expand Up @@ -34,7 +34,7 @@ To install/manage a release named `celo-mainnet-fullnode` connected to `mainnet`

```bash
# Select the chart release to use
CHART_RELEASE="oci://us-west1-docker.pkg.dev/celo-testnet/clabs-public-oci/celo-fullnode --version=0.5.9" # Use remote chart and specific version
CHART_RELEASE="oci://us-west1-docker.pkg.dev/celo-testnet/clabs-public-oci/celo-fullnode --version=0.5.10" # Use remote chart and specific version
CHART_RELEASE="./" # Use this local folder

# (Only for local chart) Sync helm dependencies
Expand Down
2 changes: 1 addition & 1 deletion charts/celo-fullnode/templates/delete-pod-cronjob.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@ spec:
kubectl delete pod -n {{ .Release.Namespace }} {{ printf "%s-%d" (include "common.fullname" .) (.Values.deletePodCronJob.podIndex | int) }} $extraFlagsPod
# Delete pvc that are not currently used
extraFlagsPod="{{ .Values.deletePodCronJob.extraFlagsPvc }}"
extraFlagsPvc="{{ .Values.deletePodCronJob.extraFlagsPvc }}"
unused_pvcs="$(kubectl describe pvc | grep -E "^Name:.*$|^Namespace:.*$|^Used By:.*$" | grep -B 2 "<none>" | grep -E "^Name:.*$")"
while IFS= read -r line; do
pvc=$(echo $line | awk '{print $2}')
Expand Down

0 comments on commit 4899420

Please sign in to comment.