Ignore bosh release with no Github repo for release-notes cmd & updat…

…e trainstat url
pivotal-cf · Oct 8, 2024 · af77e11 · af77e11
1 parent 852058e
commit af77e11
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 4 deletions.
diff --git a/pkg/cargo/bump.go b/pkg/cargo/bump.go
@@ -240,13 +240,19 @@ func fetchReleasesFromRepo(ctx context.Context, repoService RepositoryReleaseLis
 }
 
 func fetchReleasesForBump(ctx context.Context, kf Kilnfile, bump Bump, client githubClientFunc) Bump {
-	lister, err := client(ctx, kf, bump.To)
+	spec, err := kf.BOSHReleaseTarballSpecification(bump.Name)
 	if err != nil {
-		log.Println(err)
 		return bump
 	}
-	spec, err := kf.BOSHReleaseTarballSpecification(bump.Name)
+
+	// Ignores release notes for Bosh releases with empty Github repository
+	if spec.GitHubRepository == "" {
+		return bump
+	}
+
+	lister, err := client(ctx, kf, bump.To)
 	if err != nil {
+		log.Println(err)
 		return bump
 	}
 

diff --git a/pkg/notes/notes_data.go b/pkg/notes/notes_data.go
@@ -102,7 +102,7 @@ type IssuesQuery struct {
 }
 
 type TrainstatQuery struct {
-	TrainstatURL string `long:"trainstat-url" short:"tu" description:"trainstat url to fetch the release notes for component bumps" default:"https://trainstat.sc2-04-pcf1-apps.oc.vmware.com"`
+	TrainstatURL string `long:"trainstat-url" short:"tu" description:"trainstat url to fetch the release notes for component bumps" default:"https://tas-trainstat.eng.tanzu.broadcom.com"`
 }
 
 func TrainstatURL() string {