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.

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

Update jenkins-stability-checklist.md #2556

Merged
merged 4 commits into from
Mar 13, 2024
Merged

Conversation

ThomasLehnerSpryker
Copy link
Collaborator

removed non supported checklist boxes
fixed non working image reference

PR Description

TBD

Checklist

  • I agree with the Code Contribution License Agreement in CONTRIBUTING.md

removed non supported checklist boxes
fixed non working image reference
@ThomasLehnerSpryker ThomasLehnerSpryker added the TW review needed PR needs Technical writer's review label Mar 12, 2024
@lenadoc lenadoc added the updated PR contains meaningful changes to some document label Mar 12, 2024

- Configure a maximum of two executors.
- Set your PHP `memory_limit` value to be less than 2 GB.
- Implement batch processing (guidelines are here and here) in your importers and be mindful of the maximum memory consumption.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ThomasLehnerSpryker please add the links to guidelines

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

- Avoid spawning an excessive number of worker. There should be no more than two workers per queue.
- Profile your jobs locally to understand their normal memory demand, especially when interacting with data.
- In a standard-sized non-production environment, run lengthy imports and sync processes lasting more than 1-2 hours.
- Refer to troubleshooting instructions for further assistance.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What troubleshooting instructions do you mean here?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can be removed, the sentence before should read "do not run"


- Configure a maximum of two executors.
- Set your PHP `memory_limit` value to be less than 2 GB.
- Implement batch processing (guidelines are here and here) in your importers and be mindful of maximum memory consumption.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ThomasLehnerSpryker please add the links to guidelines

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@lenadoc lenadoc merged commit 8d8b100 into master Mar 13, 2024
9 checks passed
@lenadoc lenadoc deleted the ThomasLehnerSpryker-patch-10 branch March 13, 2024 20:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
TW review needed PR needs Technical writer's review updated PR contains meaningful changes to some document
Development

Successfully merging this pull request may close these issues.

2 participants