Keep CRR Apply Actions row statuses in sync with the running mutations - #1250
Conversation
Hello hervedombya,My role is to assist you with the merge of this Available options
Available commands
Status report is not available. |
Waiting for approvalThe following approvals are needed before I can proceed with the merge:
Peer approvals must include at least 1 approval from the following list: |
|
LGTM |
3306a9a to
b51c1ec
Compare
|
LGTM |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## development/4 #1250 +/- ##
=================================================
- Coverage 63.79% 63.76% -0.03%
=================================================
Files 256 256
Lines 9454 9459 +5
Branches 2559 2561 +2
=================================================
+ Hits 6031 6032 +1
- Misses 3412 3416 +4
Partials 11 11 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
|
/approve |
In the queueThe changeset has received all authorizations and has been added to the The changeset will be merged in:
The following branches will NOT be impacted:
There is no action required on your side. You will be notified here once IMPORTANT Please do not attempt to modify this pull request.
If you need this pull request to be removed from the queue, please contact a The following options are set: approve |
|
I have successfully merged the changeset of this pull request
The following branches have NOT changed:
Please check the status of the associated issue ARTESCA-16787. Goodbye hervedombya. |
The CRR provisioning wizard's Apply Actions table showed every row stuck on "Pending", even after the underlying operations had already succeeded or failed — so the user got no feedback and the run looked frozen.
Each row's status is derived from the mutation instances handed to
useChainedMutations, but the config holding them was memoized without depending on those instances. React Query returns a new result object on every status change, so the memo kept the original idle instances and the rows never moved off "Pending".Depending on the mutation instances in that memo — the approach the ISV Apply Actions step already uses — rebuilds the config whenever a status changes, so rows now reflect live Success / Failed / Retry. The chain is not restarted:
useChainedMutationskeys its reset on the step ids, not the array identity.Verified against a live ARTESCA platform: the rows now update in real time through the run.