search

ceph / ceph-cookbook

Chef cookbooks for Ceph
Apache License 2.0
100 stars 108 forks source link

Release 0.9.0 #210

Closed guilhem closed 8 years ago

jjasghar commented 8 years ago

Seems reasonable :+1:

tas50 commented 8 years ago

0.8.2 would be nice, but I think we actually have a 0.9.0 here since we've added a bunch of new functionality.

guilhem commented 8 years ago

Agreed with you @tas50 civetweb is enough to bump minor :)

guilhem commented 8 years ago

@cloudm2 please don't do some crapy stuff on this repository!

This PR have to be merged after #211 (as said here https://github.com/ceph/ceph-cookbook/pull/211#issuecomment-140737631 ). Release 0.9.0 (Commit 0ea7f15f20b34429daa326d25f4edf39ab6049a5 ) doesn't include good fixes from @tas50 .

hanscj1 commented 8 years ago

Your stuff is just metadata stuff so push another one.

hanscj1 commented 8 years ago

You should have retracted and reissued another PR if you want something in order. Can not have special instructions in comments on do this before that else whatever.

guilhem commented 8 years ago

@cloudm2 that's why, IMHO, you can't be maintainer of this cookbook (but contributor absolutely).

Being maintainer require vision, communication and practice. You don't seems to have this 3.

With all this previous exchange, any Chef regular user will see what I'm talking about. (By the way, I keep just metadata as a good Chef joke ;) )

hanscj1 commented 8 years ago

It's unfortunate you feel this way. The repo will be making positive changes moving forward that will bring it up to speed (it's really far behind) and make it enterprise grade. I hope in the future you can contribute real code changes that help move this forward. As far as the supermarket is concerned, please keep it up to date with with the coming changes. Thanks.