Skip to content

Conversation

lunny
Copy link
Member

@lunny lunny commented Sep 20, 2023

Partially Fix #25041

This PR redefined the meaning of column is_active in table action_runner_token.
Before this PR, is_active means whether it has been used by any runner. If it's true, other runner cannot use it to register again.

In this PR, is_active means whether it's validated to be used to register runner. And if it's true, then it can be used to register runners until it become false. When creating a new is_active register token, any previous tokens will be set is_active to false.

@lunny lunny added the type/enhancement An improvement of existing functionality label Sep 20, 2023
@lunny lunny added this to the 1.21.0 milestone Sep 20, 2023
@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Sep 20, 2023
@github-actions github-actions bot added the modifies/api This PR adds API routes or modifies them label Sep 20, 2023
@lunny lunny added the topic/gitea-actions related to the actions of Gitea label Sep 20, 2023
@lunny lunny changed the title Redefine the meaning of Actions Registration Token Redefine the meaning of column is_active so make Actions Registration Token generation easier Sep 21, 2023
@lunny lunny changed the title Redefine the meaning of column is_active so make Actions Registration Token generation easier Redefine the meaning of column is_active to make Actions Registration Token generation easier Sep 21, 2023
@lunny lunny modified the milestones: 1.21.0, 1.22.0 Sep 21, 2023
Copy link
Member

@wolfogre wolfogre left a comment

Choose a reason for hiding this comment

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

LGTM. BTW, I don't think it can fix #25041, maybe it's a part.

@GiteaBot GiteaBot added lgtm/need 1 This PR needs approval from one additional maintainer to be merged. and removed lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. labels Sep 21, 2023
@GiteaBot GiteaBot added lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. and removed lgtm/need 1 This PR needs approval from one additional maintainer to be merged. labels Sep 27, 2023
@lunny lunny added the reviewed/wait-merge This pull request is part of the merge queue. It will be merged soon. label Sep 27, 2023
@lunny lunny added the backport/v1.21 This PR should be backported to Gitea 1.21 label Sep 27, 2023
@lunny lunny enabled auto-merge (squash) September 27, 2023 05:06
@lunny lunny merged commit 3b2da7e into go-gitea:main Sep 27, 2023
GiteaBot pushed a commit to GiteaBot/gitea that referenced this pull request Sep 27, 2023
… Token generation easier (go-gitea#27143)

Partially Fix go-gitea#25041

This PR redefined the meaning of column `is_active` in table
`action_runner_token`.
Before this PR, `is_active` means whether it has been used by any
runner. If it's true, other runner cannot use it to register again.

In this PR, `is_active` means whether it's validated to be used to
register runner. And if it's true, then it can be used to register
runners until it become false. When creating a new `is_active` register
token, any previous tokens will be set `is_active` to false.
@GiteaBot GiteaBot added the backport/done All backports for this PR have been created label Sep 27, 2023
@GiteaBot GiteaBot removed the reviewed/wait-merge This pull request is part of the merge queue. It will be merged soon. label Sep 27, 2023
lunny added a commit that referenced this pull request Sep 28, 2023
… Token generation easier (#27143) (#27304)

Backport #27143 by @lunny

Partially Fix #25041

This PR redefined the meaning of column `is_active` in table
`action_runner_token`.
Before this PR, `is_active` means whether it has been used by any
runner. If it's true, other runner cannot use it to register again.

In this PR, `is_active` means whether it's validated to be used to
register runner. And if it's true, then it can be used to register
runners until it become false. When creating a new `is_active` register
token, any previous tokens will be set `is_active` to false.

Co-authored-by: Lunny Xiao <xiaolunwen@gmail.com>
zjjhot added a commit to zjjhot/gitea that referenced this pull request Sep 28, 2023
* giteaoffical/main: (22 commits)
  Fix review UI (go-gitea#27322)
  Improve issue history dialog and make poster can delete their own history (go-gitea#27323)
  Improve branch list UI (go-gitea#27319)
  doctor: delete action entries without existing user (go-gitea#27292)
  fix orphan check for deleted branch (go-gitea#27310)
  make writing main test easier (go-gitea#27270)
  [skip ci] Updated translations via Crowdin
  Fix protected branch icon location (go-gitea#26576)
  move the `gitea admin` subcommands into separate files (go-gitea#27307)
  Fix chinese translation (go-gitea#27296)
  bump bleve (go-gitea#27300)
  Redefine the meaning of column is_active to make Actions Registration Token generation easier (go-gitea#27143)
  Use vitest globals (go-gitea#27102)
  Fix divider in subscription page (go-gitea#27298)
  Fix yaml test (go-gitea#27297)
  Enable production source maps for index.js, fix CSS sourcemaps (go-gitea#27291)
  Fix some animation bugs (go-gitea#27287)
  Add missed return to actions view fetch (go-gitea#27289)
  Fix more yaml lint errors (go-gitea#27284)
  Fix incorrect change from go-gitea#27231 (go-gitea#27275)
  ...
@go-gitea go-gitea locked as resolved and limited conversation to collaborators Dec 26, 2023
@lunny lunny deleted the lunny/redefine_actions_registration_token branch January 27, 2024 09:10
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
backport/done All backports for this PR have been created backport/v1.21 This PR should be backported to Gitea 1.21 lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. modifies/api This PR adds API routes or modifies them topic/gitea-actions related to the actions of Gitea type/enhancement An improvement of existing functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Proposal] Support reusable Registration Token of runners
4 participants