Skip to content

added certz3.1#3360

Open
priyacj wants to merge 125 commits intoopenconfig:mainfrom
priyacj:certz3.1
Open

added certz3.1#3360
priyacj wants to merge 125 commits intoopenconfig:mainfrom
priyacj:certz3.1

Conversation

@priyacj
Copy link
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

OpenConfigBot commented Aug 3, 2024

Pull Request Functional Test Report for #3360 / a46af8f

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

coveralls commented Aug 3, 2024

Pull Request Test Coverage Report for Build 22043100843

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.1%

Totals Coverage Status
Change from base Build 22042961928: 0.0%
Covered Lines: 2300
Relevant Lines: 22772

💛 - Coveralls

Copy link
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

@ksgireesha
Copy link

Pre-merge Action: @ram-mac , kindly execute this test within our environment before merging. We can proceed with the merge upon successful test completion

Copy link
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 the comments on this PR before it can be merged

@dplore
Copy link
Member

dplore commented Dec 6, 2025

@priyacj please address the open comments

@dplore
Copy link
Member

dplore commented Feb 10, 2026

@priyacj please address the open comments so we can proceed with merging this PR. Thanks!

priyacj and others added 5 commits February 10, 2026 14:15
…rtificate_rotation_test.go


probably we should have failed here instead of continued along.

Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
this is a good change/comment.

Co-authored-by: Darren Loher <dloher@google.com>
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.