-
Sub-task
-
Resolution: Unresolved
-
Major
-
None
-
None
-
False
-
-
False
-
-
Background
This task tracks the resolution of reviewer questions on PR #1279 (Ambient Mode Update Documentation) before final submission to avoid iteration cycles on GitHub.
Reference ticket: OSSM-10821 (Ambient mode upgrade testing)
Documentation Methodology
The following procedure was used to create the ambient mode update documentation:
1. Initial Testing (OSSM-10821): Executed ambient mode upgrade tests from N-1 to current version using bookinfo application on OpenShift cluster. Validated InPlace update strategy with Istio, IstioCNI, and ZTunnel components.
2. Source Analysis: Reviewed upstream Istio documentation:
- Ambient Upgrade Guide: https://istio.io/latest/docs/ambient/upgrade/helm/
- Troubleshoot ztunnel: https://istio.io/latest/docs/ambient/usage/troubleshoot-ztunnel/
- Troubleshoot Waypoint: https://istio.io/latest/docs/ambient/usage/troubleshoot-waypoint/
3. Architecture Reference: Used Red Hat Developer article on ambient mode:
4. Document Structure: Created three documentation files covering update strategies, ambient mode setup, and waypoint proxy configuration.
Open Review Questions
Five comments from code review require clarification:
| # | Location | Question |
|---|---|---|
| 1 | update-strategy.adoc | Confusion: states RevisionBased not possible for ambient, then says procedures apply to both strategies |
| 2 | update-strategy.adoc | Clarification needed on "manual revision synchronization" statement |
| 3 | istio-ambient-waypoint.adoc | Is it possible to use revisioned istiod with non-revisioned ztunnel? |
| 4 | istio-ambient-waypoint.adoc | Do waypoint proxies update automatically or require manual restart? |
| 5 | istio-ambient-mode.adoc | Same as #2 - clarification on manual sync statement |
Resolution Required
1. Clarify the actual support status of RevisionBased strategy in ambient mode
2. Remove contradictory statements from the documentation
3. Verify waypoint proxy update behavior through testing or upstream documentation
4. Update documentation to reflect accurate behavior
- relates to
-
OSSM-10821 Test Ambient mode upgrade from N-1 to the current version
-
- Closed
-