feat: multi namespace deployment support #210

deryrahman · 2022-03-09T04:11:02Z

merge proton changes & update PROTON_COMMIT hash

api/handler/v1beta1/job.go

config/loader.go

coveralls · 2022-03-14T10:57:21Z

Pull Request Test Coverage Report for Build 2021690834

185 of 201 (92.04%) changed or added relevant lines in 4 files are covered.
55 unchanged lines in 2 files lost coverage.
Overall coverage increased (+0.2%) to 72.87%

Changes Missing Coverage	Covered Lines	Changed/Added Lines	%
config/loader.go	39	45	86.67%
config/config.go	0	10	0.0%

Files with Coverage Reduction	New Missed Lines	%
ext/datastore/bigquery/bigquery.go	8	51.22%
models/plugin.go	47	2.22%

Totals
Change from base Build 2014739380:	0.2%
Covered Lines:	5345
Relevant Lines:	7335

💛 - Coveralls

…ource

…ing deployment

…ervice

… proto

…ource

api/handler/v1beta1/resource.go

cmd/commands.go

cmd/deploy.go

config/loader_test.go

mock/job.go

mock/proto_adapter.go

mock/runtime.go

… print

deryrahman linked an issue Mar 9, 2022 that may be closed by this pull request

CLI & API should Support Deployment of Single/Multiple Namespaces #184

Closed

deryrahman force-pushed the feat-deploy-multi-namespace branch from 57b021c to 6da6645 Compare March 14, 2022 07:20

sbchaos reviewed Mar 14, 2022

View reviewed changes

api/handler/v1beta1/job.go Outdated Show resolved Hide resolved

api/handler/v1beta1/job.go Outdated Show resolved Hide resolved

sbchaos reviewed Mar 14, 2022

View reviewed changes

config/loader.go Outdated Show resolved Hide resolved

sbchaos mentioned this pull request Mar 15, 2022

refactor: remove unused params and enable strict check #218

Merged

deryrahman force-pushed the feat-deploy-multi-namespace branch 2 times, most recently from 5cde709 to f960d62 Compare March 16, 2022 09:15

irainia and others added 22 commits March 21, 2022 15:37

refactor: update config and resource deployment mechanism

3630b8e

refactor: restructure deployment of project, namespaces, job, and res…

2d03c54

…ource

refactor: fix bug when sending request and response to the server dur…

3bebfd3

…ing deployment

refactor: config to load multiple namespaces

76baef9

test: load config for optimus

b8169bf

test: load config for namespaces

caffe70

fix: load duplicated namespaces

eb2e8b6

fix: error when testing job deployment

8422305

fix: requirement to set the namespace on secret

4359922

fix: generated mock by mockery

50da107

fix: bug command cannot be properly initialized

1eb63c6

refactor: rework config loader to minimise side-effect

92cceee

fix: bug when deploying job and resource from client and server side

ee79753

fix: linter

1b6a486

refactor: test on test deploy job specification when success

5841ad0

test: add test when there's grpc recv error

abe7b2b

test: add test when namespace get error

4328d4e

test: add test when adapt from proto or job spec save is failed

539e6ed

test: add test when job service keep only error

e4c5c70

test: add test when job service sync error

e6e29fa

fix: linter context in struct

c09f288

refactor: job spec deploy test

8b1d9d6

deryrahman and others added 8 commits March 21, 2022 15:39

test: add test when deploy resource specification success

a4e8dd9

test: add test when deploy resource specification error on grpc receiver

1a06548

test: add test when deploy resource specification error on namespaceS…

3f810db

…ervice

test: add test when deploy resource specification error on adapt from…

0919ed3

… proto

test: add test when deploy resource specification error on update res…

45084d4

…ource

test: add test when get job specification success and failed

e47a8cf

test: add test when get job specification failed & remove unused test

6ab4a39

refactor: simplify how to store and load config for multiple namespaces

ec4ff2d

irainia force-pushed the feat-deploy-multi-namespace branch from 3007368 to ec4ff2d Compare March 21, 2022 08:39

fix: linter issue for namespace pointer

a503334