Merge pull request #379 from defang-io/lio-cli-delete

Deprecate delete command

Author: lionello

src/cmd/cli/command/commands.go

@@ -873,6 +873,7 @@ var deleteCmd = &cobra.Command{
 	Args:        cobra.MinimumNArgs(1),
 	Aliases:     []string{"del", "rm", "remove"},
 	Short:       "Delete a service from the cluster",
+	Deprecated:  "use 'compose stop' instead",
 	RunE: func(cmd *cobra.Command, names []string) error {
 		var tail, _ = cmd.Flags().GetBool("tail")
 
@@ -969,7 +970,7 @@ var logoutCmd = &cobra.Command{
 var bootstrapCmd = &cobra.Command{
 	Use:     "cd",
 	Aliases: []string{"bootstrap"},
-	Short:   "Manually run a command with the CD task",
+	Short:   "Manually run a command with the CD task (for BYOC only)",
 }
 
 var bootstrapDestroyCmd = &cobra.Command{

src/pkg/cli/client/byoc/aws/byoc.go

@@ -372,9 +372,12 @@ func (b *ByocAws) environment() map[string]string {
 func (b *ByocAws) runCdCommand(ctx context.Context, cmd ...string) (ecs.TaskArn, error) {
 	env := b.environment()
 	if term.DoDebug {
-		debugEnv := " -"
+		debugEnv := fmt.Sprintf("AWS_REGION=%q", b.driver.Region)
+		if awsProfile := os.Getenv("AWS_PROFILE"); awsProfile != "" {
+			debugEnv += fmt.Sprintf(" AWS_PROFILE=%q", awsProfile)
+		}
 		for k, v := range env {
-			debugEnv += " " + k + "=" + v
+			debugEnv += fmt.Sprintf(" %s=%q", k, v)
 		}
 		term.Debug(debugEnv, "npm run dev", strings.Join(cmd, " "))
 	}

src/pkg/cli/composeDown.go

@@ -14,5 +14,10 @@ func ComposeDown(ctx context.Context, client client.Client) (types.ETag, error)
 		return "", err
 	}
 	term.Debug(" - Destroying project", projectName)
+
+	if DoDryRun {
+		return "", ErrDryRun
+	}
+
 	return client.Destroy(ctx)
 }