chore(dev/ci): set expiry date for access token

[amimas]

As of GitLab v16.0, all access tokens must have an expiry date. Otherwise the access token will not be created. This wasn't being done in the script that sets up a GitLab instances with an access token so that the token can be used for all acceptance tests via API calls. This worked fine before but started failing recently when latest version of GitLab was being used in the CI because the latest version is 16 and up. As a result, all the PR validations are failing because the acceptance tests are failing with '401 Unauthorized' error.

[codecov]

Codecov Report

Merging #557 (09f9d26) into main (24bdac6) will decrease coverage by 15.92%.
The diff coverage is n/a.

Additional details and impacted files

@@             Coverage Diff             @@
##             main     #557       +/-   ##
===========================================
- Coverage   82.09%   66.17%   -15.92%     
===========================================
  Files          70       70               
  Lines        2658     2658               
===========================================
- Hits         2182     1759      -423     
- Misses        476      899      +423     

see 37 files with indirect coverage changes

[amimas]

Hi @gdubicki 👋,

I'm not sure why codecov is reporting a big decrease in test coverage. This PR makes no changes to the codes that it's reporting a coverage decrease. Not too familiar with codecov, but I think this shouldn't be a blocker for this PR.

Also, because of GitLab v16 requirements of bot users must have an expiry date, couple of existing acceptance tests are failing in this PR.

=========================== short test summary info ============================
ERROR tests/acceptance/standard/test_group_members_users.py::TestGroupMembersUsers::test__remove_user_keep_bots - gitlab.exceptions.GitlabCreateError: 400: expires_at is missing
ERROR tests/acceptance/standard/test_members_enforce.py::TestMembersEnforce::test__enforce_keep_bots - gitlab.exceptions.GitlabCreateError: 400: expires_at is missing
============== 102 passed, 2 errors, 6 rerun in 223.73s (0:03:43) ==============

I believe this should be fixed as part of #554.

So, I'm leaning towards merging this PR soon even with the reported failures. It will at least resolve the issue with acceptance tests not running in the CI and unblock #554 . Thoughts?

[gdubicki]

BIG thank you for resolving this, @amimas ! I agree, let’s merge this first and deal with other failing tests later.