Skip to content

Added fail-fast=false to troubleshoot publish layer GHA - #74

Merged
jerrytfleung merged 1 commit into
swofrom
troubleshooting
Jul 17, 2026
Merged

Added fail-fast=false to troubleshoot publish layer GHA#74
jerrytfleung merged 1 commit into
swofrom
troubleshooting

Conversation

@jerrytfleung

Copy link
Copy Markdown

No description provided.

Copilot AI review requested due to automatic review settings July 17, 2026 20:21

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Updates the staging collector layer release GitHub Actions workflow to adjust matrix “fail-fast” behavior as part of troubleshooting layer publishing.

Changes:

  • Removes strategy.fail-fast: false from the build-layer job’s matrix strategy.
  • Adds strategy.fail-fast: false to the publish-layer job’s matrix strategy.
Comments suppressed due to low confidence (1)

.github/workflows/release-layer-staging-collector.yml:21

  • The PR title indicates adding fail-fast: false for troubleshooting, but this change also removes it from the build-layer matrix. That re-enables the default fail-fast behavior (cancelling the other architecture build on first failure), which can make troubleshooting harder and differs from the prior workflow behavior. If the intent is only to change publish-layer, consider restoring fail-fast: false here as well (or update the PR title/description to reflect the behavioral change).
    strategy:
      matrix:
        architecture:
          - x86_64
          - arm64

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@jerrytfleung
jerrytfleung merged commit d9aa116 into swo Jul 17, 2026
5 checks passed
@jerrytfleung
jerrytfleung deleted the troubleshooting branch July 17, 2026 20:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants