-
Notifications
You must be signed in to change notification settings - Fork 3
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
Convert rock to use snap base #10
Conversation
@delgod This PR wont work as-is. I believe for this to be usable I would need to expose an app to run |
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.
Tnx!
Let's research why it doesn't work, I don't understand the issue, I don't think |
The integration tests here are failing because of an incompatibility with The operator needs to be updated to fix this. More details can be found here |
Here is the associated |
This is depending on this PR to be merged. |
These changes can be tested with |
What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Base change
What is the current behavior? (You can also link to an open issue here)
All mongodb components are installed from percona-provided deb packages.
What is the new behavior (if this is a feature change)?
The rock is now built using the
charmed-mongodb
snap instead of the official deb packages.Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)
Yes, you'll likely have to rewrite quite a few things to deal with confinement.
Other information:
charmed-mongodb-snap repo