-
Notifications
You must be signed in to change notification settings - Fork 112
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
Adds missing "mode" to tasks with copy and file modules #1425
Adds missing "mode" to tasks with copy and file modules #1425
Conversation
Merge Failed. This change or one of its cross-repo dependencies was unable to be automatically merged with the current state of its repository. Please rebase the change and upload a new patchset. |
@@ -23,6 +23,7 @@ | |||
- name: Copy generated repos to /etc/yum.repos.d directory | |||
become: true | |||
ansible.builtin.copy: | |||
mode: "0644" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
beware, this is a directory, it needs "0755"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ack, updated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good @frenzyfriday, thank you!
Are you going to enable risky-file-permissions
lint so we can see if this now passes CI on this patch? [1]
Merge Failed. This change or one of its cross-repo dependencies was unable to be automatically merged with the current state of its repository. Please rebase the change and upload a new patchset. |
d3e6990
to
5ecf890
Compare
Build failed (check pipeline). Post https://review.rdoproject.org/zuul/buildset/918e3cb127064a8b9f0dfd8357597281 ❌ openstack-k8s-operators-content-provider NODE_FAILURE Node request 100-0007433115 failed in 0s |
rebased to check current status |
recheck |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
cifmw-molecule-install_openstack_ca
is flaky, not a failure due to this patch, I opened a patch to try improve here
The other two failures are an easy fix, I left a suggestion.
This patch is almost ready to get in.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
could only find one this time :)
and directories now look consistent at 0755 I couldn't find any more 0775
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
Thanks @frenzyfriday
recheck |
1 similar comment
recheck |
recheck after dequeuing stuck zuul jobs |
/lgtm |
1f9756b
into
openstack-k8s-operators:main
/cherry-pick 18.0.0-proposed |
@sathlan: new pull request created: #2340 In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
As a pull request owner and reviewers, I checked that: