refactor: remove gorm, use pgx for postgres #696

sbchaos · 2023-01-02T07:51:30Z

No description provided.

coveralls · 2023-01-02T07:56:27Z

Pull Request Test Coverage Report for Build 3844192529

72 of 89 (80.9%) changed or added relevant lines in 7 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.04%) to 91.544%

Changes Missing Coverage	Covered Lines	Changed/Added Lines	%
core/job/handler/v1beta1/job_adapter.go	19	23	82.61%
internal/errors/errors.go	0	5	0.0%
core/job/spec.go	34	42	80.95%

Files with Coverage Reduction	New Missed Lines	%
core/job/spec.go	1	95.46%

Totals
Change from base Build 3820861267:	-0.04%
Covered Lines:	8239
Relevant Lines:	9000

💛 - Coveralls

* fix: bug in __lib.py due to mishandling response from handle_pod_overlap (#695) * fix: add scheduler repo tests Co-authored-by: Anwar Hidayat <[email protected]>

internal/store/postgres/migrations/000046_update_job_table.up.sql

…und across projects

sbchaos added 14 commits January 2, 2023 13:18

refactor: use pgx in project repo

fdc9647

refactor: migrate namespace repository

74a29c8

refactor: move secret repository to pgx

95c4c8d

fix: fix lint and test issues

d005963

refactor: add foreign key for tenant in resource and job

809d47b

refactor: use pgx for resource and backup

c06ea5f

refactor: use pgx for job repository

57a6920

refactor: change repo for scheduler bc

779744f

refactor: move migration command to server

0442486

refactor: use pgx remove gorm

80cda9c

refactor: update tenant bounded context

d05d782

refactor: update scheduler bounded context

199ff05

refactor: update resource bounded context

473a2ff

refactor: update job bounded context

b5a4bd6

sbchaos added 2 commits January 2, 2023 16:54

fix: fix lint errors

d32a5fa

fix: fix issues for field scanning

f2f0a8d

sbchaos linked an issue Jan 3, 2023 that may be closed by this pull request

Migrate away from gorm, use pgx to query postgres db #592

Closed

5 tasks

fix: job deleted_at value is always nil issue

e699cd7

sbchaos linked an issue Jan 3, 2023 that may be closed by this pull request

Store should fetch only the data for table and not do it for associations #421

Closed

arinda-arif and others added 5 commits January 4, 2023 10:56

fix: job repository read write issues and alter job table

defa281

fix: add scheduler repo tests (#698)

e469fe3

* fix: bug in __lib.py due to mishandling response from handle_pod_overlap (#695) * fix: add scheduler repo tests Co-authored-by: Anwar Hidayat <[email protected]>

fix: fix time zone to utc

8ef3213

fix: test time zone failures

591b0d7

refactor: simplify job spec struct

534d84a

sbchaos commented Jan 4, 2023

View reviewed changes

internal/store/postgres/migrations/000046_update_job_table.up.sql Show resolved Hide resolved

arinda-arif added 3 commits January 4, 2023 17:11

fix: scheduler job repo test failures due to spec changes

2c98e05

feat: add 000046 migration down file

c9bd3ea

fix: inferred dependency resolution issue when duplicated job name fo…

3611725

…und across projects

sravankorumilli approved these changes Jan 5, 2023

View reviewed changes

sravankorumilli merged commit 5899045 into main Jan 5, 2023

sravankorumilli deleted the remove-gorm branch January 5, 2023 06:55

sbchaos linked an issue Jan 10, 2023 that may be closed by this pull request

The database migration might not run in some scenarios #689

Closed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

refactor: remove gorm, use pgx for postgres #696

refactor: remove gorm, use pgx for postgres #696

sbchaos commented Jan 2, 2023

coveralls commented Jan 2, 2023 •

edited

Loading

refactor: remove gorm, use pgx for postgres #696

refactor: remove gorm, use pgx for postgres #696

Conversation

sbchaos commented Jan 2, 2023

coveralls commented Jan 2, 2023 • edited Loading

Pull Request Test Coverage Report for Build 3844192529

💛 - Coveralls

coveralls commented Jan 2, 2023 •

edited

Loading