Skip to content

Conversation

fuziontech
Copy link
Member

@fuziontech fuziontech commented May 12, 2022

Problem

Just a tiny little cleanup.

Changes

Produce to the same topic that we expect a message on later to evaluate if Kafka is alive

👉 Stay up-to-date with PostHog coding conventions for a smoother review.

How did you test this code?

CI take the wheel

@fuziontech fuziontech force-pushed the plugin_healthcheck_Fix branch from 999c26a to fbcc692 Compare May 13, 2022 01:51
@fuziontech fuziontech requested a review from yakkomajuri May 13, 2022 01:54
Copy link
Contributor

@yakkomajuri yakkomajuri left a comment

Choose a reason for hiding this comment

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

nice, thanks for stepping on toes

@macobo macobo merged commit 80181d7 into master May 13, 2022
@macobo macobo deleted the plugin_healthcheck_Fix branch May 13, 2022 12:16
fuziontech added a commit that referenced this pull request May 13, 2022
* master:
  fix: exclusion steps cannot be selected (#9762)
  feat(lemon-button): Support `status` for `primary` buttons (#9782)
  fix: healthcheck for kafka on plugin server (#9771)
  fix(billing): Update billing success message (#9739)
  fix(plugin-server): Set transpileOnly when importing piscina code in tests (#9777)
  fix(plugin-server): Remove unused kafka reset test code (#9779)
  fix: set kafka_skip_broken_messages on dead letter queue table (#9754)
  fix(plugin-server): remove dead code from worker.test.ts (#9776)
  refactor(plugin-server): Run ingestion only on worker threads (#9738)
  fix: Plugin-server tests with kafka need to have consumer stopped (#9774)
  chore(deps): Update posthog-js to 1.21.1 (#9773)
  chore(dep): upgrading rr-web (#9772)
  fix: ouroboros inputs (#9769)
alexkim205 pushed a commit that referenced this pull request May 23, 2022
Co-authored-by: yakkomajuri <yakko.majuri@gmail.com>
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.

3 participants