mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Chun-Hung Hsiao <>
Subject Re: Review Request 70247: Added an "unversioned" `VolumeCapability`.
Date Wed, 20 Mar 2019 21:36:16 GMT

This is an automatically generated e-mail. To reply, visit:

(Updated March 20, 2019, 9:36 p.m.)

Review request for mesos, Benjamin Bannier, James DeFelice, Jie Yu, and Jan Schlicht.


Addressed Benjamin's comments and moved `evolve` from r/70248.

Bugs: MESOS-9625

Repository: mesos

Description (updated)

To support both CSI v0 and v1, the "unversioned" `VolumeCapability` is
designed to satisfy the following compatibility guarantees:

1. The unversioned `VolumeCapability` parsed from a versioned one should
   be able to used to reconstruct the original versioned
   `VolumeCapability`, and can be upgraded/downgraded to a different
   CSI version and preserve as much semantics as possible.

2. If an backward-incompatible change is introduced in future CSI
   `VolumeCapability`, the unversioned `VolumeCapability` can provide a
   way to do a backward compatible upgrade.

Diffs (updated)

  include/mesos/csi/types.hpp PRE-CREATION 
  include/mesos/csi/types.proto PRE-CREATION 
  src/CMakeLists.txt 3397c3b1d4e8a7900b2e5f870679cc7aa30b4be2 
  src/ d451d7cabe3bf5d4f5039cfac5de1b03ef891d07 
  src/csi/types.cpp PRE-CREATION 
  src/csi/utils.hpp 9145c6795c3ecdde5de5859a852763fe9aeb1ddf 
  src/csi/utils.cpp fd6f95d88caf69e2ae197cf940beb93c164565bc 




make check


Chun-Hung Hsiao

  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message