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

Fix latest-build URL for RHEL #603

Merged
merged 3 commits into from
Feb 18, 2025

Conversation

kevinAlbs
Copy link
Contributor

Fix latest-build download URL in mongodl for RHEL

Trying to download latest-build in mongodl for some RHEL distros results in error:

python ./.evergreen/mongodl.py  --version latest-build --target rhel9 --component archive
# Failed to download [https://downloads.10gen.com/linux/mongodb-linux-x86_64-enterprise-rhel9-latest.tgz]

Comparing the URL in download-mongodb.sh on before migrating to mongodl.py:

Old:     https://downloads.10gen.com/linux/mongodb-linux-x86_64-enterprise-rhel93${DEBUG}-latest.tgz
Current: https://downloads.10gen.com/linux/mongodb-linux-x86_64-enterprise-rhel9-latest.tgz

I expect some latest-build RHEL URLs include the minor version (93 instead of 9). To fix: this PR checks the URL in latest for a minor version when building the URL for the latest-build.

Testing latest-build is added to test-cli.sh.

Verified with this patch build: https://spruce.mongodb.com/version/67b39deeae39430007faa599. Failures appear unrelated.

@kevinAlbs kevinAlbs marked this pull request as ready for review February 18, 2025 12:59
@kevinAlbs kevinAlbs requested a review from blink1073 February 18, 2025 12:59
@blink1073
Copy link
Member

OIDC failure is addressed in #606

Copy link
Member

@blink1073 blink1073 left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@kevinAlbs kevinAlbs merged commit 70d56ca into mongodb-labs:master Feb 18, 2025
72 of 74 checks passed
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