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

Dockerfile for Blin automation #10

Merged
merged 4 commits into from
May 28, 2019
Merged

Dockerfile for Blin automation #10

merged 4 commits into from
May 28, 2019

Conversation

kawaii
Copy link
Member

@kawaii kawaii commented Mar 15, 2019

Adds a very rough draft Dockerfile and .dockerignore so that we can start automating testing the module ecosystem with Blin. It accepts three environment variables currently which are $RAKUDO_OLD, $RAKUDO_NEW and $MODULES. The first two will default to 2018.10 and HEAD respectively if you do not provide them. $MODULES should be a space-separated list of modules (example below), if left blank it should simply test the entire ecosystem. Be warned.

Example:

docker build -t perl6/blin:latest $PWD
docker container run -e MODULES="DBIish URI::Encode" perl6/blin:latest

@AlexDaniel
Copy link
Member

Is there any way to pull all files from the output/ directory? I'm not happy with the cat workaround.

@kawaii
Copy link
Member Author

kawaii commented Mar 15, 2019

@AlexDaniel yes, users can be directed to use a bind-mounted Docker volume like so:

docker container run -e MODULES="DBIish URI::Encode" \
                       -v /tmp/blin-output:/opt/perl6/blin/output:rw \
                       perl6/blin:latest

Which then deposits the output files into my /tmp/blin-output:

# kane @ shuppet in /tmp/blin-output [17:52:15] 
$ ll
total 52K
-rw-r--r-- 1 root root 45K Mar 15 17:52 data.json
-rw-r--r-- 1 root root  77 Mar 15 17:52 overview

Does this satisfy your requirement if part of the documentation? Of course, we still need to figure out the issues we discovered in IRC first before this sees any production use anyway. 🙂

@AlexDaniel
Copy link
Member

I guess you can merge this? I can't tell if it's useful already.

@kawaii
Copy link
Member Author

kawaii commented May 28, 2019

@AlexDaniel I don't appear to have write access to this repository, I pushed some minor updates to the PR branch though just to update it a little bit. It's not super useful currently but if Blin automation is on our to-do list then it goes some way to helping towards that goal.

@AlexDaniel
Copy link
Member

@kawaii can you try now?

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.

None yet

2 participants