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

Add report-info command. #3

Merged
merged 8 commits into from
Dec 28, 2023
Merged

Add report-info command. #3

merged 8 commits into from
Dec 28, 2023

Conversation

woblerr
Copy link
Owner

@woblerr woblerr commented Dec 22, 2023

Addedreport-info command for displaying report file.

Current functional works with gpbackup plugins (basically, with gpbackup_s3_plugin).

For gpbackup_s3_plugin it's enough to specify a valid config for the plugin.
For other plugins, it is needed to specify the full path in the repository to the folder with the file report (--plugin-report-file-path flag).

Process based on restore_data command from gpbackup plugins specification.

Information about program version, executed commandant and etc. was moved on debug log level, not on info.

The documentation has been refactored:

  • moved commands description to separate file COMMANDS.md;
  • added links for each command to README.md.

Aded e2e test for report-info command, which compares the contents of the report file from the src_data directory with the result of the report-info command.

Also some refactoring compose file and Makefile. Now, when running e2e tests, a complete test environment is always prepared for each team. So that there are no intersections.

Display additional information about program version, executed command ant etc on debug log level, not on info.
Add report-info command for displaying report file. In first implementation functional work with gpbackup plugin (basically, with gpbackup_s3_plugin).

Process based on restore_data command from gpbackup plugins specification.
Now the functions throw errors up, checking with which code to terminate the program occurs one time.

Also use defer for close sqlite connection.
@woblerr woblerr added documentation Improvements or additions to documentation enhancement New feature or request labels Dec 22, 2023
@woblerr woblerr self-assigned this Dec 22, 2023
@woblerr woblerr changed the title Add report-info commanf. Add report-info command. Dec 23, 2023
Add new report-info command to documentation.

Refactor docs:
* move commands description to separate file COMMANDS.md;
* add links for each command to README.md.

This unloaded the README file.
This test compares the contents of the report file from the src_data directory with the result of the report-info command.

Also some refactoring compose file and Makefile. Now, when running e2e tests, a complete test environment is always prepared for each team. So that there are no intersections.
@woblerr woblerr marked this pull request as ready for review December 28, 2023 20:28
@woblerr woblerr merged commit 10b0504 into master Dec 28, 2023
4 of 6 checks passed
@woblerr woblerr deleted the add_report_info branch December 28, 2023 20:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant