Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(kuma-cp) resolve status problem without a corner case #2246

Merged
merged 7 commits into from
Jun 30, 2021

Conversation

lobkovilya
Copy link
Contributor

Summary

Solves the same problem as #2245, but doesn't take into account corner case when Dataplane goes down while Control Plane is down.

Full changelog

  • [Implement ...]
  • [Fix ...]

Issues resolved

#1737

Documentation

Testing

  • Unit tests
  • E2E tests
  • Manual testing on Universal
  • Manual testing on Kubernetes

bartsmykla and others added 6 commits June 7, 2021 15:25
This test is showing us what will happen when global zone cp
will die and before it will be back again if remote zone will die
too it should reflect the zone status (offline)

Signed-off-by: Bart Smykla <bartek@smykla.com>
Signed-off-by: Bart Smykla <bartek@smykla.com>
Signed-off-by: Bart Smykla <bartek@smykla.com>
Signed-off-by: Ilya Lobkov <ilya.lobkov@konghq.com>
@lobkovilya lobkovilya requested a review from a team as a code owner June 29, 2021 15:28
Signed-off-by: Ilya Lobkov <ilya.lobkov@konghq.com>
@codecov-commenter
Copy link

Codecov Report

Merging #2246 (4c2b3c8) into master (3596f27) will increase coverage by 0.21%.
The diff coverage is 12.76%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2246      +/-   ##
==========================================
+ Coverage   51.77%   51.99%   +0.21%     
==========================================
  Files         909      914       +5     
  Lines       41004    41134     +130     
==========================================
+ Hits        21229    21386     +157     
+ Misses      17741    17686      -55     
- Partials     2034     2062      +28     
Impacted Files Coverage Δ
...ilience/resilience_multizone_universal_postgres.go 0.00% <0.00%> (ø)
api/mesh/v1alpha1/dataplane_insight_helpers.go 80.24% <100.00%> (+3.62%) ⬆️
api/mesh/v1alpha1/zone_ingress_insight_helpers.go 70.96% <100.00%> (+22.81%) ⬆️
api/system/v1alpha1/zone_insight_helpers.go 65.85% <100.00%> (+17.20%) ⬆️
pkg/kds/client/sink.go 43.58% <0.00%> (-5.13%) ⬇️
...kg/plugins/resources/k8s/native/pkg/test/within.go 100.00% <0.00%> (ø)
api/internal/util/proto/proto.go 62.50% <0.00%> (ø)
...k8s/native/controllers/proxytemplate_controller.go 0.00% <0.00%> (ø)
api/internal/util/proto/types.go 50.00% <0.00%> (ø)
... and 18 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3596f27...4c2b3c8. Read the comment docs.

@lobkovilya lobkovilya merged commit cef8cda into master Jun 30, 2021
@lobkovilya lobkovilya deleted the fix/insight-simple branch June 30, 2021 07:50
mergify bot pushed a commit that referenced this pull request Jun 30, 2021
jakubdyszkiewicz pushed a commit that referenced this pull request Jun 30, 2021
(cherry picked from commit cef8cda)

Co-authored-by: Ilya Lobkov <ilya.lobkov@konghq.com>
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.

None yet

4 participants