-
Notifications
You must be signed in to change notification settings - Fork 68
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
modified: configuration/arm64/docker-compose.yml #585
Merged
Merged
Changes from 5 commits
Commits
Show all changes
8 commits
Select commit
Hold shift + click to select a range
d9c29df
modified: configuration/arm64/docker-compose.yml
jwillmartin 49a43ad
Merge remote-tracking branch 'origin' into fix-arm64
jwillmartin b4f7638
Merge remote-tracking branch 'origin' into fix-arm64
jwillmartin 12d665a
modified: configuration/arm64/docker-compose.yml
jwillmartin a30e860
Update docker-compose.yml
jwillmartin 607b719
Merge remote-tracking branch 'origin' into fix-arm64
jwillmartin e10ea50
modified: configuration/amd64/docker-compose.yml
jwillmartin d7a8bc7
Merge remote-tracking branch 'origin' into fix-arm64
jwillmartin File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -9,9 +9,6 @@ sudo apt update -y && sudo apt upgrade -y | |
sudo apt-get install chromium-browser -y #Chrome required for CARMA platform/V2X Hub UI(?) | ||
sudo apt install curl -y #Curl for downloading files over internet | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Add curl back. See comments above. |
||
|
||
#install docker | ||
curl -L https://raw.githubusercontent.com/usdot-fhwa-stol/carma-platform/develop/engineering_tools/install-docker.sh | bash | ||
paulbourelly999 marked this conversation as resolved.
Show resolved
Hide resolved
|
||
|
||
#make passwords for mysql | ||
mkdir -p secrets && cd secrets | ||
|
||
|
@@ -38,17 +35,21 @@ fi | |
|
||
#ARM initialization | ||
cd $directory | ||
sudo apt-get -y remove docker docker-engine docker.io containerd runc | ||
sudo apt-get -y install apt-transport-https ca-certificates curl gnupg-agent software-properties-common | ||
OS=$(lsb_release -i | awk 'FS=":" {print $3;}' | awk '{print tolower($0)}') | ||
arch=$(dpkg --print-architecture) | ||
curl -fsSL https://download.docker.com/linux/$OS/gpg | sudo apt-key add - | ||
sudo add-apt-repository "deb [arch=$arch] https://download.docker.com/linux/$OS $(lsb_release -cs) stable" | ||
sudo apt-get -y install docker-ce docker-ce-cli containerd.io docker-compose-plugin | ||
sudo apt -y install python3-pip | ||
sudo pip3 install docker-compose | ||
sudo apt update -y && sudo apt upgrade -y | ||
sudo docker-compose up -d | ||
for pkg in docker.io docker-doc docker-compose podman-docker containerd runc; do sudo apt-get remove $pkg; done | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Was getting warnings about gpg key deprecation. Updated and tested using Docker's official Debian installation steps |
||
# Add Docker's official GPG key: | ||
sudo apt-get -y install ca-certificates | ||
sudo install -m 0755 -d /etc/apt/keyrings | ||
sudo curl -fsSL https://download.docker.com/linux/debian/gpg -o /etc/apt/keyrings/docker.asc | ||
sudo chmod a+r /etc/apt/keyrings/docker.asc | ||
|
||
# Add the repository to Apt sources: | ||
echo \ | ||
"deb [arch=$(dpkg --print-architecture) signed-by=/etc/apt/keyrings/docker.asc] https://download.docker.com/linux/debian \ | ||
$(. /etc/os-release && echo "$VERSION_CODENAME") stable" | \ | ||
sudo tee /etc/apt/sources.list.d/docker.list > /dev/null | ||
sudo apt-get update | ||
sudo apt-get -y install docker-ce docker-ce-cli containerd.io docker-buildx-plugin docker-compose-plugin | ||
sudo docker compose up -d | ||
|
||
#create v2xhub user | ||
cd $mysqlDir | ||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
Is the idea here that we do not want to default to having port drayage functionality on?
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.
@paulbourelly999 Yeah. It's not used by anybody at the moment. It'll be better to have it commented out unless somebody will use it.
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.
Leave a comment indicating how to get port drayage functionality back and/or to come up with a more elegant solution to not including port drayage by default.
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.
Added comments with instructions
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.
All, I think we need to leave the port_drayage as is. The expectation is that whatever was deployed in the past should be backwards compatible or scenarios would continue to work as is, unless we notify the client we're removing this feature or retiring it.
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.
Changed back