search

pinpoint-apm / pinpoint-kubernetes

Apache License 2.0
26 stars 37 forks source link

Add Pinpoint HBase Helm Chart #6

Closed seongpyoHong closed 3 years ago

seongpyoHong commented 3 years ago

Install

Please refer to pinpoint/README.md.

seongpyoHong commented 3 years ago

@RoySRose Can you review about this PR?

RoySRose commented 3 years ago

@seongpyoHong Sorry for such a late reply.

I'm going through very busy moments for the last 2 months and still on-going. I don't think I can make time to understand the whole script and discuss it with you at the moment.

Are there any further plans for this? If there is, do you think it will be better for this to be merged?

seongpyoHong commented 3 years ago

There are no other plans for this branch (hbase & zookeeper). I think we need to proceed with the next round of proceeding for the collector or flink, but it would be good if you discuss this part later when you have time.

RoySRose commented 3 years ago

Thanks for the contribution. going to merge this. Let's figure out the details after all the components are up and running.