Skip to content

GH-51217: [CI][Packaging] Don't use diffoscope - #51221

Merged
kou merged 1 commit into
apache:mainfrom
kou:debian-reprotest
Sep 9, 2026
Merged

GH-51217: [CI][Packaging] Don't use diffoscope#51221
kou merged 1 commit into
apache:mainfrom
kou:debian-reprotest

Conversation

@kou

@kou kou commented Sep 8, 2026 •
edited
Loading

Copy link
Copy Markdown
Member

Rationale for this change

reprotest step is flaky. It seems that it's killed by OOM (out-of-memory).

What changes are included in this PR?

It seems that diffoscope is heavy. Disable it.

Are these changes tested?

Yes.

Are there any user-facing changes?

No.

Copilot AI lite review requested due to automatic review settings September 8, 2026 02:03
Copilot AI reviewed Sep 8, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@kou kou added the CI: Extra: Package: Linux Run extra Linux Packages CI label Sep 8, 2026
@github-actions

github-actions Bot commented Sep 8, 2026

Copy link
Copy Markdown

⚠️ GitHub issue #51217 has been automatically assigned in GitHub to PR creator.

@github-actions github-actions Bot added the awaiting committer review Awaiting committer review label Sep 8, 2026
@kou
kou force-pushed the debian-reprotest branch from 83e996d to 41f1432 Compare September 8, 2026 06:21
Copilot AI review requested due to automatic review settings September 8, 2026 06:21
Copilot AI reviewed Sep 8, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@kou
kou force-pushed the debian-reprotest branch from 41f1432 to fcb2aba Compare September 8, 2026 06:49
Copilot AI review requested due to automatic review settings September 8, 2026 06:49
Copilot AI reviewed Sep 8, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@kou kou changed the title GH-51217: [CI][Packaging] Reduce build parallelism in reprotest GH-51217: [CI][Packaging] Don't use diffoscope Sep 8, 2026
@kou

kou commented Sep 9, 2026

Copy link
Copy Markdown
Member Author

Let's try this.

@kou
kou merged commit f39939e into apache:main Sep 9, 2026
73 of 74 checks passed
@kou
kou deleted the debian-reprotest branch September 9, 2026 01:50
@kou kou removed the awaiting committer review Awaiting committer review label Sep 9, 2026
@kou kou mentioned this pull request Sep 9, 2026
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot
Copilot review effort
Applies to this pull request for everyone.Learn more about Copilot code review.
Copilot left review comments
@assignUser assignUser Awaiting requested review from assignUser assignUser is a code owner
@jonkeane jonkeane Awaiting requested review from jonkeane jonkeane is a code owner
@raulcd raulcd Awaiting requested review from raulcd raulcd is a code owner

Assignees

No one assigned

Labels

CI: Extra: Package: Linux Run extra Linux Packages CI

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@kou