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.

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

Test Integration: Hot Reload spec.ignoreErrors #7514

Open
wants to merge 16 commits into
base: master
Choose a base branch
from

Conversation

JoshVanL
Copy link
Contributor

@JoshVanL JoshVanL commented Feb 8, 2024

Adds integration tests that assert that the spec.ignoreErrors field is respected when hot reloading a component. Namely that daprd exits error when spec.ignoreErrors=false, and continues running when spec.ignoreErrors=true.

No implementation changes.

@JoshVanL JoshVanL requested review from a team as code owners February 8, 2024 22:07
Copy link

codecov bot commented Feb 8, 2024

Codecov Report

Attention: Patch coverage is 0% with 2 lines in your changes are missing coverage. Please review.

Project coverage is 61.35%. Comparing base (091a204) to head (1449fcc).
Report is 8 commits behind head on master.

Current head 1449fcc differs from pull request most recent head 8fd298c

Please upload reports for the commit 8fd298c to get more accurate results.

Files Patch % Lines
pkg/runtime/processor/components.go 0.00% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #7514      +/-   ##
==========================================
- Coverage   61.39%   61.35%   -0.04%     
==========================================
  Files         265      265              
  Lines       22609    22609              
==========================================
- Hits        13880    13872       -8     
- Misses       7579     7586       +7     
- Partials     1150     1151       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@JoshVanL JoshVanL added the autoupdate DaprBot will keep the Pull Request up to date with master branch label Feb 12, 2024
cicoyle
cicoyle previously approved these changes Feb 26, 2024
Copy link
Contributor

@cicoyle cicoyle left a comment

Choose a reason for hiding this comment

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

LGTM. There are some errors in CI for placement vnodes and component already exists, maybe those are flaky?

@JoshVanL JoshVanL added this to the v1.14 milestone Mar 5, 2024
@JoshVanL JoshVanL force-pushed the int-test-hotreload-ignoreErrors branch 2 times, most recently from 637d323 to 826caa7 Compare March 12, 2024 16:38
artursouza
artursouza previously approved these changes Mar 26, 2024
@artursouza artursouza added the automerge Allows DaprBot to automerge and update PR if all approvals are in place label Mar 26, 2024
Adds integration tests that assert that the `spec.ignoreErrors` field
works as expected when hot reloading a component, namely that daprd
exits error when `spec.ignoreErrors=false` and continues running
when `spec.ignoreErrors=true`.

No implementation changes.

Signed-off-by: joshvanl <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge Allows DaprBot to automerge and update PR if all approvals are in place autoupdate DaprBot will keep the Pull Request up to date with master branch
Projects
Development

Successfully merging this pull request may close these issues.

None yet

4 participants