-
Notifications
You must be signed in to change notification settings - Fork 484
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
feat: Add injection of Secure MessageBus creds for eKuiper connections #3778
feat: Add injection of Secure MessageBus creds for eKuiper connections #3778
Conversation
055e29e
to
ecc9073
Compare
close edgexfoundry#3767 Signed-off-by: Leonard Goodell <[email protected]>
ecc9073
to
27152c7
Compare
Codecov Report
@@ Coverage Diff @@
## main #3778 +/- ##
==========================================
- Coverage 45.94% 45.93% -0.01%
==========================================
Files 112 112
Lines 9595 9598 +3
==========================================
+ Hits 4408 4409 +1
- Misses 4806 4807 +1
- Partials 381 382 +1
Continue to review full report at Codecov.
|
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
@lenny-intel sorry for the late review. Unfortunately, this change breaks the snaps because
(The error is not printed due to another bug: #3784) Could you please clarify if this addition depends on eKuiper 1.4.0? The Thanks |
close #3767
Signed-off-by: Leonard Goodell [email protected]
If your build fails due to your commit message not passing the build checks, please review the guidelines here: https://github.com/edgexfoundry/edgex-go/blob/main/.github/Contributing.md
PR Checklist
Please check if your PR fulfills the following requirements:
BREAKING CHANGE:
describing the break)Testing Instructions
For existing functionality not broken:
make docker_security_secretstore_setup
edgexfoundry/security-secretstore-setup:0.0.0-dev
make run
For new connections functionality testing TBD when 1.4.0-beta.1-alpine is available
New Dependency Instructions (If applicable)
N/A