[ceph] Deprecate ceph logic in favor of upstream ceph cookbook

Bug #1409943 reported by Elliott Davis
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack + Chef
Fix Released
Critical
Elliott Davis

Bug Description

Currently all the openstack chef cookbooks are based on a template that is generated in openstack-common. The template provided by openstack-common is based on legacy configuration of ceph.

I would like to propose that remove all custom logic related to ceph in favor of using the upstream ceph chef cookbook.

First, remove all files and tests related to ceph in the common cookbook.
Next, We will need to add a dependency to the ceph cookbook in nova, cinder, and glance.
Finally, remove all current ceph logic from the aforementioned cookbooks and use the LWRP provided by the ceph cookbook.

Revision history for this message
JJ Asghar (d-jj) wrote :

Mark has put in an infra patch for this: https://review.openstack.org/#/c/147250/

Changed in openstack-chef:
status: New → Confirmed
importance: Undecided → Critical
assignee: nobody → Mark Vanderwiel (vanderwl)
summary: - Deprecate ceph logic in favor of upstream ceph cookbook
+ [ceph] Deprecate ceph logic in favor of upstream ceph cookbook
Changed in openstack-chef:
milestone: none → juno-stable
Changed in openstack-chef:
assignee: Mark Vanderwiel (vanderwl) → Elliott Davis (elliott-davis)
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to cookbook-openstack-block-storage (master)

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to openstack-chef-repo (master)

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

Changed in openstack-chef:
assignee: Elliott Davis (elliott-davis) → zhiwei (chenzhiwei)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to openstack-chef-repo (master)

Reviewed: https://review.openstack.org/148487
Committed: https://git.openstack.org/cgit/stackforge/openstack-chef-repo/commit/?id=9780e5690087a1173fa847646c402de6bb68052e
Submitter: Jenkins
Branch: master

commit 9780e5690087a1173fa847646c402de6bb68052e
Author: Chen Zhiwei <email address hidden>
Date: Tue Jan 20 18:07:44 2015 +0800

    Cleanup ceph cookbook, replace with upstream ceph cookbook

    The stackforge ceph cookbook was replaced by upstream ceph cookbook,
    so remove the roles defined in stackforge ceph cookbook and replace
    with upstream ceph cookbook in Berksfile.

    Change-Id: I2d12cb6c2c480a37ee1cfeaf5b76f068b6781160
    Partial-Bug: #1409943

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to cookbook-openstack-common (master)

Reviewed: https://review.openstack.org/146655
Committed: https://git.openstack.org/cgit/stackforge/cookbook-openstack-common/commit/?id=a78ac43af2448770dd05b1510e14c17e218ebca7
Submitter: Jenkins
Branch: master

commit a78ac43af2448770dd05b1510e14c17e218ebca7
Author: Elliott Davis <email address hidden>
Date: Sat Jan 3 22:10:58 2015 -0600

    Removed all ceph logic from this cookbook

    The assumptions made in this cookbook for ceph were incorrect.
    This commit removes all ceph logic from common and shifts the
    burden of ceph management to the upstream ceph cookbook.

    Partial-Bug: #1409943

    Change-Id: Id9af0f484675992a47401d00afb81886fc4074dc

Changed in openstack-chef:
assignee: zhiwei (chenzhiwei) → Elliott Davis (elliott-davis)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to cookbook-openstack-compute (master)

Reviewed: https://review.openstack.org/146745
Committed: https://git.openstack.org/cgit/stackforge/cookbook-openstack-compute/commit/?id=557afe098937ff4cd6661e7ad9104846081c0fdc
Submitter: Jenkins
Branch: master

commit 557afe098937ff4cd6661e7ad9104846081c0fdc
Author: Elliott Davis <email address hidden>
Date: Fri Dec 19 10:30:34 2014 -0600

    Add dependency on upstream ceph cookbook for better key management

    The upstream ceph cookbook already hooks into chef for environment information.
    This patch utilizes the client LWRP to create or add ceph keys for RBD support.
    This patch also changes some default attribute names for more sane organization

    Partial-Bug: #1409943

    Change-Id: Ia87e123ae2d88e3122b8d66d8ff144a804e6d8c4

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to cookbook-openstack-image (master)

Reviewed: https://review.openstack.org/143313
Committed: https://git.openstack.org/cgit/stackforge/cookbook-openstack-image/commit/?id=da05834a09e2b72210efe2e789c7811a14b6c441
Submitter: Jenkins
Branch: master

commit da05834a09e2b72210efe2e789c7811a14b6c441
Author: Elliott Davis <email address hidden>
Date: Sat Dec 20 11:14:14 2014 -0600

    Add dependency on upstream ceph cookbook for better key management

    The upstream ceph cookbook already hooks into chef for environment information.
    This patch utilizes the client LWRP to create or add ceph keys for RBD support.
    This patch also changes some default attribute names for more sane organization

    Partial-Bug: #1409943

    Change-Id: If37159524c237eec6ca1561e2979cd44e6dfac92

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to cookbook-openstack-block-storage (master)

Reviewed: https://review.openstack.org/147414
Committed: https://git.openstack.org/cgit/stackforge/cookbook-openstack-block-storage/commit/?id=f75d819c2fa9b92d1ed2f7b4fa7bf662c38ec843
Submitter: Jenkins
Branch: master

commit f75d819c2fa9b92d1ed2f7b4fa7bf662c38ec843
Author: Elliott Davis <email address hidden>
Date: Fri Dec 19 10:31:26 2014 -0600

    Add dependency on upstream ceph cookbook for better key management

    The upstream ceph cookbook already hooks into chef for environment information.
    This patch utilizes the client LWRP to create or add ceph keys for RBD support.
    This patch also changes some default attribute names for more sane organization

    Partial-Bug: #1409943

    Change-Id: Ibba6c568d4e4d00153061458b71593cd28714e60

Revision history for this message
Mark Vanderwiel (vanderwl) wrote :

Can this be closed now?

tags: added: block-storage common compute image repo
Revision history for this message
Christopher M Luciano (cmluciano) wrote :

+1 to closing

Changed in openstack-chef:
status: In Progress → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.