feat: [sc-103119] URI is not replacing the spec but merging instead (#1537)

fix uri spec to replace origin spec

Author: nvanthao

cmd/troubleshoot/cli/run.go

@@ -273,7 +273,8 @@ func loadSupportBundleSpecsFromURIs(ctx context.Context, kinds *loader.Troublesh
 		return err
 	}
 
-	kinds.Add(moreKinds)
+	// uri spec replaces the original spec
+	*kinds = *moreKinds
 	return nil
 }
 

cmd/troubleshoot/cli/run_test.go

@@ -57,8 +57,9 @@ spec:
 	err = loadSupportBundleSpecsFromURIs(ctx, kinds)
 	require.NoError(t, err)
 
-	require.Len(t, kinds.SupportBundlesV1Beta2, 2)
-	assert.NotNil(t, kinds.SupportBundlesV1Beta2[1].Spec.Collectors[0].ClusterInfo)
+	// valid uri spec will replace the original spec
+	require.Len(t, kinds.SupportBundlesV1Beta2, 1)
+	assert.NotNil(t, kinds.SupportBundlesV1Beta2[0].Spec.Collectors[0].ClusterInfo)
 }
 
 func Test_loadSupportBundleSpecsFromURIs_TimeoutError(t *testing.T) {