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

Updating the runner of purge azure test resources workflow #1537

Merged
merged 1 commit into from
May 6, 2024

Conversation

ytimocin
Copy link
Contributor

@ytimocin ytimocin commented May 2, 2024

Updating the runner of purge azure test resources workflow

Signed-off-by: ytimocin <ytimocin@microsoft.com>
@ytimocin ytimocin force-pushed the ytimocin/updatePurgeRunner branch from eef0f1f to 0a732f6 Compare May 6, 2024 16:18
Copy link
Contributor

@kachawla kachawla left a comment

Choose a reason for hiding this comment

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

@ytimocin just curious, how did you catch it? Do we have a documentation on which runner to use?

@ytimocin
Copy link
Contributor Author

ytimocin commented May 6, 2024

@ytimocin just curious, how did you catch it? Do we have a documentation on which runner to use?

I was checking the actions of this repo and saw that this job wasn't running. It gets stuck with the following message: Waiting for a runner to pick up this job....

image

I also saw that we have these runners: https://github.com/radius-project/samples/actions/runners.

image

We don't have [self-hosted, 1ES.Pool=1ES-Radius] this runner that this job is looking for.

@ytimocin ytimocin merged commit bc43e42 into v0.33 May 6, 2024
8 checks passed
@ytimocin ytimocin deleted the ytimocin/updatePurgeRunner branch May 6, 2024 20:40
sk593 pushed a commit that referenced this pull request May 23, 2024
Signed-off-by: ytimocin <ytimocin@microsoft.com>
Signed-off-by: sk593 <shruthikumar@microsoft.com>
sk593 pushed a commit that referenced this pull request May 23, 2024
Signed-off-by: ytimocin <ytimocin@microsoft.com>
Signed-off-by: sk593 <shruthikumar@microsoft.com>
sk593 added a commit that referenced this pull request May 23, 2024
* update gh to ado sync workflow to use service principals (#1363)

* Make env configurable (#1046)

Signed-off-by: Reshma Abdul Rahim <reshmarahim.abdul@microsoft.com>

* Adding test AKS workflow to v0.32 (#1052)

* Run functional tests every 2 hours and add another workflow that runs… (#1020)

* Run functional tests every 2 hours and add another workflow that runs the tests on AKS instead of k3d

Signed-off-by: ytimocin <ytimocin@microsoft.com>

* Triggering workflow

Signed-off-by: ytimocin <ytimocin@microsoft.com>

---------

Signed-off-by: ytimocin <ytimocin@microsoft.com>

* Removing the run of the AKS workflow on PRs (#1051)

Signed-off-by: ytimocin <ytimocin@microsoft.com>

* Update the timeout (30s) for Playwright

Signed-off-by: ytimocin <ytimocin@microsoft.com>

---------

Signed-off-by: ytimocin <ytimocin@microsoft.com>

* Use retry and update eshop playwright tests to wait for the catalog to appear (#1213)

Signed-off-by: ytimocin <ytimocin@microsoft.com>

* update gh to ado sync workflow to use service principals

Signed-off-by: Will Tsai <28876888+willtsai@users.noreply.github.com>

---------

Signed-off-by: Reshma Abdul Rahim <reshmarahim.abdul@microsoft.com>
Signed-off-by: ytimocin <ytimocin@microsoft.com>
Signed-off-by: Will Tsai <28876888+willtsai@users.noreply.github.com>
Signed-off-by: Will <28876888+willtsai@users.noreply.github.com>
Co-authored-by: Reshma Abdul Rahim <61033581+Reshrahim@users.noreply.github.com>
Co-authored-by: Yetkin Timocin <ytimocin@microsoft.com>
Signed-off-by: sk593 <shruthikumar@microsoft.com>

* Workflow and playwright test updates (#1360)

Signed-off-by: ytimocin <ytimocin@microsoft.com>
Signed-off-by: sk593 <shruthikumar@microsoft.com>

* Changing the place of page.reload in the tests (#1492)

Signed-off-by: ytimocin <ytimocin@microsoft.com>
Signed-off-by: sk593 <shruthikumar@microsoft.com>

* Adding a job that will purge AWS EKS clusters every 6 hours (#1462)

Signed-off-by: ytimocin <ytimocin@microsoft.com>
Signed-off-by: sk593 <shruthikumar@microsoft.com>

* Update all deps of all apps (#1536)

Signed-off-by: ytimocin <ytimocin@microsoft.com>
Signed-off-by: sk593 <shruthikumar@microsoft.com>

* Updating the runner of purge azure test resources workflow (#1537)

Signed-off-by: ytimocin <ytimocin@microsoft.com>
Signed-off-by: sk593 <shruthikumar@microsoft.com>

* Update purget test resources to fix the workflow failures (#1552)

Signed-off-by: ytimocin <ytimocin@microsoft.com>
Signed-off-by: sk593 <shruthikumar@microsoft.com>

* Fixing the eShop catalog load issue for the UI tests (#1548)

Signed-off-by: ytimocin <ytimocin@microsoft.com>
Signed-off-by: sk593 <shruthikumar@microsoft.com>

---------

Signed-off-by: Reshma Abdul Rahim <reshmarahim.abdul@microsoft.com>
Signed-off-by: ytimocin <ytimocin@microsoft.com>
Signed-off-by: Will Tsai <28876888+willtsai@users.noreply.github.com>
Signed-off-by: Will <28876888+willtsai@users.noreply.github.com>
Signed-off-by: sk593 <shruthikumar@microsoft.com>
Co-authored-by: Will <28876888+willtsai@users.noreply.github.com>
Co-authored-by: Reshma Abdul Rahim <61033581+Reshrahim@users.noreply.github.com>
Co-authored-by: Yetkin Timocin <ytimocin@microsoft.com>
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