Skip to content
This repository has been archived by the owner on Jul 4, 2023. It is now read-only.

Debug marathonServiceRestart task #4

Closed
jeffreyolchovy opened this issue May 31, 2017 · 1 comment · Fixed by #5
Closed

Debug marathonServiceRestart task #4

jeffreyolchovy opened this issue May 31, 2017 · 1 comment · Fixed by #5

Comments

@jeffreyolchovy
Copy link
Contributor

jeffreyolchovy commented May 31, 2017

This issue is the formal documentation and verification of the following error when invoking the marathonServiceRestart task:

> marathonServiceRestart
[trace] Stack trace suppressed: run last project/*:marathonServiceRestart for the full output.
[error] (server/*:marathonServiceRestart) Unable to restart application project/version
[error] Total time: 1 s, completed May 31, 2017 5:51:14 AM
> last project/*:marathonServiceRestart
java.lang.RuntimeException: Unable to restart application project/version
	at scala.sys.package$.error(package.scala:27)
	at sbtmarathon.MarathonSettings$$anonfun$defaultSettings$7.apply(MarathonSettings.scala:130)
	at sbtmarathon.MarathonSettings$$anonfun$defaultSettings$7.apply(MarathonSettings.scala:117)
	at scala.Function1$$anonfun$compose$1.apply(Function1.scala:47)
	at sbt.$tilde$greater$$anonfun$$u2219$1.apply(TypeFunctions.scala:40)
	at sbt.std.Transform$$anon$4.work(System.scala:63)
	at sbt.Execute$$anonfun$submit$1$$anonfun$apply$1.apply(Execute.scala:228)
	at sbt.Execute$$anonfun$submit$1$$anonfun$apply$1.apply(Execute.scala:228)
	at sbt.ErrorHandling$.wideConvert(ErrorHandling.scala:17)
	at sbt.Execute.work(Execute.scala:237)
	at sbt.Execute$$anonfun$submit$1.apply(Execute.scala:228)
	at sbt.Execute$$anonfun$submit$1.apply(Execute.scala:228)
	at sbt.ConcurrentRestrictions$$anon$4$$anonfun$1.apply(ConcurrentRestrictions.scala:159)
	at sbt.CompletionService$$anon$2.call(CompletionService.scala:28)
	at java.util.concurrent.FutureTask.run(FutureTask.java:266)
	at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:511)
	at java.util.concurrent.FutureTask.run(FutureTask.java:266)
	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1142)
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:617)
	at java.lang.Thread.run(Thread.java:745)

As far as I can tell, this regression was introduced when migrating logic from the internal plugin to the oss plugin.

This issue effects the current main line. Originally reported @ #3 (comment) and verified against v0.1.1 and the HEAD of master.

@jeffreyolchovy
Copy link
Contributor Author

For app restarts, use:

POST /v2/apps/{appId}/restart: Rolling restart of all tasks of the given app

Currently, we are sending an empty POST request to the application instance URL, which, afaict, will just result with an unspecified failure. Our closed source plugin was executing this request correctly with the proper /restart terminated URL.

For more info, see https://mesosphere.github.io/marathon/docs/rest-api.html#user-content-post-v2appsappidrestart.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant