Skip to content

ci: Ignore multiple images major versions bump - #4466

Merged
aldy505 merged 2 commits into
getsentry:masterfrom
aminvakil:dependabot_major_upgrades_limit
Aug 14, 2026
Merged

ci: Ignore multiple images major versions bump#4466
aldy505 merged 2 commits into
getsentry:masterfrom
aminvakil:dependabot_major_upgrades_limit

Conversation

@aminvakil

Copy link
Copy Markdown
Collaborator

Continuation of #4462 (comment) asked by @aldy505 .

Quick note: clickhouse version is specified in build.args.BASE_IMAGE of docker-compose.yml, so it's not getting updated by dependabot.

Legal Boilerplate

Look, I get it. The entity doing business as "Sentry" was incorporated in the State of Delaware in 2015 as Functional Software, Inc. and is gonna need some rights from me in order to utilize my contributions in this here PR. So here's the deal: I retain all rights, title and interest in and to my contributions, and by keeping this boilerplate intact I confirm that Sentry can use, modify, copy, and redistribute my contributions, under Sentry's choice of terms.

@aminvakil
aminvakil force-pushed the dependabot_major_upgrades_limit branch 2 times, most recently from 5a8bc83 to 1a462f2 Compare August 13, 2026 19:35

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Want reviews to match your repository better? Bugbot Learning can learn team-specific rules from PR activity. A team admin can enable Learning in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 1a462f2. Configure here.

Comment thread .github/dependabot.yml
- ">=2"
- dependency-name: chrislusf/seaweedfs
versions:
- ">=5"

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

SeaweedFS ignore likely ineffective

Low Severity

The new chrislusf/seaweedfs ignore may not block major bumps. That image is pinned with a digest in docker-compose.yml, and Dependabot still does not honor versions ignores for digest-pinned compose images, so >=5 updates can keep opening.

Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit 1a462f2. Configure here.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

dependabot takes digests into account as well, see dependabot/dependabot-core#6628 .

@aminvakil
aminvakil force-pushed the dependabot_major_upgrades_limit branch from 1a462f2 to 1489449 Compare August 13, 2026 19:58
@aldy505
aldy505 enabled auto-merge (squash) August 14, 2026 02:36
@aldy505
aldy505 merged commit a103814 into getsentry:master Aug 14, 2026
14 checks passed
@aminvakil
aminvakil deleted the dependabot_major_upgrades_limit branch August 14, 2026 09:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

2 participants