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

feat: add ci config #3

Merged
konrad merged 2 commits from feature/automate-all-the-things into main 2023-03-10 21:30:03 +00:00
Conversation 4 Commits 2 Files Changed 1
konrad commented 2023-03-10 17:39:42 +00:00
Owner

Resolves #2

Resolves #2
konrad added 1 commit 2023-03-10 17:39:43 +00:00
continuous-integration/drone/pr Build is failing Details
b475c64b41
feat: add ci config
konrad commented 2023-03-10 17:41:19 +00:00
Poster
Owner

@vlasov-y Do you have an idea why this fails? As far as I know redis and postgres are external charts so they should not have any effect?

@vlasov-y Do you have an idea why this fails? As far as I know redis and postgres are external charts so they should not have any effect?
vlasov-y commented 2023-03-10 21:20:54 +00:00
Collaborator

@konrad Hi! That is an expected behavior. When you package helm chart you always package it with all dependencies. We depend on PostgreSQL and Redis but do not push them to the repository (they have been added to .gitignore).

Solution

Run helm dependency update before helm package . command and the issue will disappear.

@konrad Hi! That is an expected behavior. When you package helm chart you always package it with all dependencies. We depend on PostgreSQL and Redis but do not push them to the repository (they have been added to .gitignore). **Solution** Run `helm dependency update` before `helm package .` command and the issue will disappear.
vlasov-y commented 2023-03-10 21:23:40 +00:00
Collaborator

I have updated the README.md including these details

I have updated the README.md including [these details](https://kolaente.dev/vikunja/helm-chart/src/branch/main/README.md#contributing)
konrad added 1 commit 2023-03-10 21:27:42 +00:00
continuous-integration/drone/pr Build is passing Details
bf50b61398
fix: update dependencies before packaging
konrad commented 2023-03-10 21:29:46 +00:00
Poster
Owner

That seems to have been it - thanks!

That seems to have been it - thanks!
👍 1
konrad merged commit ab61974ad5 into main 2023-03-10 21:30:03 +00:00
konrad referenced this issue from a commit 2023-03-10 21:30:03 +00:00
feat: add ci config (#3)
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: vikunja/helm-chart#3
There is no content yet.
Delete Branch "feature/automate-all-the-things"

Deleting a branch is permanent. Although the deleted branch may exist for a short time before cleaning up, in most cases it CANNOT be undone. Continue?