Merge pull request #78889 from qingsenLi/k8s-190611

use klog.Info instead of klog.Infof when had no format
This commit is contained in:
Kubernetes Prow Robot 2019-06-19 13:02:20 -07:00 committed by GitHub
commit ee4776d7ec
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 8 additions and 8 deletions

View File

@ -95,10 +95,10 @@ func (m *Migrator) MigrateIfNeeded(target *EtcdVersionPair) error {
} }
return nil return nil
case current.storageVersion == storageEtcd2 && target.storageVersion == storageEtcd3: case current.storageVersion == storageEtcd2 && target.storageVersion == storageEtcd3:
klog.Infof("upgrading from etcd2 storage to etcd3 storage") klog.Info("upgrading from etcd2 storage to etcd3 storage")
current, err = m.etcd2ToEtcd3Upgrade(current, target) current, err = m.etcd2ToEtcd3Upgrade(current, target)
case current.version.Major == 3 && target.version.Major == 2: case current.version.Major == 3 && target.version.Major == 2:
klog.Infof("downgrading from etcd 3.x to 2.x") klog.Info("downgrading from etcd 3.x to 2.x")
current, err = m.rollbackToEtcd2(current, target) current, err = m.rollbackToEtcd2(current, target)
case current.version.Major == target.version.Major && current.version.Minor < target.version.Minor: case current.version.Major == target.version.Major && current.version.Minor < target.version.Minor:
stepVersion := m.cfg.supportedVersions.NextVersionPair(current) stepVersion := m.cfg.supportedVersions.NextVersionPair(current)
@ -116,7 +116,7 @@ func (m *Migrator) MigrateIfNeeded(target *EtcdVersionPair) error {
func (m *Migrator) backupEtcd2(current *EtcdVersion) error { func (m *Migrator) backupEtcd2(current *EtcdVersion) error {
backupDir := fmt.Sprintf("%s/%s", m.dataDirectory, "migration-backup") backupDir := fmt.Sprintf("%s/%s", m.dataDirectory, "migration-backup")
klog.Infof("Backup etcd before starting migration") klog.Info("Backup etcd before starting migration")
err := os.Mkdir(backupDir, 0666) err := os.Mkdir(backupDir, 0666)
if err != nil { if err != nil {
return fmt.Errorf("failed to create backup directory before starting migration: %v", err) return fmt.Errorf("failed to create backup directory before starting migration: %v", err)
@ -163,7 +163,7 @@ func (m *Migrator) rollbackEtcd3MinorVersion(current *EtcdVersionPair, target *E
return nil, err return nil, err
} }
klog.Infof("Backing up data before rolling back") klog.Info("Backing up data before rolling back")
backupDir := fmt.Sprintf("%s.bak", m.dataDirectory) backupDir := fmt.Sprintf("%s.bak", m.dataDirectory)
err = os.RemoveAll(backupDir) err = os.RemoveAll(backupDir)
if err != nil { if err != nil {
@ -195,7 +195,7 @@ func (m *Migrator) rollbackToEtcd2(current *EtcdVersionPair, target *EtcdVersion
if !(current.version.Major == 3 && current.version.Minor == 0 && target.version.Major == 2 && target.version.Minor == 2) { if !(current.version.Major == 3 && current.version.Minor == 0 && target.version.Major == 2 && target.version.Minor == 2) {
return nil, fmt.Errorf("etcd3 -> etcd2 downgrade is supported only between 3.0.x and 2.2.x, got current %s target %s", current, target) return nil, fmt.Errorf("etcd3 -> etcd2 downgrade is supported only between 3.0.x and 2.2.x, got current %s target %s", current, target)
} }
klog.Infof("Backup and remove all existing v2 data") klog.Info("Backup and remove all existing v2 data")
err := m.dataDirectory.Backup() err := m.dataDirectory.Backup()
if err != nil { if err != nil {
return nil, err return nil, err
@ -214,12 +214,12 @@ func (m *Migrator) etcd2ToEtcd3Upgrade(current *EtcdVersionPair, target *EtcdVer
} }
runner := m.newServer() runner := m.newServer()
klog.Infof("Performing etcd2 -> etcd3 migration") klog.Info("Performing etcd2 -> etcd3 migration")
err := m.client.Migrate(target.version) err := m.client.Migrate(target.version)
if err != nil { if err != nil {
return nil, err return nil, err
} }
klog.Infof("Attaching leases to TTL entries") klog.Info("Attaching leases to TTL entries")
// Now attach lease to all keys. // Now attach lease to all keys.
// To do it, we temporarily start etcd on a random port (so that // To do it, we temporarily start etcd on a random port (so that

View File

@ -172,7 +172,7 @@ func RollbackV3ToV2(migrateDatadir string, ttl time.Duration) error {
if err := snapshotter.SaveSnap(raftSnap); err != nil { if err := snapshotter.SaveSnap(raftSnap); err != nil {
return err return err
} }
klog.Infof("Finished successfully") klog.Info("Finished successfully")
return nil return nil
} }