Startrekzky commented on code in PR #224:
URL:
https://github.com/apache/incubator-devlake-website/pull/224#discussion_r981514420
##########
docs/DataModels/DevLakeDomainLayerSchema.md:
##########
@@ -428,80 +429,80 @@ Events of pull requests.
### Domain 4 - CI/CD(WIP)
-#### jobs
-
-The CI/CD schedule, not a specific task.
-
-| **field** | **type** | **length** | **description** | **key** |
-| :-------- | :------- | :--------- | :-------------- | :------ |
-| `id` | varchar | 255 | Job id | PK |
-| `name` | varchar | 255 | Name of job | |
-
-#### builds
-
-A build is an execution of a job.
-
-| **field** | **type** | **length** | **description**
| **key** |
-| :------------- | :------- | :--------- |
:--------------------------------------------------------------- | :--------- |
-| `id` | varchar | 255 | Build id
| PK |
-| `job_id` | varchar | 255 | Id of the job this build belongs to
| FK_jobs.id |
-| `name` | varchar | 255 | Name of build
| |
-| `duration_sec` | bigint | | The duration of build in seconds
| |
-| `started_date` | datetime | 3 | Started time of the build
| |
-| `status` | varchar | 255 | The result of build. The values may
be 'success', 'failed', etc. | |
-| `commit_sha` | char | 40 | The specific commit being built on.
Nullable. | |
-
#### cicd_pipelines
A cicd_pipeline is a series of builds that have connections or a standalone
build.
-| **field** | **type** | **length** | **description**
| **key** |
-| :----------------- | :-------------- | :--------- |
:-------------------------------------------------------------- | :------ |
-| `id` | varchar | 255 | This key is generated
based on details from the original plugin | PK |
-| `created_at` | datetime | 3 |
| |
-| `updated_at` | datetime | 3 |
| |
-| `name` | varchar | 255 |
| |
-| `commit_sha` | varchar | 255 |
| |
-| `branch` | varchar | 255 |
| |
-| `repo` | varchar | 255 |
| |
-| `result` | varchar | 100 |
| |
-| `status` | varchar | 100 |
| |
-| `type` | varchar | 100 | to indicate this is CI or
CD | |
-| `duration_sec` | bigint unsigned | |
| |
-| `created_date` | datetime | 3 |
| |
-| `finished_date` | datetime | 3 |
| |
-
-#### cicd_pipeline_repos
-
-A map between cic_pipeline and repo info.
-
-| **field** | **type** | **length** | **description**
| **key** |
-| :----------- | :------- | :--------- |
:-------------------------------------------------------------- | :------ |
-| `commit_sha` | varchar | 255 |
| PK |
-| `branch` | varchar | 255 |
| |
-| `repo_url` | varchar | 255 |
| |
-| `id` | varchar | 255 | This key is generated based on details
from the original plugin | PK |
-| `created_at` | datetime | 3 |
| |
-| `updated_at` | datetime | 3 |
| |
-
+| **field** | **type** | **length** | **description**
| **key** |
+| :-------- | :-------- | :--------
|:----------------------------------------------------------------| :-------- |
+| `id` | varchar | 255) | This key is generated based on details from the
original plugin | PK |
+| `created_at` | datetime | 3) | Devlake
common.NoPKModel | |
+| `updated_at` | datetime | 3) | Devlake
common.NoPKModel | |
+| `_raw_data_params` | varchar | 255) | Devlake
common.NoPKModel.RawDataOrigin | |
+| `_raw_data_table` | varchar | 255) | Devlake
common.NoPKModel.RawDataOrigin | |
+| `_raw_data_id` | bigint unsigned | Devlake common.NoPKModel.RawDataOrigin |
| |
+| `_raw_data_remark` | longtext | Devlake common.NoPKModel.RawDataOrigin
| | |
+| `name` | varchar | 255) |
| |
+| `result` | varchar | 100) | The result of this
task | |
+| `status` | varchar | 100) | The status of this
task | |
+| `type` | varchar | 100) | To indicate if this is a
DEPLOYMENT | |
+| `duration_sec` | bigint unsigned | | how long
does this task take | |
+| `started_date` | datetime | 3) | when did this
task start | |
+| `finished_date` | datetime | 3) | when did this
task finish | |
+| `environment` | varchar | 255) | To indicate if
this is PRODUCTION(dora) | |
Review Comment:
Can we change it to `To indicate the environment in which the task is
running`?
##########
docs/DataModels/DevLakeDomainLayerSchema.md:
##########
@@ -428,80 +429,80 @@ Events of pull requests.
### Domain 4 - CI/CD(WIP)
-#### jobs
-
-The CI/CD schedule, not a specific task.
-
-| **field** | **type** | **length** | **description** | **key** |
-| :-------- | :------- | :--------- | :-------------- | :------ |
-| `id` | varchar | 255 | Job id | PK |
-| `name` | varchar | 255 | Name of job | |
-
-#### builds
-
-A build is an execution of a job.
-
-| **field** | **type** | **length** | **description**
| **key** |
-| :------------- | :------- | :--------- |
:--------------------------------------------------------------- | :--------- |
-| `id` | varchar | 255 | Build id
| PK |
-| `job_id` | varchar | 255 | Id of the job this build belongs to
| FK_jobs.id |
-| `name` | varchar | 255 | Name of build
| |
-| `duration_sec` | bigint | | The duration of build in seconds
| |
-| `started_date` | datetime | 3 | Started time of the build
| |
-| `status` | varchar | 255 | The result of build. The values may
be 'success', 'failed', etc. | |
-| `commit_sha` | char | 40 | The specific commit being built on.
Nullable. | |
-
#### cicd_pipelines
A cicd_pipeline is a series of builds that have connections or a standalone
build.
-| **field** | **type** | **length** | **description**
| **key** |
-| :----------------- | :-------------- | :--------- |
:-------------------------------------------------------------- | :------ |
-| `id` | varchar | 255 | This key is generated
based on details from the original plugin | PK |
-| `created_at` | datetime | 3 |
| |
-| `updated_at` | datetime | 3 |
| |
-| `name` | varchar | 255 |
| |
-| `commit_sha` | varchar | 255 |
| |
-| `branch` | varchar | 255 |
| |
-| `repo` | varchar | 255 |
| |
-| `result` | varchar | 100 |
| |
-| `status` | varchar | 100 |
| |
-| `type` | varchar | 100 | to indicate this is CI or
CD | |
-| `duration_sec` | bigint unsigned | |
| |
-| `created_date` | datetime | 3 |
| |
-| `finished_date` | datetime | 3 |
| |
-
-#### cicd_pipeline_repos
-
-A map between cic_pipeline and repo info.
-
-| **field** | **type** | **length** | **description**
| **key** |
-| :----------- | :------- | :--------- |
:-------------------------------------------------------------- | :------ |
-| `commit_sha` | varchar | 255 |
| PK |
-| `branch` | varchar | 255 |
| |
-| `repo_url` | varchar | 255 |
| |
-| `id` | varchar | 255 | This key is generated based on details
from the original plugin | PK |
-| `created_at` | datetime | 3 |
| |
-| `updated_at` | datetime | 3 |
| |
-
+| **field** | **type** | **length** | **description**
| **key** |
+| :-------- | :-------- | :--------
|:----------------------------------------------------------------| :-------- |
+| `id` | varchar | 255) | This key is generated based on details from the
original plugin | PK |
+| `created_at` | datetime | 3) | Devlake
common.NoPKModel | |
Review Comment:
why is there a `) ` after every length, such as `3)`, `255)`? Can we remove
them?
##########
docs/DataModels/DevLakeDomainLayerSchema.md:
##########
@@ -428,80 +429,80 @@ Events of pull requests.
### Domain 4 - CI/CD(WIP)
-#### jobs
-
-The CI/CD schedule, not a specific task.
-
-| **field** | **type** | **length** | **description** | **key** |
-| :-------- | :------- | :--------- | :-------------- | :------ |
-| `id` | varchar | 255 | Job id | PK |
-| `name` | varchar | 255 | Name of job | |
-
-#### builds
-
-A build is an execution of a job.
-
-| **field** | **type** | **length** | **description**
| **key** |
-| :------------- | :------- | :--------- |
:--------------------------------------------------------------- | :--------- |
-| `id` | varchar | 255 | Build id
| PK |
-| `job_id` | varchar | 255 | Id of the job this build belongs to
| FK_jobs.id |
-| `name` | varchar | 255 | Name of build
| |
-| `duration_sec` | bigint | | The duration of build in seconds
| |
-| `started_date` | datetime | 3 | Started time of the build
| |
-| `status` | varchar | 255 | The result of build. The values may
be 'success', 'failed', etc. | |
-| `commit_sha` | char | 40 | The specific commit being built on.
Nullable. | |
-
#### cicd_pipelines
A cicd_pipeline is a series of builds that have connections or a standalone
build.
-| **field** | **type** | **length** | **description**
| **key** |
-| :----------------- | :-------------- | :--------- |
:-------------------------------------------------------------- | :------ |
-| `id` | varchar | 255 | This key is generated
based on details from the original plugin | PK |
-| `created_at` | datetime | 3 |
| |
-| `updated_at` | datetime | 3 |
| |
-| `name` | varchar | 255 |
| |
-| `commit_sha` | varchar | 255 |
| |
-| `branch` | varchar | 255 |
| |
-| `repo` | varchar | 255 |
| |
-| `result` | varchar | 100 |
| |
-| `status` | varchar | 100 |
| |
-| `type` | varchar | 100 | to indicate this is CI or
CD | |
-| `duration_sec` | bigint unsigned | |
| |
-| `created_date` | datetime | 3 |
| |
-| `finished_date` | datetime | 3 |
| |
-
-#### cicd_pipeline_repos
-
-A map between cic_pipeline and repo info.
-
-| **field** | **type** | **length** | **description**
| **key** |
-| :----------- | :------- | :--------- |
:-------------------------------------------------------------- | :------ |
-| `commit_sha` | varchar | 255 |
| PK |
-| `branch` | varchar | 255 |
| |
-| `repo_url` | varchar | 255 |
| |
-| `id` | varchar | 255 | This key is generated based on details
from the original plugin | PK |
-| `created_at` | datetime | 3 |
| |
-| `updated_at` | datetime | 3 |
| |
-
+| **field** | **type** | **length** | **description**
| **key** |
+| :-------- | :-------- | :--------
|:----------------------------------------------------------------| :-------- |
+| `id` | varchar | 255) | This key is generated based on details from the
original plugin | PK |
+| `created_at` | datetime | 3) | Devlake
common.NoPKModel | |
+| `updated_at` | datetime | 3) | Devlake
common.NoPKModel | |
+| `_raw_data_params` | varchar | 255) | Devlake
common.NoPKModel.RawDataOrigin | |
+| `_raw_data_table` | varchar | 255) | Devlake
common.NoPKModel.RawDataOrigin | |
+| `_raw_data_id` | bigint unsigned | Devlake common.NoPKModel.RawDataOrigin |
| |
+| `_raw_data_remark` | longtext | Devlake common.NoPKModel.RawDataOrigin
| | |
+| `name` | varchar | 255) |
| |
+| `result` | varchar | 100) | The result of this
task | |
+| `status` | varchar | 100) | The status of this
task | |
+| `type` | varchar | 100) | To indicate if this is a
DEPLOYMENT | |
+| `duration_sec` | bigint unsigned | | how long
does this task take | |
+| `started_date` | datetime | 3) | when did this
task start | |
+| `finished_date` | datetime | 3) | when did this
task finish | |
+| `environment` | varchar | 255) | To indicate if
this is PRODUCTION(dora) | |
+
+#### cicd_pipeline_commits
+
+| **field** | **type** | **length** | **description**
| **key** |
+| :-------- | :-------- | :--------
|:----------------------------------------------------------------| :-------- |
+| `pipeline_id` | varchar | 255) | This key is generated based on details from
the original plugin | PK |
+| `created_at` | datetime | 3) | Devlake
common.NoPKModel | |
+| `updated_at` | datetime | 3) | Devlake
common.NoPKModel | |
+| `_raw_data_params` | varchar | 255) | Devlake
common.NoPKModel.RawDataOrigin | |
+| `_raw_data_table` | varchar | 255) | Devlake
common.NoPKModel.RawDataOrigin | |
+| `_raw_data_id` | bigint unsigned | Devlake common.NoPKModel.RawDataOrigin |
| |
+| `_raw_data_remark` | longtext | Devlake common.NoPKModel.RawDataOrigin
| | |
+| `commit_sha` | varchar | 255) | The commit that trigger this pipeline
| PK |
+| `branch` | varchar | 255) | The branch that trigger this pipeline
| |
+| `repo` | varchar | 255) |
| |
+| `repo_id` | varchar | 255) | The repo that this pipeline belongs to
| |
+| `repo_url` | longtext | |
| |
+
+#### cicd_pipeline_relationships
+
+| **field** | **type** | **length** | **description**
| **key** |
+| :-------- | :-------- | :--------
|:--------------------------------------------------| :-------- |
+| `parent_pipeline_id` | varchar | 255) | The pipeline trigger child_pipeline
| PK |
+| `child_pipeline_id` | varchar | 255) | The pipeline that is triggered by
parent_pipeline | PK |
+| `created_at` | datetime | 3) | Devlake
common.NoPKModel | |
+| `updated_at` | datetime | 3) | Devlake
common.NoPKModel | |
+| `_raw_data_params` | varchar | 255) | Devlake
common.NoPKModel.RawDataOrigin | |
+| `_raw_data_table` | varchar | 255) | Devlake
common.NoPKModel.RawDataOrigin | |
+| `_raw_data_id` | bigint unsigned | Devlake common.NoPKModel.RawDataOrigin |
| |
+| `_raw_data_remark` | longtext | Devlake common.NoPKModel.RawDataOrigin
| | |
#### cicd_tasks
A cicd_task is a single job of ci/cd.
-| **field** | **type** | **length** | **description**
| **key** |
-| :----------------- | :-------------- | :--------- |
:-------------------------------------------------------------- | :------ |
-| `id` | varchar | 255 | This key is generated
based on details from the original plugin | PK |
-| `created_at` | datetime | 3 |
| |
-| `updated_at` | datetime | 3 |
| |
-| `name` | varchar | 255 |
| |
-| `pipeline_id` | varchar | 255 |
| |
-| `result` | varchar | 100 |
| |
-| `status` | varchar | 100 |
| |
-| `type` | varchar | 100 | to indicate this is CI
or CD | |
-| `duration_sec` | bigint unsigned | |
| |
-| `started_date` | datetime | 3 |
| |
-| `finished_date` | datetime | 3 |
| |
+| **field** | **type** | **length** | **description**
| **key** |
+| :-------- | :--------
|:-------------------------------|:----------------------------------------------------------------|
:-------- |
+| `id` | varchar | 255) | This key is generated
based on details from the original plugin | PK |
+| `created_at` | datetime | 3) | Devlake
common.NoPKModel | |
+| `updated_at` | datetime | 3) | Devlake
common.NoPKModel | |
+| `_raw_data_params` | varchar | 255) | Devlake
common.NoPKModel.RawDataOrigin | |
+| `_raw_data_table` | varchar | 255) | Devlake
common.NoPKModel.RawDataOrigin | |
+| `_raw_data_id` | bigint unsigned | Devlake common.NoPKModel.RawDataOrigin |
| |
+| `_raw_data_remark` | longtext | Devlake common.NoPKModel.RawDataOrigin
| | |
+| `name` | varchar | 255) |
| |
+| `pipeline_id` | varchar | 255) | The id of
pipeline | |
+| `result` | varchar | 100) | The result of this
task | |
+| `status` | varchar | 100) | The status of this
task | |
+| `type` | varchar | 100) | To indicate if this is a
DEPLOYMENT | |
+| `duration_sec` | bigint unsigned | | how long
does this task take | |
+| `started_date` | datetime | 3) | when did this
task start | |
+| `finished_date` | datetime | 3) | when did this
task finish | |
+| `environment` | varchar | 255) | To indicate if
this is PRODUCTION(dora) | |
Review Comment:
Same as above
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]