[build] Remove Apache Incubator branding after graduation to TLP - #3750
Conversation
There was a problem hiding this comment.
Pull request overview
This PR updates Apache Fluss’ repository content to reflect graduation from the Apache Incubator to a Top-Level Project (TLP), removing Incubator branding across build/release metadata, documentation, website, and client bindings.
Changes:
- Removes Incubator-specific branding and disclaimer text (e.g., DISCLAIMER files, “(Incubating)” labels) across docs, website, Dockerfiles, and Maven module metadata.
- Updates release documentation and release scripts to use TLP processes/paths (single community/PMC vote, dist paths under
/fluss, and artifact naming without-incubating). - Re-enables the Git tag ↔ crate version verification composite action in CI.
Reviewed changes
Copilot reviewed 82 out of 84 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| website/static/img/asf_logo_wide.svg | Adds ASF footer logo asset for the website. |
| website/static/img/apache-incubator.svg | Removes the Apache Incubator logo asset. |
| website/src/pages/index.tsx | Removes “(Incubating)” from homepage hero copy. |
| website/src/pages/downloads.md | Updates downloads page headings/text to drop “(Incubating)”. |
| website/src/css/custom.css | Adjusts footer logo styling to keep ASF logo legible. |
| website/docusaurus.config.ts | Updates site title and footer branding to ASF (not Incubator). |
| website/docs/apis/index.md | Removes “(incubating)” from API docs intro copy. |
| website/community/how-to-release/verifying-a-fluss-shaded-release.md | Updates verification guide for TLP paths/naming and PMC voting language. |
| website/community/how-to-release/verifying-a-fluss-release.md | Updates verification guide for TLP dist paths/naming and PMC voting language. |
| website/community/how-to-release/release-manager-preparation.md | Updates release-manager guidance from PPMC/incubator to PMC/TLP. |
| website/community/how-to-release/creating-a-fluss-shaded-release.md | Updates shaded release guide for TLP process, paths, and naming. |
| website/community/how-to-release/creating-a-fluss-release.mdx | Updates main release guide for TLP process, paths, and naming. |
| website/community/fluss-team.md | Updates team roster and role descriptions from PPMC to PMC. |
| website/community/become-a-committer.md | Updates committer/PMC terminology and governance text. |
| tools/releasing/create_binary_release.sh | Removes packaging of DISCLAIMER in binary release artifacts. |
| tools/ci/fluss-ci-tools/pom.xml | Drops “(Incubating)” from module name. |
| README.md | Removes “(Incubating)” branding from the repository README. |
| pom.xml | Removes incubator description text and “(Incubating)” project name. |
| NOTICE-bin | Removes “(incubating)” from NOTICE-bin header line. |
| NOTICE | Removes “(incubating)” from NOTICE header line. |
| helm/NOTICE | Removes “(incubating)” from Helm NOTICE header line. |
| helm/DISCLAIMER | Deletes Helm incubator DISCLAIMER. |
| fluss-test-utils/pom.xml | Drops “(Incubating)” from module name. |
| fluss-test-coverage/pom.xml | Drops “(Incubating)” from module name/description. |
| fluss-spark/pom.xml | Drops “(Incubating)” from Spark parent module name. |
| fluss-spark/fluss-spark-ut/pom.xml | Drops “(Incubating)” from Spark UT module name. |
| fluss-spark/fluss-spark-common/pom.xml | Drops “(Incubating)” from Spark common module name. |
| fluss-spark/fluss-spark-3.5/pom.xml | Drops “(Incubating)” from Spark 3.5 module name. |
| fluss-spark/fluss-spark-3.4/pom.xml | Drops “(Incubating)” from Spark 3.4 module name. |
| fluss-server/pom.xml | Drops “(Incubating)” from server module name/description. |
| fluss-rust/website/docs/release/verifying-a-release-candidate.md | Updates fluss-rust RC verification docs for TLP naming/paths/voting. |
| fluss-rust/website/docs/release/create-release.md | Updates fluss-rust release creation docs for TLP naming/paths/voting. |
| fluss-rust/website/docs/index.md | Removes “(incubating)” from fluss-rust docs intro. |
| fluss-rust/scripts/release.sh | Updates fluss-rust release artifact naming (no -incubating). |
| fluss-rust/README.md | Removes “(Incubating)” branding from fluss-rust README. |
| fluss-rust/crates/fluss/src/lib.rs | Removes “(Incubating)” from Rust crate docs. |
| fluss-rust/crates/fluss/README.md | Removes “(Incubating)” from crate README. |
| fluss-rust/crates/fluss/Cargo.toml | Updates crate description to drop “(Incubating)”. |
| fluss-rust/bindings/python/pyproject.toml | Updates Python binding description to drop “(incubating)”. |
| fluss-rust/bindings/python/PYPI_README.md | Updates PyPI README copy to drop “(Incubating)”. |
| fluss-rust/bindings/elixir/README.md | Updates Elixir binding README copy to drop “(Incubating)”. |
| fluss-rust/bindings/elixir/lib/fluss.ex | Updates Elixir module docstring to drop “(Incubating)”. |
| fluss-rust/bindings/cpp/README.md | Updates C++ binding README to drop “(Incubating)”. |
| fluss-rust/.licenserc.yaml | Removes DISCLAIMER from licenser ignore list. |
| fluss-rpc/pom.xml | Drops “(Incubating)” from RPC module name/description. |
| fluss-protogen/pom.xml | Drops “(Incubating)” from protogen module name/description. |
| fluss-protogen/fluss-protogen-tests/pom.xml | Drops “(Incubating)” from protogen tests module name. |
| fluss-protogen/fluss-protogen-maven-plugin/pom.xml | Drops “(Incubating)” from protogen maven plugin module name. |
| fluss-protogen/fluss-protogen-generator/pom.xml | Drops “(Incubating)” from protogen generator module name. |
| fluss-metrics/pom.xml | Drops “(Incubating)” from metrics parent module name. |
| fluss-metrics/fluss-metrics-prometheus/pom.xml | Drops “(Incubating)” from metrics Prometheus module name. |
| fluss-metrics/fluss-metrics-jmx/pom.xml | Drops “(Incubating)” from metrics JMX module name. |
| fluss-metrics/fluss-metrics-influxdb/pom.xml | Drops “(Incubating)” from metrics InfluxDB module name. |
| fluss-lake/pom.xml | Drops “(Incubating)” from lake parent module name. |
| fluss-lake/fluss-lake-paimon/pom.xml | Drops “(Incubating)” from lake Paimon module name. |
| fluss-lake/fluss-lake-lance/pom.xml | Drops “(Incubating)” from lake Lance module name. |
| fluss-lake/fluss-lake-iceberg/pom.xml | Drops “(Incubating)” from lake Iceberg module name. |
| fluss-lake/fluss-lake-hudi/pom.xml | Drops “(Incubating)” from lake Hudi module name. |
| fluss-kafka/pom.xml | Drops “(Incubating)” from Kafka compatibility module name. |
| fluss-jmh/pom.xml | Drops “(Incubating)” from JMH module name. |
| fluss-flink/pom.xml | Drops “(Incubating)” from Flink parent module name. |
| fluss-flink/fluss-flink-tiering/pom.xml | Drops “(Incubating)” from Flink tiering module name. |
| fluss-flink/fluss-flink-common/pom.xml | Drops “(Incubating)” from Flink common module name. |
| fluss-flink/fluss-flink-2.2/pom.xml | Drops “(Incubating)” from Flink 2.2 module name. |
| fluss-flink/fluss-flink-1.20/pom.xml | Drops “(Incubating)” from Flink 1.20 module name. |
| fluss-flink/fluss-flink-1.19/pom.xml | Drops “(Incubating)” from Flink 1.19 module name. |
| fluss-flink/fluss-flink-1.18/pom.xml | Drops “(Incubating)” from Flink 1.18 module name. |
| fluss-filesystems/pom.xml | Drops “(Incubating)” from filesystems parent module name. |
| fluss-filesystems/fluss-fs-s3/pom.xml | Drops “(Incubating)” from S3 filesystem module name. |
| fluss-filesystems/fluss-fs-oss/pom.xml | Drops “(Incubating)” from OSS filesystem module name. |
| fluss-filesystems/fluss-fs-obs/pom.xml | Drops “(Incubating)” from OBS filesystem module name. |
| fluss-filesystems/fluss-fs-hdfs/pom.xml | Drops “(Incubating)” from HDFS filesystem module name. |
| fluss-filesystems/fluss-fs-hadoop/pom.xml | Drops “(Incubating)” from Hadoop filesystem module name. |
| fluss-filesystems/fluss-fs-hadoop-shaded/pom.xml | Drops “(Incubating)” from shaded Hadoop filesystem module name. |
| fluss-filesystems/fluss-fs-gs/pom.xml | Drops “(Incubating)” from GCS filesystem module name. |
| fluss-filesystems/fluss-fs-azure/pom.xml | Drops “(Incubating)” from Azure filesystem module name. |
| fluss-docgen/pom.xml | Drops “(Incubating)” from docgen module name. |
| fluss-dist/pom.xml | Drops “(Incubating)” from dist module name. |
| fluss-common/pom.xml | Drops “(Incubating)” from common module name. |
| fluss-client/pom.xml | Drops “(Incubating)” from client module name. |
| docker/quickstart-flink/Dockerfile | Removes incubator disclaimer comments from Dockerfile. |
| docker/fluss/Dockerfile | Removes incubator disclaimer comments from Dockerfile. |
| DISCLAIMER | Deletes the root DISCLAIMER (Incubator disclaimer). |
| .github/actions/verify-tag-version/action.yml | Re-enables tag↔crate version check logic for releases. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| TAG_VERSION="${GITHUB_REF#refs/tags/v}" | ||
| CRATE_VERSION=$(sed -n '/^\[workspace.package\]/,/^\[/p' Cargo.toml | grep '^\s*version\s*=' | head -1 | sed -E 's/.*"([^"]+)".*/\1/') | ||
| base() { echo "$1" | sed -E 's/-rc(\.[0-9]+|[0-9]+)$//'; } |
There was a problem hiding this comment.
It seems we need to adapt this?
There was a problem hiding this comment.
Fixed. The action now reads fluss-rust/Cargo.toml explicitly.
wuchong
left a comment
There was a problem hiding this comment.
Thanks @naivedogger , PR #3764 has now merged the website/TLP cleanup into main. Please rebase this branch onto the latest main and drop the website changes already covered there so this PR does not duplicate or conflict with the merged work. While resolving the overlap, please preserve the useful additions in this branch that go beyond the basic branding cleanup—especially updating the release-guide examples from 0.8.0 to the post-incubation 1.0.0, which will be more useful for future releases, and the order&descritpion of the PMC Chair in Fluss Team page updates.
| ```bash | ||
| RELEASE_VERSION="0.8.0-incubating" | ||
| SHORT_RELEASE_VERSION="0.8" | ||
| RELEASE_VERSION="1.0.0" |
There was a problem hiding this comment.
This guide and its expected artifacts now use 1.0.0, but helm/Chart.yaml still sets both version and appVersion to 0.10.0-incubating, while helm/values.yaml still defaults image.tag to 0.10.0-incubating. With the current tree, helm package produces fluss-0.10.0-incubating.tgz, but create_helm_packages.sh subsequently tries to sign fluss-1.0.0.tgz, so release creation fails; installing the chart also deploys the old image by default. Please update the chart version to 1.0.0 and align appVersion and image.tag with the actual Docker release tag (1.0 or 1.0.0).
There was a problem hiding this comment.
Fixed. The Helm chart version, appVersion, and default image tag are now aligned to 1.0.0.
| src: "/img/apache-incubator.svg", | ||
| href: "https://incubator.apache.org/", | ||
| alt: "Apache Incubator logo" | ||
| src: "/img/asf_logo_wide.svg", |
There was a problem hiding this comment.
I would suggest to remove the ASF logo in the footer, this is not required for a TLP and it makes the footer too large.
There was a problem hiding this comment.
Fixed. This PR no longer adds the ASF logo.
Fluss has graduated from the Apache Incubator to a Top-Level Project. Remove incubation branding across the repository: - Delete DISCLAIMER and helm/DISCLAIMER; drop the DISCLAIMER copy step in the binary release script and the fluss-rust license-check ignore entry. - Drop '(Incubating)' from all Maven POM names and remove the incubation disclaimer paragraphs from module descriptions. - Remove '(incubating)' from NOTICE, NOTICE-bin and helm/NOTICE. - Update README, both Dockerfiles, and the fluss-rust sources/bindings. - Website: remove the incubation footer disclaimer, replace the Apache Incubator footer logo with the ASF logo (rendered white on the dark footer via CSS), and update page copy. - Community: rename PPMC to PMC in become-a-committer. Version strings, historical release/download links, the fluss-team roster (mentor/PMC composition), and release-process docs are intentionally left for release/INFRA/PMC follow-up.
Follow-up to da55dee: complete the remaining Incubator removal items now that Fluss has graduated to a Top-Level Project. - Update the PMC roster in fluss-team.md to the board-approved 14-member PMC: Jark Wu as Chair, drop Mentor/PPMC role definitions, add PMC Chair / PMC Member / Committer definitions. - Convert all release guides to TLP form (how-to-release/*, fluss-rust release docs, release.sh): replace two-phase Incubator PMC vote with a single community/PMC vote, remove IPMC / general@incubator / PPMC / Incubator Release Checklist references, update dist paths from /incubator/fluss to /fluss. - Drop "-incubating" from artifact naming; use not-yet-released future versions (Fluss 1.0.0, fluss-shaded 1.0, fluss-rust 1.0.0) as examples to avoid fabricating artifact names that never existed. - Re-enable the tag/crate version check in verify-tag-version action (disabled when release tags carried "-incubating"; fluss-rust is now at 1.0.0 without the suffix). Download-page links, KEYS URLs, and version strings pointing at the currently published incubating artifacts are intentionally left until the first TLP release and the corresponding INFRA dist-directory setup.
|
addition to the comment of @wuchong: sorry for hijacking parts of your PR @naivedogger, i somehow missed it 🤦🏻♂️ i'll also adapt the description of your issue and remove the points that are already done thanks 🤓 |
37b02a9 to
82a0b3f
Compare
|
Thanks @wuchong for the comments. Rebased onto the latest main and removed changes already covered by #3764. The 1.0.0 release examples and PMC Chair updates are preserved, PTAL. @michaelkoepf No worries, thanks! I have rebased the PR and removed the overlapping changes. |
wuchong
left a comment
There was a problem hiding this comment.
Thanks @naivedogger , it looks good to me in general.
Just one question, it seems we left the release.sh and release documentation under fluss-rust untouched, is there a plan to update them in follow-up PRs?
|
@wuchong Yes, I plan to address this in a follow-up PR. |
|
@naivedogger The ci fails. PTAL |
Closes #3749
Removes Apache Incubator branding now that Fluss has graduated to a TLP: