Skip to content

Site-workflow: add workflow_call trigger#3520

Merged
snazy merged 4 commits intoapache:mainfrom
snazy:site-callable
Jan 29, 2026
Merged

Site-workflow: add workflow_call trigger#3520
snazy merged 4 commits intoapache:mainfrom
snazy:site-callable

Conversation

@snazy
Copy link
Member

@snazy snazy commented Jan 23, 2026

This trigger allows the Hugo Site publication workflow to be called from the versioned-docs branch.

This change also introduces a concurrency group to prevent concurrent site publications.

Fixes #3516

This trigger allows the Hugo Site publication workflow to be called from the `versioned-docs` branch.

This change also introduces a concurrency group to prevent concurrent site publications.

Fixes apache#3516
@snazy snazy marked this pull request as ready for review January 24, 2026 08:41
dimas-b
dimas-b previously approved these changes Jan 26, 2026
@github-project-automation github-project-automation bot moved this from PRs In Progress to Ready to merge in Basic Kanban Board Jan 26, 2026
@snazy
Copy link
Member Author

snazy commented Jan 27, 2026

Thanks for the approvals!
I still want to double-check the approach in a toy repo before I merge the PRs.


- name: Deploy Static Site to GitHub
if: (github.event_name == 'push' && github.repository == 'apache/polaris') || github.event_name == 'workflow_call'
if: github.event_name == 'push' && github.repository == 'apache/polaris'
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The github context is populated with the information of the calling workflow

@snazy snazy requested a review from dimas-b January 28, 2026 16:28
# Ensure that no two site publications happen concurrently.
# 1 job per reference can run concurrently, no cancellation
concurrency:
group: ${{ github.ref }}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So, github.ref could be refs/heads/versioned-docs (when called programmatically) or main... and those jobs can still run concurrently? Is that expected?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good point - updated this

Co-authored-by: Dmitri Bourlatchkov <[email protected]>
@snazy snazy requested a review from dimas-b January 28, 2026 16:34
@snazy snazy merged commit 5ed1010 into apache:main Jan 29, 2026
15 checks passed
@github-project-automation github-project-automation bot moved this from Ready to merge to Done in Basic Kanban Board Jan 29, 2026
@snazy snazy deleted the site-callable branch January 29, 2026 14:41
snazy added a commit to snazy/polaris that referenced this pull request Feb 2, 2026
The change apache#3520 added a concurrency group, accidentally a _static_ concurrency group which applies to both pushed to "main" and all PRs, which is wrong.
sungwy pushed a commit to sungwy/polaris that referenced this pull request Feb 7, 2026
This trigger allows the Hugo Site publication workflow to be called from the `versioned-docs` branch.

This change also introduces a concurrency group to prevent concurrent site publications.

Contributed to apache#3516

Co-authored-by: Dmitri Bourlatchkov <[email protected]>
snazy added a commit to snazy/polaris that referenced this pull request Feb 11, 2026
* Nit: fix shell-check warnings for await-s3.sh (apache#3593)

* Nit: add upgrade notes for develocity plugin (apache#3599)

* fix(deps): update dependency org.apache.hive:hive-exec to v2.3.10 (apache#3580)

* fix(deps): update dependency com.google.cloud:google-cloud-storage-bom to v2.62.1 (apache#3600)

* Change quick start example from MinIO to RustFS backend (apache#3586)

* chore(deps): update gradle/actions digest to f29f5a9 (apache#3602)

* Simplify getting started example for Ceph (apache#3587)

* Simplify getting started example for Ceph

* Simplify getting started example for Ceph

* fix(deps): update dependency io.smallrye.config:smallrye-config-core to v3.16.0 (apache#3604)

* chore(deps): update docker.io/mongo docker tag to v8.2.4 (apache#3606)

* fix(deps): update dependency net.ltgt.gradle:gradle-errorprone-plugin to v5 (apache#3605)

* chore(deps): update plugin com.gradle.develocity to v4.3.2 (apache#3598)

* chore(deps): update localstack/localstack docker tag to v4.13 (apache#3609)

* Guides: use `sql` code block (apache#3590)

Replaces the "anonymous" code blocks with `sql` code blocks, which is easier to "copy + paste" for users.

* Spark guide: document the launch command in the Markdown (apache#3592)

Replaces the "invisible" launch-docker.sh script with a "visible" `shell` code block, so the commands are immediately visible to users.

* Minio guide: fix service dependencies (apache#3594)

Use the "long" options for compose service dependencies, and adds the explicit option `restart: no` option for setup tasks, which is necessary to let docker-compose correctly interpret the termination of such tasks.

* Ozone guide: Fix service dependencies (apache#3595)

Use the "long" options for compose service dependencies, add healthchecks, uses more healthcheck-retries for slow-ish machines, and adds the explicit option `restart: no` option for setup tasks, which is necessary to let docker-compose correctly interpret the termination of such tasks.

* Rustfs guide: fix service dependencies (apache#3596)

Just adds a necessary service dependency

* Telemetry guide: fix setup-polaris service (apache#3597)

Fixes the polaris-setup service to use the "working" way/syntax, in alignment to other guides.

* Site-workflow: add `workflow_call` trigger (apache#3520)

This trigger allows the Hugo Site publication workflow to be called from the `versioned-docs` branch.

This change also introduces a concurrency group to prevent concurrent site publications.

Contributed to apache#3516

Co-authored-by: Dmitri Bourlatchkov <[email protected]>

* [Issue 3209] Surface Intermediate Data as part of Polaris Events (apache#3456)

Adds a RequestScoped annotation to AttributeMap, which allows us to inject this bean into intermediary business logic and be able to patch data from there back out to the PolarisEvent.

@dimas-b had a great idea that we may be able to retire the Delegator model eventually by setting a "closing" hook on the RequestScoped bean - but for simplicity, I am introducing this smaller change that will solve the GH issue linked first and will investigate that approach later.

* Getting-Started: Jaeger upgraded from V1 to V2 (apache#3603)

* Jaeger V2

* Jaeger V2

* Update Gradle to 9.3.1 (apache#3615)

* Last merged commit 90ba09c

---------

Co-authored-by: Mend Renovate <[email protected]>
Co-authored-by: Yong Zheng <[email protected]>
Co-authored-by: Dmitri Bourlatchkov <[email protected]>
Co-authored-by: Adnan Hemani <[email protected]>
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.

Eagerly trigger website publication when pushing to the versioned-docs branch

2 participants