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

set patronictl config file #552

Merged
merged 3 commits into from
Jan 25, 2023
Merged

set patronictl config file #552

merged 3 commits into from
Jan 25, 2023

Conversation

jleni
Copy link
Contributor

@jleni jleni commented Jan 22, 2023

What this PR does / why we need it

Points patronictl to the correct config file.
This means that people can run the CLI without using -c /etc/timescaledb/patroni.yaml

Which issue this PR fixes

Minimizes the risk of mistakes such as

#179
#347
#365
#418

Checklist

@jleni jleni requested a review from a team as a code owner January 22, 2023 17:35
charts/timescaledb-single/Chart.yaml Outdated Show resolved Hide resolved
charts/timescaledb-single/Chart.yaml Outdated Show resolved Hide resolved
@jleni
Copy link
Contributor Author

jleni commented Jan 25, 2023

Yes, I just noticed! sorry about that.
Rebased and updated

@jleni
Copy link
Contributor Author

jleni commented Jan 25, 2023

sorry the rebase removed a line. Fixing that now

@nhudson nhudson enabled auto-merge (squash) January 25, 2023 16:34
Copy link
Contributor

@nhudson nhudson left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks for the contribution!!

@nhudson nhudson merged commit 555e383 into timescale:main Jan 25, 2023
@jleni jleni deleted the feat/patronictl branch January 25, 2023 20:55
klaus385 pushed a commit to klaus385/timescaledb-kubernetes that referenced this pull request Apr 26, 2023
* set patronictl config file

* bump chart version

* revert line removal
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.

2 participants