Skip to content
This repository has been archived by the owner on Oct 22, 2021. It is now read-only.

Enable persi nfs support #382

Closed
gaktive opened this issue Feb 5, 2020 · 8 comments
Closed

Enable persi nfs support #382

gaktive opened this issue Feb 5, 2020 · 8 comments
Labels
changelog Issue must be present in the release notes. Priority: Critical Size: 8 Status: Not Implemented This issue will not be implemented due to reasons that should be listed on issue conversation thread suse-cap Type: Enhancement New feature or request
Milestone
2.4.0

Comments

@gaktive
Copy link
Collaborator

gaktive commented Feb 5, 2020 •
edited by andreas-kupries
Loading

Is your feature request related to a problem? Please describe.
We need to be able to support persistent storage for apps in the case where Eirini isn't in use. Persi is a volume service broker that can tackle this via mapfs and nfs-server. We have this in scf but it'll be very much needed in kubecf.

Blocking #29
@gaktive gaktive added this to the 1.0.0 milestone Feb 5, 2020
@gaktive gaktive modified the milestones: 1.0.0, 1.1.0 Feb 5, 2020
@gaktive gaktive added the Type: Documentation Improvements or additions to documentation label Feb 21, 2020
@gaktive
Copy link
Collaborator Author

gaktive commented Feb 21, 2020

Based on SUSE/doc-cap#144, we'll need to verify how to do this since documentation has run into problems following upstream material.

@fargozhu fargozhu added the Status: Accepted This issue will be implemented in a near future label Mar 6, 2020
@andreas-kupries andreas-kupries mentioned this issue Mar 9, 2020
Merged
@fargozhu fargozhu removed this from the 1.1.0 milestone Mar 12, 2020
@viovanov viovanov added Size: 8 Priority: Critical and removed Type: Documentation Improvements or additions to documentation Priority: Medium labels Apr 30, 2020
@mook-as mook-as self-assigned this May 1, 2020
@viovanov viovanov assigned viovanov and unassigned mook-as May 5, 2020
@viovanov viovanov added suse-cap and removed suse-cap labels May 5, 2020
@viovanov viovanov removed the suse-cap label May 14, 2020
@viovanov viovanov removed their assignment May 14, 2020
@gaktive gaktive mentioned this issue May 19, 2020
Closed
@gaktive gaktive removed the suse-cap label May 22, 2020
@mook-as
Copy link
Contributor

mook-as commented May 27, 2020

@viovanov Would it be possible to fork https://github.com/cloudfoundry/nfs-volume-release and https://github.com/cloudfoundry/mapfs-release while we get things in gear, so I have somewhere to PR in-progress things to? Currently this is necessary as upstream has no BPM support. I'm also not sure yet how my changes would fare on a BOSH VM based deployment.

This was referenced May 28, 2020
Open
Draft
Closed
@mook-as mook-as added the Status: Blocked Dependencies on other issues and/or pull requests label Jun 1, 2020
@mook-as
Copy link
Contributor

mook-as commented Jun 1, 2020

Marking Blocked based on quarks-operator dependencies.

@mook-as mook-as removed their assignment Jun 16, 2020
@fargozhu fargozhu added this to the jumanji milestone Jul 8, 2020
@gaktive
Copy link
Collaborator Author

gaktive commented Jul 28, 2020

Duplicate of #957

@gaktive gaktive marked this as a duplicate of #957 Jul 28, 2020
@gaktive gaktive closed this as completed Jul 28, 2020
@mook-as
Copy link
Contributor

mook-as commented Jul 28, 2020

@gaktive The issue to do a task can't be a dupe of the PR to fix it, can it‽

@mook-as
Copy link
Contributor

mook-as commented Jul 30, 2020

To clarify: we're planning on:

  • Ignoring persi-NFS for diego.
  • Supporting persi in Eirini
    • Via storage classes only

@fargozhu fargozhu added Status: Not Implemented This issue will not be implemented due to reasons that should be listed on issue conversation thread and removed Status: Accepted This issue will be implemented in a near future Status: Blocked Dependencies on other issues and/or pull requests labels Aug 24, 2020
@fargozhu fargozhu removed this from the jumanji milestone Aug 24, 2020
@fargozhu fargozhu added Status: Done Implemented and PR merged changelog Issue must be present in the release notes. and removed Status: Not Implemented This issue will not be implemented due to reasons that should be listed on issue conversation thread labels Aug 24, 2020
@fargozhu fargozhu added this to the 2.4.0 milestone Aug 24, 2020
@fargozhu fargozhu added Type: Enhancement New feature or request Status: Not Implemented This issue will not be implemented due to reasons that should be listed on issue conversation thread and removed Status: Done Implemented and PR merged labels Aug 24, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
changelog Issue must be present in the release notes. Priority: Critical Size: 8 Status: Not Implemented This issue will not be implemented due to reasons that should be listed on issue conversation thread suse-cap Type: Enhancement New feature or request
Projects
None yet
Milestone
2.4.0
Development

Successfully merging a pull request may close this issue.

persi-nfs support cloudfoundry-incubator/kubecf
4 participants
@fargozhu @viovanov @mook-as @gaktive