Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove temp testing workflow & route on-demand workflow to selfhosted runner #1282

Merged
merged 1 commit into from
Aug 29, 2023

Conversation

rzats
Copy link
Contributor

@rzats rzats commented Aug 25, 2023

Summary:

  • Removes the one-time "performance testing testing" workflow from Performance testing testing workflow #1253.
  • Modifies the performance testing workflow to route it to our self-hosted test runner, gh-runner-prod-cc-prv-01.

Prerequisites:

  • Unless it is a documentation hotfix it should be merged against the dev branch
  • Branch is up-to-date with the branch to be merged with, i.e. dev
  • Build is successful
  • Code is cleaned up and formatted

@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@rzats rzats requested a review from korlaxxalrok August 25, 2023 21:40
Copy link
Contributor

@korlaxxalrok korlaxxalrok left a comment

Choose a reason for hiding this comment

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

Yep, looks good!

(This may not work quite as expected until we modify/remove the WireGuard setup, but if it can be left as is will "just work" then even better, so I guess we try and see what happens)

@rzats rzats merged commit f7da659 into dev Aug 29, 2023
@rzats rzats deleted the rzatserkovnyi/self-hosted branch August 29, 2023 14:34
rzats added a commit that referenced this pull request Oct 12, 2023
rzats added a commit that referenced this pull request Oct 12, 2023
rzats added a commit that referenced this pull request Oct 24, 2023
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.

2 participants