problem w/ app name that is a substring of another app name

tendrilinc / marathon-autoscaler

A simple autoscaler for Marathon applications

https://hub.docker.com/r/tendril/marathon-autoscaler/

Apache License 2.0

41 stars 16 forks source link

problem w/ app name that is a substring of another app name #24

Closed kevinwsin closed 7 years ago

kevinwsin commented 7 years ago

Hi, I ran into a problem where I was testing marathon-autoscaler w/ two Marathon apps, one named "nginx" and the other "nginx-2." Only the "nginx" app was set with the marathon autoscaler tags and it looks like marathon-autoscaler was skipping over that b/c the label lookup was not working properly (I think due to using in instead of == in the app name lookups). I got around it by removing the "nginx-2" app from my Marathon instance. Thanks.

kernelpanek-segfault commented 7 years ago

@kevinwsin Thank you for finding this problem! Once my colleague can review my change and merge the PR, we'll have a new docker image built.

kernelpanek-segfault commented 7 years ago

This issue is resolved.