Skip to content
This repository has been archived by the owner on Jun 20, 2024. It is now read-only.

Send verification error to Caboose #184

Merged
merged 7 commits into from
Aug 8, 2023

Conversation

aarshkshah1992
Copy link
Collaborator

Send verification error to Caboose.

@aarshkshah1992
Copy link
Collaborator Author

Caboose PR at filecoin-saturn/caboose#141.

}
return blkRead.block, nil
}
return nil, fmt.Errorf("received a nil block with no error")
return nil, caboose.ErrInvalidResponse{
Message: fmt.Sprintf("received a nil block with no error"),
Copy link
Collaborator

Choose a reason for hiding this comment

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

make this a constant error?

@aarshkshah1992
Copy link
Collaborator Author

Failing to build because of Error: ../../../go/pkg/mod/go.opentelemetry.io/otel/exporters/otlp/[email protected]/internal/otlpconfig/options.go:100:62: undefined: internal.GetUserAgentHeader.

@willscott
Copy link
Collaborator

@aarshkshah1992 - pull in the staging branch - it's got the fix for dependencies / current otel versions

@aarshkshah1992
Copy link
Collaborator Author

@willscott You mean the Bifrost staging version or Caboose staging version ?

@willscott
Copy link
Collaborator

i mean 06077bc

@aarshkshah1992
Copy link
Collaborator Author

Confirmed in staging that this works. Saturn is now getting verification errors from Caboose.

@aarshkshah1992 aarshkshah1992 merged commit 0bbc0c4 into feat/backpressure Aug 8, 2023
28 checks passed
@aarshkshah1992 aarshkshah1992 deleted the feat/verification-error branch August 8, 2023 13:17
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants