Increase partition count and test for swap #15

Merged
tcooper merged 1 commits from fix_issue_11 into develop 2021-08-22 01:39:48 +00:00
tcooper commented 2021-08-20 04:29:16 +00:00 (Migrated from github.com)

Description

Updates partition count in tests/disk_custom_standard_partition_ext4_postinstall.pm to account for Rocky default partitioning which includes swap partition and adds postinstall test to verify swap is active.

Closes #11

How Has This Been Tested?

  • POST of test for dvd-iso flavor with pass of disk_custom_standard_partition_ext4_postinstall test suite.

disk_custom_standard_partition_ext4_postinstall

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • Any dependent changes have been merged and published in downstream modules
## Description Updates partition count in `tests/disk_custom_standard_partition_ext4_postinstall.pm` to account for Rocky default partitioning which includes `swap` partition and adds postinstall test to verify `swap` is active. Closes #11 ## How Has This Been Tested? - [x] POST of test for dvd-iso flavor with pass of `disk_custom_standard_partition_ext4_postinstall` test suite. ![disk_custom_standard_partition_ext4_postinstall](https://user-images.githubusercontent.com/542846/130179536-120e80ef-ae78-4ba6-b27b-623bd019c96d.png) ## Checklist: - [x] My code follows the style guidelines of this project - [x] I have performed a self-review of my own code - [x] I have commented my code, particularly in hard-to-understand areas - [ ] I have made corresponding changes to the documentation - [x] My changes generate no new warnings - [ ] Any dependent changes have been merged and published in downstream modules
tcooper commented 2021-08-20 04:32:48 +00:00 (Migrated from github.com)

@StackKorora since develop is not the default branch for this repo auto-close of related issue on merge doesn't work. Please close issue #11 manually after merge. I'll check with @brianclemens about getting perms and/or have him set default branch to develop for this repo.

@StackKorora since `develop` is not the default branch for this repo auto-close of related issue on merge doesn't work. Please close issue #11 manually after merge. I'll check with @brianclemens about getting perms and/or have him set default branch to `develop` for this repo.
brianclemens commented 2021-08-20 07:58:39 +00:00 (Migrated from github.com)

@tcooper Default branch updated 👌

@tcooper Default branch updated 👌
lumarel commented 2021-08-20 16:16:58 +00:00 (Migrated from github.com)

Just tested this patch for short, and can confirm that it works for both paths (bios and uefi) 👌🏻

Just tested this patch for short, and can confirm that it works for both paths (bios and uefi) 👌🏻
Sign in to join this conversation.
No reviewers
No Milestone
No project
No Assignees
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: testing/os-autoinst-distri-rocky-migrated#15
No description provided.