Skip to content

added certz3.1#3360

Open
priyacj wants to merge 154 commits into
openconfig:mainfrom
priyacj:certz3.1
Open

added certz3.1#3360
priyacj wants to merge 154 commits into
openconfig:mainfrom
priyacj:certz3.1

Conversation

@priyacj
Copy link
Copy Markdown
Contributor

@priyacj priyacj commented Aug 3, 2024

certz3.1 gNSI Server Certificate Rotation

@priyacj priyacj requested a review from a team as a code owner August 3, 2024 04:08
@priyacj priyacj requested a review from a team August 3, 2024 04:08
@OpenConfigBot
Copy link
Copy Markdown

OpenConfigBot commented Aug 3, 2024

Pull Request Functional Test Report for #3360 / 4cb4618

Virtual Devices

Device Test Test Documentation Job Raw Log
Arista cEOS status
Certz-3: Server Certificate Rotation
Cisco 8000E status
Certz-3: Server Certificate Rotation
Cisco XRd status
Certz-3: Server Certificate Rotation
Juniper ncPTX status
Certz-3: Server Certificate Rotation
Nokia SR Linux status
Certz-3: Server Certificate Rotation
Openconfig Lemming status
Certz-3: Server Certificate Rotation

Hardware Devices

Device Test Test Documentation Raw Log
Arista 7808 status
Certz-3: Server Certificate Rotation
Cisco 8808 status
Certz-3: Server Certificate Rotation
Juniper PTX10008 status
Certz-3: Server Certificate Rotation
Nokia 7250 IXR-10e status
Certz-3: Server Certificate Rotation

Help

@coveralls
Copy link
Copy Markdown

coveralls commented Aug 3, 2024

Pull Request Test Coverage Report for Build 22115486988

Details

  • 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 10.091%

Totals Coverage Status
Change from base Build 22113180336: 0.0%
Covered Lines: 2300
Relevant Lines: 22792

💛 - Coveralls

Copy link
Copy Markdown
Contributor

@morrowc morrowc left a comment

Choose a reason for hiding this comment

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

there are some Check() fialure things to address here

Comment thread feature/security/gnsi/certz/tests/internal/setup_service/setup_service.go Outdated
Comment thread feature/security/gnsi/certz/tests/internal/setup_service/setup_service.go Outdated
@ksgireesha ksgireesha requested a review from dplore April 14, 2026 06:37
@dplore
Copy link
Copy Markdown
Member

dplore commented Apr 14, 2026

/fptest virtual

@dplore
Copy link
Copy Markdown
Member

dplore commented Apr 14, 2026

Thanks for all the changes @priyacj. Please share a link to a log of the test results. FYI: this is needed for every PR

@dplore
Copy link
Copy Markdown
Member

dplore commented Apr 14, 2026

@priyacj note this test fails the kne/virtual environment. At a minimum it seems like some required files are missing?
See
Log

@dplore
Copy link
Copy Markdown
Member

dplore commented Apr 14, 2026

@morrowc @mihirpitale-googler in future, please ensure test is producing expected results before approving. Thanks!

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.

please address comments regarding test logs and possibly missing dependencies

@dplore
Copy link
Copy Markdown
Member

dplore commented May 15, 2026

/fptest virtual

@dplore
Copy link
Copy Markdown
Member

dplore commented May 15, 2026

@priyacj note this test fails the kne/virtual environment. At a minimum it seems like some required files are missing?
See
Log

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

juniper-pr new-requirement-pr The PR introduces a new test scenario/requirement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.