Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: f3: upgrade to latest go-f3 #12325

Closed
wants to merge 5 commits into from
Closed

Conversation

masih
Copy link
Member

@masih masih commented Jul 30, 2024

Related Issues

None.

Proposed Changes

Upgrade to the latest F3 for caching, additional metrics and enhanced logging.

Additional Info

WIP: Being actively experimented on butterflynet
Current set to head of filecoin-project/go-f3#528

Checklist

Before you mark the PR ready for review, please make sure that:

  • Commits have a clear commit message.
  • PR title is in the form of of <PR type>: <area>: <change being made>
    • example: fix: mempool: Introduce a cache for valid signatures
    • PR type: fix, feat, build, chore, ci, docs, perf, refactor, revert, style, test
    • area, e.g. api, chain, state, mempool, multisig, networking, paych, proving, sealing, wallet, deps
  • Update CHANGELOG.md or signal that this change does not need it.
    • If the PR affects users (e.g., new feature, bug fix, system requirements change), update the CHANGELOG.md and add details to the UNRELEASED section.
    • If the change does not require a CHANGELOG.md entry, do one of the following:
      • Add [skip changelog] to the PR title
      • Add the label skip/changelog to the PR
  • New features have usage guidelines and / or documentation updates in
  • Tests exist for new functionality or change in behavior
  • CI is green

@masih masih force-pushed the masih/f3-dep-up-metrics branch from 823081c to 1c638a8 Compare August 5, 2024 11:39
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@masih masih changed the title Upgrade to latest f3 for additional metrics chore: f3: upgrade to latest go-f3 Aug 12, 2024
@github-actions github-actions bot dismissed their stale review August 12, 2024 19:42

PR title now matches the required format.

@Stebalien
Copy link
Member

Sorry, didn't see this PR. Merged #12382

@Stebalien Stebalien closed this Aug 13, 2024
@Stebalien Stebalien deleted the masih/f3-dep-up-metrics branch August 13, 2024 23:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants