Skip to content

Conversation

@wolfboys
Copy link
Member

[Improve] docker-push improvement

Verifying this change

This change is a trivial rework / code cleanup without any test coverage.

(or)

This change is already covered by existing tests, such as (please describe tests).

(or)

This change added tests and can be verified as follows:

Does this pull request potentially affect one of the following parts

  • Dependencies (does it add or upgrade a dependency): (yes / no)

@wolfboys wolfboys requested a review from Copilot September 10, 2025 01:14
@github-actions github-actions bot added the INFRA label Sep 10, 2025

This comment was marked as outdated.

@wolfboys wolfboys requested a review from Copilot September 10, 2025 01:18

This comment was marked as outdated.

@wolfboys wolfboys requested a review from Copilot September 10, 2025 01:21
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR modifies the Docker image build workflow triggers to improve the build and deployment process. The changes replace the release-based trigger with a more flexible tag-based approach and add scheduled builds.

  • Replace release trigger with tag-based triggering for version tags
  • Add scheduled daily builds at 1:40 AM UTC
  • Maintain push triggers for release branches

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@sonarqubecloud
Copy link

Copy link
Contributor

@VampireAchao VampireAchao left a comment

Choose a reason for hiding this comment

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

LGTM

@VampireAchao VampireAchao merged commit 09a1895 into apache:dev Sep 10, 2025
42 checks passed
@wolfboys wolfboys deleted the docker-push branch September 10, 2025 02:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants