Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

PHYDEV should not be mandatory for ovswitch #2241

Closed
2 of 41 tasks
baby-gnu opened this issue Jun 29, 2018 · 3 comments
Closed
2 of 41 tasks

PHYDEV should not be mandatory for ovswitch #2241

baby-gnu opened this issue Jun 29, 2018 · 3 comments
Assignees
@vholer
Labels
Category: Drivers - Network Priority: High Status: Accepted Type: Bug
Milestone
Release 5.6

Comments

@baby-gnu
Copy link
Contributor

baby-gnu commented Jun 29, 2018 •
edited
Loading

Bug Report

Version of OpenNebula

  • 5.2.2
  • 5.4.0
  • 5.4.1
  • 5.4.2
  • 5.4.3
  • 5.4.4
  • 5.4.5
  • 5.4.6
  • 5.4.7
  • 5.4.8
  • 5.4.9
  • 5.4.10
  • 5.4.11
  • 5.4.12
  • 5.4.13
  • 5.4.14
  • 5.4.15
  • Development build

Component

  • Authorization (LDAP, x509 certs...)
  • Command Line Interface (CLI)
  • Contextualization
  • Documentation
  • Federation and HA
  • Host, Clusters and Monitorization
  • KVM
  • Networking
  • Orchestration (OpenNebula Flow)
  • Packages
  • Scheduler
  • Storage & Images
  • Sunstone
  • Upgrades
  • User, Groups, VDCs and ACL
  • vCenter

Description

An incompatible change is made to ovswitch driver : the PHYDEV attribute is now mandatory.

Here is the vnet template I used for ONE 5.4:

NAME = "CR_aca"
VN_MAD = ovswitch
BRIDGE = vswitch
AR=[
  TYPE = "ETHER",
  SIZE = "254"
]

Expected Behavior

I can create completely isolated vnet without the PHYDEV attribute.

Actual Behavior

I can't create the vnet with the following error:

Fri Jun 29 14:58:24 2018 [Z0][ReM][D]: Req:8608 UID:0 one.vn.allocate invoked , "NAME = "CR_aca"
VN_M...", -1
Fri Jun 29 14:58:24 2018 [Z0][VNM][E]: PHYDEV is mandatory for driver ovswitch
Fri Jun 29 14:58:24 2018 [Z0][ReM][E]: Req:8608 UID:0 one.vn.allocate result FAILURE [one.vn.allocate] PHYDEV is mandatory for driver ovswitch

How to reproduce

cat > /tmp/vnet.tmpl <<EOF
NAME = "CR_aca"
VN_MAD = ovswitch
BRIDGE = vswitch
AR=[
  TYPE = "ETHER",
  SIZE = "254"
]
EOF

onevnet create /tmp/vnet.tmpl

Progress Status

  • Branch created
  • Code committed to development branch
  • Testing - QA
  • Documentation
  • Release notes - resolved issues, compatibility, known issues
  • Code committed to upstream release/hotfix branches
  • Documentation committed to upstream release/hotfix branches
@baby-gnu
Copy link
Contributor Author

I found #1807 and I'm more confused.

I think the BRIDGE is mandatory since to create the network interface libvirt use something like:

ovs-vsctl add-port <BRIDGE> one-<VMID>-<INTERFACEID>

But the PHYDEV should not.

Regards.

@baby-gnu
Copy link
Contributor Author

Plus the documentation says it's optional for ovswitch:

Regards.

@baby-gnu baby-gnu changed the title PHYDEV is mandatory for ovswitch PHYDEV is mandatory for ovswitch Jun 29, 2018
@baby-gnu baby-gnu changed the title PHYDEV is mandatory for ovswitch PHYDEV should not be mandatory for ovswitch Jun 29, 2018
@tinova tinova added this to the Release 5.6 milestone Jul 2, 2018
rsmontero pushed a commit that referenced this issue Jul 4, 2018
B #2241: Change PHY_DEV mandatory'ness for ovswitch. Do not require b…
b5443ed 
…ridge when

it is generated.
abelCoronado93 pushed a commit to abelCoronado93/one that referenced this issue Jul 4, 2018
B OpenNebula#2241: PHYDEV shouldn't be mandatory for ovswitch [Sunstone]
ede6d95
rsmontero pushed a commit that referenced this issue Jul 4, 2018
B #2241: PHYDEV shouldn't be mandatory for ovswitch [Sunstone]
e2e8894
@rsmontero
Copy link
Member

Fix for 5.6 oned and Sunstone.

rsmontero pushed a commit that referenced this issue Jul 21, 2022
@jloboescalona2
F #5888: Add Storage->Image tab (#2241)
f5f8266 
Co-authored-by: Sergio Betanzos <sbetanzos@opennebula.io>
rsmontero pushed a commit that referenced this issue Jul 27, 2022
@jloboescalona2 @rsmontero
F #5888: Add Storage->Image tab (#2241)
b81e242 
Co-authored-by: Sergio Betanzos <sbetanzos@opennebula.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@vholer vholer

Labels
Category: Drivers - Network Priority: High Status: Accepted Type: Bug
Projects
None yet
Milestone
Release 5.6
Development

No branches or pull requests
4 participants
@tinova @vholer @rsmontero @baby-gnu