Skip to content

[DO NOT MERGE] Auto PR to update dependencies of istio#609

Closed
istio-testing wants to merge 1 commit intomasterfrom
autoUpdateDeps_86fe06687a41ba1d9983c5774bbca01a
Closed

[DO NOT MERGE] Auto PR to update dependencies of istio#609
istio-testing wants to merge 1 commit intomasterfrom
autoUpdateDeps_86fe06687a41ba1d9983c5774bbca01a

Conversation

@istio-testing
Copy link
Copy Markdown
Collaborator

This PR will be merged automatically once checks are successful.

@istio-merge-robot
Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
We suggest the following additional approver: ldemailly

Assign the PR to them by writing /assign @ldemailly in a comment when ready.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these OWNERS Files:

You can indicate your approval by writing /approve in a comment
You can cancel your approval by writing /approve cancel in a comment

@ldemailly
Copy link
Copy Markdown
Member

/test all

@istio-merge-robot
Copy link
Copy Markdown

/test all [submit-queue is verifying that this PR is safe to merge]

@istio-merge-robot
Copy link
Copy Markdown

Automatic merge from submit-queue

@istio-merge-robot
Copy link
Copy Markdown

/test all [submit-queue is verifying that this PR is safe to merge]

@istio-merge-robot
Copy link
Copy Markdown

Automatic merge from submit-queue

@istio-merge-robot
Copy link
Copy Markdown

/test all [submit-queue is verifying that this PR is safe to merge]

@istio-merge-robot
Copy link
Copy Markdown

Automatic merge from submit-queue

@istio-merge-robot
Copy link
Copy Markdown

/test all [submit-queue is verifying that this PR is safe to merge]

@istio-merge-robot
Copy link
Copy Markdown

Automatic merge from submit-queue

@istio-merge-robot
Copy link
Copy Markdown

/test all [submit-queue is verifying that this PR is safe to merge]

@istio-merge-robot
Copy link
Copy Markdown

Automatic merge from submit-queue

@istio-merge-robot
Copy link
Copy Markdown

/test all [submit-queue is verifying that this PR is safe to merge]

@istio-merge-robot
Copy link
Copy Markdown

Automatic merge from submit-queue

@ldemailly
Copy link
Copy Markdown
Member

@yutongz that /test-all did seem to rebase but also throw the bot into a loop ?

@istio-merge-robot
Copy link
Copy Markdown

/test all [submit-queue is verifying that this PR is safe to merge]

@istio-merge-robot
Copy link
Copy Markdown

Automatic merge from submit-queue

@yutongz
Copy link
Copy Markdown
Contributor

yutongz commented Aug 29, 2017

@ldemailly Oh, yeah, right, that's the tricky part. The reason why @sebastienvas removed the rebase check from setting is because, that will stop merge-bot from merging. In istio/istio, all required rules apply on admin so now if the pr needs rebase, even merge-robot cannot force merge it.

@istio-merge-robot
Copy link
Copy Markdown

/test all [submit-queue is verifying that this PR is safe to merge]

@istio-merge-robot
Copy link
Copy Markdown

Automatic merge from submit-queue

@istio-merge-robot
Copy link
Copy Markdown

/test all [submit-queue is verifying that this PR is safe to merge]

@istio-merge-robot
Copy link
Copy Markdown

Automatic merge from submit-queue

@istio-merge-robot
Copy link
Copy Markdown

/test all [submit-queue is verifying that this PR is safe to merge]

@istio-merge-robot
Copy link
Copy Markdown

Automatic merge from submit-queue

@istio-merge-robot
Copy link
Copy Markdown

/test all [submit-queue is verifying that this PR is safe to merge]

@istio-merge-robot
Copy link
Copy Markdown

Automatic merge from submit-queue

@istio-merge-robot
Copy link
Copy Markdown

/test all [submit-queue is verifying that this PR is safe to merge]

@istio-merge-robot
Copy link
Copy Markdown

Automatic merge from submit-queue

@istio-testing
Copy link
Copy Markdown
Collaborator Author

@istio-testing: The following tests failed, say /retest to rerun them all:

Test name Commit Details Rerun command
prow/e2e-suite-rbac-no_auth.sh a9c1fad link /test e2e-suite-rbac-no_auth
prow/e2e-suite-no_rbac-auth.sh a9c1fad link /test e2e-suite-no_rbac-auth
prow/e2e-suite-rbac-auth.sh a9c1fad link /test e2e-suite-rbac-auth
Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@rshriram rshriram deleted the autoUpdateDeps_86fe06687a41ba1d9983c5774bbca01a branch September 12, 2017 02:36
0x01001011 pushed a commit to thedemodrive/istio that referenced this pull request Jul 16, 2020
* Deprecate int percent in favor of FractionalPercent

This commit deprecates the integer percent field in Delay and Abort
types in favor of the new FractionalPercent type which allows finer
control.

Signed-off-by: Venil Noronha <veniln@vmware.com>

* Remove FractionalPercent and use double for percentage

Signed-off-by: Venil Noronha <veniln@vmware.com>

* Fix docs related to the percentage field

Signed-off-by: Venil Noronha <veniln@vmware.com>

* Copy envoy.type.Percent proto definition

* The "type" in envoy.type.Percent is renamed to "types" in
istio.envoy.types.Percent to avoid a keyword conflict in Golang.

* The Makefile splits the *.pb.go file generation for v2alpha1/*.proto
and types/*.proto in order to avoid the "inconsistent package names"
error in protoc-gen-go.

Signed-off-by: Venil Noronha <veniln@vmware.com>

* Use istio.envoy.types.Percent for percentage

Signed-off-by: Venil Noronha <veniln@vmware.com>

* Revert copy envoy.type.Percent and its usage

Signed-off-by: Venil Noronha <veniln@vmware.com>

* Add and use Percent type in virtual_service.proto

Signed-off-by: Venil Noronha <veniln@vmware.com>
luksa pushed a commit to luksa/istio that referenced this pull request Sep 22, 2022
…o#609)

* Build container images with docker

Signed-off-by: Jacek Ewertowski <jewertow@redhat.com>

* Use maistra proxy and CentOS Stream 8 as a base image for proxyv2

Signed-off-by: Jacek Ewertowski <jewertow@redhat.com>

* Test VMs using CentOS Stream 8

Signed-off-by: Jacek Ewertowski <jewertow@redhat.com>

* Temporarily skip failing security tests

Signed-off-by: Jacek Ewertowski <jewertow@redhat.com>

* Temporarily skip tests suite telemetry.prometheus.wasm

Signed-off-by: Jacek Ewertowski <jewertow@redhat.com>

Signed-off-by: Jacek Ewertowski <jewertow@redhat.com>
luksa pushed a commit to luksa/istio that referenced this pull request Apr 11, 2024
* OSSM-2378: Use Maistra proxy CentOS Stream 8 img (istio#713)

* OSSM-2378: Use Maistra proxy CentOS Stream 8 img

For integration tests. Based on (istio#609)

* Build container images with docker

Signed-off-by: Jacek Ewertowski <jewertow@redhat.com>

* Use maistra proxy and CentOS Stream 8 as a base image for proxyv2

Signed-off-by: Jacek Ewertowski <jewertow@redhat.com>

* Test VMs using CentOS Stream 8

Signed-off-by: Jacek Ewertowski <jewertow@redhat.com>

* Temporarily skip failing security tests

Signed-off-by: Jacek Ewertowski <jewertow@redhat.com>

* Temporarily skip tests suite telemetry.prometheus.wasm

Signed-off-by: Jacek Ewertowski <jewertow@redhat.com>

Signed-off-by: Jacek Ewertowski <jewertow@redhat.com>

* fix(tests): enables fixed integration tests

Signed-off-by: Jacek Ewertowski <jewertow@redhat.com>
Co-authored-by: Jacek Ewertowski <jewertow@redhat.com>

* Use our proxy (istio#801)

Signed-off-by: Jacek Ewertowski <jewertow@redhat.com>

* OSSM-5958: Remove wasm extensions (istio#937)

* Do not rely on WASM extensions

Since maistra/proxy#253

* Remove wasm extensions from EnvoyFilters

Signed-off-by: Jacek Ewertowski <jewertow@redhat.com>

* Disable tests for WASM extensions

Signed-off-by: Jacek Ewertowski <jewertow@redhat.com>

* Remove commented code

Signed-off-by: Jacek Ewertowski <jewertow@redhat.com>

---------

Signed-off-by: Jacek Ewertowski <jewertow@redhat.com>
Co-authored-by: Jonh Wendell <jwendell@redhat.com>

* Disable failing tests

Signed-off-by: Jacek Ewertowski <jewertow@redhat.com>

* Fix TestTrustDomainValidation - change expected TLS error to make it work with OpenSSL

Signed-off-by: Jacek Ewertowski <jewertow@redhat.com>

---------

Signed-off-by: Jacek Ewertowski <jewertow@redhat.com>
Co-authored-by: Bartosz Majsak <bartosz.majsak@gmail.com>
Co-authored-by: Jonh Wendell <jonh.wendell@redhat.com>
Co-authored-by: Jonh Wendell <jwendell@redhat.com>
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.

6 participants