-
Notifications
You must be signed in to change notification settings - Fork 1
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
Conversation
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? |
@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 |
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 😄 |
note: version is the kurtosis-cdk GitHub action version and should be very
latest), this is not the actual kurtosis-cdk repo version
…On Fri, Jun 21, 2024 at 8:06 AM Victor Castell ***@***.***> wrote:
***@***.**** commented on this pull request.
------------------------------
In .github/workflows/regression-tests.yml
<#3 (comment)>:
> + description: 0xPolygon/kurtosis-cdk (release tag)
+ required: false
+ default: v0.2.0
+ bake_time:
+ description: bake time (minutes)
+ required: false
+ default: 30
+
+jobs:
+ regression_tests:
+ runs-on: ubuntu-latest
+ steps:
+ - name: Checkout Repository
+ uses: ***@***.***
+ - name: Run Regression Tests
+ uses: ***@***.***
Should we update the version?
—
Reply to this email directly, view it on GitHub
<#3 (review)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AX3X7Y2OWPA5I6OHOIVRWQDZIQQMVAVCNFSM6AAAAABGY3AQ7OVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDCMZSGYYTKNJUG4>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Oh yes sorry! |
Superseded by #11 |
Description
add the following features:
How Has This Been Tested?
Fully here, e2e devent workflow and cdk integration tests all working as expected:
Screenshots (if appropriate):
Additional context
devTools jira ticket: DVT-1464