Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(create-release): add polling for status of airgap build enabled release #80

Merged
merged 18 commits into from
Mar 24, 2025

Conversation

DexterYan
Copy link
Member

@DexterYan DexterYan commented Mar 19, 2025

For create-release workflow

  1. Add new option timeout-minutes, default is 20 minutes.
  2. Add polling for airgap build status of release, which is promoted to airgap enabled channle
  3. Generate airgap download url and airgap build status for user to monitor

sc-120959

demo

Copy link
Member

@jdewinne jdewinne left a comment

Choose a reason for hiding this comment

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

Also a new version of replicated-lib needs to be added once available.

@@ -11,6 +11,12 @@ export async function actionCreateRelease() {
const releaseVersion = core.getInput('version')
const releaseNotes = core.getInput('release-notes')
const apiEndpoint = core.getInput("replicated-api-endpoint") || process.env.REPLICATED_API_ENDPOINT;
const parsedTimeout = parseInt(core.getInput('timeout-minutes') || '20');
Copy link
Member

Choose a reason for hiding this comment

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

This should also be added to action.yml

Copy link
Member Author

Choose a reason for hiding this comment

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

thank you! I have updated action.yml and README.md


if (resolvedChannel.buildAirgapAutomatically) {
try {
const status = await pollForAirgapReleaseStatus(apiClient, appSlug, resolvedChannel.id, +release.sequence, "built", timeoutMinutes);
Copy link
Member

Choose a reason for hiding this comment

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

This should be configurable by the end-user defining if they want to wait for the Air Gap result or not. Default should be false.

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 idea. I have added wait-for-airgap-build

jdewinne
jdewinne previously approved these changes Mar 20, 2025
@DexterYan
Copy link
Member Author

Here is the demo which shows the create-release worked as we expected.

https://github.com/DexterYan/replicated-airgap-embedded-cluster-deploy-flow/actions/runs/13982591708/job/39150600678

@DexterYan DexterYan requested a review from jdewinne March 21, 2025 01:41
@jdewinne jdewinne merged commit 49b440d into main Mar 24, 2025
13 checks passed
@jdewinne jdewinne deleted the dx/sc-120959/update-create-release branch March 24, 2025 22:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants