-
Notifications
You must be signed in to change notification settings - Fork 951
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
Function deployment fails when running on Node 15.8.0 #3120
Comments
+1 Same issue, v15, which is what (was) available on the M1 apple-silicon brew installery of late, leading to the this workflow (here) HTH. What a curiosity these errors. Supporting a vendor's software.... |
+1, also seeing this issue on Node v15.8.0 |
Hi folks - we are actively taking a look at the issue. Stay tuned. |
It looks like the issue is related to archiverjs/node-archiver#491. They've patched up the issue, but we are using an older version of archiver that did not receive the patch. We'll need to try updating the archiver version and make sure nothing breaks. Anyone interesting in helping with the changes? |
Thank you, @taeold! When do you expect you'll release an updated version? |
I' am use M1 apple-silicon to. I'am am waiting for your update 🥇 |
Patch submitted. It will be included in the next release for firebaes-tools package (AFAIK we don't have a regular release schedule, but we do get out a release ~once a week). |
The fix is available in the latest firebase-tools package (>9.4.0). |
@taeold this still doesn't fix the Google Cloud Builds for Firebase functions CICD. I'm getting same error during Cloud Build. |
This seems to have fixed the issue on x64 binaries, but not on arm |
@kmcnellis Based on my reading of #3129, I'm a bit confused. Are you saying that:
If (1), then we should focus on resolving #3129 not this issue. |
@sudcha23 Can you share the error messages you are seeing in your Cloud Build setup? Are you seeing the same error message as described in the original issue description? If so, I don't have a clue what's going on and will have to find a way to reproduce the error you are seeing. |
It's (2) M1 running the firebase-cli built for ARM fails to deploy CF3s When it runs the x86 binaries (1), the deploy works fine. |
The error is:
In the cloud build logs:
|
Seeing bad ZIP archives again in #3120 (comment):
This error was exactly what we hoped to fix with #3140. At the end of the day, we expect the crc32-stream package includes archiverjs/node-crc32-stream#31 (we need to use version 4.0.2 of this package). I'm not sure whether the latest version of the crc32-stream package (v4.0.2) still doesn't fix the issue for M1 Macs, or the issue lies somewhere else. Since I don't have a M1 Mac, I'm going to keep asking you both @kmcnellis @donhmorris more questions (sorry :'( ). Can you check the version of crc32-stream package you are currently using. Assuming that you are using nvm and downloaded firebase cli via |
Hm, I uninstalled node & firebase fully before checking that, and I can't replicate anymore. Maybe @donhmorris can? |
I have the same error trying to follow https://cloud.google.com/build/docs/deploying-builds/deploy-firebase
I basically have no control on the architecture/hardware as I am using cloud build's one.
Note that deploying storage works
Local hardware is Mac x64 but so far I understand my issue, this is irrelevant. |
I think you guys have moved beyond my knowledge of the firebase cli. I have never dug into the code for it. |
@innovation-club-net Can you open up a new issue with the appropriate details (e.g. version of the firesbase cli and debug logs would be helpful)? You are using node12, and I'd like to use this issue to track deployment issues for node15. |
Here are my steps I followed several times with the same results:
I have found you are asking for a headache if you don't install xcode first. For example, git comes from xcode. xcode does some things to your mac when you run it to get things setup for development. I have done this process 3 times |
any luck? |
@harihara6 The problem you described depends on the version of node and firebase-cli in your builder image. Please share your setup - I've been unable to reproduce the issue (using node 16 in my builder image). |
Hey @alsohas. We need more information to resolve this issue but there hasn't been an update in 7 weekdays. I'm marking the issue as stale and if there are no new updates in the next 3 days I will close it automatically. If you have more information that will help us get to the bottom of this, just add a comment! |
Since there haven't been any recent updates here, I am going to close this issue. @alsohas if you're still experiencing this problem and want to continue the discussion just leave a comment here and we are happy to re-open this. |
Archiver version override required, see firebase/firebase-tools#3120 archiverjs/node-archiver#491
[REQUIRED] Environment info
firebase-tools: 9.3.0
Node Version: 15.8.0
Platform: Linux 5.4.89-1-MANJARO
[REQUIRED] Test case
Deploy the function generated initially when you run
firebase init
and select functions.[REQUIRED] Steps to reproduce
Deploy the function above:
[REQUIRED] Expected behavior
Expected behavior is to deploy the function(s) without any errors.
[REQUIRED] Actual behavior
Deployments fails with the following output message in local terminal:
Cloud console had the following output in log explorer:
Workaround
I switched to Node version 14.x.x and everything worked fine. This may be a duplicate of #3108. The SO link they provided also has more details so it might be worth looking into. In #3108 they seem to have trouble deploying with node versions aside from 15.x.x. They mention that deploy requires keyboard input in GCP; in my logs this does show up but seems selects an input automatically anyway, the selected input being
[N]one
. I am listing the summary for this particular log message below. Note, this happens before the error message I attached farther up the document.Followed by:
The text was updated successfully, but these errors were encountered: