Skip to content
This repository has been archived by the owner on Oct 23, 2024. It is now read-only.

Changed releaseVersion to temporarily remove Eremetic. #1650

Merged
merged 1 commit into from
Feb 1, 2018

Conversation

ssylvester87
Copy link

If Eremetic package is deployed under a Marathon deployment group, it can cause the mesos-master process to crash and take Marathon down with it. Temporarily removing package from the Universe until the edge case that causes this can be resolved.
See https://jira.mesosphere.com/browse/COPS-2254

This is due to a bug that can crash mesos-master. See COPS-2254
in internal Jira.
@jsancio
Copy link
Contributor

jsancio commented Jan 23, 2018

Why temporarily? What needs to happens for us to add this package? Why not update the package so that it doesn't crash Marathon?

@ssylvester87
Copy link
Author

It's a community supported package, not one supported by Mesosphere. I've submitted COPS-2270 as well so that we can reach out to the appropriate owners to get the package fixed, but it's been suggested to get it removed from the Universe in the meantime.

@surdy
Copy link
Contributor

surdy commented Jan 23, 2018

@jsancio The framework for the eremetic package registers with slashed (/) in the framework ID. Currently Mesos does not have checks around this and the slashed lead to mesos master failing. Mesos is going to add some validation to prevent master from failing in case of invalid framework ID https://issues.apache.org/jira/browse/MESOS-8470. But until them this package is a risk to the cluster causing master failures.

Hence we want to "disable" this package until we have a version of Mesos with a check to avoid this issue. Unless @ryadav88 can work with the package committer to fix the framework to not use slashes in the framework ID

@ryadav88
Copy link
Contributor

ryadav88 commented Feb 1, 2018

@gisjedi is the original submitter.
I've talked to him and created this issue: #1684
Merging this PR for now. Jonathan will send a new PR soon.

@ryadav88 ryadav88 merged commit f7dc7a6 into version-3.x Feb 1, 2018
@ryadav88 ryadav88 deleted the eremetic-temp-remove branch February 1, 2018 21:22
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 this pull request may close these issues.

4 participants