feat: add timeout for copying build context on kaniko

[hiro-o918]

Related: #7887

Description

I found that uploading build context to kaniko pods sometimes stacks when copying build context on same conditions.
I inspected a pod on stack and found three things.

• initContainer keeps running
• /kaniko/buildcontext is empty
• /tmp/complete file is not created

I did not figure out exact reasons, but it supposed to be caused by creating tarball and uploading to pods.

Current implementation does not has timeout on creating tar, so I implemented this by passing context.Context.

Problem
This implementation assumes that the copying to pod must finish in 5 minutes, otherwise fail.
So We need to pass timeout duration if required.

User facing changes (remove if N/A)

All copying steps to kaniko pod must be finished in 5 minutes.

[codecov]

Codecov Report

Merging #8329 (175f45e) into main (290280e) will decrease coverage by 4.61%.
The diff coverage is 54.59%.

❗ Current head 175f45e differs from pull request most recent head d64a772. Consider uploading reports for the commit d64a772 to get more accurate results

@@            Coverage Diff             @@
##             main    #8329      +/-   ##
==========================================
- Coverage   70.48%   65.87%   -4.61%     
==========================================
  Files         515      605      +90     
  Lines       23150    29805    +6655     
==========================================
+ Hits        16317    19634    +3317     
- Misses       5776     8696    +2920     
- Partials     1057     1475     +418     

Impacted Files	Coverage Δ
cmd/skaffold/app/cmd/completion.go	13.04% <0.00%> (-1.25%)	⬇️
cmd/skaffold/app/cmd/config/list.go	65.21% <ø> (ø)
cmd/skaffold/app/cmd/config/set.go	88.72% <ø> (ø)
cmd/skaffold/app/cmd/config/util.go	54.28% <ø> (ø)
cmd/skaffold/app/cmd/credits.go	100.00% <ø> (ø)
cmd/skaffold/app/cmd/credits/export.go	0.00% <0.00%> (ø)
cmd/skaffold/app/cmd/deploy.go	40.90% <0.00%> (-12.94%)	⬇️
cmd/skaffold/app/cmd/flags.go	93.00% <ø> (+2.18%)	⬆️
cmd/skaffold/app/cmd/generate_pipeline.go	60.00% <ø> (ø)
cmd/skaffold/app/cmd/inspect_modules.go	65.00% <ø> (ø)
... and 416 more

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

[hiro-o918]

• I fixed lint error
• modify configuration to check timeout of tar_test which failed on only windows
  • But I'm not sure the reason, do you have any idea about this?

[aaron-prindle]

Thanks for the PR here @hiro-o918! In looking at out CI/CD tests it seems the test TestAddFileToTarTimeout is failing:

--- FAIL: TestAddFileToTarTimeout (0.01s)
    --- FAIL: TestAddFileToTarTimeout/TestAddFileToTarTimeout (0.01s)
        tar_test.go:302: expected error, but returned none

[hiro-o918]

@aaron-prindle
Thank you for checking this PR.
The test fails only on windows, and I do not have any windows machine now.
I'll investigate the why context timeout does not work on windows but if your team knows any clues, please tell me 🙏

[hiro-o918]

Windows error is fixed by increasing the duration of sleeping to wait timeout.
I suppose that It may be caused by difference among OSs to handle go routine.

[aaron-prindle]

LGTM! Thanks for the PR here @hiro-o918!