default cinder.conf should follow API deprecation warning

[David Comay]

The default version of Cinder's cinder.conf delivered in the Juno 2014.2.1 includes the following two entries (and corresponding configuration entries in cinder/common/config.py)

# DEPRECATED: Deploy v1 of the Cinder API. (boolean value)
#enable_v1_api=true

# Deploy v2 of the Cinder API. (boolean value)
#enable_v2_api=true

As a result, deploying with the stock cinder.conf results in the following WARNING to be logged

WARNING cinder.api [-] The v1 api is deprecated and will be removed after the Juno release. You should set enable_v1_api=false and enable_v2_api=true in your cinder.conf file.

To avoid confusion and further bug reports, switching the value of enable_v1_api default to false for the remaining Juno releases should be considered.

[git-harry (git-harry)]

There is a review is progress for this: https://review.openstack.org/#/c/145703/

[Sean McGinnis (sean-mcginnis)]

Automatically unassigning due to inactivity.

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.openstack.org/379657

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/379657
Committed: https://git.openstack.org/cgit/openstack/cinder/commit/?id=7fcca079ff63f1a1b6d4d3067508883f01515add
Submitter: Jenkins
Branch: master

commit 7fcca079ff63f1a1b6d4d3067508883f01515add
Author: Sean McGinnis <email address hidden>
Date: Thu Sep 29 13:28:29 2016 -0500

    Disable API v1 by default

    The v1 API has been deprecated by default since Juno. While we've
    determined we can't get rid of it in the foreseeable future, we
    shouldn't enable it by default. This may encourage any remaining
    users to migrate.

    This will also get rid of some warning messages in the logs on
    startup warning of the deprecated API.

    Change-Id: Ieea9a6c764eb1b55ad399a96e218d773ad4c59a5
    Partial-bug: #1413102

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: stable/newton
Review: https://review.openstack.org/398930

[OpenStack Infra (hudson-openstack)]

Change abandoned by jichenjc (<email address hidden>) on branch: stable/newton
Review: https://review.openstack.org/398930

[Sean McGinnis (sean-mcginnis)]

I think the associated patched actual resolves this since using the default cinder.conf will no long result in deprecation warnings being emitted.