Skip to content

Add option to allow command execution. #20

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

cynthia
Copy link
Contributor

@cynthia cynthia commented Mar 17, 2021

원격 호스트에 명령을 실행할 수 있는 기능입니다. (geofront-cli ssh 에서만 동작합니다.)

@cynthia
Copy link
Contributor Author

cynthia commented Mar 17, 2021

@kanghyojun 님이 맞는지 모르겠습니다만, 리뷰 부탁드립니다.

@kanghyojun
Copy link
Contributor

@pbzweihander @Jhuni0123 제가 보기에는 옵션에 문제는 없어보이는데 테스트할 수 있는 머신이 없기도해서 ㅋㅋ 스포카 분들이 봐주실 수 있을까요?

@cynthia
Copy link
Contributor Author

cynthia commented Mar 30, 2021

만약에 스포카에서 더 이상 사용/유지보수하지 않는 코드베이스이 저희가 부담을 가하고 있다면, commiter/reviewer로 넣어주시고 저희쪽에서 유지 보수 부담을 가져갈 의향도 있습니다. (저희쪽에선 적극적으로 활용중입니다.)

@dahlia 님 어떠신지요?

@kanghyojun
Copy link
Contributor

제가 스포카에서 퇴사하게 되어 없는것이라 ㅎㅎ 스포카에서 판단해주셔야할거같습니다!

Copy link
Contributor

@dahlia dahlia left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

머지해도 좋을 것 같습니다.

만약에 스포카에서 더 이상 사용/유지보수하지 않는 코드베이스이 저희가 부담을 가하고 있다면, commiter/reviewer로 넣어주시고 저희쪽에서 유지 보수 부담을 가져갈 의향도 있습니다. (저희쪽에선 적극적으로 활용중입니다.)

@dahlia 님 어떠신지요?

프로젝트 시작한 입장에서는 좋은 것 같습니다.

  1. @spoqa 분들께서 봐주시고 괜찮다고 보신다면 메인테이너 권한을 드리면 어떨까요?
  2. 아니면 @spoqa 팀 분들께 일체의 관리 업무(저장소 설정을 고치는 등)가 부담된다면, 아예 이참에 별도 org 계정을 만들어서 옮기는 것도 가능할 것 같습니다. (이렇게 하기로 한다면 제가 org 계정을 만들어서 알려드리겠습니다.)

@spoqa 분들께서 결정하신 뒤에 알려주시면 좋을 것 같습니다! @pbzweihander @Jhuni0123 @wisemuji

@pbzweihander
Copy link

pbzweihander commented Mar 31, 2021

답변이 늦어져서 죄송합니다!

스포카에서 현재 유지보수가 불가능할 정도는 아닌데, 슬랙-깃헙 연동 앱이 망가져서 알림을 못 받아서 대응이 늦어지게 되었습니다.
스포카에서는 현재 geofront를 엄청 자주 사용하고 있지 않아서 유지보수 우선순위가 조금 떨어진 상태입니다. 그렇다고 cynthia 님을 메인테이너로 추가만 하기보다는 커멘트 남겨주신 것 처럼 깃헙 org를 따로 만드는 것이 좋을 것 같습니다.

@dahlia org 계정 만들고 알려주시면 geofront-cli와 geofront 리포를 트랜스퍼하겠으니 부탁드립니다!

@dahlia
Copy link
Contributor

dahlia commented Mar 31, 2021

@pbzweihander 말씀드리려고 했는데 벌써 다 옮겨주셨군요. 💦 저녁 시간인데 빠른 처리 감사합니다!

@dahlia
Copy link
Contributor

dahlia commented Mar 31, 2021

@cynthia 님도 org에 초대드렸으니 수락 부탁드립니다.

@cynthia
Copy link
Contributor Author

cynthia commented Mar 31, 2021

@dahlia 수락했습니다. 감사합니다!

Copy link
Contributor

@dahlia dahlia left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

그런데 생각해보니, ssh 커맨드는 -X/--exec라는 옵션이 따로 있지 않고 ssh user@host -- echo foo bar (--는 뺄 수 있음) 같이 쓰는데요, 그 컨벤션을 그대로 따르는 게 좋지 않을까요?

@cynthia
Copy link
Contributor Author

cynthia commented Mar 31, 2021

따르는 게 좋지 않을까요?

좋을거라고는 생각했는데, trailing freeform argument에 대한 지원 선례가 없다보니 과격한 변화는 조금 꺼려져서 일단 소극적으로 접근했습니다. 조금 과격해도 상관 없다면 팔로업 패치 보낼게요.

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.

4 participants