Skip to content

Added include invalid pending queue items for the vault#2123

Open
russell-stern wants to merge 11 commits into
mainfrom
validate_observations
Open

Added include invalid pending queue items for the vault#2123
russell-stern wants to merge 11 commits into
mainfrom
validate_observations

Conversation

@russell-stern

Copy link
Copy Markdown
Contributor

No description provided.

@github-actions

github-actions Bot commented Jun 3, 2026

Copy link
Copy Markdown
Contributor

✅ API Diff Results - github.com/smartcontractkit/chainlink-common

✅ Compatible Changes (5)

pkg/capabilities/actions/vault (1)
  • ObservationError — ➕ Added
pkg/capabilities/actions/vault.(*Observation) (1)
  • GetError — ➕ Added
pkg/capabilities/actions/vault.Observation (1)
  • Error — ➕ Added
pkg/settings/cresettings.Schema (2)
  • VaultGetSecretsRelaxedConsensusEnabled — ➕ Added

  • VaultSkipInvalidPendingItemsEnabled — ➕ Added


📄 View full apidiff report

@russell-stern russell-stern force-pushed the validate_observations branch from 4292fef to 256a23f Compare June 12, 2026 19:33
@russell-stern russell-stern marked this pull request as ready for review June 15, 2026 12:48
@russell-stern russell-stern requested a review from a team as a code owner June 15, 2026 12:48
@russell-stern russell-stern force-pushed the validate_observations branch from 5ce797f to b7f9746 Compare June 29, 2026 12:34
@russell-stern russell-stern force-pushed the validate_observations branch from 16329fd to 7e603d9 Compare July 2, 2026 18:29

// ObservationError is an item-level Err(E) contribution for a pending queue request.
// When set on Observation, the observation is an error contribution for that request id.
message ObservationError {

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.

any reason this is a separate proto and not inlined?

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.

Mostly for extensibility later on in case we want to add codes/types

@russell-stern russell-stern changed the title Added cipherless observation flag for the vault Added include invalid pending queue items for the vault Jul 8, 2026
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.

4 participants