-
Notifications
You must be signed in to change notification settings - Fork 6
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
nixos/phoebus-olog: use FerretDB intead of MongoDB #232
base: master
Are you sure you want to change the base?
nixos/phoebus-olog: use FerretDB intead of MongoDB #232
Conversation
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.
Nice workaround! Thanks
Oh wait, if we change database backend, users will need to migrate their data from MongoDB to FerretDB, I'll need to document that. I'll also move that into the breaking changes. |
OK I'll mark this pull request as a draft in the meantime. |
By the way, I just checked the known differences between MongoDB and FerretDB. Something is drawing my attention:
Are we sure Olog isn't using any |
@stephane-cea Good question! I just checked, and it seems that Phoebus Olog only uses MongoDB to store attachments. Phoebus also doesn't currently seem to provide the option to send PV values in a log entry. |
In this case, I think it's safe to use FerretDB Thanks for checking 👍 |
to prevent having to compile MongoDB, which requires a lot of resources
c59c3af
to
af8f170
Compare
To prevent having to compile MongoDB, which requires a lot of resources