----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64583/#review193762 -----------------------------------------------------------
Ship it! Ship It! - Jie Yu On Dec. 13, 2017, 11:32 p.m., Chun-Hung Hsiao wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/64583/ > ----------------------------------------------------------- > > (Updated Dec. 13, 2017, 11:32 p.m.) > > > Review request for mesos, James DeFelice, Jie Yu, and Joseph Wu. > > > Repository: mesos > > > Description > ------- > > The storage local resource provider will now report two catagories of > RAW resources: resources that represents storage pools with profiles, > and resources that represents pre-existing volumes with IDs but no > profile. When applying `CREATE_VOLUME` or `CREATE_BLOCK` on pre-existing > volumes, we issue a `ValicateVolumeCapabilities` CSI call with the > default Mount or Block capabilities and convert the RAW resources into > MOUNT, PATH or BLOCK resources. When applying `DESTROY_VOLUME` or > `DESTROY_BLOCK` on these resources, they will be converted back to RAW > resources with IDs, but the pre-existing volumes won't be deleted. > > > Diffs > ----- > > src/resource_provider/storage/provider.cpp > e806f44ef33405d4a2b133576c60be56e9fe3435 > > > Diff: https://reviews.apache.org/r/64583/diff/2/ > > > Testing > ------- > > See later in chain. > > > Thanks, > > Chun-Hung Hsiao > >