Closed stefannica closed 1 week ago
[!IMPORTANT]
Review skipped
Auto reviews are disabled on this repository.
Please check the settings in the CodeRabbit UI or the
.coderabbit.yaml
file in this repository. To trigger a single review, invoke the@coderabbitai review
command.You can disable this status message by setting the
reviews.review_status
tofalse
in the CodeRabbit configuration file.
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
Describe changes
When multiple concurrent requests are made to the service connector REST API endpoints, they sometimes create a race condition that results in the following error:
This has been observed while deploying a Terraform GCP stack with the new ZenML Terraform provider against a freshly deployed ZenML server.
The root cause is the initialization of the service connector type registry is not thread safe. This PR uses a re-entrant lock to fix that.
Pre-requisites
Please ensure you have done the following:
develop
and the open PR is targetingdevelop
. If your branch wasn't based on develop read Contribution guide on rebasing branch to develop.Types of changes