-
Notifications
You must be signed in to change notification settings - Fork 142
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
feat: Implement the GetHelmRelease endpoint
Add endpoint to get a single HelmRelease. Additional changes: * Moved `ListHelmReleases` and `GetHelmRelease` endpoint definitions into their own file, to get a better separation (`helm.go`). * Moved the tests for `ListHelmReleases` and `GetHelmRelease` endpoints into the corresponding test file (`helm_test.go`). Closes #1498
- Loading branch information
Showing
10 changed files
with
876 additions
and
398 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,48 @@ | ||
package server | ||
|
||
import ( | ||
"context" | ||
|
||
helmv2 "github.com/fluxcd/helm-controller/api/v2beta1" | ||
"github.com/weaveworks/weave-gitops/core/server/types" | ||
pb "github.com/weaveworks/weave-gitops/pkg/api/core" | ||
) | ||
|
||
func (cs *coreServer) ListHelmReleases(ctx context.Context, msg *pb.ListHelmReleasesRequest) (*pb.ListHelmReleasesResponse, error) { | ||
k8s, err := cs.k8s.Client(ctx) | ||
if err != nil { | ||
return nil, doClientError(err) | ||
} | ||
|
||
l := &helmv2.HelmReleaseList{} | ||
|
||
if err := list(ctx, k8s, temporarilyEmptyAppName, msg.Namespace, l); err != nil { | ||
return nil, err | ||
} | ||
|
||
var results []*pb.HelmRelease | ||
for _, repository := range l.Items { | ||
results = append(results, types.HelmReleaseToProto(&repository)) | ||
} | ||
|
||
return &pb.ListHelmReleasesResponse{ | ||
HelmReleases: results, | ||
}, nil | ||
} | ||
|
||
func (cs *coreServer) GetHelmRelease(ctx context.Context, msg *pb.GetHelmReleaseRequest) (*pb.GetHelmReleaseResponse, error) { | ||
k8s, err := cs.k8s.Client(ctx) | ||
if err != nil { | ||
return nil, doClientError(err) | ||
} | ||
|
||
helmRelease := helmv2.HelmRelease{} | ||
|
||
if err = get(ctx, k8s, msg.Name, msg.Namespace, &helmRelease); err != nil { | ||
return nil, err | ||
} | ||
|
||
return &pb.GetHelmReleaseResponse{ | ||
HelmRelease: types.HelmReleaseToProto(&helmRelease), | ||
}, nil | ||
} |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,109 @@ | ||
package server | ||
|
||
import ( | ||
"context" | ||
"testing" | ||
|
||
helmv2 "github.com/fluxcd/helm-controller/api/v2beta1" | ||
. "github.com/onsi/gomega" | ||
pb "github.com/weaveworks/weave-gitops/pkg/api/core" | ||
"github.com/weaveworks/weave-gitops/pkg/kube" | ||
"k8s.io/apimachinery/pkg/util/rand" | ||
"sigs.k8s.io/controller-runtime/pkg/client" | ||
) | ||
|
||
func TestListHelmReleases(t *testing.T) { | ||
g := NewGomegaWithT(t) | ||
ctx := context.Background() | ||
|
||
c, cleanup := makeGRPCServer(k8sEnv.Rest, t) | ||
defer cleanup() | ||
|
||
_, k, err := kube.NewKubeHTTPClientWithConfig(k8sEnv.Rest, "") | ||
g.Expect(err).NotTo(HaveOccurred()) | ||
|
||
appName := "myapp" | ||
ns := newNamespace(ctx, k, g) | ||
|
||
newHelmRelease(ctx, appName, ns.Name, k, g) | ||
|
||
res, err := c.ListHelmReleases(ctx, &pb.ListHelmReleasesRequest{ | ||
Namespace: ns.Name, | ||
}) | ||
g.Expect(err).NotTo(HaveOccurred()) | ||
g.Expect(res.HelmReleases).To(HaveLen(1)) | ||
g.Expect(res.HelmReleases[0].Name).To(Equal(appName)) | ||
} | ||
|
||
func TestGetHelmRelease(t *testing.T) { | ||
g := NewGomegaWithT(t) | ||
ctx := context.Background() | ||
|
||
c, cleanup := makeGRPCServer(k8sEnv.Rest, t) | ||
defer cleanup() | ||
|
||
_, k, err := kube.NewKubeHTTPClientWithConfig(k8sEnv.Rest, "") | ||
g.Expect(err).NotTo(HaveOccurred()) | ||
|
||
appName := "myapp" + rand.String(5) | ||
ns1 := newNamespace(ctx, k, g) | ||
ns2 := newNamespace(ctx, k, g) | ||
ns3 := newNamespace(ctx, k, g) | ||
|
||
newHelmRelease(ctx, appName, ns1.Name, k, g) | ||
newHelmRelease(ctx, appName, ns2.Name, k, g) | ||
|
||
// Get app from ns1. | ||
response, err := c.GetHelmRelease(ctx, &pb.GetHelmReleaseRequest{ | ||
Name: appName, | ||
Namespace: ns1.Name, | ||
}) | ||
|
||
g.Expect(err).NotTo(HaveOccurred()) | ||
g.Expect(response.HelmRelease.Name).To(Equal(appName)) | ||
g.Expect(response.HelmRelease.Namespace).To(Equal(ns1.Name)) | ||
|
||
// Get app from ns2. | ||
response, err = c.GetHelmRelease(ctx, &pb.GetHelmReleaseRequest{ | ||
Name: appName, | ||
Namespace: ns2.Name, | ||
}) | ||
|
||
g.Expect(err).NotTo(HaveOccurred()) | ||
g.Expect(response.HelmRelease.Name).To(Equal(appName)) | ||
g.Expect(response.HelmRelease.Namespace).To(Equal(ns2.Name)) | ||
|
||
// Get app from ns3, should fail. | ||
response, err = c.GetHelmRelease(ctx, &pb.GetHelmReleaseRequest{ | ||
Name: appName, | ||
Namespace: ns3.Name, | ||
}) | ||
|
||
g.Expect(err).To(HaveOccurred()) | ||
} | ||
|
||
func newHelmRelease( | ||
ctx context.Context, | ||
appName, nsName string, | ||
k client.Client, | ||
g *GomegaWithT, | ||
) helmv2.HelmRelease { | ||
release := helmv2.HelmRelease{ | ||
Spec: helmv2.HelmReleaseSpec{ | ||
Chart: helmv2.HelmChartTemplate{ | ||
Spec: helmv2.HelmChartTemplateSpec{ | ||
SourceRef: helmv2.CrossNamespaceObjectReference{ | ||
Kind: "GitRepository", | ||
Name: "somesource", | ||
}, | ||
}, | ||
}, | ||
}, | ||
} | ||
release.Name = appName | ||
release.Namespace = nsName | ||
|
||
g.Expect(k.Create(ctx, &release)).To(Succeed()) | ||
|
||
return release | ||
} |
Oops, something went wrong.