refactor: Remove use of Compress function in sample profile due to issues - Githubissues

edgexfoundry / app-service-configurable

Owner: Applications WG

Apache License 2.0

18 stars 47 forks source link

refactor: Remove use of Compress function in sample profile due to issues #495

Closed lenny-goodell closed 1 year ago

lenny-goodell commented 1 year ago

Compress function will eventual panic the service.

Signed-off-by: Leonard Goodell leonard.goodell@intel.com

If your build fails due to your commit message not passing the build checks, please review the guidelines here: https://github.com/edgexfoundry/app-service-configurable/blob/main/.github/CONTRIBUTING.md

PR Checklist

Please check if your PR fulfills the following requirements:

[x] I am not introducing a breaking change (if you are, flag in conventional commit message with BREAKING CHANGE: describing the break)
[x ] I am not introducing a new dependency (add notes below if you are)
[ ] I have added unit tests for the new feature or bug fix (if not, why?) N/A
[x] I have fully tested (add details below) this the new feature or bug fix (if not, why?)
[ ] I have opened a PR for the related docs change (if not, why?)

Testing Instructions

Run service with sample profile Verify after 10 mins it does not panic.

New Dependency Instructions (If applicable)

lenny-goodell commented 1 year ago

@bnevis-i this will unblock using app-sample in the helm. We randomly selected the Compress function for one of the function pipelines.