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.

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

fix consumer group bug #737

Merged
merged 2 commits into from
Mar 23, 2023
Merged

fix consumer group bug #737

merged 2 commits into from
Mar 23, 2023

Conversation

diego-escobedo
Copy link
Collaborator

@diego-escobedo diego-escobedo commented Mar 23, 2023

Resolves LOT-643, resolves LOT-693

@linear
Copy link

linear bot commented Mar 23, 2023

@diego-escobedo diego-escobedo temporarily deployed to Development March 23, 2023 02:26 — with GitHub Actions Inactive
@linear
Copy link

linear bot commented Mar 23, 2023

LOT-693 Bug - Event Ingestion/Guidance with Go Broken

### Summary

Bug encountered when fully switching over to Porter, which means we were fully using go ingestion in production. This also occurs locally for me (although not sure if everyone has this problem locally, no way to verify this).

The ingestion is able to send the message to the kafka topic, however the guidance does not read the message and doesn't even print anything after

2023/03/22 18:31:54 Before polling for messages...

So messages are stuck right now in Kafka it seems.

I believe that there is no key on the message and that now since we don't have cust_Id that could be an issue as well.

### Steps to reproduce

Try to send an event to the /api/track/ either locally or on the cloud and see if it pops up on the events preview page in metrics.

Here is the event I am sending {

"batch": \[

    {

        "customer_id": "cust_ddfae050b12c4c3a9e092917299296c4",

        "event_name": "mainnet_transaction",

        "idempotency_id": "123456",

        "properties": {

            "shard_id": "2",

            "shard_type": "professional",

            "change": 5

        },

        "time_created": "2023-03-22T19:53:25.665Z"

    }

\]

}

@diego-escobedo diego-escobedo temporarily deployed to Development March 23, 2023 02:44 — with GitHub Actions Inactive
@diego-escobedo diego-escobedo temporarily deployed to Development March 23, 2023 02:44 — with GitHub Actions Inactive
@diego-escobedo diego-escobedo merged commit f030219 into main Mar 23, 2023
@diego-escobedo diego-escobedo deleted the diego/fix-ingestion branch March 23, 2023 03:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant