istio.io icon indicating copy to clipboard operation
istio.io copied to clipboard

quick guide on how to migrate from sidecars to ambient

Open nauticalmike opened this issue 1 year ago • 6 comments

Description

quick guide on how to migrate from sidecars to ambient

cc. @bleggett

Reviewers

  • [x] Ambient
  • [x] Docs
  • [ ] Installation
  • [ ] Networking
  • [ ] Performance and Scalability
  • [ ] Extensions and Telemetry
  • [ ] Security
  • [ ] Test and Release
  • [ ] User Experience
  • [ ] Developer Infrastructure
  • [ ] Localization/Translation

nauticalmike avatar Apr 17 '24 14:04 nauticalmike

Hi @nauticalmike. Thanks for your PR.

I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

istio-testing avatar Apr 17 '24 14:04 istio-testing

/ok-to-test

ericvn avatar Apr 17 '24 15:04 ericvn

@nauticalmike: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
lint_istio.io 9296bdf2e4a429bc53d970bfff6c45bb8fd6cee1 link true /test lint

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

istio-testing avatar Apr 17 '24 15:04 istio-testing

cc @kfaseela who asked me about this before

linsun avatar Apr 17 '24 15:04 linsun

/hold

I don't think we should merge this doc for 1.22, as it doesn't work for waypoint. Might be working for 1.21 but for 1.22, sidecars aren't aware of waypoints thus this won't work.

Again this was decided by TOC to be out of scope for 1.22 - I'd recommend us focus on must items for 1.22 first for the remaining 2 weeks before 1.22 ships.

linsun avatar Apr 30 '24 13:04 linsun