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

feat: add database snapshot functionality to Helm chart #9956

Open
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

carolinaecalderon
Copy link
Contributor

@carolinaecalderon carolinaecalderon commented Sep 18, 2024

Ticket

CM-448

Description

See notes: https://hpe-aiatscale.atlassian.net/wiki/spaces/ENGINEERIN/pages/1716191279/Guide+Backporting+fixes+to+past+release+branches
TLDR; add Helm values such that you can easily snapshot/backup OR restore a database's persistent volume.

Test Plan

Manual testing or N/A

Checklist

  • Changes have been manually QA'd
  • New features have been approved by the corresponding PM
  • User-facing API changes have the "User-facing API Change" label
  • Release notes have been added as a separate file under docs/release-notes/
    See Release Note for details.
  • Licenses have been included for new code which was copied and/or modified from any external code

@cla-bot cla-bot bot added the cla-signed label Sep 18, 2024
Copy link

netlify bot commented Sep 18, 2024

Deploy Preview for determined-ui canceled.

Name Link
🔨 Latest commit 05b4c40
🔍 Latest deploy log https://app.netlify.com/sites/determined-ui/deploys/66ec76a884aa1e0008e95736

Copy link

codecov bot commented Sep 18, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 59.18%. Comparing base (867eb31) to head (05b4c40).
Report is 23 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #9956   +/-   ##
=======================================
  Coverage   59.18%   59.18%           
=======================================
  Files         751      751           
  Lines      104462   104462           
  Branches     3598     3599    +1     
=======================================
  Hits        61824    61824           
  Misses      42506    42506           
  Partials      132      132           
Flag Coverage Δ
backend 43.80% <ø> (ø)
harness 72.75% <ø> (ø)
web 54.33% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

@carolinaecalderon carolinaecalderon changed the title Carolinac/rp db snapshot feat: add database snapshot functionality to Helm chart Sep 19, 2024
@carolinaecalderon carolinaecalderon marked this pull request as ready for review September 19, 2024 21:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant