Clarify duplicate host management in federated reporting docs#3645
Open
larsewi wants to merge 2 commits into
Open
Clarify duplicate host management in federated reporting docs#3645larsewi wants to merge 2 commits into
larsewi wants to merge 2 commits into
Conversation
571c83a to
1d4687e
Compare
craigcomstock
requested changes
May 12, 2026
| - hosts are able to "float", re-bootstrap or failover to several different feeder hubs | ||
| - hosts may be cloned and not have their hostkey refreshed by running `cf-key` and refreshing `$(sys.workdir)/ppkeys/localhost.pub`. | ||
| CFEngine provides two mechanisms for resolving these cross-feeder duplicates. | ||
| They address the same underlying problem with different approaches, and they can be enabled together: |
Contributor
There was a problem hiding this comment.
Can they be enabled together? The destination-side would take only add the newest record into the superhub database, without the duplicate entries in the superhub the source-side process would have nothing to delete on the feeders. I don't think this comment is accurate.
Contributor
Author
There was a problem hiding this comment.
I don't see why not. But I guess it would not make any difference, so might as well remove it ;)
Reframe the "Duplicate host management" intro around the approach distinction rather than separate lifecycle scenarios: both mechanisms address the same underlying problem (same hostkey across multiple feeders), with distributed cleanup as a source-side cleanup and handle duplicate hostkeys as a destination-side filter. Note that neither deduplicates clones reporting to the same feeder, since a feeder's __hosts table is keyed on hostkey. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
The script has since been changed to ask for admin username and password separately (defaulting to admin) and to prompt for a 2FA code when the admin account has 2FA enabled. Update the bullet list and the example console session to match the current prompts. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
1d4687e to
e21408e
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
dupschema on the superhub during import).__hoststable is keyed on hostkey (verified againstnova/db/schema.sql).distributed_cleanup.pyprompts in the bullet list and the example console session to match the current script: separate admin username/password prompts (defaulting toadmin), and 2FA prompts when enabled.Example output