Skip to content
This repository has been archived by the owner on Mar 17, 2024. It is now read-only.

Added collection and export of Earliest Offset #86

Merged
merged 2 commits into from
Oct 7, 2019

Conversation

graphex
Copy link
Contributor

@graphex graphex commented Oct 2, 2019

kafka_partition_earliest_offset is now gathered and exported for all of the topic partitions which had been gathering the latest offset. Closes #67

kafka_partition_earliest_offset is now gathered and exported for all of the topic partitions which had been gathering the latest offset.
@seglo seglo self-requested a review October 3, 2019 14:23
Copy link
Owner

@seglo seglo left a 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
Copy link
Owner

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}
Copy link
Owner

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.

@seglo seglo self-requested a review October 7, 2019 19:46
Copy link
Owner

@seglo seglo left a 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!

Copy link
Owner

@seglo seglo left a 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.

@seglo
Copy link
Owner

seglo commented Oct 7, 2019

I smoke tested the branch though and it looks good. 👍

@graphex-sumo
Copy link

Ok changed the readme references to be for 0.5.4. Wasn't sure if I should change the update_readme_version.sh back to 0.5.0 or leave it as ${VERSION}

@seglo seglo merged commit 52ad430 into seglo:master Oct 7, 2019
@seglo seglo added this to the 0.5.5 milestone Nov 11, 2019
anbarasantr pushed a commit to anbarasantr/kafka-lag-exporter that referenced this pull request Nov 24, 2019
* 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
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Capture earliest available offset for topics
3 participants