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

Add kickstart test to test vlan creation #115

Closed

Conversation

jkonecny12
Copy link
Member

Steps of testing:

  • create vlan with network --device=link --vlanid=150 --bootproto=dhcp ks command
  • test ifcfg file in post section

I'm saving content of the configuration file and name of the vlan device to special file for debugging purpose.

@jkonecny12 jkonecny12 added the master Please, use the `f39` label instead. label May 22, 2015
%end

%post
CONTENT=$(cat /etc/sysconfig/network-scripts/$(ls /etc/sysconfig/network-scripts/ | grep ".150"))
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think you don't need to read the whole file here you can just get its name and then copy it. Then use the file name for the sed below to get the device. Also the second part of this could use a * pattern instead of the grep.

@vpodzime
Copy link
Contributor

Other than the above this looks good to me.

@clumens
Copy link
Contributor

clumens commented May 26, 2015

Yeah, looks good to me now too.

@jkonecny12
Copy link
Member Author

Thank you for your responses.
I'll change it locally and push it.

@jkonecny12 jkonecny12 closed this May 27, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
master Please, use the `f39` label instead.
3 participants