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.

Sign up for GitHub

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

39: add jolokia for zookeeper monitoring #40

Closed
wants to merge 2 commits into from
Closed

39: add jolokia for zookeeper monitoring #40

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
1b7d6a2
39: add jolokia for zookeeper monitoring
Sep 28, 2017
2d1e490
39: after review
olga-casian Oct 5, 2017
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion Dockerfile
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -57,6 +57,6 @@ COPY scm-source.json /scm-source.json
WORKDIR ${HOME}
USER ${USER}

EXPOSE 2181 2888 3888 8181 8778
EXPOSE 2181 2888 3888 8181 8778 8779
Copy link
Contributor

Choose a reason for hiding this comment

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

Can you please add port mapping here: https://github.com/zalando-nakadi/exhibitor-appliance/blob/master/exhibitor-appliance.yaml#L51


ENTRYPOINT ["bash", "-ex", "/opt/exhibitor/run.sh"]
5 changes: 5 additions & 0 deletions exhibitor-appliance.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -49,6 +49,7 @@ SenzaComponents:
3888: 3888
8181: 8181
8778: 8778
8779: 8779
root: True
environment:
S3_BUCKET: "{{Arguments.ExhibitorBucket}}"
Expand Down Expand Up @@ -110,6 +111,10 @@ Resources:
FromPort: 8778
ToPort: 8778
CidrIp: 0.0.0.0/0
- IpProtocol: tcp
FromPort: 8779
ToPort: 8779
CidrIp: 0.0.0.0/0
ExhibitorLoadBalancer:
Type: AWS::ElasticLoadBalancing::LoadBalancer
Properties:
Expand Down
1 change: 1 addition & 0 deletions exhibitor.conf.tmpl
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -11,3 +11,4 @@ election-port=3888
zoo-cfg-extra=tickTime\=2000&initLimit\=10&syncLimit\=5&quorumListenOnAllIPs\=true
auto-manage-instances-settling-period-ms=0
auto-manage-instances=1
java-environment=#!/bin/bash \nexport JAVA_TOOL_OPTIONS="-javaagent:/opt/jolokia-jvm-${JOLOKIA_VERSION}-agent.jar=port=8779,host=0.0.0.0"
Copy link
Contributor

Choose a reason for hiding this comment

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

Can you please explain what is java-environment and how this thing is used? I failed to find documentation for it

Copy link
Author

Choose a reason for hiding this comment

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

The value of java-environment is placed in Config -> Ensemble -> java.env script. It is "Script to write as the “java.env” file which gets executed as part of the ZooKeeper start script. Optional – leave blank for no file." docs

2 changes: 1 addition & 1 deletion run.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -46,7 +46,7 @@ fi

exec 2>&1

java -javaagent:/opt/jolokia-jvm-1.3.7-agent.jar=port=8778,host=0.0.0.0 -jar /opt/exhibitor/exhibitor.jar \
java -javaagent:/opt/jolokia-jvm-${JOLOKIA_VERSION}-agent.jar=port=8778,host=0.0.0.0 -jar /opt/exhibitor/exhibitor.jar \
--port 8181 --defaultconfig exhibitor.conf \
--configtype $CONFIG_TYPE \
--hostname ${HOSTNAME} \
Expand Down