Change Details

Due to the way we store revision metadata for package loader. A snapshot can reference a previous revision with metadata that are not nixguix related. Thus, the expected "integrity" (from the nixguix loader point of view) field is not found. Which makes the loader crash. This diff allows the loader to continue its work without crashing by considering such revision to be fetched again. pros: - the loader works cons: - redo all the ingestion for noop in the storage side Note: - missing tests Related to T2371 Related to T2411

This eventually catch any malformed revision (from a metadata standpoint) which would made the loader crash (when it tries to resolve the known artifacts). This catches the error, logs information about it and then continue on loading. Related to T2371

Due to the way we storeThis eventually catch any malformed revision metadata for package loader. A snapshot can(from a metadata standpoint) which reference a previous revision with metadata that are not nixguix related. Thus, the expected "integrity" (from the nixguix loader point of view) field is not found. Which makes the loader crashwould made the loader crash (when it tries to resolve the known artifacts). This diff allows the loader to continue its work without crashing by considering such revision to be fetched again. pros: - the loader works cons: - redo all the ingestion for noop in the storage side Note: - missing testscatches the error, logs information about it and then continue on loading. Related to T2371 Related to T2411