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.

Sign up for GitHub

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

Jump to bottom

[internal/comparetest] Ensure all the fields are checked #17865

Closed
Tracked by #17544
dmitryax opened this issue Jan 19, 2023 · 0 comments · Fixed by #18050
Closed
Tracked by #17544

[internal/comparetest] Ensure all the fields are checked #17865

dmitryax opened this issue Jan 19, 2023 · 0 comments · Fixed by #18050
Assignees
@dmitryax

Comments

@dmitryax
Copy link
Member

dmitryax commented Jan 19, 2023 •
edited
Loading

Currently, some fields are ignored, e.g., many are missed in the number datapoints comparison. We need to ensure all data fields are validated and introduce explicit options to ignore some fields like timestamps.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@dmitryax dmitryax

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[pkg/pdatatest] Ensure all fields are checked dmitryax/opentelemetry-collector-contrib
1 participant
@dmitryax