Bump 'v1beta1' API to 'v1'

And simultaneously change the group domain to 'helm.fluxcd.io'.

chart/helm-operator/README.md

@@ -48,7 +48,7 @@ Install podinfo by referring to its Helm repository:
 
 ```sh
 cat <<EOF | kubectl apply -f -
-apiVersion: flux.weave.works/v1beta1
+apiVersion: helm.fluxcd.io/v1
 kind: HelmRelease
 metadata:
   name: podinfo
@@ -119,7 +119,7 @@ helm -i helm-operator fluxcd/helm-operator \
 You can refer to a chart from your private Git with:
 
 ```yaml
-apiVersion: flux.weave.works/v1beta1
+apiVersion: helm.fluxcd.io
 kind: HelmRelease
 metadata:
   name: some-app

chart/helm-operator/templates/crd.yaml

@@ -3,7 +3,7 @@
 apiVersion: apiextensions.k8s.io/v1beta1
 kind: CustomResourceDefinition
 metadata:
-  name: helmreleases.flux.weave.works
+  name: helmreleases.helm.fluxcd.io
   labels:
     app: {{ template "helm-operator.name" . }}
     chart: {{ template "helm-operator.chart" . }}
@@ -12,7 +12,7 @@ metadata:
   annotations:
     "helm.sh/resource-policy": keep
 spec:
-  group: flux.weave.works
+  group: helm.fluxcd.io
   names:
     kind: HelmRelease
     listKind: HelmReleaseList
@@ -22,9 +22,9 @@ spec:
   scope: Namespaced
   subresources:
     status: {}
-  version: v1beta1
+  version: v1
   versions:
-    - name: v1beta1
+    - name: v1
       served: true
       storage: true
   validation:

cmd/helm-operator/main.go

@@ -185,7 +185,7 @@ func main() {
 	// setup shared informer for HelmReleases
 	nsOpt := ifinformers.WithNamespace(*namespace)
 	ifInformerFactory := ifinformers.NewSharedInformerFactoryWithOptions(ifClient, *chartsSyncInterval, nsOpt)
-	fhrInformer := ifInformerFactory.Flux().V1beta1().HelmReleases()
+	hrInformer := ifInformerFactory.Helm().V1().HelmReleases()
 
 	// setup workqueue for HelmReleases
 	queue := workqueue.NewNamedRateLimitingQueue(workqueue.DefaultControllerRateLimiter(), "ChartRelease")
@@ -195,7 +195,7 @@ func main() {
 	rel := release.New(log.With(logger, "component", "release"), helmClient)
 	chartSync := chartsync.New(
 		log.With(logger, "component", "chartsync"),
-		chartsync.Clients{KubeClient: *kubeClient, IfClient: *ifClient, FhrLister: fhrInformer.Lister()},
+		chartsync.Clients{KubeClient: *kubeClient, IfClient: *ifClient, HrLister: hrInformer.Lister()},
 		rel,
 		queue,
 		chartsync.Config{LogDiffs: *logReleaseDiffs, UpdateDeps: *updateDependencies, GitTimeout: *gitTimeout, GitPollInterval: *gitPollInterval},
@@ -206,12 +206,12 @@ func main() {
 	// NB: the operator needs to do its magic with the informer
 	// _before_ starting it or else the cache sync seems to hang at
 	// random
-	opr := operator.New(log.With(logger, "component", "operator"), *logReleaseDiffs, kubeClient, fhrInformer, queue, chartSync)
+	opr := operator.New(log.With(logger, "component", "operator"), *logReleaseDiffs, kubeClient, hrInformer, queue, chartSync)
 	go ifInformerFactory.Start(shutdown)
 
 	// wait for the caches to be synced before starting _any_ workers
 	mainLogger.Log("info", "waiting for informer caches to sync")
-	if ok := cache.WaitForCacheSync(shutdown, fhrInformer.Informer().HasSynced); !ok {
+	if ok := cache.WaitForCacheSync(shutdown, hrInformer.Informer().HasSynced); !ok {
 		mainLogger.Log("error", "failed to wait for caches to sync")
 		os.Exit(1)
 	}
@@ -225,7 +225,7 @@ func main() {
 
 	// the status updater, to keep track of the release status for
 	// every HelmRelease
-	statusUpdater := status.New(ifClient, fhrInformer.Lister(), helmClient)
+	statusUpdater := status.New(ifClient, hrInformer.Lister(), helmClient)
 	go statusUpdater.Loop(shutdown, log.With(logger, "component", "statusupdater"))
 
 	// start HTTP server

deploy/flux-helm-release-crd.yaml

@@ -2,9 +2,9 @@
 apiVersion: apiextensions.k8s.io/v1beta1
 kind: CustomResourceDefinition
 metadata:
-  name: helmreleases.flux.weave.works
+  name: helmreleases.helm.fluxcd.io
 spec:
-  group: flux.weave.works
+  group: helm.fluxcd.io
   names:
     kind: HelmRelease
     listKind: HelmReleaseList
@@ -14,9 +14,9 @@ spec:
   scope: Namespaced
   subresources:
     status: {}
-  version: v1beta1
+  version: v1
   versions:
-    - name: v1beta1
+    - name: v1
       served: true
       storage: true
   validation:

docs/references/helmrelease-custom-resource.md

@@ -6,7 +6,7 @@ resource definition](https://github.com/fluxcd/flux/blob/master/deploy-helm/flux
 look like this:
 
 ```yaml
-apiVersion: flux.weave.works/v1beta1
+apiVersion: helm.fluxcd.io/v1
 kind: HelmRelease
 metadata:
   name: rabbit
@@ -103,7 +103,7 @@ values.yaml`, but inlined into the `HelmRelease` manifest. For
 example,
 
 ```yaml
-apiVersion: flux.weave.works/v1beta1
+apiVersion: helm.fluxcd.io/v1
 kind: HelmRelease
 # metadata: ...
 spec:
@@ -211,7 +211,7 @@ detects a change in values and/or the chart.
 ### Configuration
 
 ```yaml
-apiVersion: flux.weave.works/v1beta1
+apiVersion: helm.fluxcd.io/v1
 kind: HelmRelease
 # metadata: ...
 spec:

hack/update-codegen.sh

@@ -31,7 +31,7 @@ chmod +x ${CODEGEN_PKG}/generate-groups.sh
 # :magic:
 ${CODEGEN_PKG}/generate-groups.sh all \
     github.com/fluxcd/helm-operator/pkg/client github.com/fluxcd/helm-operator/pkg/apis \
-    "flux.weave.works:v1beta1" \
+    "helm.fluxcd.io:v1" \
     --output-base "${TEMP_DIR}" \
     --go-header-file "${SCRIPT_ROOT}/hack/custom-boilerplate.go.txt"
 

pkg/apis/helm.fluxcd.io/register.go

@@ -0,0 +1,5 @@
+package helm
+
+const (
+	GroupName = "helm.fluxcd.io"
+)

pkg/apis/helm.fluxcd.io/v1/doc.go

@@ -0,0 +1,5 @@
+// +k8s:deepcopy-gen=package,register
+// +groupName=helm.fluxcd.io
+// Package v1 is the v1 version of the API. The prior
+// version was flux.weave.works/v1beta1.
+package v1

pkg/apis/flux.weave.works/v1beta1/register.go → pkg/apis/helm.fluxcd.io/v1/register.go

@@ -1,15 +1,17 @@
-package v1beta1
+package v1
 
 import (
 	metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
 	"k8s.io/apimachinery/pkg/runtime"
 	"k8s.io/apimachinery/pkg/runtime/schema"
 
-	fluxintegrations "github.com/fluxcd/helm-operator/pkg/apis/flux.weave.works"
+	"github.com/fluxcd/helm-operator/pkg/apis/helm.fluxcd.io"
 )
 
+const Version = "v1"
+
 // SchemeGroupVersion is group version used to register these objects
-var SchemeGroupVersion = schema.GroupVersion{Group: fluxintegrations.GroupName, Version: "v1beta1"}
+var SchemeGroupVersion = schema.GroupVersion{Group: helm.GroupName, Version: Version}
 
 // Resource takes an unqualified resource and returns a Group qualified GroupResource
 func Resource(resource string) schema.GroupResource {

pkg/apis/flux.weave.works/v1beta1/types.go → pkg/apis/helm.fluxcd.io/v1/types.go

@@ -1,11 +1,11 @@
-package v1beta1
+package v1
 
 import (
 	"fmt"
 	"strings"
 
 	"github.com/ghodss/yaml"
-	v1 "k8s.io/api/core/v1"
+	"k8s.io/api/core/v1"
 	metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
 	"k8s.io/helm/pkg/chartutil"
 
@@ -27,46 +27,46 @@ type HelmRelease struct {
 // ResourceID returns an ID made from the identifying parts of the
 // resource, as a convenience for Flux, which uses them
 // everywhere.
-func (fhr HelmRelease) ResourceID() resource.ID {
-	return resource.MakeID(fhr.Namespace, "HelmRelease", fhr.Name)
+func (hr HelmRelease) ResourceID() resource.ID {
+	return resource.MakeID(hr.Namespace, "HelmRelease", hr.Name)
 }
 
 // ReleaseName returns the configured release name, or constructs and
 // returns one based on the namespace and name of the HelmRelease.
 // When the HelmRelease's metadata.namespace and spec.targetNamespace
 // differ, both are used in the generated name.
 // This name is used for naming and operating on the release in Helm.
-func (fhr HelmRelease) ReleaseName() string {
-	if fhr.Spec.ReleaseName == "" {
-		namespace := fhr.GetDefaultedNamespace()
-		targetNamespace := fhr.GetTargetNamespace()
+func (hr HelmRelease) ReleaseName() string {
+	if hr.Spec.ReleaseName == "" {
+		namespace := hr.GetDefaultedNamespace()
+		targetNamespace := hr.GetTargetNamespace()
 
 		if namespace != targetNamespace {
 			// prefix the releaseName with the administering HelmRelease namespace as well
-			return fmt.Sprintf("%s-%s-%s", namespace, targetNamespace, fhr.Name)
+			return fmt.Sprintf("%s-%s-%s", namespace, targetNamespace, hr.Name)
 		}
-		return fmt.Sprintf("%s-%s", targetNamespace, fhr.Name)
+		return fmt.Sprintf("%s-%s", targetNamespace, hr.Name)
 	}
 
-	return fhr.Spec.ReleaseName
+	return hr.Spec.ReleaseName
 }
 
 // GetDefaultedNamespace returns the HelmRelease's namespace
 // defaulting to the "default" if not set.
-func (fhr HelmRelease) GetDefaultedNamespace() string {
-	if fhr.GetNamespace() == "" {
+func (hr HelmRelease) GetDefaultedNamespace() string {
+	if hr.GetNamespace() == "" {
 		return "default"
 	}
-	return fhr.Namespace
+	return hr.Namespace
 }
 
 // GetTargetNamespace returns the configured release targetNamespace
 // defaulting to the namespace of the HelmRelease if not set.
-func (fhr HelmRelease) GetTargetNamespace() string {
-	if fhr.Spec.TargetNamespace == "" {
-		return fhr.GetDefaultedNamespace()
+func (hr HelmRelease) GetTargetNamespace() string {
+	if hr.Spec.TargetNamespace == "" {
+		return hr.GetDefaultedNamespace()
 	}
-	return fhr.Spec.TargetNamespace
+	return hr.Spec.TargetNamespace
 }
 
 // ValuesFromSource represents a source of values.
@@ -184,21 +184,21 @@ type HelmReleaseSpec struct {
 }
 
 // GetTimeout returns the install or upgrade timeout (defaults to 300s)
-func (r HelmRelease) GetTimeout() int64 {
-	if r.Spec.Timeout == nil {
+func (hr HelmRelease) GetTimeout() int64 {
+	if hr.Spec.Timeout == nil {
 		return 300
 	}
-	return *r.Spec.Timeout
+	return *hr.Spec.Timeout
 }
 
 // GetValuesFromSources maintains backwards compatibility with
 // ValueFileSecrets by merging them into the ValuesFrom array.
-func (r HelmRelease) GetValuesFromSources() []ValuesFromSource {
-	valuesFrom := r.Spec.ValuesFrom
+func (hr HelmRelease) GetValuesFromSources() []ValuesFromSource {
+	valuesFrom := hr.Spec.ValuesFrom
 	// Maintain backwards compatibility with ValueFileSecrets
-	if r.Spec.ValueFileSecrets != nil {
+	if hr.Spec.ValueFileSecrets != nil {
 		var secretKeyRefs []ValuesFromSource
-		for _, ref := range r.Spec.ValueFileSecrets {
+		for _, ref := range hr.Spec.ValueFileSecrets {
 			s := &v1.SecretKeySelector{LocalObjectReference: ref}
 			secretKeyRefs = append(secretKeyRefs, ValuesFromSource{SecretKeyRef: s})
 		}

pkg/apis/flux.weave.works/v1beta1/types_test.go → pkg/apis/helm.fluxcd.io/v1/types_test.go

@@ -1,4 +1,4 @@
-package v1beta1
+package v1
 
 import (
 	"testing"