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

feat(helm): added Helm chart to allow installation of the driver #12

Merged
merged 4 commits into from
Dec 15, 2023

Conversation

shanduur
Copy link
Member

@shanduur shanduur commented Dec 13, 2023

What this PR does / why we need it:

This pull request introduces the following changes:

  1. Functional Helm Chart: Added a functional Helm chart to enhance deployment capabilities. This chart likely includes configurations and templates necessary for deploying the application.

  2. Support for User Provided Secret: Extended the Helm chart to include support for user-provided secrets. This enhancement allows users to securely supply sensitive information, improving flexibility and security in deployments.

  3. Installation Documentation: Added an installation command to the documentation. This command provides clear and concise instructions on how to install the application, making it easier for users to get started.

Which issue(s) this PR resolves:

N/A

Special notes for your reviewer:

Before installing the chart, you need to run the make image and replace some of the default values, as there is no image published yet.

helm install linode-cosi-driver ./helm/linode-cosi-driver/ \
    --set=apiToken=test \
    --set=driver.image.repository=docker.io/linode/linode-cosi-driver \
    --set=driver.image.tag=v0.0.0-da1949 \
    --namespace=linode-cosi-driver \
    --create-namespace

Additional documentation e.g., enhancement proposals, usage docs, etc.:

  1. Installation instructions are added to the README.md;
  2. Additional documentation for the helm chart is autogenerated in the helm/linode-cosi-driver/README.md.

@shanduur shanduur added the kind/feature Categorizes issue or PR as related to a new feature. label Dec 13, 2023
@shanduur shanduur self-assigned this Dec 13, 2023
@shanduur shanduur force-pushed the feat/helm/installation branch 2 times, most recently from 80c4a6b to ea08e3d Compare December 13, 2023 15:41
luthermonson
luthermonson previously approved these changes Dec 14, 2023
@shanduur shanduur merged commit c9853f4 into linode:main Dec 15, 2023
7 checks passed
@shanduur shanduur deleted the feat/helm/installation branch December 15, 2023 16:39
@shanduur shanduur mentioned this pull request Mar 27, 2024
shanduur pushed a commit that referenced this pull request Mar 27, 2024
🤖 I have created a release *beep* *boop*
---


<details><summary>0.1.0</summary>

##
[0.1.0](v0.1.0...v0.1.0)
(2024-03-27)


### Features

* added base repo scaffolding
([#2](#2))
([905538a](905538a))
* automatically adjust GOMAXPROCS based on the CPU quota
([#36](#36))
([ebbb0f0](ebbb0f0))
* **client:** added Linode client
([#9](#9))
([4e30bf2](4e30bf2))
* **docker:** added base dockerfile
([#3](#3))
([a2246f0](a2246f0))
* **helm:** added Helm chart to allow installation of the driver
([#12](#12))
([c9853f4](c9853f4))
* **o11y:** added observability with OpenTelemetry
([#15](#15))
([337b7ce](337b7ce))
* **provisioner:** provisioner server implementation
([#18](#18))
([de570f7](de570f7))


### Bug Fixes

* correct suffix in labels
([#8](#8))
([98c3c46](98c3c46))
* **deps:** bump actions/checkout from 3 to 4
([#7](#7))
([04a44ce](04a44ce))
* **deps:** bump actions/setup-go from 4 to 5
([#10](#10))
([65ddd5f](65ddd5f))
* **deps:** bump github.com/go-resty/resty/v2 from 2.11.0 to 2.12.0
([#41](#41))
([666dc45](666dc45))
* **deps:** bump github.com/go-resty/resty/v2 from 2.9.1 to 2.11.0
([#19](#19))
([7055a98](7055a98))
* **deps:** bump github.com/grpc-ecosystem/go-grpc-middleware/v2 from
2.0.1 to 2.1.0
([#42](#42))
([dfecaa2](dfecaa2))
* **deps:** bump github.com/linode/linodego from 1.25.1-* to 1.26.0
([#16](#16))
([78526d4](78526d4))
* **deps:** bump github.com/linode/linodego from 1.26.0 to 1.27.0
([#21](#21))
([0042442](0042442))
* **deps:** bump github.com/linode/linodego from 1.27.0 to 1.29.0
([#38](#38))
([9797a25](9797a25))
* **deps:** bump github.com/linode/linodego from 1.29.0 to 1.30.0
([#43](#43))
([fd5aebd](fd5aebd))
* **deps:** bump github.com/linode/linodego from 1.30.0 to 1.31.0
([#44](#44))
([e4b354f](e4b354f))
* **deps:** bump golangci/golangci-lint-action from 3 to 4
([#31](#31))
([2896353](2896353))
* **deps:** bump google.golang.org/grpc from 1.59.0 to 1.60.0
([#14](#14))
([3125f23](3125f23))
* **deps:** bump google.golang.org/grpc from 1.60.0 to 1.60.1
([#17](#17))
([13153c1](13153c1))
* **deps:** bump google.golang.org/grpc from 1.60.1 to 1.61.0
([#29](#29))
([1ad35ad](1ad35ad))
* **deps:** bump google.golang.org/grpc from 1.61.0 to 1.61.1
([#35](#35))
([b13493f](b13493f))
* **deps:** bump google.golang.org/grpc from 1.61.1 to 1.62.0
([#39](#39))
([10571c9](10571c9))
* **deps:** bump google.golang.org/grpc from 1.62.0 to 1.62.1
([#40](#40))
([183b58e](183b58e))
* **deps:** bump the otel-dependencies group with 10 updates
([#32](#32))
([7dd63f4](7dd63f4))
* **deps:** bump the otel-dependencies group with 10 updates
([#37](#37))
([a3f3a1d](a3f3a1d))
* **deps:** bump the otel-dependencies group with 5 updates
([#28](#28))
([3bf403d](3bf403d))
* **linters:** updated linter rules
([#33](#33))
([1a1af77](1a1af77))


### Continuous Integration

* added release automation
([#20](#20))
([2d0af79](2d0af79))
</details>

<details><summary>linode-cosi-driver: 0.1.0</summary>

##
[0.1.0](linode-cosi-driver-v0.1.0...linode-cosi-driver-v0.1.0)
(2024-03-27)


### Features

* automatically adjust GOMAXPROCS based on the CPU quota
([#36](#36))
([ebbb0f0](ebbb0f0))
* **docker:** added base dockerfile
([#3](#3))
([a2246f0](a2246f0))
* **helm:** added Helm chart to allow installation of the driver
([#12](#12))
([c9853f4](c9853f4))
* **o11y:** added observability with OpenTelemetry
([#15](#15))
([337b7ce](337b7ce))
* **provisioner:** provisioner server implementation
([#18](#18))
([de570f7](de570f7))


### Continuous Integration

* added release automation
([#20](#20))
([2d0af79](2d0af79))
</details>

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Signed-off-by: Mateusz Urbanek <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/feature Categorizes issue or PR as related to a new feature.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants