Skip to content

ACNA4315: ISV Install retry - changed from just fetch http calls to f…#122

Merged
elan-llm merged 1 commit intoadobe:masterfrom
elan-llm:ISVInstallRetry
Mar 12, 2026
Merged

ACNA4315: ISV Install retry - changed from just fetch http calls to f…#122
elan-llm merged 1 commit intoadobe:masterfrom
elan-llm:ISVInstallRetry

Conversation

@elan-llm
Copy link
Copy Markdown
Contributor

@elan-llm elan-llm commented Mar 10, 2026

JIRA: https://jira.corp.adobe.com/browse/ACNA-4315?filter=-1

Root cause: The SDK was using createFetch() from @adobe/aio-lib-core-networking - a plain fetch with proxy support but no retry. Any transient 5XX from Console API (in this case a 504 originating from Events service) failed the entire install immediately.

Fix: Replacing createFetch() with HttpExponentialBackoff from the same @adobe/aio-lib-core-networking library. This adds automatic retry with exponential backoff (3 attempts, 100ms/200ms/400ms) on all 5XX and 429 responses. All SDK methods get retry automatically - no changes needed in the provisioner service itself.

Depends: Provisioner's aio-lib-console version has to bumped to ^5.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • I have signed the Adobe Open Source CLA.
  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

@codecov
Copy link
Copy Markdown

codecov bot commented Mar 11, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@elan-llm elan-llm merged commit 2a6c767 into adobe:master Mar 12, 2026
11 checks passed
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.

4 participants