search

mesosphere / marathon

Deploy and manage containers (including Docker) on top of Apache Mesos at scale.
https://mesosphere.github.io/marathon/
Apache License 2.0
4.07k stars 845 forks source link

/v2/groups PUT doesn't honor previous instances value #659

Closed sttts closed 9 years ago

sttts commented 10 years ago

Create an application group and scale an app to some value other than 1. Use the same json definition, but leave out the "instances" lines. Use a PUT on /v2/groups/ to update the group. Marathon scales down to 1.

sttts commented 10 years ago

I found a similar, probably related issue:

Change a health check and update an app with PUT. The health check (here an COMMAND health check) is attached to the mesos task, but marathon will not restart the task and therefore the health check will not be successful.

ConnorDoyle commented 10 years ago

This is actually the intended behavior for this method. PUTs are only PATCH-like for apps for the time being. In the future, proper support for PATCH is planned (see #203 #223 #251) and the current PUT behavior will go away. If those plans change, we will update this PUT to behave like the current PUT /v2/apps/{appId}

gkleiman commented 9 years ago

@stts: I couldn't reproduce what you describe and this issue is pretty old. Please reopen this if you think the problems still exist.