Skip to content
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.

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

Change base image to Debian 11 #2009

Merged
merged 1 commit into from
Feb 6, 2024
Merged

Change base image to Debian 11 #2009

merged 1 commit into from
Feb 6, 2024

Conversation

kaidokert
Copy link
Member

@kaidokert kaidokert commented Nov 22, 2023

This updates base runner images to Debian11.

b/312772759

@kaidokert kaidokert force-pushed the dev11 branch 2 times, most recently from 7d6ba12 to d968e22 Compare November 28, 2023 21:58
@kaidokert kaidokert force-pushed the dev11 branch 2 times, most recently from dd1a2d6 to f5e8a8d Compare January 9, 2024 19:32
@kaidokert kaidokert force-pushed the dev11 branch 2 times, most recently from fca64be to 36bc556 Compare January 22, 2024 19:05
@kaidokert kaidokert marked this pull request as ready for review January 22, 2024 19:06
@kaidokert kaidokert force-pushed the dev11 branch 2 times, most recently from b108971 to 0b4e7ca Compare January 23, 2024 02:01
This updates base runner images to Debian11.

Also update leak check manifests to match.

b/312772759
Copy link

codecov bot commented Feb 6, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (2d6cae9) 58.41% compared to head (27e953b) 59.28%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2009      +/-   ##
==========================================
+ Coverage   58.41%   59.28%   +0.87%     
==========================================
  Files        1902     1711     -191     
  Lines       94709    78442   -16267     
==========================================
- Hits        55325    46506    -8819     
+ Misses      39384    31936    -7448     

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

@kaidokert
Copy link
Member Author

kaidokert commented Feb 6, 2024

I believe this can go in, but then still need to re-land go/cobalt-cl/260221 ( now go/cobalt-cl/274780 )

@kaidokert kaidokert merged commit a37421c into youtube:main Feb 6, 2024
382 of 383 checks passed
sherryzy pushed a commit to sherryzy/cobalt that referenced this pull request Feb 7, 2024
This updates base runner images to Debian11.

b/291067151
b/291066320
b/312772759
b/321746394

(cherry picked from commit a37421c)
sherryzy pushed a commit to sherryzy/cobalt that referenced this pull request Feb 21, 2024
This updates base runner images to Debian11.

b/291067151
b/291066320
b/312772759
b/321746394

(cherry picked from commit a37421c)
sherryzy added a commit that referenced this pull request Feb 21, 2024
This updates base runner images to Debian11.

b/291067151
b/291066320
b/312772759
b/321746394

(cherry picked from commit a37421c)

Co-authored-by: Kaido Kert <[email protected]>
@kaidokert kaidokert deleted the dev11 branch June 30, 2024 19:45
dahlstrom-g pushed a commit that referenced this pull request Feb 6, 2025
VCM used VideoCaptureController raw pointers in a number of places,
including as a field in VCM::CaptureDeviceStartRequest.

This CL replaces the field and some other usages with a scoped_refptr
to prevent dangling pointers.

(cherry picked from commit 3524ce528548d1d743a6aa6e339ecb5a186c22bc)

Bug: 382135228
Change-Id: I1bd5f95bdf57631227034beb8bb076f258606378
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6088073
Commit-Queue: Guido Urdaneta <[email protected]>
Reviewed-by: Dale Curtis <[email protected]>
Cr-Original-Commit-Position: refs/heads/main@{#1396301}
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6099792
Reviewed-by: Guido Urdaneta <[email protected]>
Commit-Queue: Gyuyoung Kim (xWF) <[email protected]>
Cr-Commit-Position: refs/branch-heads/6478@{#2009}
Cr-Branched-From: e6143acc03189c5e52959545b110d6d17ecd5286-refs/heads/main@{#1300313}
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