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

[TECH DEBT] Update os.listdir to using os.walk for Git Pillar data traversal #67722

Closed
dmurphy18 opened this issue Feb 10, 2025 · 1 comment
Closed
Assignees

Comments

@dmurphy18
Copy link
Contributor

Description of the tech debt to be addressed, include links and screenshots

Need to implement os.walk for handling traversal of Git Pillar branch names with a '/' slash in the Git branch name,

See Issue #65467 (comment), and PR #66954

Was about to try this approach after proving it's possibility in a test program, on the Salt 3006.x branch when the PR was merged, given it had passed all tests, and was ready for merger, and marked for the 3006.10 milestone, and that milestone is close to release.

Hence tech-debt to improve as suggested in the Issue link above

Versions Report

(Provided by running salt --versions-report. Please also mention any differences in master/minion versions.)

@dmurphy18 dmurphy18 self-assigned this Feb 10, 2025
@dmurphy18 dmurphy18 changed the title [TECH DEBT] Update os.listdir to using os.walk for Git Pillar data tranversal [TECH DEBT] Update os.listdir to using os.walk for Git Pillar data traversal Feb 10, 2025
@dmurphy18 dmurphy18 added this to the Sulfur v3006.10 milestone Feb 13, 2025
@dmurphy18
Copy link
Contributor Author

Closing since PR #67723 is merged.

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

No branches or pull requests

1 participant