Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add GANTRY_UPDATE_NUM_WORKERS and other fixes #29

Merged
merged 19 commits into from
Feb 18, 2024
Merged

Add GANTRY_UPDATE_NUM_WORKERS and other fixes #29

merged 19 commits into from
Feb 18, 2024

Conversation

shizunge
Copy link
Owner

  • Added GANTRY_UPDATE_NUM_WORKERS to run updates in parallel.
  • Skip updating when failing to login to registries.
  • Fixed obtaining the current container name.
  • Fixed obtaining the service mode and the number of running tasks.
  • Enabled running tests in parallel, e.g. bash shellspec --jobs 50.
  • Added post build tests in the on-push workflow.

closes #28
closes #26

@shizunge
[auth] Do not run Gantry if log into registry failed.
dcf6892
@shizunge
[gantry] When calling wait_service_state, do not add --complete
23d6723 
We do not add --detach to docker_global_job should, thus that function should check converge.
@shizunge
[tests] Use "run" instead of "call" in tests.
814cc45 
"run" starts a subroutine. So each test will have their own independent env.
@shizunge
[tests] add test_login_file_not_exist
9bf488b
@shizunge
[workflows] add post build tests.
80b46bf
@shizunge
[gantry] fix the problem about incorrectly obtaining the current cont…
87ae432 
…ainer name when running outside the container.

This blocks run tests in parallel.
@shizunge
[lib] improve logging. fix test checks using env variables in subproc…
3c5715e 
…ess.
@shizunge
[tests] move tests, based on option category in readme.md
8f24bbf
@shizunge
add lock to static variables.
91c67d8
@shizunge
add gantry_service_parallel_spec.sh
8e41c2c
@shizunge
run updates in parallel
746e490
@shizunge
[tests] check exact number of services updated.
aece1ea
@shizunge
[gantry] update logging & reporting
21878c6
@shizunge
[tests] be able to handle more than 10 test services.
0bd5b6f
@shizunge
[gantry] fix getting number of running tasks and getting mode.
666a4e8
@shizunge
[tests] add test_SERVICES_EXCLUDED_multiple_services
06b2d8e
@shizunge
[shellcheck] disable SC2317 for tests
44febdc 
shellcheck incorrectly fired this checks on only one test: test_no_running_tasks_replicated
@shizunge
Copy link
Owner Author 

  • Added GANTRY_UPDATE_NUM_WORKERS to run updates in parallel.

  • Skip updating when failing to login to registries.

  • Fixed obtaining the current container name.

  • Fixed obtaining the service mode and the number of running tasks.

  • Enabled running tests in parallel, e.g. bash shellspec --jobs 50.

  • Added post build tests in the on-push workflow.

@shizunge shizunge merged commit 24fe8dd into main Feb 18, 2024
29 checks passed
@shizunge shizunge deleted the parallel branch February 22, 2024 03:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Updates of a multi-service stack are super slow Shell vs Normal Programming Language
1 participant
@shizunge