Skip to content

Configuration controller concurrent map accesss #729

Description

@Skarlso
        2025-08-18T07:49:58Z	LEVEL(-4)	cache	cache hit for data	{"controller": "resource", "controllerGroup": "delivery.ocm.software", "controllerKind": "Resource", "Resource": {"name":"resource-1","namespace":"ocm-system"}, "namespace": "ocm-system", "name": "resource-1", "reconcileID": "79182e1e-50ff-4652-ab55-b17fb620c9b0", "name": "sha-285901145611383924", "tag": "1.0.0", "repository": "registry.ocm-system.svc.cluster.local:5000/sha-285901145611383924"}
        fatal error: concurrent map read and map write
        
        goroutine 401 [running]:
        internal/runtime/maps.fatal({0x3c73d02?, 0xeb8f4f?})
        	/opt/hostedtoolcache/go/1.24.4/x64/src/runtime/panic.go:1058 +0x18
        gopkg.in/op/go-logging%2ev1.(*moduleLeveled).GetLevel(...)
        	/home/runner/go/pkg/mod/gopkg.in/op/go-logging.v1@v1.0.0-20160211212156-b2cb9fa56473/level.go:91
        gopkg.in/op/go-logging%2ev1.(*moduleLeveled).IsEnabledFor(0xc0005c3300, 0x5, {0x3c0e21a?, 0xc005a066b8?})
        	/home/runner/go/pkg/mod/gopkg.in/op/go-logging.v1@v1.0.0-20160211212156-b2cb9fa56473/level.go:109 +0x2c
        gopkg.in/op/go-logging%2ev1.(*Logger).IsEnabledFor(...)
        	/home/runner/go/pkg/mod/gopkg.in/op/go-logging.v1@v1.0.0-20160211212156-b2cb9fa56473/logger.go:132
        gopkg.in/op/go-logging%2ev1.(*Logger).log(0xc0004268a0, 0x5, {0x3c4e0d9, 0x19}, {0x0, 0x0, 0x0})
        	/home/runner/go/pkg/mod/gopkg.in/op/go-logging.v1@v1.0.0-20160211212156-b2cb9fa56473/logger.go:136 +0x56
        gopkg.in/op/go-logging%2ev1.(*Logger).Debugf(...)
        	/home/runner/go/pkg/mod/gopkg.in/op/go-logging.v1@v1.0.0-20160211212156-b2cb9fa56473/logger.go:244
        github.com/mikefarah/yq/v4/pkg/yqlib.(*CandidateNode).UnmarshalYAML(0xc006267a40, 0xc007aafae0, 0xc0063d6930)
        	/home/runner/go/pkg/mod/github.com/mikefarah/yq/v4@v4.45.4/pkg/yqlib/candidate_node_yaml.go:114 +0x4dc
        github.com/mikefarah/yq/v4/pkg/yqlib.(*CandidateNode).decodeIntoChild(0xc0062669a0, 0xc007aafae0, 0xc0063d6930)
        	/home/runner/go/pkg/mod/github.com/mikefarah/yq/v4@v4.45.4/pkg/yqlib/candidate_node_yaml.go:101 +0xa5
        github.com/mikefarah/yq/v4/pkg/yqlib.(*CandidateNode).UnmarshalYAML(0xc0062669a0, 0xc007aaf720, 0xc0063d6930)
        	/home/runner/go/pkg/mod/github.com/mikefarah/yq/v4@v4.45.4/pkg/yqlib/candidate_node_yaml.go:132 +0x65e
        github.com/mikefarah/yq/v4/pkg/yqlib.(*CandidateNode).decodeIntoChild(0xc0062667e0, 0xc007aaf720, 0xc0063d6930)
        	/home/runner/go/pkg/mod/github.com/mikefarah/yq/v4@v4.45.4/pkg/yqlib/candidate_node_yaml.go:101 +0xa5
        github.com/mikefarah/yq/v4/pkg/yqlib.(*CandidateNode).UnmarshalYAML(0xc0062667e0, 0xc007aaf680, 0xc0063d6930)
        	/home/runner/go/pkg/mod/github.com/mikefarah/yq/v4@v4.45.4/pkg/yqlib/candidate_node_yaml.go:159 +0x8bc
        github.com/mikefarah/yq/v4/pkg/yqlib.(*CandidateNode).decodeIntoChild(0xc006266460, 0xc007aaf680, 0xc0063d6930)
        	/home/runner/go/pkg/mod/github.com/mikefarah/yq/v4@v4.45.4/pkg/yqlib/candidate_node_yaml.go:101 +0xa5
        github.com/mikefarah/yq/v4/pkg/yqlib.(*CandidateNode).UnmarshalYAML(0xc006266460, 0xc007aaf540, 0xc0063d6930)
        	/home/runner/go/pkg/mod/github.com/mikefarah/yq/v4@v4.45.4/pkg/yqlib/candidate_node_yaml.go:132 +0x65e
        github.com/mikefarah/yq/v4/pkg/yqlib.(*CandidateNode).decodeIntoChild(0xc006b64fc0, 0xc007aaf540, 0xc0063d6930)
        	/home/runner/go/pkg/mod/github.com/mikefarah/yq/v4@v4.45.4/pkg/yqlib/candidate_node_yaml.go:101 +0xa5
        github.com/mikefarah/yq/v4/pkg/yqlib.(*CandidateNode).UnmarshalYAML(0xc006b64fc0, 0xc007aaebe0, 0xc0063d6930)
        	/home/runner/go/pkg/mod/github.com/mikefarah/yq/v4@v4.45.4/pkg/yqlib/candidate_node_yaml.go:132 +0x65e
        github.com/mikefarah/yq/v4/pkg/yqlib.(*CandidateNode).decodeIntoChild(0xc0056875e0, 0xc007aaebe0, 0xc0063d6930)
        	/home/runner/go/pkg/mod/github.com/mikefarah/yq/v4@v4.45.4/pkg/yqlib/candidate_node_yaml.go:101 +0xa5
        github.com/mikefarah/yq/v4/pkg/yqlib.(*CandidateNode).UnmarshalYAML(0xc0056875e0, 0xc007db7b80, 0xc0063d6930)
        	/home/runner/go/pkg/mod/github.com/mikefarah/yq/v4@v4.45.4/pkg/yqlib/candidate_node_yaml.go:132 +0x65e
        github.com/mikefarah/yq/v4/pkg/yqlib.(*CandidateNode).decodeIntoChild(0xc005686a80, 0xc007db7b80, 0xc0063d6930)
        	/home/runner/go/pkg/mod/github.com/mikefarah/yq/v4@v4.45.4/pkg/yqlib/candidate_node_yaml.go:101 +0xa5
        github.com/mikefarah/yq/v4/pkg/yqlib.(*CandidateNode).UnmarshalYAML(0xc005686a80, 0xc007db7540, 0xc0063d6930)
        	/home/runner/go/pkg/mod/github.com/mikefarah/yq/v4@v4.45.4/pkg/yqlib/candidate_node_yaml.go:132 +0x65e
        github.com/mikefarah/yq/v4/pkg/yqlib.(*yamlDecoder).Decode(0xc00849a4d0)
        	/home/runner/go/pkg/mod/github.com/mikefarah/yq/v4@v4.45.4/pkg/yqlib/decoder_yaml.go:136 +0x26a
        ocm.software/ocm/api/utils/subst.Parse({0xc006ab6000, 0x70c, 0x1000})
        	/home/runner/go/pkg/mod/ocm.software/ocm@v0.27.0/api/utils/subst/subst.go:83 +0x309
        ocm.software/ocm/api/utils/subst.ParseFile({0xc001dba198, 0x16}, {0xc005a07170?, 0x15?, 0x1?})
        	/home/runner/go/pkg/mod/ocm.software/ocm@v0.27.0/api/utils/subst/subst.go:35 +0xd5
        ocm.software/ocm/api/ocm/ocmutils/localize.Substitute({0xc0067e27e0, 0x3, 0x316?}, {0x42a5f00, 0xc0083028e0})
        	/home/runner/go/pkg/mod/ocm.software/ocm@v0.27.0/api/ocm/ocmutils/localize/subst.go:21 +0x1d6
        github.com/open-component-model/ocm-controller/controllers.(*MutationReconcileLooper).configure(0xc000b9cae8, {0x4270da0, 0xc00198eb10}, {0xc0060c1000, 0x2600, 0x3000}, {0xc007e94700, 0x316, 0x380}, 0xc007e8d508, ...)
        	/home/runner/work/ocm-controller/ocm-controller/controllers/mutation_reconcile_looper.go:181 +0x429
        github.com/open-component-model/ocm-controller/controllers.(*MutationReconcileLooper).mutate(0x14?, {0x4270da0?, 0xc00198eb10?}, 0x42b90d8?, {0xc0060c1000?, 0x0?, 0x408b32?}, {0xc007e94700, 0x316, 0x380}, ...)
        	/home/runner/work/ocm-controller/ocm-controller/controllers/mutation_reconcile_looper.go:981 +0x7a
        github.com/open-component-model/ocm-controller/controllers.(*MutationReconcileLooper).mutateConfigRef(0xc000b9cae8, {0x4270da0, 0xc00198eb10}, {0x42bd2d8, 0xc007e8d400}, 0xc007e8d508, {0xc0060c1000, 0x2600, 0x3000})
        	/home/runner/work/ocm-controller/ocm-controller/controllers/mutation_reconcile_looper.go:1023 +0x336
        github.com/open-component-model/ocm-controller/controllers.(*MutationReconcileLooper).performMutation(0xc000b9cae8, {0x4270da0, 0xc00198eb10}, {0x42bd2d8, 0xc007e8d400}, 0xc007e8d508, {0xc0060c1000, 0x2600, 0x3000})
        	/home/runner/work/ocm-controller/ocm-controller/controllers/mutation_reconcile_looper.go:123 +0x99
        github.com/open-component-model/ocm-controller/controllers.(*MutationReconcileLooper).ReconcileMutationObject(0xc000b9cae8, {0x4270da0, 0xc00198eb10}, {0x42bd2d8, 0xc007e8d400})
        	/home/runner/work/ocm-controller/ocm-controller/controllers/mutation_reconcile_looper.go:93 +0x290
        github.com/open-component-model/ocm-controller/controllers.(*ConfigurationReconciler).reconcile(0xc000b9ca80, {0x4270da0?, 0xc00198eb10?}, 0xc007e8d400)
        	/home/runner/work/ocm-controller/ocm-controller/controllers/configuration_controller.go:309 +0x145
        github.com/open-component-model/ocm-controller/controllers.(*ConfigurationReconciler).Reconcile(0xc000b9ca80, {0x4270da0, 0xc00198eb10}, {{{0xc001dd4c76?, 0x3c0c839?}, {0xc006e12260?, 0x100?}}})
        	/home/runner/work/ocm-controller/ocm-controller/controllers/configuration_controller.go:291 +0x785
        sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller[...]).Reconcile(0xc00198e5a0?, {0x4270da0?, 0xc00198eb10?}, {{{0xc001dd4c76?, 0x0?}, {0xc006e12260?, 0x0?}}})
        	/home/runner/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.21.0/pkg/internal/controller/controller.go:119 +0xbf
        sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller[...]).reconcileHandler(0x42ba7e0, {0x4270dd8, 0xc000844690}, {{{0xc001dd4c76, 0xa}, {0xc006e12260, 0x1d}}}, 0x0)
        	/home/runner/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.21.0/pkg/internal/controller/controller.go:340 +0x3ad
        sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller[...]).processNextWorkItem(0x42ba7e0, {0x4270dd8, 0xc000844690})
        	/home/runner/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.21.0/pkg/internal/controller/controller.go:300 +0x21b
        sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller[...]).Start.func2.1()
        	/home/runner/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.21.0/pkg/internal/controller/controller.go:202 +0x85
        created by sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller[...]).Start.func2 in goroutine 216
        	/home/runner/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.21.0/pkg/internal/controller/controller.go:198 +0x28f
        

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Type

    Fields

    Priority

    None yet

    Projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions