rest api v2 app update ignores uri #170

Closed
cap opened this Issue Mar 6, 2014 · 2 comments

Comments

Projects
None yet
3 participants
@cap

cap commented Mar 6, 2014

uris appear in the AppUpdate class, but when it comes time to apply the update uris are ignored. I noticed this in an older version, and it looks like this behavior was propagated through a refactoring. Is there a good reason not to update uris this way?

https://github.com/mesosphere/marathon/blob/master/src/main/scala/mesosphere/marathon/api/v2/AppUpdate.scala#L52

@ssorallen

This comment has been minimized.

Show comment Hide comment
@ssorallen

ssorallen Mar 10, 2014

Contributor

It seems like an oversight that this is not already possible. @ConnorDoyle, should this be added?

Contributor

ssorallen commented Mar 10, 2014

It seems like an oversight that this is not already possible. @ConnorDoyle, should this be added?

@ConnorDoyle

This comment has been minimized.

Show comment Hide comment
@ConnorDoyle

ConnorDoyle Mar 11, 2014

Contributor

Absolutely. Thanks for reporting. Will fix this ASAP.

Contributor

ConnorDoyle commented Mar 11, 2014

Absolutely. Thanks for reporting. Will fix this ASAP.

@aquamatthias aquamatthias locked and limited conversation to collaborators Mar 27, 2017

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.