Skip to content
This repository was archived by the owner on May 14, 2025. It is now read-only.

Add InfluxDB/Grafana charts for Task-run duration #3304

Closed
wants to merge 2 commits into from

Conversation

tzolov
Copy link
Contributor

@tzolov tzolov commented Jul 3, 2019

  • augment the docker-compose-influx to enable task.management.metrics.xxx for influx.
  • add initial scdf-task-batch grafana dashboard to springcloud/spring-cloud-dataflow-grafana-influxdb image.
  • update prometheus, myslq, grafana and influx image versions.

Depends on spring-cloud/spring-cloud-task/pull/612
Depends on spring-attic/spring-cloud-task-app-starters-core/issues/14

Resolves #3295

 - augment the docker-compose-influx to enable task.management.metrics.xxx for influx.
 - add initial scdf-task-batch grafana dashboard to springcloud/spring-cloud-dataflow-grafana-influxdb image.
 - update prometheus, myslq, grafana and influx image versions.

 Depends on spring-cloud/spring-cloud-task/pull/612
 Depends on spring-attic/spring-cloud-task-app-starters-core/issues/14

 Resolves spring-attic#3295
@sabbyanandan sabbyanandan self-assigned this Jul 3, 2019
Copy link
Contributor

@sabbyanandan sabbyanandan left a comment

Choose a reason for hiding this comment

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

Except for that one question, all the changes look straightforward.

@@ -35,6 +35,8 @@ services:
dataflow-server:
image: springcloud/spring-cloud-dataflow-server:${DATAFLOW_VERSION:?DATAFLOW_VERSION is not set!}
container_name: dataflow-server
volumes:
- '/tmp/apps:/apps/'
Copy link
Contributor

Choose a reason for hiding this comment

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

Any specific reasons why we need this volume mount? I don't see it being used in the associated changes in this PR, though.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Nice catch! I will remove it.
I used this volume to register and test the locally build metrics enabled Task apps.

 - this volume was used for testing with locally build apps
Copy link
Contributor

@sabbyanandan sabbyanandan left a comment

Choose a reason for hiding this comment

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

Let's go!

@sabbyanandan
Copy link
Contributor

Merged in ca67cc5.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

Add InfluxDB + Grafana charts for Task-run duration
2 participants