Bug 1138638 - 1-click install should not accept metadata without distversion tag.
1-click install should not accept metadata without distversion tag.
Status: RESOLVED FIXED
Classification: openSUSE
Product: openSUSE Tumbleweed
Classification: openSUSE
Component: YaST2
Current
Other Other
: P2 - High : Normal (vote)
: ---
Assigned To: YaST Team
Jiri Srain
https://trello.com/c/sNnvndya
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2019-06-19 04:28 UTC by Andrei Borzenkov
Modified: 2020-01-24 10:45 UTC (History)
3 users (show)

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


Attachments
y2logs (2.65 MB, application/x-compressed-tar)
2019-06-25 17:14 UTC, Andrei Borzenkov
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Andrei Borzenkov 2019-06-19 04:28:03 UTC
There are more and more reports (both forums and mailing lists) from users who used 1-Click install on stable releases and ended up having Tumbleweed repository configured for them. See e.g. this one which includes summary of other reports:

https://forums.opensuse.org/showthread.php/536470-Accidentally-installed-Factory-repository?p=2906194#post2906194

In all cases the problem is that user clicks on Tumbleweed version which does not provide distversion tag in its metadata.

According to XML schema for 1-Click metadata, distversion tag is mandatory. Why YaST oneclick installer silently accepts invalid XML without this tag?

At the very least installer must warn users multiple times that it cannot verify that package is actually intended for distribution user is using and provide as much information as possible for user to verify manually (presence of "tumbleweed" in URL would be some strong hint).

As it stands currently 1-Click installer became rather useless due to this issue.
Comment 1 Felix Miata 2019-06-19 05:50:32 UTC
It's worse than useless. It causes:
1-much time wasted by users trying to recover Leap functionality (not always successful) and by those helping users needing instruction to recover or understand what went wrong with functionality that simply works in Ubuntu
2-possible derogation of openSUSE reputation
3-data loss risk.
Comment 3 Andrei Borzenkov 2019-06-25 17:14:40 UTC
Created attachment 808472 [details]
y2logs

https://software.opensuse.org/ymp/openSUSE:Factory/standard/gnome-mahjongg.ymp?base=openSUSE%3AFactory&query=gnome-mahjongg

The problem is aggravated by the fact that software.o.o always suggests Tumbleweed 1-click link even when user is on Leap. User just clicks on link that *you* (openSUSE) offer. That should be a separate bug. I think it was reported, but cannot find it now.

bor@10:~> zypper lr -d
#  | Alias                               | Name                                    | Enabled | GPG Check | Refresh | Priority | Type   | URI                                                                      | Service
---+-------------------------------------+-----------------------------------------+---------+-----------+---------+----------+--------+--------------------------------------------------------------------------+--------
 1 | download.opensuse.org-non-oss       | Main Repository (NON-OSS)               | Yes     | (r ) Yes  | Yes     |   99     | rpm-md | http://download.opensuse.org/distribution/leap/15.1/repo/non-oss/        |        
 2 | download.opensuse.org-non-oss_1     | Update Repository (Non-Oss)             | Yes     | (r ) Yes  | Yes     |   99     | rpm-md | http://download.opensuse.org/update/leap/15.1/non-oss/                   |        
 3 | download.opensuse.org-oss           | Main Repository (OSS)                   | Yes     | (r ) Yes  | Yes     |   99     | rpm-md | http://download.opensuse.org/distribution/leap/15.1/repo/oss/            |        
 4 | download.opensuse.org-oss_1         | Main Update Repository                  | Yes     | (r ) Yes  | Yes     |   99     | rpm-md | http://download.opensuse.org/update/leap/15.1/oss                        |        
 5 | http-download.opensuse.org-6ebff5e5 | openSUSE:Factory                        | Yes     | (r ) Yes  | Yes     |   99     | rpm-md | http://download.opensuse.org/tumbleweed/repo/oss/                        |        
 6 | repo-debug                          | openSUSE-Leap-15.1-Debug                | No      | ----      | ----    |   99     | NONE   | http://download.opensuse.org/debug/distribution/leap/15.1/repo/oss/      |        
 7 | repo-debug-non-oss                  | openSUSE-Leap-15.1-Debug-Non-Oss        | No      | ----      | ----    |   99     | NONE   | http://download.opensuse.org/debug/distribution/leap/15.1/repo/non-oss/  |        
 8 | repo-debug-update                   | openSUSE-Leap-15.1-Update-Debug         | No      | ----      | ----    |   99     | NONE   | http://download.opensuse.org/debug/update/leap/15.1/oss/                 |        
 9 | repo-debug-update-non-oss           | openSUSE-Leap-15.1-Update-Debug-Non-Oss | No      | ----      | ----    |   99     | NONE   | http://download.opensuse.org/debug/update/leap/15.1/non-oss/             |        
10 | repo-source                         | openSUSE-Leap-15.1-Source               | No      | ----      | ----    |   99     | NONE   | http://download.opensuse.org/source/distribution/leap/15.1/repo/oss/     |        
11 | repo-source-non-oss                 | openSUSE-Leap-15.1-Source-Non-Oss       | No      | ----      | ----    |   99     | NONE   | http://download.opensuse.org/source/distribution/leap/15.1/repo/non-oss/ |        
bor@10:~>
Comment 4 Steffen Winterfeldt 2019-07-05 14:37:00 UTC
I fully agree.

The schema makes the 'distversion' attribute to the 'group' element mandatory.
We should stick to that and reject (not warn) invalid XML.

That said I see 'distversion' missing everywhere on s.o.o - so enforcing this
will probably annoy people even more.

I guess this needs to be sorted out.
Comment 5 Steffen Winterfeldt 2019-07-05 14:37:16 UTC
Tracking in YaST Scrum board.
Comment 6 Steffen Winterfeldt 2019-10-24 11:05:08 UTC
There's an open issue for the obs that needs to be resolved first:

https://github.com/openSUSE/open-build-service/issues/7458
Comment 7 Steffen Winterfeldt 2019-11-15 10:34:43 UTC
fixed 1-click installer

https://github.com/yast/yast-metapackage-handler/pull/18
Comment 8 Swamp Workflow Management 2020-01-24 10:40:06 UTC
This is an autogenerated message for OBS integration:
This bug (1138638) was mentioned in
https://build.opensuse.org/request/show/766789 Factory / yast2-metapackage-handler