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

support spiderpool #809

Merged
merged 3 commits into from
Oct 25, 2023
Merged

support spiderpool #809

merged 3 commits into from
Oct 25, 2023

Conversation

biqiangwu
Copy link
Contributor

@biqiangwu biqiangwu commented Sep 25, 2023

不管是后台定期、还是 egep/egcep 事件,都全量更新网关节点上的路由

@biqiangwu biqiangwu added the pr/not-ready not ready for merging label Sep 25, 2023
@codecov
Copy link

codecov bot commented Sep 25, 2023

Codecov Report

Merging #809 (01d8ed3) into main (ce4ec22) will not change coverage.
Report is 14 commits behind head on main.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #809   +/-   ##
=======================================
  Coverage   39.50%   39.50%           
=======================================
  Files          18       18           
  Lines        2511     2511           
=======================================
  Hits          992      992           
  Misses       1356     1356           
  Partials      163      163           
Flag Coverage Δ
unittests 39.50% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Coverage Δ
pkg/config/config.go 48.14% <ø> (ø)

charts/values.yaml Outdated Show resolved Hide resolved
charts/values.yaml Outdated Show resolved Hide resolved
@weizhoublue
Copy link
Collaborator

weizhoublue commented Sep 26, 2023

(1)spiderpool 场景下的 get started
(2)spiderpool 场景下的 E2E

@biqiangwu biqiangwu force-pushed the feat/underlay_CNI_supports branch 15 times, most recently from fccd6f3 to 7a0c22f Compare October 13, 2023 13:16
@biqiangwu biqiangwu added release/feature-new feature pr recoded in new release and removed pr/not-ready not ready for merging labels Oct 13, 2023
charts/values.yaml Outdated Show resolved Hide resolved
@biqiangwu biqiangwu force-pushed the feat/underlay_CNI_supports branch 2 times, most recently from bbc3441 to eb0c1a2 Compare October 20, 2023 09:30
@biqiangwu
Copy link
Contributor Author

@biqiangwu
Copy link
Contributor Author

(1)spiderpool 场景下的 get started (2)spiderpool 场景下的 E2E

1、已添加
2、可以复用已有的 E2E 用例

pkg/agent/police.go Outdated Show resolved Hide resolved
pkg/agent/police.go Outdated Show resolved Hide resolved
pkg/agent/police.go Outdated Show resolved Hide resolved
pkg/agent/police.go Outdated Show resolved Hide resolved
pkg/agent/police.go Outdated Show resolved Hide resolved
charts/values.yaml Outdated Show resolved Hide resolved
@biqiangwu biqiangwu force-pushed the feat/underlay_CNI_supports branch 2 times, most recently from 5588222 to 73b1fcc Compare October 24, 2023 06:13
```
# "1.1.1.1/32", "2.2.2.2/32" 为外部服务地址
kubectl patch spidercoordinators default --type='merge' -p '{"spec": {"hijackCIDR": ["1.1.1.1/32", "2.2.2.2/32"]}}'
```
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

对于已经运行的pod,还需要重启,这些路由才会生效

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done

[ -z "${SPIDERPOOL_VERSION}" ] && SPIDERPOOL_VERSION=$(helm show chart spiderpool/spiderpool | awk '/version/ {print $2}')

helm install spiderpool spiderpool/spiderpool --namespace kube-spiderpool --set multus.multusCNI.defaultCniCRName=${MACVLAN_MULTUS_NAME} --wait --debug --version=${SPIDERPOOL_VERSION}
createMultusCon
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

(1)所有的镜像源指定,要遵循 E2E_CHINA_IMAGE_REGISTRY 定义,方便国内开发环境 setup
(2)spiderpool 用 latest 镜像,方便及时发现问题

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done

Signed-off-by: dcwbq <biqiang.wu@daocloud.io>
@weizhoublue
Copy link
Collaborator

@weizhoublue weizhoublue changed the title underlay CNI supports support spiderpool Oct 25, 2023
@weizhoublue weizhoublue merged commit 94708aa into main Oct 25, 2023
58 checks passed
@weizhoublue weizhoublue deleted the feat/underlay_CNI_supports branch October 25, 2023 14:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release/feature-new feature pr recoded in new release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants