-
Notifications
You must be signed in to change notification settings - Fork 78
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
process-user-data: Fix the assignment of config filenames #1941
process-user-data: Fix the assignment of config filenames #1941
Conversation
Tested with docker provider. Sample image for test - |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code LGTM. Thanks
For reference the docker-provider e2e tests are working here for me except for the MTLS test that Pradipta has identified is due to the the use of the curl image and we don't think is a blocker. The podvm build failed in the CI e2e tests due to docker rate limits :( |
The config files were incorrectly assigned resulting in agent-config.toml not being created. Signed-off-by: Pradipta Banerjee <pradipta.banerjee@gmail.com>
7763928
to
2e313eb
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks.
I manually verified the PR on Azure. @wainersm verified on AWS. |
39a5288
into
confidential-containers:main
The config files were incorrectly assigned resulting in agent-config.toml not being created.