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

Add dockerfile #2

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

BriannaBrown-hub
Copy link

I ran into an issue with my python 2 versus python 3 locally and tripped down the rabbit hole of pyenv. My solution was to build a docker container with ubuntu and python and thought this might be helpful for other folks using amaxa.

Concerns I have with the current solution:

  • I wasn't sure what the level of documentation for docker should be, as it currently stands, the readme file's explanation is pretty bare bones. I'm happy to fill this out more.
  • I'm not clear on how to run the .gitlab-ci.yml to run the tests. I haven't run the tests at all as of yet.
  • To run the amaxa commands I've added a volume to the assets folder locally. The below convention has been working for me, i'm happy to add this to the readme file.
    docker run -it -v amaxa/assets:/amaxa/assets/ amaxa:my_local_amaxa bash

briannabrownOE and others added 3 commits March 20, 2019 11:16
Added instructions on how to build and run a docker container that will run Amaxa.
@@ -335,4 +343,4 @@ Future plans include:

## What Does Amaxa Mean?

[ἄμαξα](http://www.perseus.tufts.edu/hopper/text?doc=Perseus%3Atext%3A1999.04.0058%3Aentry%3Da\)%2Fmaca) is the Ancient Greek word for a wagon.
Copy link
Author

Choose a reason for hiding this comment

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

I think this added a new line, and I struggled with my editor to pull it out.

@@ -14,6 +14,14 @@ Make sure to invoke within a Python 3.6+ virtual environment or specify Python 3

Amaxa is operating system-agnostic. It has been tested primarily on Linux but is also known to work in a MINGW Windows 7 environment.

Using Amaxa within Docker, execute
Copy link
Author

Choose a reason for hiding this comment

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

I'm fairly new to docker and not sure what level of documentation people expect by including the docker image. The current explanation assumes some knowledge of docker. I'm happy to fill this out with a bit more detail.

@davidmreed
Copy link
Owner

Thank you for this PR, Brianna! Docker is not my expertise either, so I would like to take some time to study how best to include this within the Amaxa distribution. Please accept my apologies if I take a while to act on the PR.

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