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

@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
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 <[email protected]>

* Modify init command

Signed-off-by: t-kikuc <[email protected]>

* Rename init command to initialize

Signed-off-by: t-kikuc <[email protected]>

* Create pipectl int for ECS

Signed-off-by: t-kikuc <[email protected]>

* Add init for ECS

Signed-off-by: t-kikuc <[email protected]>

* Convert json.RawMessage to struct

Signed-off-by: t-kikuc <[email protected]>

* Use original structs of ECS for pipectl init

Signed-off-by: t-kikuc <[email protected]>

* Init k8s app for pipectl init, but not implemented

Signed-off-by: t-kikuc <[email protected]>

* Fix init input(wip)

Signed-off-by: t-kikuc <[email protected]>

* Add mock prompt reader

Signed-off-by: t-kikuc <[email protected]>

* Reractor prompt reader

Signed-off-by: t-kikuc <[email protected]>

* Modify mock reader to public

Signed-off-by: t-kikuc <[email protected]>

* Refactored prompt reader

Signed-off-by: t-kikuc <[email protected]>

* Add yaml snapshot test

Signed-off-by: t-kikuc <[email protected]>

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

This reverts commit 4097fb7.

Signed-off-by: t-kikuc <[email protected]>

* Refactor pipectl init

Signed-off-by: t-kikuc <[email protected]>

* Remove go-yaml from go.mod

Signed-off-by: t-kikuc <[email protected]>

* Modify tests to Parallel

Signed-off-by: t-kikuc <[email protected]>

* Fix reference of loop variable in tests

Signed-off-by: t-kikuc <[email protected]>

* Enable interruption signal for pipectl init

Signed-off-by: t-kikuc <[email protected]>

* Add LoadBalancerName to ECS TargetGroup

Signed-off-by: t-kikuc <[email protected]>

* Use Reader in tests

Signed-off-by: t-kikuc <[email protected]>

* Remove stdinReader

Signed-off-by: t-kikuc <[email protected]>

* Remove mockReader

Signed-off-by: t-kikuc <[email protected]>

* Add Prompt and PromptInput types

Signed-off-by: t-kikuc <[email protected]>

* Remove promptReader

Signed-off-by: t-kikuc <[email protected]>

* Refactored init,exporter

Signed-off-by: t-kikuc <[email protected]>

* Update Copyright

Signed-off-by: t-kikuc <[email protected]>

* Refactored nits

Signed-off-by: t-kikuc <[email protected]>

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

Signed-off-by: Yoshiki Fujikane <[email protected]>
Signed-off-by: t-kikuc <[email protected]>

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

Signed-off-by: t-kikuc <[email protected]>

* Add RC Release Procedure (#4749)

Signed-off-by: t-kikuc <[email protected]>

* Fix nits

Signed-off-by: t-kikuc <[email protected]>

* Fix error variable name

Signed-off-by: t-kikuc <[email protected]>

---------

Signed-off-by: t-kikuc <[email protected]>
Signed-off-by: Yoshiki Fujikane <[email protected]>
Co-authored-by: Yoshiki Fujikane <[email protected]>
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
@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.

3 participants