From 310d4f48e687a2a8944707603796b8416ad6f2a4 Mon Sep 17 00:00:00 2001 From: Avesh Agarwal Date: Mon, 28 Sep 2015 22:08:38 -0400 Subject: [PATCH] Fixes erroneous declaration of oomAdjuster. Fixes duplication of execer, dbus, iptInterface. Fixes default assignment of SyncPeriod. --- cmd/kube-proxy/app/server.go | 19 ++++++++----------- 1 file changed, 8 insertions(+), 11 deletions(-) diff --git a/cmd/kube-proxy/app/server.go b/cmd/kube-proxy/app/server.go index b71ccf8197..25ff6cdb85 100644 --- a/cmd/kube-proxy/app/server.go +++ b/cmd/kube-proxy/app/server.go @@ -90,7 +90,7 @@ func (s *ProxyServerConfig) AddFlags(fs *pflag.FlagSet) { fs.Var(&s.PortRange, "proxy-port-range", "Range of host ports (beginPort-endPort, inclusive) that may be consumed in order to proxy service traffic. If unspecified (0-0) then ports will be randomly chosen.") fs.StringVar(&s.HostnameOverride, "hostname-override", s.HostnameOverride, "If non-empty, will use this string as identification instead of the actual hostname.") fs.StringVar(&s.ProxyMode, "proxy-mode", "", "Which proxy mode to use: 'userspace' (older, stable) or 'iptables' (experimental). If blank, look at the Node object on the Kubernetes API and respect the '"+experimentalProxyModeAnnotation+"' annotation if provided. Otherwise use the best-available proxy (currently userspace, but may change in future versions). If the iptables proxy is selected, regardless of how, but the system's kernel or iptables versions are insufficient, this always falls back to the userspace proxy.") - fs.DurationVar(&s.SyncPeriod, "iptables-sync-period", 30*time.Second, "How often iptables rules are refreshed (e.g. '5s', '1m', '2h22m'). Must be greater than 0.") + fs.DurationVar(&s.SyncPeriod, "iptables-sync-period", s.SyncPeriod, "How often iptables rules are refreshed (e.g. '5s', '1m', '2h22m'). Must be greater than 0.") fs.BoolVar(&s.MasqueradeAll, "masquerade-all", false, "If using the pure iptables proxy, SNAT everything") fs.BoolVar(&s.CleanupAndExit, "cleanup-iptables", false, "If true cleanup iptables rules and exit.") } @@ -151,20 +151,22 @@ func NewProxyServerDefault(config *ProxyServerConfig) (*ProxyServer, error) { protocol = utiliptables.ProtocolIpv6 } + // Create a iptables utils. + execer := exec.New() + dbus := utildbus.New() + iptInterface := utiliptables.New(execer, dbus, protocol) + // We ommit creation of pretty much everything if we run in cleanup mode if config.CleanupAndExit { - execer := exec.New() - dbus := utildbus.New() - IptInterface := utiliptables.New(execer, dbus, protocol) return &ProxyServer{ - IptInterface: IptInterface, + IptInterface: iptInterface, }, nil } // TODO(vmarmol): Use container config for this. var oomAdjuster *oom.OOMAdjuster if config.OOMScoreAdj != 0 { - oomAdjuster := oom.NewOOMAdjuster() + oomAdjuster = oom.NewOOMAdjuster() if err := oomAdjuster.ApplyOOMScoreAdj(0, config.OOMScoreAdj); err != nil { glog.V(2).Info(err) } @@ -203,11 +205,6 @@ func NewProxyServerDefault(config *ProxyServerConfig) (*ProxyServer, error) { recorder := eventBroadcaster.NewRecorder(api.EventSource{Component: "kube-proxy", Host: hostname}) eventBroadcaster.StartRecordingToSink(client.Events("")) - // Create a iptables utils. - execer := exec.New() - dbus := utildbus.New() - iptInterface := utiliptables.New(execer, dbus, protocol) - var proxier proxy.ProxyProvider var endpointsHandler proxyconfig.EndpointsConfigHandler