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

Error in building project to Amplify Console #32

Closed
Arc-dev opened this issue Aug 12, 2019 · 3 comments
Closed

Error in building project to Amplify Console #32

Arc-dev opened this issue Aug 12, 2019 · 3 comments

Comments

@Arc-dev
Copy link

Arc-dev commented Aug 12, 2019

Hey folks,

I just tied to build this into my Amplify Console account and ran into a build error. Looks like it's related to this GitHub issue[1] with the newest version of the CLI.

I've edited my forked repo and tried pushing back the version to 1.10.0/1.9.0 as well as adding the unsafe-perm as below,

npm -g install @aws-amplify/cli --unsafe-perm=true

Both of these seem to not fix the issue due to other dependencies.

Cheers!
Ref
[1]. aws-amplify/amplify-cli#1996

@Arc-dev
Copy link
Author

Arc-dev commented Aug 13, 2019

Turns out I'm just an idiot!

I attempted to deploy this twice and it turns out it was crashing due to a Stripe role that was created the first time. I removed all my stacks and rebuilt using the yml build file with the --unsafe-perm true tag.
npm install -g @aws-amplify/cli --unsafe-perm=true # install latest amplify-CLI

This pushed me past the DynamoDb mock dependency so I'll fork the repo and add this change into the yml file for now.

@heitorlessa
Copy link

hey @Arc-dev,

Thanks for reporting that - I'll pin Amplify CLI version in amplify.yml file.

Ideally you wouldn't need to pass --unsafe-perm - I'll create an issue for that and link this up and pin to the last known working version

@heitorlessa
Copy link

Also noticed today that the latest CLI introduced IdentityPoolId resource where it fails in new deployments for Console - Fixing all of that in an upcoming commit

heitorlessa added a commit to heitorlessa/aws-serverless-airline-booking that referenced this issue Aug 23, 2019
heitorlessa added a commit to heitorlessa/aws-serverless-airline-booking that referenced this issue Sep 3, 2019
heitorlessa added a commit that referenced this issue Sep 4, 2019
…s-airline-booking into heitorlessa-twitch

* 'twitch' of https://github.com/heitorlessa/aws-serverless-airline-booking: (90 commits)
  #57 disable log-processing stack
  fix: only subscribe to airline fns, disable own subs too
  fix: readd powertools env vars
  revert: add requests library
  fix: aplify DDB Encryption and Auth error
  improv: update gitignore to reflect backends
  improv: Update main readme w/ deployment instructions
  fix: #32 pin CLI version and IdentityPoolId issue
  fix: remove duplicate build target
  feat: impl custom metric for payment
  feat: impl custom metric for booking
  improv: add logger debug
  improv: raises exception in invalid metric units
  feat: add log_metric for custom metrics
  improv: use simplified logger
  feat: add cold_start key in lambda logs
  feat: return pre-configured logger
  improv: remove dataclass
  fix: cold_start annotation placement
  fix: remove duplicate tracing exception metadata
  ...
heitorlessa added a commit that referenced this issue Sep 4, 2019
* heitorlessa-twitch: (90 commits)
  #57 disable log-processing stack
  fix: only subscribe to airline fns, disable own subs too
  fix: readd powertools env vars
  revert: add requests library
  fix: aplify DDB Encryption and Auth error
  improv: update gitignore to reflect backends
  improv: Update main readme w/ deployment instructions
  fix: #32 pin CLI version and IdentityPoolId issue
  fix: remove duplicate build target
  feat: impl custom metric for payment
  feat: impl custom metric for booking
  improv: add logger debug
  improv: raises exception in invalid metric units
  feat: add log_metric for custom metrics
  improv: use simplified logger
  feat: add cold_start key in lambda logs
  feat: return pre-configured logger
  improv: remove dataclass
  fix: cold_start annotation placement
  fix: remove duplicate tracing exception metadata
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants