jumpstarter-dev / jumpstarter

http://docs.jumpstarter.dev/
3 stars 3 forks source link

Make use of new status method #123

Closed NickCao closed 1 month ago

NickCao commented 1 month ago

Got some race condition on the controller side: https://github.com/jumpstarter-dev/jumpstarter-controller/actions/runs/11294698671/job/31416030560#step:9:75

The client could be too eager to connect to a newly acquired lease that the exporter is not even listening yet.

NickCao commented 1 month ago

Got some race condition on the controller side: https://github.com/jumpstarter-dev/jumpstarter-controller/actions/runs/11294698671/job/31416030560#step:9:75

The client could be too eager to connect to a newly acquired lease that the exporter is not even listening yet.

Resolved by allowing the client to wait for the exporter: https://github.com/jumpstarter-dev/jumpstarter-controller/pull/53