Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

sync from master. #7

Merged
merged 120 commits into from
Jan 30, 2021
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
120 commits
Select commit Hold shift + click to select a range
abf8cd8
tools: fill in the required args for Api::Impl (#14554)
ynqa Jan 10, 2021
876a363
rate limit: add computed descriptors (#14448)
kyessenov Jan 10, 2021
8a4b701
[fuzz] fix minor fuzz bugs (#14593)
asraa Jan 10, 2021
12c42b5
http: expose encoded headers/trailers via callbacks (#14544)
snowp Jan 11, 2021
845f92a
Implement request header processing in ext_proc (#14385)
gbrail Jan 11, 2021
5c801b2
1.17.0 release (#14624)
mattklein123 Jan 11, 2021
748a900
kick off v1.18.0 (#14637)
mattklein123 Jan 11, 2021
74327f2
filter manager: drop assert (#14633)
Jan 11, 2021
dcf8340
docs: update ext_proc docs to reflect implementation status (#14636)
gbrail Jan 11, 2021
c2522c6
[tls] Expose ServerContextImpl::selectTlsContext (#14592)
cretz Jan 12, 2021
dcf575a
http: support creating filters with match tree (#14430)
snowp Jan 12, 2021
e9ffbc5
fix empty connection debug logs (#14666)
ramaraochavali Jan 12, 2021
e53227e
tcp_proxy: ignore transfer encoding in HTTP/1.1 CONNECT responses (#1…
irozzo-1A Jan 12, 2021
7b5544b
ci: fix docs tag build (#14653)
lizan Jan 12, 2021
a836c8e
HTTP health checker: handle GOAWAY from HTTP2 upstreams (#13599)
mpuncel Jan 12, 2021
0e1b388
upstream: clean up feature parsing code (#14629)
alyssawilk Jan 12, 2021
658ee30
upstream: force a full rebuild on host weight changes (#14569)
mattklein123 Jan 12, 2021
efe73e8
access log: add support for command formatter extensions (#14512)
Jan 12, 2021
c5718c3
test: improving dynamic_forward_proxy coverage (#14672)
alyssawilk Jan 13, 2021
1d01c75
access_logs: removing disallow_unbounded_access_logs (#14677)
alyssawilk Jan 13, 2021
84dd87a
wasm: replace the obsolete contents in wasm-cc's README with docs lin…
nak3 Jan 13, 2021
df6f057
grpc-json-transcoder: support root path (#14585)
TAOXUY Jan 13, 2021
8d05b0c
ecds: add config source for network filter configs (#14674)
kyessenov Jan 13, 2021
9d1690d
fix comment for parameters end_stream of decodeData/encodeData. (#14620)
wangfakang Jan 13, 2021
445026e
[fuzz] Fix bugs in HPACK fuzz test (#14638)
asraa Jan 13, 2021
c151936
v3 packages updates for omit_canary_hosts proto (#14117)
ankatare Jan 13, 2021
a4bf246
streaminfo/mocks: delay filter_state_ dereference (#14612)
brianwolfe Jan 13, 2021
ad7d36f
http: support passing match result action to filter (#14462)
snowp Jan 13, 2021
a376d36
refactor: use unitfloat in more places (#14396)
akonradi Jan 13, 2021
9727c70
[tls] add missing built in cipher stat names (#14676)
asraa Jan 13, 2021
a89a5d0
docs: adding coverage walkthroguh (#14688)
alyssawilk Jan 13, 2021
b874400
local ratelimit: Add descriptor support in HTTP Local Rate Limiting (…
kyessenov Jan 13, 2021
1c43e39
http: prefetch for upstreams (#14143)
alyssawilk Jan 13, 2021
4bd430c
docs: Give a hint to specify type_url instead (#14562)
dio Jan 13, 2021
a4d8d91
Remove flaky_on_windows tag from proxy_filter_integration_test (#14680)
rmiller14 Jan 13, 2021
d830384
upstream: Fix moving EDS hosts between priorities. (#14483)
JonathanO Jan 13, 2021
7b5cf87
examples: Add TLS SNI sandbox (#13975)
phlax Jan 13, 2021
4534383
[utility]: Change behavior of main thread verification utility (#14660)
chaoqin-li1123 Jan 13, 2021
b794aea
Windows build: Add repository cache to CI (#14678)
sunjayBhatia Jan 13, 2021
4cadf72
ext-proc: Support "immediate_response" options for request headers (…
gbrail Jan 14, 2021
de02955
stats: convert tag extractor regexs to Re2 (#14519)
rojkov Jan 14, 2021
7653adf
hcm: removing envoy.reloadable_features.early_errors_via_hcm #14641 (…
alyssawilk Jan 14, 2021
8f092c6
test: Fix O(1/32k) flakiness in H2 flood tests that disable writes ba…
antoniovicente Jan 14, 2021
fb0ad4c
tls: add missing stats for signature algorithms. (#14703)
PiotrSikora Jan 14, 2021
e221742
connection: tighten network connection buffer limits (#14333)
antoniovicente Jan 14, 2021
09d112c
xdstp: LDS glob collection support. (#14311)
htuch Jan 14, 2021
e9f9df3
listener manager: avoid unique -> shared conversion (#14693)
Jan 14, 2021
a62c200
proto: re-implement RepeatedPtrUtil::hash(). (#14701)
PiotrSikora Jan 14, 2021
eca8065
tcp: setting nodelay on all connections (#14574)
alyssawilk Jan 14, 2021
cfd1532
test: Add multiheader TE + Content-Length test (#14686)
yanavlasov Jan 14, 2021
fda60bb
http2: Flip the upstream H2 frame flood and abuse checks to ON by def…
yanavlasov Jan 14, 2021
e3bbb5f
Fix the emsdk patching. (#14673)
jwendell Jan 14, 2021
38a6fe5
test: print test parameters meaningfully (#14604)
akonradi Jan 14, 2021
77d7cec
Migrate v2 thrift_filter to v3 api and corresponding docs changes. (#…
ankatare Jan 15, 2021
18db4c9
http: reinstating prior connect timeout behavior (#14685)
alyssawilk Jan 15, 2021
1d1b708
Fix typo (#14716)
hs0210 Jan 15, 2021
17e8151
master -> main (#14729)
mattklein123 Jan 16, 2021
5abba78
readme: fix logo URL (#14733)
mattklein123 Jan 16, 2021
62ca83a
Bump nghttp2 to 1.42.0 (#14730)
wrowe Jan 17, 2021
84d9126
Pick up current bazel-build-tools tag (#14734)
wrowe Jan 17, 2021
e34729a
access-logger: support request/response headers size (#14692)
TAOXUY Jan 18, 2021
38efc57
dynamic_forward_proxy: envoy.reloadable_features.enable_dns_cache_cir…
Shikugawa Jan 19, 2021
deaf537
Add support for google::protobuf::ListValue formatting (#14518)
itamarkam Jan 19, 2021
da1b0e4
tls: improve TLS handshake/read/write error log (#14600)
Shikugawa Jan 19, 2021
7d48928
config: switch from std::set to absl::flat_hash_set for resource name…
htuch Jan 19, 2021
72db81d
network filters: avoid unnecessary std::shared_ptrs (#14711)
Jan 19, 2021
610900e
docs: Updated version history with 1.13.8 release notes. (#14742)
cpakulski Jan 19, 2021
62762b5
Dispatcher: keeps a stack of tracked objects. (#14573)
KBaichoo Jan 19, 2021
e899c7b
bootstrap-extensions: fix a crash on http callout (#14478)
yuval-k Jan 19, 2021
2562fbb
overload: create scaled timers via the dispatcher (#14679)
akonradi Jan 20, 2021
1fe95aa
http: removing nvoy.reloadable_features.fix_upgrade_response #14643 …
alyssawilk Jan 20, 2021
7fca50b
http: removing envoy.reloadable_features.fixed_connection_close (#14705)
alyssawilk Jan 20, 2021
f2f6943
Revert "network filters: avoid unnecessary std::shared_ptrs (#14711)"…
Jan 20, 2021
ddba1f0
Clarify Consecutive Gateway Failure docs (#14738)
mattmb Jan 20, 2021
48d9989
thrift proxy: fix crash when using payload_passthrough (#14723)
Jan 20, 2021
c82616d
thrift proxy: add comments explaining local replies (#14754)
Jan 21, 2021
4c5af64
wasm: update V8 to v8.9.255.6. (#14764)
PiotrSikora Jan 21, 2021
1febcef
Request headers to add (#14747)
alyssawilk Jan 21, 2021
53eb5c1
tls: update BoringSSL to fbbf8781 (4324). (#14763)
PiotrSikora Jan 21, 2021
fda946c
docs: change getting started order (#14774)
mattklein123 Jan 21, 2021
055ca90
oauth2: set accept header on access_token request (#14538)
riptl Jan 21, 2021
b95006e
router: Remove envoy.reloadable_features.consume_all_retry_headers (#…
numerodix Jan 21, 2021
b501569
docs: API review checklist (#14399)
markdroth Jan 21, 2021
1b0c190
[docs] Add guidance on ENVOY_BUG in STYLE.md (#14575)
asraa Jan 21, 2021
b591644
filters: Add test/server:filter_config_test (#14746)
auni53 Jan 21, 2021
42c75fa
dns: removing envoy.reloadable_features.fix_wildcard_matching #14644 …
alyssawilk Jan 21, 2021
8de5ad9
test: FakeUpstream threading fixes (#14526)
antoniovicente Jan 21, 2021
62f1c83
server: add FIPS mode statistic indicating FIPS compliance (#14719)
raakella Jan 21, 2021
44b5cb7
Add error_state to all config dump resources (#14689)
lidizheng Jan 22, 2021
fd3a942
docs: fix two typos in jwt_authn_filter (#14796)
deejay1 Jan 22, 2021
bb1a16e
tcp: adding logs and debug checks (#14771)
alyssawilk Jan 22, 2021
8a7ca18
google_grpc: attempt to reduce lock contention between completionThre…
antoniovicente Jan 22, 2021
2da224f
filters: Add dependencies.proto (#14750)
auni53 Jan 22, 2021
5bb39c0
tools: Syncing api/BUILD file to generated_api_shadow (#14792)
adisuissa Jan 22, 2021
f8fcf7f
Add debug log for slow config updates for GRPC subscriptions (#14343)
adamsjob Jan 22, 2021
23a97fb
oauth2 filter: Make OAuth scopes configurable. (#14168)
andreyprezotto Jan 23, 2021
a87c9cf
upstream: Optimize LoadStatsReporter::startLoadReportPeriod implement…
antoniovicente Jan 25, 2021
9753819
ext_proc: Implement response path for headers only (#14713)
gbrail Jan 25, 2021
acb9226
http: error handling for bad factory type (#14769)
alyssawilk Jan 25, 2021
6952ef8
docs: fix oauth2 filter example (#14805)
ngoyal16 Jan 25, 2021
90534eb
[kill request] Add API validation to avoid config crash (#14398)
asraa Jan 25, 2021
8814014
overload: scale transport socket connect timeout (#13800)
akonradi Jan 25, 2021
0bc3fc9
tls: separate out cert validation logic from ContextImpl (#14757)
mathetake Jan 26, 2021
b74d0eb
cleanup: Remove unused method ServerConnectionImpl::sendProtocolError…
antoniovicente Jan 26, 2021
86b2f72
doc: polish http/upgrades documentation (#14825)
spacewander Jan 26, 2021
cafccb2
Fix error message (#14817)
auni53 Jan 26, 2021
274e883
grpc_stats: switch default of `stats_for_all_methods` (#14815)
ggreenway Jan 26, 2021
f5468fa
scaled range timer: guard against queue deletion during timer fire (#…
Jan 27, 2021
18212b5
common/http: Remove chromium_url (#14583)
dio Jan 27, 2021
53124e8
quiche: add 100 continue support (#14246)
danzh2010 Jan 27, 2021
55da7a2
Remove extraneous word from release notes (#14840)
ggreenway Jan 27, 2021
268b94b
Update flaky test script to print more details and detect flaky excep…
rmiller14 Jan 27, 2021
4e5c4b9
grpc-web: Fix 503 handling for gRPC Web text (#14516)
dio Jan 28, 2021
2ee9543
docs: Add admin section back to quick-start demo (#14833)
sandipb Jan 28, 2021
ffe97a3
Changes the link to download Kafka (#14858)
davinci26 Jan 29, 2021
4dbcb63
[header map] remove integer setter for string headers (#14773)
asraa Jan 29, 2021
bad818c
Step 6: Add clang cl pipeline for Windows (#14668)
wrowe Jan 29, 2021
5ef3f8d
transcoding: Teach transcoding filter to be aware of per vhost / per …
gagata Jan 29, 2021
b941f6e
[ci] test azp coverage artifact logging (#14691)
asraa Jan 29, 2021
5899f35
test: Break up router_test to avoid compiler OOM under ASAN (#14844)
yanavlasov Jan 29, 2021
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
The table of contents is too big for display.
Diff view
Diff view
  •  
  •  
  •  
2 changes: 1 addition & 1 deletion .azure-pipelines/cve_scan.yml
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ schedules:
displayName: Hourly CVE scan
branches:
include:
- master
- main
always: true

pool:
Expand Down
47 changes: 44 additions & 3 deletions .azure-pipelines/pipelines.yml
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
trigger:
branches:
include:
- "master"
- "main"
- "release/v*"
tags:
include:
Expand Down Expand Up @@ -54,6 +54,7 @@ stages:
- script: ci/run_envoy_docker.sh 'ci/do_ci.sh docs'
workingDirectory: $(Build.SourcesDirectory)
env:
AZP_BRANCH: $(Build.SourceBranch)
ENVOY_DOCKER_BUILD_DIR: $(Build.StagingDirectory)
BAZEL_REMOTE_CACHE: grpcs://remotebuildexecution.googleapis.com
BAZEL_REMOTE_INSTANCE: projects/envoy-ci/instances/default_instance
Expand Down Expand Up @@ -303,6 +304,7 @@ stages:
- script: ci/run_envoy_docker.sh 'ci/do_ci.sh docs'
workingDirectory: $(Build.SourcesDirectory)
env:
AZP_BRANCH: $(Build.SourceBranch)
ENVOY_DOCKER_BUILD_DIR: $(Build.StagingDirectory)
BAZEL_REMOTE_CACHE: grpcs://remotebuildexecution.googleapis.com
BAZEL_REMOTE_INSTANCE: projects/envoy-ci/instances/default_instance
Expand All @@ -321,7 +323,6 @@ stages:
workingDirectory: $(Build.SourcesDirectory)
env:
AZP_BRANCH: $(Build.SourceBranch)
AZP_SHA1: $(Build.SourceVersion)

- stage: verify
dependsOn: ["docker"]
Expand Down Expand Up @@ -384,8 +385,13 @@ stages:
pool:
vmImage: "windows-latest"
steps:
- task: Cache@2
inputs:
key: '"windows.release" | ./WORKSPACE | **/*.bzl'
path: $(Build.StagingDirectory)/repository_cache
continueOnError: true
- bash: ci/run_envoy_docker.sh ci/windows_ci_steps.sh
displayName: "Run Windows CI"
displayName: "Run Windows msvc-cl CI"
env:
CI_TARGET: "windows"
ENVOY_DOCKER_BUILD_DIR: "$(Build.StagingDirectory)"
Expand All @@ -409,6 +415,41 @@ stages:
artifactName: windows.release
condition: always()

- job: clang_cl
timeoutInMinutes: 120
pool:
vmImage: "windows-latest"
steps:
- task: Cache@2
inputs:
key: '"windows.release" | ./WORKSPACE | **/*.bzl'
path: $(Build.StagingDirectory)/repository_cache
continueOnError: true
- bash: ci/run_envoy_docker.sh ci/windows_ci_steps.sh
displayName: "Run Windows clang-cl CI"
env:
CI_TARGET: "windows"
ENVOY_DOCKER_BUILD_DIR: "$(Build.StagingDirectory)"
SLACK_TOKEN: $(SLACK_TOKEN)
REPO_URI: $(Build.Repository.Uri)
BUILD_URI: $(Build.BuildUri)
ENVOY_RBE: "true"
BAZEL_BUILD_EXTRA_OPTIONS: "--config=remote-ci --config=remote-clang-cl --jobs=$(RbeJobs) --flaky_test_attempts=2"
BAZEL_REMOTE_CACHE: grpcs://remotebuildexecution.googleapis.com
BAZEL_REMOTE_INSTANCE: projects/envoy-ci/instances/default_instance
GCP_SERVICE_ACCOUNT_KEY: $(GcpServiceAccountKey)
- task: PublishTestResults@2
inputs:
testResultsFiles: "**/bazel-out/**/testlogs/**/test.xml"
testRunTitle: "clang-cl"
searchFolder: $(Build.StagingDirectory)/tmp
condition: always()
- task: PublishBuildArtifacts@1
inputs:
pathtoPublish: "$(Build.StagingDirectory)/envoy"
artifactName: windows.clang-cl
condition: always()

- job: docker
dependsOn: ["release"]
timeoutInMinutes: 120
Expand Down
4 changes: 2 additions & 2 deletions .bazelrc
Original file line number Diff line number Diff line change
Expand Up @@ -245,8 +245,8 @@ build:remote-clang-cl --config=clang-cl
build:remote-clang-cl --config=rbe-toolchain-clang-cl

# Docker sandbox
# NOTE: Update this from https://github.com/envoyproxy/envoy-build-tools/blob/master/toolchains/rbe_toolchains_config.bzl#L8
build:docker-sandbox --experimental_docker_image=envoyproxy/envoy-build-ubuntu:11efa5680d987fff33fde4af3cc5ece105015d04
# NOTE: Update this from https://github.com/envoyproxy/envoy-build-tools/blob/main/toolchains/rbe_toolchains_config.bzl#L8
build:docker-sandbox --experimental_docker_image=envoyproxy/envoy-build-ubuntu:c8fa4235714003ba0896287ee2f91cae06e0e407
build:docker-sandbox --spawn_strategy=docker
build:docker-sandbox --strategy=Javac=docker
build:docker-sandbox --strategy=Closure=docker
Expand Down
2 changes: 1 addition & 1 deletion .devcontainer/Dockerfile
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
FROM gcr.io/envoy-ci/envoy-build:11efa5680d987fff33fde4af3cc5ece105015d04
FROM gcr.io/envoy-ci/envoy-build:c8fa4235714003ba0896287ee2f91cae06e0e407

ARG USERNAME=vscode
ARG USER_UID=501
Expand Down
4 changes: 2 additions & 2 deletions .github/ISSUE_TEMPLATE/non--crash-security--bug.md
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@ returned, etc.
> Include sample requests, environment, etc. All data and inputs
required to reproduce the bug.

>**Note**: The [Envoy_collect tool](https://github.com/envoyproxy/envoy/blob/master/tools/envoy_collect/README.md)
>**Note**: The [Envoy_collect tool](https://github.com/envoyproxy/envoy/blob/main/tools/envoy_collect/README.md)
gathers a tarball with debug logs, config and the following admin
endpoints: /stats, /clusters and /server_info. Please note if there are
privacy concerns, sanitize the data prior to sharing the tarball/pasting.
Expand All @@ -46,4 +46,4 @@ sharing.

*Call Stack*:
> If the Envoy binary is crashing, a call stack is **required**.
Please refer to the [Bazel Stack trace documentation](https://github.com/envoyproxy/envoy/tree/master/bazel#stack-trace-symbol-resolution).
Please refer to the [Bazel Stack trace documentation](https://github.com/envoyproxy/envoy/tree/main/bazel#stack-trace-symbol-resolution).
4 changes: 2 additions & 2 deletions .github/workflows/get_build_targets.sh
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ readonly SEARCH_FOLDER="//source/common/..."
set -e -o pipefail

function get_targets() {
# Comparing the PR HEAD with the upstream master HEAD.
# Comparing the PR HEAD with the upstream main HEAD.
git diff --name-only HEAD FETCH_HEAD | while IFS= read -r line
do
# Only targets under those folders.
Expand All @@ -23,6 +23,6 @@ function get_targets() {
}

# Fetching the upstream HEAD to compare with and stored in FETCH_HEAD.
git fetch https://github.com/envoyproxy/envoy.git master 2>/dev/null
git fetch https://github.com/envoyproxy/envoy.git main 2>/dev/null

export BUILD_TARGETS_LOCAL=$(echo $(get_targets))
2 changes: 2 additions & 0 deletions CODEOWNERS
Original file line number Diff line number Diff line change
Expand Up @@ -160,3 +160,5 @@ extensions/filters/http/oauth2 @rgs1 @derekargueta @snowp
/*/extensions/filters/common/local_ratelimit @mattklein123 @rgs1
# HTTP Kill Request
/*/extensions/filters/http/kill_request @qqustc @htuch
# Rate limit expression descriptor
/*/extensions/rate_limit_descriptors/expr @kyessenov @lizan
23 changes: 15 additions & 8 deletions CONTRIBUTING.md
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,7 @@ versioning guidelines:

* Features may be marked as deprecated in a given versioned API at any point in time, but this may
only be done when a replacement implementation and configuration path is available in Envoy on
master. Deprecators must implement a conversion from the deprecated configuration to the latest
main. Deprecators must implement a conversion from the deprecated configuration to the latest
`vNalpha` (with the deprecated field) that Envoy uses internally. A field may be deprecated if
this tool would be able to perform the conversion. For example, removing a field to describe
HTTP/2 window settings is valid if a more comprehensive HTTP/2 protocol options field is being
Expand All @@ -73,7 +73,7 @@ versioning guidelines:
`envoy.features.enable_all_deprecated_features` is set to true. Finally, following the deprecation
of the API major version where the field was first marked deprecated, the entire implementation
code will be removed from the Envoy implementation.
* This policy means that organizations deploying master should have some time to get ready for
* This policy means that organizations deploying main should have some time to get ready for
breaking changes at the next major API version. This is typically a window of at least 12 months
or until the organization moves to the next major API version.
* The breaking change policy also applies to source level extensions (e.g., filters). Code that
Expand All @@ -99,7 +99,7 @@ versioning guidelines:

Please see [support/README.md](support/README.md) for more information on these hooks.

* Create your PR.
* Create your PR. If your PR adds new code, it should include tests [covering](source/docs/coverage.md) the new code.
* Tests will automatically run for you.
* We will **not** merge any PR that is not passing tests.
* PRs are expected to have 100% test coverage for added code. This can be verified with a coverage
Expand Down Expand Up @@ -144,7 +144,7 @@ versioning guidelines:
* If your PR involves any changes to
[envoy-filter-example](https://github.com/envoyproxy/envoy-filter-example) (for example making a new
branch so that CI can pass) it is your responsibility to follow through with merging those
changes back to master once the CI dance is done.
changes back to main once the CI dance is done.
* If your PR is a high risk change, the reviewer may ask that you runtime guard
it. See the section on runtime guarding below.

Expand Down Expand Up @@ -189,18 +189,18 @@ maintainer's discretion. Generally all runtime guarded features will be set true
release is cut. Old code paths for refactors can be cleaned up after a release and there has been
some production run time. Old code for behavioral changes will be deprecated after six months.
Runtime features are set true by default by inclusion in
[source/common/runtime/runtime_features.cc](https://github.com/envoyproxy/envoy/blob/master/source/common/runtime/runtime_features.cc)
[source/common/runtime/runtime_features.cc](https://github.com/envoyproxy/envoy/blob/main/source/common/runtime/runtime_features.cc)

There are four suggested options for testing new runtime features:

1. Create a per-test Runtime::LoaderSingleton as done in [DeprecatedFieldsTest.IndividualFieldDisallowedWithRuntimeOverride](https://github.com/envoyproxy/envoy/blob/master/test/common/protobuf/utility_test.cc)
1. Create a per-test Runtime::LoaderSingleton as done in [DeprecatedFieldsTest.IndividualFieldDisallowedWithRuntimeOverride](https://github.com/envoyproxy/envoy/blob/main/test/common/protobuf/utility_test.cc)
2. Create a [parameterized test](https://github.com/google/googletest/blob/master/googletest/docs/advanced.md#how-to-write-value-parameterized-tests)
where the set up of the test sets the new runtime value explicitly to
GetParam() as outlined in (1).
3. Set up integration tests with custom runtime defaults as documented in the
[integration test README](https://github.com/envoyproxy/envoy/blob/master/test/integration/README.md)
[integration test README](https://github.com/envoyproxy/envoy/blob/main/test/integration/README.md)
4. Run a given unit test with the new runtime value explicitly set true or false as done
for [runtime_flag_override_test](https://github.com/envoyproxy/envoy/blob/master/test/common/runtime/BUILD)
for [runtime_flag_override_test](https://github.com/envoyproxy/envoy/blob/main/test/common/runtime/BUILD)

Runtime code is held to the same standard as regular Envoy code, so both the old
path and the new should have 100% coverage both with the feature defaulting true
Expand Down Expand Up @@ -233,6 +233,13 @@ and false.
* If a PR includes a deprecation/breaking change, notification should be sent to the
[envoy-announce](https://groups.google.com/forum/#!forum/envoy-announce) email list.

# API changes

If you change anything in the [api tree](https://github.com/envoyproxy/envoy/tree/master/api),
please read the [API Review
Checklist](https://github.com/envoyproxy/envoy/tree/master/api/review_checklist.md)
and make sure that your changes have addressed all of the considerations listed there.

# Adding new extensions

For developers adding a new extension, one can take an existing extension as the starting point.
Expand Down
4 changes: 2 additions & 2 deletions DEPENDENCY_POLICY.md
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,7 @@ Dependency declarations must:
and `urls` to reference the version. If you need to reference version `X.Y.Z` as `X_Y_Z`, this
may appear in a string as `{underscore_version}`, similarly for `X-Y-Z` you can use
`{dash_version}`.
* Versions should prefer release versions over master branch GitHub SHA tarballs. A comment is
* Versions should prefer release versions over main branch GitHub SHA tarballs. A comment is
necessary if the latter is used. This comment should contain the reason that a non-release
version is being used.
* Provide accurate entries for `use_category`. Please think carefully about whether there are data
Expand Down Expand Up @@ -112,7 +112,7 @@ basis:

* Extension [CODEOWNERS](CODEOWNERS) should update extension specific dependencies.

Where possible, we prefer the latest release version for external dependencies, rather than master
Where possible, we prefer the latest release version for external dependencies, rather than main
branch GitHub SHA tarballs.

## Dependency shepherds
Expand Down
30 changes: 15 additions & 15 deletions DEVELOPER.md
Original file line number Diff line number Diff line change
@@ -1,38 +1,38 @@
# Developer documentation

Envoy is built using the Bazel build system. Our CI on Azure Pipelines builds, tests, and runs coverage against
all pull requests and the master branch.
all pull requests and the main branch.

To get started building Envoy locally, see the [Bazel quick start](https://github.com/envoyproxy/envoy/blob/master/bazel/README.md#quick-start-bazel-build-for-developers).
To run tests, there are Bazel [targets](https://github.com/envoyproxy/envoy/blob/master/bazel/README.md#testing-envoy-with-bazel) for Google Test.
To generate a coverage report, there is a [coverage build script](https://github.com/envoyproxy/envoy/blob/master/bazel/README.md#coverage-builds).
To get started building Envoy locally, see the [Bazel quick start](https://github.com/envoyproxy/envoy/blob/main/bazel/README.md#quick-start-bazel-build-for-developers).
To run tests, there are Bazel [targets](https://github.com/envoyproxy/envoy/blob/main/bazel/README.md#testing-envoy-with-bazel) for Google Test.
To generate a coverage report, there is a [coverage build script](https://github.com/envoyproxy/envoy/blob/main/bazel/README.md#coverage-builds).

If you plan to contribute to Envoy, you may find it useful to install the Envoy [development support toolchain](https://github.com/envoyproxy/envoy/blob/master/support/README.md), which helps automate parts of the development process, particularly those involving code review.
If you plan to contribute to Envoy, you may find it useful to install the Envoy [development support toolchain](https://github.com/envoyproxy/envoy/blob/main/support/README.md), which helps automate parts of the development process, particularly those involving code review.

Below is a list of additional documentation to aid the development process:

- [General build and installation documentation](https://www.envoyproxy.io/docs/envoy/latest/start/start)

- [Building and testing Envoy with Bazel](https://github.com/envoyproxy/envoy/blob/master/bazel/README.md)
- [Building and testing Envoy with Bazel](https://github.com/envoyproxy/envoy/blob/main/bazel/README.md)

- [Managing external dependencies with Bazel](https://github.com/envoyproxy/envoy/blob/master/bazel/EXTERNAL_DEPS.md)
- [Managing external dependencies with Bazel](https://github.com/envoyproxy/envoy/blob/main/bazel/EXTERNAL_DEPS.md)

- [Guide to Envoy Bazel rules (managing `BUILD` files)](https://github.com/envoyproxy/envoy/blob/master/bazel/DEVELOPER.md)
- [Guide to Envoy Bazel rules (managing `BUILD` files)](https://github.com/envoyproxy/envoy/blob/main/bazel/DEVELOPER.md)

- [Using Docker for building and testing](https://github.com/envoyproxy/envoy/tree/master/ci)
- [Using Docker for building and testing](https://github.com/envoyproxy/envoy/tree/main/ci)

- [Guide to contributing to Envoy](https://github.com/envoyproxy/envoy/blob/master/CONTRIBUTING.md)
- [Guide to contributing to Envoy](https://github.com/envoyproxy/envoy/blob/main/CONTRIBUTING.md)

- [Overview of Envoy's testing frameworks](https://github.com/envoyproxy/envoy/blob/master/test/README.md)
- [Overview of Envoy's testing frameworks](https://github.com/envoyproxy/envoy/blob/main/test/README.md)

- [Overview of how to write integration tests for new code](https://github.com/envoyproxy/envoy/blob/master/test/integration/README.md)
- [Overview of how to write integration tests for new code](https://github.com/envoyproxy/envoy/blob/main/test/integration/README.md)

- [Envoy filter example project (how to consume and extend Envoy as a submodule)](https://github.com/envoyproxy/envoy-filter-example)

- [Performance testing Envoy with `tcmalloc`/`pprof`](https://github.com/envoyproxy/envoy/blob/master/bazel/PPROF.md)
- [Performance testing Envoy with `tcmalloc`/`pprof`](https://github.com/envoyproxy/envoy/blob/main/bazel/PPROF.md)

And some documents on components of Envoy architecture:

- [Envoy flow control](https://github.com/envoyproxy/envoy/blob/master/source/docs/flow_control.md)
- [Envoy flow control](https://github.com/envoyproxy/envoy/blob/main/source/docs/flow_control.md)

- [Envoy's subset load balancer](https://github.com/envoyproxy/envoy/blob/master/source/docs/subset_load_balancer.md)
- [Envoy's subset load balancer](https://github.com/envoyproxy/envoy/blob/main/source/docs/subset_load_balancer.md)
8 changes: 4 additions & 4 deletions EXTENSION_POLICY.md
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@ The following procedure will be used when proposing new extensions for inclusion
2. All extensions must be sponsored by an existing maintainer. Sponsorship means that the
maintainer will shepherd the extension through design/code reviews. Maintainers can self-sponsor
extensions if they are going to write them, shepherd them, and maintain them.

Sponsorship serves two purposes:
* It ensures that the extension will ultimately meet the Envoy quality bar.
* It makes sure that incentives are aligned and that extensions are not added to the repo without
Expand All @@ -24,7 +24,7 @@ The following procedure will be used when proposing new extensions for inclusion
*If sponsorship cannot be found from an existing maintainer, an organization can consider
[doing the work to become a maintainer](./GOVERNANCE.md#process-for-becoming-a-maintainer) in
order to be able to self-sponsor extensions.*

3. Each extension must have two reviewers proposed for reviewing PRs to the extension. Neither of
the reviewers must be a senior maintainer. Existing maintainers (including the sponsor) and other
contributors can count towards this number. The initial reviewers will be codified in the
Expand Down Expand Up @@ -105,7 +105,7 @@ The `security_posture` is one of:
* `unknown`: This is functionally equivalent to `requires_trusted_downstream_and_upstream`, but acts
as a placeholder to allow us to identify extensions that need classifying.
* `data_plane_agnostic`: Not relevant to data plane threats, e.g. stats sinks.

An assessment of a robust security posture for an extension is subject to the following guidelines:

* Does the extension have fuzz coverage? If it's only receiving fuzzing
Expand All @@ -122,7 +122,7 @@ An assessment of a robust security posture for an extension is subject to the fo
* Does the extension have active [CODEOWNERS](CODEOWNERS) who are willing to
vouch for the robustness of the extension?
* Is the extension absent a [low coverage
exception](https://github.com/envoyproxy/envoy/blob/master/test/per_file_coverage.sh#L5)?
exception](https://github.com/envoyproxy/envoy/blob/main/test/per_file_coverage.sh#L5)?

The current stability and security posture of all extensions can be seen
[here](https://www.envoyproxy.io/docs/envoy/latest/intro/arch_overview/security/threat_model#core-and-extensions).
Expand Down
Loading