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

Build evergreen specific test targets only for non-windows platforms #1918

Conversation

niranjanyardi
Copy link
Contributor

@niranjanyardi niranjanyardi commented Nov 6, 2023

Reverts #1873 partially
Internal failures have been fixed.

Note: For disabling evergreen specific tests for all non-evergreen platforms b/309493306 needs to be fixed.
As a step in the right direction we are just disabling building disabling evergreen specific tests for windows platforms at the moment.

b/302008891

@niranjanyardi niranjanyardi force-pushed the revert-1873-revert-1662-separate_eg_modular branch from 77b979a to 6d620e0 Compare November 6, 2023 20:29
@niranjanyardi niranjanyardi changed the base branch from revert-1873-revert-1662-separate_eg_modular to main November 6, 2023 20:33
@niranjanyardi niranjanyardi force-pushed the revert-1873-revert-1662-separate_eg_modular branch from 2ce1f1d to a563218 Compare November 6, 2023 20:38
@niranjanyardi
Copy link
Contributor Author

niranjanyardi commented Nov 6, 2023

Running internal tests : go/cobalt-cl/269031 : passed

Change-Id: Iac3266df52a776df3eb4bdc814ad8413586d55c3
Copy link

codecov bot commented Nov 6, 2023

Codecov Report

Merging #1918 (4347409) into main (2277260) will decrease coverage by 0.03%.
Report is 1 commits behind head on main.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #1918      +/-   ##
==========================================
- Coverage   57.77%   57.75%   -0.03%     
==========================================
  Files        1915     1915              
  Lines       95118    95118              
==========================================
- Hits        54959    54934      -25     
- Misses      40159    40184      +25     

see 8 files with indirect coverage changes

@niranjanyardi niranjanyardi merged commit 34c46ca into youtube:main Nov 6, 2023
333 of 334 checks passed
@niranjanyardi niranjanyardi added the cp-24.lts.1+ Cherry Pick to the 24.lts.1+ branch label Nov 6, 2023
cobalt-github-releaser-bot pushed a commit that referenced this pull request Nov 6, 2023
…1918)

Reverts #1873 partially
Internal failures have been fixed.

Note: For disabling evergreen specific tests for all non-evergreen
platforms b/309493306 needs to be fixed.
As a step in the right direction we are just disabling building
disabling evergreen specific tests for windows platforms at the moment.

b/302008891

(cherry picked from commit 34c46ca)
niranjanyardi added a commit that referenced this pull request Nov 7, 2023
Adding TODO statements from feedback in
#1918

b/309493306
cobalt-github-releaser-bot pushed a commit that referenced this pull request Nov 7, 2023
Adding TODO statements from feedback in
#1918

b/309493306

(cherry picked from commit 5eb5f80)
niranjanyardi added a commit that referenced this pull request Nov 7, 2023
…1918)

Reverts #1873 partially
Internal failures have been fixed.

Note: For disabling evergreen specific tests for all non-evergreen
platforms b/309493306 needs to be fixed.
As a step in the right direction we are just disabling building
disabling evergreen specific tests for windows platforms at the moment.

b/302008891

(cherry picked from commit 34c46ca)
@niranjanyardi niranjanyardi added cp-24.lts.1+ Cherry Pick to the 24.lts.1+ branch and removed cp-24.lts.1+ Cherry Pick to the 24.lts.1+ branch labels Nov 7, 2023
niranjanyardi added a commit that referenced this pull request Nov 7, 2023
Adding TODO statements from feedback in
#1918

b/309493306

(cherry picked from commit 5eb5f80)
niranjanyardi added a commit that referenced this pull request Nov 7, 2023
…non-windows platforms (#1924)

Refer to the original PR: #1918

Reverts #1873 partially
Internal failures have been fixed.

Note: For disabling evergreen specific tests for all non-evergreen
platforms b/309493306 needs to be fixed.
As a step in the right direction we are just disabling building
disabling evergreen specific tests for windows platforms at the moment.

b/302008891

Co-authored-by: Niranjan Yardi <[email protected]>
cobalt-github-releaser-bot pushed a commit that referenced this pull request Nov 8, 2023
Adding TODO statements from feedback in
#1918

b/309493306

(cherry picked from commit 5eb5f80)
niranjanyardi added a commit that referenced this pull request Nov 8, 2023
…1937)

Refer to the original PR: #1925

Adding TODO statements from feedback in
#1918

b/309493306

Co-authored-by: Niranjan Yardi <[email protected]>
Rongo-JL pushed a commit to Rongo-JL/cobalt that referenced this pull request Dec 19, 2023
…outube#1918)

Reverts youtube#1873 partially
Internal failures have been fixed.

Note: For disabling evergreen specific tests for all non-evergreen
platforms b/309493306 needs to be fixed.
As a step in the right direction we are just disabling building
disabling evergreen specific tests for windows platforms at the moment.

b/302008891
Rongo-JL pushed a commit to Rongo-JL/cobalt that referenced this pull request Dec 19, 2023
Adding TODO statements from feedback in
youtube#1918

b/309493306
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cp-24.lts.1+ Cherry Pick to the 24.lts.1+ branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants