Skip to content

feat: add github_enterprise_actions_hosted_runner#3017

Open
nico34638 wants to merge 3 commits intointegrations:mainfrom
nico34638:main
Open

feat: add github_enterprise_actions_hosted_runner#3017
nico34638 wants to merge 3 commits intointegrations:mainfrom
nico34638:main

Conversation

@nico34638
Copy link

@nico34638 nico34638 commented Dec 22, 2025 •
edited
Loading

Resolves #3016

Before the change?

  • No possibility to create hosted runner at enterprise level

After the change?

  • It's possible

Pull request checklist

  • Schema migrations have been created if needed (example)
  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been reviewed and added / updated if needed (for bug fixes / features)

Does this introduce a breaking change?

Please see our docs on breaking changes to help!

  • Yes
  • No

@github-actions
Copy link

github-actions bot commented Dec 22, 2025

👋 Hi! Thank you for this contribution! Just to let you know, our GitHub SDK team does a round of issue and PR reviews twice a week, every Monday and Friday! We have a process in place for prioritizing and responding to your input. Because you are a part of this community please feel free to comment, add to, or pick up any issues/PRs that are labeled with Status: Up for grabs. You & others like you are the reason all of this works! So thank you & happy coding! 🚀

@deiga
Copy link
Collaborator

deiga commented Dec 24, 2025

Hey @nico34638 👋

Thank you very much for your contribution! We're currently working on a major release of the provider and that will possibly delay getting this PR merged.

While that's happening, could I bother you for a few changes?

  • We're moving towards using Context-aware provider functions (ReadContext, CreateContext, etc.). Could you refactor your PR to use those? :)
  • Another major upcoming change will be generating docs based on Schema contents. Could you ensure that all levels of the Schema have Description fields which contain all necessary information for the docs?

Support for Hosted runners for enterprises was added in v70 of go-github. Our upcoming major release will upgrade to v77. Could you rebase your branch on top of #2891 and use the SDK methods instead of manual HTTP requests? :)

@nico34638
Copy link
Author

nico34638 commented Dec 25, 2025

Okay, I made the changes you requested. I rebased on your branch with go github/v77 and changed the description fields. Is that okay with you? I tested it, and with the changes, it still works.

@deiga
Copy link
Collaborator

deiga commented Jan 9, 2026

Hey @nico34638
Could you rebase your PR onto main now, we've upgraded the go-github version? 🙏

@nico34638
Copy link
Author

nico34638 commented Jan 10, 2026

Hey @deiga, It's done. Do you have any idea when the provider's next release will be?

deiga
deiga suggested changes Jan 10, 2026
View reviewed changes
deiga
deiga suggested changes Jan 11, 2026
View reviewed changes
Copy link
Collaborator

@deiga deiga left a comment

Choose a reason for hiding this comment

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

Please ensure the test structure follows the current pattern

deiga
deiga suggested changes Jan 13, 2026
View reviewed changes
@nico34638 nico34638 force-pushed the main branch 3 times, most recently from 36951d4 to 9ac9b53 Compare January 17, 2026 14:27
@nico34638
Copy link
Author

nico34638 commented Jan 17, 2026

I have corrected the comments

@nico34638
Copy link
Author

nico34638 commented Jan 21, 2026

Hi @deiga do you any news ?

@deiga deiga added this to the v6.11.0 Release milestone Jan 21, 2026
@deiga deiga added Type: Feature New feature or request New resource labels Jan 21, 2026
stevehipwell
stevehipwell requested changes Feb 3, 2026
View reviewed changes
Copy link
Collaborator

@stevehipwell stevehipwell left a comment

Choose a reason for hiding this comment

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

Thanks for the PR @nico34638, I've added some initial review comments.

website/docs/r/enterprise_actions_hosted_runner.html.markdown
* Runner creation and updates may take several minutes as GitHub provisions the infrastructure.
* Static public IPs are subject to account limits. Check your enterprise's limits before enabling.

## Getting Available Images and Sizes
Copy link
Collaborator

@stevehipwell stevehipwell Feb 3, 2026

Choose a reason for hiding this comment

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

Why not add a data source for this?

Copy link
Author

@nico34638 nico34638 Feb 4, 2026

Choose a reason for hiding this comment

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

I can do this in the same PR ?

@nico34638
Copy link
Author

nico34638 commented Feb 4, 2026

Following the last review, I'm taking this opportunity to add two data sources for hosted enterprise runners:
one to list them and one to get the details.

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

Reviewers

@stevehipwell stevehipwell Awaiting requested review from stevehipwell

1 more reviewer

@deiga deiga deiga approved these changes

Reviewers whose approvals may not affect merge requirements

Requested changes must be addressed to merge this pull request.

Assignees

No one assigned

Labels

New resource Type: Feature New feature or request

Projects

None yet

Milestone

v6.12.0 Release

Development

Successfully merging this pull request may close these issues.

[FEAT]: enterprise_actions_hosted_runner

3 participants

@nico34638 @deiga @stevehipwell