Fix bug that Launcher could not load the new format of Piped config

[nghialv]

What this PR does / why we need it:

Which issue(s) this PR fixes:

Fixes #

Does this PR introduce a user-facing change?:

NONE

[pipecd-bot]

Some issues were detected while linting go source files in your changes.

[pipecd-bot]

type name will be used as launcher.LauncherSpec by other packages, and that stutters; consider calling this Spec

https://golang.org/wiki/CodeReviewComments#package-names

[pipecd-bot]

Code coverage for golang is 32.41%. This pull request decreases coverage by -0.05%.

File	Function	Base	Head	Diff
pkg/app/launcher/cmd/launcher/launcher.go	Config.Validate	--	0.00%	+0.00%
pkg/app/launcher/cmd/launcher/launcher.go	LauncherSpec.LoadPipedKey	--	0.00%	+0.00%
pkg/app/launcher/cmd/launcher/launcher.go	parseConfig	0.00%	0.00%	+0.00%

[knanao]

Nice catch!
/lgtm

[khanhtc1202]

nits, is it enough to make this struct just a private struct?

[knanao]

I thought it would be okay to define it in /config. but it's up to you.

[nghialv]

Sure. Fixed them. 👍

[nghialv]

I thought it would be okay to define it in /config. but it's up to you.

I feel you. That is nice too. Let me apply your suggestion. 👍

[pipecd-bot]

Code coverage for golang is 32.41%. This pull request decreases coverage by -0.05%.

File	Function	Base	Head	Diff
pkg/app/launcher/cmd/launcher/launcher.go	launcherConfig.Validate	--	0.00%	+0.00%
pkg/app/launcher/cmd/launcher/launcher.go	launcherSpec.LoadPipedKey	--	0.00%	+0.00%
pkg/app/launcher/cmd/launcher/launcher.go	parseConfig	0.00%	0.00%	+0.00%

[khanhtc1202]

Nice 👍
/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.

[khanhtc1202]

btw, I think we may need to add something like: "Fix bug that Launcher could not load the new format of Piped config" as this PR introduces a user-facing change block.

[nghialv]

btw, I think we may need to add something like: "Fix bug that Launcher could not load the new format of Piped config" as this PR introduces a user-facing change block.

Oops. I was thinking that I had already configured it.