Name change: s/timstclair/tallclair/

kubernetes · Jul 10, 2017 · a2f2e1d · a2f2e1d
1 parent 0744964
commit a2f2e1d
Show file tree

Hide file tree

Showing 38 changed files with 55 additions and 55 deletions.
diff --git a/OWNERS_ALIASES b/OWNERS_ALIASES
@@ -52,7 +52,7 @@ aliases:
     - pmorie
     - resouer
     - sjpotter
-    - timstclair
+    - tallclair
     - tmrts
     - vishh
     - yifan-gu

diff --git a/api/OWNERS b/api/OWNERS
@@ -33,7 +33,7 @@ reviewers:
 - pwittrock
 - roberthbailey
 - ncdc
-- timstclair
+- tallclair
 - yifan-gu
 - eparis
 - mwielgus

diff --git a/hack/update_owners.py b/hack/update_owners.py
@@ -111,7 +111,7 @@ def get_maintainers():
            "nikhiljindal", "piosz", "pmorie", "pwittrock", "Q-Lee", "quinton-hoole",
            "Random-Liu", "rmmh", "roberthbailey", "saad-ali", "smarterclayton",
            "soltysh", "spxtr", "sttts", "thelinuxfoundation", "thockin",
-           "timothysc", "timstclair", "vishh", "wojtek-t", "xiang90", "yifan-gu",
+           "timothysc", "tallclair", "vishh", "wojtek-t", "xiang90", "yifan-gu",
            "yujuhong", "zmerlynn"}
     return sorted(ret - SKIP_MAINTAINERS)
 

diff --git a/pkg/api/OWNERS b/pkg/api/OWNERS
@@ -33,7 +33,7 @@ reviewers:
 - pwittrock
 - roberthbailey
 - ncdc
-- timstclair
+- tallclair
 - yifan-gu
 - eparis
 - mwielgus

diff --git a/pkg/api/testapi/OWNERS b/pkg/api/testapi/OWNERS
@@ -10,7 +10,7 @@ reviewers:
 - nikhiljindal
 - bprashanth
 - erictune
-- timstclair
+- tallclair
 - eparis
 - soltysh
 - madhusudancs

diff --git a/pkg/api/testing/OWNERS b/pkg/api/testing/OWNERS
@@ -20,7 +20,7 @@ reviewers:
 - justinsb
 - pwittrock
 - roberthbailey
-- timstclair
+- tallclair
 - yifan-gu
 - eparis
 - soltysh

diff --git a/pkg/api/v1/OWNERS b/pkg/api/v1/OWNERS
@@ -27,7 +27,7 @@ reviewers:
 - justinsb
 - roberthbailey
 - ncdc
-- timstclair
+- tallclair
 - eparis
 - timothysc
 - piosz

diff --git a/pkg/api/validation/OWNERS b/pkg/api/validation/OWNERS
@@ -26,7 +26,7 @@ reviewers:
 - justinsb
 - pwittrock
 - roberthbailey
-- timstclair
+- tallclair
 - eparis
 - soltysh
 - piosz

diff --git a/pkg/apis/OWNERS b/pkg/apis/OWNERS
@@ -29,7 +29,7 @@ reviewers:
 - justinsb
 - pwittrock
 - ncdc
-- timstclair
+- tallclair
 - yifan-gu
 - eparis
 - mwielgus

diff --git a/pkg/apis/apps/OWNERS b/pkg/apis/apps/OWNERS
@@ -9,7 +9,7 @@ reviewers:
 - sttts
 - saad-ali
 - ncdc
-- timstclair
+- tallclair
 - timothysc
 - dims
 - errordeveloper

diff --git a/pkg/apis/extensions/OWNERS b/pkg/apis/extensions/OWNERS
@@ -19,7 +19,7 @@ reviewers:
 - janetkuo
 - justinsb
 - ncdc
-- timstclair
+- tallclair
 - mwielgus
 - timothysc
 - soltysh

diff --git a/pkg/client/OWNERS b/pkg/client/OWNERS
@@ -33,7 +33,7 @@ reviewers:
 - justinsb
 - roberthbailey
 - ncdc
-- timstclair
+- tallclair
 - yifan-gu
 - eparis
 - mwielgus

diff --git a/pkg/client/unversioned/OWNERS b/pkg/client/unversioned/OWNERS
@@ -22,7 +22,7 @@ reviewers:
 - zmerlynn
 - janetkuo
 - ncdc
-- timstclair
+- tallclair
 - mwielgus
 - timothysc
 - feiskyer

diff --git a/pkg/features/kube_features.go b/pkg/features/kube_features.go
@@ -28,7 +28,7 @@ const (
 	// // alpha: v1.X
 	// MyFeature utilfeature.Feature = "MyFeature"
 
-	// owner: @timstclair
+	// owner: @tallclair
 	// beta: v1.4
 	AppArmor utilfeature.Feature = "AppArmor"
 
@@ -44,7 +44,7 @@ const (
 	// alpha: v1.4
 	DynamicKubeletConfig utilfeature.Feature = "DynamicKubeletConfig"
 
-	// owner: timstclair
+	// owner: tallclair
 	// alpha: v1.5
 	//
 	// StreamingProxyRedirects controls whether the apiserver should intercept (and follow)

diff --git a/pkg/kubelet/OWNERS b/pkg/kubelet/OWNERS
@@ -2,7 +2,7 @@ approvers:
 - Random-Liu
 - dchen1107
 - derekwaynecarr
-- timstclair
+- tallclair
 - vishh
 - yujuhong
 reviewers:

diff --git a/pkg/kubelet/dockershim/docker_streaming.go b/pkg/kubelet/dockershim/docker_streaming.go
@@ -121,7 +121,7 @@ func (ds *dockerService) PortForward(req *runtimeapi.PortForwardRequest) (*runti
 	if err != nil {
 		return nil, err
 	}
-	// TODO(timstclair): Verify that ports are exposed.
+	// TODO(tallclair): Verify that ports are exposed.
 	return ds.streamingServer.GetPortForward(req)
 }
 

diff --git a/pkg/kubelet/kubelet_node_status.go b/pkg/kubelet/kubelet_node_status.go
@@ -696,7 +696,7 @@ func (kl *Kubelet) setNodeReadyCondition(node *v1.Node) {
 	}
 
 	// Append AppArmor status if it's enabled.
-	// TODO(timstclair): This is a temporary message until node feature reporting is added.
+	// TODO(tallclair): This is a temporary message until node feature reporting is added.
 	if newNodeReadyCondition.Status == v1.ConditionTrue &&
 		kl.appArmorValidator != nil && kl.appArmorValidator.ValidateHost() == nil {
 		newNodeReadyCondition.Message = fmt.Sprintf("%s. AppArmor enabled", newNodeReadyCondition.Message)

diff --git a/pkg/kubelet/kubelet_pods.go b/pkg/kubelet/kubelet_pods.go
@@ -1418,7 +1418,7 @@ func (kl *Kubelet) RunInContainer(podFullName string, podUID types.UID, containe
 	if container == nil {
 		return nil, fmt.Errorf("container not found (%q)", containerName)
 	}
-	// TODO(timstclair): Pass a proper timeout value.
+	// TODO(tallclair): Pass a proper timeout value.
 	return kl.runner.RunInContainer(container.ID, cmd, 0)
 }
 

diff --git a/pkg/kubelet/kuberuntime/kuberuntime_container.go b/pkg/kubelet/kuberuntime/kuberuntime_container.go
@@ -764,7 +764,7 @@ func (m *kubeGenericRuntimeManager) GetAttach(id kubecontainer.ContainerID, stdi
 // RunInContainer synchronously executes the command in the container, and returns the output.
 func (m *kubeGenericRuntimeManager) RunInContainer(id kubecontainer.ContainerID, cmd []string, timeout time.Duration) ([]byte, error) {
 	stdout, stderr, err := m.runtimeService.ExecSync(id.ID, cmd, 0)
-	// NOTE(timstclair): This does not correctly interleave stdout & stderr, but should be sufficient
+	// NOTE(tallclair): This does not correctly interleave stdout & stderr, but should be sufficient
 	// for logging purposes. A combined output option will need to be added to the ExecSyncRequest
 	// if more precise output ordering is ever required.
 	return append(stdout, stderr...), err

diff --git a/pkg/kubelet/lifecycle/handlers.go b/pkg/kubelet/lifecycle/handlers.go
@@ -55,7 +55,7 @@ func (hr *HandlerRunner) Run(containerID kubecontainer.ContainerID, pod *v1.Pod,
 	switch {
 	case handler.Exec != nil:
 		var msg string
-		// TODO(timstclair): Pass a proper timeout value.
+		// TODO(tallclair): Pass a proper timeout value.
 		output, err := hr.commandRunner.RunInContainer(containerID, handler.Exec.Command, 0)
 		if err != nil {
 			msg := fmt.Sprintf("Exec lifecycle hook (%v) for Container %q in Pod %q failed - error: %v, message: %q", handler.Exec.Command, container.Name, format.Pod(pod), err, string(output))

diff --git a/pkg/kubelet/server/stats/summary.go b/pkg/kubelet/server/stats/summary.go
@@ -73,7 +73,7 @@ func (sp *summaryProviderImpl) Get() (*stats.Summary, error) {
 		}
 	}
 
-	// TODO(timstclair): Consider returning a best-effort response if any of the following errors
+	// TODO(tallclair): Consider returning a best-effort response if any of the following errors
 	// occur.
 	node, err := sp.provider.GetNode()
 	if err != nil {

diff --git a/pkg/kubelet/server/streaming/server.go b/pkg/kubelet/server/streaming/server.go
@@ -101,7 +101,7 @@ var DefaultConfig = Config{
 	SupportedPortForwardProtocols:   portforward.SupportedProtocols,
 }
 
-// TODO(timstclair): Add auth(n/z) interface & handling.
+// TODO(tallclair): Add auth(n/z) interface & handling.
 func NewServer(config Config, runtime Runtime) (Server, error) {
 	s := &server{
 		config:  config,
@@ -193,7 +193,7 @@ func (s *server) GetPortForward(req *runtimeapi.PortForwardRequest) (*runtimeapi
 
 func (s *server) Start(stayUp bool) error {
 	if !stayUp {
-		// TODO(timstclair): Implement this.
+		// TODO(tallclair): Implement this.
 		return errors.New("stayUp=false is not yet implemented")
 	}
 
@@ -210,7 +210,7 @@ func (s *server) Start(stayUp bool) error {
 }
 
 func (s *server) Stop() error {
-	// TODO(timstclair): Implement this.
+	// TODO(tallclair): Implement this.
 	return errors.New("not yet implemented")
 }
 

diff --git a/pkg/master/OWNERS b/pkg/master/OWNERS
@@ -31,7 +31,7 @@ reviewers:
 - justinsb
 - roberthbailey
 - ncdc
-- timstclair
+- tallclair
 - mwielgus
 - timothysc
 - soltysh

diff --git a/pkg/registry/OWNERS b/pkg/registry/OWNERS
@@ -29,7 +29,7 @@ reviewers:
 - pwittrock
 - roberthbailey
 - ncdc
-- timstclair
+- tallclair
 - eparis
 - mwielgus
 - timothysc

diff --git a/pkg/security/podsecuritypolicy/provider.go b/pkg/security/podsecuritypolicy/provider.go
@@ -225,7 +225,7 @@ func (s *simpleProvider) ValidatePodSecurityContext(pod *api.Pod, fldPath *field
 
 	allErrs = append(allErrs, s.strategies.SysctlsStrategy.Validate(pod)...)
 
-	// TODO(timstclair): ValidatePodSecurityContext should be renamed to ValidatePod since its scope
+	// TODO(tallclair): ValidatePodSecurityContext should be renamed to ValidatePod since its scope
 	// is not limited to the PodSecurityContext.
 	if len(pod.Spec.Volumes) > 0 && !psputil.PSPAllowsAllVolumes(s.psp) {
 		allowedVolumes := psputil.FSTypeToStringSet(s.psp.Spec.Volumes)

diff --git a/plugin/pkg/admission/noderestriction/OWNERS b/plugin/pkg/admission/noderestriction/OWNERS
@@ -1,8 +1,8 @@
 approvers:
 - deads2k
 - liggitt
-- timstclair
+- tallclair
 reviewers:
 - deads2k
 - liggitt
-- timstclair
+- tallclair
diff --git a/plugin/pkg/auth/authorizer/node/OWNERS b/plugin/pkg/auth/authorizer/node/OWNERS
@@ -1,9 +1,9 @@
 approvers:
-- timstclair
+- tallclair
 - liggitt
 - deads2k
 reviewers:
-- timstclair
+- tallclair
 - liggitt
 - deads2k
 - ericchiang
diff --git a/staging/src/k8s.io/apimachinery/OWNERS b/staging/src/k8s.io/apimachinery/OWNERS
@@ -18,4 +18,4 @@ reviewers:
 - gmarek
 - sttts
 - ncdc
-- timstclair
+- tallclair
diff --git a/staging/src/k8s.io/apimachinery/pkg/api/errors/OWNERS b/staging/src/k8s.io/apimachinery/pkg/api/errors/OWNERS
@@ -14,7 +14,7 @@ reviewers:
 - erictune
 - saad-ali
 - janetkuo
-- timstclair
+- tallclair
 - eparis
 - timothysc
 - dims

diff --git a/staging/src/k8s.io/apimachinery/pkg/api/resource/OWNERS b/staging/src/k8s.io/apimachinery/pkg/api/resource/OWNERS
@@ -7,7 +7,7 @@ reviewers:
 - mikedanese
 - saad-ali
 - janetkuo
-- timstclair
+- tallclair
 - eparis
 - timothysc
 - jbeda

diff --git a/staging/src/k8s.io/apiserver/OWNERS b/staging/src/k8s.io/apiserver/OWNERS
@@ -13,5 +13,5 @@ reviewers:
 - liggitt
 - sttts
 - ncdc
-- timstclair
+- tallclair
 - timothysc
diff --git a/staging/src/k8s.io/apiserver/pkg/features/kube_features.go b/staging/src/k8s.io/apiserver/pkg/features/kube_features.go
@@ -27,14 +27,14 @@ const (
 	// // alpha: v1.4
 	// MyFeature() bool
 
-	// owner: timstclair
+	// owner: tallclair
 	// alpha: v1.5
 	//
 	// StreamingProxyRedirects controls whether the apiserver should intercept (and follow)
 	// redirects from the backend (Kubelet) for streaming requests (exec/attach/port-forward).
 	StreamingProxyRedirects utilfeature.Feature = "StreamingProxyRedirects"
 
-	// owner: timstclair
+	// owner: tallclair
 	// alpha: v1.7
 	//
 	// AdvancedAuditing enables a much more general API auditing pipeline, which includes support for

diff --git a/staging/src/k8s.io/apiserver/pkg/storage/OWNERS b/staging/src/k8s.io/apiserver/pkg/storage/OWNERS
@@ -13,7 +13,7 @@ reviewers:
 - mikedanese
 - liggitt
 - ncdc
-- timstclair
+- tallclair
 - timothysc
 - hongchaodeng
 - krousey

diff --git a/staging/src/k8s.io/apiserver/pkg/storage/etcd/OWNERS b/staging/src/k8s.io/apiserver/pkg/storage/etcd/OWNERS
@@ -12,7 +12,7 @@ reviewers:
 - luxas
 - janetkuo
 - roberthbailey
-- timstclair
+- tallclair
 - timothysc
 - dims
 - hongchaodeng

diff --git a/staging/src/k8s.io/client-go/OWNERS b/staging/src/k8s.io/client-go/OWNERS
@@ -33,7 +33,7 @@ reviewers:
 - justinsb
 - roberthbailey
 - ncdc
-- timstclair
+- tallclair
 - yifan-gu
 - eparis
 - mwielgus

diff --git a/test/e2e_node/OWNERS b/test/e2e_node/OWNERS
@@ -1,6 +1,6 @@
 approvers:
 - Random-Liu
-- timstclair
+- tallclair
 - vishh
 - derekwaynecarr
 reviewers:

diff --git a/test/e2e_node/apparmor_test.go b/test/e2e_node/apparmor_test.go
@@ -200,7 +200,7 @@ func expectSoftRejection(status v1.PodStatus) {
 }
 
 func isAppArmorEnabled() bool {
-	// TODO(timstclair): Pass this through the image setup rather than hardcoding.
+	// TODO(tallclair): Pass this through the image setup rather than hardcoding.
 	if strings.Contains(framework.TestContext.NodeName, "-gci-dev-") {
 		gciVersionRe := regexp.MustCompile("-gci-dev-([0-9]+)-")
 		matches := gciVersionRe.FindStringSubmatch(framework.TestContext.NodeName)
-Original file line number
+Diff line change
@@ Expand Up / @@ -18,4 +18,4 @@ reviewers: @@
     - gmarek
     - sttts
     - ncdc
-    - timstclair
+    - tallclair