Skip to content
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.

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

Metric for container network traffic #27

Closed
RadiatorSpring opened this issue Oct 22, 2020 · 3 comments
Closed

Metric for container network traffic #27

RadiatorSpring opened this issue Oct 22, 2020 · 3 comments
Labels
help wanted We think the change requested makes sense and would welcome a PR.

Comments

@RadiatorSpring
Copy link

Issue

It would be nice to have a metric that can track the network bandwidth traffic of each app on CF so that we can be more precise when choosing what rate and burst are best suited for our installation.

Context

We have tried finding such metric in silk-cni without success maybe we missed something. Also it would be nice to have a general understanding of the needs of the apps on a CF installation have, so that we can warn our customers if they are actively using too much bandwidth

@ameowlia
Copy link
Member

ameowlia commented Oct 26, 2020

Hi @RadiatorSpring,

It also seems related to this issue on cf-networking.

I see the value in such metric, however, I don't think we will be able to prioritize it soon. I suggest submitting a proposal (which we can provide feedback on) and a PR if you want to get these features in.

Also it would be nice to have a general understanding of the needs of the apps on a CF installation have

This is highly dependent on what the app is doing. There is no one-size-fits-all advice for how to manage your CF.

@MarcPaquette
Copy link
Contributor

Hi @ameowlia and @RadiatorSpring ,

Is there any further conversation to be had on this topic or is this okay to close out?

Thanks!

@MarcPaquette MarcPaquette moved this from Inbox to Pending Review | Discussion in Application Runtime Platform Working Group Aug 28, 2023
MarcPaquette added a commit to MarcPaquette/community that referenced this issue Sep 22, 2023
Adding Marc Paquette to Approvers for Networking

# App Runtime Platform: Networking Contributions
###PRs Commented on/Reviewed:
- 2023-09-13T12:54:57Z: [Add release-binaries.bash](cloudfoundry/routing-api-cli#17)
### Issues that may be relevant:
- 2019-07-10T22:36:21Z: [Configuring GitBot is recommended](cloudfoundry/policy_client#1)
- 2020-10-22T10:34:06Z: [Metric for container network traffic](cloudfoundry/silk-release#27)
- 2022-04-20T16:04:16Z: [Removing dynamic egress code as it's functionality has been superseded](cloudfoundry/policy_client#2)
- 2022-04-20T20:07:31Z: [Removal of Dynamic Egress functionality in favor of Dynamic ASG feature](cloudfoundry/policy_client#3)
- 2022-04-20T20:42:19Z: [Dynamic egress removal](cloudfoundry/silk-release#57)
- 2022-04-26T14:29:44Z: [Added ops file to local project](cloudfoundry/cf-networking-onboarding#3)
- 2022-05-04T19:40:17Z: [Dynamic egress removal](cloudfoundry/cf-networking-release#143)
- 2022-06-01T18:54:14Z: [Address bug with clain clean up error](cloudfoundry/silk-release#62)
- 2022-07-01T14:19:33Z: [Add link to ops file needed to disable Dynamic ASGs](cloudfoundry/cf-networking-release#155)
- 2023-02-09T14:59:02Z: [Bump StartResponseDelayInterval to 30 seconds](cloudfoundry/healthchecker-release#2)
- 2023-03-17T19:19:15Z: [Update 010-what-is-gorouter.markdown](cloudfoundry/cf-networking-onboarding#21)
- 2023-04-03T20:36:12Z: [Tests for docker_test were failing with VCS issues](cloudfoundry/silk-release#77)
### Code contributions:
- 2021-10-29T14:15:50Z: [Revert "Revert "Remove promote job""](cloudfoundry/cf-networking-helpers@8fd9676)
- 2021-10-29T14:15:50Z: [Revert "Revert "Remove promote job""](cloudfoundry/cf-networking-helpers@b0fa33a)
- 2021-12-20T22:10:40Z: [Remove unused RequestTimeout field](cloudfoundry/cf-networking-release@be95c04)
- 2021-12-21T00:52:33Z: [add ASGPoller from policy-server that queries capi for security groups](cloudfoundry/cf-networking-release@0e5c597)
- 2022-01-04T19:30:29Z: [Fix fly-execute helper script](cloudfoundry/silk-release@292e7e6)
- 2022-01-04T23:58:11Z: [Fix fly-execute helper script](cloudfoundry/silk-release@292e7e6)
- 2022-01-06T18:34:41Z: [Add bosh properties/templates for syncing ASG data into policy-server](cloudfoundry/cf-networking-release@cebbc85)
- 2022-01-06T18:58:41Z: [Updated package specs with new go dependency files](cloudfoundry/cf-networking-release@c851b15)
- 2022-01-07T22:06:09Z: [Get all tests passing and rebase](cloudfoundry/cf-networking-release@24c1b98)
- 2022-02-09T22:13:14Z: [Add enable_asg_syncing property to vxlan-policy-agent](cloudfoundry/silk-release@309e0de)
- 2022-02-14T14:53:58Z: [Clean up orphaned chains](cloudfoundry/silk-release@d9aea80)
- 2022-02-18T14:46:17Z: [Add support for cni-wrapper-plugin to call vxlan-policy-agent to sync ASG rules](cloudfoundry/silk-release@ab591de)
- 2022-04-06T15:16:22Z: [Initial work to remove Dynamic Egress Policy features](cloudfoundry/cf-networking-release@90b2ad5)
- 2022-04-06T19:31:13Z: [Remove handlers, api, store, and fakes](cloudfoundry/cf-networking-release@574bece)
- 2022-04-07T20:06:51Z: [Add Database migrations to remove Dynamic egress tables](cloudfoundry/cf-networking-release@3f0e230)
- 2022-04-18T19:30:50Z: [Remove documentation referencing Dynamic Egress in favor of Dynamic ASG](cloudfoundry/silk-release@cb57f71)
- 2022-04-20T15:59:06Z: [Removing dymanic egress code as it's functionality has been superseeded](cloudfoundry/policy_client@ab37279)
- 2022-04-20T17:33:32Z: [Merge pull request cloudfoundry#2 from cloudfoundry/dynamic-egress-removal](cloudfoundry/policy_client@5d4bff3)
- 2022-04-20T20:05:52Z: [Removal of Dynamic Egress functionality in favor of Dynamic ASG feature](cloudfoundry/policy_client@a33b7c3)
- 2022-04-26T14:02:26Z: [Make ops file local to project](cloudfoundry/cf-networking-onboarding@648fc4f)
- 2022-04-26T14:27:48Z: [Added ops file to local project](cloudfoundry/cf-networking-onboarding@22a1dbb)
- 2022-05-03T14:29:19Z: [Removal of dynamic egress code](cloudfoundry/cf-networking-release@9526fa7)
- 2022-05-03T15:51:38Z: [Addressing issue with flaky test](cloudfoundry/cf-networking-release@f8efe59)
- 2022-05-04T19:36:20Z: [Remove Dynamic Egress from Acceptance tests and documentation](cloudfoundry/cf-networking-release@975e5bf)
- 2022-07-01T14:21:28Z: [Add link to ops file needed to disable Dynamic ASGs (cloudfoundry#155)](cloudfoundry/cf-networking-release@d3861d5)
- 2023-01-18T20:29:27Z: [Release v0.0.1](cloudfoundry/healthchecker-release@88b9e64)
- 2023-02-09T14:58:59Z: [Bump StartResponseDelayInterval to 30 seconds](cloudfoundry/healthchecker-release@813d542)
- 2023-03-17T19:19:09Z: [Update 010-what-is-gorouter.markdown](cloudfoundry/cf-networking-onboarding@78f6228)
- 2023-04-03T22:32:47Z: [Tests for docker_test were failing with VCS issues (cloudfoundry#77)](cloudfoundry/silk-release@6f4a8a2)
- 2023-06-12T15:29:39Z: [Merge pull request cloudfoundry#3 from geofffranks/develop](cloudfoundry/healthchecker-release@c7fe702)
@ameowlia
Copy link
Member

Closing because this issue is finished.

@github-project-automation github-project-automation bot moved this from Pending Review | Discussion to Done in Application Runtime Platform Working Group Sep 25, 2023
ameowlia pushed a commit to cloudfoundry/community that referenced this issue Sep 26, 2023
Adding Marc Paquette to Approvers for Networking

# App Runtime Platform: Networking Contributions
###PRs Commented on/Reviewed:
- 2023-09-13T12:54:57Z: [Add release-binaries.bash](cloudfoundry/routing-api-cli#17)
### Issues that may be relevant:
- 2019-07-10T22:36:21Z: [Configuring GitBot is recommended](cloudfoundry/policy_client#1)
- 2020-10-22T10:34:06Z: [Metric for container network traffic](cloudfoundry/silk-release#27)
- 2022-04-20T16:04:16Z: [Removing dynamic egress code as it's functionality has been superseded](cloudfoundry/policy_client#2)
- 2022-04-20T20:07:31Z: [Removal of Dynamic Egress functionality in favor of Dynamic ASG feature](cloudfoundry/policy_client#3)
- 2022-04-20T20:42:19Z: [Dynamic egress removal](cloudfoundry/silk-release#57)
- 2022-04-26T14:29:44Z: [Added ops file to local project](cloudfoundry/cf-networking-onboarding#3)
- 2022-05-04T19:40:17Z: [Dynamic egress removal](cloudfoundry/cf-networking-release#143)
- 2022-06-01T18:54:14Z: [Address bug with clain clean up error](cloudfoundry/silk-release#62)
- 2022-07-01T14:19:33Z: [Add link to ops file needed to disable Dynamic ASGs](cloudfoundry/cf-networking-release#155)
- 2023-02-09T14:59:02Z: [Bump StartResponseDelayInterval to 30 seconds](cloudfoundry/healthchecker-release#2)
- 2023-03-17T19:19:15Z: [Update 010-what-is-gorouter.markdown](cloudfoundry/cf-networking-onboarding#21)
- 2023-04-03T20:36:12Z: [Tests for docker_test were failing with VCS issues](cloudfoundry/silk-release#77)
### Code contributions:
- 2021-10-29T14:15:50Z: [Revert "Revert "Remove promote job""](cloudfoundry/cf-networking-helpers@8fd9676)
- 2021-10-29T14:15:50Z: [Revert "Revert "Remove promote job""](cloudfoundry/cf-networking-helpers@b0fa33a)
- 2021-12-20T22:10:40Z: [Remove unused RequestTimeout field](cloudfoundry/cf-networking-release@be95c04)
- 2021-12-21T00:52:33Z: [add ASGPoller from policy-server that queries capi for security groups](cloudfoundry/cf-networking-release@0e5c597)
- 2022-01-04T19:30:29Z: [Fix fly-execute helper script](cloudfoundry/silk-release@292e7e6)
- 2022-01-04T23:58:11Z: [Fix fly-execute helper script](cloudfoundry/silk-release@292e7e6)
- 2022-01-06T18:34:41Z: [Add bosh properties/templates for syncing ASG data into policy-server](cloudfoundry/cf-networking-release@cebbc85)
- 2022-01-06T18:58:41Z: [Updated package specs with new go dependency files](cloudfoundry/cf-networking-release@c851b15)
- 2022-01-07T22:06:09Z: [Get all tests passing and rebase](cloudfoundry/cf-networking-release@24c1b98)
- 2022-02-09T22:13:14Z: [Add enable_asg_syncing property to vxlan-policy-agent](cloudfoundry/silk-release@309e0de)
- 2022-02-14T14:53:58Z: [Clean up orphaned chains](cloudfoundry/silk-release@d9aea80)
- 2022-02-18T14:46:17Z: [Add support for cni-wrapper-plugin to call vxlan-policy-agent to sync ASG rules](cloudfoundry/silk-release@ab591de)
- 2022-04-06T15:16:22Z: [Initial work to remove Dynamic Egress Policy features](cloudfoundry/cf-networking-release@90b2ad5)
- 2022-04-06T19:31:13Z: [Remove handlers, api, store, and fakes](cloudfoundry/cf-networking-release@574bece)
- 2022-04-07T20:06:51Z: [Add Database migrations to remove Dynamic egress tables](cloudfoundry/cf-networking-release@3f0e230)
- 2022-04-18T19:30:50Z: [Remove documentation referencing Dynamic Egress in favor of Dynamic ASG](cloudfoundry/silk-release@cb57f71)
- 2022-04-20T15:59:06Z: [Removing dymanic egress code as it's functionality has been superseeded](cloudfoundry/policy_client@ab37279)
- 2022-04-20T17:33:32Z: [Merge pull request #2 from cloudfoundry/dynamic-egress-removal](cloudfoundry/policy_client@5d4bff3)
- 2022-04-20T20:05:52Z: [Removal of Dynamic Egress functionality in favor of Dynamic ASG feature](cloudfoundry/policy_client@a33b7c3)
- 2022-04-26T14:02:26Z: [Make ops file local to project](cloudfoundry/cf-networking-onboarding@648fc4f)
- 2022-04-26T14:27:48Z: [Added ops file to local project](cloudfoundry/cf-networking-onboarding@22a1dbb)
- 2022-05-03T14:29:19Z: [Removal of dynamic egress code](cloudfoundry/cf-networking-release@9526fa7)
- 2022-05-03T15:51:38Z: [Addressing issue with flaky test](cloudfoundry/cf-networking-release@f8efe59)
- 2022-05-04T19:36:20Z: [Remove Dynamic Egress from Acceptance tests and documentation](cloudfoundry/cf-networking-release@975e5bf)
- 2022-07-01T14:21:28Z: [Add link to ops file needed to disable Dynamic ASGs (#155)](cloudfoundry/cf-networking-release@d3861d5)
- 2023-01-18T20:29:27Z: [Release v0.0.1](cloudfoundry/healthchecker-release@88b9e64)
- 2023-02-09T14:58:59Z: [Bump StartResponseDelayInterval to 30 seconds](cloudfoundry/healthchecker-release@813d542)
- 2023-03-17T19:19:09Z: [Update 010-what-is-gorouter.markdown](cloudfoundry/cf-networking-onboarding@78f6228)
- 2023-04-03T22:32:47Z: [Tests for docker_test were failing with VCS issues (#77)](cloudfoundry/silk-release@6f4a8a2)
- 2023-06-12T15:29:39Z: [Merge pull request #3 from geofffranks/develop](cloudfoundry/healthchecker-release@c7fe702)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted We think the change requested makes sense and would welcome a PR.
Projects
None yet
Development

No branches or pull requests

3 participants