Skip to content

DAOS-18706,18693 control: Remove shadow variable#17741

Open
kjacque wants to merge 1 commit intorelease/2.6from
kjacque/2.6/daos-pool-list-shadow
Open

DAOS-18706,18693 control: Remove shadow variable#17741
kjacque wants to merge 1 commit intorelease/2.6from
kjacque/2.6/daos-pool-list-shadow

Conversation

@kjacque
Copy link
Contributor

@kjacque kjacque commented Mar 19, 2026

'pool' was created as a new variable and didn't update the variable at the higher scope. The higher scoped version of the variable was left as nil.

Features: pool

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).

'pool' was created as a new variable and didn't update the
variable at the higher scope. The higher scoped version of
the variable was left as nil.

Features: pool

Signed-off-by: Kris Jacque <kris.jacque@hpe.com>
@kjacque kjacque requested review from a team as code owners March 19, 2026 22:57
@kjacque kjacque self-assigned this Mar 19, 2026
@github-actions
Copy link

@kjacque kjacque added release-2.6.5 approved-to-merge PR has received release branch merge approval labels Mar 19, 2026
@daosbuild3
Copy link
Collaborator

Test stage Functional Hardware Medium Verbs Provider completed with status UNSTABLE. https://jenkins-3.daos.hpc.amslabs.hpecorp.net/job/daos-stack/job/daos//view/change-requests/job/PR-17741/2/testReport/

@daosbuild3
Copy link
Collaborator

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved-to-merge PR has received release branch merge approval release-2.6.5

Development

Successfully merging this pull request may close these issues.

5 participants