Artifactory Release Lifecycle Management - Add Export bundle command

artifactory/cli.go

@@ -948,37 +948,6 @@ func GetCommands() []cli.Command {
 	})
 }
 
-func getSplitCount(c *cli.Context, defaultSplitCount, maxSplitCount int) (splitCount int, err error) {
-	splitCount = defaultSplitCount
-	err = nil
-	if c.String("split-count") != "" {
-		splitCount, err = strconv.Atoi(c.String("split-count"))
-		if err != nil {
-			err = errors.New("The '--split-count' option should have a numeric value. " + cliutils.GetDocumentationMessage())
-		}
-		if splitCount > maxSplitCount {
-			err = errors.New("The '--split-count' option value is limited to a maximum of " + strconv.Itoa(maxSplitCount) + ".")
-		}
-		if splitCount < 0 {
-			err = errors.New("the '--split-count' option cannot have a negative value")
-		}
-	}
-	return
-}
-
-func getMinSplit(c *cli.Context, defaultMinSplit int64) (minSplitSize int64, err error) {
-	minSplitSize = defaultMinSplit
-	if c.String(cliutils.MinSplit) != "" {
-		minSplitSize, err = strconv.ParseInt(c.String(cliutils.MinSplit), 10, 64)
-		if err != nil {
-			err = errors.New("The '--min-split' option should have a numeric value. " + cliutils.GetDocumentationMessage())
-			return 0, err
-		}
-	}
-
-	return minSplitSize, nil
-}
-
 func getRetries(c *cli.Context) (retries int, err error) {
 	retries = cliutils.Retries
 	if c.String("retries") != "" {
@@ -1238,7 +1207,7 @@ func downloadCmd(c *cli.Context) error {
 		return err
 	}
 	fixWinPathsForDownloadCmd(downloadSpec, c)
-	configuration, err := createDownloadConfiguration(c)
+	configuration, err := cliutils.CreateDownloadConfiguration(c)
 	if err != nil {
 		return err
 	}
@@ -1299,7 +1268,7 @@ func uploadCmd(c *cli.Context) (err error) {
 		return
 	}
 	cliutils.FixWinPathsForFileSystemSourcedCmds(uploadSpec, c)
-	configuration, err := createUploadConfiguration(c)
+	configuration, err := cliutils.CreateUploadConfiguration(c)
 	if err != nil {
 		return
 	}
@@ -2421,15 +2390,6 @@ func transferSettingsCmd(_ *cli.Context) error {
 	return commands.Exec(transferSettingsCmd)
 }
 
-func getDebFlag(c *cli.Context) (deb string, err error) {
-	deb = c.String("deb")
-	slashesCount := strings.Count(deb, "/") - strings.Count(deb, "\\/")
-	if deb != "" && slashesCount != 2 {
-		return "", errors.New("the --deb option should be in the form of distribution/component/architecture")
-	}
-	return deb, nil
-}
-
 func createDefaultCopyMoveSpec(c *cli.Context) (*spec.SpecFiles, error) {
 	offset, limit, err := getOffsetAndLimitValues(c)
 	if err != nil {
@@ -2635,25 +2595,6 @@ func createDefaultDownloadSpec(c *cli.Context) (*spec.SpecFiles, error) {
 		BuildSpec(), nil
 }
 
-func createDownloadConfiguration(c *cli.Context) (downloadConfiguration *utils.DownloadConfiguration, err error) {
-	downloadConfiguration = new(utils.DownloadConfiguration)
-	downloadConfiguration.MinSplitSize, err = getMinSplit(c, cliutils.DownloadMinSplitKb)
-	if err != nil {
-		return nil, err
-	}
-	downloadConfiguration.SplitCount, err = getSplitCount(c, cliutils.DownloadSplitCount, cliutils.DownloadMaxSplitCount)
-	if err != nil {
-		return nil, err
-	}
-	downloadConfiguration.Threads, err = cliutils.GetThreadsCount(c)
-	if err != nil {
-		return nil, err
-	}
-	downloadConfiguration.SkipChecksum = c.Bool("skip-checksum")
-	downloadConfiguration.Symlink = true
-	return
-}
-
 func setTransitiveInDownloadSpec(downloadSpec *spec.SpecFiles) {
 	transitive := os.Getenv(coreutils.TransitiveDownload)
 	if transitive == "" {
@@ -2713,27 +2654,6 @@ func fixWinPathsForDownloadCmd(uploadSpec *spec.SpecFiles, c *cli.Context) {
 	}
 }
 
-func createUploadConfiguration(c *cli.Context) (uploadConfiguration *utils.UploadConfiguration, err error) {
-	uploadConfiguration = new(utils.UploadConfiguration)
-	uploadConfiguration.MinSplitSizeMB, err = getMinSplit(c, cliutils.UploadMinSplitMb)
-	if err != nil {
-		return nil, err
-	}
-	uploadConfiguration.SplitCount, err = getSplitCount(c, cliutils.UploadSplitCount, cliutils.UploadMaxSplitCount)
-	if err != nil {
-		return nil, err
-	}
-	uploadConfiguration.Threads, err = cliutils.GetThreadsCount(c)
-	if err != nil {
-		return nil, err
-	}
-	uploadConfiguration.Deb, err = getDebFlag(c)
-	if err != nil {
-		return
-	}
-	return
-}
-
 func getOffsetAndLimitValues(c *cli.Context) (offset, limit int, err error) {
 	offset, err = cliutils.GetIntFlagValue(c, "offset", 0)
 	if err != nil {

artifactory/cli_test.go

@@ -142,7 +142,7 @@ func TestCreateUploadConfiguration(t *testing.T) {
 	for _, testCase := range createUploadConfigurationCases {
 		t.Run(testCase.name, func(t *testing.T) {
 			context, _ := tests.CreateContext(t, testCase.flags, []string{})
-			uploadConfiguration, err := createUploadConfiguration(context)
+			uploadConfiguration, err := cliutils.CreateUploadConfiguration(context)
 			assert.NoError(t, err)
 			assert.Equal(t, testCase.expectedMinSplit, uploadConfiguration.MinSplitSizeMB)
 			assert.Equal(t, testCase.expectedSplitCount, uploadConfiguration.SplitCount)

docs/lifecycle/export/help.go

@@ -0,0 +1,15 @@
+package export
+
+var Usage = []string{"rbe [command options] <release bundle name> <release bundle version>"}
+
+func GetDescription() string {
+	return "Export and download a Release Bundle"
+}
+
+func GetArguments() string {
+	return `	release bundle name
+		Name of the Release Bundle to export.
+
+	release bundle version
+		Version of the Release Bundle to export.`
+}

go.mod

@@ -130,9 +130,9 @@ require (
 	gopkg.in/yaml.v3 v3.0.1 // indirect
 )
 
-replace github.com/jfrog/jfrog-cli-core/v2 => github.com/jfrog/jfrog-cli-core/v2 v2.31.1-0.20240318065853-b785b7d7f693
+replace github.com/jfrog/jfrog-cli-core/v2 => github.com/eyaldelarea/jfrog-cli-core/v2 v2.0.0-20240318093441-5596edb9ddf6
 
-replace github.com/jfrog/jfrog-client-go => github.com/jfrog/jfrog-client-go v1.28.1-0.20240318065424-90669dbbcc54
+replace github.com/jfrog/jfrog-client-go => github.com/eyaldelarea/jfrog-client-go v1.28.4-0.20240318091644-cd1b2ea37878
 
 replace github.com/jfrog/jfrog-cli-security => github.com/jfrog/jfrog-cli-security v1.0.4-0.20240303113253-d82ecc02d9eb
 

go.sum

@@ -78,6 +78,10 @@ github.com/dsnet/golib v0.0.0-20171103203638-1ea166775780/go.mod h1:Lj+Z9rebOhdf
 github.com/elazarl/goproxy v0.0.0-20230808193330-2592e75ae04a h1:mATvB/9r/3gvcejNsXKSkQ6lcIaNec2nyfOdlTBR2lU=
 github.com/emirpasic/gods v1.18.1 h1:FXtiHYKDGKCW2KzwZKx0iC0PQmdlorYgdFG9jPXJ1Bc=
 github.com/emirpasic/gods v1.18.1/go.mod h1:8tpGGwCnJ5H4r6BWwaV6OrWmMoPhUl5jm/FMNAnJvWQ=
+github.com/eyaldelarea/jfrog-cli-core/v2 v2.0.0-20240318093441-5596edb9ddf6 h1:C8KH3OwVQ3x9BXtfXyc8j79UNqk7fzjw271VQmLtpHU=
+github.com/eyaldelarea/jfrog-cli-core/v2 v2.0.0-20240318093441-5596edb9ddf6/go.mod h1:cfXM0WcFuAVpRjMLqCyV4kLW8/5DQA9SidzUnZcWYyY=
+github.com/eyaldelarea/jfrog-client-go v1.28.4-0.20240318091644-cd1b2ea37878 h1:RU1vSHXmXk405ak3WizaiZH6yhfAapVrw3NY5LrVP6A=
+github.com/eyaldelarea/jfrog-client-go v1.28.4-0.20240318091644-cd1b2ea37878/go.mod h1:knm1qYrsNqkeElD7pxCoVD8DSDnanhugVs42slz48U4=
 github.com/forPelevin/gomoji v1.1.8 h1:JElzDdt0TyiUlecy6PfITDL6eGvIaxqYH1V52zrd0qQ=
 github.com/forPelevin/gomoji v1.1.8/go.mod h1:8+Z3KNGkdslmeGZBC3tCrwMrcPy5GRzAD+gL9NAwMXg=
 github.com/frankban/quicktest v1.14.6 h1:7Xjx+VpznH+oBnejlPUj8oUpdxnVs4f8XU8WnHkI4W8=
@@ -127,12 +131,8 @@ github.com/jfrog/gofrog v1.6.3 h1:F7He0+75HcgCe6SGTSHLFCBDxiE2Ja0tekvvcktW6wc=
 github.com/jfrog/gofrog v1.6.3/go.mod h1:SZ1EPJUruxrVGndOzHd+LTiwWYKMlHqhKD+eu+v5Hqg=
 github.com/jfrog/jfrog-apps-config v1.0.1 h1:mtv6k7g8A8BVhlHGlSveapqf4mJfonwvXYLipdsOFMY=
 github.com/jfrog/jfrog-apps-config v1.0.1/go.mod h1:8AIIr1oY9JuH5dylz2S6f8Ym2MaadPLR6noCBO4C22w=
-github.com/jfrog/jfrog-cli-core/v2 v2.31.1-0.20240318065853-b785b7d7f693 h1:VgaSrFtNNfgSzRFxg7kzQeqm6q/ZeetaH4uSPt0YP+Q=
-github.com/jfrog/jfrog-cli-core/v2 v2.31.1-0.20240318065853-b785b7d7f693/go.mod h1:mWr+fwcpgoNDTqBeYmXN0AU4FwC/a90MlzL0SoNrMC0=
 github.com/jfrog/jfrog-cli-security v1.0.4-0.20240303113253-d82ecc02d9eb h1:JUqwNOqzwj+1oZQXY1p/xnnH97BEMfOKGG505vGVIe4=
 github.com/jfrog/jfrog-cli-security v1.0.4-0.20240303113253-d82ecc02d9eb/go.mod h1:pOqg/hGw6eHWNyLDHeOCyq/Ntuc+wLJTV2Z8XuwXspU=
-github.com/jfrog/jfrog-client-go v1.28.1-0.20240318065424-90669dbbcc54 h1:FTrss/ffJPjTHOOhQ8P+8DrkGYkxcaHlxp12nOfeZJQ=
-github.com/jfrog/jfrog-client-go v1.28.1-0.20240318065424-90669dbbcc54/go.mod h1:NB8tYFgkWtn+wHsKC+aYC75aLnS6yW81d8JAFTBxsi0=
 github.com/jszwec/csvutil v1.10.0 h1:upMDUxhQKqZ5ZDCs/wy+8Kib8rZR8I8lOR34yJkdqhI=
 github.com/jszwec/csvutil v1.10.0/go.mod h1:/E4ONrmGkwmWsk9ae9jpXnv9QT8pLHEPcCirMFhxG9I=
 github.com/kevinburke/ssh_config v1.2.0 h1:x584FjTGwHzMwvHx18PXxbBVzfnxogHaAReU4gf13a4=

lifecycle/cli.go

@@ -12,9 +12,12 @@ import (
 	rbDeleteLocal "github.com/jfrog/jfrog-cli/docs/lifecycle/deletelocal"
 	rbDeleteRemote "github.com/jfrog/jfrog-cli/docs/lifecycle/deleteremote"
 	rbDistribute "github.com/jfrog/jfrog-cli/docs/lifecycle/distribute"
+	rbExport "github.com/jfrog/jfrog-cli/docs/lifecycle/export"
 	rbPromote "github.com/jfrog/jfrog-cli/docs/lifecycle/promote"
 	"github.com/jfrog/jfrog-cli/utils/cliutils"
 	"github.com/jfrog/jfrog-cli/utils/distribution"
+	artClientUtils "github.com/jfrog/jfrog-client-go/artifactory/services/utils"
+	"github.com/jfrog/jfrog-client-go/lifecycle/services"
 	"github.com/jfrog/jfrog-client-go/utils"
 	"github.com/jfrog/jfrog-client-go/utils/errorutils"
 	"github.com/urfave/cli"
@@ -61,6 +64,18 @@ func GetCommands() []cli.Command {
 			Category:     lcCategory,
 			Action:       distribute,
 		},
+		{
+			Name:         "release-bundle-export",
+			Aliases:      []string{"rbe"},
+			Flags:        cliutils.GetCommandFlags(cliutils.ReleaseBundleExport),
+			Usage:        rbExport.GetDescription(),
+			HelpName:     coreCommon.CreateUsage("rbe", rbExport.GetDescription(), rbExport.Usage),
+			UsageText:    rbExport.GetArguments(),
+			ArgsUsage:    common.CreateEnvVars(),
+			BashComplete: coreCommon.CreateBashCompletionFunc(),
+			Category:     lcCategory,
+			Action:       export,
+		},
 		{
 			Name:         "release-bundle-delete-local",
 			Aliases:      []string{"rbdell"},
@@ -242,6 +257,24 @@ func deleteRemote(c *cli.Context) error {
 	return commands.Exec(deleteCmd)
 }
 
+func export(c *cli.Context) error {
+	lcDetails, err := createLifecycleDetailsByFlags(c)
+	if err != nil {
+		return err
+	}
+	exportCmd, modifications := initReleaseBundleExportCmd(c)
+	downloadConfig, err := cliutils.CreateDownloadConfiguration(c)
+	if err != nil {
+		return err
+	}
+	exportCmd.
+		SetServerDetails(lcDetails).
+		SetReleaseBundleExportModifications(modifications).
+		SetDownloadConfiguration(*downloadConfig)
+
+	return commands.Exec(exportCmd)
+}
+
 func validateDistributeCommand(c *cli.Context) error {
 	if err := distribution.ValidateReleaseBundleDistributeCmd(c); err != nil {
 		return err
@@ -287,3 +320,21 @@ func splitRepos(c *cli.Context, reposOptionKey string) []string {
 	}
 	return nil
 }
+
+func initReleaseBundleExportCmd(c *cli.Context) (command *lifecycle.ReleaseBundleExportCommand, modifications services.Modifications) {
+	command = lifecycle.NewReleaseBundleExportCommand().
+		SetReleaseBundleName(c.Args().Get(0)).
+		SetReleaseBundleVersion(c.Args().Get(1)).
+		SetProject(c.String(cliutils.Project)).
+		SetTargetPath(c.String(cliutils.TargetDirectory))
+
+	modifications = services.Modifications{
+		PathMappings: []artClientUtils.PathMapping{
+			{
+				Input:  c.String(cliutils.PathMappingPattern),
+				Output: c.String(cliutils.PathMappingTarget),
+			},
+		},
+	}
+	return
+}

lifecycle_test.go

@@ -16,6 +16,7 @@ import (
 	"github.com/jfrog/jfrog-client-go/lifecycle/services"
 	clientUtils "github.com/jfrog/jfrog-client-go/utils"
 	"github.com/jfrog/jfrog-client-go/utils/errorutils"
+	"github.com/jfrog/jfrog-client-go/utils/io/fileutils"
 	clientTestUtils "github.com/jfrog/jfrog-client-go/utils/tests"
 	"github.com/stretchr/testify/assert"
 	"net/http"
@@ -73,6 +74,10 @@ func TestLifecycle(t *testing.T) {
 	// Assert no artifacts were promoted to prod repo 2.
 	assertExpectedArtifacts(t, tests.SearchAllProdRepo2, []string{})
 
+	// Export release lifecycle bundle archive
+	exportRb(t, tests.LcRbName2, number2)
+	defer deleteExportedReleaseBundle(t, tests.LcRbName2)
+
 	// TODO Temporarily disabling till distribution on testing suite is stable.
 	/*
 		distributeRb(t)
@@ -84,6 +89,10 @@ func TestLifecycle(t *testing.T) {
 
 }
 
+func deleteExportedReleaseBundle(t *testing.T, rbName string) {
+	assert.NoError(t, os.RemoveAll(rbName))
+}
+
 func assertExpectedArtifacts(t *testing.T, specFileName string, expected []string) {
 	searchProdSpec, err := tests.CreateSpec(specFileName)
 	assert.NoError(t, err)
@@ -118,6 +127,18 @@ func createRb(t *testing.T, specName, sourceOption, rbName, rbVersion string, sy
 	assert.NoError(t, lcCli.Exec(argsAndOptions...))
 }
 
+func exportRb(t *testing.T, rbName, rbVersion string) {
+	output := lcCli.RunCliCmdWithOutput(t, "rbe", rbName, rbVersion)
+	var exportResp *services.ReleaseBundleExportedStatusResponse
+	if !assert.NoError(t, json.Unmarshal([]byte(output), &exportResp)) {
+		return
+	}
+	assert.Equal(t, exportResp.Status, services.ExportCompleted)
+	exists, err := fileutils.IsDirExists(rbName, false)
+	assert.NoError(t, err)
+	assert.Equal(t, true, exists)
+}
+
 /*
 func distributeRb(t *testing.T) {
 	distributionRulesPath := filepath.Join(tests.GetTestResourcesPath(), "distribution", tests.DistributionRules)

utils/cliutils/commandsflags.go

@@ -123,6 +123,7 @@ const (
 	ReleaseBundleDistribute   = "release-bundle-distribute"
 	ReleaseBundleDeleteLocal  = "release-bundle-delete-local"
 	ReleaseBundleDeleteRemote = "release-bundle-delete-remote"
+	ReleaseBundleExport       = "release-bundle-export"
 
 	// Access Token Create commands keys
 	AccessTokenCreate = "access-token-create"
@@ -448,13 +449,14 @@ const (
 	deleteFromDist        = "delete-from-dist"
 
 	// Common release-bundle-* v1&v2 flags
-	DistRules      = "dist-rules"
-	site           = "site"
-	city           = "city"
-	countryCodes   = "country-codes"
-	sync           = "sync"
-	maxWaitMinutes = "max-wait-minutes"
-	CreateRepo     = "create-repo"
+	DistRules       = "dist-rules"
+	site            = "site"
+	city            = "city"
+	countryCodes    = "country-codes"
+	sync            = "sync"
+	maxWaitMinutes  = "max-wait-minutes"
+	CreateRepo      = "create-repo"
+	TargetDirectory = target + "-directory"
 
 	// *** Xray Commands' flags ***
 	// Base flags
@@ -1713,6 +1715,10 @@ var flagsMap = map[string]cli.Flag{
 		Name:  Reference,
 		Usage: "[Default: false] Generate a Reference Token (alias to Access Token) in addition to the full token (available from Artifactory 7.38.10)` `",
 	},
+	TargetDirectory: cli.StringFlag{
+		Name:  TargetDirectory,
+		Usage: "[Optional] Specify the destination PATH where the downloaded file will be saved. If not provided, the file will be saved in the current directory.` `",
+	},
 }
 
 var commandFlags = map[string][]string{
@@ -2016,6 +2022,10 @@ var commandFlags = map[string][]string{
 		platformUrl, user, password, accessToken, serverId, deleteQuiet, lcDryRun, DistRules, site, city, countryCodes,
 		lcSync, maxWaitMinutes, lcProject,
 	},
+	ReleaseBundleExport: {
+		platformUrl, user, password, accessToken, serverId, lcPathMappingTarget, lcPathMappingPattern, Project,
+		rbRepo, downloadMinSplit, downloadSplitCount, TargetDirectory,
+	},
 	// Mission Control's commands
 	McConfig: {
 		mcUrl, mcAccessToken, mcInteractive,