Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Auto deploy #8

Merged
merged 13 commits into from
May 28, 2021
Merged

Auto deploy #8

merged 13 commits into from
May 28, 2021

Conversation

applejag
Copy link
Contributor

@applejag applejag commented May 26, 2021

Followed this guide: https://jamiemagee.co.uk/blog/how-to-host-your-helm-chart-repository-on-github/

Changes to CI pipeline

Changes to wharf-helm chart

  • Moved wharf-helm chart into charts/ dir
  • Added some metadata to charts/wharf-helm/Chart.yaml
  • Added charts/wharf-helm/ci/test-values.yaml for "chart-testing"
  • Added hotfixes until Feature/hotfixes #6 is merged
  • Added configurability of readinessProbe & livenessProbe for API & web
  • Changed CHANGELOG.md, we no longer have dates

Other changes

  • Disabled "remark-lint-maximum-line-length" in charts/ dir as we're using autogenerated docs via helm-docs
  • Updated docs in README.md a lot

Closes #5

@applejag applejag force-pushed the feature/auto-deploy branch 19 times, most recently from cef8624 to 91c7554 Compare May 27, 2021 08:41
@applejag applejag self-assigned this May 27, 2021
@applejag applejag added the enhancement New feature or request label May 27, 2021
README.md Show resolved Hide resolved
@applejag applejag marked this pull request as ready for review May 27, 2021 09:15
.github/kubeval.sh Outdated Show resolved Hide resolved
.github/workflows/ci.yaml Show resolved Hide resolved
.github/workflows/ci.yaml Show resolved Hide resolved
.github/workflows/ci.yaml Show resolved Hide resolved
.github/workflows/ci.yaml Show resolved Hide resolved
Co-authored-by: Mikael Berrin <pikabanga@gmail.com>
@applejag
Copy link
Contributor Author

@Pikabanga I'll let you resolve the discussions here if you find the answers adequate

@applejag applejag merged commit 819db7f into master May 28, 2021
@applejag applejag deleted the feature/auto-deploy branch May 28, 2021 15:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Publish chart via GitHub pages
3 participants