feat: [DBOPS-1363]: Onboarding two new static steps LiquibaseUpdateSQLStep & LiquibaseRollbackSQLStep #627
Conversation
bikramHarness
requested review from
archit-harness,
harshjain12,
ivan-83,
rohitk-h,
hemanthmantri,
dgarg-harness,
yogesh-chauhan,
vikyathharekal,
abosii-harness,
abhinavsingh1196,
abhinavmittal-harness,
vlica-harness,
puthrayaharness,
anilchowdhury-harness,
deepharness,
utkarshchoubey-harness and
nassergonzalez
as code owners
|
|
bikramHarness
force-pushed
the
dbops-1363
branch
from
f82b305 to
2b73859
Compare
v0/pipeline/steps/common/dbops-liquibase-rollback-sql-step-node.yaml
Outdated
Show resolved
Hide resolved
v0/pipeline/steps/common/dbops-liquibase-update-sql-step-info.yaml
Outdated
Show resolved
Hide resolved
| type: string | ||
| command: | ||
| type: string | ||
| settings: |
There was a problem hiding this comment.
why only its present in update-sql and not rollback-sql
There was a problem hiding this comment.
plus i think it should be globalSettings as per current field
There was a problem hiding this comment.
Same, copied from existing step
v0/pipeline/steps/common/dbops-liquibase-update-sql-step-info.yaml
Outdated
Show resolved
Hide resolved
| properties: | ||
| dbInstance: | ||
| type: string | ||
| dbSchema: | ||
| type: string | ||
| connectorRef: | ||
| type: string | ||
| settings: | ||
| oneOf: | ||
| - $ref: ../../common/parameter-field-map-string-json-node.yaml | ||
| - type: string | ||
| command: | ||
| type: string | ||
| delegateSelectors: | ||
| oneOf: | ||
| - type: array | ||
| items: | ||
| type: string | ||
| - type: string |
There was a problem hiding this comment.
why its getting repeated again properties?
There was a problem hiding this comment.
Took this from the existing LiquibaseCustomCommand,
should we remove this
| - $ref: ../../steps/common/dbops-liquibase-command-step-node.yaml | ||
| - $ref: ../../steps/common/files-upload-step-node.yaml | ||
| - $ref: ../../steps/custom/event-listener-step-node.yaml | ||
| - $ref: ../../steps/common/dbops-liquibase-update-sql-step-node.yaml |
There was a problem hiding this comment.
update not to have liquibase
bikramHarness
force-pushed
the
dbops-1363
branch
from
9b3d94d to
e68edbb
Compare
| - $ref: ../../steps/common/dbops-liquibase-command-step-node.yaml | ||
| - $ref: ../../steps/common/files-upload-step-node.yaml | ||
| - $ref: ../../steps/custom/event-listener-step-node.yaml | ||
| - $ref: ../../steps/common/db-schema-update-sql-step-node.yaml |
There was a problem hiding this comment.
i dont think we need to support here, only cd and custom should be fine
There was a problem hiding this comment.
removed from approval stage
bot2-harness
force-pushed
the
dbops-1363
branch
from
62b845c to
798d5ca
Compare
…LStep & LiquibaseRollbackSQLStep
bot2-harness
force-pushed
the
dbops-1363
branch
from
798d5ca to
38520f8
Compare
Onboarding two new static steps, LiquibaseUpdateSQLStep & LiquibaseRollbackSQLStep