-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
✨ Add tilt-based development tooling #1894
Conversation
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: ncdc The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
This is rough around the edges & I still need to test a few scenarios, but I wanted to post for 👀 |
Also, I noticed after I added the new |
Also note, if you want to use this with CAPA, it requires a corresponding CAPA PR that I haven't opened yet. Will do soon. |
@chuckha this isn't 100% ready for review, but if you wanted to see what I've been working on... |
@ncdc what if only the capi (and capd, since it's in repo) config was in the Tiltfile here, and we added an additional Tiltfile to the AWS provider as well? This would require running tilt multiple times (once for each provider), but wouldn't require configuration as to where the other repos are |
I can pull the aws configuration and move it to capa (and have this tiltfile include it), but I'd still like to have a single primary tiltfile/ |
db73316
to
4f0961f
Compare
Yup, this approach looks like an evolution of what I got. Including a provider Tiltfile works well enough I think, with the config saying which provider you're currently working with. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
/hold
feel free to unhold if there are no other comments
Re-enabling WIP as live update isn't working consistently - need to fix. |
Fixed live updating (arg missing slash) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
one minor comment, looks great.
Signed-off-by: Andy Goldstein <goldsteina@vmware.com>
/hold cancel |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
Tested on Linux |
tested on macOS /lgtm |
/test pull-cluster-api-integration |
What this PR does / why we need it:
Add tilt-based development tooling to simplify rapid iterative development.
Which issue(s) this PR fixes (optional, in
fixes #<issue number>(, fixes #<issue_number>, ...)
format, will close the issue(s) when PR gets merged):Fixes #