Skip to content

Use upb textformat library to dump out raw xDS proto messages#23140

Closed
markdroth wants to merge 124 commits intogrpc:masterfrom
markdroth:xds_logging.alt
Closed

Use upb textformat library to dump out raw xDS proto messages#23140
markdroth wants to merge 124 commits intogrpc:masterfrom
markdroth:xds_logging.alt

Conversation

@markdroth
Copy link
Copy Markdown
Member

@markdroth markdroth commented Jun 4, 2020

Built on #23122.

ce1a399a19 Text format serializer for upb_msg (#242)
888f35cae6 Merge pull request #241 from haberman/conformance-fixes
46b93f8cea A bit more cleanup in the decoder.
ad2eb65a4b Refactored conformance_upb to use reflection, and fixed a decoder bug.
a202c5f84d Merge pull request #234 from haberman/parser
4c974cf72d Fixed generated files.
9a870d957f Removed upb_decframe and made ptr an explicit parameter and return.
a6c54729df Added UPB_ASSUME(), to work around warnings when optimization is enabled:
9e1f89ef2c Merge pull request #224 from haberman/maps
ce0496eb86 Merge branch 'master' into maps
e911aae5f6 Factored upb_map_entry into a shared place.
4c1a97ae9c Merge branch 'master' into maps
59fe620fa0 Merge branch 'maps' of github.com:haberman/upb into maps
744f8588da Cleanup to remove END_GROUP from descriptortype -> type tables.
f9efbcd5d6 Added missing append fallback.
c4b64e6a20 Slight simplification: NULL arena will avoid creating a new sub-object.
d541566a7b Moved upb_array_new() to upb/reflection.h where it belongs.
059f226d41 Unit tests for maps generated code.
520ddc1f11 c89 fixes.
806c8c9c6e Removed obsolete testing files.
2a85bef825 Generated code interface for maps is complete, though not yet tested.
7f5fe52dfa Fixes for non-C89 code.
6c2d732082 Fixed upb's map parsing to overwrite existing elements.
090a0c33a4 Fixed VLA error and rewrote the map parsing code to be clearer.
0fbae939d2 Removed stray fprintf().
572ba75d1c Removed comma after final enumerator.
c9135e5276 Fixed the build.
d040aa1302 Merge branch 'master' into maps
e18541a9dd Added some missing files.
92509cc3b2 Rename lua_test.
382f92a87f Maps encode and decode successfully!
4c57b1fefd More progress on Lua extension.
d6c3152c0b Added more Lua tests that are passing.
ae66e571d4 Fixed some bugs and added a few more tests.
bfc86d3577 Fixed many bugs, basic Lua test passes!
b518b06d75 Lua test program is loaded successfully.
6ae4a2694c Merge branch 'maps' of github.com:haberman/upb into maps
cc6db9fb0b Fixed crash bug.
88d996132e Added Lua main.c test driver program.
626ec4bfcf Everything builds, test pass except test_decoder.
5239655b99 WIP.
23825332e1 WIP.
27b95c969a WIP.
9a360ad43d Moved legacy_msg_reflection.{c,h} -> reflection.{c.h}.
dc58b657ee New reflection API doesn't need types as parameters for map/array.
c486da3970 WIP.
b76040cfcc Merge branch 'maps' of github.com:haberman/upb into maps
cc8e894b63 Merge branch 'master' into maps
946880c105 Merge branch 'master' into maps
1461da5056 WIP.
0a07f2714b Merge branch 'master' into maps
18de110b00 Merge branch 'master' into maps
283857f308 WIP.
5dea3f8486 Merge branch 'master' into maps
07ac6f0e8e Merge branch 'master' into maps
0c64c4b594 WIP.

git-subtree-dir: third_party/upb
git-subtree-split: ce1a399a19f11683d58ba4c2569ec3fdd5a67621
3d955e684c Added "extern C" blocks to textencode. (#244)
ca512852f3 Fixed parsing for string->double maps. (#243)

git-subtree-dir: third_party/upb
git-subtree-split: 3d955e684c91feff188de1f2509e55aae291762a
4c6dcc3c6b [textformat]: added missing newline when a message opens. (#245)

git-subtree-dir: third_party/upb
git-subtree-split: 4c6dcc3c6bdb7039995768bf5f95ac2c4a6e9102
ca84145476 [decoder] fix for buf=NULL, size=0 case. (#246)

git-subtree-dir: third_party/upb
git-subtree-split: ca84145476ddd722c6d67d7cf985d1d0aefcd893
markdroth added 25 commits June 3, 2020 11:18
b717575cef Added -Wextra and -Wshorten-64-to-32 and fixed resulting errors. (#289)
2e335ab9d2 Merge pull request #288 from haberman/ubsan
408d34797d Disable UBSan, Clang is not available in upb's Kokoro image atm.
6b808a4072 Fixed all UBSan issues and added UBSan CI checks.

git-subtree-dir: third_party/upb
git-subtree-split: b717575ceff5e759a28f3ed3d60a65eee041a30c
363e39c171 Fix for extra compiler warnings. (#290)

git-subtree-dir: third_party/upb
git-subtree-split: 363e39c171a08b89bd546189d1244a6e191cacf0
@markdroth markdroth added the release notes: no Indicates if PR should not be in release notes label Jun 8, 2020
@markdroth markdroth changed the title Xds logging.alt Use upb textformat library to dump out raw xDS proto messages Jun 8, 2020
@markdroth markdroth closed this Jul 9, 2020
@markdroth markdroth deleted the xds_logging.alt branch July 9, 2020 20:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release notes: no Indicates if PR should not be in release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants