[WFLY-20915] Remove bitnami's postgresql Helm Chart#1079
[WFLY-20915] Remove bitnami's postgresql Helm Chart#1079emmartins merged 1 commit intowildfly:mainfrom
Conversation
|
@jmesnil I am going to delegate the review to the owner and tester, but before I do that please update the README-source.adoc instead of the README.adoc, the latter is just a flattened version of the former, built automatically by a workflow |
|
The README-source.adoc does not require any changes, they were done in the ancillary .adoc files. |
I see, that's ok then, you can keep it, it will - at most - be rebuilt and updated after |
Instead, the DB is deployed directly by the Helm Chart Update the `wildfly` Helm chart to actually pulls the DB credentials from the config map (as stated in the doc) instead of hard-coding them in the values.yaml file. This fixes https://issues.redhat.com/browse/WFLY-20915 Signed-off-by: Jeff Mesnil <jmesnil@ibm.com>
9e83fa3 to
e255655
Compare
|
@jmesnil do we still need this unique (in the context of quickstarts) design of Charts.yaml + values.yaml , or could we also switch to the common design of all other quickstarts, with charts/helm.yaml ? This would allow us to fully reuse the shared-docs content related with Helm charts. |
|
@emmartins We still need this structure as we are deploying a PostgreSQL resource in addition to the WildFly resources. |
Instead, the DB is deployed directly by the Helm Chart
Update the
wildflyHelm chart to actually pulls the DB credentials from the config map (as stated in the doc) instead of hard-coding them in the values.yaml file.This fixes https://issues.redhat.com/browse/WFLY-20915