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

Allow forcing ownership even when using --rpm-use-file-permissions #679

Merged
merged 1 commit into from Apr 22, 2014

Conversation

Projects
None yet
1 participant
@jordansissel
Owner

jordansissel commented Apr 22, 2014

The rpm-use-file-permissions setting asks each file who owns it and what
the mode is. Sometimes you only want the mode, or sometimes the user is
not correct for deployment. Now, specifying --rpm-user will force the
owner to be the given user but still respect the file modes gathered
by --rpm-use-file-permissions.

The intent of this change is:

  • --rpm-use-file-permissions still works the same, when alone
  • --rpm-user forces the 'user' owner regardless of the above
  • --rpm-group forces the 'group' owner regardless of the above

This was identified by https://logstash.jira.com/browse/LOGSTASH-2012

Allow forcing ownership even when using --rpm-use-file-permissions
The rpm-use-file-permissions setting asks each file who owns it and what
the mode is. Sometimes you only want the mode, or sometimes the user is
not correct for deployment. Now, specifying --rpm-user will force the
owner to be the given user but still respect the file modes gathered
by --rpm-use-file-permissions.

The intent of this change is:

* --rpm-use-file-permissions still works the same, when alone
* --rpm-user forces the 'user' owner regardless of the above
* --rpm-group forces the 'group' owner regardless of the above

This was identified by https://logstash.jira.com/browse/LOGSTASH-2012

jordansissel added a commit that referenced this pull request Apr 22, 2014

Merge pull request #679 from jordansissel/fix-rpm-permissions-and-use…
…r-settings

Allow forcing ownership even when using --rpm-use-file-permissions

@jordansissel jordansissel merged commit a5f36de into master Apr 22, 2014

@jordansissel jordansissel deleted the fix-rpm-permissions-and-user-settings branch Apr 22, 2014

prof-milki pushed a commit to prof-milki/xpm that referenced this pull request Dec 18, 2014

jls
Merge pull request jordansissel#679 from jordansissel/fix-rpm-permiss…
…ions-and-user-settings

Allow forcing ownership even when using --rpm-use-file-permissions

prof-milki pushed a commit to prof-milki/xpm that referenced this pull request Dec 27, 2014

Merge pull request jordansissel#679 from jordansissel/fix-rpm-permiss…
…ions-and-user-settings

Allow forcing ownership even when using --rpm-use-file-permissions

jordansissel added a commit that referenced this pull request Apr 24, 2015

Merge pull request #679 from jordansissel/fix-rpm-permissions-and-use…
…r-settings

Allow forcing ownership even when using --rpm-use-file-permissions

jordansissel added a commit that referenced this pull request Jun 20, 2016

Merge pull request #679 from jordansissel/fix-rpm-permissions-and-use…
…r-settings

Allow forcing ownership even when using --rpm-use-file-permissions
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment