Skip to content

Generate release vote e-mail as Github Step Summary#3150

Merged
pingtimeout merged 4 commits intoapache:mainfrom
pingtimeout:release-workflow-email
Dec 23, 2025
Merged

Generate release vote e-mail as Github Step Summary#3150
pingtimeout merged 4 commits intoapache:mainfrom
pingtimeout:release-workflow-email

Conversation

@pingtimeout
Copy link
Contributor

This fixes #3079. See the last job of this https://github.com/pingtimeout/polaris/actions/runs/19641289254 workflow for a demo (screenshot attached)

image

@pingtimeout
Copy link
Contributor Author

Thanks @snazy , it should be good now. See this run for an example : https://github.com/pingtimeout/polaris/actions/runs/20437060916. Now that it was run with dry-run mode so there was no staging repo created and the ID is missing from the URL. But the URL matches those used in previous releases so that should be good. Screenshot for posterity.

image

@github-project-automation github-project-automation bot moved this from PRs In Progress to Ready to merge in Basic Kanban Board Dec 23, 2025
@pingtimeout pingtimeout merged commit 4b3d4d1 into apache:main Dec 23, 2025
15 checks passed
@pingtimeout pingtimeout deleted the release-workflow-email branch December 23, 2025 14:23
@github-project-automation github-project-automation bot moved this from Ready to merge to Done in Basic Kanban Board Dec 23, 2025
pingtimeout added a commit to pingtimeout/polaris that referenced this pull request Dec 24, 2025
Now that apache#3150 has been merged, a VOTE e-mail is pre-generated by the
third workflow so that there is less room for error.  The release guide
has been updated to reflect this.

There is no pre-generated Incubator vote e-mail as part of the third
workflow.  This is acceptable in that eventually, after graduation, that
would become unnecessary.  But also that it could cause confusion as
there would be two proposed e-mail bodies in the third workflow.
Recommendation is to let the release manager created the Incubator vote
e-mail manually.
pingtimeout added a commit to pingtimeout/polaris that referenced this pull request Dec 24, 2025
Now that apache#3150 has been merged, a VOTE e-mail is pre-generated by the
third workflow so that there is less room for error.  The release guide
has been updated to reflect this.

There is no pre-generated Incubator vote e-mail as part of the third
workflow.  This is acceptable in that eventually, after graduation, that
would become unnecessary.  But also that it could cause confusion as
there would be two proposed e-mail bodies in the third workflow.
Recommendation is to let the release manager created the Incubator vote
e-mail manually.
pingtimeout added a commit that referenced this pull request Jan 8, 2026
Now that #3150 has been merged, a VOTE e-mail is pre-generated by the
third workflow so that there is less room for error.  The release guide
has been updated to reflect this.

There is no pre-generated Incubator vote e-mail as part of the third
workflow.  This is acceptable in that eventually, after graduation, that
would become unnecessary.  But also that it could cause confusion as
there would be two proposed e-mail bodies in the third workflow.
Recommendation is to let the release manager created the Incubator vote
e-mail manually.
evindj pushed a commit to evindj/polaris that referenced this pull request Jan 26, 2026
evindj pushed a commit to evindj/polaris that referenced this pull request Jan 26, 2026
Now that apache#3150 has been merged, a VOTE e-mail is pre-generated by the
third workflow so that there is less room for error.  The release guide
has been updated to reflect this.

There is no pre-generated Incubator vote e-mail as part of the third
workflow.  This is acceptable in that eventually, after graduation, that
would become unnecessary.  But also that it could cause confusion as
there would be two proposed e-mail bodies in the third workflow.
Recommendation is to let the release manager created the Incubator vote
e-mail manually.
snazy added a commit to snazy/polaris that referenced this pull request Feb 11, 2026
* Disable renovate bot for openapi generator cli (apache#3306)

* Fix openapi-generator-cli version in build system

* Fix openapi-generator-cli version in build system

* Build: Ensure reproducible .properties files (apache#3089)

This is a safety net in case Properties are generated anywhere during the build.

* Generate release vote e-mail as Github Step Summary (apache#3150)

* Improve error handling in quickstart setup script (apache#3288)

Add error detection and validation to API calls in the quickstart
docker-compose setup with clear error messages for easier debugging.

* (doc): Fix tools doc and add tool doc for mcp server (apache#3311)

* chore(deps): update actions/stale digest to a21a081 (apache#3326)

* Add cancel release candidate workflow (apache#3321)

This commit fixes apache#3080

* fix(deps): update quarkus platform and group to v3.30.5 (apache#3329)

* chore(deps): update actions/checkout digest to 8e8c483 (apache#3319)

* fix(deps): update dependency org.agrona:agrona to v2.4.0 (apache#3333)

* (feat): Helm: add priority class name support for helm (apache#3310)

* Add priority class name support for helm

* Update changelog

* Update default port for metrics from 8282 to 8182 (apache#3335)

Corrects the default metrics port from 8282 to 8182 and clarifies the available metrics endpoints in the documentation.

* fix(deps): update dependency software.amazon.awssdk:bom to v2.40.16 (apache#3336)

* Add KMS options to catalogs create CLI (apache#3330)

* Add Polaris blog about KMS (apache#3331)

* Add Polaris blog about KMS

Following up on apache#2802

* Use mainstream PostgreSQLContainer (apache#3345)

Migrate from the deprecated PostgreSQLContainer class to the
mainstream one as suggested by javadoc.

* fix(deps): update dependency org.apache.iceberg:iceberg-bom to v1.10.1 (apache#3317)

* Enforce mypy for CLI (apache#3305)

* fix(deps): update dependency software.amazon.awssdk:bom to v2.41.1 (apache#3351)

* Last merged commit e75eb4b

---------

Co-authored-by: Yong Zheng <yongzheng0809@gmail.com>
Co-authored-by: Pierre Laporte <pierre@pingtimeout.fr>
Co-authored-by: Tamas Mate <50709850+tmater@users.noreply.github.com>
Co-authored-by: Mend Renovate <bot@renovateapp.com>
Co-authored-by: rishii-19-works <rishipandey1912@gmail.com>
Co-authored-by: Dmitri Bourlatchkov <dmitri.bourlatchkov@gmail.com>
snazy added a commit to snazy/polaris that referenced this pull request Feb 11, 2026
* NoSQL: reduce heap pressure when running tests (apache#3267)

Some tests generate a lot of realms, likely one realm per test case. While the amount of data per realm is not much, it is nontheless nice to remove that data immediately (for tests).

The maintenance service, which purges data of eligible realms, cannot be run against the in-memory backend (different JVM).

This change adds a rather "test only" workaround to purge the realm data in the in-memory backend immediately.

* fix(deps): update dependency org.projectnessie.cel:cel-bom to v0.6.0 (apache#3356)

* chore: Suppress unchecked case from mock (apache#3322)

This is to fix javac `warning: [unchecked] unchecked conversion TriConsumer<String, TableIdentifier, MetricsReport> mockConsumer = mock(TriConsumer.class);`

* Add Polaris Console on the tools set (apache#3355)

* Move remaining build-time server properties to runtime/defaults (apache#3341)

The `runtime/defaults` module is meant as a holder of default
Polaris Server runtime properties.

This change moves a few remaining properties from the `application.properties`
file under `runtime/server` into `runtime/defaults` to avoid any possible
confusion regarding the location of effective server properties.

Note: The Admin tool has its own `application.properties` file.

* Deprecate untyped `RealmConfig.getConfig()` (apache#3323)

* `getConfig(String)` has a generic return type, but the call path that gets the value does not perform any type validation.

* Deprecate this method in favour of well-typed `getConfig(PolarisConfiguration)`

* Migrate the single use case in Polaris code to the well-typed method.

* fix(deps): update dependency io.smallrye.common:smallrye-common-annotation to v2.15.0 (apache#3365)

* Fix error handler parameters in TaskExecutorImpl (apache#3358)

Use correct exception variable inside `CompletableFuture.exceptionallyComposeAsync()`

* fix(deps): update dependency org.junit:junit-bom to v5.14.2 (apache#3363)

* fix(deps): update dependency com.gradleup.shadow:shadow-gradle-plugin to v9.3.1 (apache#3361)

* Propagate previous task exceptions as "suppressed" (apache#3367)

* Propagate previous task exceptions as "suppressed"

Task retries may fail in different ways in each attempt, however
only the last exception used to be exposed to the caller.

This change propagates exceptions from all previous tasks execution
attempts as "suppressed" exceptions chained to the final tasks
failure exception.

* Introduce idempotency records schema and Postgres-backed IdempotencyStore (apache#3205)

This PR adds the persistence foundation for REST idempotency in Polaris:
Defines an idempotency_records table (Postgres) with key, binding, liveness, and finalization fields.
Introduces a storage‑agnostic IdempotencyRecord model and IdempotencyStore SPI in polaris-core.
Implements PostgresIdempotencyStore in polaris-relational-jdbc

* fix(deps): update dependency ch.qos.logback:logback-classic to v1.5.24 (apache#3369)

* fix(deps): update immutables to v2.12.1 (apache#3368)

* Remove unnecessary version spec in jdbc persistence build file (apache#3373)

The testcontainers BOM is already included, so the version spec, which doesn't match the BOM version, is unnecessary.

* Fix typo (apache#3376)

* Cosmetic: sort lines in libs.versions.toml (apache#3133)

* fix(deps): update dependency com.github.dasniko:testcontainers-keycloak to v4.1.0 (apache#3375)

* Remove Admin tests from required_status_checks (apache#3370)

* fix(deps): update quarkus platform and group to v3.30.6 (apache#3374)

* [Bug] Fix a bug that causing error when setting `write.data.path` to be a subdirectory of the table location (apache#3371)

Currently, when updating write.data.path of the table to a subdir under the table location, it will fail the location overlap check. For example

spark-sql> ALTER TABLE tb1 SET TBLPROPERTIES (
  'write.data.path' = '<tableLocation>/alternative_data'
);

org.apache.iceberg.exceptions.ForbiddenException: Forbidden: Unable to create table at location 's3://<table_location>' because it conflicts with existing table or namespace at location 's3://<table_location>`
IcebergCatalog.validateNoLocationOverlap(...) constructs a virtual PolarisEntity for overlap checking, but it did not set the entity name. When fetching the siblings of the table, it fails to filter out itself and thus the check mistaken considered that the write.data.path conflict with the table's own base location. (isChildOf)

This PR fix the issue by adding name to the virtual PolarisEntity and add a unit and a integration test.

* Update release guide to reference the proposed vote e-mail (apache#3377)

Now that apache#3150 has been merged, a VOTE e-mail is pre-generated by the
third workflow so that there is less room for error.  The release guide
has been updated to reflect this.

There is no pre-generated Incubator vote e-mail as part of the third
workflow.  This is acceptable in that eventually, after graduation, that
would become unnecessary.  But also that it could cause confusion as
there would be two proposed e-mail bodies in the third workflow.
Recommendation is to let the release manager created the Incubator vote
e-mail manually.

* Last merged commit b324fc3

---------

Co-authored-by: Mend Renovate <bot@renovateapp.com>
Co-authored-by: Dmitri Bourlatchkov <dmitri.bourlatchkov@gmail.com>
Co-authored-by: JB Onofré <jbonofre@apache.org>
Co-authored-by: Huaxin Gao <huaxin.gao11@gmail.com>
Co-authored-by: Yong Zheng <yongzheng0809@gmail.com>
Co-authored-by: Yufei Gu <yufei@apache.org>
Co-authored-by: Honah (Jonas) J. <honahx@apache.org>
Co-authored-by: Pierre Laporte <pierre@pingtimeout.fr>
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.

Releasy: prominently propose RC + release email subject + body

2 participants