feat(ui): ignore errors that occured during logical data/dump restore
Description
Add additional checkboxes to ignore errors that occurred during logical data dump (logicalDump
) or restore (logicalRestore
)
Related issue
Examples
Additional checkboxes on the Configuration form
Checklist
-
MR description has been reviewed -
MR changes are functionally tested -
MR does NOT have API/CLI changes OR there are API/CLI changes and they have been reviewed & DOCS ARE ADJUSTED (reference doc, etc) -
MR does NOT have UI changes OR there are UI changes and they have been reviewed & UX IS REVIEWED
Merge request reports
Activity
requested review from @akartasov and @NikolayS
assigned to @Adrinlol
mentioned in merge request !699 (merged)
- Resolved by Lasha Kakabadze
Thanks, @Adrinlol
The checkboxes should go to the proper section –
Section "retrieval"
.Additionally, I've noticed a mess of a little bit for the dump and restore options. Let's clarify it and fix?
-- there is a subtitle
Subsection "retrieval.spec.logicalDump"
, but then we have a mix of both dump and restore options.There source of truth is here: https://postgres.ai/docs/reference-guides/database-lab-engine-configuration-reference, and here https://gitlab.com/postgres-ai/database-lab/-/blob/master/engine/configs/config.example.logical_generic.yml
We have two subsections:
-
retrieval.spec.logicalDump
which can be found here in the example config: https://gitlab.com/postgres-ai/database-lab/-/blob/master/engine/configs/config.example.logical_generic.yml#L196. All dump options should be inside this subsection. -
retrieval.spec.logicalRestore
https://gitlab.com/postgres-ai/database-lab/-/blob/master/engine/configs/config.example.logical_generic.yml#L274. All restore options should be inside this section. Let's add this subsection, and move restore options there (pg_restore jobs, pg_restore customOptions)
And the new checkboxes should follow the same rule.
Edited by Nikolay Samokhvalov -
added 1 commit
- b53020d3 - add restore subsection, move inputs and checkboxes
- Resolved by Lasha Kakabadze
@Adrinlol the updated screenshots look good
mentioned in commit 8d97c742
mentioned in issue #501 (closed)
mentioned in merge request !729 (closed)
mentioned in merge request !730 (merged)