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

api-server: add the fallback page #640

Merged
merged 3 commits into from
Jun 19, 2020
Merged

Conversation

cwen0
Copy link
Member

@cwen0 cwen0 commented Jun 18, 2020

Signed-off-by: cwen0 [email protected]

What problem does this PR solve?

  • use index.html as the fallback page

cwen0 added 2 commits June 18, 2020 19:04
Signed-off-by: cwen0 <[email protected]>
@codecov-commenter
Copy link

codecov-commenter commented Jun 18, 2020

Codecov Report

Merging #640 into master will decrease coverage by 0.16%.
The diff coverage is 27.58%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #640      +/-   ##
==========================================
- Coverage   55.78%   55.61%   -0.17%     
==========================================
  Files          68       68              
  Lines        4383     4398      +15     
==========================================
+ Hits         2445     2446       +1     
- Misses       1768     1782      +14     
  Partials      170      170              
Impacted Files Coverage Δ
controllers/networkchaos/ipset/ipset.go 37.14% <0.00%> (ø)
pkg/store/experiment/experiment.go 32.20% <0.00%> (-6.58%) ⬇️
pkg/utils/chaosdaemon.go 51.21% <0.00%> (ø)
pkg/utils/grpc.go 0.00% <0.00%> (ø)
controllers/twophase/types.go 54.20% <40.00%> (-1.68%) ⬇️
controllers/networkchaos/netem/types.go 47.27% <50.00%> (ø)
controllers/podchaos/containerkill/types.go 64.38% <100.00%> (ø)
controllers/timechaos/types.go 61.44% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6117650...3b80507. Read the comment docs.

Copy link
Contributor

@yeya24 yeya24 left a comment

Choose a reason for hiding this comment

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

LGTM

@@ -18,7 +18,7 @@ import (
"net/http"
"os"

"github.com/shurcooL/vfsgen"
"github.com/cwen0/vfsgen"
Copy link
Contributor

Choose a reason for hiding this comment

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

Is this change intended? You'd like to use own fork here?

Copy link
Member Author

Choose a reason for hiding this comment

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

The origin repo doesn't support setting the fallback and the repo is rarely updated, but we need this feature. cwen0/vfsgen@c353227

Copy link
Member Author

Choose a reason for hiding this comment

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

I will try to push this pr to the origin repo.

@g1eny0ung g1eny0ung self-requested a review June 19, 2020 01:59
Copy link
Member

@g1eny0ung g1eny0ung left a comment

Choose a reason for hiding this comment

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

LGTM.

@cwen0
Copy link
Member Author

cwen0 commented Jun 19, 2020

/merge

@ti-srebot
Copy link
Contributor

/run-all-tests

@cwen0 cwen0 merged commit 7a01a5d into chaos-mesh:master Jun 19, 2020
@cwen0 cwen0 deleted the fix_api_server branch June 19, 2020 02:34
shonge pushed a commit to shonge/chaos-mesh that referenced this pull request Jul 7, 2020
* api-server: add fallback

Signed-off-by: cwen0 <[email protected]>

* make clean

Signed-off-by: cwen0 <[email protected]>
Signed-off-by: shonge <[email protected]>
sjwsl pushed a commit to sjwsl/chaos-mesh that referenced this pull request May 6, 2021
* api-server: add fallback

Signed-off-by: cwen0 <[email protected]>

* make clean

Signed-off-by: cwen0 <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants