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

Merge cgroups and environment variables for the resource allocation table. #56

Merged
merged 7 commits into from
Jan 13, 2022

Conversation

SgtCoin
Copy link
Contributor

@SgtCoin SgtCoin commented Jan 10, 2022

Adding in a few brief lines about cgroups in the storage provider (sealing workers) section. Added all the environment variables for overriding the resource allocation table in lotus configuration. I am not sure what all we need for cgroups as its a linux feature and not specifically a lotus feature.

@netlify
Copy link

netlify bot commented Jan 10, 2022

❌ Deploy Preview for blissful-liskov-5fa2cf failed.

🔨 Explore the source changes: 79fa10b

🔍 Inspect the deploy log: https://app.netlify.com/sites/blissful-liskov-5fa2cf/deploys/61e04a3b782b5b0007063039

@SgtCoin SgtCoin linked an issue Jan 10, 2022 that may be closed by this pull request
Comment on lines 239 to 247
* 32G environment:
```
AP_32G_BASE_MIN_MEMORY=1073741824
AP_32G_GPU_UTILIZATION=0
AP_32G_MAX_MEMORY=4294967296
AP_32G_MAX_PARALLELISM=1
AP_32G_MAX_PARALLELISM_GPU=0
AP_32G_MIN_MEMORY=4294967296
C1_32G_BASE_MIN_MEMORY=1073741824
Copy link
Contributor

Choose a reason for hiding this comment

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

These things are super long. I'm gonna try and add them into a <details> tag.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I wasnt sure of the best way to format it, but a drop down is a great idea.

Copy link
Contributor

Choose a reason for hiding this comment

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

Adds a <details> shortcode: #61

@johnnymatthews
Copy link
Contributor

Aaaaand the build is failing because it can't find the shortcode. @SgtCoin can you merge main into this branch please? Don't think I can do it since it's your PR. That should sort this error.

@johnnymatthews johnnymatthews merged commit 34d41a7 into filecoin-project:main Jan 13, 2022
johnnymatthews added a commit that referenced this pull request Jan 24, 2022
* Change table (#44)

* Update multisig.md

Add in description for cancel a multisig transaction and inspect msig params with lotus-shed.

* Update multisig.md

format updates to cancel and inspect processes

* Update mining-architectures.md

* Update mining-architectures.md

* Formatting fixes.

* Adds closing codeblock.

Co-authored-by: Johnny <9611008+johnnymatthews@users.noreply.github.com>

* Moves back to original theme. (#39)

* Moves back to original theme.

* Adds proper homescreen.

* Temp config change for Netlify preview.

* Adds Lotus logo as SVG to topbar nav.

* Sorting layout for homepage.

* Adds placeholder text to homepage.

* Completes homepage copy and layout.

* Attempts to fix logo SVG.

* Adds link to install from homepage.

* Adds getting started section.

* Adds basic progress bar movement.

* Adds progress bar for scrolling.

* Adds plaintext option for code blocks.

* Adds plaintext to code blocks.

* Links to new 'What is Lotus' page.

* Changes identifer to make title easier to read.

* Adds placeholders for getting started content.

* Adds styling for progressbar in dark mode.

* Orders all pages. Again.

* Changes unfinished posts to draft.

* Adds draft warning to content.

* Fixing broken external links.

* Edits external links.

* Hides 'Edit' page for now.

* Further external link fixes.

* Fix/minor style edits (#45)

* Updates GitHub link.

* Fixes external links.

* Adds ZH version dropdown.

* Adds ZH language to dropdown.

* Cleans up ZH menu.

* Corrects base URL for search.

* Fix typo (#47)

* Adds minor error fixes from filecoin PR. (#50)

* Adds minor error fixes from filecoin PR.

* Fixes mutliple formatting errors.

* add partial retrieval CLI and update client CLI (#54)

* Updates Mermaid package. (#55)

* Update/default configs (#58)

* Adds default Lotus config.

* Adds default SP config.

* Adds a details shortcode. (#61)

* typofix & bullet addition (#46)

* Merge cgroups and environment variables for the resource allocation table. (#56)

* Update seal-workers.md

* Update configuration.md

* Update seal-workers.md

* Apply suggestions from code review

* Adds detail shortcode blocks.

* Adds empty line after paragraph and before title.

Co-authored-by: Johnny <9611008+johnnymatthews@users.noreply.github.com>

* Updates markdown-it to latest version dev. (#62)

* Adds warning callout for config updates. (#59)

* Adds warning callout for config updates.

* Adds config upgrade warning for miner and client.

* Storage provider updates and copy edits (#49)

* Remove out of date info on control addresses

Referenced feature is in current release.

* Clarify resources re worker windows

This actually resolves #1021 in the old filecoin-docs (https://github.com/filecoin-project/filecoin-docs/issues/1021). The issue title now refers to clarifying what happens in worker windows, but the original request was just to make this part of the docs make sense as a sentence, and the table below gives enough detail for the reader to calculate what could be done with a given set of resources, which was the other part of the question in original issue #1021.

* Updated Lotus worker co-location section

Modified co-location info for multi GPU setups since support was added to last RC and appears to be moving forward. Also added use case (which I can confirm is feasible) for co-location related to parallel GPU task processing.

* Reorganized storage groups section

I found the new documentation on storage groups a bit confusing so I thought I'd attempt to help clarify it a bit. Also corrected some errors/typos in config file and storage attach command syntax. One thing that's still not clear (to me) from this documentation is what the use case would be for having more than one value in Groups for any given sectorstore file. It also might be helpful to suggest how storage groups could work in an environment where some storage locations are shared and others are local, as well as considerations when workers have multiple (local) sector stores, but I haven't spent enough time configuring my own storage groups to be able to add that at this point.

I moved the storage groups documentation under a new heading at the bottom for multi-worker setups, in hopes that eventually there might be more robust official documentation on best practices for multiple workers since that's what most people seem to doing.

* Adds minor formatting tweaks.

* Addresses broken relref links.

Co-authored-by: Johnny <9611008+johnnymatthews@users.noreply.github.com>

* add chiese version of Developer section

Co-authored-by: sgt_coin <91992934+SgtCoin@users.noreply.github.com>
Co-authored-by: Johnny <9611008+johnnymatthews@users.noreply.github.com>
Co-authored-by: Andy Schwab <59572086+andyschwab@users.noreply.github.com>
Co-authored-by: LexLuthr <88259624+LexLuthr@users.noreply.github.com>
Co-authored-by: Mosh <1306020+mishmosh@users.noreply.github.com>
Co-authored-by: shotcollin <virtueisitsownreward@protonmail.com>
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.

Add docs for scheduler cgroup settings
2 participants