Bug 1244893 - CLI does not validate if --neutron-tunnel-types matches --neutron-network-type
Summary: CLI does not validate if --neutron-tunnel-types matches --neutron-network-type
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat OpenStack
Classification: Red Hat
Component: python-rdomanager-oscplugin
Version: 7.0 (Kilo)
Hardware: Unspecified
OS: Unspecified
high
unspecified
Target Milestone: y2
: 7.0 (Kilo)
Assignee: Brad P. Crochet
QA Contact: Marius Cornea
URL:
Whiteboard:
: 1244894 (view as bug list)
Depends On:
Blocks: 1282835
TreeView+ depends on / blocked
 
Reported: 2015-07-20 17:31 UTC by Dan Sneddon
Modified: 2023-02-22 23:02 UTC (History)
12 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
: 1282835 (view as bug list)
Environment:
Last Closed: 2015-11-17 15:09:39 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Gerrithub.io 244982 0 None None None Never
Gerrithub.io 246187 0 None None None Never
OpenStack gerrit 221321 0 None None None Never
OpenStack gerrit 240545 0 None None None Never

Description Dan Sneddon 2015-07-20 17:31:56 UTC
Description of problem:
If you deploy the overcloud with --neutron-tunnel-types vxlan, but you don't specify the --neutron-network-type, then you end up with an unusable network configuration on the overcloud. All of the OVS agents are configured for VXLAN, but the Neutron networks are GRE, so nothing can communicate.

Version-Release number of selected component (if applicable):
puddle-2015-07-17.2

How reproducible:
100%

Steps to Reproduce:
1. Deploy with --neutron-tunnel-types vxlan but do not specify the --neutron-network-type
2.
3.

Actual results:
Neutron networks are configured as GRE, but the OVS agents are configured for VXLAN tunneling.

Expected results:
The CLI should throw an error if the --neutron-network-type does not match at least one of the --neutron-tunnel-types.

Additional info:

Comment 3 Mike Burns 2015-07-20 17:40:22 UTC
*** Bug 1244894 has been marked as a duplicate of this bug. ***

Comment 6 Udi Kalifon 2015-09-08 12:07:34 UTC
I got no error when I passed "--neutron-tunnel-types vxlan --neutron-network-type gre" on the command line. Deployment succeeded (I didn't test the resulting overcloud, but it deployed).

Comment 7 Brad P. Crochet 2015-09-08 13:09:06 UTC
@Udi

Can you include the version of python-rdomanager-oscplugin?

Comment 8 Brad P. Crochet 2015-09-08 13:52:54 UTC
Verified that version was python-rdomanager-oscplugin-0.0.10-1.el7

Comment 10 Marius Cornea 2015-09-16 12:27:47 UTC
openstack overcloud deploy --templates ~/templates/my-overcloud -e ~/templates/my-overcloud/environments/network-isolation.yaml -e ~/templates/network-environment.yaml --control-scale 1 --compute-scale 1 --ceph-storage-scale 0 --neutron-tunnel-types vxlan

results in 

[heat-admin@overcloud-controller-0 ~]$ sudo grep tenant_network /etc/neutron/plugin.ini  | grep -v ^#
tenant_network_types = gre

I think neutron-network-type defaults to gre in overcloud_deploy.

Expected result:
The same result as when passing specific neutron-network-type:

[stack@instack ~]$ openstack overcloud deploy --templates ~/templates/my-overcloud -e ~/templates/my-overcloud/environments/network-isolation.yaml -e ~/templates/network-environment.yaml --control-scale 1 --compute-scale 1 --ceph-storage-scale 0 --neutron-tunnel-types vxlan --neutron-network-type gre
ERROR: openstack Neutron network type must be in Neutron tunnel types (vxlan) 

[stack@instack ~]$ rpm -qa | grep oscplugin
python-rdomanager-oscplugin-0.0.10-3.el7ost.noarch

Comment 12 Alexander Chuzhoy 2015-09-18 20:54:33 UTC
Verified:

Environment:
python-rdomanager-oscplugin-0.0.10-4.el7ost.noarch


Attempted to deploy with command:
openstack overcloud deploy --templates --control-scale 3 --compute-scale 1 --ceph-storage-scale 0 -e <yaml>  --ntp-server x.x.x.x   --timeout 90 --compute-flavor compute --control-flavor control --neutron-tunnel-types gre


Resulted in:
ERROR: openstack Neutron network type must be specified when Neutron tunnel types is specified

Comment 13 Dan Sneddon 2015-09-21 22:54:56 UTC
I have one report that when deploying with --neutron-network-type vlan the user was required to add --neutron-tunnel-types vlan.

That is not correct. We do want to make sure that when the network type is GRE or VXLAN that the neutron-tunnel-types includes that network type, but not when the network type is VLAN. When the network type is VLAN, the neutron-tunnel-types shouldn't be required.

Comment 14 Marius Cornea 2015-09-22 12:08:22 UTC
(In reply to Dan Sneddon from comment #13)
> I have one report that when deploying with --neutron-network-type vlan the
> user was required to add --neutron-tunnel-types vlan.
> 
> That is not correct. We do want to make sure that when the network type is
> GRE or VXLAN that the neutron-tunnel-types includes that network type, but
> not when the network type is VLAN. When the network type is VLAN, the
> neutron-tunnel-types shouldn't be required.

I confirm that:

[stack@instack ~]$ openstack overcloud deploy --templates ~/templates/my-overcloud -e ~/templates/my-overcloud/environments/network-isolation.yaml -e ~/templates/network-environment.yaml --control-scale 3 --compute-scale 1 --ntp-server clock.redhat.com -e ~/templates/snmpd.yaml --neutron-network-type vlan
ERROR: openstack Neutron tunnel types must be specified when Neutron network type is specified 

Before moving it to assigned do you think --neutron-network-vlan-ranges should be required if --neutron-network-type is set to vlan?

Comment 15 Dan Sneddon 2015-09-22 16:20:11 UTC
(In reply to Marius Cornea from comment #14)
> (In reply to Dan Sneddon from comment #13)
> > I have one report that when deploying with --neutron-network-type vlan the
> > user was required to add --neutron-tunnel-types vlan.
> > 
> > That is not correct. We do want to make sure that when the network type is
> > GRE or VXLAN that the neutron-tunnel-types includes that network type, but
> > not when the network type is VLAN. When the network type is VLAN, the
> > neutron-tunnel-types shouldn't be required.
> 
> I confirm that:
> 
> [stack@instack ~]$ openstack overcloud deploy --templates
> ~/templates/my-overcloud -e
> ~/templates/my-overcloud/environments/network-isolation.yaml -e
> ~/templates/network-environment.yaml --control-scale 3 --compute-scale 1
> --ntp-server clock.redhat.com -e ~/templates/snmpd.yaml
> --neutron-network-type vlan
> ERROR: openstack Neutron tunnel types must be specified when Neutron network
> type is specified 
> 
> Before moving it to assigned do you think --neutron-network-vlan-ranges
> should be required if --neutron-network-type is set to vlan?

That's a great point, yes we probably should require --neutron-network-vlan-ranges when vlan mode is selected.

Comment 16 Marius Cornea 2015-09-22 16:38:49 UTC
Ok, so when --neutron-network-type is set to vlan --neutron-tunnel-types should not be required but instead --neutron-network-vlan-ranges should be passed.

Comment 17 Dan Sneddon 2015-09-22 16:44:16 UTC
(In reply to Marius Cornea from comment #16)
> Ok, so when --neutron-network-type is set to vlan --neutron-tunnel-types
> should not be required but instead --neutron-network-vlan-ranges should be
> passed.

Yes, that is correct.

Comment 25 Udi Shkalim 2015-11-16 08:42:01 UTC
This bug is blocking us from deploying with VLAN and verify functionality

Comment 27 chris alfonso 2015-11-17 15:09:39 UTC
Closing this bug after cloning it for OSP8 to take care of the last corner case. The rest of it is in the current release.


Note You need to log in before you can comment on or make changes to this bug.