Skip to content

Adding two tests for 400ZR chromatic dispersion and eSNR#2236

Merged
dplore merged 36 commits into
openconfig:mainfrom
ahsaanyousaf:main
Oct 20, 2023
Merged

Adding two tests for 400ZR chromatic dispersion and eSNR#2236
dplore merged 36 commits into
openconfig:mainfrom
ahsaanyousaf:main

Conversation

@ahsaanyousaf
Copy link
Copy Markdown
Contributor

Tests to ensure 400ZR chromatic dispersion and electrical signal to noise ratio is streamed correctly.

@OpenConfigBot
Copy link
Copy Markdown

OpenConfigBot commented Oct 10, 2023

Pull Request Functional Test Report for #2236 / 7974b27

No tests identified for validation.

Help

@ahsaanyousaf ahsaanyousaf marked this pull request as ready for review October 10, 2023 04:49
@ahsaanyousaf ahsaanyousaf requested a review from a team as a code owner October 10, 2023 04:49
@ahsaanyousaf ahsaanyousaf requested a review from a team October 10, 2023 04:49
Comment thread feature/platform/transceiver/zr_chromatic_dispersion_test/README.MD Outdated
Comment thread feature/platform/transceiver/zr_chromatic_dispersion_test/README.MD Outdated
Comment thread feature/platform/transceiver/zr_electrical_signal_to_noise_ratio_test/README.MD Outdated
Comment thread feature/platform/transceiver/zr_chromatic_dispersion_test/README.MD Outdated
Comment thread feature/platform/transceiver/zr_chromatic_dispersion_test/README.MD Outdated
Comment thread feature/platform/transceiver/zr_electrical_signal_to_noise_ratio_test/README.MD Outdated
Comment thread feature/platform/transceiver/zr_esnr_and_cd_test/README.MD Outdated
Comment thread feature/platform/transceiver/zr_esnr_and_cd_test/README.MD Outdated
Comment thread feature/platform/transceiver/zr_esnr_and_cd_test/README.MD Outdated
Comment thread feature/platform/transceiver/zr_esnr_and_cd_test/README.MD
@coveralls
Copy link
Copy Markdown

coveralls commented Oct 18, 2023

Pull Request Test Coverage Report for Build 6577197996

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 46.333%

Totals Coverage Status
Change from base Build 6574336113: 0.0%
Covered Lines: 1409
Relevant Lines: 3041

💛 - Coveralls

Copy link
Copy Markdown

@ejbrever ejbrever left a comment

Choose a reason for hiding this comment

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

LGTM, thanks.

Comment thread feature/platform/transceiver/zr_esnr_and_cd_test/README.MD Outdated
Comment thread feature/platform/transceiver/zr_esnr_and_cd_test/README.MD Outdated
Copy link
Copy Markdown
Member

@dplore dplore left a comment

Choose a reason for hiding this comment

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

Small correction to format of a path

Comment thread feature/platform/transceiver/zr_esnr_and_cd_test/README.MD Outdated
@dplore dplore merged commit 0283fd7 into openconfig:main Oct 20, 2023
frasieroh pushed a commit to aristanetworks/openconfig-featureprofiles that referenced this pull request Oct 20, 2023
…2236)

* Add tests for 400ZR chromatic dispersion and eSNR.
frasieroh pushed a commit to aristanetworks/openconfig-featureprofiles that referenced this pull request Oct 30, 2023
…2236)

* Add tests for 400ZR chromatic dispersion and eSNR.
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.

5 participants