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

Use sqlx for iml-devices #2025

Merged
merged 2 commits into from
Jun 30, 2020
Merged

Use sqlx for iml-devices #2025

merged 2 commits into from
Jun 30, 2020

Conversation

jgrund
Copy link
Member

@jgrund jgrund commented Jun 29, 2020

Signed-off-by: Joe Grund jgrund@whamcloud.io


This change is Reviewable

@jgrund jgrund self-assigned this Jun 29, 2020
@jgrund jgrund force-pushed the sqlx-support branch 12 times, most recently from 479ec77 to 355718a Compare June 30, 2020 02:21
Base automatically changed from ip1981/1801-client-mounts to master June 30, 2020 13:23
Switch to using SQLx instead of diesel for iml-devices.
All queries are verified at compile time either against the sqlx-data.json file by default
or against the schema of a running db.

  - Switch to SQLx.
  - Add CI check to ensure schema is up to date.
  - Ensure device-scanner is built and installed for install-iml-local
    provisions.

Signed-off-by: Joe Grund <jgrund@whamcloud.io>
@jgrund jgrund requested a review from a team June 30, 2020 14:32
@jgrund jgrund added this to the IML EX V3 milestone Jun 30, 2020
@jgrund jgrund marked this pull request as ready for review June 30, 2020 14:32
ip1981
ip1981 previously approved these changes Jun 30, 2020
Copy link
Member

@ip1981 ip1981 left a comment

Choose a reason for hiding this comment

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

It's SQL over the wire anyway :)

johnsonw
johnsonw previously approved these changes Jun 30, 2020
ip1981
ip1981 previously approved these changes Jun 30, 2020
Signed-off-by: Joe Grund <jgrund@whamcloud.io>
@jgrund jgrund requested a review from johnsonw June 30, 2020 18:52
@jgrund jgrund merged commit a714020 into master Jun 30, 2020
@jgrund jgrund deleted the sqlx-support branch June 30, 2020 18:57
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
3 participants