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

Proj 1 dev #4

Merged
merged 11 commits into from
May 7, 2023
Merged

Proj 1 dev #4

merged 11 commits into from
May 7, 2023

Conversation

sts2885
Copy link
Owner

@sts2885 sts2885 commented May 7, 2023

프로젝트 1이 마무리 되었으니, main으로 release 합니다. 언젠가는 release branch를 만들어서 운영했으면 좋겠네요!

Son Taeseon and others added 11 commits May 2, 2023 00:26
 Add 1 : VPC created
Add 2 : Public, private subnet added at each .tf file
Add 3 : igw , public route table added, no route rule yet
Add 3 : nat , eip_for_nat, private route table added
=> Add 3 is not completed. It will be completed at next time.
Step 1 make vpc -> project1_dev로 merge를 승인합니다.
 step 2 - adjust custom vpc(instead of default one) completed
VPC include 3 tier subnet (web, was, db).
And each subnet contains auto scaling group and load balancer, and simple apache server to health check.
Next time, deploy monitoring system and maybe web was db software.
 Project 1 completed.
Modified 1 : public, app, db -> web, was, db
Modified 2 : ALB positioned at public subnet, and web tier goes to private subnet
Modified 3 : subnet {pub, app, db} -> subnet {pub, web, was, db}, so public cidr splitted to pub and web
ADD 1 : print output ip and domain names of ELB
ADD 2 : Use user data instead of using direct code
…system

Ds 50  . step 3 deploy monitoring system
@sts2885
Copy link
Owner Author

sts2885 commented May 7, 2023

review required는 여기다가 한줄 적으면 되는 걸까요?

Copy link
Owner Author

@sts2885 sts2885 left a comment

Choose a reason for hiding this comment

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

프로젝트 1 성공을 축하합니다! (자축)
-> 근데 나 혼자라 Approve를 할 수 없어요... (시무룩)
-> bypass해서 merge 해야 겠네요!

10. dash board import

제 블로그를 참고해주신다면 감사하겠습니다.(그냥 들어가서 datasource만 추가해주면 됩니다.)
https://cherry-blossome-in-march.tistory.com/5
Copy link
Owner Author

Choose a reason for hiding this comment

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

세상에 merge 해보기 위해서 code review라는 걸 처음으로 해봤네요! im-d-team/Dev-Docs#11 여기서 하는 법을 알게 됐어요!

@sts2885 sts2885 merged commit 0eb8f3a into main May 7, 2023
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.

1 participant