Use `AgentTokensFinder` to find single agent tokens

changed target branch from master to pam/agent-tokens-finder

added backend database databasereview pending labels

	1 Warning
	This merge request does not refer to an existing milestone.

	1 Message
	This merge request adds or changes files that require a review from the Database team.

This merge request requires a database review. To make sure these changes are reviewed, take the following steps:

Ensure the merge request has database and databasereview pending labels. If the merge request modifies database files, Danger will do this for you.
Prepare your MR for database review according to the docs.
Assign and mention the database reviewer suggested by Reviewer Roulette.

The following files require a review from the Database team:

app/finders/clusters/agent_tokens_finder.rb

Reviewer roulette

Changes that require review have been detected!

Please refer to the table below for assigning reviewers and maintainers suggested by Danger in the specified category:

Category	Reviewer	Maintainer
backend	Dominic Bauer (`@bauerdominic`) (UTC+1)	Vitali Tatarintev (`@ck3g`) (UTC+1)
database	Eulyeon K. (`@euko`) (UTC+9)	Adam Hegyi (`@ahegyi`) (UTC+1)

To spread load more evenly across eligible reviewers, Danger has picked a candidate for each review slot, based on their timezone. Feel free to override these selections if you think someone else would be better-suited or use the GitLab Review Workload Dashboard to find other available reviewers.

To read more on how to use the reviewer roulette, please take a look at the Engineering workflow and code review guidelines. Please consider assigning a reviewer or maintainer who is a domain expert in the area of the merge request.

Once you've decided who will review this merge request, assign them as a reviewer! Danger does not automatically notify them for you.

If needed, you can retry the danger-review job that generated this comment.

Generated by Danger

added devopsconfigure [DEPRECATED] groupconfigure [DEPRECATED] maintenancerefactor sectionops typemaintenance labels and removed database databasereview pending labels

Allure report

allure-report-publisher generated test report!

e2e-review-qa: test report for e212a783

expand test summary

+-----------------------------------------------------------------------------------------+
|                                     suites summary                                      |
+------------------------------------+--------+--------+---------+-------+-------+--------+
|                                    | passed | failed | skipped | flaky | total | result |
+------------------------------------+--------+--------+---------+-------+-------+--------+
| Create                             | 28     | 0      | 1       | 0     | 29    | ✅     |
| Plan                               | 49     | 0      | 1       | 0     | 50    | ✅     |
| Manage                             | 39     | 0      | 4       | 3     | 43    | ❗     |
| Configure                          | 0      | 0      | 3       | 0     | 3     | ➖     |
| Verify                             | 12     | 0      | 1       | 0     | 13    | ✅     |
| Govern                             | 10     | 0      | 5       | 1     | 15    | ❗     |
| Feature flag handler sanity checks | 9      | 0      | 0       | 0     | 9     | ✅     |
| Package                            | 0      | 0      | 1       | 0     | 1     | ➖     |
| Version sanity check               | 0      | 0      | 1       | 0     | 1     | ➖     |
+------------------------------------+--------+--------+---------+-------+-------+--------+
| Total                              | 147    | 0      | 17      | 4     | 164   | ❗     |
+------------------------------------+--------+--------+---------+-------+-------+--------+

mentioned in merge request !103300 (merged)

changed the description

mentioned in merge request !103170 (merged)

added 4 commits

f6261f2f...2cec222d - 2 commits from branch pam/agent-tokens-finder
3794f664 - Use AgentTokensFinder in API endpoint for fetching a single token
bc9aa460 - Add test for revoked agent_token

Compare with previous version

added database databasereview pending labels

changed the description

resolved all threads

marked this merge request as ready

Hi @Alexand @timofurrer could you also give this a please.

requested review from @Alexand and @timofurrer

@partiaga I really appreciate the great MR descriptions in all of the MRs in this series. Thank you!

approved this merge request

@timofurrer, thanks for approving this merge request.

This is the first time the merge request is approved. To ensure full test coverage, a new pipeline will be started shortly.

For more info, please refer to the following links:

added 1 deleted label

removed review request for @timofurrer

resolved all threads

added 1 commit

03476a44 - Test status of the agent token

Compare with previous version

deleted the pam/agent-tokens-finder branch. This merge request now targets the master branch

added 1113 commits

03476a44...6892a52f - 1110 commits from branch master
1013234c - Use AgentTokensFinder in API endpoint for fetching a single token
adb81e1f - Add test for revoked agent_token
b2ec84a9 - Test status of the agent token

Compare with previous version

Hi @Alexand did you have any comments about this MR? Will assign to the maintainer if not

Btw, the review-app deploy step is running into timeouts and currently being investigated, thus the failed pipeline. The MR code still be ok to review

mentioned in merge request !103897 (merged)

Thanks, @partiaga.

I've only had 2 very minor suggestions. Since those are really trivial to fix, I'll approve this so you don't have to wait for another review cycle from me. So feel free to pass forward to the database and backend maintainers after you address those 2 threads. But also feel free to send it back to me if you'd like.