Skip to content
This repository has been archived by the owner on Jun 24, 2022. It is now read-only.

Allow switching between oss and standard packages #521

Merged
merged 5 commits into from
Jan 8, 2019
Merged

Conversation

Crazybus
Copy link
Contributor

Before only going from the standard package to oss was supported. Now
this works properly in both directions.

Fixes: #520

Before only going from the standard package to oss was supported. Now
this works properly in both directions.
How it was previously "worked" but just ignored trying to remove the
package. Luckily it was caught by the test-kitchen testing!
@fedelemantuano
Copy link

Hi @Crazybus,

if you switch from oss to xpack or vice versa, you will have two two yum repo in system.
To have a clean config it's better remove the old one.
I know that change only the name, maybe you could use a unique name for repos oss and xpack.

@Crazybus
Copy link
Contributor Author

@fedelemantuano Great catch! I have updated the PR so that it will remove the unused repo.

@fedelemantuano
Copy link

Hi @Crazybus, I read the new commit but I don't understand these lines.

@Crazybus
Copy link
Contributor Author

Crazybus commented Jan 2, 2019

Hi @Crazybus, I read the new commit but I don't understand these lines.

@fedelemantuano Good because it was wrong :P. I have fixed it up to remove the file instead of adding it (again). I also added some tests to make sure this is actually working properly. After adding the tests I found out that the Debian version was also only removing the package and not the repo so I have fixed that up too!

@Crazybus Crazybus requested a review from itsmed January 2, 2019 10:18
@fedelemantuano
Copy link

Yes I saw the Debian part, so I'm fixing my ansible-kibana role too.

Copy link
Contributor

@itsmed itsmed left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@Crazybus Crazybus merged commit a7b07e4 into master Jan 8, 2019
@Crazybus Crazybus deleted the swapsies branch January 8, 2019 18:54
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants