Skip to content

Reduce log spam#21

Merged
ieQu1 merged 3 commits into
masterfrom
dev/clarify-hooks
Jun 18, 2026
Merged

Reduce log spam#21
ieQu1 merged 3 commits into
masterfrom
dev/clarify-hooks

Conversation

@ieQu1

@ieQu1 ieQu1 commented Jun 18, 2026

Copy link
Copy Markdown
Member

No description provided.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR clarifies hook behavior by introducing named priority bounds for built-in hooks, adjusts a few tracepoint calls for consistency, and changes default site/cluster ID generation to use uppercase hex encoding.

Changes:

  • Generate default site/cluster IDs as uppercase hex instead of base64.
  • Add ?max_hook_prio / ?min_hook_prio macros and use them for built-in hook registration priorities.
  • Simplify/standardize some ?tp(...) tracepoint invocations in membership sync paths.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.

File Description
src/classy_node.erl Switches default persistent ID generation from base64 to uppercase hex.
src/classy_membership.erl Updates several tracepoint calls to a 2-argument form and slightly reformats one trace call.
src/classy_internal.hrl Introduces hook priority bound macros used by built-in hook registration.
src/classy_hook.erl Uses the new hook priority macros for default/built-in hooks.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@ieQu1 ieQu1 marked this pull request as ready for review June 18, 2026 19:48
@ieQu1 ieQu1 changed the title Dev/clarify hooks Reduce log spam Jun 18, 2026
@ieQu1 ieQu1 merged commit f9dc631 into master Jun 18, 2026
5 checks passed
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.

2 participants