Skip to content

Conversation

@yuzefovich
Copy link
Member

@yuzefovich yuzefovich commented Oct 20, 2025

Backport 1/1 commits from #155721 on behalf of @yuzefovich.


We just made a change to include skipped FKs (those that we deemed "irrelevant") in the commented out form. This commit improves that logic by adding a comment for why these FKs are commented out, to reduce possible confusion.

Epic: None
Release note: None


Release justification: low-risk supportability improvement.

We just made a change to include skipped FKs (those that we deemed
"irrelevant") in the commented out form. This commit improves that logic
by adding a comment for why these FKs are commented out, to reduce
possible confusion.

Release note: None
@yuzefovich yuzefovich force-pushed the blathers/backport-release-25.2-155721 branch from f0f4332 to 0b9a680 Compare October 20, 2025 20:24
@blathers-crl blathers-crl bot added blathers-backport This is a backport that Blathers created automatically. O-robot Originated from a bot. labels Oct 20, 2025
@blathers-crl blathers-crl bot requested a review from michae2 October 20, 2025 20:24
@blathers-crl
Copy link

blathers-crl bot commented Oct 20, 2025

Thanks for opening a backport.

Before merging, please confirm that it falls into one of the following categories (select one):

  • Non-production code changes. Includes test-only changes, build system changes, etc.
  • Fixes for serious issues. Defined in the policy as correctness, stability, or security issues, data corruption/loss, significant performance regressions, breaking working and widely used functionality, or an inability to detect and debug production issues.
  • Other approved changes. These changes must be gated behind a disabled-by-default feature flag unless there is a strong justification not to.

Add a brief release justification to the PR description explaining your selection.

Also, confirm that the change does not break backward compatibility and complies with all aspects of the backport policy.

All backports must be reviewed by the TL and EM for the owning area.

@blathers-crl blathers-crl bot added backport Label PR's that are backports to older release branches T-sql-queries SQL Queries Team labels Oct 20, 2025
@blathers-crl
Copy link

blathers-crl bot commented Oct 20, 2025

It looks like your PR touches production code but doesn't add or edit any test code. Did you consider adding tests to your PR?

🦉 Hoot! I am a Blathers, a bot for CockroachDB. My owner is dev-inf.

@blathers-crl
Copy link

blathers-crl bot commented Oct 20, 2025

❌ PR #155730 does not comply with backport policy

Confidence: high
Explanation: The pull request appears to introduce a supportability improvement by adding comments to explain skipped foreign keys in the code. Although the PR includes a 'Release justification:' in the body indicating that it is a 'low-risk supportability improvement', this does not meet the CockroachDB backport policy for critical bugs or feature flags. The change is not behind a feature flag nor does it fix a critical bug as defined in the policy.
Recommendation: Reconsider the backport or add a feature flag to gate the change

🦉 Hoot! I am a Blathers, a bot for CockroachDB. My owner is dev-inf.

@cockroach-teamcity
Copy link
Member

This change is Reviewable

Copy link
Collaborator

@michae2 michae2 left a comment

Choose a reason for hiding this comment

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

I disagree with blathers. This is extremely low risk and was specifically asked for by Support after an escalation.

@yuzefovich yuzefovich merged commit 4928bd0 into cockroachdb:release-25.2 Oct 21, 2025
15 of 16 checks passed
@yuzefovich yuzefovich deleted the blathers/backport-release-25.2-155721 branch October 21, 2025 17:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport Label PR's that are backports to older release branches blathers-backport This is a backport that Blathers created automatically. O-robot Originated from a bot. T-sql-queries SQL Queries Team target-release-25.2.8

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants