Install multiple plugins by passing in a file argument to porter plugins install #2504

VinozzZ · 2022-12-28T22:33:29Z

What does this change

This PR adds a new --file flag to the porter plugins install command.
This allows users to define multiple plugins in a file and install them with a single porter command.

What issue does it fix

Closes #2491

Notes for the reviewer

Maybe we need to define a schema for this plugins config file?

Checklist

Did you write tests?
Did you write documentation?
Did you change porter.yaml or a storage document record? Update the corresponding schema file.
If this is your first pull request, please add your name to the bottom of our Contributors list. Thank you for making Porter better! 🙇‍♀️

Reviewer Checklist

Comment with /azp run test-porter-release if a magefile or build script was modified
Comment with /azp run porter-integration if it's a non-trivial PR

Signed-off-by: Yingrong Zhao <yingrong.zhao@gmail.com>

pkg/plugins/install.go

pkg/plugins/install_test.go

pkg/porter/plugins.go

sgettys · 2023-01-05T17:47:45Z

Need docs/schema around new plugins config file format

Signed-off-by: Yingrong Zhao <yingrong.zhao@gmail.com>

…ins install (getporter#2504) Signed-off-by: Yingrong Zhao <yingrong.zhao@gmail.com>

VinozzZ force-pushed the plugins-install-file branch 3 times, most recently from c18d7a2 to feca084 Compare January 4, 2023 20:08

add --file flag to porter plugins install command

b36ede4

Signed-off-by: Yingrong Zhao <yingrong.zhao@gmail.com>

VinozzZ force-pushed the plugins-install-file branch from feca084 to b36ede4 Compare January 4, 2023 20:17

VinozzZ marked this pull request as ready for review January 4, 2023 20:17

VinozzZ requested review from carolynvs and vdice as code owners January 4, 2023 20:17

VinozzZ requested review from bdegeeter and sgettys January 5, 2023 17:21