Skip to content

siad -M flag updates

Matthew Sevey requested to merge sevey/siad-detect-accounting into master

MERGE REQUEST

MR Guidelines

Overview

Stemming from this discussion, this MR makes 2 updates to the siad -M modules flag.

  1. It adds parsing of module names, so users can do siad -M renter now. This helps prevent issues from users forgetting a module.
  2. It automatically enables accounting and feemanager if the wallet is enabled. This is to again protect users from forgetting modules that are not required but provided useful information/utility.

Example for Visual Changes

siad modules output
https://siasky.net/AABzHfVwlwJqrhlOCqG8A0TOMrvQtogYg1-OZNt8y_gvjw

Checklist

Review and complete the checklist to ensure that the MR is complete before assigned to an approver.

  • All new methods or updated methods have clear docstrings
  • Testing added or updated for new methods
  • Any new packages are added to Makefile and .gitlab-ci.yml
  • API documentation updated for API updates
  • Module README.md updated for changes to workflow
  • Issue added to Sia-UI repo for new supporting features
  • Changelog File Created

Issues Closed

Part of #4516 (closed)

Edited by Matthew Sevey

Merge request reports