Skip to content

Conversation

@addaleax
Copy link
Collaborator

Our serverless cluster is no longer a serverless cluster, since they are being converted to Flex tier clusters.

This does not remove every mention of serverless from the Compass source code, although I imagine we would be able to do so relatively soon.

Refs: https://www.mongodb.com/docs/atlas/billing/serverless-instance-costs/

Description

Checklist

  • New tests and/or benchmarks are included
  • Documentation is changed or added
  • If this change updates the UI, screenshots/videos are added and a design review is requested
  • I have signed the MongoDB Contributor License Agreement (https://www.mongodb.com/legal/contributor-agreement)

Motivation and Context

  • Bugfix
  • New feature
  • Dependency update
  • Misc

Open Questions

Dependents

Types of changes

  • Backport Needed
  • Patch (non-breaking change which fixes an issue)
  • Minor (non-breaking change which adds functionality)
  • Major (fix or feature that would cause existing functionality to change)

Our serverless cluster is no longer a serverless cluster,
since they are being converted to Flex tier clusters.

This does not remove every mention of serverless from the
Compass source code, although I imagine we would be able to
do so relatively soon.

Refs: https://www.mongodb.com/docs/atlas/billing/serverless-instance-costs/
@addaleax addaleax marked this pull request as ready for review October 29, 2025 13:46
@addaleax addaleax requested a review from a team as a code owner October 29, 2025 13:46
@addaleax addaleax requested review from Copilot and lerouxb October 29, 2025 13:46
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR removes serverless cluster connection tests from the Compass codebase, as MongoDB's serverless clusters are being converted to Flex tier clusters. The changes clean up test code, environment variables, and configuration files that were previously used for serverless connectivity testing.

Key Changes:

  • Removed serverless-specific test cases from connection test suites
  • Cleaned up serverless-related environment variables from configuration files
  • Removed serverless connection string construction and test utilities

Reviewed Changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.

File Description
packages/data-service/src/connect.spec.ts Removed serverless host import, connection string constant, and serverless connectivity test
packages/compass-e2e-tests/tests/connection.test.ts Removed serverless host from environment variable checks and deleted two serverless connection tests (connection string and form-based)
.evergreen/functions.yml Removed E2E_TESTS_SERVERLESS_HOST environment variable declaration
.evergreen/connectivity-tests/run.sh Removed E2E_TESTS_SERVERLESS_HOST from Docker container environment variable pass-through

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@codeowners-service-app
Copy link

Assigned nbbeeken for team compass-developers because lerouxb is out of office.

@addaleax addaleax added the no-title-validation Skips validation of PR titles (conventional commit adherence + JIRA ticket inclusion) label Oct 29, 2025
@addaleax addaleax merged commit 4c9632f into main Oct 31, 2025
60 of 63 checks passed
@addaleax addaleax deleted the drop-serverless branch October 31, 2025 12:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-title-validation Skips validation of PR titles (conventional commit adherence + JIRA ticket inclusion)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants