Skip to content

fix(solana): use smaller batch size for the "appendSigner" instructions#381

Merged
gustavogama-cll merged 1 commit intomainfrom
ggama/fix-use-smaller-batch-size-append-signers
May 14, 2025
Merged

fix(solana): use smaller batch size for the "appendSigner" instructions#381
gustavogama-cll merged 1 commit intomainfrom
ggama/fix-use-smaller-batch-size-append-signers

Conversation

@gustavogama-cll
Copy link
Contributor

@gustavogama-cll gustavogama-cll commented May 14, 2025

The original value (45) has caused issues in mainnet:
https://chainlink-core.slack.com/archives/C085YEEAEUD/p1747251207380869

We're reducing to 20 out of caution and will adjust again once we're able to implement proper chunking tests.

@changeset-bot
Copy link

changeset-bot bot commented May 14, 2025

🦋 Changeset detected

Latest commit: 94cae51

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@smartcontractkit/mcms Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@gustavogama-cll gustavogama-cll force-pushed the ggama/fix-use-smaller-batch-size-append-signers branch from 19ce3f5 to 94cae51 Compare May 14, 2025 19:53
@cl-sonarqube-production
Copy link

@gustavogama-cll gustavogama-cll marked this pull request as ready for review May 14, 2025 19:59
@gustavogama-cll gustavogama-cll requested a review from a team as a code owner May 14, 2025 19:59
@gustavogama-cll gustavogama-cll enabled auto-merge May 14, 2025 19:59
@gustavogama-cll gustavogama-cll added this pull request to the merge queue May 14, 2025
Merged via the queue into main with commit 0b3000a May 14, 2025
11 checks passed
@gustavogama-cll gustavogama-cll deleted the ggama/fix-use-smaller-batch-size-append-signers branch May 14, 2025 20:09
github-merge-queue bot pushed a commit that referenced this pull request May 14, 2025
This PR was opened by the [Changesets
release](https://github.com/changesets/action) GitHub action. When
you're ready to do a release, you can merge this and the packages will
be published to npm automatically. If you're not ready to do a release
yet, that's fine, whenever you add more changesets to main, this PR will
be updated.


# Releases
## @smartcontractkit/mcms@0.19.3

### Patch Changes

- [#384](#384)
[`80e82ad`](80e82ad)
Thanks [@gustavogama-cll](https://github.com/gustavogama-cll)! -
fix(solana): use smaller batch "append instruction" chunk size

- [#381](#381)
[`0b3000a`](0b3000a)
Thanks [@gustavogama-cll](https://github.com/gustavogama-cll)! -
fix(solana): use smaller batch size for the "appendSigner" instructions

Co-authored-by: app-token-issuer-engops[bot] <144731339+app-token-issuer-engops[bot]@users.noreply.github.com>
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.

3 participants