Skip to content

Cloudbuild pipeline #3

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 78 commits into
base: main
Choose a base branch
from
Open

Cloudbuild pipeline #3

wants to merge 78 commits into from

Conversation

@harryoverton-protected
Copy link
Collaborator

@harryoverton-protected harryoverton-protected commented May 8, 2025

No description provided.

@harryoverton-protected
added initial cloudbuild configs for testing builds
cb507ae
@harryoverton-protected
fix logging options for cloudbuild
4851793
@harryoverton-protected
fix path export
d06fe01
@harryoverton-protected
fix path export
dd3ace8
@harryoverton-protected
fix path export
a35901f
@harryoverton-protected
fix substitutions conflict
cff6ff9
@harryoverton-protected
fix Install Dependencies steps
3c96b1c
@harryoverton-protected
fix Install Dependencies steps
1d79def
@harryoverton-protected
fix typo
996a35c
@harryoverton-protected
fixed install deps step
82da992
@harryoverton-protected
fixed install deps step
8d59081
@harryoverton-protected
fixed install deps step
aaab405
@harryoverton-protected
use custom cloudbuild runners for build steps
ade3e19
@harryoverton-protected
specify totalbrowser-builder image tag
acf76a6
@harryoverton-protected
use sh entrypoint not bash for alpine images
d206c42
@harryoverton-protected
make sure workdir is created
0a88fd8
@harryoverton-protected
updated builder images to use ubuntu
64c24ef
@harryoverton-protected
fix failed github auth
b3aa355
@harryoverton-protected
fix typo
e3f2ea7
@harryoverton-protected
fix typo
e9bcc5a
@harryoverton-protected
added missing volume
386b601
@harryoverton-protected
use full path when using gn
6fc4569
@harryoverton-protected
Pre-create the args.gn file for gn
f5de122
@harryoverton-protected
fix gn gen arg specification
61d3cb7
@harryoverton-protected
fix gn gen arg specification
9a8fb33
@harryoverton-protected
fix gn gen arg specification
757cf40
@harryoverton-protected
fix gn gen arg specification
245ebc3
@harryoverton-protected
fix gn gen arg specification
559ea71
@harryoverton-protected
fix branding script step
47968e0
@harryoverton-protected
run branding-script step from correct dir
ccd5bd4
@harryoverton-protected
added github pr workflow to enforce reviews for internal changes
4a07c30
@harryoverton-protected
added github pr workflow to enforce reviews for internal changes
7063e6e
@harryoverton-protected
debug whole output dir
507f15e
@harryoverton-protected
added some suggested troubleshooting checks and logging
b0da5cb
@harryoverton-protected
Revert "added some suggested troubleshooting checks and logging"
84badb9 
This reverts commit b0da5cb.
@harryoverton-protected
Revert "debug whole output dir"
5f9ac24 
This reverts commit 507f15e.
@harryoverton-protected
fix issue finding remote head as well as potential timeout issues
20b0daf
@harryoverton-protected
try check out with a dept of 0
3d1110e
@harryoverton-protected
play about with the git fetch depths to see if we can reduce the exec…
0bf337d 
…ution time for fetching
@harryoverton-protected
added better error handling and increased fetch depth
fe4cfb3
@harryoverton-protected
increase checkout fetch to 10000
9c76566
@harryoverton-protected
increase git fetch dept and add some more debugging
b18c599
@harryoverton-protected
reduced fetch dept
c79413f
@harryoverton-protected
more debug steps...
f2a9e72
@harryoverton-protected
test new workflow
a53e35a
@harryoverton-protected
remove redundant include
f957cfa
@harryoverton-protected
treat prRequiresReview as bool
8fc3e46
@harryoverton-protected
only allow reviews from approved list of reviewers
fbce205
@harryoverton-protected
add comment informing who needs to complete the review for a spceific pr
0344faa
@harryoverton-protected
fix automated github comments
66dea89
@github-actions
Copy link

github-actions bot commented May 8, 2025

Action Required: Review needed because this PR contains commits not found in the upstream repository.

Please request a review from one of the approved reviewers: @l0hn

@github-actions
Copy link

github-actions bot commented May 9, 2025

Action Required: Review needed because this PR contains commits not found in the upstream repository.

Please request a review from one of the approved reviewers: @l0hn

@github-actions
Copy link

github-actions bot commented May 10, 2025

Action Required: Review needed because this PR contains commits not found in the upstream repository.

Please request a review from one of the approved reviewers: @l0hn

@github-actions
Copy link

github-actions bot commented May 12, 2025

Action Required: Review needed because this PR contains commits not found in the upstream repository.

Please request a review from one of the approved reviewers: @l0hn

@harryoverton-protected
limit how fast we run checkout-branch step in cloud build to avoid hi…
1d6cc38 
…tting rate limits and getting blocked from pulling external deps
@github-actions
Copy link

github-actions bot commented May 12, 2025

Action Required: Review needed because this PR contains commits not found in the upstream repository.

Please request a review from one of the approved reviewers: @l0hn

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

review-required

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@harryoverton-protected