Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Replace obsolete versions into apps/v1 #1012

Merged
merged 24 commits into from Oct 21, 2019
Merged
Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
24 commits
Select commit Hold shift + click to select a range
3083616
removed extensions
Yisaer Oct 14, 2019
ff95dc5
remove v1beta2
Yisaer Oct 14, 2019
9a42afb
remove apps/v1beta1
Yisaer Oct 15, 2019
75f98da
Merge remote-tracking branch 'upstream/master' into fix_obsolete_vers…
Yisaer Oct 15, 2019
080fb74
Merge remote-tracking branch 'upstream/master' into fix_obsolete_vers…
Yisaer Oct 15, 2019
6d7c979
chang go code
Yisaer Oct 15, 2019
c57eab3
fix
Yisaer Oct 15, 2019
e9fff9a
fix
Yisaer Oct 15, 2019
902eeeb
deepcopy gen
Yisaer Oct 15, 2019
d0459bf
Merge branch 'master' into fix_obsolete_versions
Yisaer Oct 16, 2019
456358c
Merge remote-tracking branch 'upstream/master' into fix_obsolete_vers…
Yisaer Oct 16, 2019
e994ae0
Update kind-cluster-build.sh
Yisaer Oct 16, 2019
a05e72b
Merge branch 'master' into fix_obsolete_versions
Yisaer Oct 16, 2019
b95db14
fix conlict
Yisaer Oct 16, 2019
37f62b6
Merge branch 'fix_obsolete_versions' of https://github.com/Yisaer/tid…
Yisaer Oct 16, 2019
e1cf2f4
go mod tidy
Yisaer Oct 17, 2019
d92d990
Merge remote-tracking branch 'upstream/master' into fix_obsolete_vers…
Yisaer Oct 17, 2019
986cf6d
Merge branch 'master' into fix_obsolete_versions
Yisaer Oct 18, 2019
d9c23fa
Merge remote-tracking branch 'upstream/master' into fix_obsolete_vers…
Yisaer Oct 18, 2019
a2e321b
Merge remote-tracking branch 'upstream/master' into fix_obsolete_vers…
Yisaer Oct 18, 2019
71403a3
clean code
Yisaer Oct 18, 2019
0497d08
Merge branch 'master' into fix_obsolete_versions
Yisaer Oct 21, 2019
aee37b5
clean code
Yisaer Oct 21, 2019
d432c97
Merge branch 'master' into fix_obsolete_versions
Yisaer Oct 21, 2019
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
2 changes: 1 addition & 1 deletion manifests/webhook.yaml
Expand Up @@ -105,5 +105,5 @@ webhooks:
rules:
- operations: [ "UPDATE" ]
apiGroups: [ "apps", "" ]
apiVersions: ["v1beta1"]
apiVersions: ["v1"]
resources: ["statefulsets"]
2 changes: 1 addition & 1 deletion pkg/apis/pingcap.com/v1alpha1/tidbcluster_test.go
Expand Up @@ -17,7 +17,7 @@ import (
"testing"

. "github.com/onsi/gomega"
apps "k8s.io/api/apps/v1beta1"
apps "k8s.io/api/apps/v1"
corev1 "k8s.io/api/core/v1"
metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
"k8s.io/apimachinery/pkg/types"
Expand Down
12 changes: 1 addition & 11 deletions pkg/apis/pingcap.com/v1alpha1/types.go
Expand Up @@ -14,7 +14,7 @@
package v1alpha1

import (
apps "k8s.io/api/apps/v1beta1"
apps "k8s.io/api/apps/v1"
corev1 "k8s.io/api/core/v1"
metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
"k8s.io/apimachinery/pkg/types"
Expand Down Expand Up @@ -59,7 +59,6 @@ const (

// +genclient
// +k8s:deepcopy-gen:interfaces=k8s.io/apimachinery/pkg/runtime.Object

// +k8s:openapi-gen=true
// TidbCluster is the control script's spec
type TidbCluster struct {
Expand All @@ -76,7 +75,6 @@ type TidbCluster struct {
}

// +k8s:deepcopy-gen:interfaces=k8s.io/apimachinery/pkg/runtime.Object

// +k8s:openapi-gen=true
// TidbClusterList is TidbCluster list
type TidbClusterList struct {
Expand Down Expand Up @@ -292,9 +290,6 @@ type TiKVFailureStore struct {

// +genclient
// +k8s:deepcopy-gen:interfaces=k8s.io/apimachinery/pkg/runtime.Object

// +k8s:openapi-gen=true
// Backup is a backup of tidb cluster.
Yisaer marked this conversation as resolved.
Show resolved Hide resolved
type Backup struct {
metav1.TypeMeta `json:",inline"`
// +k8s:openapi-gen=false
Expand All @@ -306,7 +301,6 @@ type Backup struct {
}

// +k8s:deepcopy-gen:interfaces=k8s.io/apimachinery/pkg/runtime.Object

// +k8s:openapi-gen=true
// BackupList contains a list of Backup.
type BackupList struct {
Expand Down Expand Up @@ -420,7 +414,6 @@ type BackupStatus struct {

// +genclient
// +k8s:deepcopy-gen:interfaces=k8s.io/apimachinery/pkg/runtime.Object

// +k8s:openapi-gen=true
// BackupSchedule is a backup schedule of tidb cluster.
type BackupSchedule struct {
Expand All @@ -434,7 +427,6 @@ type BackupSchedule struct {
}

// +k8s:deepcopy-gen:interfaces=k8s.io/apimachinery/pkg/runtime.Object

// +k8s:openapi-gen=true
// BackupScheduleList contains a list of BackupSchedule.
type BackupScheduleList struct {
Expand Down Expand Up @@ -472,7 +464,6 @@ type BackupScheduleStatus struct {

// +genclient
// +k8s:deepcopy-gen:interfaces=k8s.io/apimachinery/pkg/runtime.Object

// +k8s:openapi-gen=true
// Restore represents the restoration of backup of a tidb cluster.
type Restore struct {
Expand All @@ -486,7 +477,6 @@ type Restore struct {
}

// +k8s:deepcopy-gen:interfaces=k8s.io/apimachinery/pkg/runtime.Object

// +k8s:openapi-gen=true
// RestoreList contains a list of Restore.
type RestoreList struct {
Expand Down
8 changes: 4 additions & 4 deletions pkg/apis/pingcap.com/v1alpha1/zz_generated.deepcopy.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion pkg/controller/controller_utils_test.go
Expand Up @@ -21,7 +21,7 @@ import (

. "github.com/onsi/gomega"
"github.com/pingcap/tidb-operator/pkg/apis/pingcap.com/v1alpha1"
apps "k8s.io/api/apps/v1beta1"
apps "k8s.io/api/apps/v1"
batchv1 "k8s.io/api/batch/v1"
corev1 "k8s.io/api/core/v1"
metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
Expand Down
12 changes: 6 additions & 6 deletions pkg/controller/stateful_set_control.go
Expand Up @@ -21,13 +21,13 @@ import (
"github.com/pingcap/tidb-operator/pkg/apis/pingcap.com/v1alpha1"
tcinformers "github.com/pingcap/tidb-operator/pkg/client/informers/externalversions/pingcap.com/v1alpha1"
v1listers "github.com/pingcap/tidb-operator/pkg/client/listers/pingcap.com/v1alpha1"
apps "k8s.io/api/apps/v1beta1"
apps "k8s.io/api/apps/v1"
corev1 "k8s.io/api/core/v1"
apierrors "k8s.io/apimachinery/pkg/api/errors"
utilruntime "k8s.io/apimachinery/pkg/util/runtime"
appsinformers "k8s.io/client-go/informers/apps/v1beta1"
appsinformers "k8s.io/client-go/informers/apps/v1"
"k8s.io/client-go/kubernetes"
appslisters "k8s.io/client-go/listers/apps/v1beta1"
appslisters "k8s.io/client-go/listers/apps/v1"
"k8s.io/client-go/tools/cache"
"k8s.io/client-go/tools/record"
"k8s.io/client-go/util/retry"
Expand Down Expand Up @@ -56,7 +56,7 @@ func NewRealStatefuSetControl(kubeCli kubernetes.Interface, setLister appslister

// CreateStatefulSet create a StatefulSet in a TidbCluster.
func (sc *realStatefulSetControl) CreateStatefulSet(tc *v1alpha1.TidbCluster, set *apps.StatefulSet) error {
_, err := sc.kubeCli.AppsV1beta1().StatefulSets(tc.Namespace).Create(set)
_, err := sc.kubeCli.AppsV1().StatefulSets(tc.Namespace).Create(set)
// sink already exists errors
if apierrors.IsAlreadyExists(err) {
return err
Expand All @@ -76,7 +76,7 @@ func (sc *realStatefulSetControl) UpdateStatefulSet(tc *v1alpha1.TidbCluster, se
err := retry.RetryOnConflict(retry.DefaultBackoff, func() error {
// TODO: verify if StatefulSet identity(name, namespace, labels) matches TidbCluster
var updateErr error
updatedSS, updateErr = sc.kubeCli.AppsV1beta1().StatefulSets(ns).Update(set)
updatedSS, updateErr = sc.kubeCli.AppsV1().StatefulSets(ns).Update(set)
if updateErr == nil {
glog.Infof("TidbCluster: [%s/%s]'s StatefulSet: [%s/%s] updated successfully", ns, tcName, ns, setName)
return nil
Expand All @@ -99,7 +99,7 @@ func (sc *realStatefulSetControl) UpdateStatefulSet(tc *v1alpha1.TidbCluster, se

// DeleteStatefulSet delete a StatefulSet in a TidbCluster.
func (sc *realStatefulSetControl) DeleteStatefulSet(tc *v1alpha1.TidbCluster, set *apps.StatefulSet) error {
err := sc.kubeCli.AppsV1beta1().StatefulSets(tc.Namespace).Delete(set.Name, nil)
err := sc.kubeCli.AppsV1().StatefulSets(tc.Namespace).Delete(set.Name, nil)
sc.recordStatefulSetEvent("delete", tc, set, err)
return err
}
Expand Down
2 changes: 1 addition & 1 deletion pkg/controller/stateful_set_control_test.go
Expand Up @@ -22,7 +22,7 @@ import (
apierrors "k8s.io/apimachinery/pkg/api/errors"
"k8s.io/apimachinery/pkg/runtime"
"k8s.io/client-go/kubernetes/fake"
appslisters "k8s.io/client-go/listers/apps/v1beta1"
appslisters "k8s.io/client-go/listers/apps/v1"
core "k8s.io/client-go/testing"
"k8s.io/client-go/tools/cache"
"k8s.io/client-go/tools/record"
Expand Down
2 changes: 1 addition & 1 deletion pkg/controller/tidbcluster/tidb_cluster_control_test.go
Expand Up @@ -25,7 +25,7 @@ import (
"github.com/pingcap/tidb-operator/pkg/controller"
mm "github.com/pingcap/tidb-operator/pkg/manager/member"
"github.com/pingcap/tidb-operator/pkg/manager/meta"
apps "k8s.io/api/apps/v1beta1"
apps "k8s.io/api/apps/v1"
corev1 "k8s.io/api/core/v1"
apiequality "k8s.io/apimachinery/pkg/api/equality"
metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
Expand Down
6 changes: 3 additions & 3 deletions pkg/controller/tidbcluster/tidb_cluster_controller.go
Expand Up @@ -27,7 +27,7 @@ import (
mm "github.com/pingcap/tidb-operator/pkg/manager/member"
"github.com/pingcap/tidb-operator/pkg/manager/meta"
"github.com/pingcap/tidb-operator/pkg/pdapi"
apps "k8s.io/api/apps/v1beta1"
apps "k8s.io/api/apps/v1"
corev1 "k8s.io/api/core/v1"
"k8s.io/apimachinery/pkg/api/errors"
metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
Expand All @@ -36,7 +36,7 @@ import (
kubeinformers "k8s.io/client-go/informers"
"k8s.io/client-go/kubernetes"
eventv1 "k8s.io/client-go/kubernetes/typed/core/v1"
appslisters "k8s.io/client-go/listers/apps/v1beta1"
appslisters "k8s.io/client-go/listers/apps/v1"
"k8s.io/client-go/tools/cache"
"k8s.io/client-go/tools/record"
"k8s.io/client-go/util/workqueue"
Expand Down Expand Up @@ -84,7 +84,7 @@ func NewController(
recorder := eventBroadcaster.NewRecorder(v1alpha1.Scheme, corev1.EventSource{Component: "tidbcluster"})

tcInformer := informerFactory.Pingcap().V1alpha1().TidbClusters()
setInformer := kubeInformerFactory.Apps().V1beta1().StatefulSets()
setInformer := kubeInformerFactory.Apps().V1().StatefulSets()
svcInformer := kubeInformerFactory.Core().V1().Services()
epsInformer := kubeInformerFactory.Core().V1().Endpoints()
pvcInformer := kubeInformerFactory.Core().V1().PersistentVolumeClaims()
Expand Down
4 changes: 2 additions & 2 deletions pkg/controller/tidbcluster/tidb_cluster_controller_test.go
Expand Up @@ -23,7 +23,7 @@ import (
"github.com/pingcap/tidb-operator/pkg/apis/pingcap.com/v1alpha1"
"github.com/pingcap/tidb-operator/pkg/client/clientset/versioned/fake"
informers "github.com/pingcap/tidb-operator/pkg/client/informers/externalversions"
apps "k8s.io/api/apps/v1beta1"
apps "k8s.io/api/apps/v1"
corev1 "k8s.io/api/core/v1"
metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
"k8s.io/apimachinery/pkg/types"
Expand Down Expand Up @@ -320,7 +320,7 @@ func newStatefuSet(tc *v1alpha1.TidbCluster) *apps.StatefulSet {
return &apps.StatefulSet{
TypeMeta: metav1.TypeMeta{
Kind: "StatefulSet",
APIVersion: "apps/v1beta1",
APIVersion: "apps/v1",
},
ObjectMeta: metav1.ObjectMeta{
Name: "test-statefuset",
Expand Down
8 changes: 4 additions & 4 deletions pkg/manager/member/pd_member_manager.go
Expand Up @@ -24,22 +24,22 @@ import (
"github.com/pingcap/tidb-operator/pkg/manager"
"github.com/pingcap/tidb-operator/pkg/pdapi"
"github.com/pingcap/tidb-operator/pkg/util"
apps "k8s.io/api/apps/v1beta1"
apps "k8s.io/api/apps/v1"
corev1 "k8s.io/api/core/v1"
"k8s.io/apimachinery/pkg/api/errors"
"k8s.io/apimachinery/pkg/api/resource"
metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
"k8s.io/apimachinery/pkg/util/intstr"
"k8s.io/apimachinery/pkg/util/uuid"
"k8s.io/client-go/listers/apps/v1beta1"
"k8s.io/client-go/listers/apps/v1"
corelisters "k8s.io/client-go/listers/core/v1"
)

type pdMemberManager struct {
pdControl pdapi.PDControlInterface
setControl controller.StatefulSetControlInterface
svcControl controller.ServiceControlInterface
setLister v1beta1.StatefulSetLister
setLister v1.StatefulSetLister
svcLister corelisters.ServiceLister
podLister corelisters.PodLister
epsLister corelisters.EndpointsLister
Expand All @@ -55,7 +55,7 @@ type pdMemberManager struct {
func NewPDMemberManager(pdControl pdapi.PDControlInterface,
setControl controller.StatefulSetControlInterface,
svcControl controller.ServiceControlInterface,
setLister v1beta1.StatefulSetLister,
setLister v1.StatefulSetLister,
svcLister corelisters.ServiceLister,
podLister corelisters.PodLister,
epsLister corelisters.EndpointsLister,
Expand Down
16 changes: 8 additions & 8 deletions pkg/manager/member/pd_member_manager_test.go
Expand Up @@ -26,7 +26,7 @@ import (
"github.com/pingcap/tidb-operator/pkg/controller"
"github.com/pingcap/tidb-operator/pkg/label"
"github.com/pingcap/tidb-operator/pkg/pdapi"
apps "k8s.io/api/apps/v1beta1"
apps "k8s.io/api/apps/v1"
corev1 "k8s.io/api/core/v1"
"k8s.io/apimachinery/pkg/api/errors"
metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
Expand Down Expand Up @@ -238,7 +238,7 @@ func TestPDMemberManagerSyncUpdate(t *testing.T) {
set.Status.CurrentRevision = "pd-1"
set.Status.UpdateRevision = "pd-1"
observedGeneration := int64(1)
set.Status.ObservedGeneration = &observedGeneration
set.Status.ObservedGeneration = observedGeneration
})
} else {
fakeSetControl.SetStatusChange(test.statusChange)
Expand Down Expand Up @@ -328,7 +328,7 @@ func TestPDMemberManagerSyncUpdate(t *testing.T) {
expectTidbClusterFn: func(g *GomegaWithT, tc *v1alpha1.TidbCluster) {
g.Expect(tc.Status.ClusterID).To(Equal("1"))
g.Expect(tc.Status.PD.Phase).To(Equal(v1alpha1.NormalPhase))
g.Expect(*tc.Status.PD.StatefulSet.ObservedGeneration).To(Equal(int64(1)))
g.Expect(tc.Status.PD.StatefulSet.ObservedGeneration).To(Equal(int64(1)))
g.Expect(len(tc.Status.PD.Members)).To(Equal(3))
g.Expect(tc.Status.PD.Members["pd1"].Health).To(Equal(true))
g.Expect(tc.Status.PD.Members["pd2"].Health).To(Equal(true))
Expand Down Expand Up @@ -500,7 +500,7 @@ func TestPDMemberManagerPdStatefulSetIsUpgrading(t *testing.T) {
setUpdate: func(set *apps.StatefulSet) {
set.Status.CurrentRevision = "v1"
set.Status.UpdateRevision = "v2"
set.Status.ObservedGeneration = func() *int64 { var i int64; i = 1000; return &i }()
set.Status.ObservedGeneration = func() int64 { var i int64; i = 1000; return i }()
Yisaer marked this conversation as resolved.
Show resolved Hide resolved
},
hasPod: false,
updatePod: nil,
Expand Down Expand Up @@ -617,7 +617,7 @@ func TestPDMemberManagerUpgrade(t *testing.T) {
set.Status.CurrentRevision = "pd-1"
set.Status.UpdateRevision = "pd-1"
observedGeneration := int64(1)
set.Status.ObservedGeneration = &observedGeneration
set.Status.ObservedGeneration = observedGeneration
},
expectStatefulSetFn: func(g *GomegaWithT, set *apps.StatefulSet, err error) {
g.Expect(err).NotTo(HaveOccurred())
Expand Down Expand Up @@ -717,7 +717,7 @@ func TestPDMemberManagerSyncPDSts(t *testing.T) {
set.Status.CurrentRevision = "pd-1"
set.Status.UpdateRevision = "pd-1"
observedGeneration := int64(1)
set.Status.ObservedGeneration = &observedGeneration
set.Status.ObservedGeneration = observedGeneration
},
expectStatefulSetFn: func(g *GomegaWithT, set *apps.StatefulSet, err error) {
g.Expect(err).NotTo(HaveOccurred())
Expand Down Expand Up @@ -746,7 +746,7 @@ func TestPDMemberManagerSyncPDSts(t *testing.T) {
set.Status.CurrentRevision = "pd-1"
set.Status.UpdateRevision = "pd-1"
observedGeneration := int64(1)
set.Status.ObservedGeneration = &observedGeneration
set.Status.ObservedGeneration = observedGeneration
},
expectStatefulSetFn: func(g *GomegaWithT, set *apps.StatefulSet, err error) {
g.Expect(err).NotTo(HaveOccurred())
Expand All @@ -769,7 +769,7 @@ func TestPDMemberManagerSyncPDSts(t *testing.T) {
func newFakePDMemberManager() (*pdMemberManager, *controller.FakeStatefulSetControl, *controller.FakeServiceControl, *pdapi.FakePDControl, cache.Indexer, cache.Indexer, *controller.FakePodControl) {
cli := fake.NewSimpleClientset()
kubeCli := kubefake.NewSimpleClientset()
setInformer := kubeinformers.NewSharedInformerFactory(kubeCli, 0).Apps().V1beta1().StatefulSets()
setInformer := kubeinformers.NewSharedInformerFactory(kubeCli, 0).Apps().V1().StatefulSets()
svcInformer := kubeinformers.NewSharedInformerFactory(kubeCli, 0).Core().V1().Services()
podInformer := kubeinformers.NewSharedInformerFactory(kubeCli, 0).Core().V1().Pods()
epsInformer := kubeinformers.NewSharedInformerFactory(kubeCli, 0).Core().V1().Endpoints()
Expand Down
2 changes: 1 addition & 1 deletion pkg/manager/member/pd_scaler.go
Expand Up @@ -22,7 +22,7 @@ import (
"github.com/pingcap/tidb-operator/pkg/controller"
"github.com/pingcap/tidb-operator/pkg/label"
"github.com/pingcap/tidb-operator/pkg/pdapi"
apps "k8s.io/api/apps/v1beta1"
apps "k8s.io/api/apps/v1"
corelisters "k8s.io/client-go/listers/core/v1"
)

Expand Down
2 changes: 1 addition & 1 deletion pkg/manager/member/pd_scaler_test.go
Expand Up @@ -23,7 +23,7 @@ import (
"github.com/pingcap/tidb-operator/pkg/controller"
"github.com/pingcap/tidb-operator/pkg/label"
"github.com/pingcap/tidb-operator/pkg/pdapi"
apps "k8s.io/api/apps/v1beta1"
apps "k8s.io/api/apps/v1"
corev1 "k8s.io/api/core/v1"
"k8s.io/apimachinery/pkg/api/errors"
metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
Expand Down
2 changes: 1 addition & 1 deletion pkg/manager/member/pd_upgrader.go
Expand Up @@ -20,7 +20,7 @@ import (
"github.com/pingcap/tidb-operator/pkg/apis/pingcap.com/v1alpha1"
"github.com/pingcap/tidb-operator/pkg/controller"
"github.com/pingcap/tidb-operator/pkg/pdapi"
apps "k8s.io/api/apps/v1beta1"
apps "k8s.io/api/apps/v1"
corelisters "k8s.io/client-go/listers/core/v1"
)

Expand Down
2 changes: 1 addition & 1 deletion pkg/manager/member/pd_upgrader_test.go
Expand Up @@ -22,7 +22,7 @@ import (
"github.com/pingcap/tidb-operator/pkg/controller"
"github.com/pingcap/tidb-operator/pkg/label"
"github.com/pingcap/tidb-operator/pkg/pdapi"
apps "k8s.io/api/apps/v1beta1"
apps "k8s.io/api/apps/v1"
corev1 "k8s.io/api/core/v1"
metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
"k8s.io/apimachinery/pkg/types"
Expand Down
2 changes: 1 addition & 1 deletion pkg/manager/member/scaler.go
Expand Up @@ -21,7 +21,7 @@ import (
"github.com/pingcap/tidb-operator/pkg/controller"
"github.com/pingcap/tidb-operator/pkg/label"
"github.com/pingcap/tidb-operator/pkg/pdapi"
apps "k8s.io/api/apps/v1beta1"
apps "k8s.io/api/apps/v1"
"k8s.io/apimachinery/pkg/api/errors"
corelisters "k8s.io/client-go/listers/core/v1"
)
Expand Down