[ 
https://issues.apache.org/jira/browse/MESOS-9956?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16919445#comment-16919445
 ] 

Benno Evers commented on MESOS-9956:
------------------------------------

{noformat}
commit 43b86da531a889b1c4b1d7ca6acb2eb924ea01e1 (origin/master, master)
Author: Chun-Hung Hsiao <chhs...@apache.org>
Date:   Fri Aug 30 13:04:22 2019 +0200

    Gracefully handled duplicated volumes from non-conforming CSI plugins.
    
    If the SLRP uses a plugin that does not conform to the CSI spec and
    reports duplicated volumes, the duplicate would be removed.
    
    Review: https://reviews.apache.org/r/71414/


commit b18ce53fe8e49e6f030efe89e0976a9f72ad8b50 (1.9.x, origin/1.9.x)
Author: Chun-Hung Hsiao <chhs...@apache.org>
Date:   Fri Aug 30 13:05:37 2019 +0200

    Gracefully handled duplicated volumes from non-conforming CSI plugins.
    
    If the SLRP uses a plugin that does not conform to the CSI spec and
    reports duplicated volumes, the duplicate would be removed.
    
    Review: https://reviews.apache.org/r/71414/
{noformat}

> CSI plugins reporting duplicated volumes will crash the agent.
> --------------------------------------------------------------
>
>                 Key: MESOS-9956
>                 URL: https://issues.apache.org/jira/browse/MESOS-9956
>             Project: Mesos
>          Issue Type: Bug
>          Components: storage
>            Reporter: Chun-Hung Hsiao
>            Assignee: Chun-Hung Hsiao
>            Priority: Blocker
>              Labels: mesosphere, storage
>
> The CSI spec requires volumes to be uniquely identifiable by ID, and thus 
> SLRP currently assumes that a {{ListVolumes}} call does not return duplicated 
> volumes. However, if a SLRP uses a non-conforming CSI plugin that reports 
> duplicated volumes, these volumes would corrupt the SLRP checkpoint and cause 
> the agent to crash at the next reconciliation:
> {noformat}
>  F0829 07:13:55.171332 12721 provider.cpp:1089] Check failed: 
> !checkpointedMap.contains(resource.disk().source().id()){noformat}
> MESOS-9254 introduces periodic reconciliation which make this problem much 
> easier to manifest.



--
This message was sent by Atlassian Jira
(v8.3.2#803003)

Reply via email to