[EYPP-165] 클라우드에서 관리되는 환경 변수에 대한 지원 추가
3717 단어 ey-cookbooks-stable-v5
묘사
패치 설명
기존 레시피(env vars::cloud)의 일부로 새 레시피(env vars)를 만들었습니다.레시피의 목적은
env.cloud
디렉터리에 새 파일을 생성하는 것이다shared/config
.새로 생성된 파일 원본 파일을 응용 프로그램 서버에 보낼 수 있도록 ((
env.custom
파일과 같은 방식) 몇 가지 방법도 수정했다.권장 릴리즈 노트
위험을 예측하다
낮았어
관련된 구성 요소
새 레시피:
cookbooks/env_vars/recipes/cloud.rb
새로운 레시피의 삽입점:
cookbooks/ey-application/recipes/create.rb
응용 프로그램 서버의 구성 템플릿이 업데이트되었습니다.
cookbooks/node/templates/default/app_control.sh.erb
cookbooks/passenger5/templates/default/app_control.erb
cookbooks/php/templates/default/fpm-pool.conf.erb
cookbooks/pm2/templates/default/app_control.sh.erb
cookbooks/puma/templates/default/app_control.erb
cookbooks/sidekiq/files/default/sidekiq
cookbooks/unicorn/templates/default/unicorn.initd.sh.erb
테스트 완료 설명
기존 인스턴스에 이 레시피 적용
기존 인스턴스를 새 스택으로 업그레이드합니다.파일
dna.json
에 구성된 변수가 없으므로 빈 파일이 생성되고 변경 사항이 없습니다.여기에 나의 변경 사항을 포함하는 창고가 하나 있다. rz_test-0.1업데이트된 인스턴스에 이 레시피 적용
일부 임시 환경에 custom branch of AWSM 배포나는 mission를 사용하여 테스트를 진행했다.
클라우드 패널에 환경 변수를 설정합니다. (초기 접근 옵션에서 이 기능을 열어야 합니다.)테스트에 사용할 변수는 here입니다.
인스턴스 변경 사항을 적용하여 변수 채우기
dna.json
응용 프로그램 측의 변수 목록 검사: variables in the app
품질 보증 설명
상술한 테스트 설명을 따르다.
토론 #1
@rzaharenkov는 다음 형식에 따라 홍보 설명을 업데이트해도 괜찮겠습니까?예제는 닫힌 PRs를 참조하십시오.감사합니다!## Description of your patch
## Recommended Release Notes
## Estimated risk
## Components involved
## Description of testing done
## QA Instructions
토론 #2
@radamanthus, 업데이트토론 #셋
참고로 개발 창고 버전을 구축할 때 몇 가지 이유로 테스트를 통과할 수 없습니다.https://dredd.engineyard.com/suites/f961b8ab-f677-4d2e-bf06-033d4b9022b4.보시다시피 테스트 세트 자체가 이 줄에서 실패했습니다.https://github.com/engineyard/dredd/blob/master/scenarios/solo.rb#L41.나는 그것을 어떻게 디버깅하거나 테스트하는지 모른다.Dredd 빌드에 장애가 발생했지만 성공적으로 스택을 풀었고 스택이 정상적으로 작동합니다.나의 마지막 평론을 무시해 주세요.Dredd 문제가 해결되었습니다.코드에서 문제가 해결되었습니다.https://dredd.engineyard.com/suites/0c20b1e6-8f4d-47db-b4a5-839b9fd7f19d토론 #4
LGTM토론 #5
열심히 노력해 주셔서 감사합니다!코드가 괜찮아 보이는데 v5-3.0.31 이후에 테스트할 PRs에 포함시킬 수 있을 것 같습니다미래에 내가 요구할 작은 일은 단독 제출에서 공백을 정리하는 것이다.Google squash 제출 규칙을 위반한 것으로 알고 있지만, 이렇게 하면 심사자들이 청소와 관련된 변경 사항이나 실제 코드 변경 사항을 찾아야 하는지 쉽게 볼 수 있습니다.
토론 #6
감사합니다 @radamanthus!너의 평론은 100% 유효하다.다음에는 최대한 단독으로 약속할게요.문제는 내가 파일을 저장할 때 슈프림이 무심코 이 점을 해냈다는 것이다.나는 그것을 합병할 것이다.너는 그것이 언제 발표되는지 아니?
토론 #7
@rzaharenkov 감사합니다!다음 버전으로 넘어갈 거예요.그러나 발행판은 새로운 긴급 업데이트를 요구한다. 나는 다음 주까지 이 업데이트를 한 버전에 발표한 후에 발표할 것이다.다음에는 코드 변경을 다음 버전으로 직접 통합하지 마십시오.게시 관리 프로세스 (@dbeckstead와 나) 를 통합시킵니다.
토론 #8
아, 맞다. 나는 단지 그것이 어떤 이유로 끊겼다고 생각했을 뿐이다.3주 전에 허가했으니 아직 흥분하지 마세요.Reference
이 문제에 관하여([EYPP-165] 클라우드에서 관리되는 환경 변수에 대한 지원 추가), 우리는 이곳에서 더 많은 자료를 발견하고 링크를 클릭하여 보았다 https://github.com/engineyard/ey-cookbooks-stable-v5/issues/237텍스트를 자유롭게 공유하거나 복사할 수 있습니다.하지만 이 문서의 URL은 참조 URL로 남겨 두십시오.
우수한 개발자 콘텐츠 발견에 전념 (Collection and Share based on the CC Protocol.)