fix: upload corresponding latest snapshot sha256sum

[samuelarogbonlo]

Summary of changes
Changes introduced in this pull request:

• upload corresponding latest snapshot sha256sum

Reference issue to close (if applicable)

part #370

Other information and links

[lemmih]

Let's enumerate the files to be uploaded rather than using a wildcard.

[samuelarogbonlo]

Let's enumerate the files to be uploaded rather than using a wildcard.

updated, @lemmih what do you think ?

[lemmih]

Let's enumerate the files to be uploaded rather than using a wildcard.

updated, @lemmih what do you think ?

You're still using a wildcard rather than enumerating the files. We only want to upload the archive and checksum files. Any other files in that directory should not be uploaded.

[LesnyRumcajs]

@lemmih Looking at https://github.com/ChainSafe/forest-iac/blob/main/cf/prune-latest/src/index.ts, it seems that adding corresponding checksum files would decrease the number of recent snapshots by half. Are we okay with that? Or should that worker be updated accordingly?

[lemmih]

@lemmih Looking at https://github.com/ChainSafe/forest-iac/blob/main/cf/prune-latest/src/index.ts, it seems that adding corresponding checksum files would decrease the number of recent snapshots by half. Are we okay with that? Or should that worker be updated accordingly?

If the Infra team is ready to take over then we don't have to worry about retention. Let's see what Josh at the meeting later today. If they need more time, we'll bump our retention (which is still quite a bit lower than required by the contract, iirc).

[LesnyRumcajs]

Forest snapshot service discontinued in favour of the infra snapshot service.