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

118/feature/docker ci #121

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

118/feature/docker ci #121

wants to merge 9 commits into from

Conversation

AaronYoung5
Copy link
Collaborator

No description provided.

@AaronYoung5 AaronYoung5 self-assigned this Nov 21, 2023
@AaronYoung5 AaronYoung5 force-pushed the 118/feature/docker-ci branch from 9a03c76 to 29b62d9 Compare November 21, 2023 20:59
@AaronYoung5
Copy link
Collaborator Author

@StefanCaldararu kinda hype, no longer need to build. I've just built some cross-platform images on my machine and pushed them. I want these images to be updated whenever a commit happens on master, but it's hard to test the github action since it needs to be in master. I think it works (i tested the gh action locally at least).

https://hub.docker.com/repository/docker/uwsbel/art/general

@StefanCaldararu
Copy link
Contributor

StefanCaldararu commented Nov 22, 2023

@StefanCaldararu kinda hype, no longer need to build. I've just built some cross-platform images on my machine and pushed them. I want these images to be updated whenever a commit happens on master, but it's hard to test the github action since it needs to be in master. I think it works (i tested the gh action locally at least).

https://hub.docker.com/repository/docker/uwsbel/art/general

sweet I'll take a look in the AM, at the airport rn. Trade PR reviews?
(p.s. documentation doesn't count)

Copy link
Contributor

@StefanCaldararu StefanCaldararu left a comment

Choose a reason for hiding this comment

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

Lots of questions:

  • Still have to build art-oak even though it is a service in main?
  • Can't build other branches, so I don't really see this getting a ton of use. Everyone is using other branches, and all development happens on other branches (not master), so how does this help much? ig if your branch is similar enough to master you can pull the built container and then switch branches, right? But everyones branches are still pretty far behind so won't be able to use this
  • you mentioned pushing a new image every time a commit happens to main. This seems like there will be some pretty extreme data storage / etc. What happens when I just push a documentation change, for example? Wouldn't it make sense to just have more of an "abbreviated" history of images that is kept up to date?

@@ -146,6 +144,8 @@ as this is _not_ a volume.
The user profile is updated to add the python binary directory to `PYTHONPATH` and
the lib directory is appended to `LD_LIBRARY_PATH`.

**REMOVE_OPTIX** _(Default: `false`)_: Whether to remove the optix library after building chrono. This should be `'true'` if you plan to push the image to some public repository.
Copy link
Contributor

Choose a reason for hiding this comment

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

why is this not default to true?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Cause we can't push optix binaries/source, against the terms

@AaronYoung5 AaronYoung5 force-pushed the 118/feature/docker-ci branch from 8472c59 to 9525522 Compare January 27, 2024 22:41
@AaronYoung5
Copy link
Collaborator Author

AaronYoung5 commented Jan 27, 2024

Still have to build art-oak even though it is a service in main?

By default, only chrono, dev, and vnc are pushed. But you can push a local image with atk dev -c push -s <service>. Github doesn't have agx/jetsons connected to github actions, so not possible build agx images there.

Can't build other branches, so I don't really see this getting a ton of use. Everyone is using other branches, and all development happens on other branches (not master), so how does this help much? ig if your branch is similar enough to master you can pull the built container and then switch branches, right? But everyones branches are still pretty far behind so won't be able to use this

I mean this is also true for any change we make to main rn, right?

you mentioned pushing a new image every time a commit happens to main. This seems like there will be some pretty extreme data storage / etc. What happens when I just push a documentation change, for example? Wouldn't it make sense to just have more of an "abbreviated" history of images that is kept up to date?

I think this makes sense, like the action isn't on by default, you have to explicitly run the action. As a note, docker images are layers, so like each RUN command is an additional layer on top of the FROM image you built with. So if a change happens that doesn't effect the image, i.e. docs, no additional information is actually stored on dockerhub since no layers changed. Does this make sense? Does this change your opinion 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.

2 participants