1449 Integration test for triggering build in multiple authorized nodes environment #1643
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
|
|
fishseabowl
reviewed
Feb 21, 2023
| pidlist="$pidlist$pid"; | ||
| done | ||
| echo -e "\nPyrsia processes:" | ||
| ps -u -q $pidlist |
There was a problem hiding this comment.
When I run the tests on Mac OS, it reports:
Pyrsia processes:
ps: No user named '-q'
Could you help to look into it? Thanks
There was a problem hiding this comment.
Added fix, @fishseabowl could you please to try it again?
fishseabowl
reviewed
Feb 21, 2023
|
|
||
| kill_processes | ||
|
|
||
| echo ""; echo "All test scenarios are done successfully!" |
There was a problem hiding this comment.
It is better to add a newline at the end of the file
There was a problem hiding this comment.
Hi @fishseabowl
merged last changes from 'main'-branch and found that increasing of timeout for awaiting result 'pyrsia inspect-log docker --image' is required.
Please take a look.
fishseabowl
reviewed
Feb 21, 2023
|
Is it intentionally separated from the integration tests repo |
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.
Part of: #1449
Test is provided as bash script and consists of following steps:
The logs and pids of started nodes are present inside directory: /tmp/pyrsia-manual-tests/.log and /tmp/pyrsia-manual-tests/.pid
Test is located in pyrsia repository' by path: tests/test_multiple_authorized_nodes.sh
It requires to specify following paramters:
Example:
test_multiple_authorized_nodes.sh $PYRSIA_HOME $PYRSIA_BUILD_PIPELINE_HOME