Skip to content

Commit

Permalink
Pass task log rate limit to Diego
Browse files Browse the repository at this point in the history
[#182311441](https://www.pivotaltracker.com/story/show/182311441)

Co-authored-by: Matthew Kocher <[email protected]>
  • Loading branch information
2 people authored and rroberts2222 committed Aug 19, 2022
1 parent bb31b14 commit fc22209
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 0 deletions.
1 change: 1 addition & 0 deletions lib/cloud_controller/diego/task_recipe_builder.rb
Original file line number Diff line number Diff line change
Expand Up @@ -26,6 +26,7 @@ def build_app_task(config, task)
disk_mb: task.disk_in_mb,
egress_rules: @egress_rules.running_protobuf_rules(task.app),
log_guid: task.app.guid,
log_rate_limit_bytes_per_second: task.log_rate_limit,
log_source: TASK_LOG_SOURCE,
max_pids: config.get(:diego, :pid_limit),
memory_mb: task.memory_in_mb,
Expand Down
1 change: 1 addition & 0 deletions lib/diego/bbs/models/task_pb.rb

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Original file line number Diff line number Diff line change
Expand Up @@ -344,6 +344,7 @@ module Diego
app: app,
disk_in_mb: 1024,
memory_in_mb: 2048,
log_rate_limit: 3072,
sequence_id: 9
)
end
Expand Down Expand Up @@ -468,6 +469,7 @@ module Diego
expect(result.log_guid).to eq(app.guid)
expect(result.memory_mb).to eq(2048)
expect(result.disk_mb).to eq(1024)
expect(result.log_rate_limit_bytes_per_second).to eq(3072)
expect(result.environment_variables).to eq(lifecycle_environment_variables)
expect(result.legacy_download_user).to eq('vcap')
expect(result.root_fs).to eq('preloaded:potato-stack')
Expand Down Expand Up @@ -610,6 +612,7 @@ module Diego

expect(result.disk_mb).to eq(1024)
expect(result.memory_mb).to eq(2048)
expect(result.log_rate_limit_bytes_per_second).to eq(3072)
expect(result.log_guid).to eq(app.guid)
expect(result.privileged).to be(false)
expect(result.egress_rules).to eq([
Expand Down

0 comments on commit fc22209

Please sign in to comment.