Skip to content
This repository was archived by the owner on Jan 9, 2025. It is now read-only.

fix: telepresence http server check #251

Merged
merged 4 commits into from
Sep 27, 2024
Merged

fix: telepresence http server check #251

Changes from 1 commit
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
8a7e17a
fix telepresence http server check
leoporoli Sep 26, 2024
c76ef72
remove all Kardinal namespaces when cluster resources is empty
leoporoli Sep 26, 2024
ba44d21
Merge branch 'lporoli/fix-clean-resources' into lporoli/fix-tel-inter
leoporoli Sep 26, 2024
936e9db
Merge branch 'main' into lporoli/fix-tel-inter
leoporoli Sep 27, 2024
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
5 changes: 1 addition & 4 deletions kardinal-cli/cmd/root.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -301,7 +301,7 @@ var telepresenceInterceptCmd = &cobra.Command{

// Is the port open and HTTP
if err := isPortOpenAndHTTP(localPort); err != nil {
log.Fatalf("An error occurred checking port '%s': %s", localPort, err)
log.Fatalf("An error occurred checking HTTP server on port '%s': %s", localPort, err)
}

// is Traffic-manager installed
Expand Down Expand Up @@ -424,9 +424,6 @@ func isPortOpenAndHTTP(localPortStr string) error {
}
defer resp.Body.Close()

if resp.StatusCode != http.StatusOK {
return stacktrace.NewError("the HTTP server in 'httpServerAddr' did not return a successful response. Got '%d'", resp.StatusCode)
}
return nil
}

Expand Down
Loading