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

fix(console): update prometheuses resourceinfo #1669

Merged
merged 1 commit into from
Nov 23, 2021

Conversation

jo-hnny
Copy link
Collaborator

@jo-hnny jo-hnny commented Nov 23, 2021

What type of PR is this?

Uncomment only one /kind <> line, hit enter to put that in a new line, and remove leading whitespace from that line:

/kind bug

What this PR does / why we need it:
1、关闭prometheuses时,之前使用了prometheuses对象上的selfLInk属性,由于selfLInk属性废弃,导致请求失败,现在改用name属性拼接
2、之前会同时请求platform前缀的prometheuses列表和monitor前缀的prometheuses列表,并且合并两者结果,现在修改了resourceInfo,彻底删除platform前缀的请求

Which issue(s) this PR fixes:

Fixes #

Special notes for your reviewer:

Does this PR introduce a user-facing change?:


Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.:


@jo-hnny jo-hnny requested a review from Yacent as a code owner November 23, 2021 07:35
@leoryu leoryu merged commit 6dc5b3a into tkestack:master Nov 23, 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.

2 participants