Skip to content
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

Add SQLite3 support #157

Merged
merged 6 commits into from
May 16, 2023

Conversation

maxlambrecht
Copy link
Contributor

Pull request check list

  • Proper tests/regressions included?
  • Documentation updated?

Affected functionality
Add support for SQLite datastore.
Convert the connection string config for the datastore to a Datastore block in the providers, supporting types postgres and sqlite3.

Description of change
Move the Datastore to the Catalog.
Add migrations and queries SQLs files for SQLite.
Refactor tests to run common test code for postgres and sqlite engines.
Improve some struct names.

Which issue this pull requests fixes

Victorblsilveira and others added 5 commits May 16, 2023 12:11
…#150)

Implementing Admin Handlers

Signed-off-by: Victor Vieira Barros Leal da Silveira <victorblsilveira@gmail.com>
Signed-off-by: Max Lambrecht <maxlambrecht@gmail.com>
Signed-off-by: Max Lambrecht <maxlambrecht@gmail.com>
Signed-off-by: Max Lambrecht <maxlambrecht@gmail.com>
Signed-off-by: Max Lambrecht <maxlambrecht@gmail.com>
Copy link
Collaborator

@Victorblsilveira Victorblsilveira left a comment

Choose a reason for hiding this comment

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

Awesome, great work, everything looks fine. I would say to move the fake datastore to the test/fake folder, but this is up to you. All good +1.

Signed-off-by: Max Lambrecht <maxlambrecht@gmail.com>
@sonarcloud
Copy link

sonarcloud bot commented May 16, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 8 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@maxlambrecht
Copy link
Contributor Author

Awesome, great work, everything looks fine. I would say to move the fake datastore to the test/fake folder, but this is up to you. All good +1.

I had done it already in the second or third commit. Now I removed the TODO.
Thanks

@maxlambrecht maxlambrecht merged commit 909bb28 into HewlettPackard:main May 16, 2023
@maxlambrecht maxlambrecht deleted the add-sqlite-support branch May 16, 2023 18:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants