Skip to content

DAOS-19244 packaging: create /var/daos writable by daos_server#18626

Merged
phender merged 3 commits into
masterfrom
michael-hennecke/2.9/DAOS-19244
Jul 9, 2026
Merged

DAOS-19244 packaging: create /var/daos writable by daos_server#18626
phender merged 3 commits into
masterfrom
michael-hennecke/2.9/DAOS-19244

Conversation

@Michael-Hennecke

@Michael-Hennecke Michael-Hennecke commented Jul 8, 2026

Copy link
Copy Markdown
Collaborator

Add a /var/daos directory that is writable for daos_server, which can then be used as a persistent location for control_meta and the mountpounts for the SCM/tmpfs filesystems (which should not be placed into /tmp).

Add a /var/daos directory that is writeable for daos_server,
which can then be used as a persistent location for control_meta
and the mountpounts for the SCM/tmpfs filesystems (which should
not be placed into /tmp).

Signed-off-by: Michael Hennecke <michael.hennecke@hpe.com>
@github-actions

github-actions Bot commented Jul 8, 2026

Copy link
Copy Markdown

Ticket title is '/tmp/<cluster_name>/logs/control_metadata/daos_control directory is not available after node reboot on aurora (DAOS nodes) [MDonSSD]'
Status is 'Open'
Labels: 'aurora_post_at,md_on_ssd,scrubbed_2.8,test_2.8.0rc'
https://daosio.atlassian.net/browse/DAOS-19244

Michael Hennecke added 2 commits July 8, 2026 12:02
fix typo

Signed-off-by: Michael Hennecke <michael.hennecke@hpe.com>
update daos.spec with changes from daos.sh

Signed-off-by: Michael Hennecke <michael.hennecke@hpe.com>

@tanabarr tanabarr left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

should we change any default locations within the product code based on this change?

@Michael-Hennecke Michael-Hennecke changed the title DAOS-19244 packaging: create /var/daos writeable by daos_server DAOS-19244 packaging: create /var/daos writable by daos_server Jul 8, 2026
@Michael-Hennecke

Copy link
Copy Markdown
Collaborator Author

should we change any default locations within the product code based on this change?

yes, after this lands we should change the server.yml example files to use this persistent location. i would do this in a separate PR. for control_meta, i thik there is no default in the code itself, correct?

@Michael-Hennecke Michael-Hennecke requested a review from a team July 8, 2026 22:37
@Michael-Hennecke

Copy link
Copy Markdown
Collaborator Author

please land - trivy scan error is unrelated.

@phender phender left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

For changes like this we should be adding tests to confirm the change - ci/rpm/test_daos_node.sh would be a good place to verify the /var/daos exists, is owned by the correct user, and persists or is removed upon RPM removal.

@phender phender requested a review from a team July 9, 2026 15:19
@phender phender merged commit 1950031 into master Jul 9, 2026
42 of 43 checks passed
@phender phender deleted the michael-hennecke/2.9/DAOS-19244 branch July 9, 2026 15:26
@Michael-Hennecke

Copy link
Copy Markdown
Collaborator Author

For changes like this we should be adding tests to confirm the change - ci/rpm/test_daos_node.sh would be a good place to verify the /var/daos exists, is owned by the correct user, and persists or is removed upon RPM removal.

Makes sense for the creation, although i'd point out that we do not have a check for the /var/log/daos creation either ;-)
so we should add a test for that as well...

For the RPM uninstall, my personal take would be to not remove that directory, it may contain other data relevant to the admin that they do not want removed.

phender pushed a commit that referenced this pull request Jul 9, 2026
…) (#18629)

Add a /var/daos directory that is writable for daos_server,
which can then be used as a persistent location for control_meta
and the mountpounts for the SCM/tmpfs filesystems (which should
not be placed into /tmp).

Signed-off-by: Michael Hennecke <michael.hennecke@hpe.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

5 participants