ceph: add initial ceph support. reading and writing m/ceph.conf #9

Merged
amery merged 13 commits from pr-amery-ceph into main 1 year ago
amery commented 1 year ago
Owner
There is no content yet.
amery added 13 commits 1 year ago
a8849b747c vscode: add ceph to the dictionary
0fb8c1d44b zones: introduce Zones.CephFSID and Zones.GetCephFSID()
025b9072b4 zones: introduce Machine.CephMonitor field
226d529223 zones: introduce Zone.GetCephMonitors()
c083a144ea zones: introduce GenCephConfig()
d8657770d5 zones: store ceph FSID on scan
cc7526ceb5 zones: extend scan to ensure every zone has a ceph monitor
75dee617ef env: set ceph monitors variables
4200db12cb jpictl: write m/ceph.conf on sync
amery requested review from karasz 1 year ago
karasz requested changes 1 year ago
karasz left a comment
Owner

Fine and dandy but segfaults if ceph.conf is not in the root of the machines. Please allow for ceph.conf to be in the proper place.

Fine and dandy but segfaults if ceph.conf is not in the root of the machines. Please allow for ceph.conf to be in the proper place.
karasz reviewed 1 year ago
}
// revive:disable:cyclomatic
// revive:disable:cognitive-complexity
karasz commented 1 year ago
Owner

Please also add // TODO: or // FIXME: explaining why this is needed

Please also add // TODO: or // FIXME: explaining why this is needed
karasz marked this conversation as resolved
amery added 10 commits 1 year ago
e3ab931eb1 zones: introduce Zone.GetCephMonitors()
bf4bfeb3fc zones: introduce GenCephConfig()
b772ec0a3d zones: store ceph FSID on scan
cea8362fe6 zones: extend scan to ensure every zone has a ceph monitor
fe081a4297 env: set ceph monitors variables
f10ea1dc22 jpictl: write m/ceph.conf on sync
Poster
Owner

bug fixed and // TODO: added to the complex parser function

bug fixed and `// TODO:` added to the complex parser function
amery requested review from karasz 1 year ago
karasz approved these changes 1 year ago
karasz left a comment
Owner

LGTMN

LGTMN
amery merged commit 1fe1cf940d into main 1 year ago
amery deleted branch pr-amery-ceph 1 year ago
Poster
Owner

v0.5.0 tagged

`v0.5.0` tagged

Reviewers

karasz approved these changes 1 year ago
The pull request has been merged as 1fe1cf940d.
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Loading…
There is no content yet.