Skip to content
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

Moving marathon away from ubuntu and starting from mesosphere dockerhub #272

Merged
merged 5 commits into from
May 22, 2020

Conversation

missingcharacter
Copy link
Contributor

These changes work locally, I've not tested TravisCI yet.

If you are ok with using pre-built container images, I'd change the tests to build everytime and just pull the images from https://hub.docker.com/r/missingcharacter/marathon-python/tags

It might be a good idea for you guys to create your own repository in hub.docker.com for the images to be pulled from.

@@ -1,12 +1,29 @@
#!/bin/bash
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would be happy if you got rid of this script entirely.

@solarkennedy
Copy link
Contributor

I would be just as happy if you only tested against 1 version (latest) of marathon and didn't try to do the whole matrix thing with the args and junk. I don't think it is worth it now.

@missingcharacter
Copy link
Contributor Author

@solarkennedy
Copy link
Contributor

Woot! Thanks! This is above and beyond!

@solarkennedy solarkennedy merged commit d8d74a5 into thefactory:master May 22, 2020
@solarkennedy
Copy link
Contributor

@mbabineau can you make @missingcharacter a contributor?

@missingcharacter missingcharacter deleted the fix-tests branch May 22, 2020 01:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants