|
In the non-HA case I can't see this being an issue, since the only requirement we place on report hashes is that they be unique. We don't use them for diffing like factsets and catalogs.
In the HA case the concern is that pre-existing PuppetDB A syncs to new PuppetDB B, and the two end up with identical reports that are hashed differently.
I think we may have the same issue with factsets (now that timestamps have been removed from the hash) and catalogs (now that we use certname instead of name). These are more transient because we don't store historical data, but could cause confusion during upgrade.
|