feat: add CRE enabled flag domain configuration#861
Conversation
🦋 Changeset detectedLatest commit: 445b256 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
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 |
There was a problem hiding this comment.
Pull request overview
Adds a new optional CRE configuration block to domain.yaml so domains can opt in (via an enabled flag) without breaking existing configs.
Changes:
- Introduces a
CREconfig struct with anenabledflag. - Adds an optional
crefield to eachEnvironmentin the domain config. - Adds a changeset bumping
chainlink-deployments-frameworkminor for the new config surface.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| engine/cld/config/domain/domain.go | Adds the new optional cre environment configuration type/field. |
| .changeset/nasty-needles-bathe.md | Records a minor version bump for the new domain config fields. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
You can also share your feedback on Copilot code review. Take the survey.
| // Environment represents a single environment configuration. | ||
| type Environment struct { | ||
| NetworkTypes []string `mapstructure:"network_types" yaml:"network_types"` | ||
| Datastore DatastoreType `mapstructure:"datastore" yaml:"datastore"` | ||
| CRE *CRE `mapstructure:"cre" yaml:"cre,omitempty"` | ||
| } |
|
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 ## chainlink-deployments-framework@0.88.0 ### Minor Changes - [#861](#861) [`ec0bb20`](ec0bb20) Thanks [@ecPablo](https://github.com/ecPablo)! - add CRE configuration fields to domain config --------- Co-authored-by: app-token-issuer-engops[bot] <144731339+app-token-issuer-engops[bot]@users.noreply.github.com>
Adds a new config struct to the
domain.yamlto allow users to opt in on the installation of the CRE CLI in their domain. I left it as a struct even if just has 1 fields to allow it to easily add new configs as we get more usage in the future. Using a pointer (*CRE) makes this a non-breaking change to prevent domains requiring to add this field.