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

Add reference to the blog that shows how to install control plane on ECS #4746

Merged
merged 1 commit into from
Jan 10, 2024

Conversation

ffjlabo
Copy link
Member

@ffjlabo ffjlabo commented Jan 10, 2024

What this PR does / why we need it:

Added the link for PipeCD best practice 02 - control plane on ECS

Cursor_と_Install_Control_Plane___PipeCD Installing_Control_Plane_on_ECS___PipeCD

Which issue(s) this PR fixes:

Fixes #

Does this PR introduce a user-facing change?:

  • How are users affected by this change: no
  • Is this breaking change: no
  • How to migrate (if breaking change): no

Signed-off-by: Yoshiki Fujikane <ffjlabo@gmail.com>
@ffjlabo
Copy link
Member Author

ffjlabo commented Jan 10, 2024

Added for dev and v0.45.x

Copy link

codecov bot commented Jan 10, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (449844d) 30.84% compared to head (16d4be7) 30.85%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #4746      +/-   ##
==========================================
+ Coverage   30.84%   30.85%   +0.01%     
==========================================
  Files         222      222              
  Lines       26037    26037              
==========================================
+ Hits         8031     8035       +4     
+ Misses      17356    17353       -3     
+ Partials      650      649       -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Currently, we provide the example of deploying Control Plane to ECS using terraform.

Please refer to the blog post :)
[PipeCD best practice 02 - control plane on ECS]({{< ref "/blog/control-plane-on-ecs.md" >}} "PipeCD best practice 02 - control plane on ECS").
Copy link
Member Author

Choose a reason for hiding this comment

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

Checked whether to work the link and it is OK ✅
ref: https://gohugo.io/content-management/cross-references/#use-of-ref-and-relref

Copy link
Member

@t-kikuc t-kikuc left a comment

Choose a reason for hiding this comment

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

LGTM 🚀

@ffjlabo ffjlabo enabled auto-merge (squash) January 10, 2024 02:49
Copy link
Member

@khanhtc1202 khanhtc1202 left a comment

Choose a reason for hiding this comment

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

Nice

@ffjlabo ffjlabo merged commit ab76373 into master Jan 10, 2024
14 checks passed
@ffjlabo ffjlabo deleted the add-docs-ref-for-installing-control-plane-on-ecs branch January 10, 2024 05:01
t-kikuc pushed a commit that referenced this pull request Jan 19, 2024
…ECS (#4746)

Signed-off-by: Yoshiki Fujikane <ffjlabo@gmail.com>
Signed-off-by: t-kikuc <tkikuchi07f@gmail.com>
t-kikuc added a commit that referenced this pull request Jan 19, 2024
* Add pipectl init command but not implemented

Signed-off-by: t-kikuc <tkikuchi07f@gmail.com>

* Modify init command

Signed-off-by: t-kikuc <tkikuchi07f@gmail.com>

* Rename init command to initialize

Signed-off-by: t-kikuc <tkikuchi07f@gmail.com>

* Create pipectl int for ECS

Signed-off-by: t-kikuc <tkikuchi07f@gmail.com>

* Add init for ECS

Signed-off-by: t-kikuc <tkikuchi07f@gmail.com>

* Convert json.RawMessage to struct

Signed-off-by: t-kikuc <tkikuchi07f@gmail.com>

* Use original structs of ECS for pipectl init

Signed-off-by: t-kikuc <tkikuchi07f@gmail.com>

* Init k8s app for pipectl init, but not implemented

Signed-off-by: t-kikuc <tkikuchi07f@gmail.com>

* Fix init input(wip)

Signed-off-by: t-kikuc <tkikuchi07f@gmail.com>

* Add mock prompt reader

Signed-off-by: t-kikuc <tkikuchi07f@gmail.com>

* Reractor prompt reader

Signed-off-by: t-kikuc <tkikuchi07f@gmail.com>

* Modify mock reader to public

Signed-off-by: t-kikuc <tkikuchi07f@gmail.com>

* Refactored prompt reader

Signed-off-by: t-kikuc <tkikuchi07f@gmail.com>

* Add yaml snapshot test

Signed-off-by: t-kikuc <tkikuchi07f@gmail.com>

* Revert "Init k8s app for pipectl init, but not implemented"

This reverts commit 4097fb7.

Signed-off-by: t-kikuc <tkikuchi07f@gmail.com>

* Refactor pipectl init

Signed-off-by: t-kikuc <tkikuchi07f@gmail.com>

* Remove go-yaml from go.mod

Signed-off-by: t-kikuc <tkikuchi07f@gmail.com>

* Modify tests to Parallel

Signed-off-by: t-kikuc <tkikuchi07f@gmail.com>

* Fix reference of loop variable in tests

Signed-off-by: t-kikuc <tkikuchi07f@gmail.com>

* Enable interruption signal for pipectl init

Signed-off-by: t-kikuc <tkikuchi07f@gmail.com>

* Add LoadBalancerName to ECS TargetGroup

Signed-off-by: t-kikuc <tkikuchi07f@gmail.com>

* Use Reader in tests

Signed-off-by: t-kikuc <tkikuchi07f@gmail.com>

* Remove stdinReader

Signed-off-by: t-kikuc <tkikuchi07f@gmail.com>

* Remove mockReader

Signed-off-by: t-kikuc <tkikuchi07f@gmail.com>

* Add Prompt and PromptInput types

Signed-off-by: t-kikuc <tkikuchi07f@gmail.com>

* Remove promptReader

Signed-off-by: t-kikuc <tkikuchi07f@gmail.com>

* Refactored init,exporter

Signed-off-by: t-kikuc <tkikuchi07f@gmail.com>

* Update Copyright

Signed-off-by: t-kikuc <tkikuchi07f@gmail.com>

* Refactored nits

Signed-off-by: t-kikuc <tkikuchi07f@gmail.com>

* Add reference to the blog that shows how to install control plane on ECS (#4746)

Signed-off-by: Yoshiki Fujikane <ffjlabo@gmail.com>
Signed-off-by: t-kikuc <tkikuchi07f@gmail.com>

* Bump follow-redirects from 1.15.2 to 1.15.4 in /web (#4747)

Signed-off-by: t-kikuc <tkikuchi07f@gmail.com>

* Add RC Release Procedure (#4749)

Signed-off-by: t-kikuc <tkikuchi07f@gmail.com>

* Fix nits

Signed-off-by: t-kikuc <tkikuchi07f@gmail.com>

* Fix error variable name

Signed-off-by: t-kikuc <tkikuchi07f@gmail.com>

---------

Signed-off-by: t-kikuc <tkikuchi07f@gmail.com>
Signed-off-by: Yoshiki Fujikane <ffjlabo@gmail.com>
Co-authored-by: Yoshiki Fujikane <40124947+ffjlabo@users.noreply.github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
nnnkkk7 pushed a commit to nnnkkk7/pipecd that referenced this pull request Feb 1, 2024
…ECS (pipe-cd#4746)

Signed-off-by: Yoshiki Fujikane <ffjlabo@gmail.com>
@github-actions github-actions bot mentioned this pull request Feb 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants