Merge pull request #252 from replicatedhq/fix-statefulset-status

marccampbell · web-flow · commit 2289e0055879 · 2020-09-03T14:00:16.000-07:00
Fix statefulset status
diff --git a/pkg/analyze/deployment_status.go b/pkg/analyze/deployment_status.go
@@ -24,7 +24,7 @@ func analyzeDeploymentStatus(analyzer *troubleshootv1beta2.DeploymentStatus, get
 	var status *appsv1.DeploymentStatus
 	for _, deployment := range deployments {
 		if deployment.Name == analyzer.Name {
-			status = &deployment.Status
+			status = deployment.Status.DeepCopy()
 		}
 	}
 
diff --git a/pkg/analyze/statefulset_status.go b/pkg/analyze/statefulset_status.go
@@ -24,7 +24,7 @@ func analyzeStatefulsetStatus(analyzer *troubleshootv1beta2.StatefulsetStatus, g
 	var status *appsv1.StatefulSetStatus
 	for _, statefulset := range statefulsets {
 		if statefulset.Name == analyzer.Name {
-			status = &statefulset.Status
+			status = statefulset.Status.DeepCopy()
 		}
 	}
 
diff --git a/pkg/apis/troubleshoot/v1beta1/zz_generated.deepcopy.go b/pkg/apis/troubleshoot/v1beta1/zz_generated.deepcopy.go
@@ -16,7 +16,7 @@ See the License for the specific language governing permissions and
 limitations under the License.
 */
 
-// Code generated by controller-gen. DO NOT EDIT.
+// autogenerated by controller-gen object, do not modify manually
 
 package v1beta1
 
diff --git a/pkg/apis/troubleshoot/v1beta2/zz_generated.deepcopy.go b/pkg/apis/troubleshoot/v1beta2/zz_generated.deepcopy.go
@@ -16,7 +16,7 @@ See the License for the specific language governing permissions and
 limitations under the License.
 */
 
-// Code generated by controller-gen. DO NOT EDIT.
+// autogenerated by controller-gen object, do not modify manually
 
 package v1beta2
 

Original file line number	Diff line number	Diff line change
`@@ -24,7 +24,7 @@ func analyzeDeploymentStatus(analyzer *troubleshootv1beta2.DeploymentStatus, get`
`24`	`24`	`var status *appsv1.DeploymentStatus`
`25`	`25`	`for _, deployment := range deployments {`
`26`	`26`	`if deployment.Name == analyzer.Name {`
`27`		`- status = &deployment.Status`
	`27`	`+ status = deployment.Status.DeepCopy()`
`28`	`28`	`}`
`29`	`29`	`}`
`30`	`30`
Original file line number	Diff line number	Diff line change
`@@ -24,7 +24,7 @@ func analyzeStatefulsetStatus(analyzer *troubleshootv1beta2.StatefulsetStatus, g`
`24`	`24`	`var status *appsv1.StatefulSetStatus`
`25`	`25`	`for _, statefulset := range statefulsets {`
`26`	`26`	`if statefulset.Name == analyzer.Name {`
`27`		`- status = &statefulset.Status`
	`27`	`+ status = statefulset.Status.DeepCopy()`
`28`	`28`	`}`
`29`	`29`	`}`
`30`	`30`