feat: add DB initialization module #556
Commits on Feb 7, 2024
-
feat(data-formats): move StoredItem from rendezvous
The `StoredItem` struct is no longer needed just in the rendezvous server, we are making it public and moving it to the data-formats lib so that it can be easily included in other modules. Signed-off-by: Irene Diez <[email protected]>
-
feat: add DB initialization module
This provides an interface defining the methods that we expect to be used for OV handling operations. Added an implementation for sqlite and postgress. Added db creation files and tests. Signed-off-by: Irene Diez <[email protected]>
-
Adds new dependencies and creates a sqlite database to be used in the tests. Signed-off-by: Irene Diez <[email protected]>
-
feat: add the Manufacturer DB store
This adds the store interface methods for the SqliteManufacturerStore. Signed-off-by: Irene Diez <[email protected]>
-
feat: don't allow db settings in per-device config
Signed-off-by: Irene Diez <[email protected]>
-
feat: add the Owner DB store configuration
This adds the store trait for the Sqlite Owner DB. Signed-off-by: Irene Diez <[email protected]>
-
feat: add the Rendezvous DB store configuration
This implements the store trait for the Sqlite Rendezvous DB. Signed-off-by: Irene Diez <[email protected]>
-
feat(store): change store initializations based on DB type
When selecting to use the DB as the store type in the configuration files, this requires to select the DB type (sqlite or postgres) and the server type. Signed-off-by: Irene Diez <[email protected]>
-
fix(data-formats): error when trying to parse an empty slice
When we are trying to parse raw data to make an ownership voucher we did not check whether the slice was empty or not, causing a panic in such case. This change explicitly checks if the slice is empty or not, and in the former case returns an error. Signed-off-by: Irene Diez <[email protected]>
-
feat(owner-tool): add an option to export OVs from the Manufacturer DB
This adds the `export_manufacturer_vouchers` option to the Owner CLI, which allows to export OVs from the Manufacturer Server's DB to a directory. The options are to export a single OV by GUID or all of them. Updated the DB trait to add the `select *` query that yields all the OVs in the Manufacturer. Signed-off-by: Irene Diez <[email protected]>
-
feat(owner-tool): add an option to import OVs to the Owner DB
Signed-off-by: Irene Diez <[email protected]>
-
fix(db): do not bail on non-critical errors
We are going to propagate the errors unless those are related to getting a connection to the DB. Signed-off-by: Irene Diez <[email protected]>
-
fix(db): remove dotenv handling
Signed-off-by: Irene Diez <[email protected]>
-
feat(store): add query_ovs_db method to store trait
The `query_data` method makes no sense when using databases since it uses different types of lists to query the required data, when that's a single query in a database; thereby we are adding the `query_ovs_db` method which explicitly queries the required data from the database. We are also adding a new error type `StoreError::MethodNotAvailable` that will be yielded when we call `query_data` from a database store implementation, or when we call the new `query_ovs_db` method from a directory store implementation. Signed-off-by: Irene Diez <[email protected]>
-
feat(owner-server): add DB handling to report_to_rendezvous
When the Owner server needs to report OVs to the Rendezvous server, check whether we have the old `query_data` method available or the new `query_ovs_db` method to choose the most effective one. Signed-off-by: Irene Diez <[email protected]>
-
feat(store): add Database error type
Replace all the `StoreError::Unspecified` errors with the new `StoreError::Database` when there is an internal database error. Signed-off-by: Irene Diez <[email protected]>
-
fix(db): change select_ov_to2_performed_and_ov_to0_less_than query
This changes the filters so that they correctly take into account NULL values. In order to set a NULL value we need to use None in rust, but for filtering instead of using None we need to filter by `is_null()`. Signed-off-by: Irene Diez <[email protected]>
-
fix(store): expect StoredItem in rendezvous store
This changes the `store_data` method so that it expects a Value of type `StoredItem` for the serialization and deserialization of data. Signed-off-by: Irene Diez <[email protected]>
-
chore(containers): update DB dependencies
Signed-off-by: Irene Diez <[email protected]>
-
feat(store): add postgresql store variant
Signed-off-by: Irene Diez <[email protected]>
-
docs: add database usage and configuration
Signed-off-by: Irene Diez <[email protected]>
-
-
chore: update spec to add sql files and unify migrations
Since the migrations are unified in a single folder the tests have been updated accordingly. Signed-off-by: Irene Diez <[email protected]>
-
fix: adds necessary crates and scripts to dev container
Adds sqlite and libpq crates to devcontainer creation for successful building and testing in dev container. Also adds diesel scripts to dev container build to enable successful dev container integration testing. Signed-off-by: djach7 <[email protected]>
-
Signed-off-by: Irene Diez <[email protected]>
