-
Notifications
You must be signed in to change notification settings - Fork 1.1k
Enable container test execution #48763
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
Conversation
7446be0
to
0d90f8c
Compare
We're encountering test failures because
@dotnet/domestic-cat @marcpopMSFT — is there a recommended workaround or best practice for handling this? |
Can you clarify which part is going wrong? There's normally a metadata folder somewhere next to process path, so we're expecting to be able to access the spot right next to dotnet.exe or dotnet on linux. You can move the standard workload metadata folder that we write to if you set user local. |
Thanks for the follow-up! Just to clarify, the issue is happening in the context of the two tests in this PR:
These tests are failing in our containerized CI environment due to dotnet workload restore attempting to write to a read-only path:
I tried setting the following environment variables in the test setup:
However, the error still occurs, which suggests that some part of the workload infrastructure is still trying to write to the default location. Is there another recommended way to fully redirect or disable workload-related writes in this scenario? |
I believe that if you create a userlocal file, that is, a 0-byte file here (like C:\Program Files\dotnet\metadata\workloads\10.0.100\userlocal), it will only write to a directory under your DOTNET_CLI_HOME folder. I'm not really sure what's being written. Are these from updating advertising manifests? If so, perhaps we could disable that? |
Thanks, that’s super helpful! I wasn’t aware of the userlocal file trick — I’ll try creating a 0-byte file at:
|
463da35
to
e7403bf
Compare
@Forgind Thanks for the suggestion! I looked into the Unless there's a better workaround, I might have to temporarily skip these two. |
I looked into the code a bit, and the only thing that I can think of that should be being written is advertising manifests, but unfortunately there isn't really an option to disable that, so I don't know that there's a 'natural' answer. You could probably add a way to opt out of updating advertising manifests from the workload restore command, but I think that would be too much effort for something like this, so I think disabling it is the right choice. |
No description provided.