Skip to content
This repository has been archived by the owner on Jan 18, 2024. It is now read-only.

Add permissions to Role in timescaledb-single, fixing #599 #601

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

geilername
Copy link

@geilername geilername commented Mar 29, 2023

What this PR does / why we need it

Add permission to create and modify services to the Role used in timescaledb-single chart.

Which issue this PR fixes

(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged)

Special notes for your reviewer

Checklist

@geilername geilername requested a review from a team as a code owner March 29, 2023 13:56
@CLAassistant
Copy link

CLAassistant commented Mar 29, 2023

CLA assistant check
All committers have signed the CLA.

@joschiwald
Copy link
Contributor

That issue is already fixed on patroni side. You could use the patroni-dcs-failsafe tags of the timescale/timescaledb-ha images, which uses a more current patroni version.

@hongbo-miao
Copy link

hongbo-miao commented Apr 28, 2023

Thanks the information from @joschiwald ! 😃

I succeed using new patroni-dcs-failsafe tag by

helm upgrade \
  timescale \
  timescaledb-single \
  --install \
  --repo=https://charts.timescale.com \
  --namespace=my-timescale \
  --create-namespace \
  --values=my-values.yaml

my-values.yaml

image:
  repository: timescale/timescaledb-ha
  tag: pg15.2-ts2.10.2-patroni-dcs-failsafe-p0

without any errors.

So maybe just need fix by updating tag instead.

Copy link

@RLutsch RLutsch left a comment

Choose a reason for hiding this comment

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

had the same issue, this fixed it.

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

Successfully merging this pull request may close these issues.

TimescaleDB Single ServiceAccount missing create:service permission
5 participants