-
Notifications
You must be signed in to change notification settings - Fork 5
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
First stage only DNS setup tests #77
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just a suggestion, but feel free to ignore it.
aytests/dns_setup.sh
Outdated
|
||
set -e -x | ||
|
||
fgrep -q 'WRITE_HOSTNAME_TO_HOSTS="no"' /etc/sysconfig/network/dhcp || exit 1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If I am not wrong, as you're using set -e
, you won't need to call exit. It will stop on the first command that fails.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah that is exactly what happens when doing copy paste development ;-) I'll tweak it, thanks
package/aytests-tests.changes
Outdated
|
||
- Test for configuring DNS when running first stage only | ||
- Bumping version since the feature is SLE-15 only | ||
- 1.2.0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Thanks!
Does "See yast/yast-network#520" mean that we must wait with merging until after that one is merged? |
It doesn't make sense to merge it before that, so yes |
4b5c906
to
557c1e1
Compare
Fixed conflict in the changelog. Thanks for reviews. |
See yast/yast-network#520