Skip to content

[build] Include legal files in Fluss server image (release-1.0) - #4428

Open
wuchong wants to merge 1 commit into
apache:release-1.0from
wuchong:codex/fix-docker-license-files-1.0
Open

wuchong wants to merge 1 commit into
apache:release-1.0from
wuchong:codex/fix-docker-license-files-1.0

Conversation

@wuchong

@wuchong wuchong commented Sep 18, 2026

Copy link
Copy Markdown
Member

Purpose

Linked issue: #4426

Backport of #4427 to release-1.0. The Fluss server Docker image is built from build-target, which can be regenerated without LICENSE or NOTICE during the RC release process.

Brief change log

  • Add LICENSE-bin and NOTICE-bin to the binary assembly as LICENSE and NOTICE.
  • Remove the release script's post-package copy, so the assembly is the single source of truth.
  • Compare the generated legal files byte-for-byte in CI and smoke-test their presence in the server image.
  • Include the legal files in the workflow trigger and image cache key.
  • Document the same content checks in the release guide.

Tests

  • ./mvnw -B --no-transfer-progress clean package -pl fluss-dist -am -DskipTests
  • cmp LICENSE-bin build-target/LICENSE
  • cmp NOTICE-bin build-target/NOTICE
  • Verified the generated tar contains fluss-1.0-SNAPSHOT/LICENSE and fluss-1.0-SNAPSHOT/NOTICE.
  • Built an offline verification image from the locally cached RC3 base and verified both container files and their SHA-256 hashes. The direct Dockerfile build could not refresh the base image because the local Docker registry mirror returned EOF; CI will exercise the normal build.
  • bash -n tools/releasing/create_binary_release.sh
  • Parsed .github/workflows/client-integration.yml as YAML.

API and Format

No API or storage format changes.

Documentation

The release guide now requires byte-for-byte comparison of the generated legal files before building the server image.

Add LICENSE-bin and NOTICE-bin to the binary assembly so clean Maven builds and downstream Docker images retain the required legal files. Verify their contents and presence in CI, and document the release check.\n\nFixes apache#4426\n\nCo-Authored-By: Codex <noreply@openai.com>\nAI-Model: gpt-5\nAI-Contributed/Feature: 35/35\nAI-Contributed/UT: 0/0
AI-Contributed/Feature: 35/35
AI-Contributed/UT: 0/0
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant