uccross / skyhookdm-ceph-cls

Skyhook Data Management: Storage and management of tabular data in Ceph.
https://www.skyhookdm.com
GNU Lesser General Public License v2.1
13 stars 9 forks source link

Upgrade repo to Ceph-Nautilus #13

Closed jlefevre closed 4 years ago

jlefevre commented 4 years ago

Move from Luminous to Nautilus. Although Octopus stable version was just released we will wait on this for more user adoption.

ivotron commented 4 years ago

current WIP for this is on branch https://github.com/uccross/skyhookdm-ceph-cls/tree/master (specifically this commit).

So far, we only moved the parent repo from luminous to nautilus. Same instructions and codebase as the main luminous branch of this CLS repo, the only changes are in the README and CI pipeline as seen here: https://github.com/uccross/skyhookdm-ceph-cls/compare/nautilus

The above causes the following error: https://travis-ci.com/github/uccross/skyhookdm-ceph-cls/builds/168964597

ivotron commented 4 years ago

From @jlefevre :

it seems they just changed the signature of encode to explicitly specify the type as 3rd arg (e.g., uint64_t) whereas it used to be implicitly inferred. i'll work on that error first, seems reasonable to fix.

ivotron commented 4 years ago

this issue is blocking the progress of the Hello World project. At this point we cannot deploy rook unless we're able to run on Nautilus.

In addition, new features/fixes are still being developed on the luminous branch (e.g. #41), which means there's technical debt being accumulated, since those changes will need to be ported to nautilus as well since now they're on separate branches.

Is there anything we can do to help with this?

jlefevre commented 4 years ago

Most of the changes are done now, debugging.

jlefevre commented 4 years ago

I merged PR #45 , all working now in popper env, so this should unblock Jayjeet. I'm leaving this open for now since there is still a problem with vstart, which we also use for development, so will continue working on that part. Please add comment here if still blocking.