Skip to content

Conversation

@provokateurin
Copy link
Member

Summary

The logs are very spammy and make it really hard to parse the output if something fails. I also never found them useful when debugging failing tests, so I just removed them entirely (they can still be found in the file if needed).

Checklist

Signed-off-by: provokateurin <kate@provokateurin.de>
@provokateurin provokateurin added this to the Nextcloud 32 milestone Jun 3, 2025
@provokateurin provokateurin requested review from come-nc and susnux June 3, 2025 15:54
@provokateurin provokateurin requested a review from a team as a code owner June 3, 2025 15:54
@provokateurin provokateurin requested review from nfebe and skjnldsv and removed request for a team June 3, 2025 15:54
@provokateurin provokateurin enabled auto-merge June 3, 2025 15:57
@susnux
Copy link
Contributor

susnux commented Jun 3, 2025

(they can still be found in the file if needed)

But not in CI anymore.

@provokateurin
Copy link
Member Author

But were they ever useful there? When the reading the CI logs I have the same problem as locally: It's very hard to understand what is going on with so many logs in between.

Copy link
Contributor

@come-nc come-nc left a comment

Choose a reason for hiding this comment

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

No way, that’s helping me debug #52793 CI failures, which are quite hard to debug already.

@provokateurin
Copy link
Member Author

Ok since people seem to have many different opinions, instead of removing them an option should be introduced and I would suggest that it defaults to off and CI turns it on explicitly?

This was referenced Aug 22, 2025
This was referenced Sep 2, 2025
This was referenced Sep 25, 2025
@skjnldsv skjnldsv modified the milestones: Nextcloud 32, Nextcloud 33 Sep 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants