-
Notifications
You must be signed in to change notification settings - Fork 96
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
Update nginx image in LR test workflow #7785
Conversation
Signed-off-by: sk593 <[email protected]>
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #7785 +/- ##
=======================================
Coverage 61.05% 61.05%
=======================================
Files 523 523
Lines 27361 27361
=======================================
Hits 16706 16706
Misses 9184 9184
Partials 1471 1471 ☔ View full report in Codecov by Sentry. |
Is this failing because of the version we're using? or because we're pulling it from dockerhub? |
Based on the error output, it seems like the version is the issue. The error message is specific to the image, not to docker |
We get these errors when we are installing radius also. I think we may need to add the images we are going to use to a list (if there is one) so that we don't see the warnings. Please see this issue: #7717. cc/ @sk593 @rynowak |
Radius functional test overview
Click here to see the list of tools in the current test run
Test Status⌛ Building Radius and pushing container images for functional tests... |
Haven't seen this being reproduced anymore. Going to close this in favor of addressing here: #7717 |
Description
Fixing issues with an old version of nginx during the LRT workflows during the publish TF test recipes step.
1.27
is the latest version released of nginxType of change
Fixes: #4277