Merge pull request #1207 from fibonacci1729/fix/helm-list

fix(helm-list): filter configmaps by owner=TILLER

pkg/storage/driver/cfgmaps.go

@@ -84,7 +84,10 @@ func (cfgmaps *ConfigMaps) Get(key string) (*rspb.Release, error) {
 // that filter(release) == true. An error is returned if the
 // configmap fails to retrieve the releases.
 func (cfgmaps *ConfigMaps) List(filter func(*rspb.Release) bool) ([]*rspb.Release, error) {
-	list, err := cfgmaps.impl.List(api.ListOptions{})
+	lsel := kblabels.Set{"OWNER": "TILLER"}.AsSelector()
+	opts := api.ListOptions{LabelSelector: lsel}
+
+	list, err := cfgmaps.impl.List(opts)
 	if err != nil {
 		logerrf(err, "list: failed to list")
 		return nil, err

pkg/storage/driver/records.go

@@ -125,6 +125,7 @@ func newRecord(key string, rls *rspb.Release) *record {
 
 	lbs.init()
 	lbs.set("NAME", rls.Name)
+	lbs.set("OWNER", "TILLER")
 	lbs.set("STATUS", rspb.Status_Code_name[int32(rls.Info.Status.Code)])
 	lbs.set("VERSION", strconv.Itoa(int(rls.Version)))
 

pkg/storage/storage.go

@@ -113,6 +113,7 @@ func (s *Storage) Deployed(name string) (*rspb.Release, error) {
 
 	ls, err := s.Driver.Query(map[string]string{
 		"NAME":   name,
+		"OWNER":  "TILLER",
 		"STATUS": "DEPLOYED",
 	})
 	switch {