openyurtio / openyurt

OpenYurt - Extending your native Kubernetes to edge(project under CNCF)
https://openyurt.io
Apache License 2.0
1.68k stars 391 forks source link

fix: edgex component creation cause registration errors and core-command crash #2030

Closed LavenderQAQ closed 2 months ago

LavenderQAQ commented 2 months ago

What type of PR is this?

/kind bug /sig iot

What this PR does / why we need it:

The default maxSurge value for k8s deployment is 25%, and when replicas is 1, maxSurge is 1 after rounding up. This results in the existence of 2 identical edgex components at the same time after a quick create + update operation, and edgex components use a registration mechanism. If a component preregisters and is terminated by workload, this causes the metadata component to scramble. This chaos manifests itself in the form of other components crashing all the time.

I updated auto-collector to directly inject the default maxSurge value of 0 into the config file and the issue was resolved.

Which issue(s) this PR fixes:

Fixes #2028

LavenderQAQ commented 2 months ago

/rerun

LavenderQAQ commented 2 months ago

/rerun

sonarcloud[bot] commented 2 months ago

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud