vikunja
/
helm-chart
2
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity
Labels Milestones

feat: add ci config #3

Merged
konrad merged 2 commits from feature/automate-all-the-things into main 2 weeks ago
Conversation 4 Commits 2 Files Changed 1
konrad commented 2 weeks ago
Owner

Resolves #2

Resolves #2
konrad added 1 commit 2 weeks ago
continuous-integration/drone/pr Build is failing Details
b475c64b41
feat: add ci config
konrad commented 2 weeks ago
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 2 weeks ago
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 2 weeks ago
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 2 weeks ago
continuous-integration/drone/pr Build is passing Details
bf50b61398
fix: update dependencies before packaging
konrad commented 2 weeks ago
Poster
Owner

That seems to have been it - thanks!

That seems to have been it - thanks!
👍 1
konrad merged commit ab61974ad5 into main 2 weeks ago
konrad referenced this issue from a commit 2 weeks ago
feat: add ci config (#3)
continuous-integration/drone/pr Build is passing
Details
The pull request has been merged as ab61974ad5.
Sign in to join this conversation.
Reviewers
Request review
No reviewers
Labels
Apply labels
Clear labels
No items
No Label
Milestone
Set milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Assign users
Clear assignees
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

No dependencies set.

Reference: vikunja/helm-chart#3
Write Preview
Loading…
Cancel
Save
There is no content yet.
Delete Branch 'feature/automate-all-the-things'

Deleting a branch is permanent. It CANNOT be undone. Continue?

No
Yes