Skip to content

Backport(v1.19): test_plugin_generator: stub the Apache License download (#5461) - #5466

Merged
Watson1978 merged 1 commit into
v1.19from
backport-to-v1.19/pr5461
Aug 9, 2026
Merged

Backport(v1.19): test_plugin_generator: stub the Apache License download (#5461)#5466
Watson1978 merged 1 commit into
v1.19from
backport-to-v1.19/pr5461

Conversation

@github-actions

@github-actions github-actions Bot commented Aug 9, 2026

Copy link
Copy Markdown

Which issue(s) this PR fixes:
Backport #5461
Fixes #

What this PR does / why we need it:
FluentPluginGenerator::ApacheLicense downloads the license text from apache.org every time a plugin is generated, so the test case made 12 requests per run and depended on the network.

This causes test failures when the network is unstable, so I added a stub for it.

Ref. https://github.com/fluent/fluentd/actions/runs/30705665490/job/91384215966

Docs Changes:
N/A

Release Note:
N/A

**Which issue(s) this PR fixes**:
Fixes #

**What this PR does / why we need it**:
`FluentPluginGenerator::ApacheLicense` downloads the license text from
apache.org every time a plugin is generated, so the test case made 12
requests per run and depended on the network.

This causes test failures when the network is unstable, so I added a
stub for it.

Ref.
https://github.com/fluent/fluentd/actions/runs/30705665490/job/91384215966

**Docs Changes**:
N/A

**Release Note**:
N/A

Signed-off-by: Shizuo Fujita <fujita@clear-code.com>
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@Watson1978 Watson1978 added this to the v1.19.4 milestone Aug 9, 2026
@Watson1978 Watson1978 added the CI Test/CI issues label Aug 9, 2026
@Watson1978
Watson1978 merged commit e45d249 into v1.19 Aug 9, 2026
21 checks passed
@Watson1978
Watson1978 deleted the backport-to-v1.19/pr5461 branch August 9, 2026 03:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI Test/CI issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant