Skip to content

chore: Add validator service file#1638

Merged
sergerad merged 8 commits intonextfrom
sergerad-validator-service
Feb 9, 2026
Merged

chore: Add validator service file#1638
sergerad merged 8 commits intonextfrom
sergerad-validator-service

Conversation

@sergerad
Copy link
Collaborator

@sergerad sergerad commented Feb 4, 2026

The Debian packaging automation needed to be updated to support multiple services in a single Debian package.

@sergerad sergerad added the no changelog This PR does not require an entry in the `CHANGELOG.md` file label Feb 4, 2026
Environment="OTEL_SERVICE_NAME=miden-validator"
EnvironmentFile=/lib/systemd/system/miden-validator.env
ExecStart=/usr/bin/miden-node validator start
WorkingDirectory=/opt/miden-node
Copy link
Collaborator

Choose a reason for hiding this comment

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

Bit of a meta question, but we should probably distinguish this from the node? In case both node + validator run on the same instance.

Suggested change
WorkingDirectory=/opt/miden-node
WorkingDirectory=/opt/miden-validator

A different way of doing things is to have common /opt/miden/ directory and then each service gets its own /opt/miden/<service> folder in there. But that gets dodgy..? Or maybe not.

Essentially, I think I recall that each "org" should have its own /opt directory. But I could be mistaken. I think part of the problem would then be that we probably need a miden user group to own the /opt/miden directory, which would then let the miden-node, miden-validator users act within that group. Or we just have a single miden user which handles all miden services.

Probably I'm overthinking things :)

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Seems reasonable but would opt to add validator in the current scheme we have and do a migration for all existing packages/services separately to this work

@sergerad
Copy link
Collaborator Author

sergerad commented Feb 9, 2026

Tested here https://github.com/0xMiden/miden-node/actions/runs/21810104637

will try run it on validator instance now

@sergerad sergerad merged commit 67b8170 into next Feb 9, 2026
41 of 45 checks passed
@sergerad sergerad deleted the sergerad-validator-service branch February 9, 2026 03:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no changelog This PR does not require an entry in the `CHANGELOG.md` file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants