Skip to content
New issue

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

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

Already on GitHub? # to your account

feat: implement secret change support, refactor cred mgmt #333

Merged

Conversation

ajcasagrande
Copy link
Contributor

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:

  • I am not introducing a breaking change (if you are, flag in conventional commit message with BREAKING CHANGE: describing the break)
  • 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?)
  • 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

New Dependency Instructions (If applicable)

Signed-off-by: Anthony Casagrande <anthony.j.casagrande@intel.com>
@codecov-commenter
Copy link

Codecov Report

Merging #333 (368d8b8) into main (ca50ff0) will increase coverage by 0.56%.
The diff coverage is 17.26%.

📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

@@            Coverage Diff             @@
##             main     #333      +/-   ##
==========================================
+ Coverage   33.38%   33.95%   +0.56%     
==========================================
  Files          18       18              
  Lines        1800     1832      +32     
==========================================
+ Hits          601      622      +21     
- Misses       1133     1143      +10     
- Partials       66       67       +1     
Impacted Files Coverage Δ
internal/driver/basenotificationresthandler.go 0.00% <0.00%> (ø)
internal/driver/checkstatuses.go 37.61% <0.00%> (-2.20%) ⬇️
internal/driver/onvifclient.go 19.66% <0.00%> (-3.99%) ⬇️
internal/driver/onvifdiscovery.go 27.60% <0.00%> (-0.51%) ⬇️
internal/driver/pullpointsubscriber.go 0.00% <0.00%> (ø)
internal/driver/driver.go 19.42% <29.72%> (+5.32%) ⬆️
internal/driver/credentials.go 75.75% <50.00%> (-24.25%) ⬇️
internal/driver/macmapper.go 86.56% <50.00%> (-2.50%) ⬇️

@presatish presatish requested a review from lenny-goodell May 3, 2023 00:10
go.mod Show resolved Hide resolved
internal/driver/checkstatuses.go Outdated Show resolved Hide resolved
internal/driver/onvifdiscovery.go Outdated Show resolved Hide resolved
Signed-off-by: Anthony Casagrande <anthony.j.casagrande@intel.com>
@ajcasagrande ajcasagrande requested a review from presatish May 3, 2023 17:37
internal/driver/credentials.go Outdated Show resolved Hide resolved
internal/driver/driver.go Show resolved Hide resolved
internal/driver/driver.go Outdated Show resolved Hide resolved
secret cache was being used in a redundant manner, as secret store
already caches some data.

Signed-off-by: Anthony Casagrande <anthony.j.casagrande@intel.com>
@ajcasagrande ajcasagrande requested a review from lenny-goodell May 3, 2023 21:47
Copy link
Member

@lenny-goodell lenny-goodell left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Contributor

@presatish presatish left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@presatish presatish merged commit a510015 into edgexfoundry:main May 3, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants