search

flomesh-io / fsm

Lightweight service mesh for Kubernetes East-West and North-South traffic management, uses ebpf for layer4 and pipy proxy for layer7 traffic management, support multi cluster network.
https://flomesh.io
Apache License 2.0
55 stars 8 forks source link

fix: cloud connector issuses #175

Closed cybwan closed 8 months ago

cybwan commented 8 months ago

Description:

  1. add check info with consul registration.
  2. auto create check service for consul service.
  3. use fgw's cluster ip as via gateway for local k8s services.
  4. sync with fsm gateway service name's change.
  5. fix mixed service access mode.
  6. fix access controll.
    Affected area: Functional Area
    New Functionality [X]
    CI System [ ]
    CLI Tool [ ]
    Certificate Management [ ]
    Control Plane [ ]
    Demo [ ]
    Documentation [ ]
    Egress [ ]
    Ingress [ ]
    Install [ ]
    Networking [ ]
    Observability [ ]
    Performance [ ]
    SMI Policy [ ]
    Security [ ]
    Sidecar Injection [ ]
    Tests [ ]
    Upgrade [ ]
    Other [X]

    Please answer the following questions with yes/no.

    1. Does this change contain code from or inspired by another project? no

    2. Is this a breaking change? no

    3. Has documentation corresponding to this change been updated in the fsm-docs repo (if applicable)? no

codecov-commenter commented 8 months ago

Codecov Report

Attention: 319 lines in your changes are missing coverage. Please review.

Comparison is base (1eeee08) 47.58% compared to head (964f4ff) 46.78%. Report is 3 commits behind head on release/v1.2.

Files Patch % Lines
cmd/cli/connector_enable.go 6.71% 235 Missing and 1 partial :warning:
cmd/cli/connector_disable.go 30.00% 34 Missing and 1 partial :warning:
cmd/cli/util.go 0.00% 11 Missing :warning:
pkg/providers/kube/client.go 64.51% 11 Missing :warning:
cmd/cli/egressgateway_enable.go 0.00% 9 Missing :warning:
pkg/sidecar/providers/pipy/registry/services.go 0.00% 7 Missing :warning:
cmd/cli/flb_enable.go 0.00% 3 Missing :warning:
cmd/cli/ingress_enable.go 0.00% 3 Missing :warning:
pkg/k8s/client.go 0.00% 3 Missing :warning:
pkg/catalog/access_control.go 0.00% 1 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## release/v1.2 #175 +/- ## ================================================ - Coverage 47.58% 46.78% -0.81% ================================================ Files 196 199 +3 Lines 14884 15229 +345 ================================================ + Hits 7083 7125 +42 - Misses 7757 8058 +301 - Partials 44 46 +2 ``` | [Flag](https://app.codecov.io/gh/flomesh-io/fsm/pull/175/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=None) | Coverage Δ | | |---|---|---| | [unittests](https://app.codecov.io/gh/flomesh-io/fsm/pull/175/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=None) | `46.78% <16.92%> (-0.81%)` | :arrow_down: | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=None#carryforward-flags-in-the-pull-request-comment) to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.