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

cluster-api-aws: fix machinedeployment subnet filter #219

Merged
merged 1 commit into from
Nov 13, 2023

Conversation

zugwan
Copy link
Contributor

@zugwan zugwan commented Nov 13, 2023

기존 VPC 및 서브넷을 재사용할 때 잘못된 필터링 규칙을 수정합니다.

기존 규칙

태그 중 Name 키가 -subnet-[private or public]-AZ 값을 가는 것을 이용, Name 태그의 값 중 클러스터 아이디를 포함하는 서브넷을 선정하도록 규칙 적용
예) 클러스터 이름이 cph5wipz9 일 때 private subnet의 태그 키 Name, 값 cph5wipz9-subnet-private-ap-northeast-2a

문제점

기존 Cluster API 클러스터에서 생성한 서브넷을 재사용할 경우 Name 태그는 최초 생성한 클러스터 정보를 그대로 유지하기 때문에 신규 클러스터에서는 기존 필터로는 사용 가능한 서브넷 정보를 가져올 수 없음.

개선

클러스터가 사용하려는 서브넷 (capa를 통해 생성을 하거나 기존 서브넷을 재사용하는 모든 경우에)에는 kubernetes.io/cluster/ 키가 "shared"라는 값으로 생성되기 때문에 태그의 값이 아닌 태그 자체에 클러스터 아이디가 포함되어 있는 지 여부를 확인하는 필터를 사용
예) 클러스터 이름이 cph5wipz9 일때: 태그 키 kubernetes.io/cluster/cph5wipz9, 값 "shared"

Fixes incorrect filtering rules when reusing existing VPCs and
Subnets.
@ktkfree ktkfree merged commit 68c6464 into main Nov 13, 2023
2 of 3 checks passed
@ktkfree ktkfree deleted the capa_fix_md_subnet_filter branch November 13, 2023 11:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants