Skip to content

Commit

Permalink
path fix for docker/Dockerfile and bin/single-cluster-monitor.sh (#296)
Browse files Browse the repository at this point in the history
After cloning the project, I found that docker/Dockerfile is not able to build due to the path error of kafka-monitor-start.sh. Also com/linkedin/kmf/apps/SingleClusterMonitor path was wrong in bin/single-cluster-monitor.sh file.


Co-authored-by: talipk <[email protected]>
Reviewed-by: Andrew Choi <[email protected]>
  • Loading branch information
talipkorkmaz and talipk authored Sep 28, 2020
1 parent e687c01 commit a099710
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion bin/single-cluster-monitor.sh
Original file line number Diff line number Diff line change
Expand Up @@ -9,4 +9,4 @@

base_dir=$(dirname $0)

exec $base_dir/kmf-run-class.sh com/linkedin/kmf/apps/SingleClusterMonitor $@
exec $base_dir/kmf-run-class.sh com/linkedin/xinfra/monitor/apps/SingleClusterMonitor $@
2 changes: 1 addition & 1 deletion docker/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ MAINTAINER [email protected]

WORKDIR /opt/kafka-monitor
ADD build/ build/
ADD bin/kafka-monitor-start.sh bin/kafka-monitor-start.sh
ADD bin/xinfra-monitor-start.sh bin/xinfra-monitor-start.sh
ADD bin/kmf-run-class.sh bin/kmf-run-class.sh
ADD config/xinfra-monitor.properties config/xinfra-monitor.properties
ADD config/log4j2.properties config/log4j2.properties
Expand Down

0 comments on commit a099710

Please sign in to comment.