Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
37 commits
Select commit Hold shift + click to select a range
7d5f240
wasm: update Proxy-Wasm C++ SDK to latest.
PiotrSikora Jan 17, 2022
01f7cdc
review: update use_category for emsdk.
PiotrSikora Jan 17, 2022
7c3efe8
review: add emdwp tool.
PiotrSikora Jan 17, 2022
b1459cd
review: try using /bin/false instead.
PiotrSikora Jan 17, 2022
e2b6284
review: don't advertise "per_object_debug_info" feature.
PiotrSikora Jan 18, 2022
fdae2ca
review: allow @emsdk to be imported into non-x86_64 workspace.
PiotrSikora Jan 18, 2022
539aa58
review: remove sanitizers from flags propagated to Emscripten.
PiotrSikora Jan 18, 2022
9fd86f5
review: update Proxy-Wasm C++ SDK to fix tags.
PiotrSikora Jan 18, 2022
6ec9108
review: skip some tests that fail on WAMR due to a small stack size.
PiotrSikora Jan 18, 2022
c59d140
review: add comments to the patch.
PiotrSikora Jan 18, 2022
7865cf5
review: use the default (latest) version of Emscripten.
PiotrSikora Jan 18, 2022
771b8b3
review: use uintptr_t to fix Windows build.
PiotrSikora Jan 18, 2022
9ea9536
review: update condition for excluding C++ tests.
PiotrSikora Jan 18, 2022
ff158ce
review: better approach to sanitizers.
PiotrSikora Jan 18, 2022
33ef8a8
Merge remote-tracking branch 'origin/main' into PiotrSikora/emsdk_v3.1.1
PiotrSikora Jan 18, 2022
c97a2da
review: non-ambiguous select.
PiotrSikora Jan 18, 2022
caceac1
review: fix visibility.
PiotrSikora Jan 18, 2022
36e186d
review: uintptr_t -> unsigned int.
PiotrSikora Jan 18, 2022
691f7a5
review: bump timeout for a test.
PiotrSikora Jan 18, 2022
aaae279
review: suppress warning on Windows.
PiotrSikora Jan 18, 2022
ef0f5b4
review: ignore unknown pragmas.
PiotrSikora Jan 18, 2022
c5a4f72
review: revert changes for excluding C++ tests.
PiotrSikora Jan 20, 2022
e11caee
review: revert hacks for Windows.
PiotrSikora Jan 20, 2022
60a178c
Merge remote-tracking branch 'origin/main' into PiotrSikora/emsdk_v3.1.1
PiotrSikora Jan 20, 2022
c5aeb17
Merge remote-tracking branch 'origin/main' into PiotrSikora/emsdk_v3.1.1
PiotrSikora Feb 9, 2022
29591f5
Merge remote-tracking branch 'origin/main' into PiotrSikora/emsdk_v3.1.1
PiotrSikora Mar 11, 2022
7387adf
review: update to emsdk v3.1.7 with Bazel fixes.
PiotrSikora Mar 11, 2022
7f503b0
review: undo no longer necessary changes in tests.
PiotrSikora Mar 11, 2022
1b7584c
review: use envoy_wasm_cc_binary(protobuf="lite|full").
PiotrSikora Mar 11, 2022
a61fda1
review: style.
PiotrSikora Mar 11, 2022
4a00dd7
review: bump Proxy-Wasm C++ SDK.
PiotrSikora Mar 11, 2022
89dfafc
review: update Proxy-Wasm C++ SDK.
PiotrSikora Mar 14, 2022
99d087d
Merge remote-tracking branch 'origin/main' into PiotrSikora/emsdk_v3.1.1
PiotrSikora Mar 14, 2022
b684a75
review: use merged commit.
PiotrSikora Mar 15, 2022
c87e7ce
Merge remote-tracking branch 'origin/main' into PiotrSikora/emsdk_v3.1.1
PiotrSikora Mar 15, 2022
b153d34
Merge remote-tracking branch 'origin/main' into PiotrSikora/emsdk_v3.1.1
PiotrSikora Mar 17, 2022
515752f
Merge remote-tracking branch 'origin/main' into PiotrSikora/emsdk_v3.1.1
PiotrSikora Mar 18, 2022
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 2 additions & 0 deletions bazel/dependency_imports.bzl
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,7 @@ load("@rules_antlr//antlr:deps.bzl", "antlr_dependencies")
load("@proxy_wasm_rust_sdk//bazel:dependencies.bzl", "proxy_wasm_rust_sdk_dependencies")
load("@rules_cc//cc:repositories.bzl", "rules_cc_dependencies", "rules_cc_toolchains")
load("@base_pip3//:requirements.bzl", pip_dependencies = "install_deps")
load("@emsdk//:emscripten_deps.bzl", "emscripten_deps")

# go version for rules_go
GO_VERSION = "1.17.5"
Expand All @@ -31,6 +32,7 @@ def envoy_dependency_imports(go_version = GO_VERSION):
)
rules_cc_dependencies()
rules_cc_toolchains()
emscripten_deps()

# These dependencies, like most of the Go in this repository, exist only for the API.
go_repository(
Expand Down
15 changes: 3 additions & 12 deletions bazel/repositories.bzl
Original file line number Diff line number Diff line change
Expand Up @@ -206,7 +206,7 @@ def envoy_dependencies(skip_targets = []):
_upb()
_proxy_wasm_cpp_sdk()
_proxy_wasm_cpp_host()
_emscripten_toolchain()
_emsdk()
_rules_fuzzing()
external_http_archive("proxy_wasm_rust_sdk")
_com_google_cel_cpp()
Expand Down Expand Up @@ -978,17 +978,8 @@ def _proxy_wasm_cpp_sdk():
def _proxy_wasm_cpp_host():
external_http_archive(name = "proxy_wasm_cpp_host")

def _emscripten_toolchain():
external_http_archive(
name = "emscripten_toolchain",
build_file_content = _build_all_content(exclude = [
"upstream/emscripten/cache/is_vanilla.txt",
".emscripten_sanity",
]),
patch_cmds = [
"if [[ \"$(uname -m)\" == \"x86_64\" ]]; then ./emsdk install 3.1.7 && ./emsdk activate --embedded 3.1.7; fi",
],
)
def _emsdk():
external_http_archive(name = "emsdk")
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Not dig into it, now we can install emsdk on OS except x86?


def _com_github_google_jwt_verify():
external_http_archive("com_github_google_jwt_verify")
Expand Down
2 changes: 2 additions & 0 deletions bazel/repositories_extra.bzl
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
load("@emsdk//:deps.bzl", emsdk_deps = "deps")
load("@rules_python//python:pip.bzl", "pip_install", "pip_parse")
load("@proxy_wasm_cpp_host//bazel/cargo/wasmtime:crates.bzl", "wasmtime_fetch_remote_crates")
load("//bazel/external/cargo:crates.bzl", "raze_fetch_remote_crates")
Expand Down Expand Up @@ -25,5 +26,6 @@ def _python_deps():
# Envoy deps that rely on a first stage of dependency loading in envoy_dependencies().
def envoy_dependencies_extra():
_python_deps()
emsdk_deps()
raze_fetch_remote_crates()
wasmtime_fetch_remote_crates()
19 changes: 10 additions & 9 deletions bazel/repository_locations.bzl
Original file line number Diff line number Diff line change
Expand Up @@ -1021,8 +1021,8 @@ REPOSITORY_LOCATIONS_SPEC = dict(
project_name = "WebAssembly for Proxies (C++ SDK)",
project_desc = "WebAssembly for Proxies (C++ SDK)",
project_url = "https://github.com/proxy-wasm/proxy-wasm-cpp-sdk",
version = "898aaed73e69c453eeefd08ca8c405dd056f2251",
sha256 = "8fb86b8c2ed8a546f54d06a355543f85796d86f3b3a7a0c6e0466f9d2152ef92",
version = "e30535b7c0cd3126e6401bc3769063a74bbb37be",
sha256 = "94e474ebea782225821224734ed5992fa749301e12e06b6520b8b4d4e1c05ffc",
strip_prefix = "proxy-wasm-cpp-sdk-{version}",
urls = ["https://github.com/proxy-wasm/proxy-wasm-cpp-sdk/archive/{version}.tar.gz"],
use_category = ["dataplane_ext"],
Expand All @@ -1038,7 +1038,7 @@ REPOSITORY_LOCATIONS_SPEC = dict(
"envoy.wasm.runtime.wavm",
"envoy.wasm.runtime.wasmtime",
],
release_date = "2022-03-11",
release_date = "2022-03-15",
cpe = "N/A",
),
proxy_wasm_cpp_host = dict(
Expand Down Expand Up @@ -1077,16 +1077,17 @@ REPOSITORY_LOCATIONS_SPEC = dict(
release_date = "2021-07-13",
cpe = "N/A",
),
emscripten_toolchain = dict(
emsdk = dict(
project_name = "Emscripten SDK",
project_desc = "Emscripten SDK (use by Wasm)",
project_url = "https://github.com/emscripten-core/emsdk",
version = "3.1.7",
sha256 = "bcceced0b7cad2e08375adf74ef20fa431230abbae8766bdad268c43e34f8d03",
strip_prefix = "emsdk-{version}",
# v3.1.7 with Bazel fixes
version = "0ea8f8a8707070e9a7c83fbb4a3065683bcf1799",
sha256 = "1ca0ff918d476c55707bb99bc0452be28ac5fb8f22a9260a8aae8a38d1bc0e27",
strip_prefix = "emsdk-{version}/bazel",
urls = ["https://github.com/emscripten-core/emsdk/archive/{version}.tar.gz"],
use_category = ["build"],
release_date = "2022-03-07",
use_category = ["test_only"],
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Does it mean we use a test version?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

It means this project is only compiled and used for tests (not dataplane, controlplane, etc.).
See:

USE_CATEGORIES = [
# This dependency is used in API protos.
"api",
# This dependency is used in build process.
"build",
# This dependency is used to process xDS requests.
"controlplane",
# This dependency is used in processing downstream or upstream requests (core).
"dataplane_core",
# This dependency is used in processing downstream or upstream requests (extensions).
"dataplane_ext",
# This dependecy is used for logging, metrics or tracing (core). It may process unstrusted input.
"observability_core",
# This dependecy is used for logging, metrics or tracing (extensions). It may process unstrusted input.
"observability_ext",
# This dependency does not handle untrusted data and is used for various utility purposes.
"other",
# This dependency is used only in tests.
"test_only",
# Documentation generation
"docs",
# Developer tools (not used in build or docs)
"devtools",
]

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Oh, I miss it, thanks!

release_date = "2022-03-09",
),
rules_rust = dict(
project_name = "Bazel rust rules",
Expand Down
2 changes: 1 addition & 1 deletion bazel/wasm/wasm.bzl
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
load("@proxy_wasm_cpp_sdk//bazel/wasm:wasm.bzl", "proxy_wasm_cc_binary")
load("@proxy_wasm_cpp_sdk//bazel:defs.bzl", "proxy_wasm_cc_binary")
load("@rules_rust//rust:rust.bzl", "rust_binary")
load("//bazel:envoy_select.bzl", "envoy_select_wasm_v8_bool")

Expand Down
3 changes: 0 additions & 3 deletions test/extensions/access_loggers/wasm/test_data/BUILD
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,4 @@ envoy_cc_library(
envoy_wasm_cc_binary(
name = "test_cpp.wasm",
srcs = ["test_cpp.cc"],
deps = [
"@proxy_wasm_cpp_sdk//:proxy_wasm_intrinsics_lite",
],
)
4 changes: 1 addition & 3 deletions test/extensions/bootstrap/wasm/test_data/BUILD
Original file line number Diff line number Diff line change
Expand Up @@ -108,9 +108,7 @@ envoy_wasm_cc_binary(
envoy_wasm_cc_binary(
name = "speed_cpp.wasm",
srcs = ["speed_cpp.cc"],
deps = [
"@proxy_wasm_cpp_sdk//:proxy_wasm_intrinsics_full",
],
protobuf = "full",
)

envoy_wasm_cc_binary(
Expand Down
2 changes: 1 addition & 1 deletion test/extensions/filters/http/wasm/test_data/BUILD
Original file line number Diff line number Diff line change
Expand Up @@ -154,11 +154,11 @@ envoy_wasm_cc_binary(
"test_shared_data_cpp.cc",
"test_shared_queue_cpp.cc",
],
protobuf = "lite",
deps = [
":test_cc_proto",
"//source/extensions/common/wasm/ext:declare_property_cc_proto",
"//source/extensions/common/wasm/ext:envoy_proxy_wasm_api_lib",
"@proxy_wasm_cpp_sdk//:proxy_wasm_intrinsics_lite",
"@proxy_wasm_cpp_sdk//contrib:contrib_lib",
],
)
Expand Down
3 changes: 0 additions & 3 deletions test/extensions/filters/network/wasm/test_data/BUILD
Original file line number Diff line number Diff line change
Expand Up @@ -71,7 +71,4 @@ envoy_wasm_cc_binary(
"test_panic_cpp.cc",
"test_resume_call_cpp.cc",
],
deps = [
"@proxy_wasm_cpp_sdk//:proxy_wasm_intrinsics_lite",
],
)