Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

4.2.x master default fixes #18

Open
wants to merge 8 commits into
base: master
Choose a base branch
from
Open

Conversation

Mbd06b
Copy link

@Mbd06b Mbd06b commented Oct 7, 2023

This pull request attempts to resolve all the errors and issues I encountered trying to install with the default helm chart with minimal configurations, while trying to respect the original intentions of Michael's helming approach.

  • It suggests downgrading your master chart appVersion to 6.4.2 to support the available images closest to the "6.4.x" release
  • It removes helm templating syntax in the values.yaml which fixes errors resolving the ... | default .Chart.AppVersion }}" in the image: definition of the templates
  • It fixes a pod init error with the postgresdb pod, by implementing a pattern recommended by the postgres error log:
initdb: error: directory "/var/lib/postgresql/data" exists but is not empty
It contains a lost+found directory, perhaps due to it being a mount point.
Using a mount point directly as the data directory is not recommended.
Create a subdirectory under the mount point

By specifying a meaningful subdirectory for data initialization (ie. PGDATA env var) within the mountPath.

This fixes outstanding issues:
#7

@Mbd06b Mbd06b changed the title 4.2.x master 4.2.x master default fixes Oct 7, 2023
@Mbd06b Mbd06b marked this pull request as ready for review October 7, 2023 18:03
@Mbd06b
Copy link
Author

Mbd06b commented Oct 7, 2023

You might be able to test my release with my registry here, I setup this harbor registry in order to package and debug these changes from the current default deployment

helm install myTiagaRelease oci://harbor.ethosengine.com/ethosengine/taiga --namespace taiga --create-namespace -f 
 values.yaml

@Mbd06b Mbd06b closed this Oct 8, 2023
@Mbd06b Mbd06b deleted the 4.2.x-master branch October 8, 2023 03:07
@Mbd06b Mbd06b restored the 4.2.x-master branch October 8, 2023 03:08
@Mbd06b Mbd06b reopened this Oct 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant