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.

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

Updated value of perTopicPipeline didn't enabled before restarting service #1187

Closed
cherrycl opened this issue Oct 17, 2022 · 1 comment · Fixed by #1198
Closed

Updated value of perTopicPipeline didn't enabled before restarting service #1187

cherrycl opened this issue Oct 17, 2022 · 1 comment · Fixed by #1198
Assignees
@lenny-goodell
Labels
bug Something isn't working

Comments

@cherrycl
Copy link
Contributor

🐞 Bug Report

Affected Services [REQUIRED]

The issue is located in: app-services

Is this a regression?

No

Description and Minimal Reproduction [REQUIRED]

  1. Deploy EdgeX with compose file
  2. Set PipelineMessagesProcessed to true on edgex/appservices/2.0/app-sample/Writable/Telemetry/Metrics
  3. Validate timer-count of float-pipeline is increased
  4. Updated Topics to invalid device name edgex/events/device/#/invalid-device/# on edgex/appservices/2.0/app-sample/Writable/Pipeline/PerTopicPipelines/float

timer-count of float-pipeline is still increased. After restarting app-sample service, the timer-count is not increased.

🔥 Exception or Error

🌍 Your Environment

Deployment Environment: Ubuntu

EdgeX Version [REQUIRED]: Main

Anything else relevant?
@cherrycl cherrycl added the bug Something isn't working label Oct 17, 2022
@cherrycl cherrycl mentioned this issue Oct 18, 2022
Merged
5 tasks
@lenny-goodell lenny-goodell moved this to New Issues in Application Services Oct 19, 2022
@lenny-goodell lenny-goodell self-assigned this Oct 19, 2022
@lenny-goodell lenny-goodell moved this from New Issues to In Progress in Application Services Oct 24, 2022
@lenny-goodell
Copy link
Member

I have reproduced this issue, but not root caused yet.

While reproducing this I for this bug: #1196

@lenny-goodell lenny-goodell linked a pull request Oct 24, 2022 that will close this issue
fix: Update Pipeline topics when writable pipeline changed #1198
Merged
5 tasks
@lenny-goodell lenny-goodell mentioned this issue Oct 24, 2022
Merged
5 tasks
lenny-goodell pushed a commit to lenny-goodell/app-functions-sdk-go that referenced this issue Oct 24, 2022
fix: Update Pipeline topics when writable pipeline changed
f911664 
fixes edgexfoundry#1187

Goodell <[email protected]>
@lenny-goodell lenny-goodell moved this from In Progress to In Review in Application Services Oct 24, 2022
lenny-goodell pushed a commit to lenny-goodell/app-functions-sdk-go that referenced this issue Oct 24, 2022
fix: Update Pipeline topics when writable pipeline changed
9078b7e 
fixes edgexfoundry#1187

Signed-off-by: Leonard Goodell <[email protected]>
lenny-goodell pushed a commit that referenced this issue Oct 25, 2022
fix: Update Pipeline topics when writable pipeline changed (#1198)
bf1cebf 
fixes #1187

Signed-off-by: Leonard Goodell <[email protected]>
Repository owner moved this from In Review to Done in Application Services Oct 25, 2022
@lenny-goodell lenny-goodell moved this to Todo in Kamakura Issues Nov 8, 2022
@lenny-goodell lenny-goodell moved this from Todo to Done in main in Kamakura Issues Oct 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@lenny-goodell lenny-goodell

Labels
bug Something isn't working
Projects
No open projects
Application Services
Archived in project
Kamakura Issues
Status: Done in main
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: Update Pipeline topics when writable pipeline changed lenny-goodell/app-functions-sdk-go
2 participants
@cherrycl @lenny-goodell