Mailing-List: contact dev-help@cloudstack.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@cloudstack.apache.org
From: syed <git@git.apache.org>
To: dev@cloudstack.apache.org
Reply-To: dev@cloudstack.apache.org
References: <git-pr-1600-cloudstack@git.apache.org>
In-Reply-To: <git-pr-1600-cloudstack@git.apache.org>
Subject: [GitHub] cloudstack pull request #1600: Support Backup of Snapshots for Managed Stora...
Content-Type: text/plain
Message-Id: <20161003155918.0628CDFB89@git1-us-west.apache.org>
Date: Mon,  3 Oct 2016 15:59:18 +0000 (UTC)
archived-at: Mon, 03 Oct 2016 15:59:20 -0000

GitHub user syed reopened a pull request:

    https://github.com/apache/cloudstack/pull/1600

    Support Backup of Snapshots for Managed Storage

        This PR adds an ability to Pass a new parameter, locationType,
        to the “createSnapshot” API command. Depending on the locationType,
        we decide where the snapshot should go in case of managed storage.
    
        There are two possible values for the locationType param
    
        1) `Primary`: The standard operation for managed storage is to
        keep the snapshot on the device (primary). For non-managed storage, this will
        give an error as this option is only supported for managed storage
    
        2) `Secondary`: Applicable only to managed storage. This will
        keep the snapshot on the secondary storage. For non-managed
        storage, this will result in an error.
    
        The reason for implementing this feature is to avoid a single
        point of failure for primary storage. Right now in case of managed
        storage, if the primary storage goes down, there is no easy way
        to recover data as all snapshots are also stored on the primary.
        This features allows us to mitigate that risk.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/syed/cloudstack snapshot-archive-pr

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/cloudstack/pull/1600.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #1600
    
----
commit bfad68efa35b94f67470fd8b549aca0cc7ff7d30
Author: Syed <syed1.mushtaq@gmail.com>
Date:   2016-06-30T17:37:33Z

        Support Backup of Snapshots for Managed Storage
    
        This PR adds an ability to Pass a new parameter, locationType,
        to the “createSnapshot” API command. Depending on the locationType,
        we decide where the snapshot should go in case of managed storage.
    
        There are two possible values for the locationType param
    
        1) `Standard`: The standard operation for managed storage is to
        keep the snapshot on the device. For non-managed storage, this will
        be to upload it to secondary storage. This option will be the
        default.
    
        2) `Archive`: Applicable only to managed storage. This will
        keep the snapshot on the secondary storage. For non-managed
        storage, this will result in an error.
    
        The reason for implementing this feature is to avoid a single
        point of failure for primary storage. Right now in case of managed
        storage, if the primary storage goes down, there is no easy way
        to recover data as all snapshots are also stored on the primary.
        This features allows us to mitigate that risk.

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---