-
Notifications
You must be signed in to change notification settings - Fork 35
Updating of CI-tests #197
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
Merged
dmitry-lipetsk
merged 47 commits into
postgrespro:master
from
dmitry-lipetsk:D20250226_001--ci_and_ssh
Mar 3, 2025
Merged
Updating of CI-tests #197
dmitry-lipetsk
merged 47 commits into
postgrespro:master
from
dmitry-lipetsk:D20250226_001--ci_and_ssh
Mar 3, 2025
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
A problem was detected in container with Ubuntu 24.04 tests works with "/usr/bin/pg_config" but real pg_config is "/usr/lib/postgresql/17/bin/pg_config" To resovle this problem we will call "pg_config --bindir" and use it result for BINDIR and PG_CONFIG.
… into D20250226_001--ci_and_ssh
Let's use /pg/testgres/run_tests.sh directly.
curl is installing twice.
…6_001--ci_and_ssh
…6_001--ci_and_ssh
[del] musl-dev [del] mc
/pg/testgres/run_tests.sh is used directly.
- We do multiple attempts to read pid file. - We process a case when we see that node is stopped between test and read. - We process a case when pid-file is empty.
Assert is added.
…0250226_001--ci_and_ssh
- New parameters "ignore_errors" is added. Default value is False. - Asserts are added.
…0250226_001--ci_and_ssh
This method returns nothing (None).
…0250226_001--ci_and_ssh
When we define a template we have to use "-t" option. It forces mktemp to return a path instead name. The following methods of RemoteOperations are fixed: - mkdtemp - mkstemp
…to D20250226_001--ci_and_ssh
…0250226_001--ci_and_ssh
Main - We check only an exit code to detect an error. - If someone utility returns a result through an exit code, a caller side should set ignore_errors=true and process this case itself. - If expect_error is true and no errors occurred, we raise an InvalidOperationException.
…d' into D20250226_001--ci_and_ssh
The folder "home/postgres" is not required now.
Let's rollback the new code to avoid problems with probackup2' tests.
…d' into D20250226_001--ci_and_ssh
Let's test a real pg_config.
…' into D20250226_001--ci_and_ssh
- these tests configure logging wrong and create the conflicts with root logger - these tests (local and remote) conflict with each other
…o D20250226_001--ci_and_ssh
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This patch request has two goals: