Skip to content
This repository has been archived by the owner on Jan 22, 2020. It is now read-only.

add 1.10 folder for app definition files #50

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

margaret
Copy link
Contributor

I misinterpreted mellenburg's comment on #49. Need to have a 1.10 too since that's been released. Top level file is meant to work with HEAD of master branch.

@margaret
Copy link
Contributor Author

Also updated the contents of tweeter.json for 1.10. portMappings value changed locations within the definition.

@ghost
Copy link

ghost commented Sep 15, 2017

@margaret Per https://github.com/mesosphere/marathon/blob/e4f015962906a2cd54fad10692d42b2e07850ebb/changelog.md I believe the field is networks, not network and it's an array.

@margaret
Copy link
Contributor Author

😑 Thanks. Yeah, i just noticed that

@margaret
Copy link
Contributor Author

squashed

@margaret
Copy link
Contributor Author

margaret commented Sep 16, 2017

testing via https://github.com/mesosphere/dcos-enterprise/pull/1312 (to-do is to get a CI running on this repo again)

@margaret margaret requested a review from a user September 16, 2017 00:50
@ghost
Copy link

ghost commented Sep 18, 2017

@margaret The config looks correct to me. However, as of 1.10 we are trying to encourage folks to use the MESOS container runtime. Would it make sense to change tweeter to use that instead?

@margaret
Copy link
Contributor Author

@stbof Yeah, that sounds like a good idea. I'll make a JIRA ticket for it.

@margaret
Copy link
Contributor Author

margaret commented Sep 18, 2017

😞 still can't access 1.1.1.1:30000 even with the updated app definition. Will look around some more and see if I'm accessing it the wrong way https://teamcity.mesosphere.io/viewLog.html?buildId=787750&buildTypeId=ClosedSource_MesosphereEnterpriseDcOs_IntegrationTests_TweeterDemoEnterprise

@margaret
Copy link
Contributor Author

margaret commented Sep 19, 2017

This PR is on hold - need to update the AMIs that dcos-launch uses as they are no long compatible with most recent cassandra

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.

None yet

1 participant