HomeMesosphereNo notifications. 4 unresolved issues.

Delete Legacy Storage (#4852)
ClosedAll Users

Authored by jasongilanfarr on Dec 14 2016, 8:23 PM.

Details

Summary
  • We don't support legacy storage in 1.5+ anymore, so delete all of it.
  • We still use _some_ of MarathonState - I'd really like to get rid of that to and use a Proto typeclass or something (or just have the proto stuff live in the storage layer and not the classes).
Test Plan

sbt test

Diff Detail

Repository
rMARATHON marathon
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
jasongilanfarr retitled this revision from to Delete Legacy Storage (#4852).Dec 14 2016, 8:23 PM
jasongilanfarr updated this object.
jasongilanfarr edited the test plan for this revision. (Show Details)
timcharper accepted this revision.Dec 15 2016, 12:36 AM

\\ ٩( ᐛ )و //

LGTM except you probably want to run scalariform on this to reconcile formatting.

This revision is now accepted and ready to land.Dec 15 2016, 12:36 AM
jasongilanfarr updated this object.

-Rebased

jdef requested changes to this revision.Dec 15 2016, 9:02 PM
jdef added inline comments.
src/test/scala/mesosphere/marathon/state/RootGroupTest.scala
476

why is this validation test removed?

This revision now requires changes to proceed.Dec 15 2016, 9:02 PM
src/test/scala/mesosphere/marathon/state/RootGroupTest.scala
476

oh good catch, i meant to only remove the proto one.

  • Undo deletion of a test case
jdef accepted this revision.Dec 15 2016, 10:31 PM
This revision is now accepted and ready to land.Dec 15 2016, 10:31 PM
This revision was automatically updated to reflect the committed changes.

Even if already landed: what a great patch!

src/main/scala/mesosphere/marathon/storage/migration/Migration.scala
42

Think we should remove the storage format from the version.

unterstein added inline comments.Dec 16 2016, 11:03 AM
src/main/scala/mesosphere/marathon/storage/migration/Migration.scala
42

We should add a big block of documentation within the 1.5 changelog for this minimum requirement of 1.4 running before and that you can not directly upgrade from a version < 1.4

See https://github.com/mesosphere/marathon/issues/4864

src/main/scala/mesosphere/marathon/storage/migration/Migration.scala
42

We can't remove it unfortunately. If we read a storage format that has legacy, we can't migrate.

The 1.4 and 1.5 changelog will be very explicit about this.