-
Notifications
You must be signed in to change notification settings - Fork 1.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Multistream specs #15603
base: develop
Are you sure you want to change the base?
Multistream specs #15603
Conversation
AER Report: CI Coreaer_workflow , commit , Detect Changes , Clean Go Tidy & Generate , Scheduled Run Frequency , Flakeguard Root Project / Get Tests To Run , lint , Core Tests (go_core_tests) , Flakeguard Deployment Project , Core Tests (go_core_tests_integration) , Core Tests (go_core_ccip_deployment_tests) , Core Tests (go_core_race_tests) , Core Tests (go_core_fuzz) , Flakeguard Root Project / Run Tests (github.com/smartcontractkit/chainlink/v2/core/services/llo, ubuntu-latest) , Flakeguard Root Project / Report , Flakey Test Detection , SonarQube Scan 1. MockTask does not implement Task:[Golang Lint] Source of Error:core/services/relay/evm/mercury/mocks/pipeline.go:20:23: cannot use &MockTask{} (value of type *MockTask) as "github.com/smartcontractkit/chainlink/v2/core/services/pipeline".Task value in variable declaration: *MockTask does not implement "github.com/smartcontractkit/chainlink/v2/core/services/pipeline".Task (missing method TaskStreamID) Suggested fix: Implement the 2. No such file or directory:[Golang Lint] Source of Error:cat: ./golangci-lint-report.xml: No such file or directory Suggested fix: Ensure that the linter runs successfully and generates the 3. Undefined: Stream:[Run tests] Source of Error:core/services/streams/delegate_test.go:18:53: undefined: Stream Suggested fix: Define the 4. strmSrv.id undefined:[Run tests] Source of Error:core/services/streams/delegate_test.go:52:42: strmSrv.id undefined (type *StreamService has no field or method id) Suggested fix: Add the 5. strmSrv.spec undefined:[Run tests] Source of Error:core/services/streams/delegate_test.go:53:45: strmSrv.spec undefined (type *StreamService has no field or method spec) Suggested fix: Add the 6. sr.streams undefined:[Run tests] Source of Error:core/services/streams/stream_registry_test.go:31:6: sr.streams undefined (type *streamRegistry has no field or method streams) Suggested fix: Add the 7. Too many errors:[Run tests] Source of Error:core/services/streams/stream_registry_test.go:55:21: too many errors Suggested fix: Fix the previous errors related to undefined fields/methods in the AER Report: Operator UI CI ran successfully ✅ |
f373e71
to
358a742
Compare
00ca59b
to
13192ab
Compare
} | ||
} | ||
|
||
func (s *streamRegistry) Get(streamID StreamID) (strm Stream, exists bool) { | ||
func (s *streamRegistry) Get(streamID StreamID) (p Pipeline, exists bool) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What's the relation between job ids, stream ids, and pipeline ids? Seems like job and stream are analogous, and then 1 pipeline can have many streams (which is a specific type of job). Is that correct?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah this is a bit confusing. A job
has exactly one pipeline_spec
and those have two different IDs (we almost exclusively use the jobID
to reference it though). stream_id
is a completely different thing. So a job has one pipeline which may contain many stream IDs.
Requires
Supports