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

Add managed endpoint count to status command #499

Merged
merged 1 commit into from
Sep 1, 2021

Conversation

zhaojizhuang
Copy link
Contributor

Fix #495

Add managed endpoint count to status command
Signed-off-by: zhaojizhuang [email protected]

@zhaojizhuang
Copy link
Contributor Author

zhaojizhuang commented Aug 26, 2021

[root@ ~/code/src/cilium-cli]# cilium status
    /¯¯\
 /¯¯\__/¯¯\    Cilium:         OK
 \__/¯¯\__/    Operator:       OK
 /¯¯\__/¯¯\    Hubble:         disabled
 \__/¯¯\__/    ClusterMesh:    disabled
    \__/

DaemonSet         cilium             Desired: 1, Ready: 1/1, Available: 1/1
Deployment        cilium-operator    Desired: 1, Ready: 1/1, Available: 1/1
Containers:       cilium             Running: 1
                  cilium-operator    Running: 1
Cluster Pods:     53/55 managed by Cilium
Image versions    cilium             quay.io/cilium/cilium:v1.10.3: 1
                  cilium-operator    quay.io/cilium/operator-generic:v1.10.3: 1

@zhaojizhuang zhaojizhuang temporarily deployed to ci August 26, 2021 03:40 Inactive
@zhaojizhuang zhaojizhuang temporarily deployed to ci August 26, 2021 06:20 Inactive
@zhaojizhuang
Copy link
Contributor Author

@zhaojizhuang zhaojizhuang reopened this Aug 26, 2021
@zhaojizhuang zhaojizhuang temporarily deployed to ci August 26, 2021 07:09 Inactive
@zhaojizhuang zhaojizhuang reopened this Aug 27, 2021
@zhaojizhuang zhaojizhuang temporarily deployed to ci August 27, 2021 03:01 Inactive
@zhaojizhuang zhaojizhuang reopened this Aug 27, 2021
@zhaojizhuang zhaojizhuang temporarily deployed to ci August 27, 2021 06:06 Inactive
@zhaojizhuang
Copy link
Contributor Author

AKS failed https://github.com/cilium/cilium-cli/pull/499/checks?check_run_id=3429678559

Errors:      cilium                   cilium                   Get "http://localhost:8080/apis/apps/v1/namespaces/kube-system/daemonsets/cilium": dial tcp [::1]:8080: connect: connection refused
             cilium-operator          cilium-operator          Get "http://localhost:8080/apis/apps/v1/namespaces/kube-system/deployments/cilium-operator": dial tcp [::1]:8080: connect: connection refused
Warnings:    hubble-relay             hubble-relay             hubble relay is not deployed
             clustermesh-apiserver    clustermesh-apiserver    clustermesh is not deployed

🔍 Collecting Kubernetes nodes
failed to collect sysdump: failed to collect Kubernetes nodes: Get "http://localhost:8080/api/v1/nodes": dial tcp [::1]:8080: connect: connection refused

@tklauser AKS intalled failed

@zhaojizhuang zhaojizhuang reopened this Aug 28, 2021
@zhaojizhuang zhaojizhuang temporarily deployed to ci August 28, 2021 14:16 Inactive
@zhaojizhuang zhaojizhuang reopened this Aug 30, 2021
@zhaojizhuang zhaojizhuang temporarily deployed to ci August 30, 2021 12:09 Inactive
@tklauser tklauser requested a review from joestringer August 31, 2021 17:33
Copy link
Member

@joestringer joestringer left a comment

Choose a reason for hiding this comment

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

🎉 LGTM, thanks for the submission!

@tklauser
Copy link
Member

tklauser commented Sep 1, 2021

Thanks @zhaojizhuang

PR approved by all requested teams. Azure tests (https://github.com/cilium/cilium-cli/runs/3476088449?check_suite_focus=true) and kind tests (https://github.com/cilium/cilium-cli/runs/3461510199?check_suite_focus=true) both failed because of #367. This is an external contributor's PR, so the zero flake policy doesn't apply. Merging.

@tklauser tklauser merged commit 00be808 into cilium:master Sep 1, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add managed endpoint count to status command
4 participants