- 
          
- 
                Notifications
    You must be signed in to change notification settings 
- Fork 6.2k
Support actions and reusable workflows from private repos #32562
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
Conversation
| 
 For changes in  | 
| Is this related to #24635 ? | 
| Any progress for it? | 
| Any progress there? | 
| Any movement here? This would really really help us in our Gitea Cloud account. | 
| 
 
 | 
01211b3    to
    82d368c      
    Compare
  
    203ad9c    to
    01035bd      
    Compare
  
    | 
Could it be a private org is broken? Or is this nightly act_runner from https://dl.gitea.com/ not up to date. | 
| 
 I can't reproduce this issue. Can you make test01/test-repo private and try again? | 
| well good point, missed my own repo visibility. | 
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me, we could later enhance troubleshooting
- by log more information around repository not found
- if act_runner did an authenticated pull
| act could check own repo visibility in it's payload when getting repository not found with token and log an actionable message | 
| <div class="inline field"> | ||
| <label>{{ctx.Locale.Tr "actions.actions"}}</label> | ||
| <div class="ui checkbox{{if $isActionsGlobalDisabled}} disabled{{end}}"{{if $isActionsGlobalDisabled}} data-tooltip-content="{{ctx.Locale.Tr "repo.unit_disabled"}}"{{end}}> | ||
| <input class="enable-system" name="enable_actions" type="checkbox" {{if $isActionsGlobalDisabled}}disabled{{end}} {{if $isActionsEnabled}}checked{{end}}> | 
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What is enable-system?
| {{if and .EnableActions (.Permission.CanRead ctx.Consts.RepoUnitTypeActions)}} | ||
| {{if .Repository.IsPrivate}} | 
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No indention?
| <div class="inline field"> | ||
| <label>{{ctx.Locale.Tr "actions.actions"}}</label> | ||
| <div class="ui checkbox{{if $isActionsGlobalDisabled}} disabled{{end}}"{{if $isActionsGlobalDisabled}} data-tooltip-content="{{ctx.Locale.Tr "repo.unit_disabled"}}"{{end}}> | ||
| <input class="enable-system" name="enable_actions" type="checkbox" {{if $isActionsEnabled}}checked{{end}}> | 
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Won't it cause bug since you remove this block? RepoSettingForm.EnableActions is always false now, then any edit on this page will disable the Actiosn unit?
RepoSettingForm.EnableActions  is left there and need to be removed.
| return perm, err | ||
| } | ||
| actionsCfg := repo.MustGetUnit(ctx, unit.TypeActions).ActionsConfig() | ||
| if !actionsCfg.IsCollaborativeOwner(taskRepo.OwnerID) || !taskRepo.IsPrivate { | 
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is it right if the owner's visibility is not public?
| I "fixed" these reviews in Remove unnecessary code #35761 | 
* giteaofficial/main: [skip ci] Updated translations via Crowdin Revert go-gitea#18491, fix oauth2 client link account (go-gitea#35745) Upgrade go mail to 0.7.2 (go-gitea#35748) [skip ci] Updated translations via Crowdin Support actions and reusable workflows from private repos (go-gitea#32562) Use git model to detect whether branch exist instead of gitrepo method (go-gitea#35459) Fix shutdown waitgroup panic (go-gitea#35676) Intorduce "config edit-ini" sub command to help maintaining INI config file (go-gitea#35735) Improve online runner check (go-gitea#35722) Make "update file" API can create a new file when SHA is not set (go-gitea#35738) Fix review request webhook bug (go-gitea#35339) (go-gitea#35723) Misc tool tweaks (go-gitea#35734) Update dependencies (go-gitea#35733) Make external iframe render work (go-gitea#35730) Remove mermaid margin workaround (go-gitea#35732) # Conflicts: # models/user/user.go


Resolve https://gitea.com/gitea/act_runner/issues/102
This PR allows administrators of a private repository to specify some collaborative owners. The repositories of collaborative owners will be allowed to access this repository's actions and workflows.
Settings for private repos:
This PR also moves "Enable Actions" setting to
Actions > Generalpage