Skip to content

feat: add RMN deployment changeset#2008

Open
asoliman92 wants to merge 7 commits intoasoliman/rmn-curse-adminsfrom
asoliman/new-rmn-deployment
Open

feat: add RMN deployment changeset#2008
asoliman92 wants to merge 7 commits intoasoliman/rmn-curse-adminsfrom
asoliman/new-rmn-deployment

Conversation

@asoliman92
Copy link
Copy Markdown
Contributor

No description provided.

@asoliman92 asoliman92 force-pushed the asoliman/new-rmn-deployment branch from 8b80c61 to d45c9a7 Compare April 28, 2026 04:10
Comment thread chains/evm/deployment/operations_gen_config.yaml Outdated
@asoliman92 asoliman92 marked this pull request as ready for review April 28, 2026 04:15
@asoliman92 asoliman92 requested review from a team as code owners April 28, 2026 04:15
@asoliman92 asoliman92 force-pushed the asoliman/new-rmn-deployment branch from 166fae2 to af68afa Compare April 28, 2026 04:21
@github-actions
Copy link
Copy Markdown

Metric asoliman/new-rmn-deployment asoliman/rmn-curse-admins
Coverage 70.1% 69.8%

}

// CurseRMN curses one or more subjects on an RMN v2.1.0 contract.
var CurseRMN = changesets.NewFromOnChainSequence(changesets.NewFromOnChainSequenceParams[
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

return c.ChainSel
}

var DeployRMN = changesets.NewFromOnChainSequence(changesets.NewFromOnChainSequenceParams[
Copy link
Copy Markdown
Contributor

@carte7000 carte7000 Apr 29, 2026

Choose a reason for hiding this comment

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

We might want to have this changeset be part of chainlink-ccip/deployment and create an adapter so the deployment of RMN and the CurseAdmin management CS are working across family. cc: @tt-cll, @AnieeG

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I don't think it needs to be part of this PR yet though. The adapter will look very similarly to what we do in evm with some type changes that we can do later.

ChainSel uint64
// RMN is an address ref used to look up the deployed RMN contract in the datastore.
RMN datastore.AddressRef
Args rmnops.AuthorizedCallerArgs
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

If we move this to an adapter we will need to make that field chain agnostic. i.e decouple from any go-ethereum types

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