Conversation
Codecov Report
@@ Coverage Diff @@
## main #81 +/- ##
==========================================
+ Coverage 34.78% 34.89% +0.11%
==========================================
Files 6 6
Lines 1912 1917 +5
==========================================
+ Hits 665 669 +4
Misses 1143 1143
- Partials 104 105 +1
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
samples/ubuntu-dual-stack.yaml
Outdated
There was a problem hiding this comment.
Is this a limitation of cloud-init?
There was a problem hiding this comment.
I'm not sure, but still trying to figure it out.
There was a problem hiding this comment.
Maybe it's better to use a log lib that supports file truncate?
There was a problem hiding this comment.
Ok，i'll use a lib to deal the log message.
carezkh
force-pushed
the
care-dev-dual-stack
branch
from
4b04033 to
46c2728
Compare
There was a problem hiding this comment.
The func name should be discoverXXX
There was a problem hiding this comment.
Any reason we choose this log lib? It doesn't seem to have many stars
There was a problem hiding this comment.
There are a little few log libs support rotation, this lib was imported by most users. However, it doesn't seem active and thread-safe.
Any suggestions? Or just use the built-in log without rotation beacase i have removed periodical logs that would cause the size of the log file to grow rapidly.
There was a problem hiding this comment.
For a process in a container, it's better we log to console
carezkh
force-pushed
the
care-dev-dual-stack
branch
2 times, most recently
from
3616f37 to
25c6e57
Compare
carezkh
force-pushed
the
care-dev-dual-stack
branch
2 times, most recently
from
45c6069 to
5708542
Compare
carezkh
force-pushed
the
care-dev-dual-stack
branch
from
5708542 to
cde8228
Compare
|
I was wondering whether this is going to be merged anytime in the near future? |
|
@telmich Sorry, but we are currently lacking both time and env to update and test this PR out. Feel free to take over this PR. |
3 participants
No description provided.