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

debian-install-script #233

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

debian-install-script #233

wants to merge 3 commits into from

Conversation

oliv10
Copy link
Collaborator

@oliv10 oliv10 commented Jul 9, 2024

Debian install script and updates needed to other scripts to make sure that all the dependencies are preset for installation.

Docker installation updated to use the script from Docker. Currently this will be the OS limitations for the future.

I was unable to test the Debian install script to 100% completion as my VM would not start the containers. (I think this is mostly due to running amd64 through a compatibility layer on arm.) Running the containers on an arm Debian VM are ok and run as expected.

oliv10 added 3 commits July 9, 2024 14:48
Add wget to prerequisites for script.
Updated Docker installation script to use a single command for installing and configuring Docker.
@oliv10 oliv10 requested a review from mtweimer July 9, 2024 18:50
echo "Installing prerequisites..."
sudo apt update -y
sudo apt install wget -y
curl -o packages-microsoft-prod.deb https://packages.microsoft.com/config/debian/12/packages-microsoft-prod.deb # Someone needs to make this dynamic and check the OS version.
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@codecadwallader this may need an update to dynamically check for the correct version of Debian. Currently its set to use 12. Since Debian does not come with the Microsoft repos they need to be added manually.

@codecadwallader
Copy link
Contributor

Hey @oliv10 , thanks for putting these updates together. I'm sorry that I didn't see your message sooner. I had a backlog of GitHub notifications from my open-source project and had fallen behind. I've cleared those out and will be more on top of GitHub notifications going forwards.

I actually was in the process the last couple weeks of making a self-updating Linux service that is published through an APT package that we self-host out of Azure storage. It uses the unattended upgrades component of APT to enable automatic updates so that we can push new versions out to our customers and iterate on our scanners without requiring them to re-deploy. This included some code side changes to the scanner as well so that we can do some additional things like version tracking and improve security by no longer sending the scanner password over the wire.

I took parts of these scripts and folded them into the ShieldCyber repo where I'm hosting the scripts, DEBIAN metadata, GitHub action to build and publish the APT package and repository, etc. Currently I'm doing a live pull as part of the build process to grab the compose.yml from this repo but @mtweimer and I were discussing if it made sense to move that file over as well

I do not have any of these multi-Linux enhancements you've been working on in place. Since the deployment mechanism is through the standard APT packaging route it is indeed pretty limited to Debian based distros like Ubuntu. I'm glad you've explored that pathway for the Docker side of things and we can leverage this logic when we start expanding the scanner to other Linux distros (e.g., RedHat).

It may be good for the three of us to talk sometime about our strategy going forwards. 👍

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