Skip to content
This repository has been archived by the owner on Jun 6, 2024. It is now read-only.

use api/v1/pods to get all pods #2750

Merged
merged 1 commit into from
May 14, 2019
Merged

use api/v1/pods to get all pods #2750

merged 1 commit into from
May 14, 2019

Conversation

xudifsd
Copy link
Member

@xudifsd xudifsd commented May 13, 2019

Previous watchdog use /api/v1/namespaces/{}/pods iterate all namespace to get all pods, simplify it to get via /api/v1/pods.

This can reduce number of http get watchdog issues.

@xudifsd xudifsd requested review from sterowang and mzmssg May 13, 2019 08:36
@coveralls
Copy link

Coverage Status

Coverage remained the same at 53.064% when pulling 3243e17 on dixu/watchdog-namespace into 17b005e on master.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage remained the same at 53.064% when pulling 3243e17 on dixu/watchdog-namespace into 17b005e on master.

@xudifsd xudifsd merged commit a32a51c into master May 14, 2019
@xudifsd xudifsd deleted the dixu/watchdog-namespace branch May 14, 2019 02:08
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@mzmssg mzmssg mzmssg approved these changes

@sterowang sterowang Awaiting requested review from sterowang

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@xudifsd @coveralls @mzmssg