Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: add schemaOnly to the import section #2234

Merged
merged 5 commits into from Jun 9, 2023
Merged

feat: add schemaOnly to the import section #2234

merged 5 commits into from Jun 9, 2023

Conversation

armru
Copy link
Member

@armru armru commented Jun 8, 2023

Closes #2228

@github-actions github-actions bot added backport-requested ◀️ This pull request should be backported to all supported releases release-1.18 release-1.19 release-1.20 labels Jun 8, 2023
@github-actions
Copy link
Contributor

github-actions bot commented Jun 8, 2023

❗ By default, the pull request is configured to backport to all release branches.

  • To stop backporting this pr, remove the label: backport-requested ◀️ or add the label 'do not backport'
  • To stop backporting this pr to a certain release branch, remove the specific branch label: release-x.y

@github-actions
Copy link
Contributor

github-actions bot commented Jun 8, 2023

Build Error! No Linked Issue found. Please link an issue or mention it in the body using #<issue_id>

@armru
Copy link
Member Author

armru commented Jun 8, 2023

/test limit=local

@github-actions
Copy link
Contributor

github-actions bot commented Jun 8, 2023

@armru, here's the link to the E2E on CNPG workflow run: https://github.com/cloudnative-pg/cloudnative-pg/actions/runs/5212362117

@github-actions github-actions bot added the ok to merge 👌 This PR can be merged label Jun 8, 2023
armru and others added 4 commits June 9, 2023 10:57
Closes #2228

Signed-off-by: Armando Ruocco <armando.ruocco@enterprisedb.com>
Signed-off-by: Armando Ruocco <armando.ruocco@enterprisedb.com>
Signed-off-by: Gabriele Bartolini <gabriele.bartolini@enterprisedb.com>
Signed-off-by: Jonathan Gonzalez V <jonathan.gonzalez@enterprisedb.com>
Signed-off-by: Jonathan Gonzalez V <jonathan.gonzalez@enterprisedb.com>
@sxd
Copy link
Member

sxd commented Jun 9, 2023

/test depth=push test_level=4 feature_type=importing-databases

@github-actions
Copy link
Contributor

github-actions bot commented Jun 9, 2023

@sxd, here's the link to the E2E on CNPG workflow run: https://github.com/cloudnative-pg/cloudnative-pg/actions/runs/5220778617

@sxd sxd merged commit ad0fdb2 into main Jun 9, 2023
22 checks passed
@sxd sxd deleted the dev/2228 branch June 9, 2023 12:03
cnpg-bot pushed a commit that referenced this pull request Jun 9, 2023
The import feature was previously importing everything, schema and data,
with this new feature we will be able to select only schema or schema and data.

The way is doing this it's using the `section` feature from `pg_dump` and `pg_restore`
commands, we select which section we want to dump/restore, for a full dump and restore
we use `pre-data`, `data` and `post-data` sections, while to import only the schema
the section `pre-data` and `post-data` is used.

Closes #2228

Signed-off-by: Armando Ruocco <armando.ruocco@enterprisedb.com>
Signed-off-by: Gabriele Bartolini <gabriele.bartolini@enterprisedb.com>
Signed-off-by: Jonathan Gonzalez V <jonathan.gonzalez@enterprisedb.com>
Co-authored-by: Gabriele Bartolini <gabriele.bartolini@enterprisedb.com>
Co-authored-by: Jonathan Gonzalez V <jonathan.gonzalez@enterprisedb.com>
(cherry picked from commit ad0fdb2)
cnpg-bot pushed a commit that referenced this pull request Jun 9, 2023
The import feature was previously importing everything, schema and data,
with this new feature we will be able to select only schema or schema and data.

The way is doing this it's using the `section` feature from `pg_dump` and `pg_restore`
commands, we select which section we want to dump/restore, for a full dump and restore
we use `pre-data`, `data` and `post-data` sections, while to import only the schema
the section `pre-data` and `post-data` is used.

Closes #2228

Signed-off-by: Armando Ruocco <armando.ruocco@enterprisedb.com>
Signed-off-by: Gabriele Bartolini <gabriele.bartolini@enterprisedb.com>
Signed-off-by: Jonathan Gonzalez V <jonathan.gonzalez@enterprisedb.com>
Co-authored-by: Gabriele Bartolini <gabriele.bartolini@enterprisedb.com>
Co-authored-by: Jonathan Gonzalez V <jonathan.gonzalez@enterprisedb.com>
(cherry picked from commit ad0fdb2)
cnpg-bot pushed a commit that referenced this pull request Jun 9, 2023
The import feature was previously importing everything, schema and data,
with this new feature we will be able to select only schema or schema and data.

The way is doing this it's using the `section` feature from `pg_dump` and `pg_restore`
commands, we select which section we want to dump/restore, for a full dump and restore
we use `pre-data`, `data` and `post-data` sections, while to import only the schema
the section `pre-data` and `post-data` is used.

Closes #2228

Signed-off-by: Armando Ruocco <armando.ruocco@enterprisedb.com>
Signed-off-by: Gabriele Bartolini <gabriele.bartolini@enterprisedb.com>
Signed-off-by: Jonathan Gonzalez V <jonathan.gonzalez@enterprisedb.com>
Co-authored-by: Gabriele Bartolini <gabriele.bartolini@enterprisedb.com>
Co-authored-by: Jonathan Gonzalez V <jonathan.gonzalez@enterprisedb.com>
(cherry picked from commit ad0fdb2)
armru added a commit that referenced this pull request Jun 9, 2023
The import feature was previously importing everything, schema and data,
with this new feature we will be able to select only schema or schema and data.

The way is doing this it's using the `section` feature from `pg_dump` and `pg_restore`
commands, we select which section we want to dump/restore, for a full dump and restore
we use `pre-data`, `data` and `post-data` sections, while to import only the schema
the section `pre-data` and `post-data` is used.

Closes #2228 

Signed-off-by: Armando Ruocco <armando.ruocco@enterprisedb.com>
Signed-off-by: Gabriele Bartolini <gabriele.bartolini@enterprisedb.com>
Signed-off-by: Jonathan Gonzalez V <jonathan.gonzalez@enterprisedb.com>
Co-authored-by: Gabriele Bartolini <gabriele.bartolini@enterprisedb.com>
Co-authored-by: Jonathan Gonzalez V <jonathan.gonzalez@enterprisedb.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-requested ◀️ This pull request should be backported to all supported releases ok to merge 👌 This PR can be merged release-1.18 release-1.19 release-1.20
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add "schemaOnly" option to "import" facility
5 participants