Upgrade repo to Ceph-Nautilus about skyhookdm-ceph-cls HOT 5 CLOSED

Most of the changes are done now, debugging.

from skyhookdm-ceph-cls.

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.

from skyhookdm-ceph-cls.

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

from skyhookdm-ceph-cls.

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.

from skyhookdm-ceph-cls.

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?

from skyhookdm-ceph-cls.