Skip to content

Fix bookinfo ingress and ingress-controller#77

Closed
frankbu wants to merge 7 commits intomasterfrom
demo_latest
Closed

Fix bookinfo ingress and ingress-controller#77
frankbu wants to merge 7 commits intomasterfrom
demo_latest

Conversation

@frankbu
Copy link
Copy Markdown
Contributor

@frankbu frankbu commented Mar 22, 2017

This change is Reviewable

type: LoadBalancer
ports:
- port: 80
nodePort: 32000
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please remove this line as well for type LoadBalancer.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Otherwise LGTM

@frankbu frankbu closed this Mar 22, 2017
mandarjog pushed a commit that referenced this pull request Oct 31, 2017
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
kyessenov pushed a commit to kyessenov/istio that referenced this pull request Aug 13, 2018
* all batch report changes.

* add report batch test.

* fix format

* default report batch time to 1 second.

* Log error if report call fails.

* not to call timer at start.

* fix memory leak in unit_test.
rajusharma pushed a commit to rajusharma/istio that referenced this pull request Jul 2, 2019
Signed-off-by: Douglas Reid <douglas-reid@users.noreply.github.com>
howardjohn pushed a commit to howardjohn/istio that referenced this pull request Jan 12, 2020
deva26 pushed a commit to deva26/istio that referenced this pull request Jun 24, 2020
Remove extra cmdline flags related to galley for the tsbd start command
jacob-delgado pushed a commit that referenced this pull request Aug 24, 2021
* release 1.11: use case-insensitive match for host field in authz (#73)

Same as https://github.com/istio-private/istio/pull/69

Signed-off-by: Yangmin Zhu <ymzhu@google.com>

* Strip # part in URL by default and offer a flag to opt-out

* Small fix.

* Small fix.

* Small fix.

* Small fix.

* Small fix.

Co-authored-by: Yangmin Zhu <ymzhu@google.com>
istio-testing pushed a commit that referenced this pull request Aug 25, 2021
* [1.11] Security fixes (#77)

* release 1.11: use case-insensitive match for host field in authz (#73)

Same as https://github.com/istio-private/istio/pull/69

Signed-off-by: Yangmin Zhu <ymzhu@google.com>

* Strip # part in URL by default and offer a flag to opt-out

* Small fix.

* Small fix.

* Small fix.

* Small fix.

* Small fix.

Co-authored-by: Yangmin Zhu <ymzhu@google.com>

* add test for request with # character (#81)

* Update proxy SHA

Co-authored-by: Oliver Liu <yonggangl@google.com>
Co-authored-by: Yangmin Zhu <ymzhu@google.com>
Co-authored-by: Jonh Wendell <jonh.wendell@redhat.com>
luksa pushed a commit to luksa/istio that referenced this pull request Oct 14, 2024
…er-merge_upstream_istio_master-6253864e

Automator: merge upstream changes to openshift-service-mesh/istio@master
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants