Skip to content

Commit

Permalink
Merge pull request #196 from Sums047/issue/195
Browse files Browse the repository at this point in the history
Fix for issue 195: update helm.version to 3.13.3
  • Loading branch information
lokanalla authored Mar 7, 2024
2 parents a16a243 + 00dbdc3 commit c92e2fc
Show file tree
Hide file tree
Showing 6 changed files with 11 additions and 11 deletions.
6 changes: 3 additions & 3 deletions devdocs/dev-env.md
Original file line number Diff line number Diff line change
Expand Up @@ -88,7 +88,7 @@ On ubuntu, you can do this using the same script used by the Dockerfile to bundl

```
chmod u+x ./docker/setup-helm.sh
./docker/setup-helm.sh 3.13.2
./docker/setup-helm.sh 3.13.3
```

Include as many versions as you need separated by spaces. Ensure the versions exist on the [Helm Github releases page](https://github.com/helm/helm/releases).
Expand All @@ -98,8 +98,8 @@ For other systems, look at the contents of the script to see the steps and adapt
- Download the version of the Helm you want
- Extract the archive
- Copy the `helm` binary included in the archive to any bin directory on your PATH
- Rename the `helm` binary to `helm$version` e.g helm3.13.2
- Run `helm3.13.2 --help` to verify this has worked
- Rename the `helm` binary to `helm$version` e.g helm3.13.3
- Run `helm3.13.3 --help` to verify this has worked

## Install the build dependencies

Expand Down
4 changes: 2 additions & 2 deletions devdocs/helm-chart.md
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@ The Helm chart for this driver includes the following features:

# Build Helm Chart

To build the Helm chart you will need to install Helm v3.13.2
To build the Helm chart you will need to install Helm v3.13.3

## Full Build

Expand All @@ -34,7 +34,7 @@ python3 build.py --skip-docker

# Deploy Helm Chart

To deploy the Helm chart will need Helm v3.13.2 installed
To deploy the Helm chart will need Helm v3.13.3 installed

Install the chart with the default configuration using the install command:

Expand Down
2 changes: 1 addition & 1 deletion docs/user-guide/building-a-resource.md
Original file line number Diff line number Diff line change
Expand Up @@ -102,7 +102,7 @@ For full details on templates and the syntax used, check out the [Templating](te

The Kubernetes driver can deploy any helm charts to your target Kubernetes environments as long as Helm3 has been installed on the environment.

The Helm charts must be compatible with Helm 3 (helm v.3.13.2 is included in the driver).
The Helm charts must be compatible with Helm 3 (helm v.3.13.3 is included in the driver).

The `helm` directory should include the packaged Helm charts you intend to install (`.tgz` files) and any values files you may use to configure the installation. We'll see how you install the charts and select your values file later.

Expand Down
4 changes: 2 additions & 2 deletions docs/user-guide/deployment-locations.md
Original file line number Diff line number Diff line change
Expand Up @@ -12,12 +12,12 @@ The following properties are supported by the driver:
| clientConfig | - | Y | A multiline string version of the kubectl config file used to access the target cluster (see more details [below](#obtaining-clientconfig)) |
| defaultObjectNamespace | default | N | Sets the default namespace used when deploying Kubernetes objects on a create request. This value is only used when the object does not have a specified namespace in the metadata section of it's configuration |
| driverNamespace | Value of defaultObjectNamepsace/default_object_namespace | N | Sets the namespace to be used by the driver for any Kubernetes objects it creates for management purposes |
| helm.version | 3.13.2 | N | Determines the helm client version to use when deploying helm charts (allowed values: 3.13.2) |
| helm.version | 3.13.3 | N | Determines the helm client version to use when deploying helm charts (allowed values: 3.13.3) |
| helm.tls.cacert | - | N | Contents of the CA certificate (if used) |
| helm.tls.cert | - | N | Contents of the helm client certificate |
| helm.tls.key | - | N | Contents of the helm client key |

**Note:** when using Helm your target deployment location must be using a compatible server version for 3.13.2 (check with `helm version` on the server).
**Note:** when using Helm your target deployment location must be using a compatible server version for 3.13.3 (check with `helm version` on the server).

# Obtaining clientConfig

Expand Down
2 changes: 1 addition & 1 deletion kubedriver/location/deployment_location.py
Original file line number Diff line number Diff line change
Expand Up @@ -77,7 +77,7 @@ def from_dict(dl_data):
return KubeDeploymentLocation(name, client_config, **kwargs)

def __init__(self, name, client_config, default_object_namespace=DEFAULT_NAMESPACE, crd_api_version=None, driver_namespace=None, \
cm_api_version='v1', cm_kind='ConfigMap', cm_data_field='data', helm_version='3.13.2', helm_tls=None):
cm_api_version='v1', cm_kind='ConfigMap', cm_data_field='data', helm_version='3.13.3', helm_tls=None):
super().__init__(name, client_config, default_object_namespace=default_object_namespace)
self.crd_api_version = crd_api_version
self.cm_api_version = cm_api_version
Expand Down
4 changes: 2 additions & 2 deletions tests/unit/helmclient/test_helm_client_3.py
Original file line number Diff line number Diff line change
Expand Up @@ -56,7 +56,7 @@ class TestHelmClient3(unittest.TestCase):
"""

def setUp(self):
self.client = HelmClient('kubeconfig', '3.13.2')
self.client = HelmClient('kubeconfig', '3.13.3')

def tearDown(self):
self.client.close()
Expand Down Expand Up @@ -120,7 +120,7 @@ def test_delete_command_error(self, mock_subprocess):

@patch('kubedriver.helmclient.client.subprocess')
def test_purge(self, mock_subprocess):
self.client = HelmClient('kubeconfig', '3.13.2')
self.client = HelmClient('kubeconfig', '3.13.3')
self.__mock_subprocess_response(mock_subprocess, 0, EXAMPLE_MANIFEST)
result = self.client.purge('name', 'namespace')
self.assertEqual(result, None)
Expand Down

0 comments on commit c92e2fc

Please sign in to comment.