search

cloudfoundry / cf-for-k8s

The open source deployment manifest for Cloud Foundry on Kubernetes
Apache License 2.0
300 stars 115 forks source link

Re add logging to smoke test #590

Closed MasslessParticle closed 3 years ago

MasslessParticle commented 3 years ago

#175542566

Signed-off by: Tim Downey tdowney@vmware.com

WHAT is this change about?

The smoke test for logging was unrealistic because it only emitted a single log. We made the smoke test app emit 1 log/second to overcome potential lossiness.

Does this PR introduce a change to config/values.yml?

No

Acceptance Steps

The smoke tests should be less flaky

Tag your pair, your PM, and/or team

@tcdowney @angelachin

cf-gitbot commented 3 years ago

We have created an issue in Pivotal Tracker to manage this:

https://www.pivotaltracker.com/story/show/176003602

The labels on this github issue will be updated when the story is started.

cf-rel-int-status-bot commented 3 years ago

Hello friend, it looks like your pull request has failed one or more of our checks. Please take a look! :eyes:

cf-gitbot commented 3 years ago

We have created an issue in Pivotal Tracker to manage this:

https://www.pivotaltracker.com/story/show/176019508

The labels on this github issue will be updated when the story is started.

cf-rel-int-status-bot commented 3 years ago

Hello friend, it looks like your pull request has failed one or more of our checks. Please take a look! :eyes:

jamespollard8 commented 3 years ago

LGTM!