* getting more updates in rocksdb
@ 2016-07-20 10:38 Willem Jan Withagen
2016-07-20 13:32 ` Gregory Farnum
0 siblings, 1 reply; 10+ messages in thread
From: Willem Jan Withagen @ 2016-07-20 10:38 UTC (permalink / raw)
To: Ceph Development
Hi,
I submitted a small patch to rocksdb for FreeBSD
https://github.com/facebook/rocksdb/commit/b726bf59619d43651838e03cce9f876b3560cbda
And was hopping that it would be tracked by my Ceph stuff as well.
But refetching things, it does not seem to be in the tree ceph cmake
fetches.
And I'm not fluent enough in git to actually check if my current
checkout with hash: 5647fa427c4eab87c0caf8bcbb91f5846a4623f9
should contain the above commit.
And if it is not in, would it be possible to get it in the module we
fetch for Ceph?
--WjW
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: getting more updates in rocksdb
2016-07-20 10:38 getting more updates in rocksdb Willem Jan Withagen
@ 2016-07-20 13:32 ` Gregory Farnum
2016-07-20 13:52 ` Willem Jan Withagen
0 siblings, 1 reply; 10+ messages in thread
From: Gregory Farnum @ 2016-07-20 13:32 UTC (permalink / raw)
To: Willem Jan Withagen; +Cc: Ceph Development
On Wed, Jul 20, 2016 at 3:38 AM, Willem Jan Withagen <wjw@digiware.nl> wrote:
>
> Hi,
>
> I submitted a small patch to rocksdb for FreeBSD
> https://github.com/facebook/rocksdb/commit/b726bf59619d43651838e03cce9f876b3560cbda
>
> And was hopping that it would be tracked by my Ceph stuff as well.
> But refetching things, it does not seem to be in the tree ceph cmake
> fetches.
>
> And I'm not fluent enough in git to actually check if my current
> checkout with hash: 5647fa427c4eab87c0caf8bcbb91f5846a4623f9
> should contain the above commit.
Use "git log" on both trees and search for the hashes? :)
Anyway, the Ceph submodule does not auto-update (git submodules don't
support that AFAIK). You can go into the rocksdb folder and try
checking out the latest upstream master, then committing the new hash
as a commit in the Ceph git repo (at least, once your commit is
merged; not sure if it is yet).
-Greg
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: getting more updates in rocksdb
2016-07-20 13:32 ` Gregory Farnum
@ 2016-07-20 13:52 ` Willem Jan Withagen
2016-07-20 14:23 ` Josh Durgin
0 siblings, 1 reply; 10+ messages in thread
From: Willem Jan Withagen @ 2016-07-20 13:52 UTC (permalink / raw)
To: Gregory Farnum; +Cc: Ceph Development
On 20-7-2016 15:32, Gregory Farnum wrote:
> On Wed, Jul 20, 2016 at 3:38 AM, Willem Jan Withagen <wjw@digiware.nl> wrote:
>>
>> Hi,
>>
>> I submitted a small patch to rocksdb for FreeBSD
>> https://github.com/facebook/rocksdb/commit/b726bf59619d43651838e03cce9f876b3560cbda
>>
>> And was hopping that it would be tracked by my Ceph stuff as well.
>> But refetching things, it does not seem to be in the tree ceph cmake
>> fetches.
>>
>> And I'm not fluent enough in git to actually check if my current
>> checkout with hash: 5647fa427c4eab87c0caf8bcbb91f5846a4623f9
>> should contain the above commit.
>
> Use "git log" on both trees and search for the hashes? :)
>
> Anyway, the Ceph submodule does not auto-update (git submodules don't
> support that AFAIK). You can go into the rocksdb folder and try
> checking out the latest upstream master, then committing the new hash
> as a commit in the Ceph git repo (at least, once your commit is
> merged; not sure if it is yet).
Although I (sort of) understand what you are saying, I'm not able to
reproduce the actual commands to do this. Comparing the logs I
understand. First just have to clone rocksdb.
And as far as I know the commit I'm looking for is in the current
rocksdb tree. I got commit about 22 days ago, and the current hash seems
te be from just a bit before that.
--WjW
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: getting more updates in rocksdb
2016-07-20 13:52 ` Willem Jan Withagen
@ 2016-07-20 14:23 ` Josh Durgin
2016-07-21 7:15 ` Tone Zhang
0 siblings, 1 reply; 10+ messages in thread
From: Josh Durgin @ 2016-07-20 14:23 UTC (permalink / raw)
To: Willem Jan Withagen, Gregory Farnum; +Cc: Ceph Development
On 07/20/2016 06:52 AM, Willem Jan Withagen wrote:
> On 20-7-2016 15:32, Gregory Farnum wrote:
>> On Wed, Jul 20, 2016 at 3:38 AM, Willem Jan Withagen <wjw@digiware.nl> wrote:
>>>
>>> Hi,
>>>
>>> I submitted a small patch to rocksdb for FreeBSD
>>> https://github.com/facebook/rocksdb/commit/b726bf59619d43651838e03cce9f876b3560cbda
>>>
>>> And was hopping that it would be tracked by my Ceph stuff as well.
>>> But refetching things, it does not seem to be in the tree ceph cmake
>>> fetches.
>>>
>>> And I'm not fluent enough in git to actually check if my current
>>> checkout with hash: 5647fa427c4eab87c0caf8bcbb91f5846a4623f9
>>> should contain the above commit.
>>
>> Use "git log" on both trees and search for the hashes? :)
>>
>> Anyway, the Ceph submodule does not auto-update (git submodules don't
>> support that AFAIK). You can go into the rocksdb folder and try
>> checking out the latest upstream master, then committing the new hash
>> as a commit in the Ceph git repo (at least, once your commit is
>> merged; not sure if it is yet).
>
> Although I (sort of) understand what you are saying, I'm not able to
> reproduce the actual commands to do this. Comparing the logs I
> understand. First just have to clone rocksdb.
>
> And as far as I know the commit I'm looking for is in the current
> rocksdb tree. I got commit about 22 days ago, and the current hash seems
> te be from just a bit before that.
To clarify, to update the rocksdb submodule, you'd need to first update
the ceph fork. (cat .gitmodules shows src/rocksdb comes from
https://github.com/ceph/rocksdb).
You can make a PR to ceph/rocksdb with the latest upstream code from
within your submodule checkout (src/rocksdb) which functions like a
normal git repo itself. You could e.g. 'git pull
https://github.com/facebook/rocksdb' to get the latest code there.
Then once that's merged into ceph/rocksdb, make a PR to ceph (from the
root of your ceph checkout 'git commit src/rocksdb' will update the
sha1 the rocksdb submodule references).
Josh
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: getting more updates in rocksdb
2016-07-20 14:23 ` Josh Durgin
@ 2016-07-21 7:15 ` Tone Zhang
2016-07-21 8:15 ` Willem Jan Withagen
0 siblings, 1 reply; 10+ messages in thread
From: Tone Zhang @ 2016-07-21 7:15 UTC (permalink / raw)
To: Josh Durgin, Willem Jan Withagen, Gregory Farnum,
Ceph Development
Hi,
I met the same issue, and I run "git submodule update --init
--recursive --remote" to fetch the latest commit from
facebook/rocksdb.
In fact, there is one similar PR has been raised, it is
https://github.com/ceph/rocksdb/pull/10, it is in review stage.
In fact, the PR is raised to fix the rocksdb compiling error in
AArm64, the reason the ceph/rocksdb is out of date.
Sure we should fetch all the delta from facebook/rocksdb to ceph/rocksdb.
Thanks.
Br,
Tone
On 20 July 2016 at 22:23, Josh Durgin <jdurgin@redhat.com> wrote:
> On 07/20/2016 06:52 AM, Willem Jan Withagen wrote:
>>
>> On 20-7-2016 15:32, Gregory Farnum wrote:
>>>
>>> On Wed, Jul 20, 2016 at 3:38 AM, Willem Jan Withagen <wjw@digiware.nl>
>>> wrote:
>>>>
>>>>
>>>> Hi,
>>>>
>>>> I submitted a small patch to rocksdb for FreeBSD
>>>>
>>>> https://github.com/facebook/rocksdb/commit/b726bf59619d43651838e03cce9f876b3560cbda
>>>>
>>>> And was hopping that it would be tracked by my Ceph stuff as well.
>>>> But refetching things, it does not seem to be in the tree ceph cmake
>>>> fetches.
>>>>
>>>> And I'm not fluent enough in git to actually check if my current
>>>> checkout with hash: 5647fa427c4eab87c0caf8bcbb91f5846a4623f9
>>>> should contain the above commit.
>>>
>>>
>>> Use "git log" on both trees and search for the hashes? :)
>>>
>>> Anyway, the Ceph submodule does not auto-update (git submodules don't
>>> support that AFAIK). You can go into the rocksdb folder and try
>>> checking out the latest upstream master, then committing the new hash
>>> as a commit in the Ceph git repo (at least, once your commit is
>>> merged; not sure if it is yet).
>>
>>
>> Although I (sort of) understand what you are saying, I'm not able to
>> reproduce the actual commands to do this. Comparing the logs I
>> understand. First just have to clone rocksdb.
>>
>> And as far as I know the commit I'm looking for is in the current
>> rocksdb tree. I got commit about 22 days ago, and the current hash seems
>> te be from just a bit before that.
>
>
> To clarify, to update the rocksdb submodule, you'd need to first update
> the ceph fork. (cat .gitmodules shows src/rocksdb comes from
> https://github.com/ceph/rocksdb).
>
> You can make a PR to ceph/rocksdb with the latest upstream code from
> within your submodule checkout (src/rocksdb) which functions like a
> normal git repo itself. You could e.g. 'git pull
> https://github.com/facebook/rocksdb' to get the latest code there.
>
> Then once that's merged into ceph/rocksdb, make a PR to ceph (from the
> root of your ceph checkout 'git commit src/rocksdb' will update the
> sha1 the rocksdb submodule references).
>
> Josh
>
>
> --
> To unsubscribe from this list: send the line "unsubscribe ceph-devel" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line "unsubscribe ceph-devel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: getting more updates in rocksdb
2016-07-21 7:15 ` Tone Zhang
@ 2016-07-21 8:15 ` Willem Jan Withagen
2016-07-21 8:59 ` Tone Zhang
0 siblings, 1 reply; 10+ messages in thread
From: Willem Jan Withagen @ 2016-07-21 8:15 UTC (permalink / raw)
To: Tone Zhang, Josh Durgin, Gregory Farnum, Ceph Development
On 21-7-2016 09:15, Tone Zhang wrote:
> Hi,
>
> I met the same issue, and I run "git submodule update --init
> --recursive --remote" to fetch the latest commit from
> facebook/rocksdb.
> In fact, there is one similar PR has been raised, it is
> https://github.com/ceph/rocksdb/pull/10, it is in review stage.
> In fact, the PR is raised to fix the rocksdb compiling error in
> AArm64, the reason the ceph/rocksdb is out of date.
> Sure we should fetch all the delta from facebook/rocksdb to ceph/rocksdb.
> Thanks.
>
> Br,
> Tone
>
> On 20 July 2016 at 22:23, Josh Durgin <jdurgin@redhat.com> wrote:
>> On 07/20/2016 06:52 AM, Willem Jan Withagen wrote:
>>>
>>> On 20-7-2016 15:32, Gregory Farnum wrote:
>>>>
>>>> On Wed, Jul 20, 2016 at 3:38 AM, Willem Jan Withagen <wjw@digiware.nl>
>>>> wrote:
>>>>>
>>>>>
>>>>> Hi,
>>>>>
>>>>> I submitted a small patch to rocksdb for FreeBSD
>>>>>
>>>>> https://github.com/facebook/rocksdb/commit/b726bf59619d43651838e03cce9f876b3560cbda
>>>>>
>>>>> And was hopping that it would be tracked by my Ceph stuff as well.
>>>>> But refetching things, it does not seem to be in the tree ceph cmake
>>>>> fetches.
>>>>>
>>>>> And I'm not fluent enough in git to actually check if my current
>>>>> checkout with hash: 5647fa427c4eab87c0caf8bcbb91f5846a4623f9
>>>>> should contain the above commit.
>>>>
>>>>
>>>> Use "git log" on both trees and search for the hashes? :)
>>>>
>>>> Anyway, the Ceph submodule does not auto-update (git submodules don't
>>>> support that AFAIK). You can go into the rocksdb folder and try
>>>> checking out the latest upstream master, then committing the new hash
>>>> as a commit in the Ceph git repo (at least, once your commit is
>>>> merged; not sure if it is yet).
>>>
>>>
>>> Although I (sort of) understand what you are saying, I'm not able to
>>> reproduce the actual commands to do this. Comparing the logs I
>>> understand. First just have to clone rocksdb.
>>>
>>> And as far as I know the commit I'm looking for is in the current
>>> rocksdb tree. I got commit about 22 days ago, and the current hash seems
>>> te be from just a bit before that.
>>
>>
>> To clarify, to update the rocksdb submodule, you'd need to first update
>> the ceph fork. (cat .gitmodules shows src/rocksdb comes from
>> https://github.com/ceph/rocksdb).
>>
>> You can make a PR to ceph/rocksdb with the latest upstream code from
>> within your submodule checkout (src/rocksdb) which functions like a
>> normal git repo itself. You could e.g. 'git pull
>> https://github.com/facebook/rocksdb' to get the latest code there.
>>
>> Then once that's merged into ceph/rocksdb, make a PR to ceph (from the
>> root of your ceph checkout 'git commit src/rocksdb' will update the
>> sha1 the rocksdb submodule references).
'mmmm,
If I look in ceph/rocksdb at github thru the filebrowser, the commit I
made is actually in this tree. So I guess I should be getting it.
So I guess I'm going to start recloning my sources...
--WjW
--
To unsubscribe from this list: send the line "unsubscribe ceph-devel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: getting more updates in rocksdb
2016-07-21 8:15 ` Willem Jan Withagen
@ 2016-07-21 8:59 ` Tone Zhang
2016-07-21 9:16 ` Willem Jan Withagen
0 siblings, 1 reply; 10+ messages in thread
From: Tone Zhang @ 2016-07-21 8:59 UTC (permalink / raw)
To: Willem Jan Withagen; +Cc: Josh Durgin, Gregory Farnum, Ceph Development
Hi,
Could you please share me that after reclone the ceph code you can
fetch the latest code of ceph/rocksdb or not? I mean the same as the
upstream facebook/rocksdb.
I have tried, the HEAD of ceph/rocksdb is still
5647fa427c4eab87c0caf8bcbb91f5846a4623f9. In my side, the code cloned
to local is different from the files saw in github filebrowser.
Thanks.
Br,
Tone
On 21 July 2016 at 16:15, Willem Jan Withagen <wjw@digiware.nl> wrote:
> On 21-7-2016 09:15, Tone Zhang wrote:
>> Hi,
>>
>> I met the same issue, and I run "git submodule update --init
>> --recursive --remote" to fetch the latest commit from
>> facebook/rocksdb.
>> In fact, there is one similar PR has been raised, it is
>> https://github.com/ceph/rocksdb/pull/10, it is in review stage.
>> In fact, the PR is raised to fix the rocksdb compiling error in
>> AArm64, the reason the ceph/rocksdb is out of date.
>> Sure we should fetch all the delta from facebook/rocksdb to ceph/rocksdb.
>> Thanks.
>>
>> Br,
>> Tone
>>
>> On 20 July 2016 at 22:23, Josh Durgin <jdurgin@redhat.com> wrote:
>>> On 07/20/2016 06:52 AM, Willem Jan Withagen wrote:
>>>>
>>>> On 20-7-2016 15:32, Gregory Farnum wrote:
>>>>>
>>>>> On Wed, Jul 20, 2016 at 3:38 AM, Willem Jan Withagen <wjw@digiware.nl>
>>>>> wrote:
>>>>>>
>>>>>>
>>>>>> Hi,
>>>>>>
>>>>>> I submitted a small patch to rocksdb for FreeBSD
>>>>>>
>>>>>> https://github.com/facebook/rocksdb/commit/b726bf59619d43651838e03cce9f876b3560cbda
>>>>>>
>>>>>> And was hopping that it would be tracked by my Ceph stuff as well.
>>>>>> But refetching things, it does not seem to be in the tree ceph cmake
>>>>>> fetches.
>>>>>>
>>>>>> And I'm not fluent enough in git to actually check if my current
>>>>>> checkout with hash: 5647fa427c4eab87c0caf8bcbb91f5846a4623f9
>>>>>> should contain the above commit.
>>>>>
>>>>>
>>>>> Use "git log" on both trees and search for the hashes? :)
>>>>>
>>>>> Anyway, the Ceph submodule does not auto-update (git submodules don't
>>>>> support that AFAIK). You can go into the rocksdb folder and try
>>>>> checking out the latest upstream master, then committing the new hash
>>>>> as a commit in the Ceph git repo (at least, once your commit is
>>>>> merged; not sure if it is yet).
>>>>
>>>>
>>>> Although I (sort of) understand what you are saying, I'm not able to
>>>> reproduce the actual commands to do this. Comparing the logs I
>>>> understand. First just have to clone rocksdb.
>>>>
>>>> And as far as I know the commit I'm looking for is in the current
>>>> rocksdb tree. I got commit about 22 days ago, and the current hash seems
>>>> te be from just a bit before that.
>>>
>>>
>>> To clarify, to update the rocksdb submodule, you'd need to first update
>>> the ceph fork. (cat .gitmodules shows src/rocksdb comes from
>>> https://github.com/ceph/rocksdb).
>>>
>>> You can make a PR to ceph/rocksdb with the latest upstream code from
>>> within your submodule checkout (src/rocksdb) which functions like a
>>> normal git repo itself. You could e.g. 'git pull
>>> https://github.com/facebook/rocksdb' to get the latest code there.
>>>
>>> Then once that's merged into ceph/rocksdb, make a PR to ceph (from the
>>> root of your ceph checkout 'git commit src/rocksdb' will update the
>>> sha1 the rocksdb submodule references).
>
> 'mmmm,
>
> If I look in ceph/rocksdb at github thru the filebrowser, the commit I
> made is actually in this tree. So I guess I should be getting it.
> So I guess I'm going to start recloning my sources...
>
> --WjW
>
>
--
To unsubscribe from this list: send the line "unsubscribe ceph-devel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: getting more updates in rocksdb
2016-07-21 8:59 ` Tone Zhang
@ 2016-07-21 9:16 ` Willem Jan Withagen
2016-07-21 9:37 ` Tone Zhang
0 siblings, 1 reply; 10+ messages in thread
From: Willem Jan Withagen @ 2016-07-21 9:16 UTC (permalink / raw)
To: Tone Zhang; +Cc: Josh Durgin, Gregory Farnum, Ceph Development
On 21-7-2016 10:59, Tone Zhang wrote:
> Hi,
>
> Could you please share me that after reclone the ceph code you can
> fetch the latest code of ceph/rocksdb or not? I mean the same as the
> upstream facebook/rocksdb.
> I have tried, the HEAD of ceph/rocksdb is still
> 5647fa427c4eab87c0caf8bcbb91f5846a4623f9. In my side, the code cloned
> to local is different from the files saw in github filebrowser.
Right.
That is exactly my problem as well.
https://github.com/ceph/rocksdb claims 6797e6f
Which is 17 commits behind on the master version
So what I get from the other mails, is to actually get that forward, a
commit for the new tree needs to get into ceph/ceph.
Which is what Josh and Greg are writting about.
And there is a lingering PULL from Kefu that is actually doing something
like that.
--WjW
> Thanks.
>
> Br,
> Tone
>
> On 21 July 2016 at 16:15, Willem Jan Withagen <wjw@digiware.nl> wrote:
>> On 21-7-2016 09:15, Tone Zhang wrote:
>>> Hi,
>>>
>>> I met the same issue, and I run "git submodule update --init
>>> --recursive --remote" to fetch the latest commit from
>>> facebook/rocksdb.
>>> In fact, there is one similar PR has been raised, it is
>>> https://github.com/ceph/rocksdb/pull/10, it is in review stage.
>>> In fact, the PR is raised to fix the rocksdb compiling error in
>>> AArm64, the reason the ceph/rocksdb is out of date.
>>> Sure we should fetch all the delta from facebook/rocksdb to ceph/rocksdb.
>>> Thanks.
>>>
>>> Br,
>>> Tone
>>>
>>> On 20 July 2016 at 22:23, Josh Durgin <jdurgin@redhat.com> wrote:
>>>> On 07/20/2016 06:52 AM, Willem Jan Withagen wrote:
>>>>>
>>>>> On 20-7-2016 15:32, Gregory Farnum wrote:
>>>>>>
>>>>>> On Wed, Jul 20, 2016 at 3:38 AM, Willem Jan Withagen <wjw@digiware.nl>
>>>>>> wrote:
>>>>>>>
>>>>>>>
>>>>>>> Hi,
>>>>>>>
>>>>>>> I submitted a small patch to rocksdb for FreeBSD
>>>>>>>
>>>>>>> https://github.com/facebook/rocksdb/commit/b726bf59619d43651838e03cce9f876b3560cbda
>>>>>>>
>>>>>>> And was hopping that it would be tracked by my Ceph stuff as well.
>>>>>>> But refetching things, it does not seem to be in the tree ceph cmake
>>>>>>> fetches.
>>>>>>>
>>>>>>> And I'm not fluent enough in git to actually check if my current
>>>>>>> checkout with hash: 5647fa427c4eab87c0caf8bcbb91f5846a4623f9
>>>>>>> should contain the above commit.
>>>>>>
>>>>>>
>>>>>> Use "git log" on both trees and search for the hashes? :)
>>>>>>
>>>>>> Anyway, the Ceph submodule does not auto-update (git submodules don't
>>>>>> support that AFAIK). You can go into the rocksdb folder and try
>>>>>> checking out the latest upstream master, then committing the new hash
>>>>>> as a commit in the Ceph git repo (at least, once your commit is
>>>>>> merged; not sure if it is yet).
>>>>>
>>>>>
>>>>> Although I (sort of) understand what you are saying, I'm not able to
>>>>> reproduce the actual commands to do this. Comparing the logs I
>>>>> understand. First just have to clone rocksdb.
>>>>>
>>>>> And as far as I know the commit I'm looking for is in the current
>>>>> rocksdb tree. I got commit about 22 days ago, and the current hash seems
>>>>> te be from just a bit before that.
>>>>
>>>>
>>>> To clarify, to update the rocksdb submodule, you'd need to first update
>>>> the ceph fork. (cat .gitmodules shows src/rocksdb comes from
>>>> https://github.com/ceph/rocksdb).
>>>>
>>>> You can make a PR to ceph/rocksdb with the latest upstream code from
>>>> within your submodule checkout (src/rocksdb) which functions like a
>>>> normal git repo itself. You could e.g. 'git pull
>>>> https://github.com/facebook/rocksdb' to get the latest code there.
>>>>
>>>> Then once that's merged into ceph/rocksdb, make a PR to ceph (from the
>>>> root of your ceph checkout 'git commit src/rocksdb' will update the
>>>> sha1 the rocksdb submodule references).
>>
>> 'mmmm,
>>
>> If I look in ceph/rocksdb at github thru the filebrowser, the commit I
>> made is actually in this tree. So I guess I should be getting it.
>> So I guess I'm going to start recloning my sources...
>>
>> --WjW
>>
>>
--
To unsubscribe from this list: send the line "unsubscribe ceph-devel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: getting more updates in rocksdb
2016-07-21 9:16 ` Willem Jan Withagen
@ 2016-07-21 9:37 ` Tone Zhang
2016-07-22 20:05 ` Dan Mick
0 siblings, 1 reply; 10+ messages in thread
From: Tone Zhang @ 2016-07-21 9:37 UTC (permalink / raw)
To: Willem Jan Withagen; +Cc: Josh Durgin, Gregory Farnum, Ceph Development
Hi,
Thanks a lot!
So synchronise the submodule code is a little pain. We have to
synchronise the code manually when we found we have been behind the
upstream so much. :)
Br,
Tone
On 21 July 2016 at 17:16, Willem Jan Withagen <wjw@digiware.nl> wrote:
> On 21-7-2016 10:59, Tone Zhang wrote:
>> Hi,
>>
>> Could you please share me that after reclone the ceph code you can
>> fetch the latest code of ceph/rocksdb or not? I mean the same as the
>> upstream facebook/rocksdb.
>> I have tried, the HEAD of ceph/rocksdb is still
>> 5647fa427c4eab87c0caf8bcbb91f5846a4623f9. In my side, the code cloned
>> to local is different from the files saw in github filebrowser.
>
> Right.
>
> That is exactly my problem as well.
> https://github.com/ceph/rocksdb claims 6797e6f
> Which is 17 commits behind on the master version
>
> So what I get from the other mails, is to actually get that forward, a
> commit for the new tree needs to get into ceph/ceph.
> Which is what Josh and Greg are writting about.
> And there is a lingering PULL from Kefu that is actually doing something
> like that.
>
> --WjW
>
>
>> Thanks.
>>
>> Br,
>> Tone
>>
>> On 21 July 2016 at 16:15, Willem Jan Withagen <wjw@digiware.nl> wrote:
>>> On 21-7-2016 09:15, Tone Zhang wrote:
>>>> Hi,
>>>>
>>>> I met the same issue, and I run "git submodule update --init
>>>> --recursive --remote" to fetch the latest commit from
>>>> facebook/rocksdb.
>>>> In fact, there is one similar PR has been raised, it is
>>>> https://github.com/ceph/rocksdb/pull/10, it is in review stage.
>>>> In fact, the PR is raised to fix the rocksdb compiling error in
>>>> AArm64, the reason the ceph/rocksdb is out of date.
>>>> Sure we should fetch all the delta from facebook/rocksdb to ceph/rocksdb.
>>>> Thanks.
>>>>
>>>> Br,
>>>> Tone
>>>>
>>>> On 20 July 2016 at 22:23, Josh Durgin <jdurgin@redhat.com> wrote:
>>>>> On 07/20/2016 06:52 AM, Willem Jan Withagen wrote:
>>>>>>
>>>>>> On 20-7-2016 15:32, Gregory Farnum wrote:
>>>>>>>
>>>>>>> On Wed, Jul 20, 2016 at 3:38 AM, Willem Jan Withagen <wjw@digiware.nl>
>>>>>>> wrote:
>>>>>>>>
>>>>>>>>
>>>>>>>> Hi,
>>>>>>>>
>>>>>>>> I submitted a small patch to rocksdb for FreeBSD
>>>>>>>>
>>>>>>>> https://github.com/facebook/rocksdb/commit/b726bf59619d43651838e03cce9f876b3560cbda
>>>>>>>>
>>>>>>>> And was hopping that it would be tracked by my Ceph stuff as well.
>>>>>>>> But refetching things, it does not seem to be in the tree ceph cmake
>>>>>>>> fetches.
>>>>>>>>
>>>>>>>> And I'm not fluent enough in git to actually check if my current
>>>>>>>> checkout with hash: 5647fa427c4eab87c0caf8bcbb91f5846a4623f9
>>>>>>>> should contain the above commit.
>>>>>>>
>>>>>>>
>>>>>>> Use "git log" on both trees and search for the hashes? :)
>>>>>>>
>>>>>>> Anyway, the Ceph submodule does not auto-update (git submodules don't
>>>>>>> support that AFAIK). You can go into the rocksdb folder and try
>>>>>>> checking out the latest upstream master, then committing the new hash
>>>>>>> as a commit in the Ceph git repo (at least, once your commit is
>>>>>>> merged; not sure if it is yet).
>>>>>>
>>>>>>
>>>>>> Although I (sort of) understand what you are saying, I'm not able to
>>>>>> reproduce the actual commands to do this. Comparing the logs I
>>>>>> understand. First just have to clone rocksdb.
>>>>>>
>>>>>> And as far as I know the commit I'm looking for is in the current
>>>>>> rocksdb tree. I got commit about 22 days ago, and the current hash seems
>>>>>> te be from just a bit before that.
>>>>>
>>>>>
>>>>> To clarify, to update the rocksdb submodule, you'd need to first update
>>>>> the ceph fork. (cat .gitmodules shows src/rocksdb comes from
>>>>> https://github.com/ceph/rocksdb).
>>>>>
>>>>> You can make a PR to ceph/rocksdb with the latest upstream code from
>>>>> within your submodule checkout (src/rocksdb) which functions like a
>>>>> normal git repo itself. You could e.g. 'git pull
>>>>> https://github.com/facebook/rocksdb' to get the latest code there.
>>>>>
>>>>> Then once that's merged into ceph/rocksdb, make a PR to ceph (from the
>>>>> root of your ceph checkout 'git commit src/rocksdb' will update the
>>>>> sha1 the rocksdb submodule references).
>>>
>>> 'mmmm,
>>>
>>> If I look in ceph/rocksdb at github thru the filebrowser, the commit I
>>> made is actually in this tree. So I guess I should be getting it.
>>> So I guess I'm going to start recloning my sources...
>>>
>>> --WjW
>>>
>>>
>
--
To unsubscribe from this list: send the line "unsubscribe ceph-devel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: getting more updates in rocksdb
2016-07-21 9:37 ` Tone Zhang
@ 2016-07-22 20:05 ` Dan Mick
0 siblings, 0 replies; 10+ messages in thread
From: Dan Mick @ 2016-07-22 20:05 UTC (permalink / raw)
To: Tone Zhang, Willem Jan Withagen
Cc: Josh Durgin, Gregory Farnum, Ceph Development
On 07/21/2016 02:37 AM, Tone Zhang wrote:
> Hi,
>
> Thanks a lot!
> So synchronise the submodule code is a little pain. We have to
> synchronise the code manually when we found we have been behind the
> upstream so much. :)
I also now want to unbreak the aarch64 build, so will be trying to usher
this along.
^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2016-07-22 20:05 UTC | newest]
Thread overview: 10+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-07-20 10:38 getting more updates in rocksdb Willem Jan Withagen
2016-07-20 13:32 ` Gregory Farnum
2016-07-20 13:52 ` Willem Jan Withagen
2016-07-20 14:23 ` Josh Durgin
2016-07-21 7:15 ` Tone Zhang
2016-07-21 8:15 ` Willem Jan Withagen
2016-07-21 8:59 ` Tone Zhang
2016-07-21 9:16 ` Willem Jan Withagen
2016-07-21 9:37 ` Tone Zhang
2016-07-22 20:05 ` Dan Mick
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.