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

Split cloudbuild build-all.yaml into 2 separate build commands to avoid length overflow #23321

Merged
merged 1 commit into from
Oct 24, 2022

Conversation

andy31415
Copy link
Contributor

@andy31415 andy31415 commented Oct 24, 2022

Build fails otherwise due to too long arguments, it says:

Your build failed to run: generic::invalid_argument: invalid build: invalid .steps field: build step 2 arg 0 too long (max: 4000)

Build fail otherwise due to too long arguments, it says:

```
Your build failed to run: generic::invalid_argument: invalid build: invalid .steps field: build step 2 arg 0 too long (max: 4000)
```
@andy31415 andy31415 added the hotfix urgent fix needed, can bypass review label Oct 24, 2022
@andy31415
Copy link
Contributor Author

Will merge since CI cannot validate cloudbuild compilations.

@andy31415 andy31415 merged commit d5e2725 into project-chip:master Oct 24, 2022
@github-actions
Copy link

PR #23321: Size comparison from 5d95baf to 39d2a7f

Decreases (2 builds for telink)
platform target config section 5d95baf 39d2a7f change % change
telink light-switch-app tlsr9518adk80d (read/write) 790476 790468 -8 -0.0
text 554014 554012 -2 -0.0
ota-requestor-app tlsr9518adk80d text 565046 565044 -2 -0.0
Full report (6 builds for k32w, telink)
platform target config section 5d95baf 39d2a7f change % change
k32w contact k32w0+release (read/write) 661412 661412 0 0.0
.bss 77040 77040 0 0.0
.data 2104 2104 0 0.0
.text 563156 563156 0 0.0
light k32w0+release (read/write) 671640 671640 0 0.0
.bss 74840 74840 0 0.0
.data 2060 2060 0 0.0
.text 592012 592012 0 0.0
lock k32w0+release (read/write) 633220 633220 0 0.0
.bss 75600 75600 0 0.0
.data 2080 2080 0 0.0
.text 552812 552812 0 0.0
telink light-switch-app tlsr9518adk80d (read/write) 790476 790468 -8 -0.0
bss 72480 72480 0 0.0
noinit 43520 43520 0 0.0
text 554014 554012 -2 -0.0
lighting-app tlsr9518adk80d (read/write) 805732 805732 0 0.0
bss 73240 73240 0 0.0
noinit 43520 43520 0 0.0
text 566174 566174 0 0.0
ota-requestor-app tlsr9518adk80d (read/write) 804616 804616 0 0.0
bss 74052 74052 0 0.0
noinit 43520 43520 0 0.0
text 565046 565044 -2 -0.0

adbridge pushed a commit to ARM-software/connectedhomeip that referenced this pull request Nov 18, 2022
Build fail otherwise due to too long arguments, it says:

```
Your build failed to run: generic::invalid_argument: invalid build: invalid .steps field: build step 2 arg 0 too long (max: 4000)
```
adbridge pushed a commit to ARM-software/connectedhomeip that referenced this pull request Nov 18, 2022
Build fail otherwise due to too long arguments, it says:

```
Your build failed to run: generic::invalid_argument: invalid build: invalid .steps field: build step 2 arg 0 too long (max: 4000)
```
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hotfix urgent fix needed, can bypass review integrations
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant