Skip to content

Enable presubmit in Jenkins#74

Merged
sebastienvas merged 2 commits intoistio:masterfrom
sebastienvas:demo
Mar 22, 2017
Merged

Enable presubmit in Jenkins#74
sebastienvas merged 2 commits intoistio:masterfrom
sebastienvas:demo

Conversation

@sebastienvas
Copy link
Copy Markdown
Contributor

@sebastienvas sebastienvas commented Mar 21, 2017

Jenkins uses a GKE cluster so some updates were needed.


This change is Reviewable

Jenkins uses a GKE cluster so some updates were needed.
Updated bookinfo istio deployment
@sebastienvas
Copy link
Copy Markdown
Contributor Author

jenkins rebuild istio/presubmit

@istio-testing
Copy link
Copy Markdown
Collaborator

Jenkins job istio/presubmit passed

@sebastienvas
Copy link
Copy Markdown
Contributor Author

alleluia

@sebastienvas sebastienvas requested a review from rshriram March 22, 2017 21:01
Copy link
Copy Markdown
Member

@rshriram rshriram left a comment

Choose a reason for hiding this comment

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

We still have to automate the runtime image name right? Otherwise LGTM

@sebastienvas sebastienvas merged commit 2946b84 into istio:master Mar 22, 2017
@sebastienvas sebastienvas deleted the demo branch March 22, 2017 21:22
rshriram pushed a commit that referenced this pull request Oct 30, 2017
Enable presubmit in Jenkins

Former-commit-id: 2946b84
mandarjog pushed a commit that referenced this pull request Nov 2, 2017
Enable presubmit in Jenkins

Former-commit-id: 2946b84
kyessenov pushed a commit to kyessenov/istio that referenced this pull request Aug 13, 2018
rajusharma pushed a commit to rajusharma/istio that referenced this pull request Jul 2, 2019
* Add support for istio-cni install

Signed-off-by: Tim Swanson <tiswanso@cisco.com>

* Fix istio-cni ns name in check

Signed-off-by: Tim Swanson <tiswanso@cisco.com>

* Add cleanup and fix review comments

* istio-cni: Breakup monolith helm template and address review comments

- make each k8s object its own template
- change istio_cni.enable helm var to global.proxy.injectionControl=cni|init
- remove checks for istio-cni pods from install.sh

* Fix sidecar injector template to allow old istio_cni.enable var

* Restore use of istio_cni.enabled for control of inject template
howardjohn pushed a commit to howardjohn/istio that referenced this pull request Jan 12, 2020
* Cleanup of translate package

* Review comments
deva26 added a commit to deva26/istio that referenced this pull request Jun 24, 2020
Added permission to tsbd to watch deployments, daemonsets and
jacob-delgado pushed a commit that referenced this pull request Aug 24, 2021
istio-testing pushed a commit that referenced this pull request Aug 27, 2021
* Strip # part in URL by default and offer a flag to opt-out (#76)

* Strip # part in URL by default and offer flag for opt-out.

* Small fix.

* Small fix.

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

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

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

* Update the proxy SHA (#78)

* Strip # part in URL by default and offer flag for opt-out.

* Small fix.

* Small fix.

* Update proxy SHA.

* Fix a test.

* Update Proxy SHA (#79)

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

* Update the base image for 1.10 (#87)

* Update istio.deps

Co-authored-by: Oliver Liu <yonggangl@google.com>
Co-authored-by: Yangmin Zhu <ymzhu@google.com>
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
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.

4 participants