Skip to content

Skip the Cavium root validity test until March 19th, 2026#938

Merged
hslatman merged 1 commit intomasterfrom
herman/bump-cavium-test-skip-date
Jan 20, 2026
Merged

Skip the Cavium root validity test until March 19th, 2026#938
hslatman merged 1 commit intomasterfrom
herman/bump-cavium-test-skip-date

Conversation

@hslatman
Copy link
Member

There's no new Cavium root certificate available yet. Bumping the date to skip this check with about 2 months for now.

More info at https://www.marvell.com/products/security-solutions/nitrox-hs-adapters/liquidsecurity-certificate.html

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the skip condition for the Cavium root certificate validity test to extend the skip period until March 19, 2026. The change moves from a hardcoded Unix timestamp to a more readable RFC3339 date format and improves the skip message formatting.

Changes:

  • Updated the test skip deadline from January 15, 2026 to March 19, 2026
  • Replaced Unix timestamp with RFC3339 date parsing for better readability
  • Enhanced skip message to use RFC1123 formatting

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

There's no new Cavium root certificate available yet. Bumping the
date to skip this check with about 2 months for now.

More info at https://www.marvell.com/products/security-solutions/nitrox-hs-adapters/liquidsecurity-certificate.html
@hslatman hslatman force-pushed the herman/bump-cavium-test-skip-date branch from 0dbd94e to 79419cb Compare January 20, 2026 14:04
@hslatman hslatman requested review from a team and maraino January 20, 2026 14:22
@hslatman hslatman merged commit 975c521 into master Jan 20, 2026
13 checks passed
@hslatman hslatman deleted the herman/bump-cavium-test-skip-date branch January 20, 2026 18:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants