-
Notifications
You must be signed in to change notification settings - Fork 2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Quality of life improvements #3
Merged
GMW99
merged 36 commits into
DiamondLightSource:master
from
rosalindfranklininstitute:master
Feb 4, 2022
Merged
Quality of life improvements #3
GMW99
merged 36 commits into
DiamondLightSource:master
from
rosalindfranklininstitute:master
Feb 4, 2022
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Removed the absolute to the ip command, to just be ip.:
Added work around for the edge case where the ip route return the wrong information this is now more specific command
Monitor update
Added an option to specify block type through create-osd, remove-osd.
Helpmsg now provides information about the type of RAM block devices that can be used.
This loads the brd block device and creates the relevant pv's
This loads the zram block device and creates the relevant pv's
Now only creates the pv's
This creats the block devices according to the type specified gram, ram, zram then creates the pv's and then the OSDs
File no longer need now that distrac is more generic
No requires the type of block device to specified, then remove accordingly
Fixed issue where rgw would run even when not flagged to do so
The example now include the -t flag so that they work with the new update.
Added the documentation for the addition of the BRD and ZRAM RAM BLOCK devices.
…upport Additional ram support
This adds the zram type to the allowed list within LVM - thus allowing for the creation of PV's
Removes the change that allowed for ZRAM to have PV's
Now uses the LVM config change so PV's can be made
This keeps the LVM file clean.
This change is superficial however increases maintainability.
…lumes Zram physical volumes
Removed rm command to make it safer also reduces steps of recreating the directory
removed rm and mkdir and a yes | command
Removed uneeded sudo
This remove the need for a user to have full root access and instead allows for the creation of a distrac user that can run the required commands.
The readme has been updated to reflect the changes made.
This log.conf contains all the logging and debuging configuration it is all set to 0/0 and /dev/null such that nothing is collected.
Appendeds log.conf to the default config created during mon creation
Stopping log creation
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Added the following improvements