Skip to content

Conversation

tdammers
Copy link
Contributor

@tdammers tdammers commented Feb 20, 2025

This adds KES Agent connectivity to consensus.

To use a KES Agent to source KES SignKeys and OpCerts, the praosCredentialsSource in the PraosCanBeLeader data structure can now be pointed to a domain socket address where it will look for a KES Agent.

Also covers #1077.

@@ -262,20 +265,38 @@ data PraosCanBeLeader c = PraosCanBeLeader

data PraosCredentialsSource c
= PraosCredentialsUnsound (OCert.OCert c) (KES.UnsoundPureSignKeyKES (KES c))
| PraosCredentialsAgent FilePath
Copy link
Contributor

Choose a reason for hiding this comment

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

I'm unsure if this one should use FsPath instead? cc @jorisdral

@tdammers tdammers force-pushed the tdammers/kes-agent-connectivity branch from c418528 to 42ed7a9 Compare March 12, 2025 10:42
@tdammers tdammers force-pushed the tdammers/kes-agent-connectivity branch from 42ed7a9 to d2365a8 Compare March 12, 2025 11:26
@jasagredo
Copy link
Contributor

The use of serdoc-core prevents us from using GHC 9.12.2:

Configuration is affected by the following files:
- cabal.project
- cabal.project.local
Resolving dependencies...
Error: [Cabal-7107]
Could not resolve dependencies:
[__0] trying: kes-agent-0.1.0.0 (user goal)
[__1] trying: template-haskell-2.23.0.0/installed-2522 (dependency of kes-agent)
[__2] next goal: serdoc-core (dependency of kes-agent)
[__2] rejecting: serdoc-core-0.3.0.0 (conflict: template-haskell==2.23.0.0/installed-2522, serdoc-core => template-haskell>=2.16 && <2.23)
[__2] skipping: serdoc-core; 0.2.0.0, 0.1.0.0 (has the same characteristics that caused the previous version to fail: excludes 'template-haskell' version 2.23.0.0)
[__2] fail (backjumping, conflict set: kes-agent, serdoc-core, template-haskell)
After searching the rest of the dependency tree exhaustively, these were the goals I've had most trouble fulfilling: template-haskell, serdoc-core, kes-agent

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

3 participants