Skip to content

Automatically retry snapshot publishing - 3 times#14764

Merged
jamesfredley merged 3 commits into7.0.xfrom
retry-snapshot-publish
May 22, 2025
Merged

Automatically retry snapshot publishing - 3 times#14764
jamesfredley merged 3 commits into7.0.xfrom
retry-snapshot-publish

Conversation

@jamesfredley
Copy link
Copy Markdown
Contributor

@jamesfredley jamesfredley commented May 21, 2025

nick-fields/retry is on the ASF approved list

Automatically retries 3 times and reduces time needed to babysit the workflow when repository.apache.org is experiencing read timeouts.

@jamesfredley jamesfredley self-assigned this May 21, 2025
@jamesfredley jamesfredley requested a review from Copilot May 21, 2025 23:08
@jamesfredley jamesfredley moved this to In Progress in Apache Grails May 21, 2025
@jamesfredley jamesfredley added this to the grails:7.0.0-M4 milestone May 21, 2025
Copy link
Copy Markdown
Contributor

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 introduces automatic retry logic for snapshot publishing to repository.apache.org using the nick-fields/retry action.

  • Integrates the retry action replacing the direct gradlew publish command
  • Configures retry parameters such as timeout_seconds, max_attempts, and retry_wait_seconds
Comments suppressed due to low confidence (1)

.github/workflows/gradle.yml:242

  • Verify that the 'command' parameter and the associated numeric parameters (timeout_seconds, max_attempts, retry_wait_seconds) are supported by the nick-fields/retry action and their values meet your infrastructure requirements.
command: ./gradlew publish

@jamesfredley jamesfredley changed the title Automatically retry snapshot publishing to repository.apache.org 3 times Automatically retry snapshot publishing to repository.apache.org - 3 times May 21, 2025
@jamesfredley jamesfredley changed the title Automatically retry snapshot publishing to repository.apache.org - 3 times Automatically retry snapshot publishing - 3 times May 21, 2025
@jamesfredley jamesfredley requested review from jdaugherty and matrei May 21, 2025 23:43
Copy link
Copy Markdown
Contributor

@matrei matrei left a comment

Choose a reason for hiding this comment

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

Should we add a comment explaining which problem we are solving by using the retry action (timeouts from https://repository.apache.org/content/repositories/snapshots)?

@jamesfredley
Copy link
Copy Markdown
Contributor Author

Comment added

# Attempts to address: Could not write to resource 'https://repository.apache.org/content/repositories/snapshots/...' Read timed out

@jamesfredley jamesfredley merged commit 5a4b2fb into 7.0.x May 22, 2025
11 of 25 checks passed
@github-project-automation github-project-automation bot moved this from In Progress to Done in Apache Grails May 22, 2025
@jamesfredley jamesfredley deleted the retry-snapshot-publish branch May 22, 2025 14:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants