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

[ci] Test deployment to AWS #974

Merged
merged 28 commits into from
Dec 11, 2023
Merged

Conversation

barroco
Copy link
Contributor

@barroco barroco commented Dec 8, 2023

This PR adds a deployment scenario (aws-1) to the repository which sets up a Kubernetes cluster in AWS using a user with limited permissions on the aws account. For the moment, the scenario only setup the Kubernetes infrastructure and tears it down. A Github Actions workflow is configured to run it on pull requests or manually.

This PR will be followed by two others to keep the changeset as small as possible:

  1. Deployment of the DSS using helm.
  2. Test the deployment of the DSS using interuss/monitoring.

Details of the changes:

  • Add new operations deployment utilities in /deploy/operations to create and destroy the Kubernetes cluster.
  • terraform-aws-kubernetes
    • Add aws_iam_permissions_boundary variable
    • Add iam_role_node_group_arn output
    • Retrieve token using terraform instead of CLI command
    • Various structure improvements
  • Configure the new dss-deploy github workflow

@barroco barroco changed the title [ci] Test DSS deployment to AWS [ci] Test deployment to AWS Dec 8, 2023
@barroco
Copy link
Contributor Author

barroco commented Dec 8, 2023

We expect that the workflow must exist on the master branch to get the proper permissions.
It was tested on the forked repository: https://github.com/Orbitalize/dss/actions/runs/7142862936
As discussed with @BenjaminPelletier, we will move forward with merging the PR once reviewed and will be ready to revert the change in case of issue.

@barroco barroco marked this pull request as ready for review December 11, 2023 16:48
@barroco barroco merged commit 456d0cc into interuss:master Dec 11, 2023
6 of 7 checks passed
barroco added a commit that referenced this pull request Dec 11, 2023
barroco added a commit that referenced this pull request Dec 11, 2023
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

Successfully merging this pull request may close these issues.

2 participants