Skip to content

ApiListener::UpdateObjectAuthority(): distribute auth. by object's host #10161

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.

Sign up for GitHub

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

Jump to bottom
Merged
julianbrost merged 4 commits into from
Jun 6, 2025
Merged

ApiListener::UpdateObjectAuthority(): distribute auth. by object's host #10161

julianbrost merged 4 commits into from
Jun 6, 2025

Conversation

@Al2Klimov
Copy link
Member

@Al2Klimov Al2Klimov commented Sep 18, 2024

Pin child objects of hosts (HOST!...) to the same endpoint as the host. This reduces cross-object action latency withing the same host. If all endpoints know this algorithm, we can use it.

fixes #10157

@Al2Klimov Al2Klimov requested a review from yhabteab September 18, 2024 16:00
@cla-bot cla-bot bot added the cla/signed label Sep 18, 2024
@icinga-probot icinga-probot bot added area/distributed Distributed monitoring (master, satellites, clients) enhancement New feature or request ref/IP labels Sep 18, 2024
@Al2Klimov Al2Klimov requested review from julianbrost and removed request for yhabteab October 2, 2024 08:25
@yhabteab yhabteab mentioned this pull request Oct 22, 2024
Closed
@Al2Klimov Al2Klimov added this to the 2.15.0 milestone Jan 21, 2025
yhabteab
yhabteab reviewed May 16, 2025
View reviewed changes
Copy link
Member

@yhabteab yhabteab left a comment

Choose a reason for hiding this comment

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

Apart from the comments below, please also update the documentation for this behaviour.

@Al2Klimov Al2Klimov mentioned this pull request May 16, 2025
Merged
@Al2Klimov Al2Klimov marked this pull request as ready for review May 16, 2025 13:09
yhabteab
yhabteab reviewed May 19, 2025
View reviewed changes
@Al2Klimov Al2Klimov mentioned this pull request May 20, 2025
Merged
@Al2Klimov Al2Klimov requested a review from yhabteab May 20, 2025 14:05
yhabteab
yhabteab reviewed May 21, 2025
View reviewed changes
@Al2Klimov
For the same zone, call ApiListener::UpdateObjectAuthority() in icing…
18f810a 
…a::Hello

after setting remote capabilities. They'll become important for teamwork in a zone.
Al2Klimov added 2 commits May 23, 2025 10:44
@Al2Klimov
ApiListener::UpdateObjectAuthority(): distribute auth. by object's host
6cd83ba 
Pin child objects of hosts (HOST!...) to the same endpoint as the host.
This reduces cross-object action latency withing the same host.
If all endpoints know this algorithm, we can use it.
@Al2Klimov
Move l_MyCapabilities -> ApiCapabilities::MyCapabilities
77b86bb
@yhabteab yhabteab mentioned this pull request May 28, 2025
Open
yhabteab
yhabteab approved these changes May 28, 2025
View reviewed changes
Copy link
Member

@yhabteab yhabteab left a comment

Choose a reason for hiding this comment

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

LFTM! However, b1e08ba is my commit, so someone else has to approve it :).

@julianbrost julianbrost merged commit 4e08adc into master Jun 6, 2025
26 checks passed
@julianbrost julianbrost deleted the authBYhost-10157 branch June 6, 2025 13:03
@Al2Klimov Al2Klimov mentioned this pull request Jul 2, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@julianbrost julianbrost julianbrost approved these changes

@yhabteab yhabteab yhabteab approved these changes

Assignees

@yhabteab yhabteab

Labels

area/distributed Distributed monitoring (master, satellites, clients) cla/signed enhancement New feature or request ref/IP

Projects

None yet

Milestone

2.15.0

Development

Successfully merging this pull request may close these issues.

Distribute object authority by HOST name

4 participants

@Al2Klimov @julianbrost @yhabteab