Integrate NOVUM-RGI chart into the official helm chart. #7
No reviewers
Labels
No Label
has
backport
in progress
invalid
kind
breaking
kind
bug
kind
build
kind
dependency
kind
deployment
kind
docs
kind
enhancement
kind
feature
kind
lint
kind
proposal
kind
question
kind
refactor
kind
security
kind
testing
kind
translation
kind
ui
need
backport
priority
critical
priority
low
priority
maybe
priority
medium
reviewed
duplicate
reviewed
invalid
reviewed
wontfix
skip-changelog
status
blocked
status
needs-feedback
status
needs-reviews
status
wip
upstream
gitea
upstream
other
No Milestone
No Assignees
5 Participants
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: gitea/helm-chart#7
Loading…
Reference in New Issue
Block a user
No description provided.
Delete Branch "luhahn/helm-chart:master"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
As discussed in Discord here's my PR to migrate NOVUM-RGI chart into the official helm chart.
First of all, i know that's a lot of stuff. But I gave jfeltens original chart a big rework since a lot of the approaches are deprecated.
What has changed:
I'm open for discussions about what to actually migrate and what not :)
@luhahn Could you change the title to a suitable title, current is too simple.
masterto Integrate NOVUM-RGI chart into the official helm chart.@ -1,5 +1,7 @@
MIT License
Copyright (c) 2020 NOVUM-RGI
Could you add this below Gitea copyright?
@ -2,0 +5,4 @@
Readme will be updated with examples in the next few days
# Content
<!-- vscode-markdown-toc -->
Please use Gitea toc functionality
how is this used in gitea? Couldn't find any examples
Or did you want me to remove the vscode-markdown comment the toc is in usual markdown format? I was too lazy to create the toc by hand so I used a vscode plugin which did it for me.
@ -15,7 +17,23 @@ sources:
- https://github.com/go-gitea/gitea
- https://hub.docker.com/r/gitea/gitea/
maintainers:
- name: Lucas Hahn
alphabet order
e2080106c4
to9de2d078c4
I think we could merge this one and add some ci tests after.
@ -22,0 +31,4 @@
condition: gitea.cache.enabled
- name: mysql
repository: https://charts.bitnami.com/bitnami
version: 6.14.8
Previously we were using mariadb 7.3.0 not mysql 6.14.8
That's only the chart version. It is a bit confusing in helm, you have a chart version which only referes to the chart, templates etc. and an app version, which is the actual version of the application. For mariadb this is appVersion: 8.0.21 for chart version 6.14.8
@ -1,6 +1,8 @@
MIT License
Don't need an additional blank line here
@ -2,0 +60,4 @@
* Memcached
* Mysql
## 2. <a name='Installing'></a>Installing
Who is expected to be rendering this Markdown? As far I as understood Gitea will automatically add an anchor with the name "Installing" anyway
@ -190,0 +496,4 @@
|gitea.log.x.flags|A comma separated string representing the log flags.|stdflags|
|gitea.log.x.expression| regular expression to match either the function name, file or message. Defaults to empty. Only log messages that match the expression will be saved in the logger.||
|gitea.log.x.prefix|An additional prefix for every log line in this logger. Defaults to empty.||
|gitea.log.x.colorize| Colorize the log lines by default|false|
Do you really understand what's going on here with these loggging settings?
@ -0,0 +680,4 @@
FLAGS = {{ .Values.gitea.log.x.flags }}
EXPRESSION = {{ .Values.gitea.log.x.expression }}
PREFIX = {{ .Values.gitea.log.x.prefix }}
COLORIZE = {{ .Values.gitea.log.x.colorize }}
[log.x] is not a logger - this configuration section does not make sense.
As suggested from @zeripath we changed the configuration part of the app.ini to a more generic approach. It's now possible to use all the Configurations in gitea.config
Generally once things are given to Gitea any future changes are assigned to Gitea as the copyright holder but I think that might be a discussion with the Gitea owners.
d9fcdf0b7d
tode8530ba29
de8530ba29
toe3b292724c
Things with the app.ini look much better - this autogeneration will prevent
having to constantly change the chart to add more features.
Caveats with this review:
Otherwise, LGTM
Copyright header in LICENSE file is fine (for this PR at least)
Thanks for PR