Skip to content

Conversation

carbonin
Copy link
Member

Previously the regex in NetworkInterface#parse_ip4 would also match ipv6 addresses.
This would cause an issue when only ipv6 addresses were returned by the ip addr show command.
This problem was masked by the fact that ipv4 addresses are returned first in the output and detect returns the first matching line.

@bdunne

@carbonin carbonin changed the title Dont fail when only ipv6 addresses are returned by ip addr show Dont fail when only IPv6 addresses are returned by ip addr show Nov 16, 2015
@miq-bot
Copy link
Member

miq-bot commented Nov 16, 2015

Checked commits carbonin/linux_admin@29fe376~...1637369 with ruby 1.9.3, rubocop 0.34.2, and haml-lint 0.13.0
2 files checked, 0 offenses detected
Everything looks good. 👍

bdunne added a commit that referenced this pull request Nov 30, 2015
Dont fail when only IPv6 addresses are returned by `ip addr show`
@bdunne bdunne merged commit e969487 into ManageIQ:master Nov 30, 2015
@carbonin carbonin deleted the dont_fail_with_only_ipv6 branch February 12, 2016 18:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants