Skip to content

DAOS-18046 control: Remove conflicting -o param (#18038)#18229

Merged
daltonbohning merged 1 commit into
release/2.8from
kjacque/2.8/dmg-get-acl-config
May 14, 2026
Merged

DAOS-18046 control: Remove conflicting -o param (#18038)#18229
daltonbohning merged 1 commit into
release/2.8from
kjacque/2.8/dmg-get-acl-config

Conversation

@kjacque
Copy link
Copy Markdown
Contributor

@kjacque kjacque commented May 12, 2026 •
edited
Loading

The --outfile parameter for dmg pool get-acl was aliased to -o, which conflicted with the config file parameter.

Steps for the author:

  • Commit message follows the guidelines.
  • Appropriate Features or Test-tag pragmas were used.
  • Appropriate Functional Test Stages were run.
  • At least two positive code reviews including at least one code owner from each category referenced in the PR.
  • Testing is complete. If necessary, forced-landing label added and a reason added in a comment.

After all prior steps are complete:

  • Gatekeeper requested (daos-gatekeeper added as a reviewer).

@kjacque kjacque requested review from knard38, mjmac and tanabarr May 12, 2026 15:47
@kjacque kjacque self-assigned this May 12, 2026
@kjacque kjacque requested review from a team as code owners May 12, 2026 15:47
@kjacque kjacque added clean-cherry-pick Cherry-pick from another branch that did not require additional edits approved-to-merge PR has received release branch merge approval release-2.8 labels May 12, 2026
@kjacque
DAOS-18046 control: Remove conflicting -o param (#18038)
f7f4ace 
The --outfile parameter for dmg pool get-acl was aliased to -o,
which conflicted with the config file parameter.

Features: control

Signed-off-by: Kris Jacque <kris.jacque@hpe.com>
@kjacque kjacque force-pushed the kjacque/2.8/dmg-get-acl-config branch from 110f1e5 to f7f4ace Compare May 12, 2026 15:48
@kjacque
Copy link
Copy Markdown
Contributor Author

kjacque commented May 12, 2026

Just force-pushed to add the pragma. This is a clean cherry-pick.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 12, 2026

Ticket title is 'dmg pool get-acl does not work in insecure mode'
Status is 'Awaiting backport'
Job should run at elevated priority (1)
https://daosio.atlassian.net/browse/DAOS-18046

@github-actions github-actions Bot added the priority Ticket has high priority (automatically managed) label May 12, 2026
@daosbuild3
Copy link
Copy Markdown
Collaborator

daosbuild3 commented May 14, 2026

Test stage Functional Hardware Medium MD on SSD completed with status FAILURE. https://jenkins-3.daos.hpc.amslabs.hpecorp.net//job/daos-stack/job/daos/view/change-requests/job/PR-18229/2/execution/node/1299/log

@kjacque
Copy link
Copy Markdown
Contributor Author

kjacque commented May 14, 2026

Functional hardware failure is a known issue: https://daosio.atlassian.net/browse/DAOS-18733

@kjacque kjacque added the forced-landing The PR has known failures or has intentionally reduced testing, but should still be landed. label May 14, 2026
@kjacque kjacque requested a review from a team May 14, 2026 15:16
@daltonbohning daltonbohning merged commit b313863 into release/2.8 May 14, 2026
44 of 46 checks passed
@daltonbohning daltonbohning deleted the kjacque/2.8/dmg-get-acl-config branch May 14, 2026 15:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mjmac mjmac mjmac approved these changes

@tanabarr tanabarr tanabarr approved these changes

@knard38 knard38 knard38 approved these changes

Assignees

@kjacque kjacque

Labels

approved-to-merge PR has received release branch merge approval clean-cherry-pick Cherry-pick from another branch that did not require additional edits forced-landing The PR has known failures or has intentionally reduced testing, but should still be landed. priority Ticket has high priority (automatically managed) release-2.8

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@kjacque @daosbuild3 @mjmac @tanabarr @knard38 @daltonbohning