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.

Sign up for GitHub

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

Jump to bottom

Data Availability #147

Merged
merged 22 commits into from
Jul 22, 2024
Merged

Data Availability #147

merged 22 commits into from
Jul 22, 2024

Conversation

marijanp
Copy link
Contributor

@marijanp marijanp commented Jul 10, 2024 •
edited
Loading

  • Cherry picked commits from Data availability #124
  • Added a postgres test hook for system integration testing
  • Modified kairos-cli and kairos-server tests to spawn a postgres database

Rom3dius and others added 20 commits July 9, 2024 15:37
@Rom3dius @marijanp
store transfer, withdrawal deposit + query endpoint
6f7430c
@Rom3dius @marijanp
implemented From<DBError> for AppErr
76be994
@Rom3dius @marijanp
put it all behind a feature flag
deae4fc
@Rom3dius @marijanp
use bigdecimal for numeric + only run migrations if feature enabled
19d69f6
@Rom3dius @marijanp
added migrations feature to kairos-server kairos-data
b6ecfcf
@Rom3dius @marijanp
enum + fmt + removed comments + removed blankspace etc.
84e9555
@Rom3dius @marijanp
removed all pool clones
9252b2d
@Rom3dius @marijanp
database flag for test-utils + feature flag propagation
6cad0d8
@marijanp
kairos-test-utils: implement postgres hook
28cda11
@marijanp
kairos-test-utils/kairos: add postgres url
65de666
@marijanp
tests: start kairos with database if feature-flag enabled
5ce2a35
@marijanp
kairos: enable kairos-test-utils/database
f8e082c
@marijanp
kairos-test-utils: make migrations optional
05b1228
@marijanp
kairos-data: make errors more verbose
843f84f
@marijanp
kairos-data: verbose logging
b0cb399
@marijanp
kairos-data: remove patch version
3b45d59
@marijanp
kairos-data: fix enum value capitalization
165743f
@marijanp
flake/devShell: add diesel-cli
f49356e
@marijanp
kairos-test-utils/postgres: derive Debug for PgConnection
a664f2c
@marijanp
kairos-server/tests: move posgres into individual test case to avoid …
101450a 
…dropping
Avi-D-coder
Avi-D-coder approved these changes Jul 12, 2024
View reviewed changes
Copy link
Contributor

@Avi-D-coder Avi-D-coder left a comment

Choose a reason for hiding this comment

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

tentative yes, but I have not run it yet.

@marijanp marijanp changed the title Cherry pick da Data Availability Jul 15, 2024
@marijanp marijanp self-assigned this Jul 15, 2024
koxu1996
koxu1996 approved these changes Jul 22, 2024
View reviewed changes
Copy link
Contributor

@koxu1996 koxu1996 left a comment

Choose a reason for hiding this comment

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

LGTM

@koxu1996 koxu1996 merged commit 649a74f into main Jul 22, 2024
4 checks passed
@koxu1996 koxu1996 deleted the cherry-pick-da branch July 22, 2024 12:46
@koxu1996 koxu1996 mentioned this pull request Jul 22, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@koxu1996 koxu1996 koxu1996 approved these changes

@Avi-D-coder Avi-D-coder Avi-D-coder approved these changes

@jonas089 jonas089 Awaiting requested review from jonas089

@Rom3dius Rom3dius Awaiting requested review from Rom3dius

Assignees

@marijanp marijanp

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@marijanp @koxu1996 @Avi-D-coder @Rom3dius