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

Not mounting FSx with ubuntu16.04 #896

Closed
JiaweiZhuang opened this issue Mar 1, 2019 · 4 comments
Closed

Not mounting FSx with ubuntu16.04 #896

JiaweiZhuang opened this issue Mar 1, 2019 · 4 comments
Labels

Comments

@JiaweiZhuang
Copy link

Environment:

  • AWS ParallelCluster 2.2.1
  • OS: ubuntu 16.04
  • Scheduler: Slurm
  • Master instance type: c5n.large
  • Compute instance type: c5n.18xlarge

Bug description and how to reproduce:
With base_os = ubuntu1604, FSx was created but not mounted to the cluster. Does it currently require centos7?

For example, centos7 would install Lustre client during launch as shown in /var/log/cfn-init.log:

Recipe: aws-parallelcluster::_lustre_install
  * remote_file[/opt/parallelcluster/sources/kmod-lustre-client-2.10.6.x86_64.rpm] action create (skipped due to not_if)
  * remote_file[/opt/parallelcluster/sources/lustre-client-2.10.6.x86_64.rpm] action create (skipped due to not_if)
  * yum_package[lustre_kmod] action install (up to date)
  * yum_package[lustre_client] action install (up to date)

With ubuntu, it doesn't seem to do anything with Lustre:

@JiaweiZhuang
Copy link
Author

I just notice Support for FSx Lustre with Centos 7 in release notes. Doesn't seem to be mentioned in the docs though.

@demartinofra
Copy link
Contributor

Hi @JiaweiZhuang

Yes FSx is only supported with Centos 7 in the current release.
I will update the docs accordingly and ideally we should also add a validation before cluster creation.

Thanks for reporting this!

@JiaweiZhuang
Copy link
Author

JiaweiZhuang commented Mar 1, 2019

Yes I can totally understand this because supporting other OS should not delay the current release. That's already a significant enhancement and thanks for the great work :)

@sean-smith
Copy link
Contributor

Fixed in #904

We added validation to detect an unsupported instance type. Stayed tuned for more instance types soon!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants