Skip to content

Comments

PSMDB-1925 Recognize 'psmdb-x.y.x' tags in 'set_mongo_variables.py'#1669

Merged
igorsol merged 1 commit intov8.0from
psmdb-1925-igor
Jan 29, 2026
Merged

PSMDB-1925 Recognize 'psmdb-x.y.x' tags in 'set_mongo_variables.py'#1669
igorsol merged 1 commit intov8.0from
psmdb-1925-igor

Conversation

@igorsol
Copy link

@igorsol igorsol commented Jan 28, 2026

fixes the issue with 'psmdb-x.y.x' tags during the build

Copy link

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 Bazel wrapper hook so it correctly derives MONGO_VERSION when the Git tag uses the Percona-style psmdb-x.y.z prefix, in addition to the existing upstream r-prefixed tags.

Changes:

  • Adjusts get_mongo_version() to strip either an upstream r prefix or a Percona psmdb- prefix from the git describe --abbrev=0 output, returning the cleaned version string.
  • Leaves the rest of the Bazel .bazelrc.mongo_variables generation logic unchanged, so only tag parsing behavior is modified.

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

Copy link

@ktrushin ktrushin left a comment

Choose a reason for hiding this comment

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

Looks good

Copy link

@plebioda plebioda left a comment

Choose a reason for hiding this comment

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

Looks good, except one minor comment/question.

@igorsol igorsol merged commit 4ab6133 into v8.0 Jan 29, 2026
4 checks passed
@igorsol igorsol deleted the psmdb-1925-igor branch January 29, 2026 11:38
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.

3 participants