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

Workers coloring implementation. #47

Merged
merged 8 commits into from
Apr 9, 2024

Conversation

HeeManSu
Copy link
Contributor

@HeeManSu HeeManSu commented Mar 30, 2024

  • Logs with deployment names
  • Different colors for all the workers
  • Handling data races

Screenshot from 2024-03-31 05-15-08

@Creatoon
Copy link
Contributor

Creatoon commented Apr 2, 2024

@HeeManSu I tried deploying the apps which were already present in the .metacall/faas/apps, and when one of the apps named mailchimp-dynamodb-example thrown error, you can see that same application id got different color in different lines.

Also Some logs are getting repeated, if you see the starting lines for mailchimp app logs, you will see that it says "Script loaded correctly" for 2 times

Please look into these issues, thanks

I deployed this example: https://github.com/metacall/mailchimp-dynamodb-example

nn

@HeeManSu
Copy link
Contributor Author

HeeManSu commented Apr 6, 2024

@Creatoon You are getting repeated logs (Script loaded correctly) because the Mailchimp example has two languages to run Metacall (Static Files and NodeJS).
They create two JSON files, metacall-file.json for Static Files and metacall-node.json for NodeJS.

The metacall-file.json is getting deployed successfully but metacall-node.json throwing an error.
I think the error is related to dependencies.

image

@viferga
Copy link
Member

viferga commented Apr 9, 2024

@HeeManSu there's conflicsts in:

src/utils/autoDeploy.ts
src/utils/logger.ts

Resolve them please.

@viferga viferga merged commit 1b8b77c into metacall:master Apr 9, 2024
1 check 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.

3 participants