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 orphan go routines; add waitgroups #510

Merged
merged 1 commit into from
Jun 15, 2024

Conversation

MUzairS15
Copy link

Description

This PR fixes #

Notes for Reviewers

Signed commits

  • Yes, I signed my commits.

Signed-off-by: MUzairS15 <muzair.shaikh810@gmail.com>
@MUzairS15 MUzairS15 requested a review from Jougan-0 June 15, 2024 06:39
@MUzairS15 MUzairS15 changed the title fix reace condition; add waitgroups fix race condition; add waitgroups Jun 15, 2024
@MUzairS15 MUzairS15 changed the title fix race condition; add waitgroups fix orphan go routines; add waitgroups Jun 15, 2024
@MUzairS15
Copy link
Author

// @leecalcote This should fix symlink/ file or directory not found errors in generations.

@MUzairS15 MUzairS15 merged commit ceae4d8 into meshery:master Jun 15, 2024
3 of 4 checks passed
@leecalcote
Copy link
Member

How many models do we expect that this fixes generation for? @Jougan-0

@Jougan-0
Copy link
Contributor

Jougan-0 commented Jun 15, 2024

How many models do we expect that this fixes generation for? @Jougan-0

Currently, there is a difference in just one model during initial testing. I am running some workflows to provide more concurrent answers.(On less rows of spreadsheet)

@Jougan-0
Copy link
Contributor

After running the Model Generator on Control (Old Meshkit) and Expermintal ( This meshkit PR)
Control Workflow ( 534 models)
Experimental Workflow (542 models)
The total amount of extra model generated are 8.

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