Skip to content

Update github runners to oci gh arc runners#1326

Merged
javier-aliaga merged 11 commits into
dapr:masterfrom
koksay:update-github-runner
Apr 27, 2026
Merged

Update github runners to oci gh arc runners#1326
javier-aliaga merged 11 commits into
dapr:masterfrom
koksay:update-github-runner

Conversation

@koksay

@koksay koksay commented Apr 29, 2025

Copy link
Copy Markdown
Contributor

Description

CNCF has hosted ephemeral GitHub runners on Oracle that we want projects to use instead of the GitHub-hosted ones, which now incur a cost to use.

Please direct any questions to me, @jeefy, @krook, and @RobertKielty. (or #cncf-ci-infra channel on the CNCF Slack)

Signed-off-by: Koray Oksay <koray.oksay@gmail.com>
@koksay koksay requested review from a team as code owners April 29, 2025 08:27
@artur-ciocanu

Copy link
Copy Markdown
Contributor

@cicoyle could you please take a look, I am not sure how CNCF handles this and how Dapr Java SDK should be affected.

koksay and others added 2 commits April 29, 2025 23:06
Signed-off-by: Koray Oksay <koray.oksay@gmail.com>
cicoyle
cicoyle previously requested changes May 6, 2025

@cicoyle cicoyle left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Hello @koksay - thanks for the PR here. Mind providing a link to a discussion or where I may find more information about the reason for this change?

Was there a particular reason for the dapr/java-sdk being chosen for this change? From a dapr ecosystem perspective, we will want consistency across all SDKs.

@koksay

koksay commented May 6, 2025

Copy link
Copy Markdown
Contributor Author

Hi @cicoyle, the main reason is the cost of the runners (for some projects, it's runners on Equinix), and we're visiting all CNCF projects for this change. @jeefy can provide more information here, or you can also join the #cncf-ci-infra channel on the CNCF Slack and ask any questions you may have.

@cicoyle

cicoyle commented May 6, 2025

Copy link
Copy Markdown
Contributor

inix), and we're visiting all CNCF projects for this chang

Will join and ask for clarification in the slack channel and provide additional context here for future reference

@artur-ciocanu

Copy link
Copy Markdown
Contributor

@cicoyle is this still relevant? I am wondering if we could close this PR.

@artur-ciocanu

Copy link
Copy Markdown
Contributor

@cicoyle @salaboy could follow up with someone from CNCF and figure what can we do here and if we should do anything.

@koksay

koksay commented Oct 30, 2025

Copy link
Copy Markdown
Contributor Author

Hey @artur-ciocanu, sorry — I didn't follow up on this PR. It's still relevant, for sure. As I stated at the beginning, these runners are no longer free, and we need to move all of them to the CNCF-hosted runners. I think

@cicoyle mentioned on Slack that you prefer to align all SDKs; you may move the others as well. Here are the runner types:

# amd64
oracle-vm-2cpu-8gb-x86-64
oracle-vm-4cpu-16gb-x86-64
oracle-vm-8cpu-32gb-x86-64
oracle-vm-16cpu-64gb-x86-64
oracle-vm-24cpu-96gb-x86-64
oracle-vm-32cpu-128gb-x86-64

# arm64
oracle-vm-2cpu-8gb-arm64
oracle-vm-4cpu-16gb-arm64
oracle-vm-8cpu-32gb-arm64
oracle-vm-16cpu-64gb-arm64
oracle-vm-24cpu-96gb-arm64
oracle-vm-32cpu-128gb-arm64

cc: @jeefy

@artur-ciocanu

Copy link
Copy Markdown
Contributor

@cicoyle and @salaboy are you OK with this PR, should we merge it?

@salaboy

salaboy commented Dec 8, 2025 via email

Copy link
Copy Markdown
Collaborator

@artur-ciocanu

Copy link
Copy Markdown
Contributor

@cicoyle do you see any issues in approving this PR, should I try to merge

@koksay

koksay commented Feb 26, 2026

Copy link
Copy Markdown
Contributor Author

@artur-ciocanu @cicoyle Sorry, I forgot to follow up on this PR. Would you please take a look at it?

@cicoyle

cicoyle commented Feb 26, 2026

Copy link
Copy Markdown
Contributor

@cicoyle do you see any issues in approving this PR, should I try to merge

I retriggered CI - I see no issue merging as long as CI is 🟢 👍🏻

@javier-aliaga javier-aliaga dismissed cicoyle’s stale review April 27, 2026 07:53

Stale review that has been resolved in the comments

javier-aliaga
javier-aliaga previously approved these changes Apr 27, 2026
Signed-off-by: Javier Aliaga <javier@diagrid.io>
@javier-aliaga javier-aliaga merged commit 9135b88 into dapr:master Apr 27, 2026
11 checks passed
@codecov

codecov Bot commented Apr 27, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 76.76%. Comparing base (4396215) to head (cdd4674).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff            @@
##             master    #1326   +/-   ##
=========================================
  Coverage     76.76%   76.76%           
  Complexity     2260     2260           
=========================================
  Files           241      241           
  Lines          7066     7066           
  Branches        740      740           
=========================================
  Hits           5424     5424           
  Misses         1279     1279           
  Partials        363      363           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

siri-varma pushed a commit to siri-varma/java-sdk that referenced this pull request May 10, 2026
* Update github runners to oci gh arc runners

Signed-off-by: Koray Oksay <koray.oksay@gmail.com>

* correct the runner size

Signed-off-by: Koray Oksay <koray.oksay@gmail.com>

* Move to the VM instead of container runner

Signed-off-by: Koray Oksay <koray.oksay@gmail.com>

* chore: Increase timeout for Javadocs validation job to 60 minutes

Signed-off-by: Javier Aliaga <javier@diagrid.io>

---------

Signed-off-by: Koray Oksay <koray.oksay@gmail.com>
Signed-off-by: Javier Aliaga <javier@diagrid.io>
Co-authored-by: artur-ciocanu <artur.ciocanu@gmail.com>
Co-authored-by: salaboy <Salaboy@gmail.com>
Co-authored-by: Cassie Coyle <cassie.i.coyle@gmail.com>
Co-authored-by: Javier Aliaga <javier@diagrid.io>
siri-varma pushed a commit to siri-varma/java-sdk that referenced this pull request May 10, 2026
* Update github runners to oci gh arc runners

Signed-off-by: Koray Oksay <koray.oksay@gmail.com>

* correct the runner size

Signed-off-by: Koray Oksay <koray.oksay@gmail.com>

* Move to the VM instead of container runner

Signed-off-by: Koray Oksay <koray.oksay@gmail.com>

* chore: Increase timeout for Javadocs validation job to 60 minutes

Signed-off-by: Javier Aliaga <javier@diagrid.io>

---------

Signed-off-by: Koray Oksay <koray.oksay@gmail.com>
Signed-off-by: Javier Aliaga <javier@diagrid.io>
Co-authored-by: artur-ciocanu <artur.ciocanu@gmail.com>
Co-authored-by: salaboy <Salaboy@gmail.com>
Co-authored-by: Cassie Coyle <cassie.i.coyle@gmail.com>
Co-authored-by: Javier Aliaga <javier@diagrid.io>
Signed-off-by: Siri Varma Vegiraju <s_vegiraju@apple.com>
@marcduiker

Copy link
Copy Markdown
Contributor

@holopin-bot @koksay sdk-badge Thank you! Here's a digital badge as a small token of appreciation.

@holopin-bot

holopin-bot Bot commented Jun 12, 2026

Copy link
Copy Markdown

Congratulations @koksay, the maintainer of this repository has issued you a badge! Here it is: https://holopin.io/claim/cmqbfctdq0011ks0be8jvb8f0

This badge can only be claimed by you, so make sure that your GitHub account is linked to your Holopin account. You can manage those preferences here: https://holopin.io/account.
Or if you're new to Holopin, you can simply sign up with GitHub, which will do the trick!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants