Skip to content
This repository has been archived by the owner on Jul 11, 2023. It is now read-only.

apis: add UpstreamTrafficSetting API types #4535

Merged
merged 1 commit into from
Feb 15, 2022

Conversation

shashankram
Copy link
Member

@shashankram shashankram commented Feb 14, 2022

Description:
Adds the Go types for UpstreamTrafficSetting API.

*Note the defaults are not to be encoded in
the JSON tags and will be implemented within
the controller subsequently.

Part of #4500

Affected area:

Functional Area
New Functionality [X]

Please answer the following questions with yes/no.

  1. Does this change contain code from or inspired by another project? no

    • Did you notify the maintainers and provide attribution?
  2. Is this a breaking change? no

)

// UpstreamTrafficSetting defines the settings applicable to traffic destined
// to an upstream host.
Copy link
Contributor

Choose a reason for hiding this comment

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

Examples being Circuit Breaking

Copy link
Contributor

@draychev draychev left a comment

Choose a reason for hiding this comment

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

This is good stuff! Glad to see this being built out!
Thanks @shashankram !

@shashankram shashankram added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Feb 15, 2022
@shashankram shashankram removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Feb 15, 2022
@shashankram shashankram changed the title apis: add UpstreamTraffic API types apis: add UpstreamTrafficSetting API types Feb 15, 2022
Adds the Go types for UpstreamTrafficSetting API.

*Note the defaults are not to be encoded in
the JSON tags and will be implemented within
the controller subsequently.

Part of openservicemesh#4500

Signed-off-by: Shashank Ram <[email protected]>
@shashankram shashankram merged commit 9b67c22 into openservicemesh:main Feb 15, 2022
@shashankram shashankram deleted the upstream-api branch February 15, 2022 20:27
schristoff pushed a commit to schristoff/osm that referenced this pull request Feb 22, 2022
Adds the Go types for UpstreamTrafficSetting API.

*Note the defaults are not to be encoded in
the JSON tags and will be implemented within
the controller subsequently.

Part of openservicemesh#4500

Signed-off-by: Shashank Ram <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants