Skip to content

Commit

Permalink
Merge branch 'main' into add-readiness-liveness-probe
Browse files Browse the repository at this point in the history
  • Loading branch information
davidjumani authored Nov 15, 2024
2 parents 6b5438b + 833eb9d commit 8297c90
Show file tree
Hide file tree
Showing 3 changed files with 12 additions and 6 deletions.
6 changes: 6 additions & 0 deletions changelog/v1.18.0-beta35/helm-doc-gen.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,6 @@
changelog:
- type: NON_USER_FACING
issueLink: https://github.com/solo-io/solo-projects/issues/6888
resolvesIssue: false
description: >-
Update helm public docs generation to work from main branch and only pull in released changes
8 changes: 3 additions & 5 deletions docs/cmd/generate_docs.go
Original file line number Diff line number Diff line change
Expand Up @@ -454,11 +454,9 @@ func fetchEnterpriseHelmValues(_ []string) error {
if err != nil {
return err
}
version, err := semver.NewVersion(string(semverReleaseTag))
if err != nil {
return err
}
minorReleaseTag := fmt.Sprintf("v%d.%d.x", version.Major(), version.Minor())

minorReleaseTag := "v" + string(semverReleaseTag)

files, err := githubutils.GetFilesFromGit(ctx, client, repoOwner, glooEnterpriseRepo, minorReleaseTag, path)
if err != nil {
return err
Expand Down
4 changes: 3 additions & 1 deletion pkg/github-action-utils/version.go
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,9 @@ func GetLatestEnterpriseVersion(repoRootPath string, repo string, owner string)
return err
}
defer f.Close()
enterpriseVersion, err := version.GetLatestHelmChartVersionWithMaxVersion(version.EnterpriseHelmRepoIndex, version.GlooEE, true, maxGlooEVersion)
// get the latest version from the helm repo, include unstable versions so it works from the main branches
// for LTS branches, unstable versions will be filtered out by the version constraints
enterpriseVersion, err := version.GetLatestHelmChartVersionWithMaxVersion(version.EnterpriseHelmRepoIndex, version.GlooEE, false, maxGlooEVersion)
if err != nil {
return err
}
Expand Down

0 comments on commit 8297c90

Please sign in to comment.