Skip to content
This repository was archived by the owner on Jul 11, 2023. It is now read-only.

Conversation

shalier
Copy link
Contributor

@shalier shalier commented Sep 2, 2022

Description:
Cherry pick commit 68e99eb from PR #5065 to release-v1.2 branch to fix circular dependency bug.

Testing done:

Affected area:

Functional Area
Control Plane [x]
Install [x]
Upgrade [x]

Please answer the following questions with yes/no.

  1. Does this change contain code from or inspired by another project?

    • Did you notify the maintainers and provide attribution?
  2. Is this a breaking change?

  3. Has documentation corresponding to this change been updated in the osm-docs repo (if applicable)?

steeling
steeling previously approved these changes Sep 2, 2022
keithmattix
keithmattix previously approved these changes Sep 2, 2022
Copy link
Contributor

@keithmattix keithmattix left a comment

Choose a reason for hiding this comment

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

Approving pending CI

Signed-off-by: Shalier Xia <shalierxia@microsoft.com>
@codecov-commenter
Copy link

Codecov Report

Merging #5069 (6f42427) into release-v1.2 (d2175d3) will increase coverage by 0.17%.
The diff coverage is 0.00%.

@@               Coverage Diff                @@
##           release-v1.2    #5069      +/-   ##
================================================
+ Coverage         68.62%   68.79%   +0.17%     
================================================
  Files               220      210      -10     
  Lines             15941    15657     -284     
================================================
- Hits              10939    10771     -168     
+ Misses             4950     4835     -115     
+ Partials             52       51       -1     
Flag Coverage Δ
unittests 68.79% <0.00%> (+0.17%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
cmd/osm-bootstrap/osm-bootstrap.go 45.36% <0.00%> (-0.89%) ⬇️
pkg/metricsstore/metricsstore.go 92.67% <ø> (-0.23%) ⬇️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@trstringer trstringer merged commit 0b6a18f into openservicemesh:release-v1.2 Sep 2, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants