Added virtual worker #602
Added virtual worker #602
Conversation
![sonatype-lift[bot]](https://avatars.githubusercontent.com/in/9843?s=60&v=4){kind=small}
| ) | ||
|
|
||
| func getRandHexStr(length int) string { | ||
| r := rand.New(rand.NewSource(time.Now().UnixNano())) |
There was a problem hiding this comment.
G404: Use of weak random number generator (math/rand instead of crypto/rand)
(at-me in a reply with help or ignore)
micahhausler
force-pushed
the
worker/virtual-worker
branch
2 times, most recently
from
9aeacad to
5417ada
Compare
Codecov Report
@@ Coverage Diff @@
## main #602 +/- ##
=======================================
Coverage 45.86% 45.86%
=======================================
Files 56 56
Lines 3268 3268
=======================================
Hits 1499 1499
Misses 1686 1686
Partials 83 83 Continue to review full report at Codecov.
|
micahhausler
force-pushed
the
worker/virtual-worker
branch
from
5417ada to
c291002
Compare
micahhausler
force-pushed
the
worker/virtual-worker
branch
from
c291002 to
12adba2
Compare
micahhausler
force-pushed
the
worker/virtual-worker
branch
from
12adba2 to
3921cb3
Compare
|
|
||
| func getRandHexStr(length int) string { | ||
| // intentionally weak RNG. This is only for fake output | ||
| r := rand.New(rand.NewSource(time.Now().UnixNano())) |
There was a problem hiding this comment.
G404: Use of weak random number generator (math/rand instead of crypto/rand)
(at-me in a reply with help or ignore)
Was this a good recommendation?
[ 🙁 Not relevant ] - [ 😕 Won't fix ] - [ 😑 Not critical, will fix ] - [ 🙂 Critical, will fix ] - [ 😊 Critical, fixing now ]
micahhausler
force-pushed
the
worker/virtual-worker
branch
from
3921cb3 to
2c1045c
Compare
Signed-off-by: Micah Hausler <[email protected]>
micahhausler
force-pushed
the
worker/virtual-worker
branch
from
2c1045c to
c4d4735
Compare
Add virtual tink worker command
Description
Why is this needed
This worker is helpful if you want to test Tinkerbell API changes while not actually executing a worker that executes docker containers.
Fixes: #
How Has This Been Tested?
Tested locally against the Tinkerbell API. I'll have an E2E test in a future PR that uses the no-op logger and container manager.
How are existing users impacted? What migration steps/scripts do we need?
No impact to existing users.
Checklist:
I have: