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 with GitHub actions #373

Open
wants to merge 10 commits into
base: master
Choose a base branch
from

Conversation

Thirsty2
Copy link

@Thirsty2 Thirsty2 commented Oct 4, 2020

This Dockerfile and associated github actions builds an image based on Alpine Linux 3.12. The docker build executes regression tests automatically, and publishes the container image as a github package if the build and tests succeed.

@caryr
Copy link
Collaborator

caryr commented Jan 9, 2021

@steveicarus is there any reason this should not be merged. Do we need more information regarding exactly what this is doing/how it will be used? @martinwhitaker do you have any comments? This all seems reasonable, but I must admit I've not looked at Docker much.

@caryr
Copy link
Collaborator

caryr commented Jan 9, 2021

And a question for @Thirsty2 what is your intention for having a containerized version of Icarus? I can think of some, but would appreciate understanding your perspective as well.

@Thirsty2
Copy link
Author

Thirsty2 commented Jan 9, 2021 via email

@caryr
Copy link
Collaborator

caryr commented Jan 9, 2021

I assume this is build automation for you (e.g in an academic/learning setting someone would build a container and then that could be used by all students with no system, version, etc issues).

@martinwhitaker
Copy link
Collaborator

I don't have any objections to this, but I don't understand the need to include it either.

@caryr
Copy link
Collaborator

caryr commented Jan 9, 2021

Yes Martin, that's basically where I am. I can see where this could be useful for the case I mentioned earlier and maybe there are others I've not thought of. I will admit my vision is likely clouded by years of building a huge number of tools from source so all this has mostly become second nature to me.

To me the biggest issue preventing me from saying this is a no brainier is you would like to have GTKWave available, but from what I read GUI programs are not really supported.

@Thirsty2
Copy link
Author

Thirsty2 commented Jan 10, 2021 via email

@steveicarus
Copy link
Owner

I think the Dockerfile files are OK as a source of build documentation, if nothing else. They are harmless.

However, I wonder if the deploy action would eat into our free action compute time budget. The CI actions are already time consuming. do we really want to include docker build and deploys with all the other build actions?

@caryr
Copy link
Collaborator

caryr commented Jan 10, 2021

I believe the deploy should certainly not be part of the normal CI. For stable it should only be for an actual release, for devel maybe quarterly or monthly or maybe never. Is there a different pool that can be used for the docker build as compared to our existing CI?

@caryr
Copy link
Collaborator

caryr commented Jan 11, 2021

Hi Doug, I assume this is just the recipe for building containers and there is no risk of this creating extra load where it builds every time we push a change. Once you confirm I will merge this.

@Thirsty2
Copy link
Author

Thirsty2 commented Jan 11, 2021 via email

@steveicarus steveicarus force-pushed the master branch 8 times, most recently from c698c31 to fa85cc7 Compare April 16, 2022 03:14
@steveicarus steveicarus force-pushed the master branch 7 times, most recently from 9728397 to 0dc8596 Compare April 16, 2022 03:49
@DeflateAwning
Copy link
Contributor

This would be great to get integrated into the project!

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.

5 participants