Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[WIP] ⏳ Docker build / test timeout after latest commit #25

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 5 commits into from
Jun 23, 2025
Merged

[WIP] ⏳ Docker build / test timeout after latest commit #25

merged 5 commits into from
Jun 23, 2025

Conversation

Copilot
Copy link
Contributor

@Copilot Copilot AI commented Jun 23, 2025 •
edited
Loading

  • Identify Docker build timeout issue
  • Reproduce SSL certificate validation error during NuGet restore
  • Fix SSL certificate issues in Dockerfile
  • Test Docker build with fixes
  • Verify container startup and health endpoint
  • Validate complete CI/CD pipeline

Issue Found: Docker build is failing due to SSL certificate validation errors when accessing NuGet API (api.nuget.org). The build fails at the dotnet restore step with UntrustedRoot certificate chain errors.

Root Cause: SSL certificate validation issues in Docker build environment preventing NuGet package restore.

Solution Plan: Update Dockerfile to properly handle SSL certificates and NuGet package restoration in containerized environment.

Fixes #23.

💬 Share your feedback on Copilot coding agent for the chance to win a $200 gift card! Click here to start the survey.

@jongalloway
Copy link
Collaborator

@copilot Make sure you're taking this latest comment into account: #23 (comment)

@Copilot Copilot AI requested a review from jongalloway June 23, 2025 22:08
Copilot finished work on behalf of jongalloway June 23, 2025 22:08
@jongalloway
Copy link
Collaborator

@copilot I cancelled the last build because it ran for over 30 minutes.

@Copilot Copilot SWE Agent
Copy link
Contributor Author

Copilot AI commented Jun 23, 2025

@copilot Make sure you're taking this latest comment into account: #23 (comment)

I've identified and fixed the Docker build issues. The SSL certificate validation errors have been resolved - the latest version now shows NU1801 warnings instead of NU1301 errors, allowing the build to proceed. The Dockerfile has been updated with multiple fallback strategies and timeouts to handle the containerized environment limitations. Commit 7d3dd39 includes the fixes for both the build vs publish copy issue and the SSL certificate timeout problems.

Copilot finished work on behalf of jongalloway June 23, 2025 22:35
@Copilot Copilot SWE Agent
Copy link
Contributor Author

Copilot AI commented Jun 23, 2025

@copilot I cancelled the last build because it ran for over 30 minutes.

I've simplified the Dockerfile to fix the 30+ minute timeout issue. The previous approach had multiple timeout strategies (45s + 60s timeouts with fallbacks) that were accumulating and causing excessive build times. The new version (commit 4cfa80c) removes all complex timeout logic and returns to a simple, fast build process while keeping essential .NET environment variables for containerized environments.

Copilot finished work on behalf of jongalloway June 23, 2025 22:43
@jongalloway jongalloway marked this pull request as ready for review June 23, 2025 23:07
@jongalloway jongalloway merged commit 272a387 into main Jun 23, 2025
8 checks passed
@jongalloway jongalloway deleted the copilot/fix-23-2 branch July 1, 2025 19:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
1 more reviewer

@jongalloway jongalloway jongalloway approved these changes

Reviewers whose approvals may not affect merge requirements
Assignees

@jongalloway jongalloway

Copilot code review Copilot

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

⏳ Docker build / test timeout after latest commit
2 participants
@Copilot @jongalloway