Skip to content

Conversation

@computate
Copy link
Member

parent 9cc5541
author Christopher Tate computate@computate.org 1677561053 -0700 committer Christopher Tate computate@computate.org 1677684028 -0700

  • fix a bug with postgresql.enabled value
  • add docs for new serviceAccountName value for required security on OpenShift
  • allow customizing the postgresqlDatabase value and PGDATABASE environment variable on the betydb container
  • allow disabling the creation of a betydb Secret in order to use an ExternalSecret from a vault instead

@computate
Copy link
Member Author

@robkooper I noticed I didn't have a value for serviceAccount.enabled, so I added it in addition to serviceAccount.create and set them both to false by default:

serviceAccount:

  # Specifies whether a service account should be added to the deployment
  enabled: false

  # Specifies whether a service account should be created
  create: false

  # Annotations to add to the service account
  annotations: {}

  # The name of the service account to use.
  # If not set and create is true, a name is generated using the fullname template
  name: ""

robkooper and others added 2 commits May 7, 2024 23:24
…aints

- Fix a bug with postgresql.enabled value
- Add optional serviceAccount to deployment for required security on OpenShift
- Add docs for the new serviceAccount values
- Allow customizing the postgresqlDatabase value and PGDATABASE environment variable on the betydb container
- Allow disabling the creation of a betydb Secret in order to use an ExternalSecret from a vault instead
- Remove the conflicting PGDATABASE in the init container that points to the bety database
- Removing the PG_TABLE=sessions and PG_DATABASE=bety environment variable
- When the PG_TABLE=sessions environment variable is set, the check-postgresql init container fails. When the deployment fails, the argocd application never completes and enters the post-install phase. The load-db Job depends on the post-install hook running, so the bety database is never created.
- Remove post-install helm hook from load-db Job so that the bety database gets created for the Deployment
- Added string around the -r 0 parameter of the command

parent 9cc5541
author Christopher Tate <computate@computate.org> 1677561053 -0700
committer Christopher Tate <computate@computate.org> 1680705101 -0600
gpgsig -----BEGIN PGP SIGNATURE-----

 iQEzBAABCAAdFiEE8gjca+F8TU1M4A3jZbVOLA7tOVIFAmQthk0ACgkQZbVOLA7t
 OVIK1AgA1wYe2xmMRpyz/qQfGH7UpdoLv+az9JReXlW2+cda41VYaQahcEUSuRy3
 c3iJynk5uoaO9QcR8gKGThmuO6kZYpJceyaPGP/ms+KBT+rgN9XFcUPQIGjuRIoj
 OplUx9IqKPrTqVLb7H448QHNDq+oST43vvmzxmXei5KSHSxq6Vrjgiqe+3UoyjdM
 HidKEaIYddajhpbzGiHZ96klIq5TNiU+q1xsuTzT5fsFkiPzwrpom5H5MlQ31EGg
 CeKpBPtG7qNI3zrK3gi9//4n8BlYqh14ncr/7Ju5MxP9bjIHyL22ksasMYL3HMxu
 ggwyLGRQmhbpsXoRHVbxelyi381a7w==
 =pmxl
 -----END PGP SIGNATURE-----
@robkooper
Copy link
Member

Let me update my pr for postgres to update the documentation as pointed out by @infotroph and I can merge both.

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.

2 participants