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

Run integration tests against deployment #2962

Draft
wants to merge 9 commits into
base: main
Choose a base branch
from

Conversation

soapy1
Copy link
Contributor

@soapy1 soapy1 commented Feb 14, 2025

Reference Issues or PRs

#2855

What does this implement/fix?

This PR updates the integration tests to support running against an existing cluster. You can do this by providing the --existing-deployment flag (opposed to the --cloud flag) when running the integration tests. --existing-deployment gets a path to an existing nebari config dir (must contain the nebari-config.yaml and the output dir /stages). For example see How to test this PR?

Also adds these tests to the Local Integration Tests GHA workflow.

Put a x in the boxes that apply

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds a feature)
  • Breaking change (fix or feature that would cause existing features not to work as expected)
  • Documentation Update
  • Code style update (formatting, renaming)
  • Refactoring (no functional changes, no API changes)
  • Build related changes
  • Other (please describe):

Testing

  • Did you test the pull request locally?
  • Did you add new tests?

How to test this PR?

Create a local deploy of nebari and then run the integration tests.

$ mkdir nebari-local
$ nebari init ....
$ nebari deploy ....
$ cd ./tests/tests_integration
$  pytest -vvv -s --existing-deployment=../../nebari-local .

@soapy1 soapy1 force-pushed the run-integration-tests-against-deployment branch from 17f2825 to 0be0ff9 Compare February 14, 2025 21:56
@soapy1 soapy1 force-pushed the run-integration-tests-against-deployment branch from 41caaed to 8faddb2 Compare February 14, 2025 22:51
@soapy1 soapy1 force-pushed the run-integration-tests-against-deployment branch from f1e7883 to f857acc Compare February 14, 2025 23:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: New 🚦
Development

Successfully merging this pull request may close these issues.

1 participant