Unshallow repo after workspace is ready

[jeanp413]

Description

Unshallows a repo automatically in the background

Related Issue(s)

Fixes #6796

How to test

1. Open a workspace
2. Check that after a few seconds(maybe minutes depending on the repo size) the repo has been unshallowed git rev-parse --is-shallow-repository

Release Notes

NONE

Documentation

[codecov]

Codecov Report

Merging #6857 (739aec6) into main (a36fd97) will increase coverage by 18.10%.
The diff coverage is 0.00%.

@@             Coverage Diff             @@
##             main    #6857       +/-   ##
===========================================
+ Coverage   19.04%   37.15%   +18.10%     
===========================================
  Files           2       19       +17     
  Lines         168     4578     +4410     
===========================================
+ Hits           32     1701     +1669     
- Misses        134     2743     +2609     
- Partials        2      134      +132     

Flag	Coverage Δ
components-local-app-app-linux-amd64	?
components-local-app-app-linux-arm64	?
components-local-app-app-windows-386	?
components-local-app-app-windows-amd64	?
components-local-app-app-windows-arm64	?
components-supervisor-app	37.15% <0.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
components/supervisor/pkg/supervisor/supervisor.go	6.05% <0.00%> (ø)
components/local-app/pkg/auth/auth.go
components/local-app/pkg/auth/pkce.go
components/supervisor/pkg/dropwriter/dropwriter.go	73.46% <0.00%> (ø)
components/supervisor/pkg/ports/served-ports.go	80.00% <0.00%> (ø)
...mponents/supervisor/pkg/supervisor/notification.go	83.95% <0.00%> (ø)
components/supervisor/pkg/supervisor/tasks.go	58.56% <0.00%> (ø)
components/supervisor/pkg/ports/exposed-ports.go	0.00% <0.00%> (ø)
components/supervisor/pkg/terminal/service.go	32.38% <0.00%> (ø)
components/supervisor/pkg/ports/slirp4netns.go	0.00% <0.00%> (ø)
... and 12 more

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 915cd41...739aec6. Read the comment docs.

[aledbf]

Unshallows a repo automatically in the background

I thought this idea was rejected when I proposed it in #6464

[jeanp413]

Not sure why it was not included in the original PR but some of us (myself included) have been having issues with the shallow clone so we agreed to unshallow the repo (codespaces also unshallows in the background)

[svenefftinge]

I think we can do without any fetching on prebuilds.

[svenefftinge]

Unshallows a repo automatically in the background

I thought this idea was rejected when I proposed it in #6464

There has been discussions around pressure on always doing unshallow right after ready. Which made us think about more complex solutions (time-based, only certain cases, etc.). Also my analyzation of codespaces seems to have been wrong, as Jean Pierre found that they indeed just always do an unshallow after start. So, I think we should do the same here now as our current approach has negative effects as mentioned by Jean Pierre.

[jeanp413]

/werft run

👍 started the job as gitpod-build-jp-unshallow.7

[akosyakov]

/werft run

👍 started the job as gitpod-build-jp-unshallow.9

[csweichel]

We've just merged a fix for the broken chart. Please rebase which should solve the issue

[jeanp413]

Ready for review cc @akosyakov

[akosyakov]

/lgtm

[roboquat]

LGTM label has been added.

Git tree hash: 7750ed822b8b487cb00897db665c9892677683a1

[roboquat]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: akosyakov

Associated issue: #6796

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

• components/supervisor/OWNERS [akosyakov]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment