Skip to content

Conversation

@scj643
Copy link
Contributor

@scj643 scj643 commented Nov 17, 2025

Change summary

Add the option to allow subdirectories to be used for the git commit-archive location.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Code style update (formatting, renaming)
  • Refactoring (no functional changes)
  • Migration from an old Vyatta component to vyos-1x, please link to related PR inside obsoleted component
  • Other (please describe):

Related Task(s)

https://vyos.dev/T7940

How to test / Smoketest result

Add a git config with an extra subdirectory path after the config object.

Checklist:

  • I have read the CONTRIBUTING document
  • I have linked this PR to one or more Phabricator Task(s)
  • I have run the components SMOKETESTS if applicable
  • My commit headlines contain a valid Task id
  • My change requires a change to the documentation
  • I have updated the documentation accordingly

@github-actions
Copy link

github-actions bot commented Nov 17, 2025


PR title does not match the required format

@github-actions
Copy link

github-actions bot commented Nov 17, 2025

All contributors have signed the CLA ✍️ ✅
Posted by the CLA Assistant Lite bot.

@scj643
Copy link
Contributor Author

scj643 commented Nov 17, 2025

I have read the CLA Document and I hereby sign the CLA

vyosbot added a commit to vyos/vyos-cla-signatures that referenced this pull request Nov 17, 2025
@scj643 scj643 force-pushed the git-archive-subdir branch 2 times, most recently from 174d8c7 to f11fcd6 Compare November 17, 2025 16:20
@scj643 scj643 force-pushed the git-archive-subdir branch from f11fcd6 to b3a3a04 Compare November 17, 2025 16:21
@github-actions
Copy link

CI integration 👍 passed!

Details

CI logs

  • CLI Smoketests (no interfaces) 👍 passed
  • CLI Smoketests VPP 👍 passed
  • CLI Smoketests (interfaces only) 👍 passed
  • Config tests 👍 passed
  • Config tests VPP 👍 passed
  • RAID1 tests 👍 passed
  • TPM tests 👍 passed

@scj643
Copy link
Contributor Author

scj643 commented Nov 24, 2025

I think this will need to actually become https://vyos.dev/T6304 in order to actually make it easy to implement.

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

Labels

Development

Successfully merging this pull request may close these issues.

1 participant