Skip to content
This repository was archived by the owner on Feb 12, 2024. It is now read-only.

Various updates and cleanups #10

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

Various updates and cleanups #10

wants to merge 2 commits into from

Conversation

FWest98
Copy link

@FWest98 FWest98 commented Nov 6, 2020

I did some updates and cleanups:

  • Update image version from 0.7.1 to 0.9.0
  • Update Ingress definition to use the new API; the beta-api will no longer be supported in newer version and is currently deprecated
  • Cleanup templates using nindent vs indent
  • Mounting the configuration of LdapAdmin as a file, instead of putting the configuration in environment variables. This was more stable in my experience
  • Added flexibility to ingress hosts definition by allowing a custom path per host.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant