Move the --farm flag to farm build command

The option `farm` which is used to specify the farm to be used, is moved to farm build command from farm command.

closes #20752

Signed-off-by: Chetan Giradkar <cgiradka@redhat.com>
This commit is contained in:
Chetan Giradkar
2023-11-28 15:50:27 +00:00
parent 06c41b614d
commit 31df9c237e
5 changed files with 27 additions and 23 deletions

View File

@@ -19,6 +19,7 @@ type buildOptions struct {
buildOptions common.BuildFlagsWrapper
local bool
platforms []string
farm string
}
var (
@@ -55,6 +56,16 @@ func init() {
buildCommand.PersistentFlags().StringSliceVar(&buildOpts.platforms, platformsFlag, nil, "Build only on farm nodes that match the given platforms")
common.DefineBuildFlags(buildCommand, &buildOpts.buildOptions, true)
podmanConfig := registry.PodmanConfig()
farmFlagName := "farm"
// If remote, don't read the client's containers.conf file
defaultFarm := ""
if !registry.IsRemote() {
defaultFarm = podmanConfig.ContainersConfDefaultsRO.Farms.Default
}
flags.StringVar(&buildOpts.farm, farmFlagName, defaultFarm, "Farm to use for builds")
}
func build(cmd *cobra.Command, args []string) error {
@@ -98,8 +109,8 @@ func build(cmd *cobra.Command, args []string) error {
}
defaultFarm := cfg.Farms.Default
if farmCmd.Flags().Changed("farm") {
f, err := farmCmd.Flags().GetString("farm")
if cmd.Flags().Changed("farm") {
f, err := cmd.Flags().GetString("farm")
if err != nil {
return err
}

View File

@@ -16,27 +16,9 @@ var (
}
)
var (
// Temporary struct to hold cli values.
farmOpts = struct {
Farm string
}{}
)
func init() {
registry.Commands = append(registry.Commands, registry.CliCommand{
Command: farmCmd,
})
farmCmd.Hidden = true
flags := farmCmd.Flags()
podmanConfig := registry.PodmanConfig()
farmFlagName := "farm"
// If remote, don't read the client's containers.conf file
defaultFarm := ""
if !registry.IsRemote() {
defaultFarm = podmanConfig.ContainersConfDefaultsRO.Farms.Default
}
flags.StringVarP(&farmOpts.Farm, farmFlagName, "f", defaultFarm, "Farm to use for builds")
}