Skip to content
Snippets Groups Projects

feat: --config parameter to specify picodata configuration file

Merged Georgy Moshkin requested to merge gmoshkin/config.yaml into master

Summary

  • Remove --init-cfg option: now it will say to use --config instead

  • fix: move tiers configuration to config.yaml

Change configuration file format:

  • before:
tier:
  storage:
    replication_factor: 2
  • after:
cluster:
  tiers:
    storage:
      replication-factor: 2
  • feat: try reading ./config.yaml by default

  • feat: validate configuration against storage (currently only instance_id)

  • feat: --config parameter to specify picodata configuration file

Also here we refactor all the places where we checked command line parameters, because now PicodataConfig is the main source of this information.


  • Close #...
  • Target branch: master
  • Changes should be cherry-picked to 24.1: yes / no
  • Docs follow-up: not necessary / new issue
Edited by Georgy Moshkin

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
Please register or sign in to reply
Loading