Skip to content

Conversation

@artonge
Copy link
Contributor

@artonge artonge commented Sep 5, 2025

Composer does not like being run as root.

Composer does not like being run as root.

Signed-off-by: Louis Chemineau <louis@chmn.me>
@artonge artonge requested a review from a team as a code owner September 5, 2025 10:00
@artonge artonge requested review from CarlSchwan, leftybournes and provokateurin and removed request for a team September 5, 2025 10:00
@artonge artonge self-assigned this Sep 5, 2025
@artonge artonge added 3. to review Waiting for reviews tests Related to tests labels Sep 5, 2025
@artonge artonge added this to the Nextcloud 33 milestone Sep 5, 2025
@provokateurin
Copy link
Member

Why would the user in the docker container be root? It always worked as-is for me 🤔

@artonge
Copy link
Contributor Author

artonge commented Sep 5, 2025

Why would the user in the docker container be root? It always worked as-is for me 🤔

Not sure. I am using podman, so might lead to differences. But I though podman was non-root by default.

Edit: got confused, it just does not need root to run. Both seems to default to root if nothing else is set.

@artonge artonge requested a review from come-nc September 8, 2025 13:54
Copy link
Member

@nickvergessen nickvergessen left a comment

Choose a reason for hiding this comment

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

I have no clue about the docker one, but looks okay.

@provokateurin provokateurin merged commit 0baaebd into master Sep 16, 2025
148 checks passed
@provokateurin provokateurin deleted the artonge/feat/run_integration_tests_as_www-data branch September 16, 2025 08:35
@skjnldsv skjnldsv modified the milestones: Nextcloud 33, Nextcloud 32 Oct 2, 2025
@nextcloud-bot nextcloud-bot mentioned this pull request Jan 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews tests Related to tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants