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.

Sign up for GitHub

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

Jump to bottom

[EPIC] Ambient mode support in Sail Operator #500

Open
6 of 7 tasks
sridhargaddam opened this issue Nov 29, 2024 · 0 comments
Open
6 of 7 tasks

[EPIC] Ambient mode support in Sail Operator #500

sridhargaddam opened this issue Nov 29, 2024 · 0 comments
Assignees
@sridhargaddam
Labels
ambient epic
Milestone
v1.0

Comments

@sridhargaddam
Copy link
Contributor

sridhargaddam commented Nov 29, 2024 •
edited
Loading

Epic Description

Add necessary support in the Sail Operator to deploy Istio in Ambient mode.
@sridhargaddam sridhargaddam self-assigned this Nov 29, 2024
@sridhargaddam sridhargaddam mentioned this issue Dec 5, 2024
Merged
sridhargaddam added a commit to sridhargaddam/sail-operator that referenced this issue Dec 6, 2024
@sridhargaddam
Implement ztunnel controller in Sail Operator
1464eb2 
This PR implements the ztunnel controller and the associated unit tests
in Sail Operator.

Related to: istio-ecosystem#500

Signed-off-by: Sridhar Gaddam <[email protected]>
@sridhargaddam sridhargaddam mentioned this issue Dec 6, 2024
Merged
istio-testing pushed a commit that referenced this issue Dec 9, 2024
@sridhargaddam
Implement ztunnel controller in Sail Operator (#512)
1189a85 
This PR implements the ztunnel controller and the associated unit tests
in Sail Operator.

Related to: #500

Signed-off-by: Sridhar Gaddam <[email protected]>
sridhargaddam added a commit to sridhargaddam/sail-operator that referenced this issue Dec 17, 2024
@sridhargaddam
Fix undeploy when using Ambient mode
e350cc6 
When make undeploy is executed in an Ambient deployment, it hangs.
This happens because the Sail Operator is deleted, but a finalizer
remains on the ztunnel resource. As the finalizer isn't removed, the
ztunnel CR and the associated CRD cannot be deleted.

Related to: istio-ecosystem#500
Signed-off-by: Sridhar Gaddam <[email protected]>
@sridhargaddam sridhargaddam mentioned this issue Dec 17, 2024
Merged
istio-testing pushed a commit that referenced this issue Dec 17, 2024
@sridhargaddam
Fix undeploy when using Ambient mode (#533)
1d78f98 
When make undeploy is executed in an Ambient deployment, it hangs.
This happens because the Sail Operator is deleted, but a finalizer
remains on the ztunnel resource. As the finalizer isn't removed, the
ztunnel CR and the associated CRD cannot be deleted.

Related to: #500
Signed-off-by: Sridhar Gaddam <[email protected]>
sridhargaddam added a commit to sridhargaddam/sail-operator that referenced this issue Dec 23, 2024
@sridhargaddam
e2e tests for validating Ambient deployment
0432c84 
This PR implements e2e tests for control plane and dataplane validation
of Ambient profile in the Sail Operator.

Related to: istio-ecosystem#500
Signed-off-by: Sridhar Gaddam <[email protected]>
@sridhargaddam sridhargaddam mentioned this issue Dec 23, 2024
Merged
sridhargaddam added a commit to sridhargaddam/sail-operator that referenced this issue Dec 24, 2024
@sridhargaddam
e2e tests for validating Ambient deployment
e281e30 
This PR implements e2e tests for control plane and dataplane validation
of Ambient profile in the Sail Operator.

Related to: istio-ecosystem#500
Signed-off-by: Sridhar Gaddam <[email protected]>
sridhargaddam added a commit to sridhargaddam/sail-operator that referenced this issue Jan 8, 2025
@sridhargaddam
e2e tests for validating Ambient deployment
6ba21b1 
This PR implements e2e tests for control plane and dataplane validation
of Ambient profile in the Sail Operator.

Related to: istio-ecosystem#500
Signed-off-by: Sridhar Gaddam <[email protected]>
sridhargaddam added a commit to sridhargaddam/sail-operator that referenced this issue Jan 10, 2025
@sridhargaddam
e2e tests for validating Ambient deployment
2b8ede1 
This PR implements e2e tests for control plane and dataplane validation
of Ambient profile in the Sail Operator.

Related to: istio-ecosystem#500
Signed-off-by: Sridhar Gaddam <[email protected]>
istio-testing pushed a commit that referenced this issue Jan 16, 2025
@sridhargaddam
e2e tests for validating Ambient deployment (#541)
4e9cf6a 
* e2e tests for validating Ambient deployment

This PR implements e2e tests for control plane and dataplane validation
of Ambient profile in the Sail Operator.

Related to: #500
Signed-off-by: Sridhar Gaddam <[email protected]>

* Fix flaky error seen while running tests

As part of the test framework, when istio and istio-cni namespaces are
deleted, we were not waiting until the namespaces are deleted. Because
of this, we were seeing some race conditions in subsequent tests. This
PR fixes it.

Signed-off-by: Sridhar Gaddam <[email protected]>

* Verify ztunnel listening sockets in the application pod ns

Signed-off-by: Sridhar Gaddam <[email protected]>

* Avoid platform while running the e2e tests

Signed-off-by: Sridhar Gaddam <[email protected]>

---------

Signed-off-by: Sridhar Gaddam <[email protected]>
@dgn dgn added this to the v1.0 milestone Jan 17, 2025
sridhargaddam added a commit to sridhargaddam/sail-operator that referenced this issue Jan 20, 2025
@sridhargaddam
Allow overriding Ambient parameters in IstioCNI
ee7eb36 
Even though we can install Ambient using the Sail Operator,
we're unable to override some of the Ambient parameters in
the IstioCNI chart because the CNIAmbientConfig is missing
from the CNIConfig struct. This PR fixes it.

Related to: istio-ecosystem#500
Signed-off-by: Sridhar Gaddam <[email protected]>
@sridhargaddam sridhargaddam mentioned this issue Jan 20, 2025
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@sridhargaddam sridhargaddam

Labels
ambient epic
Projects
None yet
Milestone
v1.0
Development

No branches or pull requests
2 participants
@dgn @sridhargaddam