Merge pull request #176 from jpbetz/fix-noop-granular-case

Fix noop granular case for atomic<->granular schema changes

Author: k8s-ci-robot

typed/reconcile_schema.go

@@ -258,12 +258,13 @@ func buildGranularFieldSet(path fieldpath.Path, value *TypedValue) (*fieldpath.S
 	if err != nil {
 		return nil, errorf("toFieldSet: %v", err)
 	}
-	result := fieldpath.NewSet(path)
-	resultAtPath := descendToPath(result, path)
 	if valueFieldSetAtPath, ok := fieldSetAtPath(valueFieldSet, path); ok {
+		result := fieldpath.NewSet(path)
+		resultAtPath := descendToPath(result, path)
 		*resultAtPath = *valueFieldSetAtPath
+		return result, nil
 	}
-	return result, nil
+	return nil, nil
 }
 
 func fieldSetAtPath(node *fieldpath.Set, path fieldpath.Path) (*fieldpath.Set, bool) {

typed/reconcile_schema_test.go

@@ -267,6 +267,29 @@ var reconcileCases = []reconcileTestCase{{
 		_P("unchanged", "numeric"),
 	),
 	fixedFields: nil, // indicates no change
+}, {
+	name:         "no-change-empty-granular",
+	rootTypeName: "v1",
+	oldSchema:    granularSchema("v1"),
+	newSchema:    granularSchema("v1"),
+	liveObject: typed.YAMLObject(`
+struct: {}
+list: []
+objectList:
+  - keyA: a1
+    keyB: b1
+stringMap: {}
+unchanged: {}
+`),
+	oldFields: _NS(
+		_P("struct"),
+		_P("list"),
+		_P("objectList"),
+		_P("objectList", _KBF("keyA", "a1", "keyB", "b1")),
+		_P("objectList", _KBF("keyA", "a1", "keyB", "b1"), "value"),
+		_P("unchanged"),
+	),
+	fixedFields: nil, // indicates no change
 }}
 
 func TestReconcileFieldSetWithSchema(t *testing.T) {