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

Add scripts to build kpack with Tilt #1345

Merged
merged 2 commits into from
Oct 25, 2023
Merged

Add scripts to build kpack with Tilt #1345

merged 2 commits into from
Oct 25, 2023

Conversation

xtreme-shane-lattanzio
Copy link
Contributor

This PR introduces tilt to kpack. Each image is built individually. The triggers may be slightly off so they may need to be tweaked but I have tested that kpack does deploy and function properly.

The config-logging configmap needs to be commented out for the k8s_kind image swapping to function properly.

The reason for both pack and local resources are for live loading. I am still working on validating that this works properly but wanted to put something up in case there was any feedback since I am just getting started with kpack. Any feedback welcome!

@xtreme-shane-lattanzio xtreme-shane-lattanzio requested a review from a team as a code owner October 5, 2023 15:43
@codecov-commenter
Copy link

Codecov Report

Merging #1345 (b9e77c3) into main (ef0f7c7) will decrease coverage by 0.08%.
Report is 22 commits behind head on main.
The diff coverage is n/a.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

@@            Coverage Diff             @@
##             main    #1345      +/-   ##
==========================================
- Coverage   67.41%   67.34%   -0.08%     
==========================================
  Files         133      133              
  Lines        8093     8210     +117     
==========================================
+ Hits         5456     5529      +73     
- Misses       2200     2231      +31     
- Partials      437      450      +13     

see 13 files with indirect coverage changes

@sambhav sambhav merged commit 7ba039c into main Oct 25, 2023
3 checks passed
@sambhav sambhav deleted the local_tilt branch October 25, 2023 09:59
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.

4 participants