Skip to content

Local setup for end2end tests#3062

Merged
pcapriotti merged 30 commits intodevelopfrom
pcapriotti/end2end
Feb 14, 2023
Merged

Local setup for end2end tests#3062
pcapriotti merged 30 commits intodevelopfrom
pcapriotti/end2end

Conversation

@pcapriotti
Copy link
Contributor

@pcapriotti pcapriotti commented Feb 7, 2023

This introduces a replacement for integration.sh, written in python, that is able to run two (or more) backends at the same time. It works by creating configuration files on the fly for a set of services, modifying them so that they work together on a new set of ports.

TODO

  • Set up local SRV records
  • Add spawn timeout
  • Add a second cannon instance
  • Spawn only one backend when INTEGRATION_FEDERATION_TESTS is not set
  • db-reset should reset the second test keyspace as well
  • Remove legacy integration.sh script
  • The two backends should use different elasticsearch indices

Checklist

  • Add a new entry in an appropriate subdirectory of changelog.d
  • Read and follow the PR guidelines

@zebot zebot added the ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist label Feb 7, 2023
@pcapriotti pcapriotti marked this pull request as ready for review February 7, 2023 15:17
Copy link
Contributor

@stefanwire stefanwire left a comment

Choose a reason for hiding this comment

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

Just a nit left.

@pcapriotti pcapriotti marked this pull request as draft February 9, 2023 11:08
@pcapriotti pcapriotti marked this pull request as ready for review February 9, 2023 13:03
@pcapriotti pcapriotti merged commit 6a6bcd7 into develop Feb 14, 2023
@pcapriotti pcapriotti deleted the pcapriotti/end2end branch February 14, 2023 15:24
@smatting smatting mentioned this pull request Mar 7, 2023
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

Comments