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

[CICD-29] Stop erroneous build failures due to attempt to rsync mu-plugin subdirectories #13

Merged
merged 3 commits into from
Jun 30, 2022

Conversation

daniel-savo
Copy link
Contributor

JIRA Ticket

CICD-29

What Are We Doing Here

When adding all install files to a repository, the GHA build was failing due to rsync attempting to update the mu-plugin/wpe-cache-plugin subdirectories:

rsync: failed to set times on "/home/wpe-user/sites/dscicdtest1/wp-content/mu-plugins/wpe-cache-plugin": Operation not permitted (1)
rsync: failed to set times on "/home/wpe-user/sites/dscicdtest1/wp-content/mu-plugins/wpe-cache-plugin/css": Operation not permitted (1)
rsync: failed to set times on "/home/wpe-user/sites/dscicdtest1/wp-content/mu-plugins/wpe-cache-plugin/js": Operation not permitted (1)
rsync: failed to set times on "/home/wpe-user/sites/dscicdtest1/wp-content/mu-plugins/wpe-cache-plugin/js/components": Operation not permitted (1)
rsync: failed to set times on "/home/wpe-user/sites/***/wp-content/mu-plugins/wpe-cache-plugin/js/dist": Operation not permitted (1)
rsync: failed to set times on "/home/wpe-user/sites/***/wp-content/mu-plugins/wpe-cache-plugin/js/services": Operation not permitted (1)
rsync: failed to set times on "/home/wpe-user/sites/***/wp-content/mu-plugins/wpe-cache-plugin/js/utils": Operation not permitted (1)
rsync: failed to set times on "/home/wpe-user/sites/***/wp-content/mu-plugins/wpe-cache-plugin/security": Operation not permitted (1)
rsync: failed to set times on "/home/wpe-user/sites/***/wp-content/mu-plugins/wpe-cache-plugin/view": Operation not permitted (1)

This PR fixes the excludes paths to ensure this doesn't happen.

@daniel-savo daniel-savo requested a review from a team as a code owner June 30, 2022 14:54
Copy link
Contributor

@marksabbath marksabbath left a comment

Choose a reason for hiding this comment

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

LGTM!

@daniel-savo daniel-savo merged commit dc3acfe into main Jun 30, 2022
@daniel-savo daniel-savo deleted the CICD-29 branch June 30, 2022 14:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants