Fix renaming order shown on terminal

When finishing a rolling update, a message is shown : 'Renaming oldRc to newRc' In my case that'd be: 'Renaming nginx to nginx-df8d9fafc5007d83968cd37c8353d52e' That naming order is incorrect, as the new name is temporary, and the final name should be the old one

Fix renaming order shown on terminal
f2fcf659 · Alvaro [Andor] · GitHub · 4f69d7ed · f2fcf659
Commit f2fcf659 authored Nov 18, 2016 by Alvaro [Andor] Committed by GitHub Nov 18, 2016
Hide whitespace changes
Inline Side-by-side

Showing with 1 addition and 1 deletion

rolling_updater.go pkg/kubectl/rolling_updater.go +1 -1

No files found.
--- a/pkg/kubectl/rolling_updater.go
+++ b/pkg/kubectl/rolling_updater.go
@@ -520,7 +520,7 @@ func (r *RollingUpdater) cleanupWithClients(oldRc, newRc *api.ReplicationControl
 		if err := r.rcClient.ReplicationControllers(r.ns).Delete(oldRc.Name, nil); err != nil {
 			return err
 		}
-		fmt.Fprintf(config.Out, "Renaming %s to %s\n", oldRc.Name, newRc.Name)
+		fmt.Fprintf(config.Out, "Renaming %s to %s\n", newRc.Name, oldRc.Name)
 		return Rename(r.rcClient, newRc, oldRc.Name)
 	case PreserveRollingUpdateCleanupPolicy:
 		return nil