Skip to content

Conversation

@masaori335
Copy link
Contributor

Dump all HostDBRecords to see current health state of each origin servers in the HostDB.

@masaori335 masaori335 added this to the 10.2.0 milestone Oct 23, 2025
@masaori335 masaori335 self-assigned this Oct 23, 2025
@masaori335 masaori335 added HostDB traffic_ctl traffic_ctl related work. labels Oct 23, 2025
brbzull0
brbzull0 previously approved these changes Oct 23, 2025
Copy link
Contributor

@brbzull0 brbzull0 left a comment

Choose a reason for hiding this comment

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

looks good. Just one comment.

brbzull0
brbzull0 previously approved these changes Oct 24, 2025
Copy link
Contributor

@brbzull0 brbzull0 left a comment

Choose a reason for hiding this comment

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

Thanks.

brbzull0
brbzull0 previously approved these changes Oct 30, 2025
@masaori335
Copy link
Contributor Author

@bneradt Updated to follow your comment. PTAL!

@bneradt
Copy link
Contributor

bneradt commented Oct 31, 2025

@bneradt Updated to follow your comment. PTAL!

Looks good, but we'll have to make the compiler happy by returning "" at the end now:

../src/mgmt/rpc/handlers/hostdb/HostDB.cc:51:1: error: control reaches end of non-void function [-Werror=return-type]
   51 | }
      | ^

@masaori335 masaori335 merged commit bf0eb13 into apache:master Nov 5, 2025
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

HostDB traffic_ctl traffic_ctl related work.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants