Skip to content

Remove FV tests for crypto package tied to FIPS #11507

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

Open
stevegaossou wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Remove FV tests for crypto package tied to FIPS #11507

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
6daefeb
Remove FV tests for crypto package tied to FIPS.
stevegaossou Dec 6, 2025
d1b1bed
Merge branch 'master' into stevegaossou-remove-crypto-tests-fips
stevegaossou Jan 7, 2026
fe55b57
Addressing check-dirty failure.
stevegaossou Jan 14, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
13 changes: 0 additions & 13 deletions .semaphore/semaphore-scheduled-builds.yml
View file
Open in desktop

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

23 changes: 5 additions & 18 deletions .semaphore/semaphore.yml
View file
Open in desktop

Large diffs are not rendered by default.

13 changes: 0 additions & 13 deletions .semaphore/semaphore.yml.d/blocks/20-crypto.yml
View file
Open in desktop

This file was deleted.

29 changes: 0 additions & 29 deletions crypto/Makefile
View file
Open in desktop

This file was deleted.

28 changes: 0 additions & 28 deletions crypto/deps.txt
View file
Open in desktop

This file was deleted.

17 changes: 0 additions & 17 deletions crypto/fv/expected-nmap.log
View file
Open in desktop

This file was deleted.

32 changes: 0 additions & 32 deletions crypto/fv/main/main.go
View file
Open in desktop

This file was deleted.

10 changes: 0 additions & 10 deletions crypto/pkg/tls/fipstls.go
View file
Open in desktop

This file was deleted.

6 changes: 0 additions & 6 deletions crypto/pkg/tls/nonfipstls.go
View file
Open in desktop

This file was deleted.

2 changes: 0 additions & 2 deletions crypto/pkg/tls/tls.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,6 @@ import (
"strings"

apiv3 "github.com/projectcalico/api/pkg/apis/projectcalico/v3"
log "github.com/sirupsen/logrus"
)

// Ciphers supported by TLS 1.2
Expand Down Expand Up @@ -99,7 +98,6 @@ func ParseTLSCiphers(ciphers string) ([]uint16, error) {
// NewTLSConfig returns a tls.Config with the recommended default settings for Calico components. Based on build flags,
// boringCrypto may be used and fips strict mode may be enforced, which can override the parameters defined in this func.
func NewTLSConfig() (*tls.Config, error) {
log.WithField("BuiltWithBoringCrypto", BuiltWithBoringCrypto).Debug("creating a TLS config")
env := os.Getenv("TLS_CIPHER_SUITES")
ciphers, err := ParseTLSCiphers(env)
if err != nil {
Expand Down