-
Notifications
You must be signed in to change notification settings - Fork 21
SERVER-216 #383
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
SERVER-216 #383
Conversation
|
Why is there no gha-test-main.sh script in this PR? |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #383 +/- ##
==========================================
+ Coverage 69.01% 69.09% +0.08%
==========================================
Files 92 92
Lines 21549 21673 +124
==========================================
+ Hits 14871 14975 +104
- Misses 6678 6698 +20 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
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've committed needed changes for makefiles and with that this file should be changed to:
function build_packages(){
if [ "$ENV_DISTRO" = "" ]; then
echo "ENV_DISTRO is not set"
return
fi
chown -R root:root .
GIT_DIR=$(git rev-parse --show-toplevel)
cd "$GIT_DIR"
echo "building package for $BUILD_DISTRO"
if [[ $ENV_DISTRO == "ubuntu" ]]; then
make deb
elif [[ $ENV_DISTRO == "debian" ]]; then
make deb
elif [[ $ENV_DISTRO == "el" ]]; then
make rpm
elif [[ $ENV_DISTRO == "amzn" ]]; then
make rpm
else
make tar
fi
mkdir -p /tmp/output/$ENV_DISTRO
cp -a $GIT_DIR/pkg/target/* /tmp/output/$ENV_DISTRO
}
pvinh-spike
left a comment
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.
We need to add this line in all *.yaml :
git config --global --add safe.directory "$GITHUB_WORKSPACE"
…c version conflicts
…ent glibc version conflicts" This reverts commit 189608b.
No, they take different build arguments and have different purposes. |
What does that do? |
…ERVER-216 # Conflicts: # .github/packaging/common
…c version conflicts
aa0b9e2 to
904493a
Compare
Built packages with links to JFrog are available in the
Actionstab aboveIntegration tests are here.
Test cases download the current build from JFrog and install it in a fresh container, then execute it with the --help argument
You need a JFrog username and token to execute the test cases, an example is in the README.sh