Skip to content

ci: add clang-format-5.0#1205

Merged
htuch merged 1 commit intomasterfrom
mattklein123-patch-1
Jul 5, 2017
Merged

ci: add clang-format-5.0#1205
htuch merged 1 commit intomasterfrom
mattklein123-patch-1

Conversation

@mattklein123
Copy link
Member

This change just adds the binary. I will use the new image and make the format
fixes in a follow up change.

This change just adds the binary. I will use the new image and make the format 
fixes in a follow up change.
@mattklein123
Copy link
Member Author

@lyft/network-team @htuch

Copy link
Member

@htuch htuch left a comment

Choose a reason for hiding this comment

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

I think you'll find there's probably going to be some code churn when you first switch to clang-format-5.0, even if the same clang format is used. FYI.

@htuch htuch merged commit 146fb40 into master Jul 5, 2017
@htuch htuch deleted the mattklein123-patch-1 branch July 5, 2017 13:31
htuch added a commit that referenced this pull request Jul 5, 2017
rshriram pushed a commit to rshriram/envoy that referenced this pull request Oct 30, 2018
)

* Try Cluster warming

* Fix build errors

* build fix

* more build

* optional fix

* use absl::optional

* info.duration() removed

* more fixes

* use absl:optional in tests

* Use the latest Envoy from master

* Run clang-format

* Even newer Envoy

* Use requestComplete() instead of duration()

* Fix

* use requestComplete

* Use value_or()
mathetake pushed a commit that referenced this pull request Mar 3, 2026
**Description**

In standalone mode, it takes a while for the Envoy process to start.
This PR adds some feedback during the Envoy startup process so that
users can know when Envoy is ready to accept connections:

```
$ ./out/aigw-darwin-arm64 run --debug
(...)
looking up the latest patch for Envoy version 1.35
1.35.3 is already downloaded
starting: /tmp/envoy-gateway/versions/1.35.3/bin/envoy in run directory /tmp/envoy-gateway/runs/1758020241848418000
[2025-09-16 12:57:22.144][18234916][warning][config] [source/server/options_impl_platform_default.cc:9] CPU number provided by HW thread count (instead of cpuset).
time=2025-09-16T12:57:22.781+02:00 level=INFO msg="Waiting for Envoy to be ready..." status=PRE_INITIALIZING
time=2025-09-16T12:57:24.781+02:00 level=INFO msg="Waiting for Envoy to be ready..." status=PRE_INITIALIZING
time=2025-09-16T12:57:26.781+02:00 level=INFO msg="Waiting for Envoy to be ready..." status=PRE_INITIALIZING
time=2025-09-16T12:57:28.781+02:00 level=INFO msg="Waiting for Envoy to be ready..." status=PRE_INITIALIZING
time=2025-09-16T12:57:30.781+02:00 level=INFO msg="Waiting for Envoy to be ready..." status=PRE_INITIALIZING
time=2025-09-16T12:57:32.781+02:00 level=INFO msg="Waiting for Envoy to be ready..." status=PRE_INITIALIZING
time=2025-09-16T12:57:34.781+02:00 level=INFO msg="Waiting for Envoy to be ready..." status=PRE_INITIALIZING
time=2025-09-16T12:57:36.780+02:00 level=INFO msg="Waiting for Envoy to be ready..." status=PRE_INITIALIZING
time=2025-09-16T12:57:38.782+02:00 level=INFO msg="Envoy is ready!" status=LIVE
```

**Related Issues/PRs (if applicable)**

N/A

**Special notes for reviewers (if applicable)**

N/A

Signed-off-by: Ignasi Barrera <ignasi@tetrate.io>
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