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

Remove pipectl quickstart command #5268

Merged
merged 2 commits into from
Oct 9, 2024
Merged

Conversation

khanhtc1202
Copy link
Member

@khanhtc1202 khanhtc1202 commented Oct 8, 2024

What this PR does / why we need it:

We will use kubectl with quickstart manifests for quickstart, instead of pipectl quickstart command

Which issue(s) this PR fixes:

Fixes #5148

Does this PR introduce a user-facing change?:

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

Signed-off-by: khanhtc1202 <khanhtc1202@gmail.com>
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.

thank you.

Would you remove the following line in docs?

quickstart Quick prepare PipeCD control plane in quickstart mode.

Copy link

codecov bot commented Oct 8, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 24.28%. Comparing base (3dbf164) to head (2288b8b).
Report is 19 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #5268      +/-   ##
==========================================
+ Coverage   24.08%   24.28%   +0.19%     
==========================================
  Files         441      440       -1     
  Lines       47099    46754     -345     
==========================================
+ Hits        11345    11352       +7     
+ Misses      34850    34499     -351     
+ Partials      904      903       -1     

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

Signed-off-by: khanhtc1202 <khanhtc1202@gmail.com>
@khanhtc1202
Copy link
Member Author

@t-kikuc You're right, I remove that as well 👍

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.

thank you.

Although docs/content/en/docs-v0.49.x/user-guide/command-line-tool.md is not required to change, let's recommend kubectl quickstart instead.

Copy link
Contributor

@Warashi Warashi left a comment

Choose a reason for hiding this comment

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

LGTM

@khanhtc1202 khanhtc1202 merged commit 4b2499e into master Oct 9, 2024
18 checks passed
@khanhtc1202 khanhtc1202 deleted the remove-quickstart-subcommand branch October 9, 2024 03:07
github-actions bot pushed a commit that referenced this pull request Oct 9, 2024
* Remove pipectl quickstart command

Signed-off-by: khanhtc1202 <khanhtc1202@gmail.com>

* Remove the quickstart from docs

Signed-off-by: khanhtc1202 <khanhtc1202@gmail.com>

---------

Signed-off-by: khanhtc1202 <khanhtc1202@gmail.com>
Signed-off-by: pipecd-bot <pipecd.dev@gmail.com>
t-kikuc added a commit that referenced this pull request Oct 9, 2024
* Fix the workflow publishes quickstart manifests (#5266)

* Add pull-requests permission to the workflow

Signed-off-by: Shinnosuke Sawada-Dazai <shin@warashi.dev>

* Set the base branch to the default branch of the repository

Signed-off-by: Shinnosuke Sawada-Dazai <shin@warashi.dev>

* Use `github.ref_name` to determine the version of manifests

Signed-off-by: Shinnosuke Sawada-Dazai <shin@warashi.dev>

* Checkout master branch to make PR to master branch

Signed-off-by: Shinnosuke Sawada-Dazai <shin@warashi.dev>

---------

Signed-off-by: Shinnosuke Sawada-Dazai <shin@warashi.dev>
Signed-off-by: pipecd-bot <pipecd.dev@gmail.com>

* Remove pipectl quickstart command (#5268)

* Remove pipectl quickstart command

Signed-off-by: khanhtc1202 <khanhtc1202@gmail.com>

* Remove the quickstart from docs

Signed-off-by: khanhtc1202 <khanhtc1202@gmail.com>

---------

Signed-off-by: khanhtc1202 <khanhtc1202@gmail.com>
Signed-off-by: pipecd-bot <pipecd.dev@gmail.com>

* Enabled to configure the interval of livestate store for Lambda (#5269)

* add liveStateInterval for Lambda

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

* add docs of liveStateInterval

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

* fix a test

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

* Rename to 'awsAPIPollingInterval'

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

---------

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

* Update RELEASE to v0.49.2 and sync docs of v0.49.x (#5272)

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

---------

Signed-off-by: Shinnosuke Sawada-Dazai <shin@warashi.dev>
Signed-off-by: pipecd-bot <pipecd.dev@gmail.com>
Signed-off-by: khanhtc1202 <khanhtc1202@gmail.com>
Signed-off-by: t-kikuc <tkikuchi07f@gmail.com>
Co-authored-by: Shinnosuke Sawada-Dazai <shin@warashi.dev>
Co-authored-by: Khanh Tran <32532742+khanhtc1202@users.noreply.github.com>
Co-authored-by: Tetsuya Kikuchi <97105818+t-kikuc@users.noreply.github.com>
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.

Easier quickstart for PipeCD
3 participants