-
Notifications
You must be signed in to change notification settings - Fork 820
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
improve error messaging for "Your socket connection to the server... will be closed" #11835
Comments
Hey @Alambre020 👋 thanks for raising this! The reproduction steps do not seem to follow the typical paths for pushing Amplify projects. What is the use case for deleting the |
Hey @josefaidt, I was actually working with @Alambre020 to try to get him spun up on a personal project, and this is where we got stuck. (We hit a few other CLI snags as well, but I'm chalking that up to forgetting what order to run initial CLI setup commands in for now.) Removing the
Feel free to pull me to help repro this one, as I own the original app + source. |
Hey @svidgen and @Alambre020 perhaps we can update the docs to suggest only adding it to When sharing projects, the If you do not remove the Additionally, if you do not remove this file I would be able to safely initialize the project using a different environment name such as |
This was the intended use-case. Initially we tried to use |
Thanks for clarifying @svidgen ! What sort of overrides issues were you running into? |
@Alambre020 I chatted with @josefaidt and he was able to pull the repo and push it without issue. Hopefully we just mindlessly flipped some commands. Let's try this fresh and make sure to run things strictly in this order:
@josefaidt Thanks for your time. @Alambre020 or I will post back when we've had a chance to retry this. |
@Alambre020 Do you have a non-Ubuntu machine you can try this on? |
@svidgen, it hasn't been turned on in long time but i can try tomorrow |
@svidgen @josefaidt , I was able to test on a windows machine. I got through the init successfully. The push failed. " There was an error pulling the backend environment dev. " It cannot find .aws\conifg |
Hey @Alambre020 does the
|
Hey @Alambre020 I do not recognize that directory name, but after a bit of googling it seems like this is related to Cygwin? Are you using Cygwin by chance? Can you check the value of your |
Hey @Alambre020 are you still experiencing this issue? |
@josefaidt , yes sorry... Been away from this in a while and will have to check this week. Last i remember i tried on an old windows machine and still had issues. |
@josefaidt, my home variable is C:\Users\Nick\AppData\Roaming\SPB_Data which does seem to have the file that the error says is missing: I do not see any cygwin installed. |
Hey @Alambre020 no worries about cygwin, I was doing some digging on this path to understand what app Additionally, there is a typo in that path ( |
Hey @Alambre020 are you still experiencing this? |
@josefaidt , yes. Sorry for the delay and thank you for continuing to check in. I will have time to look at this again this weekend. I will have to double check the patching... If it is the cgwin... What should I try first? I'm not really familiar with it or not sure what i would've installed it for. |
@josefaidt @svidgen , after renaming the folder I did get farther in the process. It's possible I named it incorrectly but should this be a folder that the amplify process should create? Now the process timed out: |
@josefaidt when i run the push again it says there are no changes? so maybe the first pushed worked after renaming the .aws folder? |
Hey @Alambre020 thanks for checking on that! That error seems to be coming in after the CloudFormation push was successful, where there is some project state and files that are being uploaded at the end. This error and subsequent push attempt leads me to believe the operation was likely successful and the connection was not closed properly. I will mark this as a bug to improve the error handling and messaging here https://github.com/aws-amplify/amplify-cli/blob/dev/packages/amplify-provider-awscloudformation/src/aws-utils/aws-s3.ts#L134 Despite the oddity it does appear to be successful. |
How did you install the Amplify CLI?
npm
If applicable, what version of Node.js are you using?
v19.0.1
Amplify CLI Version
10.6.2
What operating system are you using?
Ubuntu
Did you make any manual changes to the cloud resources managed by Amplify? Please describe the changes made.
There is an existing override.ts but is the original from functional deployments.
Describe the bug
Initializing an existing project under my own account.
Removed the amplify team-provider-info.json file.
Getting error on amplify push:
**
:~/Documents/DeckAnalyzer$ amplify push
✖ There was an error pulling the backend environment devtwo.
🛑 Packaging overrides failed.
Command failed with exit code 1: yarn install
Resolution: There may be errors in your overrides file. If so, fix the errors and try again.
Learn more at: https://docs.amplify.aws/cli/project/troubleshooting/
Session Identifier: 23f96950-8041-44f9-8a30-2a76743c0494
**
Expected behavior
Expected project to initialize and deploy.
Reproduction steps
amplify init
delete team-provider-info.json
amplify push
Project Identifier
Project Identifier: caa1b95f02a190bcb0b4ee7bcb9e7b6f
Log output
Additional information
No response
Before submitting, please confirm:
The text was updated successfully, but these errors were encountered: