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
ci: add multus canary test without holder pods #14159
base: master
Are you sure you want to change the base?
ci: add multus canary test without holder pods #14159
Conversation
This pull request has merge conflicts that must be resolved before it can be merged. @subhamkrai please rebase it. https://rook.io/docs/rook/latest/Contributing/development-flow/#updating-your-fork |
@@ -1614,7 +1614,7 @@ jobs: | |||
name: ${{ github.job }}-${{ matrix.ceph-image }} | |||
additional-namespace: kube-system | |||
|
|||
csi-hostnetwork-disabled: | |||
multus-cluster-network-without-holder: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could we also rename the test above to be multus-cluster-network-with-holder
so it's clear in the outputs which is which?
|
||
- name: wait for prepare pod | ||
run: tests/scripts/github-action-helper.sh wait_for_prepare_pod 2 | ||
|
||
- name: wait for ceph to be ready | ||
run: IS_POD_NETWORK=true tests/scripts/github-action-helper.sh wait_for_ceph_to_be_ready osd 2 | ||
run: IS_POD_NETWORK=true IS_MULTUS=true tests/scripts/github-action-helper.sh wait_for_ceph_to_be_ready osd 2 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Won't IS_MULTUS=true
cause the test to fail because of the block below?
tests/scripts/validate_cluster.sh
:
if [ -n "$IS_MULTUS" ]; then
echo "verifying csi holder interfaces (multus ones must be present)"
kubectl -n rook-ceph exec -t ds/csi-rbdplugin-holder-my-cluster -- grep eth0 /proc/net/dev
kubectl -n rook-ceph exec -t ds/csi-cephfsplugin-holder-my-cluster -- grep eth0 /proc/net/dev
kubectl -n rook-ceph exec -t ds/csi-nfsplugin-holder-my-cluster -- grep eth0 /proc/net/dev
fi
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yeah probably and also the routing part is still pending to add
fb6c912
to
c7d7e12
Compare
this commit adds changes to deploy multus with holder pods disabled and also remove ci test csi-hostnetwork-disabled as this not really not supported in rook. Signed-off-by: subhamkrai <srai@redhat.com>
c7d7e12
to
4a39b31
Compare
this commit adds changes to deploy multus with
holder pods disabled and also remove ci test
csi-hostnetwork-disabled as this not really not
supported in rook.
Issue resolved by this Pull Request:
Resolves #13922
Checklist: