Skip to content

Conversation

@dli357
Copy link
Contributor

@dli357 dli357 commented Oct 20, 2021

This feature adds the ability for a user to directly specify custom k8s volumes and volume mounts in the CDAP CR.

Custom volumes and volume mounts specified in the upper-level CR will be applied to all services, and custom volumes and volume mounts specified in the particular service will only be applied to that service. Custom volume mounts for a service are applied to all containers and init containers for that service.

@dli357 dli357 requested review from chtyim and wyzhang October 20, 2021 06:56
@dli357 dli357 self-assigned this Oct 20, 2021
@google-cla google-cla bot added the cla: yes label Oct 20, 2021
@dli357 dli357 force-pushed the feature/CDAP-18568-Custom-Volumes branch 2 times, most recently from e036adc to 20ea947 Compare October 20, 2021 07:11
@dvitiiuk
Copy link

Checked that this change satisfying the next requirements (which caused the creation of #72):

  • managing of file-based secure store on a separated volume
  • storing RBAC-related artifacts on a separated volume.

@dli357 dli357 force-pushed the feature/CDAP-18568-Custom-Volumes branch from 20ea947 to fad9f18 Compare December 1, 2021 02:59
@dli357 dli357 force-pushed the feature/CDAP-18568-Custom-Volumes branch 3 times, most recently from 0f4251e to fdf0de4 Compare January 11, 2022 21:13
@dli357 dli357 requested a review from wyzhang January 13, 2022 19:45
@dli357 dli357 force-pushed the feature/CDAP-18568-Custom-Volumes branch from fdf0de4 to 4d1075c Compare January 18, 2022 21:54
…stem Services

    [CDAP-18568] Switch name to AdditionalVolumes and AdditionalVolumeMounts and added clarifying comments
@dli357 dli357 force-pushed the feature/CDAP-18568-Custom-Volumes branch from 4d1075c to f71a513 Compare January 18, 2022 22:33
@dli357 dli357 merged commit 9d9da64 into develop Jan 18, 2022
@dli357 dli357 deleted the feature/CDAP-18568-Custom-Volumes branch January 18, 2022 22:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants