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
Extend documentation for lib/partition_setup.pm #8674
Conversation
lib/partition_setup.pm
Outdated
@@ -209,7 +210,7 @@ sub set_partition_size { | |||
|
|||
resize_partition(); | |||
|
|||
Method assumes that correct disk is already selected | |||
Method assumes that correct disk is already selected. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Method assumes that correct disk is already selected. | |
Method assumes that correct disk is already selected. |
lib/partition_setup.pm
Outdated
@@ -396,9 +400,12 @@ sub addlv { | |||
|
|||
=head2 addboot | |||
|
|||
addboot(); | |||
addboot($part_size, [%default_boot_sizes]); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
addboot($part_size, [%default_boot_sizes]); | |
addboot($part_size); |
%default_boot_sizes
is not a parameter.
lib/partition_setup.pm
Outdated
|
||
Adds a boot partition, calls C<addpart> for this | ||
C<$part_size> is the size of partition. | ||
C<%default_boot_sizes> is pre-defined default boot size. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
C<%default_boot_sizes> is pre-defined default boot size. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this still there?
%default_boot_sizes
is not a parameter.
lib/partition_setup.pm
Outdated
|
||
Example: | ||
|
||
take_first_disk iscsi => 1; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
take_first_disk iscsi => 1; | |
take_first_disk(iscsi => 1); |
lib/partition_setup.pm
Outdated
|
||
Selects the first disk in the list to be partitioned | ||
Take the first disk to be partitioned. Take first storage ng if it is C<is_storage_ng>. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you rephrase that?
I don't really understand what you want to say in the 2nd sentence.
updated now |
lib/partition_setup.pm
Outdated
Deletes all existing partitions in the expert partitioner | ||
Despite the name it does not check if it is run on a storage ng system | ||
Deletes all existing partitions in the expert partitioner. | ||
Despite the name it does not check if it is run on a storage ng system, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Despite the name it does not check if it is run on a storage ng system, | |
Despite the name it does not check if it is run on a storage ng system, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
trailing space
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
removed now
lib/partition_setup.pm
Outdated
|
||
Adds a boot partition, calls C<addpart> for this | ||
C<$part_size> is the size of partition. | ||
C<%default_boot_sizes> is pre-defined default boot size. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this still there?
%default_boot_sizes
is not a parameter.
see https://progress.opensuse.org/issues/57917