Skip to content

Skip building the current major version in the next major lane - #1103

Open
Watson1978 wants to merge 1 commit into
fluent:masterfrom
Watson1978:fix-nex-major-build-rpm
Open

Skip building the current major version in the next major lane#1103
Watson1978 wants to merge 1 commit into
fluent:masterfrom
Watson1978:fix-nex-major-build-rpm

Conversation

@Watson1978

Copy link
Copy Markdown
Contributor

"Build next major" ran "rake yum:build" twice: once for the current version and once more for the v7 one in v7-test. The first build's artifacts are discarded, since all three upload-artifact steps for them are guarded by next-major != 'true', so it was pure waste. The deb counterpart already guards the step this way; only the rpm one was left behind.

Measured on the AlmaLinux 10 x86_64 lane of run 30994758570:

  • Build rake yum:build (v6) 8m27s
  • Build next major rake yum:build (v6) 6m56s <- discarded
    rake yum:build (v7) 6m16s

To keep the RHEL9 and RHEL10 ABI checks meaningful, move them after the v7 build and point them at whichever repositories directory the lane actually produced. This makes the next major lane check the v7 packages instead of repeating what the "Build" lane already checks.

Also fail loudly when no rpm is found to check, so that a wrong directory cannot be mistaken for a clean result.

"Build next major" ran "rake yum:build" twice: once for the current version
and once more for the v7 one in v7-test. The first build's artifacts are
discarded, since all three upload-artifact steps for them are guarded by
next-major != 'true', so it was pure waste. The deb counterpart already
guards the step this way; only the rpm one was left behind.

Measured on the AlmaLinux 10 x86_64 lane of run 30994758570:

  * Build             rake yum:build (v6)   8m27s
  * Build next major  rake yum:build (v6)   6m56s  <- discarded
                      rake yum:build (v7)   6m16s

To keep the RHEL9 and RHEL10 ABI checks meaningful, move them after the v7
build and point them at whichever repositories directory the lane actually
produced. This makes the next major lane check the v7 packages instead of
repeating what the "Build" lane already checks.

Also fail loudly when no rpm is found to check, so that a wrong directory
cannot be mistaken for a clean result.

Signed-off-by: Shizuo Fujita <fujita@clear-code.com>
@Watson1978
Watson1978 force-pushed the fix-nex-major-build-rpm branch from a9a2dad to b2b9e9d Compare August 6, 2026 01:43
@Watson1978

Copy link
Copy Markdown
Contributor Author
Before After
スクリーンショット_20260806_105639 スクリーンショット_20260806_105527

Reduced 17m in Build next major to 9m.

@Watson1978
Watson1978 marked this pull request as ready for review August 6, 2026 01:59
@Watson1978 Watson1978 added the backport to v6.0 Need to be backported LTS label Aug 6, 2026
@Watson1978
Watson1978 requested a review from kenhys August 6, 2026 01:59

@kenhys kenhys 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.

LGTM.

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

Labels

backport to v6.0 Need to be backported LTS

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants