Skip to content

Upgrade to contra-tracer 0.2.1#5368

Open
f-f wants to merge 1 commit into
mainfrom
f-f/contra-tracer-0.2
Open

Upgrade to contra-tracer 0.2.1#5368
f-f wants to merge 1 commit into
mainfrom
f-f/contra-tracer-0.2

Conversation

@f-f
Copy link
Copy Markdown

@f-f f-f commented May 13, 2026

Description

As part of the integration for 11.1, we'll be removing the legacy tracing iohk-monitoring-framework - that needs contra-tracer >= 0.2.1, which this patch implements.

Notes:

  • this is a starting point that compiles so I can integrate it downstream, @IntersectMBO/ouroboros-network-maintainers please feel free to take over if it's the wrong approach
  • I had to add an SRP to the git repo because the cabal.project file is set to override hackage packages with the CHaP ones, and we already have a contra-tracer on CHaP as well, set to an earlier version. I am not sure what a better approach would be, so I'd be happy to hear suggestions on this (also for future occurrences)

Checklist

Quality

  • Commit sequence makes sense and have useful messages, see ref.
  • New tests are added and existing tests are updated.
  • Self-reviewed the PR.

Maintenance

  • Linked an issue or added the PR to the current sprint of ouroboros-network project.
  • Added labels.
  • Updated changelog files.
  • The documentation has been properly updated, see ref.

@f-f f-f requested a review from a team as a code owner May 13, 2026 17:01
@github-project-automation github-project-automation Bot moved this to In Progress in Ouroboros Network May 13, 2026
Copy link
Copy Markdown
Collaborator

@coot coot left a comment

Choose a reason for hiding this comment

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

It LGTM, you just need to fix warnings since Contra.Tracer re-exports the >$<, and you'll also need to add QuickCheck < 2.18 constraints, since it renamed/deprecated a function we use.

@f-f f-f force-pushed the f-f/contra-tracer-0.2 branch from bdacb91 to 2f6836a Compare May 15, 2026 14:23
@f-f f-f requested a review from a team as a code owner May 15, 2026 14:23
@f-f f-f force-pushed the f-f/contra-tracer-0.2 branch from 2f6836a to 9e8ce37 Compare May 15, 2026 14:29
@f-f
Copy link
Copy Markdown
Author

f-f commented May 15, 2026

@coot thanks! I have applied the changes but I am getting a few test failures, could you have a look at those?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: In Progress

Development

Successfully merging this pull request may close these issues.

2 participants