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

Initial version of the Load Balancer taxonomy #149

Merged
merged 2 commits into from
Apr 4, 2024

Conversation

cfisher-scottlogic
Copy link
Contributor

Adds Cloud Load Balancer to CCC service taxonomy as per #133.

@crawfordchanel crawfordchanel mentioned this pull request Mar 27, 2024
14 tasks
Copy link
Contributor

@AdrianHammond AdrianHammond left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@cfisher-scottlogic Hi Cara. - Re line 53 - CCC-050335 - this looks to specific capability for AWS Outpost? I think the Taxonomy should be vendor agnostic so I think we should update / remove this line, what do you think? Many thanks Adrian

Copy link
Contributor

@mlysaght2017 mlysaght2017 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Very comprehensive, nice job

| CCC-050332 | HTTP header based routing | Direct incoming requests to different backend resources based on the values of HTTP headers. |
| CCC-050333 | Traffic mirroring | Can duplicate incoming network traffic and send it to a secondary destination for monitoring, analysis, or testing purposes. |
| CCC-050334 | Traffic splitting / weighted routing | Can distribute incoming traffic across multiple backend resources based on predefined weights or percentages. (e.g. for canary deployments, A/B testing, blue-green deployments, or gradual traffic migrations) |
| CCC-050335 | Outpost support | Ability to extend cloud-provisioned load balancing services to on-premises or edge locations using dedicated hardware or software appliances (outposts). |
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks vendor specific

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@cfisher-scottlogic Hi Cara. - Re line 53 - CCC-050335 - this looks to specific capability for AWS Outpost? I think the Taxonomy should be vendor agnostic so I think we should update / remove this line, what do you think? Many thanks Adrian

Looks vendor specific

You're bang on, this is AWS specific terminology. However, I think the capability (on premises/edge load balancing) is agnostic, so perhaps just a rewording of the feature is required here. How about 'On-premises Cloud Integration', or 'Hybrid Cloud Connectivity'?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thats works for me @cfisher-scottlogic

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorted! Also changed description slightly: ea952e0.

@@ -50,7 +50,7 @@ use in financial services ecosystems.
| CCC-050332 | HTTP header based routing | Direct incoming requests to different backend resources based on the values of HTTP headers. |
| CCC-050333 | Traffic mirroring | Can duplicate incoming network traffic and send it to a secondary destination for monitoring, analysis, or testing purposes. |
| CCC-050334 | Traffic splitting / weighted routing | Can distribute incoming traffic across multiple backend resources based on predefined weights or percentages. (e.g. for canary deployments, A/B testing, blue-green deployments, or gradual traffic migrations) |
| CCC-050335 | Outpost support | Ability to extend cloud-provisioned load balancing services to on-premises or edge locations using dedicated hardware or software appliances (outposts). |
| CCC-050335 | Hybrid cloud connectivity | Can extend cloud-provisioned load balancing services across on-premises, private, and other public cloud infrastructures. |
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Contributor

@AdrianHammond AdrianHammond left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This LGTM - not certain if its okay for me to merge. @mark-rushing thoughts?

Copy link

@markrushing markrushing left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Agreed and approved

Copy link
Contributor

@AdrianHammond AdrianHammond left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Merging post discussion on all project meeting on 04/04.

@AdrianHammond AdrianHammond merged commit f2e97ed into finos:main Apr 4, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants