Skip to content
This repository has been archived by the owner on Jun 6, 2024. It is now read-only.

Add environment and handle corner cases #2525

Merged
merged 1 commit into from
Apr 10, 2019
Merged

Conversation

wangdian
Copy link
Member

  1. Add dev-box environment for storage plugin.
  2. Handle case when setup storage server with no storage user.

2. Handle case when setup storage server with no storage user.
@wangdian wangdian requested a review from ydye April 10, 2019 06:33
@coveralls
Copy link

Coverage Status

Coverage remained the same at 52.932% when pulling def4a54 on diawang/storage_fix into 06f7311 on master.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage remained the same at 52.932% when pulling def4a54 on diawang/storage_fix into 06f7311 on master.

@wangdian wangdian merged commit 07e4e5f into master Apr 10, 2019
@wangdian
Copy link
Member Author

Test case:

  1. Use a clean PAI instance or check you k8s configmap in [your ip]/kubernetes-dashboard/#!/configmap?namespace=default, backup storage-user and remove it.
  2. In dev box. Go to /pai/contrib/storage_plugin, run
    python storagectl.py server set [teamname] nfs [nfs ip] [nfs root path] --sharedfolders data --privatefolders users
  3. The test case passed if no exception occurs.
  4. Recover storage-user configmap

@wangdian wangdian deleted the diawang/storage_fix branch November 28, 2019 08:53
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants