[MBUILDCACHE-71] buildinfo.xml should be stored after storing the project's artifacts

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.0.1
• Fix Version/s: 1.2.0
• Labels: pull-request-available
• Assignee: Olivier Lamy
• Reporter: Amir Hadadi
• Created: 01/Sep/23 11:59
• Updated: 09/Apr/24 06:20
• Resolved: 29/Mar/24 09:05

Description

In certain failure cases it's possible that only part of a module artifacts get stored in the local and remote storage. In that case if buildinfo.xml got stored, the extension will later incorrectly attempt to restore the partially cached build.

Because buildinfo.xml is used as an indication that the cached artifacts exist, I propose to reorder the logic in 

CacheControllerImpl.save()

 so that 

localCache.saveBuildInfo(cacheResult, build);

will happen after the project's artifacts are stored.

Issue Links

• links to: GitHub Pull Request #138