Skip to content

ConsensusQueue: wire no-op ReportingPlugin - #23901

Draft
jmank88 wants to merge 1 commit into
developfrom
CRE-6002-consensus-queue-wiring
Draft

jmank88 wants to merge 1 commit into
developfrom
CRE-6002-consensus-queue-wiring

Conversation

@jmank88

@jmank88 jmank88 commented Oct 5, 2026 •

Copy link
Copy Markdown
Contributor

@jmank88
jmank88 requested review from MStreet3 and agparadiso October 5, 2026 13:13
@github-actions

github-actions Bot commented Oct 5, 2026

Copy link
Copy Markdown
Contributor

✅ No conflicts with other open PRs targeting develop

@github-actions

github-actions Bot commented Oct 5, 2026

Copy link
Copy Markdown
Contributor

I see you updated files related to core. Please run make gocs in the root directory to add a changeset as well as in the text include at least one of the following tags:

  • #added For any new functionality added.
  • #breaking_change For any functionality that requires manual action for the node to boot.
  • #bugfix For bug fixes.
  • #changed For any change to the existing functionality.
  • #db_update For any feature that introduces updates to database schema.
  • #deprecation_notice For any upcoming deprecation functionality.
  • #internal For changesets that need to be excluded from the final changelog.
  • #nops For any feature that is NOP facing and needs to be in the official Release Notes for the release.
  • #removed For any functionality/config that is removed.
  • #updated For any functionality that is updated.
  • #wip For any change that is not ready yet and external communication about it should be held off till it is feature complete.

@jmank88
jmank88 force-pushed the CRE-6002-consensus-queue-wiring branch from 78e9b52 to e04cd2d Compare October 5, 2026 13:18
@trunk-io

trunk-io Bot commented Oct 5, 2026 •

Copy link
Copy Markdown

Static Badge   Static Badge   Static Badge

View Full Report ↗︎ ⋅ Docs

@jmank88
jmank88 force-pushed the CRE-6002-consensus-queue-wiring branch from e04cd2d to 1c33ed0 Compare October 5, 2026 13:33
type contractTransmitter struct {
lggr logger.Logger
fromAccount ocrtypes.Account
eventSink triggers.EventSink

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.

so at the very least I would expect the ability to map a workflow ID to an event sink. we chose to have just the single engine registry and this is wrapped by the coordinator. If we wire as-is, we need a TODO to come back and address this dependency

@cl-sonarqube-production

Copy link
Copy Markdown

Quality Gate failed Quality Gate failed

Failed conditions
7 New Major Issues (required ≤ 5)
C Reliability Rating on New Code (required ≥ A)

See analysis details on SonarQube

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE SonarQube for IDE

@github-actions

github-actions Bot commented Oct 27, 2026 •

Copy link
Copy Markdown
Contributor

CORA - Analysis Skipped

Reason: The number of code owners (4) is less than the minimum required (5) and/or the number of CODEOWNERS entries with changed files (4) is less than the minimum required (2).

@jmank88
jmank88 force-pushed the CRE-6002-consensus-queue-wiring branch from 3d49d6a to ec7c4a4 Compare October 29, 2026 12:17
@jmank88 jmank88 changed the title add ContractTransmitter; wire Limits from job spec; write random data to KV; add validation case ConsensusQueue: wire no-op ReportingPlugin Oct 29, 2026
@jmank88
jmank88 added this pull request to stack #24103 October 29, 2026 12:20
return errors.New("could not find bootstrap node in topology, exactly one bootstrap node is required")
}

_, ocrPeeringCfg, err := cre.PeeringCfgs(bootstrap)

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.

Error: ../../../../system-tests/lib/cre/features/queue/queue.go:92:27: assignment mismatch: 3 variables but cre.PeeringCfgs returns 2 values

This branch has not been deployed

No deployments
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