futurewei-cloud / alcor

Alcor: Cloud native SDN platform powered by Kubernetes and Istio
MIT License
32 stars 33 forks source link

Fix data plane manager to support goal state v2 #699

Closed DavidLiu506 closed 2 years ago

DavidLiu506 commented 2 years ago
  1. Add application properties to dpm_manager.yaml.
  2. Data plane manger using grpc client to send goal state in GoalStateV2.
  3. Patch subnet state and router state for goal state v2.
  4. DPM process router configuration function using goal state v2.
  5. Allow empty routing rule for route state creation.
lgtm-com[bot] commented 2 years ago

This pull request introduces 3 alerts when merging 15a60e9c464fbf2a67885eb60d583e9e42b88ca2 into d040632057f4d09a2268503c48398d134d7cabcd - view on LGTM.com

new alerts:

lgtm-com[bot] commented 2 years ago

This pull request introduces 3 alerts when merging dcbe16c40e326237babe5c5949e4d9fb14c93493 into d040632057f4d09a2268503c48398d134d7cabcd - view on LGTM.com

new alerts:

lgtm-com[bot] commented 2 years ago

This pull request introduces 3 alerts when merging 4b5428eb572295b7007a3768069658bcfc64f6f7 into d040632057f4d09a2268503c48398d134d7cabcd - view on LGTM.com

new alerts:

lgtm-com[bot] commented 2 years ago

This pull request introduces 3 alerts when merging 3f98c400c24e67f1bfd4672e98beaf25733cbfd9 into d040632057f4d09a2268503c48398d134d7cabcd - view on LGTM.com

new alerts:

lgtm-com[bot] commented 2 years ago

This pull request introduces 3 alerts when merging ac1cab55d927f3268697539dd0ddf624f7a77049 into d040632057f4d09a2268503c48398d134d7cabcd - view on LGTM.com

new alerts:

cj-chung commented 2 years ago

LGTM if @DavidLiu506 resolves the comments.

lgtm-com[bot] commented 2 years ago

This pull request introduces 3 alerts when merging fd7b860e751acb8865612c3a1df29bc295b34e37 into d040632057f4d09a2268503c48398d134d7cabcd - view on LGTM.com

new alerts: