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

Grafana Version Update #524

Merged
merged 1 commit into from
Oct 10, 2023
Merged

Conversation

Dhruv-J
Copy link
Contributor

@Dhruv-J Dhruv-J commented Oct 4, 2023

This PR updates the Grafana docker image used in order to support future plugin development. It also updates the clickhouse-datasource-plugin version to allow additional value types and functionality.

This PR updates the Grafana docker image used in order to support future plugin
development. It also updates the clickhouse-datasource-plugin version to allow
additional value types and functionality.

Signed-off-by: Dhruv-J <dhruvj@vmware.com>
@Dhruv-J
Copy link
Contributor Author

Dhruv-J commented Oct 4, 2023

Will check all utilities and ensure that they work before requesting review.

@codecov
Copy link

codecov bot commented Oct 4, 2023

Codecov Report

Merging #524 (e2a6f3f) into main (65e74c7) will not change coverage.
Report is 1 commits behind head on main.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #524   +/-   ##
=======================================
  Coverage   70.51%   70.51%           
=======================================
  Files          40       40           
  Lines        5253     5253           
=======================================
  Hits         3704     3704           
  Misses       1372     1372           
  Partials      177      177           
Flag Coverage Δ *Carryforward flag
kind-e2e-tests 62.34% <ø> (ø)
kind-multi-cluster-e2e-tests 43.17% <ø> (ø)
python-coverage 55.75% <ø> (ø) Carriedforward from 65e74c7
unit-tests 70.17% <ø> (+0.02%) ⬆️

*This pull request uses carry forward flags. Click here to find out more.

Copy link
Contributor

@yuntanghsu yuntanghsu left a comment

Choose a reason for hiding this comment

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

lgtm

@Dhruv-J
Copy link
Contributor Author

Dhruv-J commented Oct 9, 2023

/theia-test-e2e

@Dhruv-J Dhruv-J merged commit e9289a9 into antrea-io:main Oct 10, 2023
45 of 46 checks passed
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