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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: externalID map should not include external_ids #3385

Merged
merged 1 commit into from
Nov 7, 2023

Conversation

xujunjie-cover
Copy link
Member

@xujunjie-cover xujunjie-cover commented Nov 7, 2023

Pull Request

What type of this PR

Examples of user facing changes:

  • Bug fixes

Which issue(s) this PR fixes

Fixes #3377

WHAT

馃 Generated by Copilot at f94510b

Simplify the external_ids key format for logical switch ports associated with security groups in pkg/controller/security_group.go. This improves code readability and consistency.

馃 Generated by Copilot at f94510b

ListLogicalSwitchPorts
Simplifies external_ids
A clear autumn code

HOW

馃 Generated by Copilot at f94510b

  • Simplify the external_ids key for security group association in logical switch ports (link)

Signed-off-by: xujunjie-cover <xujunjielxx@163.com>
@bobz965
Copy link
Collaborator

bobz965 commented Nov 7, 2023

@xujunjie-cover please fix the DCO by git commit -s

@oilbeater oilbeater added bug Something isn't working need backport labels Nov 7, 2023
@oilbeater oilbeater merged commit 75bbeb2 into kubeovn:master Nov 7, 2023
10 checks passed
oilbeater pushed a commit that referenced this pull request Nov 7, 2023
Signed-off-by: xujunjie-cover <xujunjielxx@163.com>
bobz965 pushed a commit that referenced this pull request Nov 8, 2023
Signed-off-by: xujunjie-cover <xujunjielxx@163.com>
bobz965 pushed a commit to bobz965/kube-ovn that referenced this pull request Nov 15, 2023
Signed-off-by: xujunjie-cover <xujunjielxx@163.com>
Signed-off-by: bobz965 <zhangbingbing2_yewu@cmss.chinamobile.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working need backport
Projects
None yet
Development

Successfully merging this pull request may close these issues.

v1.12.1璁剧疆SecurityGroup浠ュ悗锛屽厑璁告墍鏈夌殑icmp锛岃鍒欎笉鐢熸晥
3 participants