* merging pr 5311 was a noop
@ 2015-07-30 20:15 Loic Dachary
0 siblings, 0 replies; only message in thread
From: Loic Dachary @ 2015-07-30 20:15 UTC (permalink / raw)
To: Yan, Zheng; +Cc: Ceph Development
[-- Attachment #1: Type: text/plain, Size: 694 bytes --]
Hi Yan,
This actually is an interesting case :-) Yesterday you merged https://github.com/ceph/ceph/pull/5311 into hammer-12088. But you did it after hammer-12088 was merged into hammer ( https://github.com/ceph/ceph/pull/5222#issuecomment-125881146 ). Therefore the commit from pr 5311 never actually made it to hammer.
Fortunately it's easy to sort out: you just need to review https://github.com/ceph/ceph/pull/5427 which has the same commit as https://github.com/ceph/ceph/pull/5311 (I just took the liberty of editing the commit message to add <> around the mail name and a reference to the commit that introduced the leak).
Cheers
--
Loïc Dachary, Artisan Logiciel Libre
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 198 bytes --]
^ permalink raw reply [flat|nested] only message in thread
only message in thread, other threads:[~2015-07-30 20:15 UTC | newest]
Thread overview: (only message) (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-07-30 20:15 merging pr 5311 was a noop Loic Dachary
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.