Skip to content

CSHARP-6039: Implement a new EG variable to override the default UPLOAD_BUCKET path#2008

Open
sanych-sun wants to merge 1 commit into
mongodb:mainfrom
sanych-sun:CSHARP-6039
Open

CSHARP-6039: Implement a new EG variable to override the default UPLOAD_BUCKET path#2008
sanych-sun wants to merge 1 commit into
mongodb:mainfrom
sanych-sun:CSHARP-6039

Conversation

@sanych-sun
Copy link
Copy Markdown
Member

No description provided.

@sanych-sun sanych-sun requested a review from adelinowona May 22, 2026 22:15
@sanych-sun sanych-sun requested a review from a team as a code owner May 22, 2026 22:15
Copilot AI review requested due to automatic review settings May 22, 2026 22:15
@sanych-sun sanych-sun added the chore Non–user-facing code changes (tests, build scripts, etc.). label May 22, 2026
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the Evergreen configuration to support a new variable (upload_prefix) that can override the default S3 key prefix previously derived from UPLOAD_BUCKET when uploading/downloading build artifacts.

Changes:

  • Removes the implicit UPLOAD_BUCKET="${project}" export/expansion wiring.
  • Replaces all S3 remote_file paths that used ${UPLOAD_BUCKET}/... with ${upload_prefix}... (upload build archive, logs, NuGet packages, apicompat artifacts).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread evergreen/evergreen.yml
Comment on lines 62 to 66
export MONGO_ORCHESTRATION_HOME="$DRIVERS_TOOLS/.evergreen/orchestration"
export MONGODB_BINARIES="$DRIVERS_TOOLS/mongodb/bin"
export UPLOAD_BUCKET="${project}"
export PROJECT_DIRECTORY="$(pwd)"

cat <<EOT > expansion.yml
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

${upload_prefix} is a new EG variable. It's available globally from repo.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

chore Non–user-facing code changes (tests, build scripts, etc.).

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants