Skip to content

ci: wait for publisher release links artifact#271

Merged
yejseo01 merged 7 commits into
mainfrom
release_polls_for_links
May 27, 2026
Merged

ci: wait for publisher release links artifact#271
yejseo01 merged 7 commits into
mainfrom
release_polls_for_links

Conversation

@yejseo01
Copy link
Copy Markdown
Contributor

Changes

  • Polls topo-publisher for the newly created sign-and-release workflow run on branch/ref v6.
  • Retries run discovery every 10 seconds, up to 30 attempts.
  • Adds a new step that waits for the publisher run to produce the release-links artifact.
  • Polls for release-links every 60 seconds, up to 30 attempts.
  • Downloads the release-links artifact into publisher-artifacts once it exists.
  • Upload retrieved release links artifact for verification

@yejseo01 yejseo01 requested a review from a team as a code owner May 20, 2026 15:15
@yejseo01 yejseo01 marked this pull request as draft May 20, 2026 15:21
@yejseo01
Copy link
Copy Markdown
Contributor Author

There are some comcurrency issues I want to address before it's ready to be reviewed

@yejseo01
Copy link
Copy Markdown
Contributor Author

this needs to be merged first - https://github.com/Arm-Debug/topo-publisher/pull/38/changes

@yejseo01 yejseo01 marked this pull request as ready for review May 21, 2026 08:02
Comment thread .github/workflows/release.yaml Outdated
Comment thread .github/workflows/release.yaml Outdated
Comment thread .github/workflows/release.yaml Outdated
Comment thread .github/workflows/release.yaml Outdated
Comment thread .github/workflows/release.yaml Outdated
Copy link
Copy Markdown
Contributor

@awphi awphi left a comment

Choose a reason for hiding this comment

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

Small edge to patch as discussed but otherwise lgtm!

Comment thread .github/workflows/release.yaml Outdated
@yejseo01 yejseo01 merged commit d452bfc into main May 27, 2026
6 checks passed
@yejseo01 yejseo01 deleted the release_polls_for_links branch May 27, 2026 16:01
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.

3 participants