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

test: create a shared LV for use by two app pods #315

Merged
merged 1 commit into from
Jun 24, 2024
Merged

Conversation

dsharma-dc
Copy link
Contributor

Pull Request template

Please, go through these steps before you submit a PR.

Why is this PR required? What issue does it fix?:
Adds a test for shared volume.

What this PR does?:
Creates a LV by specifying shared parameter in the storage class so that the volume can be mounted to more than one path.

Does this PR require any upgrade changes?:
No

If the changes in this PR are manually verified, list down the scenarios covered::

Any additional information for your reviewer? :
Mention if this PR is part of any design or a continuation of previous PRs

Checklist:

  • Fixes #
  • PR Title follows the convention of <type>(<scope>): <subject>
  • Has the change log section been updated?
  • Commit has unit tests
  • Commit has integration tests
  • (Optional) Are upgrade changes included in this PR? If not, mention the issue/PR to track:
  • (Optional) If documentation changes are required, which issue on https://github.com/openebs/openebs-docs is used to track them:

tests/utils.go Outdated Show resolved Hide resolved
tests/provision_test.go Outdated Show resolved Hide resolved
@dsharma-dc dsharma-dc force-pushed the shared_lv branch 2 times, most recently from b1e7e56 to f7af68d Compare June 20, 2024 11:45
@dsharma-dc dsharma-dc requested a review from niladrih June 20, 2024 11:46
@dsharma-dc dsharma-dc merged commit bddedee into develop Jun 24, 2024
5 checks passed
@dsharma-dc dsharma-dc deleted the shared_lv branch June 24, 2024 16:29
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.

3 participants