Fix intermittent integration test failures

[rl-gitpod]

Description

2 minor changes here.
The most impactful is the ws-proxy change where an active workspace could be deleted from the cache by the stop event of the same workspace id. This can happen in integration testing or via the API, but the UI would prevent this happening in user-land as it waits for the stopped event before allowing another start.

Related Issue(s)

Fixes #5413

How to test

Run the 'TestBackup' test:

go test -count=1 -v -run TestBackup ./...

Prior to these changes it will intermittently fail, but if run multiple times in succession it would fail more 'reliably'.
Now, the following repeated test will succeed:

for i in $(seq 1 10); do go test -count=1 -v -run TestBackup ./...; done

Release Notes

NONE

[codecov]

Codecov Report

Merging #5617 (3264c6c) into main (5ecb8f5) will increase coverage by 13.15%.
The diff coverage is 85.24%.

@@             Coverage Diff             @@
##             main    #5617       +/-   ##
===========================================
+ Coverage   19.04%   32.20%   +13.15%     
===========================================
  Files           2       28       +26     
  Lines         168     5136     +4968     
===========================================
+ Hits           32     1654     +1622     
- Misses        134     3329     +3195     
- Partials        2      153      +151     

Flag	Coverage Δ
components-blobserve-app	29.31% <ø> (?)
components-local-app-app-linux	?
components-local-app-app-windows	?
components-registry-facade-app	11.63% <0.00%> (?)
components-ws-daemon-app	22.11% <0.00%> (?)
components-ws-proxy-app	69.04% <96.29%> (?)

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

Impacted Files	Coverage Δ
...omponents/registry-facade/pkg/registry/manifest.go	9.65% <0.00%> (ø)
components/ws-daemon/pkg/content/service.go	0.00% <0.00%> (ø)
components/ws-proxy/pkg/proxy/infoprovider.go	60.06% <96.29%> (ø)
components/local-app/pkg/auth/pkce.go
components/local-app/pkg/auth/auth.go
...omponents/registry-facade/pkg/registry/registry.go	0.00% <0.00%> (ø)
components/ws-proxy/pkg/proxy/workspacerouter.go	81.73% <0.00%> (ø)
components/ws-daemon/pkg/quota/size.go	87.30% <0.00%> (ø)
... and 23 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 5ecb8f5...3264c6c. Read the comment docs.

[rl-gitpod]

/hold

Refactoring to use instance id and adding tests

[rl-gitpod]

/werft run

👍 started the job as gitpod-build-rl-flaky-test-5413.16

[rl-gitpod]

/werft run

👍 started the job as gitpod-build-rl-flaky-test-5413.18

[rl-gitpod]

/werft run

👍 started the job as gitpod-build-rl-flaky-test-5413.24

[rl-gitpod]

/unhold

[csweichel]

imho the changes could go in like this, but I believe there's a chance to simplify things a little

[csweichel]

/lgtm

[roboquat]

LGTM label has been added.

Git tree hash: f97190d139ec78a38c579290c1374481b849e556

[roboquat]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: csweichel

Associated issue: #5413

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:

• OWNERS [csweichel]
• components/registry-facade/OWNERS [csweichel]
• components/ws-daemon/OWNERS [csweichel]
• components/ws-proxy/OWNERS [csweichel]

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

[aledbf]

/werft run

👍 started the job as gitpod-build-rl-flaky-test-5413.34

[aledbf]

/werft run

👍 started the job as gitpod-build-rl-flaky-test-5413.35

[aledbf]

/werft run

👍 started the job as gitpod-build-rl-flaky-test-5413.36