Skip to content

Optimizie CI Runtime by Parallelizing Slow Tests#2087

Merged
Ilyesbdlala merged 5 commits intomainfrom
maintenance/ci-optimization
Nov 7, 2023
Merged

Optimizie CI Runtime by Parallelizing Slow Tests#2087
Ilyesbdlala merged 5 commits intomainfrom
maintenance/ci-optimization

Conversation

@Ilyesbdlala
Copy link
Copy Markdown
Member

@Ilyesbdlala Ilyesbdlala commented Nov 7, 2023

Description

Checklist

  • Test your changes as thoroughly as possible before you commit them. Preferably, automate your test by unit/integration tests.
  • Make sure that all your commits are signed-off and that you are added to the Contributors file.
  • Make sure that all CI finish successfully.
  • Optional (but appreciated): Make sure that all commits are Verified.

@Ilyesbdlala
Allow concurrent running of zap integration tests
df6dd98 
Signed-off-by: Ilyes Ben Dlala <ilyes.bendlala@iteratec.com>
@Ilyesbdlala
Allow concurrent running of trivy integration tests
2f8e6c2 
Signed-off-by: Ilyes Ben Dlala <ilyes.bendlala@iteratec.com>
@Ilyesbdlala Ilyesbdlala self-assigned this Nov 7, 2023
@netlify
Copy link
Copy Markdown

netlify bot commented Nov 7, 2023 •
edited
Loading

✅ Deploy Preview for docs-securecodebox canceled.

Name Link
🔨 Latest commit c037b6e
🔍 Latest deploy log https://app.netlify.com/sites/docs-securecodebox/deploys/654a53b728b56300085c1f35

@github-actions
Copy link
Copy Markdown

github-actions bot commented Nov 7, 2023 •
edited
Loading

🦙 MegaLinter status: ⚠️ WARNING

Descriptor Linter Files Fixed Errors Elapsed time
⚠️ ACTION actionlint 1 4 0.45s
⚠️ JAVASCRIPT eslint 3 1 0.9s
⚠️ YAML prettier 1 1 0.82s

See detailed report in MegaLinter reports
Set VALIDATE_ALL_CODEBASE: true in mega-linter.yml to validate all sources, not only the diff

MegaLinter is graciously provided by OX Security

@Ilyesbdlala
Include zap-advanced in the normal CI instead of nightly
8c7347c 
the duration is nolonger too long to not include in the normal CI. It should not increase the general duration of the CI

Signed-off-by: Ilyes Ben Dlala <ilyes.bendlala@iteratec.com>
@Ilyesbdlala
Allow concurrent running of zap-advanced integration tests
eeee689 
Signed-off-by: Ilyes Ben Dlala <ilyes.bendlala@iteratec.com>
@Ilyesbdlala
Fixed the expected number of results for zap test.
c037b6e 
The concurency decreases the speed of the spider and as a result in the limited time of the test, fewer results can be found

Signed-off-by: Ilyes Ben Dlala <ilyes.bendlala@iteratec.com>
@Ilyesbdlala Ilyesbdlala changed the title Maintenance/ci optimization ci optimization Nov 7, 2023
@J12934 J12934 changed the title ci optimization Optimizie CI Runtime by Parallelizing Slow Tests Nov 7, 2023
@Ilyesbdlala Ilyesbdlala merged commit ac4f4be into main Nov 7, 2023
@Ilyesbdlala Ilyesbdlala deleted the maintenance/ci-optimization branch November 7, 2023 16:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@J12934 J12934 J12934 approved these changes

Assignees

@Ilyesbdlala Ilyesbdlala

Labels

maintenance

Projects

secureCodeBox
Archived in project

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@Ilyesbdlala @J12934