Skip to content

ci: migrate workflows from self-hosted to GitHub-hosted runners#106

Merged
MaxSiominDev merged 1 commit into
masterfrom
migrate-to-github-hosted-runners
Jul 2, 2026
Merged

ci: migrate workflows from self-hosted to GitHub-hosted runners#106
MaxSiominDev merged 1 commit into
masterfrom
migrate-to-github-hosted-runners

Conversation

@MaxSiominDev

@MaxSiominDev MaxSiominDev commented Jun 29, 2026

Copy link
Copy Markdown
Contributor

TDGClientTest queried spaces right after cluster.start(), which only waits for the containers to come up. The TDG data model (and therefore the User/person spaces) is applied asynchronously afterwards, so on slower runners the first queries hit a not-yet-created space and failed with "attempt to index ... 'space' (a nil value)". Wait for the model in @BeforeAll by probing both spaces via retryUntilSuccess before the tests run.

I haven't forgotten about:

  • Tests
  • Changelog
  • Documentation
    • JavaDoc was written
  • Commit messages comply with the guideline
  • Cleanup the code for review. See checklist

Related issues:

Copilot AI review requested due to automatic review settings June 29, 2026 16:59

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@MaxSiominDev MaxSiominDev force-pushed the migrate-to-github-hosted-runners branch from 113854e to 4edd733 Compare June 29, 2026 17:14
@MaxSiominDev MaxSiominDev merged commit 5782535 into master Jul 2, 2026
31 of 36 checks passed
@MaxSiominDev MaxSiominDev deleted the migrate-to-github-hosted-runners branch July 2, 2026 12:20
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