kibana_plugin fails version change unless installed plugin is first removed (feature request)

[ghost]

PE3.8
CentOS 6.8
x-pack 5.4.2
kibana 5.4.2 -> 5.6.8 successfully
attempt upgrade to x-pack 5.6.8

Notice: /Stage[main]/Profile::Kibana/Profile::Kibana_plugin[x-pack]/Exec[install-/opt/staging/profile/x-pack-5.6.8.zip]/returns: Attempting to transfer from file:///opt/staging/profile/x-pack-5.6.8.zip
Notice: /Stage[main]/Profile::Kibana/Profile::Kibana_plugin[x-pack]/Exec[install-/opt/staging/profile/x-pack-5.6.8.zip]/returns: Transferring 119517805 bytes....................
Notice: /Stage[main]/Profile::Kibana/Profile::Kibana_plugin[x-pack]/Exec[install-/opt/staging/profile/x-pack-5.6.8.zip]/returns: Transfer complete
Notice: /Stage[main]/Profile::Kibana/Profile::Kibana_plugin[x-pack]/Exec[install-/opt/staging/profile/x-pack-5.6.8.zip]/returns: Retrieving metadata from plugin archive
Notice: /Stage[main]/Profile::Kibana/Profile::Kibana_plugin[x-pack]/Exec[install-/opt/staging/profile/x-pack-5.6.8.zip]/returns: Extracting plugin archive
Notice: /Stage[main]/Profile::Kibana/Profile::Kibana_plugin[x-pack]/Exec[install-/opt/staging/profile/x-pack-5.6.8.zip]/returns: Extraction complete
Notice: /Stage[main]/Profile::Kibana/Profile::Kibana_plugin[x-pack]/Exec[install-/opt/staging/profile/x-pack-5.6.8.zip]/returns: Plugin x-pack already exists, please remove before installing a new version```

I've obviously solved the plugin issue superficially, but could you update the module to handle this?

feature request: handle plugin version changes

[tylerjl]

Could you share your pre-upgrade manifest and the manifest you used to upgrade? The module should already uninstall plugins before installation if the version has changed, so I'd like to reproduce this if possible.

[ghost]

Sure, I will on Monday.

…

On Sat, Mar 3, 2018, 11:04 PM Tyler Langlois ***@***.***> wrote: Could you share your pre-upgrade manifest and the manifest you used to upgrade? The module should already uninstall plugins before installation if the version has changed, so I'd like to reproduce this if possible. — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#20 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AHfXcgigf-X97XJziQUabqNnmunFI_1wks5ta5HrgaJpZM4SZAFt> .

[ghost]

My team has been dealing with a fairly all-consuming production outage so my mind has been on that. I need to review the manifest first and make sure I am aware of what I am uploading and then I will do so. It may take me a week or so to actually do this, but I have made an entry in my backlog to do so. Again I apologize for missing the date I said I would get the manifest to you and appreciate your patience.

[tylerjl]

@jonassteinberg1 no worries! I appreciate the time you're taking to help debug and solve this for others in the OSS community. I'm happy to look at the manifest whenever you get the time.