Merged
Conversation
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: sebastienvas Assign the PR to them by writing The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these OWNERS Files:You can indicate your approval by writing |
Contributor
Author
|
/assign @douglas-reid |
ldemailly
reviewed
Aug 15, 2017
| E2E_ARGS=(--use_local_cluster) | ||
|
|
||
| if [ "${CI}" == 'bootstrap' ]; then | ||
| if [ "${CI:-}" == 'bootstrap' ]; then |
Member
There was a problem hiding this comment.
this doesn't do anything different does it ? (please teach me if there is a difference between before/after)
rshriram
pushed a commit
that referenced
this pull request
Oct 30, 2017
* Update e2e.sh * Update e2e.sh Former-commit-id: 7a74914
vbatts
pushed a commit
to vbatts/istio
that referenced
this pull request
Oct 31, 2017
* Update e2e.sh * Update e2e.sh Former-commit-id: 7a74914
mandarjog
pushed a commit
that referenced
this pull request
Oct 31, 2017
* add grpc server/client * remove insecure option * add grpc test * update old tests * simplify tests * changes + increase liveness delay * reduce noise
mandarjog
pushed a commit
that referenced
this pull request
Nov 2, 2017
* Update e2e.sh * Update e2e.sh Former-commit-id: 7a74914
luksa
pushed a commit
to luksa/istio
that referenced
this pull request
Sep 15, 2022
* [ior] MAISTRA-1400 Add IOR to Pilot * [MAISTRA-1089][MAISTRA-1400][MAISTRA-1744][MAISTRA-1811]: Add IOR to Pilot (istio#135) (istio#240) * MAISTRA-1400: Add IOR to Pilot (istio#135) * MAISTRA-1400: Add IOR to Pilot * [MAISTRA-1744] Add route annotation propagation (istio#158) * MAISTRA-1811 Store resourceVersion of reconciled Gateway resource (istio#190) * MAISTRA-1089 Add support for IOR routes in all namespaces (istio#193) * MAISTRA-2131: ior: honor Gateway's httpsRedirect (istio#276) If Gateway's httpsRedirect is set to true, create the OpenShift Route with Insecure Policy set to `Redirect`. Manual cherrypick from maistra/istio#269. * MAISTRA-2149: Make IOR robust in multiple replicas (istio#282) In scenarios where multiple replicas of istiod are running, only one IOR should be in charge of keeping routes in sync with Istio Gateways. We achieve this by making sure IOR only runs in the leader replica. Also, because leader election is not 100% acurate, meaning that for a small window of time there might be two instances being the leader - which could lead to duplicated routes being created if a new gateway is created in that time frame - we also change the way the Route name is created: Instead of having a generateName field, we now explicitly pass a name to the Route object to be created. Being deterministic, it allows the Route creation to fail when there's already a Route object with the same name (created by the other leader in that time frame). Use an exclusive leader ID for IOR * Manual cherrypick of maistra/istio#275 * MAISTRA-1813: Add unit tests for IOR (istio#286) * MAISTRA-2051 fixes for maistra install * MAISTRA-2164: Refactor IOR internals (istio#295) Instead of doing lots of API calls on every event - this does not scale well with lots of namespaces - keep the state in memory, by doing an initial synchronization on start up and updating it when receiving events. The initial synchronization is more complex, as we have to deal with asynchronous events (e.g., we have to wait for the Gateway store to be warmed up). Once it's initialized, handling events as they arrive becomes trivial. Tests that make sure we do not make more calls to the API server than the necessary were added, to avoid regressions. * MAISTRA-2205: Add an option to opt-out for automatic route creation If the Istio Gateway contains the annotation `maistra.io/manageRoute: false` then IOR ignores it and doesn't attempt to create or manage route(s) for this Gateway. Also, ignore Gateways with the annotation `istio: egressgateway` as these are not meant to have routes. * Add integration test for IOR Signed-off-by: Jacek Ewertowski <jewertow@redhat.com> * OSSM-1442: IOR: Ignore UPDATE events if resourceVersions are the same (istio#516) * OSSM-1442: IOR: Ignore UPDATE events if resourceVersions are the same For some obscure reason, it looks like we may receive UPDATE events with the new object being equal to the old one. As IOR always delete and recreate routes when receiving an UPDATE event, this might lead to some service downtime, given for a few moments the route will not exist. We guard against this behavior by comparing the `resourceVersion` field of the new object and the one stored in the Route object. * Add test Co-authored-by: Brian Avery <bavery@redhat.com> Co-authored-by: Jonh Wendell <jonh.wendell@redhat.com> Fix debug log formatting OSSM-1800: Copy gateway labels to routes Simplify the comparison of resource versions We store the gateway resource version (the whole metadata actually) in the `syncRoute` object. There's no need to loop over the routes to perform the comparison. This also fix the corner case where the gateway has one host and for some reason OCP rejects the creation of the route (e.g., when hostname is already taken). In this case the `syncRoute` object exists with zero routes in it. Thus the loop is a no-op and the function wrongly returns with an error of `eventDuplicatedMessage`. By comparing directly using the `syncRoute.metadata` we fix this. OSSM-1105: Support namespace portion in gateway hostnames They are not used by routes, so we essentially ignore the namespace part - anything on the left side of a "namespace/hostname" string. OSSM-1650 Make sure initialSync and event loop behave the same (istio#551)
zmiklank
pushed a commit
to zmiklank/istio
that referenced
this pull request
Jan 30, 2026
dgn
pushed a commit
to dgn/istio
that referenced
this pull request
Feb 17, 2026
* Fix TestEgressGatewayTls security test (istio#565) Add required config for TestEgressGatewayTls security test Signed-off-by: Maxim Babushkin <mbabushk@redhat.com> Co-authored-by: Maxim Babushkin <mbabushk@redhat.com> * Fix Telemetry tests Sail Operator config (istio#552) (istio#556) Fix Telemetry integration tests configuration in Sail Operator setup flow. Signed-off-by: Maxim Babushkin <mbabushk@redhat.com> * Fix security QUIC Sail Operator config (istio#568) Add missing configuration for security QUIC based Istio integration tests. Signed-off-by: Maxim Babushkin <mbabushk@redhat.com> Co-authored-by: Maxim Babushkin <mbabushk@redhat.com> --------- Signed-off-by: Maxim Babushkin <mbabushk@redhat.com> Co-authored-by: OpenShift Cherrypick Robot <openshift-cherrypick-robot@redhat.com> Co-authored-by: Maxim Babushkin <mbabushk@redhat.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This should fix the issue where smoke test does not have logs.