[envoy / haproxy] Create new components manifests#6024
Conversation
|
|
87c8e7d to
d253b47
Compare
3ed55b9 to
4140af7
Compare
|
f0d47ee to
8855d57
Compare
cbeauchesne
left a comment
There was a problem hiding this comment.
We're getting somewhere :)
The CI prod on happroxy (and also probably the other) is not using the last release, but a dev one. We can go though this together on monday.
8855d57 to
180c90f
Compare
|
Oops good catch, it was loading the dev version by default for haproxy (because at first on the old stream-offload-processing scenario, the tests were released before the haproxy integration was released) |
180c90f to
008382e
Compare
…ifests + rename test aggregation decorator to scenario group go_proxies
008382e to
3c3ad60
Compare
cbeauchesne
left a comment
There was a problem hiding this comment.
two small cosmetic changes
b26d380 to
e7c1ca3
Compare
|
/merge |
|
View all feedbacks in Devflow UI.
This pull request is not mergeable according to GitHub. Common reasons include pending required checks, missing approvals, or merge conflicts — but it could also be blocked by other repository rules or settings.
This pull request was merged directly. |
Motivation
Changes
This PR made changes:
GO_PROXIES_.go_proxies_default: everything that should fall into the default case of the proxiesgo_proxies_appsec_blocking: executes the tests of theAPPSEC_BLOCKINGagainst proxiesNote: This could be confusing but, envoy and haproxy, are the two tested components and come with their own weblogs. Currently they only have one weblog per proxy, but this will be increased in the future.
Workflow
🚀 Once your PR is reviewed and the CI green, you can merge it!
🛟 #apm-shared-testing 🛟
Reviewer checklist
tests/ormanifests/is modified ? I have the approval from R&P teambuild-XXX-imagelabel is present