Skip to content

[SHOT-251] build: update Dockerfile to use Ubuntu 22.04 - #8195

Merged
mimartin12 merged 1 commit into
rcfrom
cherry-pick-SHOT-251
Aug 12, 2026
Merged

[SHOT-251] build: update Dockerfile to use Ubuntu 22.04#8195
mimartin12 merged 1 commit into
rcfrom
cherry-pick-SHOT-251

Conversation

@mimartin12

Copy link
Copy Markdown
Contributor

🎟️ Tracking

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

📔 Objective

Cherry-pick

@mimartin12 mimartin12 added t:bugfix Change Type - Bugfix t:deps Change Type - Dependencies and removed t:bugfix Change Type - Bugfix labels Aug 12, 2026
@mimartin12
mimartin12 marked this pull request as ready for review August 12, 2026 15:23
@mimartin12
mimartin12 requested review from a team as code owners August 12, 2026 15:23
@mimartin12
mimartin12 requested a review from keithhubner August 12, 2026 15:23
@github-actions

github-actions Bot commented Aug 12, 2026

Copy link
Copy Markdown
Contributor

🤖 Bitwarden Claude Code Review

Overall Assessment: APPROVE

This PR is a one-line cherry-pick into rc that changes the MsSql utility image base from mcr.microsoft.com/mssql/server:2025-CU5-ubuntu-24.04 to the ubuntu-22.04 variant of the same SQL Server 2025 CU5 release. The SQL Server version is unchanged, so the mssql-conf and mssql-tools18 paths used later in the Dockerfile remain valid, and the gosu/tzdata apt packages are available on Jammy. No other file in the repo pins the MsSql utility image, so nothing else needs to move with it. The change matches commit e1a9b96fb already merged to main (#8183), so rc and main stay aligned.

Code Review Details

No findings.

@codecov

codecov Bot commented Aug 12, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 63.13%. Comparing base (16a41c2) to head (87ae05e).
⚠️ Report is 1 commits behind head on rc.

Additional details and impacted files
@@            Coverage Diff             @@
##               rc    #8195      +/-   ##
==========================================
- Coverage   67.79%   63.13%   -4.67%     
==========================================
  Files        2337     2344       +7     
  Lines      101456   101864     +408     
  Branches     9167     9247      +80     
==========================================
- Hits        68787    64312    -4475     
- Misses      30362    35341    +4979     
+ Partials     2307     2211      -96     

☔ 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 enabled auto-merge (squash) August 12, 2026 15:49
@mimartin12
mimartin12 force-pushed the cherry-pick-SHOT-251 branch from e1a9b96 to 87c697d Compare August 12, 2026 16:01
@mimartin12
mimartin12 force-pushed the cherry-pick-SHOT-251 branch from 87c697d to 87ae05e Compare August 12, 2026 16:07
@mimartin12
mimartin12 merged commit efc852d into rc Aug 12, 2026
29 of 30 checks passed
@mimartin12
mimartin12 deleted the cherry-pick-SHOT-251 branch August 12, 2026 16:11
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