Skip to content

fips: update boringssl to use the main branch#39017

Merged
RyanTheOptimist merged 10 commits into
envoyproxy:mainfrom
kyessenov:fips_unpin
Apr 8, 2025
Merged

fips: update boringssl to use the main branch#39017
RyanTheOptimist merged 10 commits into
envoyproxy:mainfrom
kyessenov:fips_unpin

Conversation

@kyessenov
Copy link
Copy Markdown
Contributor

@kyessenov kyessenov commented Apr 4, 2025

Change-Id: I23c1fcf05b4713d848a2f46852525d6c2506986a

Commit Message: Update the BoringSSL FIPS build to use the same commit as a regular non-FIPS build (9/13/2024) per the revised FedRAMP policy.

Additional Description: BoringSSL considers the main branch to be the "update stream" in the policy. The new recommendation is to use the latest stable versions of the compilers, so we also bump Clang version to be the same as the regular build (from 14 to 18). Please note that this change does not include setting the compliance policy, as that would be a breaking change.

Risk Level: low (as we match the thoroughly tested non-FIPS build)
Testing: regression tests passed
Issue: #38353
Docs Changes: none
Release Notes: yes (it is quite plausible a default might have changed in the crypto settings due to the large version bump from 2022 to 2024, so this is a notable change).

Change-Id: I23c1fcf05b4713d848a2f46852525d6c2506986a
Signed-off-by: Kuat Yessenov <kuat@google.com>
@repokitteh-read-only
Copy link
Copy Markdown

As a reminder, PRs marked as draft will not be automatically assigned reviewers,
or be handled by maintainer-oncall triage.

Please mark your PR as ready when you want it to be reviewed!

🐱

Caused by: #39017 was opened by kyessenov.

see: more, trace.

@repokitteh-read-only repokitteh-read-only Bot added the deps Approval required for changes to Envoy's external dependencies label Apr 4, 2025
@repokitteh-read-only
Copy link
Copy Markdown

CC @envoyproxy/dependency-shepherds: Your approval is needed for changes made to (bazel/.*repos.*\.bzl)|(bazel/dependency_imports\.bzl)|(api/bazel/.*\.bzl)|(.*/requirements\.txt)|(.*\.patch).
envoyproxy/dependency-shepherds assignee is @RyanTheOptimist

🐱

Caused by: #39017 was opened by kyessenov.

see: more, trace.

Change-Id: I72a46e76eeeb95e462bb2527a4884b561517d272
Signed-off-by: Kuat Yessenov <kuat@google.com>
Change-Id: I0ef615ee86685217ba0966770057d4f18b09e43b
Change-Id: I56be266c8977f27a342ff9a054f1b3efe5b9aad7
Signed-off-by: Kuat Yessenov <kuat@google.com>
Change-Id: Id69f5cd24aeeea311b994ab7e1b85c6f3cd0cbc2
Signed-off-by: Kuat Yessenov <kuat@google.com>
Change-Id: I6853441787bf21c14d3d0f3c02314d7305baef6c
Signed-off-by: Kuat Yessenov <kuat@google.com>
Change-Id: I15a20717ba17484fbdbc75b958bcf2a2ade83e97
@kyessenov kyessenov closed this Apr 7, 2025
@kyessenov kyessenov reopened this Apr 7, 2025
Change-Id: Ibd75b040b3ec9d076fe9814bc9ccd8a2c4783460
Signed-off-by: Kuat Yessenov <kuat@google.com>
@kyessenov kyessenov changed the title test unpin fips fips: update boringssl to use the main branch Apr 7, 2025
@kyessenov kyessenov marked this pull request as ready for review April 7, 2025 21:33
Change-Id: If3452c8febf4c42f78d4b1f87b45eea14729ba68
Signed-off-by: Kuat Yessenov <kuat@google.com>
Change-Id: I0ea5d21fe52ac96377a7fa769c968075338f2d01
Signed-off-by: Kuat Yessenov <kuat@google.com>
Copy link
Copy Markdown
Member

@ggreenway ggreenway left a comment

Choose a reason for hiding this comment

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

Fantastic!

@ggreenway ggreenway self-assigned this Apr 7, 2025
Copy link
Copy Markdown
Contributor

@RyanTheOptimist RyanTheOptimist left a comment

Choose a reason for hiding this comment

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

Woo hoo!!

@repokitteh-read-only repokitteh-read-only Bot removed the deps Approval required for changes to Envoy's external dependencies label Apr 8, 2025
@RyanTheOptimist RyanTheOptimist merged commit e451caf into envoyproxy:main Apr 8, 2025
26 checks passed
phlax pushed a commit to yanavlasov/envoy that referenced this pull request Dec 9, 2025
Change-Id: I23c1fcf05b4713d848a2f46852525d6c2506986a

Commit Message: Update the BoringSSL FIPS build to use the same commit
as a regular non-FIPS build (9/13/2024) per the revised FedRAMP policy.

Additional Description: BoringSSL considers the main branch to be the
"update stream" in the policy. The new recommendation is to use the
latest stable versions of the compilers, so we also bump Clang version
to be the same as the regular build (from 14 to 18). Please note that
this change **does not** include setting the compliance policy, as that
would be a breaking change.

Risk Level: low (as we match the thoroughly tested non-FIPS build)
Testing: regression tests passed
Issue: envoyproxy#38353
Docs Changes: none
Release Notes: yes (it is quite plausible a default might have changed
in the crypto settings due to the large version bump from 2022 to 2024,
so this is a notable change).

---------

Signed-off-by: Kuat Yessenov <kuat@google.com>
phlax pushed a commit to yanavlasov/envoy that referenced this pull request Dec 9, 2025
Change-Id: I23c1fcf05b4713d848a2f46852525d6c2506986a

Commit Message: Update the BoringSSL FIPS build to use the same commit
as a regular non-FIPS build (9/13/2024) per the revised FedRAMP policy.

Additional Description: BoringSSL considers the main branch to be the
"update stream" in the policy. The new recommendation is to use the
latest stable versions of the compilers, so we also bump Clang version
to be the same as the regular build (from 14 to 18). Please note that
this change **does not** include setting the compliance policy, as that
would be a breaking change.

Risk Level: low (as we match the thoroughly tested non-FIPS build)
Testing: regression tests passed
Issue: envoyproxy#38353
Docs Changes: none
Release Notes: yes (it is quite plausible a default might have changed
in the crypto settings due to the large version bump from 2022 to 2024,
so this is a notable change).

---------

Signed-off-by: Kuat Yessenov <kuat@google.com>
Signed-off-by: Ryan Northey <ryan@synca.io>
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.

3 participants