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.

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

Fix solaris2 plugin network interface detection #859

Merged
merged 3 commits into from
Aug 5, 2016

Conversation

acaiafa
Copy link
Contributor

@acaiafa acaiafa commented Aug 5, 2016

The current regex in place does not properly catch the interfaces. Updating this regex allows for the network information to actually populate.

@acaiafa acaiafa changed the title updating regex fixing solaris2 ohai plugin regex Aug 5, 2016
@johnbellone
Copy link
Contributor

/cc @scotthain

@scotthain
Copy link

👍 verified. Thanks @johnbellone

@acaiafa
Copy link
Contributor Author

acaiafa commented Aug 5, 2016

@scotthain any idea how long until this makes it out?

@scotthain
Copy link

@acaiafa I don't actually know, I'll see if I can find out.

@tas50
Copy link
Contributor

tas50 commented Aug 5, 2016

Not the next chef release, but the release after that

@tas50
Copy link
Contributor

tas50 commented Aug 5, 2016

Looks good to me. 👍

@tas50 tas50 merged commit 9078ea8 into chef:master Aug 5, 2016
@tas50 tas50 changed the title fixing solaris2 ohai plugin regex Fix solaris2 plugin network interface detection Aug 11, 2016
@tas50 tas50 added Bug and removed Signed CLA labels Aug 11, 2016
@thommay thommay added Type: Bug Does not work as expected. and removed Bug labels Jan 24, 2017
@chef chef locked and limited conversation to collaborators Nov 16, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Type: Bug Does not work as expected.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants