Skip to content

Mention about the possibilities of using staging_target_path at NodeStage#386

Open
humblec wants to merge 2 commits into
container-storage-interface:masterfrom
humblec:staging
Open

Mention about the possibilities of using staging_target_path at NodeStage#386
humblec wants to merge 2 commits into
container-storage-interface:masterfrom
humblec:staging

Conversation

@humblec

@humblec humblec commented Aug 31, 2019

Copy link
Copy Markdown
Contributor

At present, it is unclear to SP that, how to use staging_target_path
and the possibilities of using the same. This patch add more details
on staging target path and its usage.

Fix # #385

Signed-off-by: Humble Chirammal hchiramm@redhat.com

@humblec

humblec commented Aug 31, 2019

Copy link
Copy Markdown
Contributor Author

/assign @msau42

@shay-berman

Copy link
Copy Markdown

I think its a good clarification.
We actually use the staging target path to store metadata file (with the discovered multipath device), in order to have that metadata during the NodePublishVolume(uses the metadata file to operate the l mkfs+mount it to target path) and for NodeUnStage (uses the metadata to clean up the multipath device).

So I think its good clarification that staging_target_path can be used for CSI driver metadata for example.

…tage.

At present, it is unclear to SP that, how to use staging_target_path
and the possibilities of using the same. This patch add more details
on `staging target path` and its usage.

Fix # container-storage-interface#385

Signed-off-by: Humble Chirammal <hchiramm@redhat.com>
@humblec

humblec commented Sep 5, 2019

Copy link
Copy Markdown
Contributor Author

@saad-ali

@msau42

msau42 commented Sep 5, 2019

Copy link
Copy Markdown

/assign @saad-ali

Comment thread spec.md Outdated
Comment thread spec.md Outdated
Signed-off-by: hchiramm <hchiramm@redhat.com>
@humblec

humblec commented Apr 23, 2020

Copy link
Copy Markdown
Contributor Author

@saad-ali @msau42 can you please review ?

@saad-ali

Copy link
Copy Markdown
Member

I have no objections to this. But please bring it up at the next CSI meeting, would be good to get more eyeballs on this before merge.

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.

6 participants