-
-
Notifications
You must be signed in to change notification settings - Fork 197
Added collection and export of Earliest Offset #86
Conversation
kafka_partition_earliest_offset is now gathered and exported for all of the topic partitions which had been gathering the latest offset.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@graphex Thanks. A few comments regarding the version changes.
README.md
Outdated
@@ -125,7 +132,7 @@ file of the accompanying Helm Chart. | |||
You can install the chart from the local filesystem. | |||
|
|||
``` | |||
helm install https://github.com/lightbend/kafka-lag-exporter/releases/download/v0.5.4/kafka-lag-exporter-0.5.4.tgz | |||
helm install https://github.com/lightbend/kafka-lag-exporter/releases/download/v0.5.5/kafka-lag-exporter-0.5.5.tgz |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do you mind reverting the changes to update the version for the example commands in the README? I'd like it to reflect the current released version for people who browse the project homepage. The release process will automatically update it.
@@ -13,5 +13,5 @@ echo helm install $RELEASE_URL | |||
sed -i -r "s/helm install http\S*/helm install ${RELEASE_URL_ESCAPED}/g" $DIR/../README.md | |||
|
|||
echo Update README.md Running Docker Image version to: | |||
echo lightbend/kafka-lag-exporter:0.5.0 | |||
echo lightbend/kafka-lag-exporter:${VERSION} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is meant to be variable.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Thanks for the contribution!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oops. Was a little premature here, still need those comments handled.
I smoke tested the branch though and it looks good. 👍 |
Ok changed the readme references to be for 0.5.4. Wasn't sure if I should change the |
* Added collection and export of Earliest Offset kafka_partition_earliest_offset is now gathered and exported for all of the topic partitions which had been gathering the latest offset. * referencing 0.5.4 in readme
kafka_partition_earliest_offset is now gathered and exported for all of the topic partitions which had been gathering the latest offset. Closes #67