Skip to content

chore: initial Vector Helm Chart and Helm Chart release process#2871

Merged
MOZGIII merged 22 commits into
masterfrom
k8s-helm
Aug 4, 2020
Merged

chore: initial Vector Helm Chart and Helm Chart release process#2871
MOZGIII merged 22 commits into
masterfrom
k8s-helm

Conversation

@MOZGIII

@MOZGIII MOZGIII commented Jun 19, 2020

Copy link
Copy Markdown
Contributor

Will be rebased after #2653 is merged. 🎉 Builds upon the work done at #2232.

This PR adds the Helm Chart to deploy Vector into Kubernetes via Helm.

It has a lot of tweaks and knobs that we want to document.

  • add comments at values.yaml
  • check if {} should be [] at the default sources, transforms, and sinks

@MOZGIII
MOZGIII force-pushed the k8s-helm branch 2 times, most recently from c70ecbc to 2b79586 Compare June 19, 2020 05:02
@Hoverbear

Copy link
Copy Markdown
Contributor

Noting we seem to prefer packages.timber.io to charts.timber.io

@MOZGIII
MOZGIII force-pushed the k8s-helm branch 2 times, most recently from 72aa002 to 958531d Compare June 23, 2020 18:19
@MOZGIII
MOZGIII force-pushed the k8s-impl branch 5 times, most recently from d05fed0 to 44cc6b1 Compare July 1, 2020 08:01
@MOZGIII
MOZGIII force-pushed the k8s-impl branch 2 times, most recently from 512e3f2 to fceed3a Compare July 6, 2020 16:14
@MOZGIII
MOZGIII force-pushed the k8s-impl branch 2 times, most recently from 2ff7036 to a5d1379 Compare July 7, 2020 20:27
@MOZGIII
MOZGIII marked this pull request as ready for review July 9, 2020 13:22
@MOZGIII MOZGIII changed the title chore: Helm Chart chore: initial Vector Helm Chart and Helm Chart release process Jul 10, 2020

@ktff ktff left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, just the documentation in values.yaml that you mentioned needs to be added.

@MOZGIII

MOZGIII commented Jul 12, 2020

Copy link
Copy Markdown
Contributor Author

I meant the documentation on the website. It's quite a challenge to present and organize the docs for all this.
I think I should add more comments to the values.yaml - they can then be used as the source data for the docs on the website.

MOZGIII added 14 commits July 29, 2020 20:10
…Chart.yaml

Signed-off-by: MOZGIII <mike-n@narod.ru>
Apparently it's no longer available, or never was available in the first
place. :(

Signed-off-by: MOZGIII <mike-n@narod.ru>
Signed-off-by: MOZGIII <mike-n@narod.ru>
Signed-off-by: MOZGIII <mike-n@narod.ru>
Signed-off-by: MOZGIII <mike-n@narod.ru>
…riding

Signed-off-by: MOZGIII <mike-n@narod.ru>
Signed-off-by: MOZGIII <mike-n@narod.ru>
Signed-off-by: MOZGIII <mike-n@narod.ru>
Signed-off-by: MOZGIII <mike-n@narod.ru>
Signed-off-by: MOZGIII <mike-n@narod.ru>
Signed-off-by: MOZGIII <mike-n@narod.ru>
Signed-off-by: MOZGIII <mike-n@narod.ru>
Signed-off-by: MOZGIII <mike-n@narod.ru>
Signed-off-by: MOZGIII <mike-n@narod.ru>
Signed-off-by: MOZGIII <mike-n@narod.ru>
@binarylogic
binarylogic requested a review from jszwedko August 3, 2020 15:25
Signed-off-by: MOZGIII <mike-n@narod.ru>
@MOZGIII
MOZGIII merged commit 4767019 into master Aug 4, 2020
@MOZGIII
MOZGIII deleted the k8s-helm branch August 4, 2020 04:16
@MOZGIII

MOZGIII commented Aug 4, 2020

Copy link
Copy Markdown
Contributor Author

This is merged, so the nightly builds should kick off, and we should see Helm Chart releases published to the repo.
If everything goes smoothly - the same CI job just needs to be copied to the release workflow.

@Xaeroxe

Xaeroxe commented Sep 15, 2020

Copy link
Copy Markdown

Hi, my org is interested in running these helm charts. Where are they being published currently?

@binarylogic

Copy link
Copy Markdown
Contributor

Hi @Xaeroxe, you can see some k8s installation examples here. We are planning to release this officially in the next week or two. Let me know that helps.

@Xaeroxe

Xaeroxe commented Sep 15, 2020

Copy link
Copy Markdown

It does, thank you!

mengesb pushed a commit to jacobbraaten/vector that referenced this pull request Dec 9, 2020
…ordotdev#2871)

* Add helm chart for Vector installation

Signed-off-by: Alex Gavrisco <alexandr@gavrisco.com>
Signed-off-by: MOZGIII <mike-n@narod.ru>

* Fix Markdown checkstyle

Signed-off-by: Alex Gavrisco <alexandr@gavrisco.com>
Signed-off-by: MOZGIII <mike-n@narod.ru>

* Correct formatting and typos at distribution/helm/vector/README.md

Signed-off-by: MOZGIII <mike-n@narod.ru>

* Use vector namespace

Signed-off-by: MOZGIII <mike-n@narod.ru>

* Adjust and update the configmap parameters

Signed-off-by: MOZGIII <mike-n@narod.ru>

* Fix more typos and formatting issues

Signed-off-by: MOZGIII <mike-n@narod.ru>

* Update the description and chart version at distribution/helm/vector/Chart.yaml

Signed-off-by: MOZGIII <mike-n@narod.ru>

* Remove metrics service and service monitor

Apparently it's no longer available, or never was available in the first
place. :(

Signed-off-by: MOZGIII <mike-n@narod.ru>

* Remove `-w` arg from vector

Signed-off-by: MOZGIII <mike-n@narod.ru>

* Move PSP to a separate file

Signed-off-by: MOZGIII <mike-n@narod.ru>

* Major audit and rewrite of the Helm chart

Signed-off-by: MOZGIII <mike-n@narod.ru>

* Reset version and appVersion at the chart manifest and hinted to overriding

Signed-off-by: MOZGIII <mike-n@narod.ru>

* Add a script for publishing Helm charts

Signed-off-by: MOZGIII <mike-n@narod.ru>

* Allow initializing empty repo

Signed-off-by: MOZGIII <mike-n@narod.ru>

* Add release-helm CI job for nightly releases

Signed-off-by: MOZGIII <mike-n@narod.ru>

* Proper version resolution

Signed-off-by: MOZGIII <mike-n@narod.ru>

* Proper support for nightly versions

Signed-off-by: MOZGIII <mike-n@narod.ru>

* Fix comments at distribution/helm/vector/values.yaml

Signed-off-by: MOZGIII <mike-n@narod.ru>

* Remove outdated README.md altogether

Signed-off-by: MOZGIII <mike-n@narod.ru>

* Add more documentation to the values.yaml

Signed-off-by: MOZGIII <mike-n@narod.ru>

* Fix a typo at helm release script

Signed-off-by: MOZGIII <mike-n@narod.ru>

* Install helm and kubectl at CI

Signed-off-by: MOZGIII <mike-n@narod.ru>

Co-authored-by: Alex Gavrisco <alexandr@gavrisco.com>
Signed-off-by: Brian Menges <brian.menges@anaplan.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

K8s: set up Helm chart release process K8s: prepare Helm charts Add Vector Kubernetes helm chart

7 participants