search

aws-solutions / mlops-workload-orchestrator

The MLOps Workload Orchestrator solution helps you streamline and enforce architecture best practices for machine learning (ML) model productionization. This solution is an extendable framework that provides a standard interface for managing ML pipelines for AWS ML services and third-party services.
https://aws.amazon.com/solutions/implementations/mlops-workload-orchestrator
Apache License 2.0
144 stars 52 forks source link

deployment folder #4

Closed smart-patrol closed 3 years ago

smart-patrol commented 3 years ago

Issue #, if available:

Description of changes: The documentation currently has the script running from the cloned, base folder. Needs to run from deployment folder to work.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

smart-patrol commented 3 years ago

FYI I also got the following:

------------------------------------------------------------------------------
[Synth] CDK Project
------------------------------------------------------------------------------
cd /home/ec2-user/environment/aws-mlops-framework/deployment/../source
npm install -g aws-cdk@1.83.0
npm ERR! code EEXIST
npm ERR! syscall symlink
npm ERR! path ../lib/node_modules/aws-cdk/bin/cdk
npm ERR! dest /home/ec2-user/.nvm/versions/node/v10.24.0/bin/cdk
npm ERR! errno -17
npm ERR! EEXIST: file already exists, symlink '../lib/node_modules/aws-cdk/bin/cdk' -> '/home/ec2-user/.nvm/versions/node/v10.24.0/bin/cdk'
npm ERR! File exists: /home/ec2-user/.nvm/versions/node/v10.24.0/bin/cdk
npm ERR! Remove the existing file and try again, or run npm
npm ERR! with --force to overwrite files recklessly.

npm ERR! A complete log of this run can be found in:
npm ERR!     /home/ec2-user/.npm/_logs/2021-03-18T05_51_09_453Z-debug.log

Ran --force at the npm install step but probably a better way to fix it.

aassadza commented 3 years ago

Hi @smart-patrol, thanks for you contribution and raising this issue. I have merged your pull request.