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(inbound): report health UP #2413

Merged
merged 7 commits into from
May 3, 2024
Merged

fix(inbound): report health UP #2413

merged 7 commits into from
May 3, 2024

Conversation

chillleader
Copy link
Member

@chillleader chillleader commented May 2, 2024

Description

Report Health.UP on startup for Webhook, Kafka, SQS and SNS inbound connectors (otherwise, the health will be stuck on UNKNOWN in the 8.6 release branch)

@chillleader chillleader self-assigned this May 2, 2024
@chillleader chillleader requested a review from a team as a code owner May 2, 2024 14:51
johnBgood
johnBgood previously approved these changes May 2, 2024
@chillleader chillleader changed the title fix(webhook): report health UP fix(inbound): report health UP May 2, 2024
@chillleader chillleader requested a review from johnBgood May 2, 2024 15:51
Copy link
Contributor

@johnBgood johnBgood left a comment

Choose a reason for hiding this comment

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

👍

@ev-codes
Copy link
Contributor

ev-codes commented May 2, 2024

@chillleader Thanks for this!

I suggest we keep this PR open until I confirm with Sara whether QA is required. If QA is required, then I think we should only merge once QA is complete.

@ev-codes ev-codes added the component:qa Task containing all details related to QA label May 3, 2024
@ev-codes
Copy link
Contributor

ev-codes commented May 3, 2024

@chillleader since this wasn't causing an issue in 8.5, I think we should remove the backport release/8.5 tag.

@ev-codes
Copy link
Contributor

ev-codes commented May 3, 2024

@chillleader sorry one more thing - could you please update the description to indicate why this change is needed?

(i.e., otherwise health will just be stuck on "unknown")

Copy link
Contributor

@ev-codes ev-codes left a comment

Choose a reason for hiding this comment

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

Thanks for addressing everything, approved!

@chillleader chillleader added this pull request to the merge queue May 3, 2024
Merged via the queue into main with commit ac7aba5 May 3, 2024
12 checks passed
@chillleader chillleader deleted the webhook-report-health branch May 3, 2024 08:55
github-actions bot pushed a commit that referenced this pull request May 3, 2024
* fix(webhook): report health

* fix(kafka): report initial health

* fix(sns): report health

* fix(sqs): report health

* fix(sqs): report health

* fix(sqs): report health

* fix(sqs): report health

(cherry picked from commit ac7aba5)
Copy link
Contributor

github-actions bot commented May 3, 2024

Successfully created backport PR for release/8.6:

chillleader added a commit that referenced this pull request May 3, 2024
* fix(webhook): report health

* fix(kafka): report initial health

* fix(sns): report health

* fix(sqs): report health

* fix(sqs): report health

* fix(sqs): report health

* fix(sqs): report health

(cherry picked from commit ac7aba5)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport release/8.6 component:qa Task containing all details related to QA
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants