Bug 1094296

Summary:	say that secondaries are intended to be used with vpn only
Product:	Red Hat Enterprise Linux 7	Reporter:	Vladimir Benes <vbenes>
Component:	NetworkManager	Assignee:	Jirka Klimes <jklimes>
Status:	CLOSED ERRATA	QA Contact:	Desktop QE <desktop-qa-list>
Severity:	unspecified	Docs Contact:
Priority:	unspecified
Version:	7.1	CC:	dcbw, jklimes, rkhan, thaller
Target Milestone:	rc
Target Release:	---
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:	NetworkManager-0.9.9.1-30.git20140326.4dba720.el7	Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2015-03-05 13:50:50 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:

Description Vladimir Benes 2014-05-05 12:51:56 UTC

Description of problem:
I can add uuid of any network I want into secondaries as neither documentation not describe say anything that it's not allowed. There is just an error written in messages that connection is not vpn. 

Version-Release number of selected component (if applicable):
NetworkManager-0.9.9.1-13.git20140326.4dba720.el7.x86_64

How reproducible:
always

Steps to Reproduce:
1.add wireless profile as an ethernet's secondary
2.nmcli connection edit id ethernet
3.set connection.secondary uuid_of_wireless_profile
4.nmcli con up id ethernet

Actual results:
both fails with error that wireless profile is not vpn in messages

Expected results:
some warning in nmcli that this is unsupported or describe saying that or man pages at least

Additional info:

Comment 1 Jirka Klimes 2014-05-23 10:30:45 UTC

The nmcli specific description has been added by this commit:
645f020 nmcli: improve setting connection.secondaries property

But we may still do some enhancements:
* update the property's description itself
* restrict setting secondaries to only VPNs in nmcli
* implement TAB completion in nmcli editor for secondaries
The changes are available in upstream branch jk/secondaries-rh1094296.

Comment 2 Thomas Haller 2014-05-23 13:19:58 UTC

Looks good to me

Comment 3 Dan Williams 2014-05-23 15:58:52 UTC

jk/secondaries-rh1094296 looks good to me.

Comment 4 Jirka Klimes 2014-05-26 08:42:32 UTC

Changes committed to upstream master:
e6f1c83 cli: TAB completion for connection.secondaries
7696238 cli: only allow setting VPN connection as secondaries in editor
e3d1e17 libnm-util: make explicit that we only allow VPN as secondaries (rh #1094296)

Comment 8 errata-xmlrpc 2015-03-05 13:50:50 UTC

Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://rhn.redhat.com/errata/RHBA-2015-0311.html