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

alibabacloud: use DescribeVSwitches to get vswitch tags #23635

Merged

Conversation

haozhangami
Copy link
Contributor

Obtaining the vswitch tags through ListTagResources is incomplete, which will cause the IP allocation to fail and require additional queries. Therefore, use DescribeVSwitches to get the vswitch tags.

Signed-off-by: Hao Zhang hao.zhang.am.i@gmail.com

use DescribeVSwitches to get vswitch tags

@haozhangami haozhangami requested a review from a team as a code owner February 8, 2023 14:17
@maintainer-s-little-helper maintainer-s-little-helper bot added the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Feb 8, 2023
@github-actions github-actions bot added the kind/community-contribution This was a contribution made by a community member. label Feb 8, 2023
@haozhangami
Copy link
Contributor Author

Hello @l1b0k please help to review, thank you

@christarazi christarazi added release-note/misc This PR makes changes that have no direct user impact. area/alibaba Impacts Alibaba based IPAM. sig/ipam IP address management, including cloud IPAM and removed dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. labels Feb 9, 2023
@haozhangami haozhangami force-pushed the use-describevswitches-to-get-vswitch-tags branch 2 times, most recently from 233a7b3 to fb01593 Compare February 24, 2023 13:20
@sayboras
Copy link
Member

/test

Obtaining the vswitch tags through ListTagResources is incomplete,
which will cause the IP allocation to fail and require additional
queries. Therefore, use DescribeVSwitches to get the vswitch tags.

Signed-off-by: Hao Zhang <hao.zhang.am.i@gmail.com>
@haozhangami haozhangami force-pushed the use-describevswitches-to-get-vswitch-tags branch from fb01593 to a57ba66 Compare February 28, 2023 07:43
@haozhangami
Copy link
Contributor Author

/test

1 similar comment
@sayboras
Copy link
Member

/test

@sayboras
Copy link
Member

👋

Just a note that only cilium team member can trigger tests via /test comment. I have re-triggered the tests, please avoid force push till the tests are done.

Thanks a lot for your contribution.

@sayboras sayboras requested a review from a team February 28, 2023 08:54
@sayboras
Copy link
Member

All CI jobs are passed (except ci-verifier is in waiting for status stage, but this job is not required as part of the changes here), reviews are in, marking this ready to merge.

@sayboras sayboras added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Feb 28, 2023
@sayboras sayboras merged commit 4269350 into cilium:master Feb 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/alibaba Impacts Alibaba based IPAM. kind/community-contribution This was a contribution made by a community member. ready-to-merge This PR has passed all tests and received consensus from code owners to merge. release-note/misc This PR makes changes that have no direct user impact. sig/ipam IP address management, including cloud IPAM
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants