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

doc: add example error message when creating DB after starting systemd service #485

Merged
merged 1 commit into from
Aug 23, 2024

Conversation

cmoussa1
Copy link
Member

Problem

The flux-accounting service depends on the creation of the flux-accounting DB before starting since it needs to establish a connection to the DB in order to run flux account commands. If this is done out of order, confusing behavior can result from running the commands, such as an attempt to write to a readonly database error.


This PR just adds a note to the flux-accounting guide about making sure the flux-accounting DB has been created before starting the flux-accounting service and gives an example of the error message that can potentially occur.

Problem: The flux-accounting service depends on the creation of the
flux-accounting DB before starting since it needs to establish a
connection to the DB in order to run "flux account" commands. If this is
done out of order, confusing behavior can result from running the
commands, such as an "attempt to write to a readonly database" error.

Add a note to the flux-accounting guide about making sure the
flux-accounting DB has been created before starting the flux-accounting
service and give an example of the error message that can potentially
occur.
@cmoussa1 cmoussa1 added the documentation Improvements or additions to documentation label Aug 23, 2024
@cmoussa1 cmoussa1 requested a review from wihobbs August 23, 2024 15:48
Copy link
Member

@wihobbs wihobbs left a comment

Choose a reason for hiding this comment

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

Note that you can use `` instead of "" in commit messages if you backslash out of them or open the editor. However that's IMO personal preference.

Thanks for getting this plumbed correctly (and on short notice) on fluke yesterday! :)

@cmoussa1
Copy link
Member Author

TIL! Thanks for the commit message tip. Glad we got it figured out. Setting MWP here

@mergify mergify bot merged commit eea54d9 into flux-framework:master Aug 23, 2024
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation merge-when-passing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants