Skip to content

Conversation

@packocz
Copy link

@packocz packocz commented May 19, 2025

I tried to make a start at updating the PSG awaiter, which is causing us a lot of headaches. We have large PSGs and update them very frequently. For some reason every now and then some of them just get stuck at "updating" status for several hours and our development pipelines basically stop.

I wonder what would be a better approach:

  • extend SfpPackageInstallationOptions and update all install/deploy commands to support a new argument
  • use an ENV variable

I've had some issues running these tests, though I originally wrote this in the old repository with lerna and there they ran fine. Nevertheless this is a draft only.

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 flxbl-sfp Guide considered?
  • Tested changes?
  • Unit Tests new and existing passing locally?

@packocz packocz force-pushed the feat/psg-awaiter-timeout branch from d3712a3 to d48b09b Compare June 30, 2025 13:42
@packocz packocz marked this pull request as ready for review June 30, 2025 13:42
@packocz
Copy link
Author

packocz commented Jun 30, 2025

I think I'm now ready with this PR. I added some tests that cover the new behaviour as well.
Still working on being able to use this locally, the nature if the awaiter feature is such that it's quite difficult to test in real life scenarios.

@azlam-abdulsalam azlam-abdulsalam self-requested a review July 18, 2025 01: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.

1 participant