Project

General

Profile

Actions

action #77251

closed

Rename scenario cryptlvm+activate_existing and remove reference to fate.suse.com/321208

Added by JERiveraMoya over 3 years ago. Updated about 3 years ago.

Status:
Closed
Priority:
Low
Assignee:
Target version:
SUSE QA - SLE 15 SP3
Start date:
2020-11-10
Due date:
% Done:

0%

Estimated time:

Description

We should find a better name for this scenario, as we don't reuse partitions in any of the scenarios, so we don't care if it is lvm or not, which could bring confusion about what was the original purpose of it (without checking the bugs and feature mentioned on it). For this one in particular we just check encrypted and unlock an specific disk. After unlocked we install the system without reusing the partition. For PowerVM we don't even have an image created with lvm encrypted system, we just encrypt the partition at the beginning or the scenario. We should reflect only the activating and the encryption in the name.

In boot_encrypt, we have:

     if (get_var('ENCRYPT_ACTIVATE_EXISTING') and !get_var('ENCRYPT_FORCE_RECOMPUTE')) {
         record_info 'bsc#993247 https://fate.suse.com/321208', 'activated encrypted partition will not be recreated as encrypted';
         return;
     }

which we should remove and remove module from the schedule when we do not expect encrypted partitions.
Please, check if module is used for older versions, and also adjust schedule for openSUSE distris.

Acceptance criteria

  1. There are no references to https://fate.suse.com/321208
  2. Come up with a new name for the test suite which reflects it's purpose better and discuss it within the team
Actions #1

Updated by JERiveraMoya over 3 years ago

  • Description updated (diff)
Actions #2

Updated by riafarov over 3 years ago

  • Priority changed from Normal to Low
  • Target version set to future
Actions #3

Updated by riafarov over 3 years ago

  • Subject changed from Rename scenario cryptlvm+activate_existing to Rename scenario cryptlvm+activate_existing and remove reference to fate.suse.com/321208
  • Description updated (diff)
  • Status changed from New to Workable
Actions #4

Updated by JRivrain over 3 years ago

  • Status changed from Workable to In Progress
  • Assignee set to JRivrain
Actions #5

Updated by JRivrain over 3 years ago

  • Status changed from In Progress to Feedback

PR merged https://github.com/os-autoinst/os-autoinst-distri-opensuse/pull/11716/files. Still some issue to solve with the job groups, we cannot delete the test suites "cryptlvm+activate_sexisting" because of messages like "Groups Functional, Staging: SLE 12, Staging: SLE 15 must be updated through the YAML template" however those do not contain cryptlvm+activate* testsuites in it. Looks like a bug.

Actions #6

Updated by JRivrain over 3 years ago

  • Status changed from Feedback to Closed

I managed to remove the old test suites, and all passed latest build.

Actions #7

Updated by riafarov about 3 years ago

  • Target version changed from future to SLE 15 SP3
Actions

Also available in: Atom PDF