Skip to content

CI: explicitly specify the service want to start for kafka and websocket example#31175

Merged
phlax merged 1 commit intoenvoyproxy:mainfrom
soulxu:fix_kafka_exmaple
Dec 5, 2023
Merged

CI: explicitly specify the service want to start for kafka and websocket example#31175
phlax merged 1 commit intoenvoyproxy:mainfrom
soulxu:fix_kafka_exmaple

Conversation

@soulxu
Copy link
Copy Markdown
Member

@soulxu soulxu commented Dec 5, 2023

Commit Message: ci: explicitly specify the service want to start for kafka example
Additional Description:
CI failed on kafka and websocket example verify test:

kafka is missing dependency kafka-client
ERROR: starting kafka .
websocket is missing dependency client-ws
ERROR: starting websocket .

Risk Level: low
Testing: CI test
Docs Changes: n/a
Release Notes: n/a
Platform Specific Features: n/a

Signed-off-by: He Jie Xu <hejie.xu@intel.com>
@soulxu
Copy link
Copy Markdown
Member Author

soulxu commented Dec 5, 2023

/assign @phlax

@soulxu soulxu changed the title ci: explicitly specify the service want to start for kafka example ci: explicitly specify the service want to start for kafka and websocket example Dec 5, 2023
@soulxu soulxu changed the title ci: explicitly specify the service want to start for kafka and websocket example CI: explicitly specify the service want to start for kafka and websocket example Dec 5, 2023
Copy link
Copy Markdown
Member

@phlax phlax left a comment

Choose a reason for hiding this comment

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

nice! lglm, thanks @soulxu

i wonder what changed - these used to work ~reliably i think

github has a pretty awful system for vm/agent versioning - basically they change things as they feel without much notice - this has broken docker/compose previously

@phlax phlax merged commit a9e51fb into envoyproxy:main Dec 5, 2023
delphisfang pushed a commit to delphisfang/envoy that referenced this pull request Dec 10, 2023
…ket example (envoyproxy#31175)

Signed-off-by: He Jie Xu <hejie.xu@intel.com>
phlax pushed a commit to phlax/envoy that referenced this pull request Dec 11, 2023
…ket example (envoyproxy#31175)

Signed-off-by: He Jie Xu <hejie.xu@intel.com>
Signed-off-by: Ryan Northey <ryan@synca.io>
phlax pushed a commit that referenced this pull request Dec 12, 2023
…ket example (#31175)

Signed-off-by: He Jie Xu <hejie.xu@intel.com>
Signed-off-by: Ryan Northey <ryan@synca.io>
phlax pushed a commit to phlax/envoy that referenced this pull request Dec 15, 2023
…ket example (envoyproxy#31175)

Signed-off-by: He Jie Xu <hejie.xu@intel.com>
Signed-off-by: Ryan Northey <ryan@synca.io>
phlax pushed a commit that referenced this pull request Dec 15, 2023
…ket example (#31175)

Signed-off-by: He Jie Xu <hejie.xu@intel.com>
Signed-off-by: Ryan Northey <ryan@synca.io>
SeanKilleen pushed a commit to SeanKilleen/envoy that referenced this pull request Apr 3, 2024
…ket example (envoyproxy#31175)

Signed-off-by: He Jie Xu <hejie.xu@intel.com>
Signed-off-by: Ryan Northey <ryan@synca.io>
Signed-off-by: Sean Killeen <SeanKilleen@gmail.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.

2 participants