Skip to content

Conversation

@Nsttt
Copy link
Member

@Nsttt Nsttt commented Sep 29, 2025

What's added in this PR?

Was added an extra upload strategy to handle multi-CDN configs. Still has a fallback for the previous config endpoint if something goes wrong with the new call.
Clickup

Screenshots

image

What's the issues or discussion related to this PR ?

Polycloud support https://app.clickup.com/t/9013031642/ZC-3503

What are the steps to test this PR?

Configure a multi CDN on app. Set that CDN to an application, then try to build that application.

(Optional) What's left to be done for this PR?

@valeriitropin asked about using the secondaries as fallback for serving the app. That would be a nice feat

(Optional) What's the potential risk and how to mitigate it?

Stop properly deploying app. Need to be careful with that

(Required) Pre-PR/Merge checklist

  • I have added/updated/opened a PR to documentation to cover this new behavior
  • I have added an explanation of my changes
  • I have written new tests (if applicable)
  • I have tested this locally (standing from a first time user point of view, never touch this app before)
  • I have/will run tests, or ask for help to add test

@nx-cloud
Copy link

nx-cloud bot commented Sep 29, 2025

🤖 Nx Cloud AI Fix Eligible

An automatically generated fix could have helped fix failing tasks for this run, but Self-healing CI is disabled for this workspace. Visit workspace settings to enable it and get automatic fixes in future runs.

To disable these notifications, a workspace admin can disable them in workspace settings.


View your CI Pipeline Execution ↗ for commit 5cdd50e

Command Status Duration Result
nx affected -t test ❌ Failed 32s View ↗
nx run e2e-deployment:e2e-test -- --passWithNoT... ✅ Succeeded 18s View ↗
nx affected -t build --skipRemoteCache ✅ Succeeded 4m 47s View ↗
nx affected -t lint ✅ Succeeded 6s View ↗

☁️ Nx Cloud last updated this comment at 2025-09-29 10:02:48 UTC

@fiorin fiorin self-assigned this Nov 24, 2025
@fiorin fiorin marked this pull request as ready for review November 24, 2025 14:34
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