Skip to content
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.

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

[Device LLRP] Drive code causing events to be sent before SDK is fully initialized which causes panic #235

Closed
lenny-goodell opened this issue May 24, 2023 · 0 comments · Fixed by #236
Assignees
Labels
bug Something isn't working
Milestone

Comments

@lenny-goodell
Copy link
Member

🐞 Bug Report

Affected Services [REQUIRED]

The issue is located in Drive Initialize

Is this a regression? Maybe, did not see issue in Levski

Yes, the previous version in which this bug was not present was: Levski

Description and Minimal Reproduction [REQUIRED]

Start Simulator
Start service
Restart the service
Service will panic when incrementing EventsSent metric because it is nil

Last portion of Driver Initialize needs to be moved to Driver Start so SDK can complete it initialization, i.e. create metric counter

Anything else relevant?

@lenny-goodell lenny-goodell added the bug Something isn't working label May 24, 2023
@lenny-goodell lenny-goodell added this to the Minnesota milestone May 24, 2023
marcpfuller added a commit to marcpfuller/device-rfid-llrp-go that referenced this issue May 24, 2023
…tart

fixes: edgexfoundry#235

Signed-off-by: Marc-Philippe Fuller <marc-philippe.fuller@intel.com>
lenny-goodell pushed a commit that referenced this issue May 24, 2023
…tart (#236)

fixes: #235

Signed-off-by: Marc-Philippe Fuller <marc-philippe.fuller@intel.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
2 participants