Thanks to visit codestin.com
Credit goes to github.com

Skip to content
This repository was archived by the owner on Mar 18, 2024. It is now read-only.

fix(pools): move recovery functionality to delete#1237

Merged
azlam-abdulsalam merged 2 commits into
mainfrom
feat/orphaned-pools
Feb 16, 2023
Merged

fix(pools): move recovery functionality to delete#1237
azlam-abdulsalam merged 2 commits into
mainfrom
feat/orphaned-pools

Conversation

@azlam-abdulsalam

Copy link
Copy Markdown
Contributor

fix(pools): move recovery functionality to delete

recovery of oprhaned scratch orgs are moved to the delete command, with an addition of '-o' flag. This will alllow the user to run dur
ing the normal cleanup or out of bound and removes the race conditions which had caused it to be reverted earlier

Checklist

All items have to be completed before a PR is merged

  • Adhere to Contribution Guidelines
  • Updates to Decision Records considered?
  • Updates to documentation at DX@Scale Guide considered?
  • Tested changes?
  • Unit Tests new and existing passing locally?

azlam-abdulsalam and others added 2 commits February 16, 2023 09:51
* feat(pool): recover orphaned scratch orgs

Scratch orgs when created during prepare can be timed out at the client side (aka at the execution
of prepare), however these scratch orgs might get created in the backend and consume an active
limit, but is not recovered by sfpowerscripts upon delete. Users have to manually delete thes orgs
out or wait till these orgs are expired, so that the active limits are back.
 This functionality
uses description filed to determine which scratch orgs were requested by sfpowerscripts and then
proceed to delete during an execution of prepare command before and after the pools are created

* [CodeFactor] Apply fixes

* Apply suggestions from code review

---------

Co-authored-by: codefactor-io <[email protected]>
recovery of oprhaned scratch orgs are moved to the delete command, with an addition of '-o' flag.
This will alllow the user to run dur
@azlam-abdulsalam azlam-abdulsalam merged commit df9e3c0 into main Feb 16, 2023
@azlam-abdulsalam azlam-abdulsalam deleted the feat/orphaned-pools branch February 16, 2023 19:52
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants