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.

Sign up for GitHub

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

Jump to bottom

No check for prerequisites for Terraform AWS-EKS install in CLI #3359

Closed
2 tasks done
armanjindal opened this issue Jun 29, 2022 · 1 comment
Closed
2 tasks done

No check for prerequisites for Terraform AWS-EKS install in CLI #3359

armanjindal opened this issue Jun 29, 2022 · 1 comment
Assignees
@armanjindal
Milestone
Terraform Install...

Comments

@armanjindal
Copy link
Contributor

armanjindal commented Jun 29, 2022 •
edited

As we modify the CLI command airy create --provider=aws to use the AWS-EKS Terraform install, instead of the current hard-coded version, we must first check the necessary binaries are installed on the installer's computer.

  • aws CLI tool is installed
  • terraform is installed
@armanjindal armanjindal self-assigned this Jun 29, 2022
@armanjindal armanjindal added epic and removed epic labels Jun 29, 2022
armanjindal added a commit that referenced this issue Jul 5, 2022
@armanjindal
[#3359] Add method to check environment pre-install
9ce07f4
@armanjindal armanjindal mentioned this issue Jul 5, 2022
Closed
armanjindal added a commit that referenced this issue Jul 6, 2022
@armanjindal
[#3359] Fixing to better and idiomatic Go
351b36e
@armanjindal armanjindal mentioned this issue Jul 7, 2022
Closed
5 tasks
armanjindal added a commit that referenced this issue Jul 8, 2022
@armanjindal
[#3397] Fixes #3359 - Checks necessary binaries
dd56cec
@armanjindal
Copy link
Contributor Author

Fixed in #3404

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@armanjindal armanjindal

Labels
None yet
Projects
None yet
Milestone
Terraform Installation of Airy-Core u...
Development

Successfully merging a pull request may close this issue.

[#3359] Add method to check environment pre-install airyhq/airy
1 participant
@armanjindal