vikunja/helm-chart
2
2
Fork 5
Code Issues 7 Pull Requests 1 Packages Projects Releases 1 Wiki Activity

Utilize bjw-s's common Helm library. #13

Merged
konrad merged 41 commits from :new-helm-chart into main 2023-11-07 14:47:24 +00:00
Conversation 14 Commits 41 Files Changed 24 +11 -7
1 changed files with 11 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 688a055c36 - Show all commits

18
README.md
View File

@ -10,13 +10,13 @@ This Helm Chart deploys both the Vikunja [frontend](https://hub.docker.com/r/vik
## Quickstart
The majority of default values defined in `values.yaml` should be compatible for your deployment. Additionally, if you utilize an Ingress for both the API and Frontend, you will be able to access the frontend out of the box. However, it won't have any default credentials, you will need to **either** enable registration, or execute `/bin/sh` on the API container and run the following command:
The majority of default values defined in `values.yaml` should be compatible for your deployment. Additionally, if you utilize an Ingress for both the API and Frontend, you will be able to access the frontend out of the box. However, it won't have any default credentials. So, you'll need to create an account using the registration button.
konrad marked this conversation as resolved Outdated
konrad commented 2023-10-10 16:36:23 +00:00
Outdated
Review
Copy Link

Registration is enabled by default, maybe change this to "you need to create an account before using it, there are no default credentials".

Not sure if we should even leave this in here as it is not specific for the helm deployment.

Registration is enabled by default, maybe change this to "you need to create an account before using it, there are no default credentials". Not sure if we should even leave this in here as it is not specific for the helm deployment.
perfectra1n commented 2023-10-11 13:59:20 +00:00
Outdated
Review
Copy Link

What do you think about this change?:

688a055c36

What do you think about this change?: https://kolaente.dev/vikunja/helm-chart/commit/688a055c369218558392df042f8ee6e16900e0f6
konrad commented 2023-10-11 14:04:41 +00:00
Outdated
Review
Copy Link

Looks good!

Looks good!
```bash
./vikunja user create --email <user@email.com> --user <user1> --password <password123>
```
That should be it!
If you do **not** wish to execute the above command in order to create a new user, you'll need to enable registration by appending
## Registration (creating users)
You can disable registration (if you do not with to allow others to register on your Vikunja), by providing the following values in your `values.yaml`:
konrad commented 2023-10-31 20:02:44 +00:00
Outdated
Review
Copy Link

I don't think this should belong here as it's not specific for the helm deployment.

I don't think this should belong here as it's not specific for the helm deployment.
perfectra1n commented 2023-11-01 04:57:20 +00:00
Outdated
Review
Copy Link

Understood - it was meant to be more of an example of how you can utilize the values of the Helm deployment to create the config.yml and make changes. Would it be adequate to perhaps rename the section to "Example of how to create config.yml" and move it to the bottom of the README.md?

Understood - it was meant to be more of an example of how you can utilize the `values` of the Helm deployment to create the `config.yml` and make changes. Would it be adequate to perhaps rename the section to "Example of how to create `config.yml`" and move it to the bottom of the README.md?
perfectra1n commented 2023-11-02 19:36:11 +00:00
Outdated
Review
Copy Link

Renamed the section, and bumped the "modification" examples to the bottom of the README.md in order to highlight how to use a pre-existing PVC. Let me know if you find that to be adequate!

ce56e07518

Renamed the section, and bumped the "modification" examples to the bottom of the README.md in order to highlight how to use a pre-existing PVC. Let me know if you find that to be adequate! https://kolaente.dev/vikunja/helm-chart/commit/ce56e07518e9187f2f406a80c94dc8fddca53a87
👍 1
konrad commented 2023-11-07 14:45:25 +00:00
Outdated
Review
Copy Link

Renaming is great!

Renaming is great!
```yaml
api:
@ -26,10 +26,14 @@ api:
data:
config.yml:
service:
enableregistration: true
enableregistration: false
```
Once you've registered your user, feel free to disable registration if you don't wish for other users to register.
If you need to create another user, you could opt to execute the following command on the `api` container:
```bash
./vikunja user create --email <user@email.com> --user <user1> --password <password123>
```
### Modifying Resources