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

Fix scp error in web app deployment #2155

Merged
merged 1 commit into from
Feb 24, 2025

Conversation

brandonpayton
Copy link
Member

Motivation for the change, related issues

We're currently experiencing an scp error when attempting to transfer mime-types.json as a single file to a the destination '~/website-deployment/'.
https://github.com/WordPress/wordpress-playground/actions/runs/13481525026/job/37667340559#step:7:16325

It seems like the home dir resolution is failing altogether or inconsistent between scp and regular ssh login. This was working before. There may have been a change in the hosting environment that inadvertently affected this script.

The previous step copies a directory to the same destination, and that is working fine.

Implementation details

As a workaround, this PR just copies mime-types.json into the local directory that is successfully copied to the server. This way, the file is transferred to the same location, and we avoid this single-file weirdness with scp.

Testing Instructions (or ideally a Blueprint)

  • CI
  • Try workflow after merging. The workflow is already currently broken, and it is faster to test this way.

@brandonpayton brandonpayton added [Type] Bug An existing feature does not function as intended [Aspect] Website labels Feb 24, 2025
@brandonpayton brandonpayton requested a review from a team February 24, 2025 16:11
@brandonpayton brandonpayton self-assigned this Feb 24, 2025
@brandonpayton brandonpayton merged commit 7a1ebb9 into trunk Feb 24, 2025
2 checks passed
@brandonpayton brandonpayton deleted the fix-public-website-deployment branch February 24, 2025 16:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Aspect] Website [Type] Bug An existing feature does not function as intended
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

1 participant