Re: Hammer Backports integration branch population

[Abhishek L <abhishek.lekshmanan@gmail.com>]

[-- Attachment #1: Type: text/plain, Size: 2866 bytes --]


Loic Dachary writes:

> [cc'ing ceph-devel for Nathan and Xinxin to read]
>
> Hi Abhishek,
>
> On 13/06/2015 07:52, Abhishek L wrote:
>> Hi Loic,
>> 
>> I've populated the integration branch for my fork[1], with pending pull
>> requests in queue against hammer milestone. Can you push it against the
>> official repo (ceph-ci ?) so that the gitbuilders can start
>> building.
>
> $ git --no-pager log --oneline --merges ceph/hammer..theanalyst/hammer-backports
> 3677fd2 Merge 4657: DNM: common/admin_socket: close socket descriptor in destructor
> 6f89359 Merge 4667: ceph.spec: update OpenSUSE BuildRequires
> 7d50492 Merge 4687: utf8 and old gcc breakage on RHEL6.5
> 26c1f8a Merge 4776: ceph: cli interactive mode does not understand quotes
> b9b015a Merge 4846: missing man pages for ceph-create-keys, ceph-disk-*
> ae2a2cb Merge 4862: Bucket header is enclosed by quotes
> e82807f Merge 4868: Clock skew causes missing summary and confuses Calamari
> f0ae884 Merge 4875: librbd: aio calls may block
> c840979 Merge 4877: admin/build-doc: script fails silently under certain circumstances
> c5f51fe Merge 4883: ceph.spec.in: ceph-common subpackage def needs tweaking for SUSE/openSUSE
> 8e3662e Merge 4884: Keystone PKI token expiration is not enforced
> 5fdace1 Merge 4885: Object copy bug
> 29c563a Merge 4886: MDS is crashed (mds/CDir.cc: 1391: FAILED assert(!is_complete()))
> 43eb346 Merge 4889: OPT_INT option interprets 3221225472 as -1073741824, and crashes in Throttle::Throttle()
> 7768fb2 Merge 4891: crush: take crashes due to invalid arg
> a5885bc Merge 4892: ceph-disk: get_partition_type fails on /dev/cciss...
> 8b79288 Merge 4899: start_flush: filter out removed snaps before determining snapc's
> 59c4d78 Merge 4902: Fixes for rados ops with snaps
> b0e083e Merge 4936: mon crashes when "ceph osd tree 85 --format json"
>
> I see a DNM here: is it intentional ?
>
> http://ceph.com/gitbuilder.cgi is building https://github.com/ceph/ceph/tree/hammer-backports

Since I used a script to extract the PR's, I guess this commit got
pulled in as well. However looking at the particular PR [1], it looks
like DNM was only marked because it shouldn't be making it to the last
hammer, (and you marked it as DNM ;) So no harm done.

[1]: https://github.com/ceph/ceph/pull/4657

>
>> 
>> Also, for finding out PR's against a milestone, I've used the github's
>> api to get the relevant PR numbers, I've updated the wiki for populating
>> the integration branch[2] Let me know if this looks okay to you. 
>
> Very convenient, great addition the snippets collection !
>
> Cheers
>
>> [1] https://github.com/theanalyst/ceph/tree/hammer-backports
>> [2] http://tracker.ceph.com/projects/ceph-releases/wiki/HOWTO_populate_the_integration_branch
>> 
>> Cheers, 
>> Abhishek
>> 

-- 
Abhishek

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 489 bytes --]