Skip to content

Add "distribute release --timeout <seconds>" option for loading release id.#1948

Merged
AnatolyPristensky merged 5 commits intomicrosoft:masterfrom
KojiNakamaru:feature/distribute_release_timeout
May 18, 2022
Merged

Add "distribute release --timeout <seconds>" option for loading release id.#1948
AnatolyPristensky merged 5 commits intomicrosoft:masterfrom
KojiNakamaru:feature/distribute_release_timeout

Conversation

@KojiNakamaru
Copy link
Copy Markdown
Contributor

Add "distribute release --timeout " option for loading release id.

The uploading with distribute release command internally performs multiple steps found in https://docs.microsoft.com/en-us/appcenter/distribution/uploading . Most steps are performed without any issue, but for large ipa/apk files the step 6

  1. Once uploaded, there is a short delay before the upload is marked as finished. Poll for this status to get the $RELEASE_ID for the next step:

may take several hours regardless of the success or failure (further details can be found in the discussion with App Center Support: Distribute Issue #KVM78S6P).

This pull request allows to specify a timeout for loadReleaseIdUntilSuccess (429fa00). With this option, a user can stop appcenter-cli in reasonable time and restart it if necessary. The pull request also contains other minor refinements (0ab83dd, 0846e65).

@DmitriyKirakosyan
Copy link
Copy Markdown
Contributor

Hi @KojiNakamaru , Thank you for your contribution!
I'll check it asap.

Comment thread src/commands/distribute/release.ts Outdated
Comment thread src/commands/distribute/release.ts Outdated
Comment thread src/commands/distribute/release.ts Outdated
@DmitriyKirakosyan
Copy link
Copy Markdown
Contributor

Hi @KojiNakamaru , I left a few comments to your changes.

@KojiNakamaru
Copy link
Copy Markdown
Contributor Author

@DmitriyKirakosyan Thank you, I reflected your comments (194d69a).

@AnatolyPristensky
Copy link
Copy Markdown
Contributor

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@AnatolyPristensky
Copy link
Copy Markdown
Contributor

Hello @KojiNakamaru, I see that CI Pipeline is failed with the following error message:
/home/vsts/work/1/s/src/commands/distribute/release.ts
278:13 error Missing radix parameter radix
456:50 error Missing radix parameter radix

Could you please have a look at it?

@KojiNakamaru
Copy link
Copy Markdown
Contributor Author

@AnatolyPristensky Thank you, I specified radix 10 again (40c6666, cf. #1948 (comment) ).

@AnatolyPristensky
Copy link
Copy Markdown
Contributor

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@AnatolyPristensky AnatolyPristensky merged commit 6dad658 into microsoft:master May 18, 2022
@KojiNakamaru KojiNakamaru deleted the feature/distribute_release_timeout branch May 18, 2022 09:36
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