mesos-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Joseph Wu <>
Subject Re: Review Request 64351: Moved generated CSI code into public directories.
Date Fri, 08 Dec 2017 00:16:54 GMT

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

(Updated Dec. 7, 2017, 4:16 p.m.)

Review request for mesos, Chun-Hung Hsiao and Jie Yu.


Remove ugly hack in favor of a `-I` flag with a TODO.  This will let the private protobufs
stay private.

Bugs: MESOS-8251

Repository: mesos


This moves the Container Storage Interface (CSI) code generated from
its protobufs into the `include` directory so that other public
headers can use them.  The `csi/spec.hpp` was moved too, because
we prefer to include a layer of header indirection when adding
generated headers into non-generated headers.

An upcoming module specifically for CSI support will need to return
objects defined in the CSI specification.  And because modules have
public interfaces, we'll need the generated headers to be public too.

Diffs (updated)

  src/ be105f485124579f70888bda69d3c37fd5a18770 
  src/csi/client.hpp 91dda0d44544c8901a9f71ef7f8338205b8ed598 
  src/csi/spec.hpp c819be30312e32275b2d17ade35ba9f500f81b80 
  src/tests/mock_csi_plugin.hpp 162dc6499a7ae880b30ec801ed399b7a3438bdc6 




See end of chain.


Joseph Wu

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