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

feature: CI-based CDK devnets with integration tests #3

Closed
wants to merge 5 commits into from

Conversation

rebelArtists
Copy link

@rebelArtists rebelArtists commented Apr 25, 2024

Description

add the following features:

  • a github actions workflow that allows cdk eng teams to create and test CDK devnets in less <20min
  • cdk eng team can custom-specify the exact commit id OR existing release tag of any upstream CDK repo, and create JIT devnet on the fly using this new github action
  • users get a clear go/no-go CI check whether the devnet is behaving as expected (i.e. verifiedBatchNumber is incrementing) and can withstand a series of transactions load tests (via polycli)

How Has This Been Tested?

Fully here, e2e devent workflow and cdk integration tests all working as expected:

Screenshots (if appropriate):

image image image image

Additional context

devTools jira ticket: DVT-1464

@vcastellm
Copy link
Contributor

Hi @rebelArtists the time is now for this 😄 , is there a way to specify one or some of the components to be the local repo Dockerfile build instead of an existing versioned image?

@rebelArtists
Copy link
Author

@vcastellm current code fully supports any existing released image OR short commit hash (will perform custom container build on the fly) for any upstream CDK repo

supporting local repo Dockerfiles would be custom functionality, would be interesting to better understand the use case and whether supporting commit hash is sufficient

@vcastellm
Copy link
Contributor

Well, actually this is good as it is for the nighly builds use case, but I'm thinking in per PR runs.

Another use case is testing in local, so I can spin up my local Kurtosis setup but exclude the service I want to run in local, but now that I think that's pretty easy to do 😄

@rebelArtists
Copy link
Author

rebelArtists commented Jun 22, 2024 via email

@vcastellm
Copy link
Contributor

note: version is the kurtosis-cdk GitHub action version and should be very latest), this is not the actual kurtosis-cdk repo version

Oh yes sorry!

@vcastellm
Copy link
Contributor

Superseded by #11

@vcastellm vcastellm closed this Jul 4, 2024
@vcastellm vcastellm deleted the dan/regression_tests branch September 17, 2024 09:43
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