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.

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

Enhance 'chown' call #263

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

FaramosCZ
Copy link

@FaramosCZ FaramosCZ commented Jan 16, 2025

  1. Use common ':' separator instead of unusual '.' one, as disscussed here:
    Add support for building and testing mariadb-container on RHEL10 #262 (comment)

  2. Use the same data type in the whole expression,
    so either only integers '27:0' or preferably only strings 'mysql:root'
    as suggested here:
    Add support for building and testing mariadb-container on RHEL10 #262 (comment)

1) Use common ':' separator instead of unusual '.' one, as disscussed here:
   sclorg#262 (comment)

2) Use the same data type in the whole expression,
   so either only integers '27:0' or preferably only strings 'mysql:root'
   as suggested here:
   sclorg#262 (comment)
Copy link

github-actions bot commented Jan 16, 2025

Pull Request validation

Failed

🔴 Review - Missing review from a member (2 required)

Success

🟢 CI - All checks have passed

@phracek
Copy link
Member

phracek commented Jan 17, 2025

[test]

Copy link

Testing Farm results

namecomposearchstatusstarted (UTC)timelogs
CentOS Stream 9 - 10.11CentOS-Stream-9x86_64✅ passed17.01.2025 09:45:199min 51stest pipeline

Copy link

github-actions bot commented Jan 17, 2025

Testing Farm results

namecomposearchstatusstarted (UTC)timelogs
CentOS Stream 10 - 10.11CentOS-Stream-10x86_64✅ passed17.01.2025 09:45:219min 58stest pipeline
CentOS Stream 9 - 10.5CentOS-Stream-9x86_64✅ passed17.01.2025 09:45:3012min 16stest pipeline
Fedora - 10.5Fedora-latestx86_64✅ passed17.01.2025 09:45:3318min 23stest pipeline

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants