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

Unify remap user tasks from os_setup role #16

Merged
merged 1 commit into from
Nov 15, 2023

Conversation

kysrpex
Copy link
Contributor

@kysrpex kysrpex commented Nov 15, 2023

Remove remap user tasks for CentOS 8 (roles/os_setup/tasks/remap_user/centos8.yml) and use the RHEL 9 tasks file for all RHEL versions.

The tasks for CentOS 8 do not work with other operating system releases, while the tasks for RHEL 9 are more general and should work across operating system versions.

Remove remap user tasks for CentOS 8 (roles/os_setup/tasks/remap_user/centos8.yml) and use the RHEL 9 tasks file for all RHEL versions.

The tasks for CentOS 8 do not work with other operating system releases, while the tasks for RHEL 9 are more general and should work across operating system versions.
@kysrpex kysrpex merged commit 672561c into usegalaxy-eu:main Nov 15, 2023
1 check passed
@kysrpex kysrpex deleted the remap_user_single_task branch November 15, 2023 13:07
@kysrpex
Copy link
Contributor Author

kysrpex commented Dec 8, 2023

@mira-miracoli wrote:

What was the issue again with the OS major version separated remap_user.py? (how it was before #16 )

There were several issues with the status before #16:

@kysrpex
Copy link
Contributor Author

kysrpex commented Dec 8, 2023

@mira-miracoli To add to the previous message, in #16 I have just blindly trusted roles/os_setup/tasks/remap_user/rhel9.yml to be correct for all our use cases (including machines where the user and/or 999 is not named galaxy), which is not the case. Hence #18.

@kysrpex
Copy link
Contributor Author

kysrpex commented Dec 8, 2023

@mira-miracoli I think in fact you already wrote about all this yourself here, sorry for not realizing earlier.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants