Thanks to visit codestin.com
Credit goes to github.com

Skip to content

fix EventToMessage event id ... InstanceMonitorDeleted error: invalid path: name is empty #910

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
cgalibern merged 2 commits into from
Nov 13, 2025
Merged

fix EventToMessage event id ... InstanceMonitorDeleted error: invalid path: name is empty #910

cgalibern merged 2 commits into from
Nov 13, 2025

Conversation

@cgalibern
Copy link
Contributor

@cgalibern cgalibern commented Nov 12, 2025

No description provided.

@cgalibern
[daemon] Refactor imon -> omon OrchestrationEnd previous race
1d5d8ae 
    ==================
    WARNING: DATA RACE
    Write at 0x00c000404280 by goroutine 44:
      github.com/opensvc/om3/util/pubsub.(*Msg).AddLabels()
          /root/dev/om3/util/pubsub/main.go:260 +0x89
      github.com/opensvc/om3/daemon/msgbus.(*ObjectOrchestrationEnd).AddLabels()
          <autogenerated>:1 +0x37
      github.com/opensvc/om3/util/pubsub.cmdPubFactory()
          /root/dev/om3/util/pubsub/main.go:790 +0x7c
      github.com/opensvc/om3/util/pubsub.(*Bus).Pub()
          /root/dev/om3/util/pubsub/main.go:775 +0x84
      github.com/opensvc/om3/daemon/omon.(*Manager).delete()
          /root/dev/om3/daemon/omon/main.go:482 +0x6f9
      github.com/opensvc/om3/daemon/omon.(*Manager).worker.func1()
          /root/dev/om3/daemon/omon/main.go:214 +0xac
      runtime.deferreturn()
          /root/god-1.23.7/go/src/runtime/panic.go:611 +0x5d
      github.com/opensvc/om3/daemon/omon.Start.func1()
          /root/dev/om3/daemon/omon/main.go:150 +0x76

    Previous read at 0x00c000404280 by goroutine 27:
      reflect.Value.IsNil()
          /root/god-1.23.7/go/src/reflect/value.go:1553 +0x14b
      encoding/json.mapEncoder.encode()
          /root/god-1.23.7/go/src/encoding/json/encode.go:734 +0xf4
      encoding/json.mapEncoder.encode-fm()
          <autogenerated>:1 +0x84
      encoding/json.structEncoder.encode()
          /root/god-1.23.7/go/src/encoding/json/encode.go:715 +0x2bd
      encoding/json.structEncoder.encode-fm()
          <autogenerated>:1 +0xe4
      encoding/json.ptrEncoder.encode()
          /root/god-1.23.7/go/src/encoding/json/encode.go:887 +0x3d3
      encoding/json.ptrEncoder.encode-fm()
          <autogenerated>:1 +0x84
      encoding/json.structEncoder.encode()
          /root/god-1.23.7/go/src/encoding/json/encode.go:715 +0x2bd
      encoding/json.structEncoder.encode-fm()
          <autogenerated>:1 +0xe4
      encoding/json.ptrEncoder.encode()
          /root/god-1.23.7/go/src/encoding/json/encode.go:887 +0x3d3
      encoding/json.ptrEncoder.encode-fm()
          <autogenerated>:1 +0x84
      encoding/json.(*encodeState).reflectValue()
          /root/god-1.23.7/go/src/encoding/json/encode.go:322 +0x83
      encoding/json.(*encodeState).marshal()
          /root/god-1.23.7/go/src/encoding/json/encode.go:298 +0xea
      encoding/json.Marshal()
          /root/god-1.23.7/go/src/encoding/json/encode.go:164 +0x12b
      github.com/opensvc/om3/daemon/daemondata.marshalEventData()
          /root/dev/om3/daemon/daemondata/data.go:442 +0xe4
      github.com/opensvc/om3/daemon/daemondata.(*data).forwardEvent()
          /root/dev/om3/daemon/daemondata/data.go:452 +0x1a
      github.com/opensvc/om3/daemon/daemondata.(*data).onSubEvent()
          /root/dev/om3/daemon/daemondata/data.go:531 +0xa30
      github.com/opensvc/om3/daemon/daemondata.(*data).run()
          /root/dev/om3/daemon/daemondata/data.go:362 +0xeae
      github.com/opensvc/om3/daemon/daemondata.Start.func1()
          /root/dev/om3/daemon/daemondata/main.go:49 +0xf8

    Goroutine 44 (running) created at:
      github.com/opensvc/om3/daemon/omon.Start()
          /root/dev/om3/daemon/omon/main.go:144 +0x116a
      github.com/opensvc/om3/daemon/imon.objectMonCreator.func1()
          /root/dev/om3/daemon/imon/orchestrate_test.go:812 +0x484

    Goroutine 27 (running) created at:
      github.com/opensvc/om3/daemon/daemondata.Start()
          /root/dev/om3/daemon/daemondata/main.go:47 +0x304
      github.com/opensvc/om3/daemon/daemontesthelper.Setup()
          /root/dev/om3/daemon/daemontesthelper/main.go:72 +0xdf8
      github.com/opensvc/om3/daemon/imon.orchestrateTestFunc()
          /root/dev/om3/daemon/imon/orchestrate_test.go:534 +0xe9
      github.com/opensvc/om3/daemon/imon.Test_Orchestrate_HA_that_dont_call_start.func1()
          /root/dev/om3/daemon/imon/orchestrate_test.go:262 +0xa4
      testing.tRunner()
          /root/god-1.23.7/go/src/testing/testing.go:1690 +0x226
      testing.(*T).Run.gowrap1()
          /root/god-1.23.7/go/src/testing/testing.go:1743 +0x44
    ==================
@cgalibern
[core] Remove unused pather interface and PathOf method from nami…
e00c83d 
…ng/path.go
@cgalibern cgalibern changed the title misc fix EventToMessage event id 791 InstanceMonitorDeleted error: invalid path: name is empty Nov 13, 2025
@cgalibern cgalibern changed the title fix EventToMessage event id 791 InstanceMonitorDeleted error: invalid path: name is empty fix EventToMessage event id ... InstanceMonitorDeleted error: invalid path: name is empty Nov 13, 2025
@cgalibern cgalibern marked this pull request as ready for review November 13, 2025 07:08
@cgalibern cgalibern merged commit cb4ff41 into opensvc:main Nov 13, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@cgalibern