- 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).
Details
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.
Build has FAILED
Link to build: https://jenkins.mesosphere.com/service/jenkins/job/public-test-marathon-phabricator/729/
See console output for more information: https://jenkins.mesosphere.com/service/jenkins/job/public-test-marathon-phabricator/729/console
Build has FAILED
Link to build: https://jenkins.mesosphere.com/service/jenkins/job/public-test-marathon-phabricator/731/
See console output for more information: https://jenkins.mesosphere.com/service/jenkins/job/public-test-marathon-phabricator/731/console
Build has FAILED
Link to build: https://jenkins.mesosphere.com/service/jenkins/job/public-test-marathon-phabricator/732/
See console output for more information: https://jenkins.mesosphere.com/service/jenkins/job/public-test-marathon-phabricator/732/console
\\ ٩( ᐛ )و //
LGTM except you probably want to run scalariform on this to reconcile formatting.
Build is green https://jenkins.mesosphere.com/service/jenkins/job/public-test-marathon-phabricator/755/ for more details.
src/test/scala/mesosphere/marathon/state/RootGroupTest.scala | ||
---|---|---|
476 | why is this validation test removed? |
src/test/scala/mesosphere/marathon/state/RootGroupTest.scala | ||
---|---|---|
476 | oh good catch, i meant to only remove the proto one. |
Build has FAILED
Link to build: https://jenkins.mesosphere.com/service/jenkins/job/public-test-marathon-phabricator/757/
See console output for more information: https://jenkins.mesosphere.com/service/jenkins/job/public-test-marathon-phabricator/757/console
Build is green https://jenkins.mesosphere.com/service/jenkins/job/public-test-marathon-phabricator/758/ for more details.
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. |
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 |
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. |
Think we should remove the storage format from the version.