Bug #4882
Error when terminating a VM with more than ten interfaces.
Status: | Closed | Start date: | 10/26/2016 | |
---|---|---|---|---|
Priority: | Normal | Due date: | ||
Assignee: | Marcel Weinberg | % Done: | 0% | |
Category: | Drivers - Network | |||
Target version: | Release 5.2.1 | |||
Resolution: | fixed | Pull request: | ||
Affected Versions: | OpenNebula 5.2 |
Description
It's quite simple:
Whenever you terminate a VM with 10 or more interfaces, you will get the following error:
Wed Oct 26 18:39:44 2016 [Z0][VMM][I]: iptables: Too many links.
Wed Oct 26 18:39:44 2016 [Z0][VMM][E]: clean: Command Error: sudo iptables -X one-121-10-i
Wed Oct 26 18:39:44 2016 [Z0][VMM][E]: clean: ["/var/tmp/one/vnm/command.rb:61:in `block in run!'", "/var/tmp/one/vnm/command.rb:58:in `each'", "/var/tmp/one/vnm/command.rb:58:in `run!'", "/var/tmp/one/vnm/security_groups_iptables.rb:345:in `nic_deactivate'", "/var/tmp/one/vnm/sg_driver.rb:128:in `block in deactivate'", "/var/tmp/one/vnm/sg_driver.rb:125:in `each'", "/var/tmp/one/vnm/sg_driver.rb:125:in `deactivate'", "/var/tmp/one/vnm/802.1Q/clean:35:in `<main>'"]
Wed Oct 26 18:39:44 2016 [Z0][VMM][I]: ExitCode: 1
Wed Oct 26 18:39:44 2016 [Z0][VMM][I]: Failed to execute network driver operation: clean.
Wed Oct 26 18:39:44 2016 [Z0][VMM][E]: Error shutting down VM: 802.1Q: -
Wed Oct 26 18:39:44 2016 [Z0][VM][I]: New LCM state is RUNNING
Wed Oct 26 18:39:45 2016 [Z0][LCM][I]: Fail to shutdown VM. Assuming that the VM is still RUNNING (will poll VM).
Wed Oct 26 18:40:16 2016 [Z0][LCM][I]: VM running but monitor state is POWEROFF
Wed Oct 26 18:40:16 2016 [Z0][VM][I]: New LCM state is SHUTDOWN_POWEROFF
Wed Oct 26 18:40:17 2016 [Z0][VM][I]: New state is POWEROFF
The VM will be terminated anyway, BUT the iptables rules and chains remain on the system.
This is caused by a regex match that does not check the whole string.
Example:
if line.match(/^#{chain}/)
I will fork the repo and creaate a pull request with a fix.
History
#1 Updated by Marcel Weinberg over 4 years ago
Created pull request #132
#2 Updated by Marcel Weinberg over 4 years ago
Created pull request 132.
#3 Updated by Javi Fontan over 4 years ago
- Status changed from Pending to Closed
- Target version set to Release 5.4
- Resolution set to fixed
Merged pull requests to both master and 5.2 branches
#4 Updated by Tino Vázquez over 4 years ago
- Target version changed from Release 5.4 to Release 5.2.1