Skip to content

Conversation

@dbieber
Copy link
Owner

@dbieber dbieber commented Nov 10, 2024

No description provided.

@dbieber
Copy link
Owner Author

dbieber commented Feb 1, 2025

Here's what should happen:

  1. We build the web app in github actions (not in the qemu emulator) -- this avoids extra dependencies on the image
    And we save the built web app as an artifact.
    This is already done with the "Build web app" and "Archive web app build" steps in build.yml

  2. When building the image, we want to copy the already built web app into the image.
    We get the web app onto the machine in the Download web app build step (already done).
    Next we need to copy it into the image.
    The "non_image_commands" feature is still a work in progress in action.yml (of https://github.com/dbieber/arm-runner-action).

dbieber and others added 4 commits February 24, 2025 20:11
- Updated supervisord.conf to use the correct settings-server directory
- Added detailed logging for the web app copy process
- Added verification step for web app build artifacts

🤖 Generated with Claude Code
Co-Authored-By: Claude <noreply@anthropic.com>
This update incorporates fixes for handling the extra_files_path and
extra_files_mnt_path parameters in the arm-runner-action repository.

🤖 Generated with Claude Code
Co-Authored-By: Claude <noreply@anthropic.com>
- Increased image size to 6000MB
- Added apt-get clean commands to manage disk space
- Updated to arm-runner-action v1.0.8 for path handling improvements

🤖 Generated with Claude Code
Co-Authored-By: Claude <noreply@anthropic.com>
- Updated paths in GitHub Actions workflow to correctly reference the web app files
- Changed from /mnt/web-app-dist to /web-app-dist to match the actual path in the VM

🤖 Generated with Claude Code
Co-Authored-By: Claude <noreply@anthropic.com>
@dbieber
Copy link
Owner Author

dbieber commented Feb 25, 2025

Thanks Claude for helping me get this into a working state!

@dbieber
Copy link
Owner Author

dbieber commented Mar 9, 2025

Going to check this in and then fix forward any issues that arise.

@dbieber dbieber merged commit ca2d8e9 into main Mar 9, 2025
2 checks passed
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