Skip to content

Commit

Permalink
Pass 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: Rebecca Roberts <[email protected]>
  • Loading branch information
acrmp and rroberts2222 committed Aug 19, 2022
1 parent 5030793 commit bb31b14
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 0 deletions.
1 change: 1 addition & 0 deletions lib/cloud_controller/diego/app_recipe_builder.rb
Original file line number Diff line number Diff line change
Expand Up @@ -68,6 +68,7 @@ def app_lrp_arguments
start_timeout_ms: health_check_timeout_in_seconds * 1000,
disk_mb: process.disk_quota,
memory_mb: process.memory, # sums up
log_rate_limit_bytes_per_second: process.log_rate_limit,
privileged: desired_lrp_builder.privileged?,
ports: ports,
log_source: LRP_LOG_SOURCE,
Expand Down
2 changes: 2 additions & 0 deletions lib/diego/bbs/models/desired_lrp_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 @@ -47,6 +47,7 @@ module Diego
instances: 21,
memory: 128,
disk_quota: 256,
log_rate_limit: 1024,
command: command,
file_descriptors: 32,
health_check_type: 'port',
Expand Down Expand Up @@ -285,6 +286,7 @@ module Diego
expect(lrp.log_source).to eq(LRP_LOG_SOURCE)
expect(lrp.max_pids).to eq(100)
expect(lrp.memory_mb).to eq(128)
expect(lrp.log_rate_limit_bytes_per_second).to eq(1024)
expect(lrp.metrics_guid).to eq(process.app.guid)

expect(lrp.metric_tags.keys.size).to eq(11)
Expand Down Expand Up @@ -872,6 +874,7 @@ module Diego
expect(lrp.log_guid).to eq(process.app.guid)
expect(lrp.max_pids).to eq(100)
expect(lrp.memory_mb).to eq(128)
expect(lrp.log_rate_limit_bytes_per_second).to eq(1024)
expect(lrp.metrics_guid).to eq(process.app.guid)

expect(lrp.metric_tags.keys.size).to eq(11)
Expand Down

0 comments on commit bb31b14

Please sign in to comment.