Add the ability to force the sync strategy via SYNC_APPLICATION command

[nghialv]

What this PR does / why we need it:

Which issue(s) this PR fixes:

Fixes #1165

Does this PR introduce a user-facing change?:

NONE

[pipecd-bot]

Code coverage for golang is 33.77%. This pull request decreases coverage by -0.04%.

File	Function	Base	Head	Diff
pkg/app/piped/planner/kubernetes/pipeline.go	buildQuickSyncPipeline	--	85.71%	+85.71%
pkg/app/piped/planner/kubernetes/kubernetes.go	Planner.Plan	0.00%	0.00%	+0.00%
pkg/app/piped/planner/kubernetes/pipeline.go	buildPipeline	85.71%	--	-85.71%

[pipecd-bot]

Code coverage for golang is 33.77%. This pull request decreases coverage by -0.04%.

File	Function	Base	Head	Diff
pkg/app/piped/planner/kubernetes/pipeline.go	buildQuickSyncPipeline	--	85.71%	+85.71%
pkg/app/piped/planner/kubernetes/kubernetes.go	Planner.Plan	0.00%	0.00%	+0.00%
pkg/app/piped/planner/kubernetes/pipeline.go	buildPipeline	85.71%	--	-85.71%

[cakecatz]

@nghialv We need to modify syncApplication like this to put syncStrategy on the request.

https://github.com/pipe-cd/pipe/blob/sync/pkg/app/web/src/api/applications.ts#L97-L105

export const syncApplication = async ({
  applicationId,
  syncStrategy, // added
}: SyncApplicationRequest.AsObject): Promise<
  SyncApplicationResponse.AsObject
> => {
  const req = new SyncApplicationRequest();
  req.setApplicationId(applicationId);
  req.setSyncStrategy(syncStrategy); // added
  return apiRequest(req, apiClient.syncApplication);
};

[nghialv]

@cakecatz Right. I added them.

[pipecd-bot]

Code coverage for golang is 33.77%. This pull request decreases coverage by -0.04%.

File	Function	Base	Head	Diff
pkg/app/piped/planner/kubernetes/pipeline.go	buildQuickSyncPipeline	--	85.71%	+85.71%
pkg/app/piped/planner/kubernetes/kubernetes.go	Planner.Plan	0.00%	0.00%	+0.00%
pkg/app/piped/planner/kubernetes/pipeline.go	buildPipeline	85.71%	--	-85.71%

[pipecd-bot]

Code coverage for golang is 33.77%. This pull request decreases coverage by -0.04%.

File	Function	Base	Head	Diff
pkg/app/piped/planner/kubernetes/pipeline.go	buildQuickSyncPipeline	--	85.71%	+85.71%
pkg/app/piped/planner/kubernetes/kubernetes.go	Planner.Plan	0.00%	0.00%	+0.00%
pkg/app/piped/planner/kubernetes/pipeline.go	buildPipeline	85.71%	--	-85.71%

[pipecd-bot]

Code coverage for javascript is 81.29%. This pull request increases coverage by 0.03%.

File	Base	Head	Diff
src/modules/applications.ts	85.71%	86.00%	+0.29%
src/fixtures/dummy-deployment.ts	100.00%	100.00%	+0.00%
src/api/applications.ts	28.21%	29.11%	+0.91%

[cakecatz]

/lgtm

[nakabonne]

Left a comment about the strategy name, but looks pretty neat

[nakabonne]

NONE is a bit confusable to me. I feel like it would be better to use AUTO or UNDEFINED or something like that.

[nghialv]

Nice. AUTO looks good.

[nakabonne]

/lgtm

[pipecd-bot]

Code coverage for golang is 33.77%. This pull request decreases coverage by -0.04%.

File	Function	Base	Head	Diff
pkg/app/piped/planner/kubernetes/pipeline.go	buildQuickSyncPipeline	--	85.71%	+85.71%
pkg/app/piped/planner/kubernetes/kubernetes.go	Planner.Plan	0.00%	0.00%	+0.00%
pkg/app/piped/planner/kubernetes/pipeline.go	buildPipeline	85.71%	--	-85.71%

[pipecd-bot]

Code coverage for javascript is 81.29%. This pull request increases coverage by 0.03%.

File	Base	Head	Diff
src/fixtures/dummy-deployment.ts	100.00%	100.00%	+0.00%
src/api/applications.ts	28.21%	29.11%	+0.91%
src/modules/applications.ts	85.71%	86.00%	+0.29%

[khanhtc1202]

/approve

[pipecd-bot]

This pull request is APPROVED by khanhtc1202.

Approvers can cancel the approval by writing /approve cancel in a comment. Any additional commits also will change this pull request to be not-approved.