Bug 1205363 - [Build 20221111] yast2_dns_server: explicit call to netconfig
[Build 20221111] yast2_dns_server: explicit call to netconfig
Status: RESOLVED FIXED
Classification: openSUSE
Product: openSUSE Tumbleweed
Classification: openSUSE
Component: YaST2
Current
Other Other
: P5 - None : Normal (vote)
: ---
Assigned To: YaST Team
Jiri Srain
https://openqa.opensuse.org/tests/286...
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2022-11-12 07:55 UTC by Dominique Leuenberger
Modified: 2023-03-08 17:38 UTC (History)
3 users (show)

See Also:
Found By: openQA
Services Priority:
Business Priority:
Blocker: Yes
Marketing QA Status: ---
IT Deployment: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Dominique Leuenberger 2022-11-12 07:55:00 UTC
## Observation

This is 'just' another case where a yast module unconditionally calls netconfig.  NM was recently changed to no longer require sysconfig-netconfig ( but use it when present )


openQA test in scenario opensuse-Tumbleweed-DVD-x86_64-yast2_ncurses@64bit fails in
[yast2_dns_server](https://openqa.opensuse.org/tests/2869764/modules/yast2_dns_server/steps/19)

## Test suite description
Maintainer: qsf-y Test for yast2 UI, ncurses only. Running on created gnome images which provides both text console for ncurses UI tests as well as the gnome environment for the GUI tests.
riafarov set TIMEOUT_SCALE to improve stability of the test.


## Reproducible

Fails since (at least) Build [20221111](https://openqa.opensuse.org/tests/2869028)


## Expected result

Last good: [20221110](https://openqa.opensuse.org/tests/2867521) (or more recent)


## Further details

Always latest result in this scenario: [latest](https://openqa.opensuse.org/tests/latest?arch=x86_64&distri=opensuse&flavor=DVD&machine=64bit&test=yast2_ncurses&version=Tumbleweed)
Comment 2 Lukas Ocilka 2022-11-18 08:28:47 UTC
Yes, this whole thing about DNS forwarding has been designed based on netconfig. Just see https://github.com/yast/yast-dns-server/blob/master/src/modules/DnsServer.pm and https://github.com/yast/yast-dns-server/blob/master/src/include/dns-server/dialog-main.rb and search for "netconfig". You will get 40 hits.

Let's drop YaST DNS Server from Tumbleweed, we really don't have time for it.
Comment 3 Dominique Leuenberger 2022-11-18 08:49:12 UTC
(In reply to Lukas Ocilka from comment #2)
> Yes, this whole thing about DNS forwarding has been designed based on
> netconfig. Just see
> https://github.com/yast/yast-dns-server/blob/master/src/modules/DnsServer.pm
> and
> https://github.com/yast/yast-dns-server/blob/master/src/include/dns-server/
> dialog-main.rb and search for "netconfig". You will get 40 hits.
> 
> Let's drop YaST DNS Server from Tumbleweed, we really don't have time for it.

That implies dropping it (or not adding it) to the ALP stack (netconfig is not supposed to exist there)

Other than that, no objection
Comment 4 Lukas Ocilka 2022-11-18 08:53:11 UTC
Yes, that also means dropping from https://build.opensuse.org/package/view_file/SUSE:ALP:Workloads/yast-mgmt-ncurses-container/Dockerfile?expand=1 but IMO it's there just by a mistake. We did not intend to support it for ALP anyway.
Comment 6 Josef Reidinger 2023-02-07 14:03:34 UTC
We found out that on dns-server depends dhcp server module and also http server module. All modules are old and perl based, so quite hard to maintain. We decided to drop all of them from TW. ALP should not be affected as only dns-server is used in container and sr to drop it is already created.
Comment 11 Stefan Hundhammer 2023-03-08 17:38:05 UTC
grep -B 5 -A 3 1205363 yast-yast2/package/yast2.changes
-------------------------------------------------------------------
Wed Feb  8 21:46:04 UTC 2023 - Josef Reidinger <jreidinger@suse.com>

- DnsServerApi: drop module. It should never be in yast2 as it
  import module from yast2-dns-server. And now dns-server is going
  to be dropped (bsc#1205363)
- 4.5.23

-------------------------------------------------------------------