Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: patch for volumeMounts #546

Merged
merged 1 commit into from
Aug 7, 2024
Merged

Conversation

bangqipropel
Copy link
Collaborator

@bangqipropel bangqipropel commented Aug 2, 2024

Reason for Change:
add volumeMounts and volume patch for initContainers
add nil check for annotations

Requirements

  • added unit tests and e2e tests (if applicable).

Issue Fixed:

Notes for Reviewers:

@bangqipropel bangqipropel changed the title patch for volumeMounts chore:patch for volumeMounts Aug 2, 2024
@bangqipropel bangqipropel force-pushed the bangqipropel/patch_for_volumeMounts branch from 0fd4b22 to a792c30 Compare August 2, 2024 17:38
@bangqipropel bangqipropel changed the title chore:patch for volumeMounts chore: patch for volumeMounts Aug 2, 2024
Copy link

codecov bot commented Aug 2, 2024

Codecov Report

Attention: Patch coverage is 50.00000% with 11 lines in your changes missing coverage. Please review.

Project coverage is 57.02%. Comparing base (ed546f0) to head (91e4eae).
Report is 26 commits behind head on main.

Files Patch % Lines
pkg/controllers/workspace_controller.go 64.70% 3 Missing and 3 partials ⚠️
pkg/resources/manifests.go 0.00% 5 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #546      +/-   ##
==========================================
- Coverage   61.10%   57.02%   -4.09%     
==========================================
  Files          29       30       +1     
  Lines        2602     2927     +325     
==========================================
+ Hits         1590     1669      +79     
- Misses        921     1160     +239     
- Partials       91       98       +7     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@bangqipropel bangqipropel force-pushed the bangqipropel/patch_for_volumeMounts branch from a792c30 to 774d1bb Compare August 3, 2024 01:30
@bangqipropel bangqipropel force-pushed the bangqipropel/patch_for_volumeMounts branch from e3ef3a5 to f538582 Compare August 5, 2024 20:53
@bangqipropel bangqipropel force-pushed the bangqipropel/patch_for_volumeMounts branch from f538582 to d5fc713 Compare August 5, 2024 21:24
@bangqipropel bangqipropel force-pushed the bangqipropel/patch_for_volumeMounts branch from 0bbcfc1 to e333389 Compare August 6, 2024 20:45
@bangqipropel bangqipropel force-pushed the bangqipropel/patch_for_volumeMounts branch from e333389 to 7082cab Compare August 6, 2024 21:47
@bangqipropel bangqipropel force-pushed the bangqipropel/patch_for_volumeMounts branch from 7082cab to 4736725 Compare August 6, 2024 22:29
@bangqipropel bangqipropel force-pushed the bangqipropel/patch_for_volumeMounts branch from 4736725 to 0492865 Compare August 6, 2024 23:02
Signed-off-by: Bangqi Zhu <[email protected]>
@Fei-Guo Fei-Guo merged commit 08066f6 into main Aug 7, 2024
6 of 8 checks passed
@Fei-Guo Fei-Guo deleted the bangqipropel/patch_for_volumeMounts branch August 7, 2024 23:45
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.

None yet

2 participants