Skip to content
This repository was archived by the owner on Jun 6, 2024. It is now read-only.

save job specific token in sperate namespace #5292

Merged
merged 9 commits into from
Feb 7, 2021
Merged

save job specific token in sperate namespace #5292

merged 9 commits into from
Feb 7, 2021

Conversation

suiguoxin
Copy link
Member

  • save job specific token in sperate namespace
  • revoke it in purge() function by check job state from DB

@suiguoxin suiguoxin requested a review from hzy46 February 7, 2021 04:24
@coveralls
Copy link

coveralls commented Feb 7, 2021 •
edited
Loading

Coverage Status

Coverage increased (+0.3%) to 34.211% when pulling 113d4c2 on suiguoxin:job-token-pr into be386f0 on microsoft:master.

@suiguoxin suiguoxin mentioned this pull request Feb 7, 2021
Closed
6 tasks
hzy46
hzy46 reviewed Feb 7, 2021
View reviewed changes
hzy46
hzy46 reviewed Feb 7, 2021
View reviewed changes
@suiguoxin suiguoxin requested a review from yiyione February 7, 2021 07:01
@suiguoxin suiguoxin requested a review from hzy46 February 7, 2021 07:04
hzy46
hzy46 reviewed Feb 7, 2021
View reviewed changes
hzy46
hzy46 reviewed Feb 7, 2021
View reviewed changes
@suiguoxin suiguoxin requested a review from hzy46 February 7, 2021 07:41
@suiguoxin suiguoxin merged commit d97c1c4 into microsoft:master Feb 7, 2021
@suiguoxin suiguoxin deleted the job-token-pr branch February 7, 2021 08:23
@suiguoxin suiguoxin mentioned this pull request Feb 8, 2021
Closed
55 tasks
@hzy46 hzy46 mentioned this pull request Feb 10, 2021
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@hzy46 hzy46 hzy46 approved these changes

@yiyione yiyione yiyione approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@suiguoxin @coveralls @hzy46 @yiyione