Skip to content

Commit

Permalink
Updated docs for CSI-Powerflex 2.12 (#1233)
Browse files Browse the repository at this point in the history
* updated docs for csi-vxflexos2.12

* updated install wizard for vxflexos2.12

* fixed unit tests

* fix indentation in generate-yaml.test.js

* fix linter issues

* fix linters in generate-yaml.test.js
  • Loading branch information
rishabhatdell authored Aug 27, 2024
1 parent e8861b8 commit aca481d
Show file tree
Hide file tree
Showing 16 changed files with 573 additions and 35 deletions.
2 changes: 1 addition & 1 deletion content/docs/csidriver/_index.md
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@ The CSI Drivers by Dell implement an interface between [CSI](https://kubernetes-
{{<table "table table-striped table-bordered table-sm">}}
| Features | PowerMax | PowerFlex | Unity XT | PowerScale | PowerStore |
|--------------------------|:--------:|:---------:|:---------:|:----------:|:----------:|
| CSI Driver version | 2.11.0 | 2.11.0 | 2.11.0 | 2.11.0 | 2.11.0 |
| CSI Driver version | 2.11.0 | 2.12.0 | 2.11.0 | 2.11.0 | 2.11.0 |
| Static Provisioning | yes | yes | yes | yes | yes |
| Dynamic Provisioning | yes | yes | yes | yes | yes |
| Expand Persistent Volume | yes | yes | yes | yes | yes |
Expand Down
2 changes: 1 addition & 1 deletion content/docs/csidriver/release/powerflex.md
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ title: PowerFlex
description: Release notes for PowerFlex CSI driver
---

## Release Notes - CSI PowerFlex v2.11.0
## Release Notes - CSI PowerFlex v2.12.0



Expand Down
4 changes: 2 additions & 2 deletions content/docs/deployment/csminstallationwizard/_index.md
Original file line number Diff line number Diff line change
Expand Up @@ -21,10 +21,10 @@ The [Dell Container Storage Modules Installation Wizard](./src/index.html) is a
| CSI PowerMax | 2.10.1 |✔️ |✔️ |
| CSI PowerMax | 2.9.1 |✔️ |✔️ |
| CSI PowerMax | 2.8.0 |✔️ |✔️ |
| CSI PowerFlex | 2.12.0 |✔️ ||
| CSI PowerFlex | 2.11.0 |✔️ ||
| CSI PowerFlex | 2.10.1 |✔️ ||
| CSI PowerFlex | 2.10.1 |✔️ ||
| CSI PowerFlex | 2.9.1 |✔️ ||
| CSI PowerFlex | 2.8.0 |✔️ ||
| CSI PowerScale | 2.11.0 |✔️ |✔️ |
| CSI PowerScale | 2.10.1 |✔️ |✔️ |
| CSI PowerScale | 2.9.1 |✔️ |✔️ |
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ weight: 5
description: Release notes for CSM Installation Wizard
---

## Release Notes - CSM Installation Wizard 1.3.0
## Release Notes - CSM Installation Wizard 1.4.0



Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
csmVersion=1.11.0
csmVersion=1.12.0
imageRepository=dellemc
controllerCount=1
nodeSelectorLabel=node-role.kubernetes.io/control-plane:
Expand Down
8 changes: 4 additions & 4 deletions content/docs/deployment/csminstallationwizard/src/index.html
Original file line number Diff line number Diff line change
Expand Up @@ -90,10 +90,10 @@
<div class="col-sm-9"></div>
<div class="col-sm-3">
<select id="csm-version" class="form-select" onchange="onCSMVersionChange(); validateInput(validateForm, CONSTANTS)" required>
<option value="1.8.0">CSM 1.8</option>
<option value="1.9.3">CSM 1.9</option>
<option value="1.10.2">CSM 1.10</option>
<option value="1.11.0" selected>CSM 1.11</option>
<option value="1.9.3">CSM 1.9</option>
<option value="1.10.2">CSM 1.10</option>
<option value="1.11.0">CSM 1.11</option>
<option value="1.12.0" selected>CSM 1.12</option>
</select>
</div>
</div>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -40,10 +40,10 @@ const CONSTANTS = {
PROPERTIES: ".properties",
HELM: "helm",
OPERATOR: "operator",
CSM_HELM_V180: "1.1.0",
CSM_HELM_V193: "1.2.1",
CSM_HELM_V1102: "1.3.2",
CSM_HELM_V1110: "1.4.0",
CSM_HELM_V1120: "1.5.0",
HELM_TAINTS: `
- key: "$KEY"
operator: "Exists"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -42,10 +42,10 @@ const CONSTANTS = {
PROPERTIES: ".properties",
HELM: "helm",
OPERATOR: "operator",
CSM_HELM_V180: "1.1.0",
CSM_HELM_V193: "1.2.1",
CSM_HELM_V1102: "1.3.2",
CSM_HELM_V1110: "1.4.0",
CSM_HELM_V1120: "1.5.0",
HELM_TAINTS: `
- key: "$KEY"
operator: "Exists"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -68,10 +68,10 @@ const CONSTANTS = {
PROPERTIES: ".properties",
HELM: "helm",
OPERATOR: "operator",
CSM_HELM_V180: "1.1.0",
CSM_HELM_V193: "1.2.1",
CSM_HELM_V1102: "1.3.2",
CSM_HELM_V1110: "1.4.0",
CSM_HELM_V1120: "1.5.0",
};

describe("GIVEN onAuthorizationChange function", () => {
Expand Down Expand Up @@ -561,8 +561,8 @@ describe("GIVEN resetTaint function", () => {
describe("GIVEN displayModules function", () => {
const testHtml = `
<select id="csm-version">
<option value="1.7.0" selected>CSM 1.7.0</option>
<option value="1.6.0">CSM 1.6.0</option>
<option value="1.10.0" selected>CSM 1.10.0</option>
<option value="1.9.0">CSM 1.9.0</option>
</select>
<input type="text" id="driver-namespace">
<div class="vgsnapshot" style="display:none"></div>
Expand Down Expand Up @@ -722,7 +722,7 @@ describe("GIVEN displayCommands function", () => {
<input id="array" value="powerstore">
<input id="installation-type" value="helm">
<input type="text" id="driver-namespace" value="csi-powerstore">
<input type="text" id="csm-version" value="1.8.0">
<input type="text" id="csm-version" value="1.12.0">
<div id="command-text-area" style="display:none">
<div id="command-title"></div>
<span id="command-note" style="display:none"></span>
Expand All @@ -737,15 +737,15 @@ describe("GIVEN displayCommands function", () => {
expect($("#command-title").text()).toEqual("Run the following commands to install");
expect($("#command-note").text()).toEqual("Ensure that the namespaces and secrets are created before installing the helm chart");
expect($("#command1").text()).toEqual("helm repo add dell https://dell.github.io/helm-charts");
expect($("#command2").text()).toEqual("helm install powerstore dell/container-storage-modules -n csi-powerstore --version 1.1.0 -f values.yaml");
expect($("#command2").text()).toEqual("helm install powerstore dell/container-storage-modules -n csi-powerstore --version 1.5.0 -f values.yaml");
});

test("SHOULD show expected commands: Installation Type: Operator", () => {
document.body.innerHTML = `
<input id="array" value="powerstore">
<input id="installation-type" value="operator">
<input type="text" id="driver-namespace" value="csi-powerstore">
<input type="text" id="csm-version" value="1.7.0">
<input type="text" id="csm-version" value="1.9.0">
<div id="command-text-area" style="display:none">
<div id="command-title"></div>
<span id="csm-operator-note" style="display:none"></span>
Expand All @@ -772,7 +772,7 @@ describe("SHOULD Disable/Enable Generate YAML button based on validation of inpu
<input id="array" value="">
<input id="installation-type" value="helm">
<input id="image-repository" value="dell">
<input id="csm-version" value="csm-1.6.0">
<input id="csm-version" value="csm-1.9.0">
<input type="number" id="controller-count" value="1">
<input type="text" id="driver-namespace" value="csm-driver">
`;
Expand All @@ -786,7 +786,7 @@ describe("SHOULD Disable/Enable Generate YAML button based on validation of inpu
<input id="array" value="powerstore">
<input id="installation-type" value="helm">
<input id="image-repository" value="dellemc">
<input id="csm-version" value="csm-1.6.0">
<input id="csm-version" value="csm-1.9.0">
<input type="number" id="controller-count" value="1">
<input type="text" id="driver-namespace" value="">
`;
Expand All @@ -801,7 +801,7 @@ describe("SHOULD Disable/Enable Generate YAML button based on validation of inpu
<input id="array" value="powerstore">
<input id="installation-type" value="helm">
<input id="image-repository" value="dell">
<input id="csm-version" value="csm-1.6.0">
<input id="csm-version" value="csm-1.9.0">
<input type="text" id="driver-namespace" value="csi-powerstore">
`;

Expand All @@ -814,7 +814,7 @@ describe("SHOULD Disable/Enable Generate YAML button based on validation of inpu
<input id="array" value="powerstore">
<input id="installation-type" value="helm">
<input id="image-repository" value="dell">
<input id="csm-version" value="csm-1.6.0">
<input id="csm-version" value="csm-1.9.0">
<input type="number" id="controller-count" value="1">
<input type="text" id="driver-namespace" value="csi-powerstore">
`;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -45,10 +45,10 @@ const CONSTANT_PARAM = {
PROPERTIES: ".properties",
HELM: "helm",
OPERATOR: "operator",
CSM_HELM_V180: "1.1.0",
CSM_HELM_V193: "1.2.1",
CSM_HELM_V1102: "1.3.2",
CSM_HELM_V1110: "1.4.0",
CSM_HELM_V1120: "1.5.0",
HELM_TAINTS: `
- key: "$KEY"
operator: "Exists"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -471,9 +471,6 @@ function displayCommands(releaseNameValue, commandTitleValue, commandNoteValue,
installationType = document.getElementById("installation-type").value
var helmChartVersion;
switch (csmVersion) {
case "1.8.0":
helmChartVersion = CONSTANTS.CSM_HELM_V180;
break;
case "1.9.3":
helmChartVersion = CONSTANTS.CSM_HELM_V193;
break;
Expand All @@ -483,8 +480,11 @@ function displayCommands(releaseNameValue, commandTitleValue, commandNoteValue,
case "1.11.0":
helmChartVersion = CONSTANTS.CSM_HELM_V1110;
break;
case "1.12.0":
helmChartVersion = CONSTANTS.CSM_HELM_V1120;
break;
default:
helmChartVersion = CONSTANTS.CSM_HELM_V1110;
helmChartVersion = CONSTANTS.CSM_HELM_V1120;
break;
}
$("#command-text-area").show();
Expand Down
Loading

0 comments on commit aca481d

Please sign in to comment.