-
Notifications
You must be signed in to change notification settings - Fork 67
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add --framework_name option #21
Conversation
@@ -10,6 +10,7 @@ marathon_apt_repo: "deb http://repos.mesosphere.com/{{ansible_distribution|lower | |||
marathon_yum_package: "marathon-{{ marathon_version }}" | |||
mesosphere_yum_repo: "http://repos.mesosphere.com/el/{{ os_version_major }}/noarch/RPMS/{{ mesosphere_releases[os_version_major] }}" | |||
|
|||
marathon_framework_name: marathon |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you please use the default value used by Marathon, which is marathon-VERSION
where version is version of marathon of course.
Build is failing if you haven't noticed. Once fixed, we can go ahead merging. |
@@ -1,6 +1,6 @@ | |||
--- | |||
marathon_playbook_version: "0.3.2" | |||
marathon_version: "0.8.1" | |||
marathon_version: "0.11.1" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
version change must be in a separate commit
please add little test to .travis.yml |
ping @fpgeek |
@fpgeek you could also set the framework_name using the new feature added yesterday by #28 |
Closing due to lack of activity. |
No description provided.