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 only one address is shown in anaconda (#1264400) #556

Conversation

jkonecny12
Copy link
Member

Only NMIP[4,6]Config object was connected to signal on change. But when inner objects (Addresses, Nameservers, Gateway) changed it doesn't notify us about this change.
Fix by connecting this inner properties too.

Resolves: rhbz#1264400

Reported-by: Marek Hruscak <mhruscak@redhat.com>

@jkonecny12 jkonecny12 added rhel7-branch master Please, use the `f39` label instead. labels Mar 16, 2016
Only NMIP[4,6]Config object was connected to signal on change. But when
inner objects (Addresses, Nameservers, Gateway) changed it doesn't
notify us about this change.
Fix by connecting this inner properties too.

Resolves: rhbz#1264400

Reported-by: Marek Hruscak <mhruscak@redhat.com>
@rvykydal
Copy link
Contributor

Looks good to me.
Just a note, on master we'll need to consider whether we need to disconnect the callbacks in disconnect_client_callbacks.

@rvykydal rvykydal added the ACK label Mar 30, 2016
@jkonecny12
Copy link
Member Author

Pushed to rhel7, it seems it must be changed to master. So I'm closing this PR and I'm going to create new for the master.

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. rhel7-branch
2 participants