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

Fix of template CRD example #670

Merged
merged 1 commit into from
Jan 18, 2023
Merged

Conversation

AleksandrNull
Copy link
Contributor

Description

  1. Prepend writefile image with quay registry path
  2. kexec version fix (there is no 1.0.1 tag)

Why is this needed

Examples do not work out-of-the-box

Tested at my local setup as-is

@codecov
Copy link

codecov bot commented Jan 18, 2023

Codecov Report

Merging #670 (80accbd) into main (ae32f7f) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #670   +/-   ##
=======================================
  Coverage   48.26%   48.26%           
=======================================
  Files          18       18           
  Lines         951      951           
=======================================
  Hits          459      459           
  Misses        484      484           
  Partials        8        8           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@jacobweinstock
Copy link
Member

thanks @AleksandrNull ! mind signing the commit? git commit --amend --signoff --no-edit so the DCO check passes, please?

1. Prepend writefile image with quay registry path
2. kexec version fix

Signed-off-by: Aleksandr Shaposhnikov <[email protected]>
@jacobweinstock jacobweinstock added the ready-to-merge Signal to Mergify to merge the PR. label Jan 18, 2023
@mergify mergify bot merged commit bb48d12 into tinkerbell:main Jan 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-to-merge Signal to Mergify to merge the PR.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants