Skip to content
This repository has been archived by the owner on May 8, 2024. It is now read-only.

Reduce the number of ELBs in the U-SPS cluster #144

Merged
merged 13 commits into from
Jan 23, 2023
Merged

Conversation

drewm-jpl
Copy link
Contributor

@drewm-jpl drewm-jpl commented Jan 23, 2023

Purpose

  • This PR reduces the number of Elastic Load Balancers (ELBs) used in the SPS cluster from seven to one. At the moment, the WPS-T API is the only service that needs an externally accessible endpoint. For this reason, the WPS-T API is now the only service with an ELB in front of it. The remaining SPS services are only accessible from within the cluster itself.
  • Minor housekeeping tasks are also included in this PR. they are described in the Proposed Changes section.

Proposed Changes

  • Remove ELBs in front of the following services: HySDS UI, Mozart REST API, Mozart Elasticsearch, GRQ REST API, GRQ Elasticsearch, minIO.
  • Bump the required Terraform version from 1.1.8 to 1.3.6.
  • Bump the required AWS resource version from 4.23.0 to 4.51.0.
  • Remove minIO.
  • Remove SPS's temporary S3 staging bucket.

Issues

Testing

  • The SPS cluster was deployed in MCP Dev and the L1B CWL workflow was successfully deployed and executed via manual calls to the WPS-T API.
  • Future work should extend the existing smoke test to automate this testing procedure. It should then be automatically run upon the opening of a PR to this repo.

@drewm-jpl drewm-jpl changed the title 120 reduce elbs Reduce the number of ELBs in the U-SPS cluster Jan 23, 2023
@drewm-jpl drewm-jpl marked this pull request as ready for review January 23, 2023 20:15
Copy link
Collaborator

@LucaCinquini LucaCinquini left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The changes look ok to me (although I have not checked every single line). Once this is merged into main I can try to deploy U-SPS on MCP-DEV

@drewm-jpl drewm-jpl merged commit 4c4725d into main Jan 23, 2023
@LucaCinquini LucaCinquini added Epic and removed Epic labels Jan 25, 2023
@drewm-jpl drewm-jpl deleted the 120-reduce-elbs branch April 12, 2023 00:42
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Reduce the number of ELBs in the U-SPS cluster
2 participants