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

fix(mountpath): fix sock file mount path in pool deployments #1605

Merged
merged 2 commits into from
Feb 6, 2020

Conversation

mittachaitu
Copy link

@mittachaitu mittachaitu commented Feb 5, 2020

Signed-off-by: mittachaitu [email protected]

What this PR does / why we need it:
This PR fixes the shadow mount points in cStor-pool deployment by
updating sock file mount path from /var/run to /var/tmp/sock.
This PR handles upgrade changes and new cStor pool deployments.

Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #openebs/openebs#2902

Special notes for your reviewer:
Tested the changes on gke(using disks) by building cstor-pool image and cstor-pool-mgmt image on top of cstor-pool image.(Test images: mittachaitu/cstor-pool:sock_path1, mittachaitu/cstor-pool-mgmt:fix_sock1)
Checklist:

  • Fixes #
  • Labelled this PR & related issue with documentation tag
  • PR messages has document related information
  • Labelled this PR & related issue with breaking-changes tag
  • PR messages has breaking changes related information
  • Labelled this PR & related issue with requires-upgrade tag
  • PR messages has upgrade related information
  • Commit has unit tests
  • Commit has integration tests

Copy link
Contributor

@shubham14bajpai shubham14bajpai left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@prateekpandey14 prateekpandey14 left a comment

Choose a reason for hiding this comment

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

lgtm

Copy link
Contributor

@mynktl mynktl left a comment

Choose a reason for hiding this comment

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

lgtm 👍

Copy link
Contributor

@vishnuitta vishnuitta left a comment

Choose a reason for hiding this comment

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

changes are good

@prateekpandey14 prateekpandey14 merged commit 4af590f into openebs-archive:master Feb 6, 2020
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