feat: support charts from sub-folders inside git repository

Signed-off-by: Dominykas Blyžė <[email protected]>

Author: dominykas

internal/gitutil/gitutil.go

@@ -31,8 +31,9 @@ import (
 var gitRepositoryURLRe = regexp.MustCompile(`^git(\+\w+)?://`)
 
 type GitRepositoryURL struct {
-	RepositoryURL string
-	GitRemoteURL  *url.URL
+	RepositoryURL          string
+	GitRemoteURL           *url.URL
+	PathUnderGitRepository string
 }
 
 // HasGitReference returns true if a git repository contains a specified ref (branch/tag)
@@ -71,8 +72,22 @@ func ParseGitRepositoryURL(repositoryURL string) (GitRepositoryURL, error) {
 		return GitRepositoryURL{}, errors.Errorf("git repository URL should not contain credentials - please use git credential helpers")
 	}
 
+	path := ""
+
+	if gitRemoteURL.Fragment != "" {
+		query, err := url.ParseQuery(gitRemoteURL.Fragment)
+		if err != nil {
+			return GitRepositoryURL{}, err
+		}
+
+		path = query.Get("subdirectory")
+	}
+
+	gitRemoteURL.Fragment = ""
+
 	return GitRepositoryURL{
-		RepositoryURL: repositoryURL,
-		GitRemoteURL:  gitRemoteURL,
+		RepositoryURL:          repositoryURL,
+		GitRemoteURL:           gitRemoteURL,
+		PathUnderGitRepository: path,
 	}, err
 }

internal/gitutil/gitutil_test.go

@@ -41,29 +41,39 @@ func TestIsGitUrl(t *testing.T) {
 func TestParseGitRepositoryURL(t *testing.T) {
 	// Test table: Given url, ParseGitRepositoryURL should return expect.
 	tests := []struct {
-		url                    string
-		expectRepositoryURL    string
-		expectGitRepositoryURL string
+		url                            string
+		expectRepositoryURL            string
+		expectGitRemoteURL             string
+		expectedPathUnderGitRepository string
 	}{
 		{
-			url:                    "git://example.com/example/chart",
-			expectRepositoryURL:    "git://example.com/example/chart",
-			expectGitRepositoryURL: "git://example.com/example/chart",
+			url:                 "git://example.com/example/chart",
+			expectRepositoryURL: "git://example.com/example/chart",
+			expectGitRemoteURL:  "git://example.com/example/chart",
 		},
 		{
-			url:                    "git+https://example.com/example/chart",
-			expectRepositoryURL:    "git+https://example.com/example/chart",
-			expectGitRepositoryURL: "https://example.com/example/chart",
+			url:                 "git+https://example.com/example/chart",
+			expectRepositoryURL: "git+https://example.com/example/chart",
+			expectGitRemoteURL:  "https://example.com/example/chart",
+		},
+		{
+			url:                            "git+https://example.com/example/chart#subdirectory=charts/some-chart",
+			expectRepositoryURL:            "git+https://example.com/example/chart#subdirectory=charts/some-chart",
+			expectGitRemoteURL:             "https://example.com/example/chart",
+			expectedPathUnderGitRepository: "charts/some-chart",
 		},
 	}
 
 	for _, test := range tests {
 		parsed, _ := ParseGitRepositoryURL(test.url)
 		if parsed.RepositoryURL != test.expectRepositoryURL {
-			t.Errorf("Expected RepositoryURL %s for %s, but got %s", test.expectRepositoryURL, test.url, parsed)
+			t.Errorf("Expected RepositoryURL %s for %s, but got %s", test.expectRepositoryURL, test.url, parsed.RepositoryURL)
+		}
+		if parsed.GitRemoteURL.String() != test.expectGitRemoteURL {
+			t.Errorf("Expected GitRemoteURL %s for %s, but got %s", test.expectGitRemoteURL, test.url, parsed.GitRemoteURL)
 		}
-		if parsed.GitRemoteURL.String() != test.expectGitRepositoryURL {
-			t.Errorf("Expected GitRemoteURL %s for %s, but got %s", test.expectGitRepositoryURL, test.url, parsed)
+		if parsed.PathUnderGitRepository != test.expectedPathUnderGitRepository {
+			t.Errorf("Expected PathUnderGitRepository %s for %s, but got %s", test.expectGitRemoteURL, test.url, parsed.PathUnderGitRepository)
 		}
 	}
 }

pkg/getter/gitgetter.go

@@ -78,7 +78,7 @@ func (g *GitGetter) get(href string) (*bytes.Buffer, error) {
 		return nil, err
 	}
 
-	ch, err := loader.LoadDir(chartTmpDir)
+	ch, err := loader.LoadDir(filepath.Join(chartTmpDir, gitURL.PathUnderGitRepository))
 	if err != nil {
 		return nil, err
 	}