-
Notifications
You must be signed in to change notification settings - Fork 349
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 possible to start installation without network (#1221109) #208
Fix possible to start installation without network (#1221109) #208
Conversation
|
||
def apply(self): | ||
_update_network_data(self.data, self.network_control_box) | ||
log.debug("network: apply ksdata %s", self.data.network) | ||
self.network_control_box.kill_nmce(msg="leaving network spoke") | ||
|
||
# if installation media is not used try if source is reachable | ||
if self.data.method.method != "cdrom" and self._network_change: |
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.
hd is another possible non-network source.
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.
True. I'll change it locally.
0bdd0b2
to
4a06127
Compare
Add check for harddrive too. |
Note that on this PR, the jenkins results are correct (even if it tested too much - don't know what's up with that). There are some pylint problems. |
Payload manager is started from network spoke if network settings change. This will start source spoke and software spoke reach-ability testing. Resolves: rhbz#1221109
4a06127
to
d239863
Compare
Yeah I already fixed it. Thank you for this PR testing @clumens . |
Looks good. |
Pushed. Thank you |
Payload manager is started from network spoke if network settings change.
This will start source spoke and software spoke reach-ability testing.
Resolves: rhbz#1221109