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.

Sign up for GitHub

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

Jump to bottom

[Backport release-v0.7] fix: cache the server version info of kubernetes #977

Merged
merged 9 commits into from
Aug 30, 2022
Merged

[Backport release-v0.7] fix: cache the server version info of kubernetes #977

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
3faae2c
fix: cache the server version info of kubernetes
Sodawyx Jul 31, 2022
32e746b
fix: fix the errors according to the comments
Sodawyx Aug 1, 2022
b5796c8
fix: fix the bug of cache the non-resource info update
Sodawyx Aug 3, 2022
663359f
fix: delete the change before
Sodawyx Aug 3, 2022
e2b3129
fix: update the change
Sodawyx Aug 4, 2022
0c0c49b
fix: update the change
Sodawyx Aug 9, 2022
3e25e8c
fix: update the unit test
Sodawyx Aug 10, 2022
b1992ce
fix: fix the go pipeline error
Sodawyx Aug 12, 2022
0c35e00
fix: fix the errors
Sodawyx Aug 13, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
fix: fix the errors 
(cherry picked from commit 7bab3ab)
  • Loading branch information
@Sodawyx @github-actions
Sodawyx authored and github-actions[bot] committed Aug 30, 2022
commit 0c35e00398276038fc775b8f401d0b5ee33fe5f7
1 change: 1 addition & 0 deletions pkg/yurthub/server/nonresource.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -59,6 +59,7 @@ func withNonResourceRequest(w http.ResponseWriter, req *http.Request) {
for i := range nonResourceReqPaths {
if req.URL.Path == nonResourceReqPaths[i] {
cacheNonResourceInfo(w, req, cfg.StorageWrapper, fmt.Sprintf("non-reosurce-info%s", nonResourceReqPaths[i]), nonResourceReqPaths[i], false)
break
}
}

Expand Down
5 changes: 3 additions & 2 deletions pkg/yurthub/server/nonresource_test.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -18,12 +18,13 @@ package server

import (
"fmt"
"github.com/openyurtio/openyurt/pkg/yurthub/cachemanager"
"github.com/openyurtio/openyurt/pkg/yurthub/storage/disk"
"net/http"
"net/http/httptest"
"os"
"testing"

"github.com/openyurtio/openyurt/pkg/yurthub/cachemanager"
"github.com/openyurtio/openyurt/pkg/yurthub/storage/disk"
)

var rootDir = "/tmp/cache-local"
Expand Down