-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Refactor Docker setup, follow best-practices in containerization and make caldera easier to deploy #3114
base: master
Are you sure you want to change the base?
Conversation
These changes are super interesting and I look forward to going through them in detail! 🚀
In the meantime, here's the minimal fix for the Docker build which was approved and just waiting on a merge: |
💯 Thank you for tackling these changes! I fully agree that this project will be much more useful when Docker deployment is trivial. 👍 |
ce513fb
to
49c54e4
Compare
Thanks @rfulwell. I made a few additional change to the Docker setup, I was wondering if you could test it yourself as well to make sure everything works? I am by no means a caldera power user and would like to make sure these changes don't break anything. |
And thank you for the original PR fixing the outdated Ubuntu version in the Dockerfile, hope that gets merged quickly as well. |
ac27f4a
to
223a63b
Compare
On our queue. |
While I don't use docker for this project, I think quite a lot of people do to quickly get started. This PR would be a great service to the community. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
some minor merge conflicts and a couple suggestions to get rid of the warning messages, and should be good to go
Follow best-practices and make caldera easier to use
c07ebac
to
29ad20c
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One final piece since it looks like there's a conflict between the setuptools in requirements.txt and in tox.ini. Can you remove the setuptools from tox.ini since we're now specifying it in requirements.txt? That should allow our workflows to run properly, and once those pass we can move this forward
requirements.txt is being used instead
@uruwhy approved and this is now on queue. Will look at asap. |
Description
This project is currently quite difficult to get working. This is one of (at least) three pull requests intended to make caldera simpler to deploy or get running locally. Docker is a great tool for this, and should be the easiest and quickest deployment option.
Unfortunately, the current Dockerfile doesn't even build (due to an outdated version of Ubuntu).
This PR:
setuptools
to work with recent versions of Python3Type of change
Please delete options that are not relevant.
How Has This Been Tested?
Repeated build tests of both flavors of caldera (full + slim). The resulting containers appear to work when interacted with.
Checklist: