Project

General

Profile

Actions

action #40067

closed

coordination #39581: [fate#319428][functional][epic][y] Adapt yast module tests to new ServiceWidget

[fate#319428][functional][y] Adapt dns yast module tests to new ServiceWidget

Added by riafarov over 6 years ago. Updated about 6 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
Category:
Bugs in existing tests
Target version:
SUSE QA (private) - Milestone 19
Start date:
2018-08-10
Due date:
2018-09-25
% Done:

0%

Estimated time:
3.00 h
Difficulty:

Description

Motivation

We got new service widget for SLE15 (see fate#319428 https://fate.suse.com/319428 ).
This PR demonstrates the difference: https://github.com/yast/yast-dhcp-server/pull/26

Here is trello card: https://trello.com/c/uAe4i9Ru/107-5-fate319428-allow-socket-activation-widget

Acceptance criteria

  1. yast2 dns test modules are adjusted to the service widget changes in SLE/Leap15 and TW

Related issues 1 (0 open1 closed)

Has duplicate openQA Tests (public) - action #40325: [functional][y] Adapt yast2_samba to the new service widgetRejectedJERiveraMoya2018-08-282018-10-09

Actions
Actions #1

Updated by riafarov over 6 years ago

  • Copied from action #39725: [fate#319428][functional][y] Adapt http yast module tests to new ServiceWidget added
Actions #2

Updated by riafarov over 6 years ago

  • Copied from deleted (action #39725: [fate#319428][functional][y] Adapt http yast module tests to new ServiceWidget)
Actions #3

Updated by riafarov over 6 years ago

  • Parent task set to #39581
Actions #4

Updated by okurz over 6 years ago

  • Status changed from New to Workable
  • Target version changed from Milestone 18 to Milestone 19
Actions #5

Updated by riafarov over 6 years ago

  • Assignee set to JERiveraMoya
Actions #6

Updated by JERiveraMoya over 6 years ago

  • Status changed from Workable to In Progress
Actions #7

Updated by JERiveraMoya over 6 years ago

  • Status changed from In Progress to Feedback

For Leap we need: PR - Fix yast2_tftp for Leap - no sevice conf. widget yet and once is merged, file a bug due to missing directives as we cannot reach yet yast2_dns: http://dhcp87.suse.cz/tests/2407#step/yast2_samba/93

Actions #8

Updated by JERiveraMoya over 6 years ago

  • Status changed from Feedback to In Progress

This error is the first time I see it: https://openqa.opensuse.org/tests/746095#step/yast2_samba/51. It happens before the expected bug. It is failing going to the tab.

Actions #9

Updated by JERiveraMoya over 6 years ago

Filed bug about missing directives in SLE 15 SP1 and Leap occurring in yast2_samba.
Leap just got the service widget: PR - service widget to yast2_tftp in Leap

Actions #10

Updated by JERiveraMoya over 6 years ago

Recap:

Actions #11

Updated by JERiveraMoya over 6 years ago

Setting up Multi-machine env in my laptop in order to integrate widget in iscsi_server and iscsi_client tests.

Actions #12

Updated by JERiveraMoya over 6 years ago

  • Status changed from In Progress to Feedback

Leap-15.1-yast2_ncurses-yast2_samba fails in bug - Missing smb.conf directives
MM setup in my local env seems to be up and running (although not always, sporadic network issues found), enough to get verification run.
PR - Add service widget for iscsi

Actions #14

Updated by JERiveraMoya about 6 years ago

VR:

Actions #15

Updated by riafarov about 6 years ago

  • Due date changed from 2018-09-04 to 2018-09-25

VR still in progress

Actions #16

Updated by JERiveraMoya about 6 years ago

  • Status changed from Feedback to In Progress

Lesson learn: we need for next time some variable at the level of medium that we can change in OSD to determine which YaST modules got the change, i.e: Y2_MODULES_NEW_SERVICE_WIDGET=samba,dns, assign those values to each YaST module test so we can match it. It would require much less PRs to adapt our tests.

PR - Add service widget for samba in TW

Actions #17

Updated by JERiveraMoya about 6 years ago

  • Status changed from In Progress to Feedback
Actions #18

Updated by JERiveraMoya about 6 years ago

This PR is still not merged (I pointed to a possible idea for next time to implement another widget but for this one we can do it by code due to almost all work is done), it was just some reflexion, not nothing to avoid to merge it :) merged! :)
Still pending on merging PR - Add service widget for iscsi
Samba problem also in TW PR - Add soft-failure samba conf for TW

Actions #19

Updated by JERiveraMoya about 6 years ago

  • Status changed from Feedback to In Progress

VR:

Actions #20

Updated by JERiveraMoya about 6 years ago

  • Status changed from In Progress to Resolved
Actions #21

Updated by okurz about 6 years ago

  • Related to action #40325: [functional][y] Adapt yast2_samba to the new service widget added
Actions #22

Updated by okurz about 6 years ago

  • Related to deleted (action #40325: [functional][y] Adapt yast2_samba to the new service widget)
Actions #23

Updated by okurz about 6 years ago

  • Has duplicate action #40325: [functional][y] Adapt yast2_samba to the new service widget added
Actions #24

Updated by okurz about 6 years ago

This is an autogenerated message for openQA integration by the openqa_review script:

This bug is still referenced in a failing openQA test: iscsi_client
https://openqa.suse.de/tests/2171138

Actions

Also available in: Atom PDF