action #51467

[functional][u] Change all occurrences of /.*_run.[' "]zypper.*/ to zypper_call

Added by SLindoMansilla 10 months ago. Updated 8 months ago.

Status:ResolvedStart date:14/05/2019
Priority:HighDue date:
Assignee:dheidler% Done:

0%

Category:Enhancement to existing tests
Target version:SUSE QA tests - Milestone 25
Difficulty:
Duration:

Description

Motivation

As discussed in the retro, to avoid problems like the wrong handling of new zypper feature "search-packages", which prompts the user to ask if a search through non-enabled modules should be done, we should always use zypper_call, so that all possible cases are handled on only one function.

Acceptance criteria

  • AC1: There are no occurrences of /.*_run.[' "]zypper.*/
  • AC2: zypper_call can handle serial console.

Related issues

Copied to openQA Tests - action #54002: [functional][u] Change even more occurrences of /.*_run.[... Feedback 14/05/2019

History

#1 Updated by SLindoMansilla 9 months ago

  • Status changed from New to Workable
  • Target version set to Milestone 25

As discussed in the grooming meeting

#2 Updated by dheidler 9 months ago

  • Status changed from Workable to In Progress
  • Assignee set to dheidler

#3 Updated by dheidler 8 months ago

  • Status changed from In Progress to Feedback

#4 Updated by okurz 8 months ago

PR merged, seems to fail in some instances, e.g. https://openqa.opensuse.org/tests/962346#step/zypper_clear_repos/13

#6 Updated by okurz 8 months ago

https://github.com/os-autoinst/os-autoinst-distri-opensuse/pull/7712 merged, zypper_clear_repos would be the next candidate :)

#8 Updated by SLindoMansilla 8 months ago

  • Status changed from Feedback to Workable

PR#7715 was merged.

#9 Updated by dheidler 8 months ago

  • Status changed from Workable to Resolved

#10 Updated by dheidler 8 months ago

  • Copied to action #54002: [functional][u] Change even more occurrences of /.*_run.[' "]zypper.*/ to zypper_call added

Also available in: Atom PDF