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

[KPT CODELAB] (1/3) New codelab dir + the sample application resources #5023

Merged
merged 1 commit into from
Nov 13, 2020

Conversation

yuwenma
Copy link
Contributor

@yuwenma yuwenma commented Nov 13, 2020

This PR adds the sample application for the kpt codelab.

Fixes: #4881 #3904

@yuwenma yuwenma requested a review from a team as a code owner November 13, 2020 01:55
@google-cla google-cla bot added the cla: yes label Nov 13, 2020
@codecov
Copy link

codecov bot commented Nov 13, 2020

Codecov Report

Merging #5023 (5da6424) into master (be1c5e0) will decrease coverage by 0.02%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #5023      +/-   ##
==========================================
- Coverage   72.20%   72.18%   -0.03%     
==========================================
  Files         365      365              
  Lines       12842    12842              
==========================================
- Hits         9273     9270       -3     
- Misses       2880     2881       +1     
- Partials      689      691       +2     
Impacted Files Coverage Δ
pkg/skaffold/docker/image.go 79.34% <0.00%> (-1.41%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update be1c5e0...2bbce9e. Read the comment docs.

@yuwenma yuwenma changed the title [KPT CODELAB] (1/3) New codelab dir + the sample application resources [WIP][KPT CODELAB] (1/3) New codelab dir + the sample application resources Nov 13, 2020
@yuwenma yuwenma changed the title [WIP][KPT CODELAB] (1/3) New codelab dir + the sample application resources [KPT CODELAB] (1/3) New codelab dir + the sample application resources Nov 13, 2020
@MarlonGamez
Copy link
Contributor

I think this should be fine to go in, since it's just the sample app files for the codelab. Took a look over everything and it seems good :) I'll run kokoro and merge assuming that goes well 👍🏼

@yuwenma
Copy link
Contributor Author

yuwenma commented Nov 13, 2020

I think this should be fine to go in, since it's just the sample app files for the codelab. Took a look over everything and it seems good :) I'll run kokoro and merge assuming that goes well 👍🏼

Thank you!

@MarlonGamez MarlonGamez merged commit 1c2e220 into GoogleContainerTools:master Nov 13, 2020
yuwenma added a commit to yuwenma/skaffold that referenced this pull request Nov 16, 2020
MarlonGamez pushed a commit that referenced this pull request Dec 11, 2020
* [kpt codelab] Add codelab for kpt validations

* update URL since 1/3 #5023 is merged

* fix nits and issues

* Change codelab content due to kpt v1 changes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add codelab to introduce kpt deployer
2 participants