Skip to content

Apply three commits from f5/otel-arrow-adapter#1

Merged
jmacd merged 3 commits into
mainfrom
jmacd/three_commits
Aug 10, 2023
Merged

Apply three commits from f5/otel-arrow-adapter#1
jmacd merged 3 commits into
mainfrom
jmacd/three_commits

Conversation

@jmacd
Copy link
Copy Markdown
Contributor

@jmacd jmacd commented Aug 10, 2023

Although I initialized this repository by pushing all the commits from the old repo, I couldn't update the new repo with three commits since then because of EasyCLA triggers failing. I am attempting to merge the same three commits via a PR.

moh-osman3 and others added 3 commits August 7, 2023 09:44
* config

* add timeout

* wip

* not working

* uncomment stream restart

* remove default and fix unit tests

* validate that user sets maxstreamlifetime

* fix tests

* fix format

* comment out closesend

* add timer

* tidy and fix mock api issue

* not updated?

* fix failed test

* check status message is exact match with EOF

* update config

* broken unit test

* fix test?

* call streamCall once

* address review feedback

---------

Co-authored-by: Laurent Quérel <laurent.querel@gmail.com>
* Update validation process documentation

* Fix exemplar issue in metrics

* Improve metric generator to explore more corner cases

* Make assert.Equiv independent of the testing framework

* Fix bug that occurs when a log record has no attribute

* Apply same optimization and fix to traces
…validation; minor fixes (#203)

* skeleton span validator

* three validation files

* three validation files (part 2)

* missing files

* diagram

* WIP validation hacks

* both do not mutate data

* add the test verifier

* Add some HOWTO

* Write up the validation connector

* edit README

* tidy comments

* fix test

---------

Co-authored-by: Laurent Quérel <laurent.querel@gmail.com>
@jmacd jmacd requested a review from lquerel August 10, 2023 14:49
@jmacd jmacd merged commit e961deb into main Aug 10, 2023
@jmacd
Copy link
Copy Markdown
Contributor Author

jmacd commented Aug 10, 2023

I didn't press the merge button. I pushed the branch with -f and ... confused

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.

3 participants