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

Operator yaml to helm chart #292

Merged
merged 9 commits into from
Apr 11, 2024
Merged

Operator yaml to helm chart #292

merged 9 commits into from
Apr 11, 2024

Conversation

Abhinandan-Purkait
Copy link
Member

Changes:

  • change install command from operator yaml based kubectl apply to helm install.
  • CI changes to use helm chart instead of operator-yaml.
  • doc changes to use deployment for k8s CSI controller
  • replaces docs wherever kube-system is used instead of openebs

README.md Outdated
@@ -51,7 +51,7 @@ The orignal v1.0 dev roadmap [is here ](https://github.com/orgs/openebs/projects
> Before installing the LVM-LocalPV driver please make sure your Kubernetes Cluster meets the following prerequisites:
> 1. All the nodes must have LVM2 utils package installed
> 2. All the nodes must have dm-snapshot Kernel Module loaded - (Device Mapper Snapshot)
> 4. You have access to install RBAC components into kube-system namespace. The OpenEBS LVM driver components are installed in kube-system namespace to allow them to be flagged as system critical components.
> 4. You have access to install RBAC components into <OPENEBS> namespace.
<BR>

> [!NOTE]
Copy link
Member Author

Choose a reason for hiding this comment

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

@dave-brace The below contents are not supported at the moment we might need to move it under a roadmap section. The users are still needed to created the PV and VG on their own for localpv LVM.

README.md Outdated Show resolved Hide resolved
upgrade/README.md Outdated Show resolved Hide resolved
docs/faq.md Outdated Show resolved Hide resolved
Copy link
Member

@avishnu avishnu left a comment

Choose a reason for hiding this comment

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

Lgtm

Signed-off-by: Abhinandan Purkait <[email protected]>
Signed-off-by: Abhinandan Purkait <[email protected]>
Adopters.md Show resolved Hide resolved
Adopters.md Show resolved Hide resolved
Signed-off-by: Abhinandan Purkait <[email protected]>
@Abhinandan-Purkait Abhinandan-Purkait merged commit 731233a into develop Apr 11, 2024
4 checks passed
@avishnu avishnu deleted the update_docs branch April 11, 2024 12:37
**NOTE:** Installation using operator YAMLs is not the supported way any longer.
We can install the latest release of OpenEBS LVM driver by running the following command:
```bash
helm repo add openebs https://openebs.github.io/openebs
Copy link

Choose a reason for hiding this comment

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

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.

5 participants