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

optimized fsm connector #171

Closed cybwan closed 8 months ago

cybwan commented 8 months ago

Description:

  1. add vm-sidecar scripts
  2. connector supports nacos.
  3. add connector sub-command to fsm cli.
  4. fix egressgateway sub-command.
  5. fsm connector supports multi vm clusters.
  6. add appendTag support to consul connector.
  7. add appendMetadata support to eureka and nacos connector.
    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 [ ]

    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: 311 lines in your changes are missing coverage. Please review.

Comparison is base (e232369) 47.58% compared to head (c68b0bf) 46.78%. Report is 5 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:
cmd/cli/egressgateway_enable.go 0.00% 9 Missing :warning:
pkg/sidecar/providers/pipy/registry/services.go 0.00% 7 Missing :warning:
pkg/providers/kube/client.go 69.23% 4 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:

: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 #171 +/- ## ================================================ - 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/171/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/171/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=None) | `46.78% <14.79%> (-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.