오픈소스 컨트리뷰트해보기

[skarltjr]

개요 :

• https://github.com/fabric8io/kubernetes-client/pull/4200

  
  fabric8은 자바에서 손쉽게 k8s api를 활용할 수 있는 k8s client 오픈소스

docker-registry와 관련된 secret resource를 생성해야하는 상황이었다. 소스코드를 살펴보니 내부에 createDockerRegistrySecret() 매서드가 존재했었는데 secretName을 지정할 수 없었고 해당 매서드로 생성된 secret은 harbor-secret이라는 이름을 갖게된다.

기여 : 

1. createDockerRegistrySecret 매서드는 디폴트로 container-image-registry-secret이라는 이름의 시크릿 생성
2. 오버로딩을 통해 secretName param을 추가하고 docker-registry-secret 생성시 secretName 파라미터를 활용하여 이름을 지정할 수 있도록했다.
3. 테스트코드 추가

느낀점 :

확실히 느꼈다.
1. 쉽게 지나칠 수 있는 중복을 최대한 줄이고자한다.
- 중복은 곧 코드양이 증가 -> 코드 해석 시간증가 -> 수정의 어려움 증가 -> 유지보수 어려움의 증가라고 느낄 수 있었다. 
3. 변수명 하나하나 충분한 고려를한다.
- 나와 남의 이해를 위해서 올바른 변수명을 고민하는건 굉장히 중요한것같다.
- 시크릿 리소스의 기본 이름을 docker-registry-secret으로 변경할까했지만 리뷰를 반영해 좀 더 포괄적인 "container-image-registry-secret"를 활용했다

6/22 merged

[skarltjr]

오예~

[skarltjr]

중복은 항상 줄여야하는가?

중복을 줄이는것은 좋다
다만 이로인한 사이드 이펙트는 항상 고려해야한다.
당연히 수정으로 인해 문제가 발생할 수 있는데

그럼 이건 어떻게 대응해야하는가?
나는 테스트코드라고 생각한다
변경으로 인해 발생할 수 있는 문제를 테스트를 통해 파악해나갈 수 있다고 생각한다.