Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Revert "Load analyzers and generators in isolated ALCs in our OOP process" #74985

Merged
merged 1 commit into from
Sep 4, 2024

Conversation

333fred
Copy link
Member

@333fred 333fred commented Sep 4, 2024

Reverts #74780. Running into exceptions in speedometer and blocking VS insertions.

@333fred 333fred requested a review from a team as a code owner September 4, 2024 00:43
@dotnet-issue-labeler dotnet-issue-labeler bot added Area-IDE untriaged Issues and PRs which have not yet been triaged by a lead labels Sep 4, 2024
@333fred 333fred enabled auto-merge September 4, 2024 00:44
@333fred 333fred removed the VSCode label Sep 4, 2024
@CyrusNajmabadi
Copy link
Member

What exceptions are you running into?

@CyrusNajmabadi
Copy link
Member

@dibarbet as well. We sent this pr through speedometer, and it passed. So it's unclear why we'd hit an issue now.

@333fred
Copy link
Member Author

333fred commented Sep 4, 2024

@dibarbet as well. We sent this pr through speedometer, and it passed. So it's unclear why we'd hit an issue now.

Unfortunately, your validation PR didn't pass speedometer. There was an infra issue, and it didn't run:

image

There are lots of filenotfound and unauthorizedaccess exceptions when attempting to load various generators in speedometer, we're reverting for expediency.

@333fred 333fred merged commit 9fbfe9f into main Sep 4, 2024
25 checks passed
@dotnet-policy-service dotnet-policy-service bot added this to the Next milestone Sep 4, 2024
@333fred 333fred deleted the revert-74780-isolatedALC branch September 4, 2024 17:29
@CyrusNajmabadi
Copy link
Member

I'm super confused. I got a pass notification. I even copied it... Ugh.

@jjonescz
Copy link
Member

jjonescz commented Sep 5, 2024

I'm super confused. I got a pass notification. I even copied it... Ugh.

That was just the DDRIT test run passing, not the Speedometer, unfortunately.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Area-IDE untriaged Issues and PRs which have not yet been triaged by a lead
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants