Skip to content

fix(container): default registry_image to nginx:latest when omitted at create time#3947

Draft
jremy42 wants to merge 1 commit intomainfrom
fix/container-registry-image-optional
Draft

fix(container): default registry_image to nginx:latest when omitted at create time#3947
jremy42 wants to merge 1 commit intomainfrom
fix/container-registry-image-optional

Conversation

@jremy42
Copy link
Copy Markdown
Contributor

@jremy42 jremy42 commented Apr 16, 2026

close #3946

@github-actions github-actions Bot added the container Container issues, bugs and feature requests label Apr 16, 2026
@jremy42 jremy42 force-pushed the fix/container-registry-image-optional branch from 844e3ca to 2706a3e Compare April 16, 2026 13:50
@codecov-commenter
Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 2.25%. Comparing base (f9ec5cb) to head (2706a3e).
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
internal/services/container/helpers_container.go 0.00% 2 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff            @@
##            main   #3947      +/-   ##
========================================
- Coverage   2.25%   2.25%   -0.01%     
========================================
  Files        516     516              
  Lines      55962   55964       +2     
========================================
  Hits        1261    1261              
- Misses     54602   54604       +2     
  Partials      99      99              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

container Container issues, bugs and feature requests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Serverless Container: default registry_image attribute is not optional

2 participants