Skip to content

Combine input & output volumes#124

Merged
TommyLike merged 1 commit intomasterfrom
feature/combine_volumes
May 5, 2019
Merged

Combine input & output volumes#124
TommyLike merged 1 commit intomasterfrom
feature/combine_volumes

Conversation

@TommyLike
Copy link
Copy Markdown
Contributor

Sync feature of combine input & output volumes.

For #122

MountPath string `json:"mountPath" protobuf:"bytes,1,opt,name=mountPath"`

// defined the PVC name
VolumeClaimName string `json:"volumeClaimName,omitempty" protobuf:"bytes,2,opt,name=volumeClaimName"`
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Why would u add this? Would prefer auto gen it.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Yeah, there would be a few codes which can be optimized, would like to do it when the initial sync patch get merged.

@TommyLike TommyLike force-pushed the feature/combine_volumes branch from fa9fcc4 to b7ecae3 Compare May 5, 2019 05:42
@TommyLike TommyLike force-pushed the feature/combine_volumes branch from b7ecae3 to 71defce Compare May 5, 2019 06:13
@dingtsh1
Copy link
Copy Markdown
Contributor

dingtsh1 commented May 5, 2019

/lgtm

@TommyLike TommyLike merged commit 40e85f4 into master May 5, 2019
@k82cn k82cn deleted the feature/combine_volumes branch May 5, 2019 08:26
kevin-wangzefeng pushed a commit to kevin-wangzefeng/volcano that referenced this pull request Jun 28, 2019
JesseStutler pushed a commit to JesseStutler/volcano that referenced this pull request Mar 26, 2025
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