Skip to content

Conversation

jwendell
Copy link
Member

No description provided.

yangminzhu and others added 30 commits February 20, 2020 12:53
Signed-off-by: Rama Chavali <rama.rao@salesforce.com>
* remove use_downstream_protocol for gateway

* comment

* writing

Co-authored-by: Yan Xue <3491507+yxue@users.noreply.github.com>
Co-authored-by: knrc <knrc@users.noreply.github.com>
* resign certificate

* test

Co-authored-by: xuzhonghu <xuzhonghu@huawei.com>
This is going to continue to break as istio.io changes. We already
disabled this on 1.5/master.
…22171)

We already honor it in `go build`, we should do the same
in `go test`.

Manual backport of istio/istio#22163
This variable needs to be set before used (set -u) and must not be
quoted, otherwise it will become an empty arg for go test if it's empty.

Co-authored-by: Jonh Wendell <jonh.wendell@redhat.com>
…506)

Co-authored-by: gargnupur <gargnupur@google.com>
(cherry picked from commit fdc6dd4)

Co-authored-by: Zhonghu Xu <xuzhonghu@huawei.com>
Signed-off-by: Yuchen Dai <silentdai@gmail.com>
* Make sure to use CNI 1.4 image when running tests under CNI

* Fix lint
* Update deps with update_deps.sh

* Dep updates again
* Run update_deps.sh

Had to do go get istio.io/operator@5f8ecc70a0f4059bfd4d0f7867d4dc76407f2f08
since update_deps.sh was not updating it.

* Run go mod tidy
* Fix Gateway helm chart for helm 3

Helm 3 threw an error that `$spec.sds` is not a function. Removing the parentheses led to the next problem: the and evaluates eager so `spec.sds.enabled` resulted in a `nil` exception.

By nesting the if, the problem is resolved

* Also fix the role

Co-authored-by: Alex Nederlof <alex@nederlof.com>
* Add files from cherrypick

* Update yaml files to pull 1.15.1 images

Co-authored-by: Brian Avery <bavery@redhat.com>
@brian-avery
Copy link
Contributor

This also fixes: Maistra-1528

@mergify mergify bot merged commit 7d3618c into maistra:maistra-1.1 Sep 4, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.