Skip to content

Conversation

@rpardini
Copy link
Member

@rpardini rpardini commented Jul 9, 2024

smee: introduce bool tink-server-insecure-tls controlling tinkerbell_insecure_tls kernel parameter

Signed-off-by: Ricardo Pardini ricardo@pardini.net

@rpardini rpardini force-pushed the add-tink-insecure-tls-flag branch from b9ea57b to d6f150f Compare July 12, 2024 06:31
mergify bot added a commit to tinkerbell/hook that referenced this pull request Jul 23, 2024
…pass it to worker as TINKERBELL_INSECURE_TLS (#234)

#### hook-bootkit: read `tinkerbell_insecure_tls` from kernel cmdline and pass it to worker as TINKERBELL_INSECURE_TLS

- this fits in with
  -  tinkerbell/smee#479
  -  tinkerbell/tink#960

Signed-off-by: Ricardo Pardini <ricardo@pardini.net>
@rpardini rpardini force-pushed the add-tink-insecure-tls-flag branch 2 times, most recently from df6f6c7 to 93f403f Compare August 17, 2024 08:25
…ll_insecure_tls` kernel parameter

- for usage with tinkerbell/tink#960

Signed-off-by: Ricardo Pardini <ricardo@pardini.net>
@rpardini rpardini force-pushed the add-tink-insecure-tls-flag branch from 93f403f to 7e803b4 Compare August 17, 2024 19:11
@codecov
Copy link

codecov bot commented Aug 17, 2024

Codecov Report

Attention: Patch coverage is 57.69231% with 11 lines in your changes missing coverage. Please review.

Project coverage is 52%. Comparing base (db21d79) to head (43f2e4a).
Report is 3 commits behind head on main.

Files Patch % Lines
cmd/smee/main.go 0% 11 Missing ⚠️
Additional details and impacted files
@@         Coverage Diff         @@
##           main   #479   +/-   ##
===================================
  Coverage    52%    52%           
===================================
  Files        26     26           
  Lines      2136   2139    +3     
===================================
+ Hits       1124   1126    +2     
- Misses      980    981    +1     
  Partials     32     32           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@jacobweinstock jacobweinstock added the ready-to-merge Signal to Mergify to merge the PR. label Aug 27, 2024
@mergify mergify bot merged commit 6e912d3 into tinkerbell:main Aug 27, 2024
mergify bot added a commit to tinkerbell/charts that referenced this pull request Oct 24, 2024
…ure-tls` (#114)

#### smee: add http.tinkServer.insecureTLS controlling `-tink-server-insecure-tls`

- this fits in with
  -  tinkerbell/smee#479
  -  tinkerbell/tink#960
  - tinkerbell/hook#234

Signed-off-by: Ricardo Pardini <ricardo@pardini.net>
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