Yurii Vlasov vlasov-y
vlasov-y commented on issue vikunja/helm-chart#10 2023-09-28 12:35:45 +00:00
Configuration Questions

@xeruf There is a default value, but I assume, you have set interface to value that does not contain port, or you set it to empty.

https://kolaente.dev/vikunja/helm-chart/src/branch/main/values.…

vlasov-y commented on issue vikunja/helm-chart#10 2023-09-18 09:16:01 +00:00
Configuration Questions

It is very hard to proxy to application on the same domain, because usually application are not ready to work under subpath different than /.

Usually, two domains are used.

vlasov-y commented on issue vikunja/helm-chart#11 2023-09-18 09:12:06 +00:00
Publish Helm chart on ArtifactHub

@konrad WDYT?

vlasov-y commented on issue vikunja/helm-chart#10 2023-09-15 12:17:11 +00:00
Configuration Questions

Oops... I thought that I will push to my mirror repository but I push to the real one 😄 5d4142c5bd

@konrad if…

vlasov-y pushed to main at vikunja/helm-chart 2023-09-15 12:14:43 +00:00
5d4142c5bd Added an option to reuse existing PVC
vlasov-y commented on issue vikunja/helm-chart#10 2023-09-15 12:08:08 +00:00
Configuration Questions

why two ingress configs? should I duplicate the config if I want both on the same domain?

It is very hard to proxy to application on the same domain, because usually application are not ready…

vlasov-y commented on pull request vikunja/helm-chart#7 2023-05-10 07:46:48 +00:00
Fix 'raw' template

@vgrassia Thanks for fixing! That was "Find&Replace" issue, like copy-paste :)

FIY @konrad I have approved, but we have to increment patch version in Chart.yaml before merge. @vgrassia Can you…

vlasov-y approved vikunja/helm-chart#7 2023-05-10 07:45:19 +00:00
Fix 'raw' template
vlasov-y commented on pull request vikunja/helm-chart#6 2023-05-03 07:32:36 +00:00
Update security context to allow frontend pod to start

Now it looks good to me @r1cebank 😃 Approving

FYI @konrad

vlasov-y commented on pull request vikunja/helm-chart#6 2023-05-03 07:31:23 +00:00
Update security context to allow frontend pod to start

Aaa, you are right :) I somewhy misunderstood the diff and thought that you change postgres to postgresql XD Sorry

vlasov-y commented on pull request vikunja/helm-chart#6 2023-05-02 11:23:35 +00:00
Update security context to allow frontend pod to start

Okay, then this line is to be reverted @r1cebank

vlasov-y commented on pull request vikunja/helm-chart#6 2023-05-02 09:49:34 +00:00
Update security context to allow frontend pod to start

@r1cebank I have tried deploy frontend with values below and it works nice. I do not see a reason to set allowPrivilegeEscalation=true

  securityContext:
    allowPrivilegeEscalation:…
vlasov-y commented on pull request vikunja/helm-chart#6 2023-05-02 09:38:50 +00:00
Update security context to allow frontend pod to start

According to this documentation paragraph , it has to be postgres, not postgresql. @konrad which value is corrent one?

vlasov-y commented on pull request vikunja/helm-chart#6 2023-05-02 09:38:50 +00:00
Update security context to allow frontend pod to start

Change #nginx to # nginx please

vlasov-y commented on pull request vikunja/helm-chart#6 2023-05-02 08:59:22 +00:00
Update security context to allow frontend pod to start

@vlasov-y can you take a look?

will review,

vlasov-y created pull request vikunja/helm-chart#4 2023-04-12 17:13:27 +00:00
Updated appVersion to 0.20.4
vlasov-y created branch release/0.20.4 in vikunja/helm-chart 2023-04-12 17:12:34 +00:00
vlasov-y pushed to release/0.20.4 at vikunja/helm-chart 2023-04-12 17:12:34 +00:00
ad5c4a631e Updated appVersion to 0.20.4