Skip to content

Conversation

cfsnate
Copy link

@cfsnate cfsnate commented Jan 23, 2022

Could not deploy CDK and SOCA without changes to IAM role. Permissions fixed in example IAM policy json.

Amazon Linux would not deploy because of lack of Dev tools, could not compile Python, OpenPBS, OpenMPI, fixed in Scheduler.sh

NodeJS 16 would not install from epel on AML2 because of libuv dependancy. Installing NodeJS 16 first then pulling required packages from repos.

Updated base AMI for us-east-2 to latest AL2.

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

@mcrozes mcrozes added the dependencies Pull requests that update a dependency file label Jan 23, 2022
@mcrozes mcrozes self-assigned this Jan 23, 2022
@mcrozes
Copy link
Collaborator

mcrozes commented Jan 23, 2022

Hello @cfsnate -

NodeJS

Thanks for reporting this issue. I was able to reproduce this issue on my environment. It seems doing yum install nodejs is not the most reliable way to install node/npm, instead we are considering switching to nvm approach (similar to what we are doing on soca_installer.sh https://github.com/awslabs/scale-out-computing-on-aws/blob/main/installer/soca_installer.sh#L100-L115)

IAM Policies

Will review and provide an update soon

@mcrozes
Copy link
Collaborator

mcrozes commented Jan 24, 2022

Hello @cfsnate -

We have been able to reproduce and fix both errors. We are currently reviewing everything and will push a new update soon.
Once complete, I will go ahead and close your PR but will credit you in the commit message (as we are pushing a couple of additional commits at the same time)

mcrozes added a commit that referenced this pull request Jan 25, 2022
- Fixed IAM policies required to install SOCA and added support for cdk boostrap (#64: Contributor @cfsnate)
- More consistent way to install EPEL repository across distros
- Better way to install SSM on the Scheduler host (similar to what we are already doing with ComputeNodes)
- Updated remote job submission to fix error with group ownership when using a remote input file
@mcrozes
Copy link
Collaborator

mcrozes commented Jan 25, 2022

Hello @cfsnate -

Thanks again for your CR. Changes have been pushed via fd6f1b6

@mcrozes mcrozes closed this Jan 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants