Skip to content

Conversation

@yacaovsnc
Copy link
Contributor

@yacaovsnc yacaovsnc commented Apr 3, 2023

  • Add websocket client in ResultsServer
  • Adding boolean to enable upload to Actions service in JobServer, this is only used for Run svc messages.

@yacaovsnc yacaovsnc requested a review from a team as a code owner April 3, 2023 20:52
@yacaovsnc
Copy link
Contributor Author

Need to queue the CI build again.

@yacaovsnc yacaovsnc closed this Apr 24, 2023
@yacaovsnc yacaovsnc reopened this Apr 24, 2023
@TingluoHuang TingluoHuang merged commit ad9a4a4 into actions:main Apr 26, 2023
nikola-jokic pushed a commit to nikola-jokic/runner that referenced this pull request May 12, 2023
* Only upload to Results with new job message type

* No need to have separate websocketFeedServer

* Linting fix

* Update src/Runner.Common/JobServerQueue.cs

Co-authored-by: Tingluo Huang <[email protected]>

* add connection timeout

* Consolidate initializing webclient to result client

* Retry websocket delivery for console logs

* Linter fix

* Do not give up for good, reconnect again in 10 minutes

* Has to reset delivered

* Only first time retry 3 times to connect to websocket

---------

Co-authored-by: Tingluo Huang <[email protected]>
AdamOlech pushed a commit to antmicro/runner that referenced this pull request Jun 17, 2025
* Only upload to Results with new job message type

* No need to have separate websocketFeedServer

* Linting fix

* Update src/Runner.Common/JobServerQueue.cs

Co-authored-by: Tingluo Huang <[email protected]>

* add connection timeout

* Consolidate initializing webclient to result client

* Retry websocket delivery for console logs

* Linter fix

* Do not give up for good, reconnect again in 10 minutes

* Has to reset delivered

* Only first time retry 3 times to connect to websocket

---------

Co-authored-by: Tingluo Huang <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants