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

Enable job profiling with influxdb #38

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

jamesbeedy
Copy link

Short howto on enabling job profiling with influxdb.

Depends on charmed-hpc/slurm-charms#70

@jamesbeedy jamesbeedy requested a review from a team as a code owner February 12, 2025 22:52
@jamesbeedy jamesbeedy requested review from AshleyCliff and removed request for a team February 12, 2025 22:52
Copy link
Contributor

@AshleyCliff AshleyCliff left a comment

Choose a reason for hiding this comment

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

Thanks for the PR! Just a few grammar changes to get your approval on and it'll be good to go!

@@ -15,5 +15,6 @@ See the how-to guides in this section for the steps to set up and deploy various

Deploy Slurm <deploy-slurm>
Deploy a shared filesystem <deploy-shared-filesystem>
Enable job profiling with InfluxDB <enable-job-profiling-with-influxdb>
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Enable job profiling with InfluxDB <enable-job-profiling-with-influxdb>
Enable job profiling <enable-job-profiling-with-influxdb>

Copy link
Contributor

Choose a reason for hiding this comment

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

Just a bit too wordy for the sidebar

(howto-enable-job-profiling-with-influxdb)=
# How to enable job profiling with InfluxDB

Charmed-HPC integrates with the [InfluxDB Charm](https://charmhub.io/influxdb) to enable job profiling in slurm.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Charmed-HPC integrates with the [InfluxDB Charm](https://charmhub.io/influxdb) to enable job profiling in slurm.
Charmed HPC can integrate with the [InfluxDB Charm](https://charmhub.io/influxdb) to enable job profiling in Slurm.


Charmed-HPC integrates with the [InfluxDB Charm](https://charmhub.io/influxdb) to enable job profiling in slurm.

This how-to guide explains how enable job profiling by deploying and integrating `influxdb` with charmed-hpc.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
This how-to guide explains how enable job profiling by deploying and integrating `influxdb` with charmed-hpc.
This guide explains how to enable job profiling by deploying and integrating `influxdb` with charmed-hpc.


## Deploy and Integrate InfluxDB

InfluxDB can be deployed using Juju in a single command.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
InfluxDB can be deployed using Juju in a single command.
InfluxDB can be deployed using Juju in a single command:

$ juju deploy influxdb
:::

Now [integrate](https://canonical-juju.readthedocs-hosted.com/en/latest/user/reference/juju-cli/list-of-juju-cli-commands/integrate/) the newly deployed influxdb charm with slurmctld.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Now [integrate](https://canonical-juju.readthedocs-hosted.com/en/latest/user/reference/juju-cli/list-of-juju-cli-commands/integrate/) the newly deployed influxdb charm with slurmctld.
Now [integrate](https://canonical-juju.readthedocs-hosted.com/en/latest/user/reference/juju-cli/list-of-juju-cli-commands/integrate/) the newly deployed influxdb charm with slurmctld:

$ juju integrate influxdb slurmctld
:::

Once the InfluxDB charm deployment and integration are complete slurm will be configured to send job profiling metrics to influxdb, enabling the use of the [`sstat`](https://slurm.schedmd.com/sstat.html) command.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Once the InfluxDB charm deployment and integration are complete slurm will be configured to send job profiling metrics to influxdb, enabling the use of the [`sstat`](https://slurm.schedmd.com/sstat.html) command.
Once the InfluxDB charm deployment and integration are complete, Slurm is configured to send job profiling metrics to influxdb, enabling the use of the [`sstat`](https://slurm.schedmd.com/sstat.html) command.

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.

2 participants