Skip to content

Conversation

@pgolm
Copy link

@pgolm pgolm commented Dec 9, 2025

The Scaleway Jobs API supports configuring local storage capacity for job definitions, but this option was not exposed in the Terraform provider. This change adds the local_storage_capacity attribute to allow users to specify local storage in MiB for their job containers.

@pgolm pgolm requested review from a team and remyleone as code owners December 9, 2025 17:36
remyleone
remyleone previously approved these changes Dec 10, 2025
@remyleone
Copy link
Member

Thanks a lot for your contribution :)

@remyleone remyleone enabled auto-merge December 10, 2025 10:22
auto-merge was automatically disabled December 10, 2025 15:30

Head branch was pushed to by a user without write access

@pgolm pgolm force-pushed the feat/job-definition-local-storage-capacity branch from 8fab6c2 to 330917c Compare December 10, 2025 15:30
The Scaleway Jobs API supports configuring local storage capacity for
job definitions, but this option was not exposed in the Terraform
provider. This change adds the local_storage_capacity attribute to
allow users to specify local storage in MiB for their job containers.
@pgolm pgolm force-pushed the feat/job-definition-local-storage-capacity branch from 330917c to f07f3f2 Compare December 10, 2025 15:50
@pgolm
Copy link
Author

pgolm commented Dec 10, 2025

I've fixed the tests.

@codecov-commenter
Copy link

codecov-commenter commented Dec 10, 2025

Codecov Report

❌ Patch coverage is 0% with 10 lines in your changes missing coverage. Please review.
✅ Project coverage is 1.71%. Comparing base (c4ee92c) to head (595d92a).
⚠️ Report is 4 commits behind head on master.
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
internal/services/jobs/jobs.go 0.00% 10 Missing ⚠️
Additional details and impacted files
@@          Coverage Diff           @@
##           master   #3521   +/-   ##
======================================
  Coverage    1.71%   1.71%           
======================================
  Files         415     415           
  Lines       45753   45763   +10     
======================================
+ Hits          785     787    +2     
- Misses      44880   44889    +9     
+ Partials       88      87    -1     

☔ 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

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants