Skip to content
This repository has been archived by the owner on Aug 29, 2020. It is now read-only.

Keeping example app.ini up to date #17

Open
bjw-s opened this issue May 12, 2020 · 4 comments
Open

Keeping example app.ini up to date #17

bjw-s opened this issue May 12, 2020 · 4 comments

Comments

@bjw-s
Copy link

bjw-s commented May 12, 2020

Currently a version of app.ini is made available through templates/gitea/gitea-config.yaml.

The contents of this file is prone to change as new versions of Gitea are released, which would mean the configmap would have to change accordingly. For example, the SHOW_MILESTONES_DASHBOARD_PAGE key is available in the official sample file, but it is not in the configmap.

If I understand the documentation correctly, it is possible to only define the values in the file that you actually wish to customize.

Following that approach, maybe the configmap should be simplified to just read the config key from values.yaml. The only thing I'm not sure of is how the database and gitea secrets should be automatically injected into the configuration file.

Hopefully this issue can be used to discuss this.

@nox-404
Copy link

nox-404 commented Jul 4, 2020

Hi there,
Would be best to be able to set everything from values.yaml I agree, I can make a PR if needed
Something like that would be great:

config: |
  < the config >
  DB_TYPE: {{ .Values.externaldb.dbType }}
  ...

or to avoid using the tpl it could be something like that:

config:
  default:
    APP_NAME: Gitea
  ...
  <section>:
    <key>: <value>

and provide defaults there, wich gives the user full customization without the hassle of merging the defaults

@cdrage
Copy link
Contributor

cdrage commented Jul 8, 2020

Hi @nox-404 unfortunately I've been pretty busy with work, if you can create a PR that'd be great!

@nox-404
Copy link

nox-404 commented Aug 18, 2020

Completely forgot about this, I'll see what I can do over the weekend

@cdrage
Copy link
Contributor

cdrage commented Aug 28, 2020

Hey all, I have donated this chart to the folks over at Gitea who have been doing an amazing job maintaining and updating the chart so it is up to date. Please open up any issues here: https://gitea.com/gitea/helm-chart and try theirs own!

Many thanks for all who've contributed.

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

No branches or pull requests

3 participants