OSSM-2378: Use Maistra proxy CentOS Stream 8 img by bartoszmajsak · Pull Request #713 · maistra/istio

bartoszmajsak · 2022-12-21T15:12:37Z

For integration tests. Based on (#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

Please provide a description of this PR:

To help us figure out who should review this PR, please put an X in all the areas that this PR affects.

Please check any characteristics that apply to this pull request.

Does not have any user-facing changes. This may include CLI changes, API changes, behavior changes, performance improvements, etc.

For integration tests. Based on (#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>

maistra-bot · 2022-12-21T15:12:39Z

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

openshift-ci · 2022-12-21T15:12:41Z

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

tests/integration/security/file_mounted_certs/p2p_mtls_test.go

tests/integration/security/filebased_tls_origination/destination_rule_tls_test.go

tests/integration/security/filebased_tls_origination/egress_gateway_origination_test.go

bartoszmajsak · 2022-12-21T16:27:43Z

/retest

jwendell · 2022-12-21T16:30:30Z

/retest

bartoszmajsak · 2022-12-21T16:51:03Z

/test integration-telemetry

openshift-ci · 2022-12-21T16:51:06Z

@bartoszmajsak: The specified target(s) for /test were not found.
The following commands are available to trigger required jobs:

/test istio-gencheck-2-4
/test istio-lint-2-4
/test istio-unit-2-4

Use /test all to run all jobs.

Details

In response to this:

/test integration-telemetry

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.

jwendell · 2022-12-21T20:13:37Z

/retest

bartoszmajsak · 2022-12-21T21:56:22Z

Thanks @jwendell

* OSSM-2378: Use Maistra proxy CentOS Stream 8 img For integration tests. Based on (maistra#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>

* OSSM-2378: Use Maistra proxy CentOS Stream 8 img (#713) * OSSM-2378: Use Maistra proxy CentOS Stream 8 img For integration tests. Based on (#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 (#801) Signed-off-by: Jacek Ewertowski <jewertow@redhat.com> * OSSM-5958: Remove wasm extensions (#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>

* OSSM-2378: Use Maistra proxy CentOS Stream 8 img (maistra#713) * OSSM-2378: Use Maistra proxy CentOS Stream 8 img For integration tests. Based on (maistra#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 (maistra#801) Signed-off-by: Jacek Ewertowski <jewertow@redhat.com> * OSSM-5958: Remove wasm extensions (maistra#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>

maistra-bot added the do-not-merge/work-in-progress label Dec 21, 2022

maistra-bot added the size/L label Dec 21, 2022

bartoszmajsak marked this pull request as ready for review December 21, 2022 15:55

maistra-bot removed the do-not-merge/work-in-progress label Dec 21, 2022

bartoszmajsak requested a review from jwendell December 21, 2022 15:57

jwendell reviewed Dec 21, 2022

View reviewed changes

fix(tests): enables fixed integration tests

78d1365

jwendell approved these changes Dec 21, 2022

View reviewed changes

jwendell added the okay to merge label Dec 21, 2022

maistra-bot merged commit 4baafa5 into maistra:maistra-2.4 Dec 21, 2022

bartoszmajsak deleted the centos8_proxy_img branch December 21, 2022 21:56

yxun mentioned this pull request May 22, 2024

[maistra-2.6] Rebase building OpenSSL-based proxy image #1008

Closed

12 tasks

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

OSSM-2378: Use Maistra proxy CentOS Stream 8 img#713

OSSM-2378: Use Maistra proxy CentOS Stream 8 img#713
maistra-bot merged 2 commits intomaistra:maistra-2.4from
bartoszmajsak:centos8_proxy_img

bartoszmajsak commented Dec 21, 2022

Uh oh!

maistra-bot commented Dec 21, 2022

Uh oh!

openshift-ci bot commented Dec 21, 2022

Uh oh!

Uh oh!

Uh oh!

Uh oh!

bartoszmajsak commented Dec 21, 2022

Uh oh!

jwendell commented Dec 21, 2022

Uh oh!

bartoszmajsak commented Dec 21, 2022

Uh oh!

openshift-ci bot commented Dec 21, 2022

Uh oh!

jwendell commented Dec 21, 2022

Uh oh!

bartoszmajsak commented Dec 21, 2022

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

Conversation

bartoszmajsak commented Dec 21, 2022

Uh oh!

maistra-bot commented Dec 21, 2022

Uh oh!

openshift-ci bot commented Dec 21, 2022

Uh oh!

Uh oh!

Uh oh!

Uh oh!

bartoszmajsak commented Dec 21, 2022

Uh oh!

jwendell commented Dec 21, 2022

Uh oh!

bartoszmajsak commented Dec 21, 2022

Uh oh!

openshift-ci bot commented Dec 21, 2022

Uh oh!

jwendell commented Dec 21, 2022

Uh oh!

bartoszmajsak commented Dec 21, 2022

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants