localVolumes Instance property is lost with Marathon restart #5165
Assignees
Comments
Closed
6 tasks
timcharper
pushed a commit
that referenced
this issue
Feb 16, 2017
Previously, all tasks were being read as LaunchedEphemeral task subtypes, and this was leading to any resident information about the task to be lost. This change fixes the issue, and adds round-trip serialization tests. Fixed #5165
timcharper
pushed a commit
that referenced
this issue
Feb 23, 2017
Summary: Previously, all tasks were being read as LaunchedEphemeral task subtypes, and this was leading to any resident information about the task to be lost. This change fixes the issue, and adds round-trip serialization tests. Fixed #5165 Backport of dad4e1f Test Plan: unit Reviewers: aquamatthias, meichstedt Subscribers: marathon-team Differential Revision: https://phabricator.mesosphere.com/D546
timcharper
pushed a commit
that referenced
this issue
Feb 23, 2017
Summary: Previously, all tasks were being read as LaunchedEphemeral task subtypes, and this was leading to any resident information about the task to be lost. This change fixes the issue, and adds round-trip serialization tests. Fixed #5165 Backport of dad4e1f Test Plan: unit Reviewers: aquamatthias, meichstedt Subscribers: marathon-team Differential Revision: https://phabricator.mesosphere.com/D546
This issue was closed.
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
It appears that
localVolumesis not persisted properly.{ "id": "/le-sleep", "cmd": "sleep 3600", "cpus": 1, "mem": 128, "disk": 0, "instances": 3, "constraints": [ [ "hostname", "UNIQUE" ] ], "container": { "type": "MESOS", "volumes": [ { "containerPath": "hi", "mode": "RW", "persistent": { "size": 16, "type": "root", "constraints": [] } } ] } }You'll see once that looks like this:
{ "localVolumes": [ { "runSpecId": "/le-sleep", "containerPath": "hi", "uuid": "79bb63c7-efe2-11e6-bcfd-469762eea0d6", "persistenceId": "le-sleep#hi#79bb63c7-efe2-11e6-bcfd-469762eea0d6" } ], "stagedAt": "2017-02-10T22:44:26.814Z", "state": "TASK_RUNNING", "ports": [ 31589 ], "startedAt": "2017-02-10T22:44:26.988Z", "version": "2017-02-10T22:44:26.429Z", "id": "le-sleep.79bb63c8-efe2-11e6-bcfd-469762eea0d6", "appId": "/le-sleep", "slaveId": "caf86dd5-62cc-47bd-82e8-1f4fef7002ea-S2", "host": "mesos-3.dev.vagrant" }Restart Marathon.
Curl the tasks endpoint again and the
localVolumesproperty will be missing.{ "stagedAt": "2017-02-10T22:44:26.814Z", "state": "TASK_RUNNING", "ports": [ 31589 ], "startedAt": "2017-02-10T22:44:26.988Z", "version": "2017-02-10T22:44:26.429Z", "id": "le-sleep.79bb63c8-efe2-11e6-bcfd-469762eea0d6", "appId": "/le-sleep", "slaveId": "caf86dd5-62cc-47bd-82e8-1f4fef7002ea-S2", "host": "mesos-3.dev.vagrant" }The text was updated successfully, but these errors were encountered: