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.

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

Print the integration test server logs in the actions #5253

Closed
joeldierkes opened this issue Feb 18, 2022 · 1 comment · Fixed by #5254 or #5337
Closed

Print the integration test server logs in the actions #5253

joeldierkes opened this issue Feb 18, 2022 · 1 comment · Fixed by #5254 or #5337
Assignees
@joeldierkes
Labels
backport Issues which are backported to additional releases enhancement Testing
Milestone
1.27.7

Comments

@joeldierkes
Copy link
Contributor

Motivation

In #5249 the server logs were needed to track down an issue. We should print them by default to have more insight in case of a bug.

Modification

Print the server logs when the integration tests are running in the actions. They should be printed after the test's run.
@joeldierkes joeldierkes self-assigned this Feb 18, 2022
joeldierkes pushed a commit to joeldierkes/rucio that referenced this issue Feb 18, 2022
Testing: Print error logs in the integration test actions rucio#5253
385e8da
@joeldierkes joeldierkes linked a pull request Feb 18, 2022 that will close this issue
Testing: Print error logs in the integration test actions #5253 #5254
Merged
joeldierkes pushed a commit to joeldierkes/rucio that referenced this issue Feb 21, 2022
Testing: Print error logs in the integration test actions rucio#5253
44fe883
bari12 added a commit that referenced this issue Feb 22, 2022
@bari12
Merge pull request #5254 from joeldierkes/feature-5253-Print_the_inte…
2682732 
…gration_test_server_logs_in_the_actions

Testing: Print error logs in the integration test actions #5253
bari12 pushed a commit that referenced this issue Feb 22, 2022
@bari12
Testing: Print error logs in the integration test actions #5253
2c79dea
bari12 pushed a commit that referenced this issue Feb 22, 2022
@bari12
Testing: Print error logs in the integration test actions #5253
d74f801
@bari12 bari12 added the backport Issues which are backported to additional releases label Feb 22, 2022
@bari12
Copy link
Member

bari12 commented Feb 22, 2022

Backport 1.26.12

@bari12 bari12 added this to the 1.27.6 milestone Feb 22, 2022
joeldierkes pushed a commit to joeldierkes/rucio that referenced this issue Feb 22, 2022
Testing: Always print error logs in the integration test actions ruci…
8aca58c 
…o#5253
piperov pushed a commit to piperov/rucio that referenced this issue Feb 25, 2022
@piperov
Testing: Print error logs in the integration test actions rucio#5253
bece649
joeldierkes pushed a commit to joeldierkes/rucio that referenced this issue Mar 8, 2022
Always print the integration test logs rucio#5253
a95f9f5 
If any job before the `Print server error logs` fails, the job itself will not
run. Since these are the cases in which the logs can provide additional
information, they should be printed.
@joeldierkes joeldierkes linked a pull request Mar 8, 2022 that will close this issue
Always print the integration test logs #5253 #5337
Merged
bari12 added a commit that referenced this issue Mar 9, 2022
@bari12
Merge pull request #5337 from joeldierkes/feature-5253-Print_the_inte…
ec4563e 
…gration_test_server_logs_in_the_actions_2

Always print the integration test logs #5253
bari12 pushed a commit that referenced this issue Mar 9, 2022
@bari12
Always print the integration test logs #5253
fce4e2b 
If any job before the `Print server error logs` fails, the job itself will not
run. Since these are the cases in which the logs can provide additional
information, they should be printed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@joeldierkes joeldierkes

Labels
backport Issues which are backported to additional releases enhancement Testing
Projects
None yet
Milestone
1.27.7
2 participants
@bari12 @joeldierkes