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

Fixes #15368 - refresh existing discovered host #276

Closed
wants to merge 1 commit into from

Conversation

lzap
Copy link
Member

@lzap lzap commented Jun 15, 2016

Existing host (when renamed) was ignored.

Also fixed 404 for show page.

Replaces #275

@cfchase
Copy link

cfchase commented Jun 16, 2016

Tested code against Sat 6.2 branch. Verified it refreshes facts and no longer duplicates the renamed discovered host.

@lzap
Copy link
Member Author

lzap commented Jun 16, 2016

Tested code against Sat 6.2 branch. Verified it refreshes facts and no longer duplicates the renamed discovered host.

Thanks, will take sometime until it lands z-stream, I suggest you to
patch the package yourself.

Later,
Lukas #lzap Zapletal

@orrabin
Copy link
Member

orrabin commented Jun 24, 2016

Tested against develop and it looks good.

@dLobatog
Copy link
Member

Merged as 7dc5b8d, thanks @lzap!

@dLobatog dLobatog closed this Jun 27, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
5 participants