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

Feature: Add podyaml and nodeyaml panel in dashboard and corresponding API #110

Merged
merged 1 commit into from
Apr 10, 2024

Conversation

wuchang0201
Copy link
Contributor

Feature show podyaml and nodeyaml, custom log

@wuchang0201
Copy link
Contributor Author

example:
image

@@ -2,28 +2,22 @@
FROM node as node
# 将代码拷贝到镜像中的工作目录
COPY ./grafana_plugin/yaml_panel /app/yaml_panel
COPY ./grafana_plugin/custom_log_panel /app/custom_log_panel
Copy link
Collaborator

Choose a reason for hiding this comment

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

why remove this plugin?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

There is a bug in custom_log, so do not use this plugin

@d3c3mber d3c3mber changed the title Feature podyaml and nodeyaml Feature: Add podyaml and nodeyaml panel in dashboard and corresponding API Apr 9, 2024
@d3c3mber
Copy link
Collaborator

d3c3mber commented Apr 9, 2024

can you please squash the commits into one? thanks

@wuchang0201 wuchang0201 force-pushed the feature_podyaml branch 2 times, most recently from d6bd641 to 3e55182 Compare April 9, 2024 04:15
Copy link
Collaborator

@linuzb linuzb left a comment

Choose a reason for hiding this comment

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

LGTM

@linuzb linuzb merged commit 86b0988 into alipay:main Apr 10, 2024
2 checks passed
@wuchang0201
Copy link
Contributor Author

#81

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.

3 participants