Project

General

Profile

Actions

Feature #17389

closed

Storage classes support in keepproxy

Added by Nico César about 3 years ago. Updated over 2 years ago.

Status:
Resolved
Priority:
Normal
Assigned To:
Category:
Keep
Target version:
Story points:
-
Release relationship:
Auto

Description

Support storage classes for PUT:

  • Propagate X-Keep-Storage-Classes headed received from clients, ensure that blocks are written to the correct storage classes
  • On success, return fulfilled X-Keep-Storage-Classes-Confirmed to client

Subtasks 1 (0 open1 closed)

Task #17819: Review 17389-keepproxy-storage-classes-confirmedResolvedPeter Amstutz06/18/2021Actions

Related issues

Related to Arvados Epics - Idea #16107: Storage classesResolved03/01/202109/30/2021Actions
Blocked by Arvados - Feature #17349: [Keep API] Revisit "replicas_desired" and "storage_classes_desired".RejectedActions
Actions #1

Updated by Nico César about 3 years ago

  • Target version set to To Be Groomed
Actions #2

Updated by Nico César about 3 years ago

Actions #3

Updated by Nico César about 3 years ago

  • Blocked by Feature #17349: [Keep API] Revisit "replicas_desired" and "storage_classes_desired". added
Actions #4

Updated by Nico César about 3 years ago

  • Description updated (diff)
Actions #5

Updated by Tom Clegg almost 3 years ago

  • Target version deleted (To Be Groomed)
  • Status changed from New to Resolved

I think this is covered in #13431 -- see func (s *ServerRequiredSuite) TestStorageClassesHeader(c *C) in source:services/keepproxy/keepproxy_test.go

Actions #6

Updated by Peter Amstutz almost 3 years ago

  • Target version set to 2021-06-23 sprint
  • Status changed from Resolved to Feedback
  • Subject changed from [keepproxy] Storage tiers to Storage classes support in keepproxy
Actions #7

Updated by Peter Amstutz almost 3 years ago

  • Description updated (diff)
Actions #8

Updated by Peter Amstutz almost 3 years ago

  • Target version changed from 2021-06-23 sprint to 2021-07-07 sprint
Actions #9

Updated by Lucas Di Pentima almost 3 years ago

  • Assigned To set to Lucas Di Pentima
  • Status changed from Feedback to New
Actions #10

Updated by Lucas Di Pentima almost 3 years ago

  • Status changed from New to In Progress
Actions #11

Updated by Lucas Di Pentima almost 3 years ago

Updates at d02322793 - branch 17389-keepproxy-storage-classes-confirmed
Test run: developer-run-tests: #2533

  • Returns the X-Keep-Storage-Classes-Confirmed header to the client on PUT success.
  • Adds test

The propagation of the X-Keep-Storage-Classes header from the client to the keepstores was already done in #13431

Actions #12

Updated by Peter Amstutz almost 3 years ago

  • Target version changed from 2021-07-07 sprint to 2021-06-23 sprint
Actions #13

Updated by Lucas Di Pentima almost 3 years ago

  • Target version changed from 2021-06-23 sprint to 2021-07-07 sprint
Actions #14

Updated by Peter Amstutz almost 3 years ago

Lucas Di Pentima wrote:

Updates at d02322793 - branch 17389-keepproxy-storage-classes-confirmed
Test run: developer-run-tests: #2533

  • Returns the X-Keep-Storage-Classes-Confirmed header to the client on PUT success.
  • Adds test

The propagation of the X-Keep-Storage-Classes header from the client to the keepstores was already done in #13431

One suggestion: add a comment explaining the logic (if the PUT succeeded, we are able to assume all the storage classes were fulfilled). Otherwise LGTM.

Actions #15

Updated by Lucas Di Pentima almost 3 years ago

  • Status changed from In Progress to Resolved
Actions #16

Updated by Peter Amstutz over 2 years ago

  • Release set to 42
Actions

Also available in: Atom PDF