-
Notifications
You must be signed in to change notification settings - Fork 15
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
Update Arangodb packaged version to using 3.12.4 #1271
Update Arangodb packaged version to using 3.12.4 #1271
Conversation
Reviewer's Guide by SourceryThis pull request updates the ArangoDB version to 3.12.4. It modifies the installation script to use the new version and updates the docker compose files to use the new image. No diagrams generated as the changes look simple and do not need a visual representation. File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
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.
Hey @JoshuaSBrown - I've reviewed your changes - here's some feedback:
Overall Comments:
- The PR description mentions documentation updates, but there don't appear to be any changes to the documentation files in the diff.
Here's what I looked at during the review
- 🟡 General issues: 1 issue found
- 🟢 Security: all looks good
- 🟢 Testing: all looks good
- 🟢 Complexity: all looks good
- 🟢 Documentation: all looks good
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
curl -OL https://download.arangodb.com/arangodb312/DEBIAN/Release.key | ||
"$SUDO_CMD" apt-key add - < Release.key |
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.
suggestion: Add error handling for the Release key download
Consider adding error checking after the curl command to ensure the Release key was downloaded successfully before proceeding with the installation.
curl -OL https://download.arangodb.com/arangodb312/DEBIAN/Release.key | |
"$SUDO_CMD" apt-key add - < Release.key | |
if ! curl -OL https://download.arangodb.com/arangodb312/DEBIAN/Release.key; then | |
echo "Failed to download ArangoDB Release key" | |
return 1 | |
fi | |
if [ ! -s Release.key ]; then | |
echo "ArangoDB Release key file is empty or missing" | |
return 1 | |
fi | |
"$SUDO_CMD" apt-key add - < Release.key |
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.
I will not add this without testing, but it looks like a good suggestion.
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.
Can this be added as a new issue to possibly refactor this part of the script.
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.
I have created an issue for your comment: #1275
PR Description
Update Arangodb version and pull officially supported version, update documentation.
Tasks
Summary by Sourcery
Update Arangodb to version 3.12.4.
Build:
Deployment:
Documentation: