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

allow override of docker image #728

Open
wants to merge 2 commits into
base: mupx
Choose a base branch
from
Open

allow override of docker image #728

wants to merge 2 commits into from

Conversation

gdw2
Copy link

@gdw2 gdw2 commented Oct 31, 2015

Use case:

My meteor app depends on having the graphicsmagick binary installed, so I want to use a docker image, based off meteord, that contains the dependency.

@@ -219,6 +219,12 @@ Meteor Up uses Docker to run and manage your app. It uses [MeteorD](https://gith

For more information see [`lib/taskLists.js`](https://github.com/arunoda/meteor-up/blob/mupx/lib/taskLists/linux.js).

#### Using a custom docker image

The default docker image (MeteorD) can be overridden by setting `dockerImage`
Copy link
Owner

Choose a reason for hiding this comment

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

It is mup.json file. Not the settings.json file.
Also, show a code sample on to do it. Just having the dockerImage key is enough.

Also, add a link to to MeteorD

Copy link
Owner

Choose a reason for hiding this comment

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

May be you can link also another document on how to extend MeteorD. May be a blog post of yours.
Anyway, that can be done later.

@arunoda
Copy link
Owner

arunoda commented Nov 3, 2015

Mention me once you done changes. It's hard to track it otherwise.
On 2015 නොවැ 4, බදාදා at පෙ.ව. 12.24 gdw2 [email protected] wrote:

This may address #728 zodern/meteor-up#728


Reply to this email directly or view it on GitHub
zodern/meteor-up#728 (comment).

@gdw2
Copy link
Author

gdw2 commented Nov 4, 2015

Thanks for the feedback. Will fix this week.

@bompi88
Copy link

bompi88 commented Nov 16, 2015

+1

@MichaelJCole
Copy link

Hey, I was just about to make these same changes. @gdw2 did they work well for you?

@gdw2 @arunoda is there any help I can offer to move this over the finish line?

@gdw2
Copy link
Author

gdw2 commented Dec 8, 2015

@arunoda . I addressed your review comments.
@MichaelJCole . It has worked well for me.

@v3rron
Copy link

v3rron commented Jan 3, 2016

@arunoda can you pls accept this pull request? thanks

@bompi88
Copy link

bompi88 commented Jan 13, 2016

@gdw2 @arunoda Shouldn't this be configurable for all deploy targets as well as the general option?

@bompi88
Copy link

bompi88 commented Jan 21, 2016

Looks like this will be the new mup? Haven't tried it with custom docker image yet, though.

@MasterJames
Copy link

It will have that. I'm pretty sure I saw a post in the new still under development version to add custom docker images.

@comigor
Copy link

comigor commented Feb 15, 2016

Oh please merge it already

@MasterJames
Copy link

This account in general has been depreciated and mupx has almost been ported to kadirahq.
For the record I am no an official representative.
All unmerged PRs need to be ported to the new version. I believe there will be a more regorious need to comply with dev&submission methods/requirements, which are still being outlined for Mantra in general. Basically meaning tests are now needed to accept Pull Requests. I don't think any or many PRs here do, mine don't.

@comigor
Copy link

comigor commented Feb 15, 2016

@MasterJames oh, nice, I wasn't aware of that. Thank you!

@eskan
Copy link

eskan commented Jun 5, 2016

very useful PR : @arunoda may you merge it ?
thanks

@MasterJames
Copy link

No PRs have been merged here for a very long time. It is considered the stable version partly for that reason.
New development is at kadirahq/meteor-up if there's a PR of interest here that is still needed in the new version then it could be redone for consideration there. It wouldn't hurt to ask the devs there if it's needed and desired first.
Oh I already commented here.

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.

8 participants