스냅샷 클리닝 중 AMI 비활성화 스냅샷 삭제 제외 처리 필요

Kim-Yul commented 6 months ago

자세히 알아보아야 하지만, 현재 파악된 부분으로 작성하였습니다.

현재 스냅샷 클리닝 코드는 AMI를 비활성화한 것을 삭제해야 한다고 인식하고 있습니다.

이로 인해 에러가 발생합니다. 로직 상 AMI가 비활성화되어 있어도 스냅샷을 삭제하지 않아야 하는데, 이 경우를 코드에 반영해야 할 것 같습니다.

이 외에 다른 스냅샷은 정상 삭제 되었습니다.

다음은 수정이 필요한 부분입니다.

kmu-leeky commented 6 months ago

AMI 가 비활성화 (disable) 되어 있을때 삭제를 하면 안되는데 지금은 삭제를 한다는거지? 그런데 AMI 가 비활성화 되어 있으면 삭제해도 되지 않을까? 비활성화시 삭제 에러가 난다면 그걸 해결해서 삭제되도록 하는게 맞지 않을까?

Kim-Yul commented 6 months ago

비활성화된 AMI라면 AMI 삭제 후 스냅샷 삭제로 코드 수정 작업 진행하겠습니다.

Kim-Yul commented 6 months ago

원하는대로 코드가 동작하지 않아 수정을 진행하였습니다. 그로 인해 예정보다 하루 늦게 완료되어 뒤늦게 진행 사항 공유해드립니다.

우선, 삭제 전에 리포트해야 할 필요성이 있을 것 같아 리포트 코드 구현했습니다.

시나리오

report 를 통해서 어떤 snapshot이 삭제될 지 알 수 있으며, management를 통해 ami 삭제 후 실제 삭제가 이루어집니다.

밑의 AMI 상태와 스냅샷 리스트는 제가 사용하고 있던 리소스입니다.

[활성화 상태의 AMI]

[태그가 있는 상태의 스냅샷]

[결과 : 슬랙 메세지]

[태그가 없는 상태의 AMI]

[결과 : 슬랙 메세지]

[비활성화 상태의 AMI]

[태그가 있는 상태의 스냅샷]

[결과 : 슬랙 메세지]

[비활성화 상태의 AMI]

[태그가 없는 상태의 스냅샷]

[결과 : 슬랙 메세지]

이전 코드는 불필요한 코드가 추가되어 있었고 무엇보다 원하는 동작대로 진행되지 않아 의도에 맞게 코드를 간결하게 수정하였습니다. 작성 완료된 코드는 커밋하도록 하겠습니다. 코드 리뷰 후 머지하도록 하겠습니다.

Kim-Yul commented 6 months ago

풀리퀘 완료하여 이슈 클로즈 하겠습니다.