Skip to content

docs: Deprecations for connect-native SDK and specific connect native APIs#17937

Merged
david-yu merged 21 commits intomainfrom
docs/connect-native
Jun 28, 2023
Merged

docs: Deprecations for connect-native SDK and specific connect native APIs#17937
david-yu merged 21 commits intomainfrom
docs/connect-native

Conversation

@david-yu
Copy link
Copy Markdown
Contributor

@david-yu david-yu commented Jun 28, 2023

Description

Deprecation of connect native SDK and specific connect native APIs to discourage new users from adopting connect native until a longer term solution exists (i.e. proxyless GRPC for XDS GH-10339) for native app integration into Consul Service Mesh.

Testing & Reproduction steps

Links

PR Checklist

  • updated test coverage
  • external facing docs updated
  • appropriate backport labels added
  • not a security concern

@david-yu david-yu added type/docs Documentation needs to be created/updated/clarified pr/no-changelog PR does not need a corresponding .changelog entry backport/1.16 This release series is no longer active on CE. Use backport/ent/1.16. labels Jun 28, 2023
@david-yu david-yu requested a review from a team as a code owner June 28, 2023 16:39
Copy link
Copy Markdown
Contributor

@trujillo-adam trujillo-adam left a comment

Choose a reason for hiding this comment

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

Made a few suggestions to address some punctuation and consistency issues.

# Service Mesh Native Integration for Go Applications

-> **Note:** The Connect Native golang SDK is currently deprecated and will be removed in a future Consul release.
The eventual removal of the SDK is dependent on the delivery of proxyless GRPC via XDS which is tracked on [GH-10339](https://github.com/hashicorp/consul/issues/10339).
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
The eventual removal of the SDK is dependent on the delivery of proxyless GRPC via XDS which is tracked on [GH-10339](https://github.com/hashicorp/consul/issues/10339).
We will replace the SDK functionality with proxy-less gRPC connections to Consul xDS. Refer to [GH-10339](https://github.com/hashicorp/consul/issues/10339) for additional information and to track progress toward the replacement functionality.

Is it correct to characterize the upcoming functionality as a replacement for the SDK?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Technically yes since all that is needed is the native golang package for XDS. I.e.

import _ "google.golang.org/grpc/xds"

@david-yu
Copy link
Copy Markdown
Contributor Author

Thanks @trujillo-adam and @im2nguyen I believe I have addressed all your comments.

Copy link
Copy Markdown
Contributor

@trujillo-adam trujillo-adam left a comment

Choose a reason for hiding this comment

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

👍🏽

@im2nguyen
Copy link
Copy Markdown
Contributor

@david-yu feel free to merge when ready!

@david-yu
Copy link
Copy Markdown
Contributor Author

Addressed @mkeeler 's comments.