Updates demo tests to use diff instead of copy of demo files#85
Merged
sebastienvas merged 4 commits intoistio:masterfrom Mar 23, 2017
Merged
Updates demo tests to use diff instead of copy of demo files#85sebastienvas merged 4 commits intoistio:masterfrom
sebastienvas merged 4 commits intoistio:masterfrom
Conversation
Collaborator
|
Jenkins job istio/presubmit passed |
rshriram
approved these changes
Mar 23, 2017
Member
rshriram
left a comment
There was a problem hiding this comment.
Please address the comment. Other than that, LGTM
tests/commonUtils.sh
Outdated
Member
There was a problem hiding this comment.
What does the above line do ? Bash has a bunch of vodoo stuff that not everyone knows:). So it might be good practice if we add comments to such constructs so that someone else could debug when needed.
tests/commonUtils.sh
Outdated
Collaborator
|
Jenkins job istio/presubmit passed |
zenlint
pushed a commit
to zenlint/istio
that referenced
this pull request
Aug 30, 2017
rshriram
pushed a commit
that referenced
this pull request
Oct 30, 2017
Updates demo tests to use diff instead of copy of demo files Former-commit-id: 3a09418
mandarjog
pushed a commit
that referenced
this pull request
Oct 31, 2017
mandarjog
pushed a commit
that referenced
this pull request
Nov 2, 2017
Updates demo tests to use diff instead of copy of demo files Former-commit-id: 3a09418
mandarjog
pushed a commit
that referenced
this pull request
Nov 2, 2017
guptasu
pushed a commit
to guptasu/istio
that referenced
this pull request
Jun 11, 2018
* Update documentation on our API surface to line up with today's world. Various other small touchups * Another round of touch-ups. Mostly bringing in comment lengths and annotating fields as required or optional in our config protos.
kyessenov
pushed a commit
to kyessenov/istio
that referenced
this pull request
Aug 13, 2018
* use uuid for deduplication_id * Only call uuid once.
howardjohn
pushed a commit
to howardjohn/istio
that referenced
this pull request
Jan 12, 2020
* Emphasize that dry run is default * Review comments
deva26
pushed a commit
to deva26/istio
that referenced
this pull request
Jun 24, 2020
* Add service def name. Use path for certs * Typo in values
jacob-delgado
pushed a commit
that referenced
this pull request
Aug 24, 2021
istio-testing
pushed a commit
that referenced
this pull request
Aug 25, 2021
* [1.9] Security fixes (#80) * release 1.9: use case-insensitive match for host field in authz (#75) Same as https://github.com/istio-private/istio/pull/70 Signed-off-by: Yangmin Zhu <ymzhu@google.com> * [1.9] Strip # part in URL by default and offer a flag to opt-out Co-authored-by: Yangmin Zhu <ymzhu@google.com> * add test for request with # character (#85) * Update base image for 1.9.8 (#88) * Update base image for 1.9.8 * Update Makefile.core.mk Co-authored-by: Oliver Liu <yonggangl@google.com> Co-authored-by: Yangmin Zhu <ymzhu@google.com>
libesz
pushed a commit
to libesz/istio
that referenced
this pull request
Nov 11, 2021
asmigala
pushed a commit
to asmigala/istio
that referenced
this pull request
Jul 24, 2024
…ase-1.22-merge_upstream_istio_release_1_22-b34c6a22 Automator: merge upstream changes to openshift-service-mesh/istio@release-1.22
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
4 participants
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 change is