-
Notifications
You must be signed in to change notification settings - Fork 805
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
blockchaos: api definition and selector implementation #2739
Conversation
Signed-off-by: YangKeao <yangkeao@chunibyo.icu>
Signed-off-by: YangKeao <yangkeao@chunibyo.icu>
Signed-off-by: YangKeao <yangkeao@chunibyo.icu>
Signed-off-by: YangKeao <yangkeao@chunibyo.icu>
[REVIEW NOTIFICATION] This pull request has been approved by:
To complete the pull request process, please ask the reviewers in the list to review by filling The full list of commands accepted by this bot can be found here. Reviewer can indicate their review by submitting an approval review. |
Signed-off-by: YangKeao <yangkeao@chunibyo.icu>
Codecov Report
@@ Coverage Diff @@
## master #2739 +/- ##
==========================================
+ Coverage 41.03% 41.07% +0.04%
==========================================
Files 144 146 +2
Lines 11747 11868 +121
==========================================
+ Hits 4820 4875 +55
- Misses 6565 6627 +62
- Partials 362 366 +4
Continue to review full report at Codecov.
|
Signed-off-by: YangKeao <yangkeao@chunibyo.icu>
Signed-off-by: YangKeao <yangkeao@chunibyo.icu>
Signed-off-by: YangKeao <yangkeao@chunibyo.icu>
Signed-off-by: YangKeao <yangkeao@chunibyo.icu>
Signed-off-by: YangKeao <yangkeao@chunibyo.icu>
Signed-off-by: YangKeao <yangkeao@chunibyo.icu>
Signed-off-by: YangKeao <yangkeao@chunibyo.icu>
@g1eny0ung This PR also modifies the dashboard (as the CI won't pass, if I don't implement the dashboard). PTAL |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The frontend code LGTM.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
/merge |
This pull request has been accepted and is ready to merge. Commit hash: 49f2393
|
* add basic type for blockchaos Signed-off-by: YangKeao <yangkeao@chunibyo.icu> * bootstrap blockchaos and nodevolumepath selector Signed-off-by: YangKeao <yangkeao@chunibyo.icu> * generate files from blockchaos Signed-off-by: YangKeao <yangkeao@chunibyo.icu> * add licenses for examples Signed-off-by: YangKeao <yangkeao@chunibyo.icu> * group the imports Signed-off-by: YangKeao <yangkeao@chunibyo.icu> * register an unknown blockchaos Signed-off-by: YangKeao <yangkeao@chunibyo.icu> * try to add dashboard for blockchaos Signed-off-by: YangKeao <yangkeao@chunibyo.icu> * add webhook Signed-off-by: YangKeao <yangkeao@chunibyo.icu> * regenerate yaml files Signed-off-by: YangKeao <yangkeao@chunibyo.icu> * frontend implementation of blockchaos Signed-off-by: YangKeao <yangkeao@chunibyo.icu> * make check Signed-off-by: YangKeao <yangkeao@chunibyo.icu> Co-authored-by: Ti Chi Robot <ti-community-prow-bot@tidb.io> Signed-off-by: rksharma95 <nikhilrk13jan@gmail.com>
…-mesh#2739)" This reverts commit 618a4cb.
…-mesh#2739)" This reverts commit 618a4cb.
…-mesh#2739)" This reverts commit 618a4cb. Signed-off-by: STRRL <im@strrl.dev>
* Revert "blockchaos: api definition and selector implementation (#2739)" This reverts commit 618a4cb. Signed-off-by: STRRL <im@strrl.dev> * chore: revert changes on frontend Signed-off-by: STRRL <im@strrl.dev> * chore: make check Signed-off-by: STRRL <im@strrl.dev> * restore yarn cache Signed-off-by: YangKeao <yangkeao@chunibyo.icu> Co-authored-by: YangKeao <yangkeao@chunibyo.icu> Co-authored-by: Ti Chi Robot <ti-community-prow-bot@tidb.io>
What problem does this PR solve?
According to the chaos-mesh/rfcs#29 , the
BlockChaos
API definition and the volume path selector implementation is here 😺 .What's changed and how it works?
chaos-mesh/rfcs#29
Document issue: #2738