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 process compose logs #2544

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

imabdulbasit
Copy link
Contributor

@imabdulbasit imabdulbasit commented Feb 5, 2025

process compose logs uploaded is actually the demo native script and not the actual logs.
This PR defines the path in process-compose.yml where the logs are stored and changes the workflow to upload the logs.

@imabdulbasit imabdulbasit changed the base branch from main to ab/lightweight2 February 5, 2025 22:49
@imabdulbasit imabdulbasit marked this pull request as draft February 5, 2025 22:50
Base automatically changed from ab/lightweight2 to main February 5, 2025 23:19
@imabdulbasit imabdulbasit changed the title Ab/fix process compose logs fix process compose logs Feb 5, 2025
@imabdulbasit imabdulbasit force-pushed the ab/fix-process-compose-logs branch from 8aac2d5 to 5f7fea3 Compare February 5, 2025 23:21
@imabdulbasit imabdulbasit marked this pull request as ready for review February 5, 2025 23:23
Copy link
Collaborator

@sveitser sveitser left a comment

Choose a reason for hiding this comment

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

I think for locally running maybe not writing to /tmp is better?

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