This repository has been archived by the owner on Feb 22, 2022. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 16.8k
Concourse to postgres dependency should be optional #1056
Comments
william-tran
pushed a commit
to autonomic-ai/charts
that referenced
this issue
May 9, 2017
@frodenas Any input on how to do this before I submit a PR? Here's what we have so far: autonomic-ai@8608a48 |
@william-tran Apologies for the late reply. I've been disconnected from this repo for a while, so unsure how conditionals work. If you were able to test your changes, submit a PR, I'll try to do some tests in the meantime. |
It's working out great plugging in our own RDS based postgres, but we're having other issues with runaway disk usage, though I have my workers not using PersistentVolumes. I'll send a PR once I can verify the whole setup is stable. |
@william-tran any progress on this? |
We were concerned over the stability of the version of Concourse we were
using (3.0.1) but it's been stable for us for a week. The stability issues
had nothing to do with postgres, but with the `btrfs` that workers were
using that would cause the fs to panic and go read only. We have a recovery
process for that case that I can include in the readme. I'll send over that
PR shortly.
On Jun 12, 2017 5:02 PM, "λex Vincent" <[email protected]> wrote:
@william-tran <https://github.com/william-tran> any progress on this?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#1056 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ABCMGc19HeyQZbxmiptZdLw3RZsiCmGgks5sDadigaJpZM4NVzVE>
.
|
@alexisvincent @frodenas PR submitted |
viglesiasce
pushed a commit
that referenced
this issue
Jul 6, 2017
* Conditional dependency on postgres. Fixes #1056 * Update readme for conditional PostgreSQL dependency Also add a section on restarting workers * Update Chart.yaml
yanns
pushed a commit
to yanns/charts
that referenced
this issue
Jul 28, 2017
* Conditional dependency on postgres. Fixes helm#1056 * Update readme for conditional PostgreSQL dependency Also add a section on restarting workers * Update Chart.yaml
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
As a user, I would like to bring my own postgres instead of using a chart dependency.
The text was updated successfully, but these errors were encountered: