migrate ocm from urfave/cli to spf13/cobra

Signed-off-by: Christian Richter <c.richter@opencloud.eu>
This commit is contained in:
Christian Richter
2025-12-02 15:27:48 +01:00
committed by Florian Schade
parent f43909de77
commit 680423707c
4 changed files with 33 additions and 30 deletions

View File

@@ -2,15 +2,16 @@ package command
import ( import (
"github.com/opencloud-eu/opencloud/services/ocm/pkg/config" "github.com/opencloud-eu/opencloud/services/ocm/pkg/config"
"github.com/urfave/cli/v2"
"github.com/spf13/cobra"
) )
// Health is the entrypoint for the health command. // Health is the entrypoint for the health command.
func Health(cfg *config.Config) *cli.Command { func Health(cfg *config.Config) *cobra.Command {
return &cli.Command{ return &cobra.Command{
Name: "health", Use: "health",
Usage: "Check health status", Short: "Check health status",
Action: func(c *cli.Context) error { RunE: func(cmd *cobra.Command, args []string) error {
// Not implemented // Not implemented
return nil return nil
}, },

View File

@@ -5,12 +5,13 @@ import (
"github.com/opencloud-eu/opencloud/pkg/clihelper" "github.com/opencloud-eu/opencloud/pkg/clihelper"
"github.com/opencloud-eu/opencloud/services/ocm/pkg/config" "github.com/opencloud-eu/opencloud/services/ocm/pkg/config"
"github.com/urfave/cli/v2"
"github.com/spf13/cobra"
) )
// GetCommands provides all commands for this service // GetCommands provides all commands for this service
func GetCommands(cfg *config.Config) cli.Commands { func GetCommands(cfg *config.Config) []*cobra.Command {
return []*cli.Command{ return []*cobra.Command{
// start this service // start this service
Server(cfg), Server(cfg),
@@ -24,11 +25,12 @@ func GetCommands(cfg *config.Config) cli.Commands {
// Execute is the entry point for the ocm command. // Execute is the entry point for the ocm command.
func Execute(cfg *config.Config) error { func Execute(cfg *config.Config) error {
app := clihelper.DefaultApp(&cli.App{ app := clihelper.DefaultAppCobra(&cobra.Command{
Name: "ocm", Use: "ocm",
Usage: "starts ocm service", Short: "starts ocm service",
Commands: GetCommands(cfg),
}) })
app.AddCommand(GetCommands(cfg)...)
app.SetArgs(os.Args[1:])
return app.RunContext(cfg.Context, os.Args) return app.ExecuteContext(cfg.Context)
} }

View File

@@ -16,21 +16,21 @@ import (
"github.com/opencloud-eu/opencloud/services/ocm/pkg/revaconfig" "github.com/opencloud-eu/opencloud/services/ocm/pkg/revaconfig"
"github.com/opencloud-eu/opencloud/services/ocm/pkg/server/debug" "github.com/opencloud-eu/opencloud/services/ocm/pkg/server/debug"
"github.com/opencloud-eu/reva/v2/cmd/revad/runtime" "github.com/opencloud-eu/reva/v2/cmd/revad/runtime"
"github.com/urfave/cli/v2"
"github.com/spf13/cobra"
) )
// Server is the entrypoint for the server command. // Server is the entrypoint for the server command.
func Server(cfg *config.Config) *cli.Command { func Server(cfg *config.Config) *cobra.Command {
return &cli.Command{ return &cobra.Command{
Name: "server", Use: "server",
Usage: fmt.Sprintf("start the %s service without runtime (unsupervised mode)", cfg.Service.Name), Short: fmt.Sprintf("start the %s service without runtime (unsupervised mode)", cfg.Service.Name),
Category: "server", PreRunE: func(cmd *cobra.Command, args []string) error {
Before: func(c *cli.Context) error {
return configlog.ReturnFatal(parser.ParseConfig(cfg)) return configlog.ReturnFatal(parser.ParseConfig(cfg))
}, },
Action: func(c *cli.Context) error { RunE: func(cmd *cobra.Command, args []string) error {
logger := logging.Configure(cfg.Service.Name, cfg.Log) logger := logging.Configure(cfg.Service.Name, cfg.Log)
traceProvider, err := tracing.GetTraceProvider(c.Context, cfg.Commons.TracesExporter, cfg.Service.Name) traceProvider, err := tracing.GetTraceProvider(cmd.Context(), cfg.Commons.TracesExporter, cfg.Service.Name)
if err != nil { if err != nil {
return err return err
} }

View File

@@ -2,16 +2,16 @@ package command
import ( import (
"github.com/opencloud-eu/opencloud/services/ocm/pkg/config" "github.com/opencloud-eu/opencloud/services/ocm/pkg/config"
"github.com/urfave/cli/v2"
"github.com/spf13/cobra"
) )
// Version prints the service versions of all running instances. // Version prints the service versions of all running instances.
func Version(cfg *config.Config) *cli.Command { func Version(cfg *config.Config) *cobra.Command {
return &cli.Command{ return &cobra.Command{
Name: "version", Use: "version",
Usage: "print the version of this binary and the running service instances", Short: "print the version of this binary and the running service instances",
Category: "info", RunE: func(cmd *cobra.Command, args []string) error {
Action: func(c *cli.Context) error {
// not implemented // not implemented
return nil return nil
}, },