Skip to content

[SHOT-251] Update Dockerfile to use Ubuntu 22.04 - #8183

Merged
mimartin12 merged 1 commit into
mainfrom
fix/stepdown-ubuntu-version
Aug 12, 2026
Merged

[SHOT-251] Update Dockerfile to use Ubuntu 22.04#8183
mimartin12 merged 1 commit into
mainfrom
fix/stepdown-ubuntu-version

Conversation

@mimartin12

@mimartin12 mimartin12 commented Aug 10, 2026

Copy link
Copy Markdown
Contributor

🎟️ Tracking

https://bitwarden.atlassian.net/browse/SHOT-251

📔 Objective

Steps down the Ubuntu version due to FIPS issues.

📸 Screenshots

@mimartin12
mimartin12 requested review from a team as code owners August 10, 2026 19:35
@mimartin12
mimartin12 requested a review from keithhubner August 10, 2026 19:35
@github-actions

github-actions Bot commented Aug 10, 2026

Copy link
Copy Markdown
Contributor

🤖 Bitwarden Claude Code Review

Overall Assessment: APPROVE

This PR changes a single line in util/MsSql/Dockerfile, moving the base image from mcr.microsoft.com/mssql/server:2025-CU5-ubuntu-24.04 to the 2025-CU5-ubuntu-22.04 variant to address FIPS issues. The SQL Server version and cumulative update level are unchanged, so no data or schema compatibility impact is expected. The remaining Dockerfile steps (gosu/tzdata install, mssql-conf telemetry opt-out, /opt/mssql-tools18/bin/sqlcmd healthcheck) and the entrypoint.sh / backup-db.sh scripts contain no Ubuntu 24.04-specific dependencies. The image builds in CI, which will surface any tag or package availability problems.

Code Review Details

No findings.

@codecov

codecov Bot commented Aug 10, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 63.50%. Comparing base (0b43527) to head (c18f6ad).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #8183   +/-   ##
=======================================
  Coverage   63.50%   63.50%           
=======================================
  Files        2338     2338           
  Lines      101524   101524           
  Branches     9179     9179           
=======================================
+ Hits        64469    64470    +1     
+ Misses      34845    34844    -1     
  Partials     2210     2210           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@mimartin12 mimartin12 changed the title Update Dockerfile to use Ubuntu 22.04 [SHOT-251] Update Dockerfile to use Ubuntu 22.04 Aug 10, 2026
@mimartin12 mimartin12 added the t:deps Change Type - Dependencies label Aug 10, 2026
@mimartin12
mimartin12 enabled auto-merge (squash) August 10, 2026 20:13
@mimartin12
mimartin12 merged commit 8414547 into main Aug 12, 2026
44 of 47 checks passed
@mimartin12
mimartin12 deleted the fix/stepdown-ubuntu-version branch August 12, 2026 14:03
mimartin12 added a commit that referenced this pull request Aug 12, 2026
mimartin12 added a commit that referenced this pull request Aug 12, 2026
mimartin12 added a commit that referenced this pull request Aug 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

t:deps Change Type - Dependencies

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants