Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Improve network integration testing #1348

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.

Sign up for GitHub

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

Jump to bottom
Merged
kba merged 11 commits into from
Dec 19, 2025
Merged

Improve network integration testing #1348

kba merged 11 commits into from
Dec 19, 2025

Conversation

@MehmedGIT
Copy link
Contributor

@MehmedGIT MehmedGIT commented Dec 17, 2025 •
edited
Loading

This PR:

  • resolves: avoid logging credentials in deployer #1346
  • updates RabbitMQ service version from 3.12 to the latest 4.2
  • prints the log file of a processing job integration test in both 5_processing_server and 6_client
  • prints a summary of the processing jobs of workflow integration tests in both 5_processing_server and 6_client. For each failed processing job, retrieves the log file of the job and prints it (in case there are any failing).

@MehmedGIT MehmedGIT mentioned this pull request Dec 17, 2025
Merged
@MehmedGIT MehmedGIT requested review from bertsky and kba December 17, 2025 02:08
@MehmedGIT MehmedGIT force-pushed the network_test_job_logs branch from 62c4b53 to 98a0ea3 Compare December 17, 2025 02:25
kba
kba approved these changes Dec 17, 2025
View reviewed changes
Copy link
Member

@kba kba left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks

bertsky
bertsky reviewed Dec 17, 2025
View reviewed changes
bertsky
bertsky approved these changes Dec 17, 2025
View reviewed changes
Copy link
Collaborator

@bertsky bertsky left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great – thanks!

@MehmedGIT
Copy link
Contributor Author

MehmedGIT commented Dec 17, 2025

@kba can be merged with the next release.

@kba kba mentioned this pull request Dec 19, 2025
Closed
@kba kba merged commit 90ffae0 into master Dec 19, 2025
17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@kba kba kba approved these changes

@bertsky bertsky bertsky approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

avoid logging credentials in deployer

4 participants

@MehmedGIT @kba @bertsky