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

docs(performance-tips): added steps to deploy io engine deamonset #160

Merged
merged 7 commits into from
Aug 31, 2023

Conversation

AJDatacore
Copy link
Contributor

No description provided.

@@ -93,17 +93,46 @@ cat /sys/devices/system/cpu/isolated
{% endtab %}
{% endtabs %}

### Deploy Mayastor daemonset
### Deploy IO-engine daemonset
Copy link
Contributor

Choose a reason for hiding this comment

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

There's no deploy of the daemonset now, it's all done via helm or kubectl edit.
Should we also show the helm command to change the coreList? CC @niladrih

Suggested change
### Deploy IO-engine daemonset
### Modify the io-engine daemonset

Copy link
Member

Choose a reason for hiding this comment

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

The core list set command for installation and upgrade would be a good idea. I'll test this one out, and post here.

Copy link
Member

@niladrih niladrih Aug 28, 2023

Choose a reason for hiding this comment

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

Install command argument: --set 'io_engine.coreList={30,31}'
Upgrade command argument: --set-args 'io_engine.coreList={30,31}'

Need to have the single quotes around it.

Copy link
Member

@niladrih niladrih left a comment

Choose a reason for hiding this comment

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

Left one comment.

quickstart/performance-tips.md Outdated Show resolved Hide resolved
Signed-off-by: AJ_Datacore <[email protected]>
Copy link
Member

@niladrih niladrih left a comment

Choose a reason for hiding this comment

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

Looks good

@AJDatacore AJDatacore merged commit e6c1803 into develop Aug 31, 2023
2 checks passed
@niladrih niladrih deleted the deploy-etcd-change branch May 20, 2024 16:26
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.

3 participants