Skip to content

Conversation

@JohnVillalovos
Copy link
Member

@JohnVillalovos JohnVillalovos commented Jul 26, 2022

The functional tests have been erratic. Current theory is that we are
starting the tests before the GitLab container is fully up and
running.

  • Add checking of the Health Check[1] endpoints.
  • Add a 20 second delay after we believe it is up and running.
  • Increase timeout from 300 to 400 seconds

[1] https://docs.gitlab.com/ee/user/admin_area/monitoring/health_check.html

@JohnVillalovos JohnVillalovos marked this pull request as draft July 26, 2022 16:15
@JohnVillalovos JohnVillalovos force-pushed the jlvillal/fix_functional_ci branch 5 times, most recently from 0a2f2a7 to 564ab74 Compare July 26, 2022 16:43
@JohnVillalovos JohnVillalovos requested a review from nejch July 26, 2022 16:43
@JohnVillalovos JohnVillalovos marked this pull request as ready for review July 26, 2022 16:43
@JohnVillalovos JohnVillalovos changed the title wip: trying to make functional more reliable test: attempt to make functional test startup more reliable Jul 26, 2022
@JohnVillalovos JohnVillalovos force-pushed the jlvillal/fix_functional_ci branch from 564ab74 to c829b26 Compare July 26, 2022 16:48
@JohnVillalovos JohnVillalovos force-pushed the jlvillal/fix_functional_ci branch from c829b26 to d349e06 Compare July 26, 2022 23:30
@codecov-commenter
Copy link

codecov-commenter commented Jul 26, 2022

Codecov Report

Merging #2188 (08f4313) into main (8ba97aa) will increase coverage by 4.00%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #2188      +/-   ##
==========================================
+ Coverage   91.56%   95.56%   +4.00%     
==========================================
  Files          81       81              
  Lines        5344     5344              
==========================================
+ Hits         4893     5107     +214     
+ Misses        451      237     -214     
Flag Coverage Δ
api_func_v4 81.41% <ø> (?)
cli_func_v4 83.06% <ø> (-0.10%) ⬇️
unit 87.31% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
gitlab/utils.py 98.59% <0.00%> (+1.40%) ⬆️
gitlab/v4/objects/members.py 94.82% <0.00%> (+1.72%) ⬆️
gitlab/client.py 98.70% <0.00%> (+3.44%) ⬆️
gitlab/types.py 98.21% <0.00%> (+3.57%) ⬆️
gitlab/v4/objects/notes.py 94.28% <0.00%> (+3.80%) ⬆️
gitlab/v4/objects/groups.py 89.40% <0.00%> (+3.97%) ⬆️
gitlab/v4/objects/pages.py 100.00% <0.00%> (+5.00%) ⬆️
gitlab/v4/objects/events.py 98.73% <0.00%> (+5.06%) ⬆️
gitlab/mixins.py 92.30% <0.00%> (+5.49%) ⬆️
gitlab/v4/objects/environments.py 100.00% <0.00%> (+5.88%) ⬆️
... and 21 more

@JohnVillalovos JohnVillalovos force-pushed the jlvillal/fix_functional_ci branch from d349e06 to 08f4313 Compare July 29, 2022 04:45
@JohnVillalovos JohnVillalovos requested a review from nejch July 29, 2022 04:45
The functional tests have been erratic. Current theory is that we are
starting the tests before the GitLab container is fully up and
running.

  * Add checking of the Health Check[1] endpoints.
  * Add a 20 second delay after we believe it is up and running.
  * Increase timeout from 300 to 400 seconds

[1] https://docs.gitlab.com/ee/user/admin_area/monitoring/health_check.html
@JohnVillalovos JohnVillalovos force-pushed the jlvillal/fix_functional_ci branch from 08f4313 to 67508e8 Compare July 29, 2022 15:39
@nejch nejch merged commit 17414f7 into main Jul 29, 2022
@nejch nejch deleted the jlvillal/fix_functional_ci branch July 29, 2022 16:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants