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

Improve deploy-data flexibility to deploy files from other checkout in same config file #462

Merged
merged 6 commits into from
Jun 12, 2024

Conversation

tlvu
Copy link
Collaborator

@tlvu tlvu commented Jun 7, 2024

Overview

This is a non breaking change.

Given the config file can specify multiple checkouts, this flexibility to
have SRC_DIR be an absolute path will allow one checkout to take files from
other checkouts, using absolute path to the other checkouts. SRC_DIR can
still be a relative path of the current checkout, as before, to preserve
backward-compatibility.

Possible use-case: re-organize the layout of various files from the various
checkouts in an intermediate location before rsyncing this intermediate
location to the final destination.

CI Operations

birdhouse_daccs_configs_branch: master
birdhouse_skip_ci: false

@github-actions github-actions bot added ci/deployment Related to deployment utilities and scripts documentation Improvements or additions to documentation labels Jun 7, 2024
Copy link
Collaborator

@fmigneault fmigneault left a comment

Choose a reason for hiding this comment

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

Change looks ok. I suggest improving the changelog description.

CHANGES.md Show resolved Hide resolved
Copy link
Collaborator

@mishaschwartz mishaschwartz left a comment

Choose a reason for hiding this comment

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

I agree with @fmigneault. The change looks good but the description needs some clarification

@tlvu
Copy link
Collaborator Author

tlvu commented Jun 10, 2024

Improved the changelog, let me know if it's clearer.

@tlvu tlvu merged commit 9d9f46c into master Jun 12, 2024
3 of 4 checks passed
@tlvu tlvu deleted the improve-deploy-data-flexibility branch June 12, 2024 20:14
@github-actions github-actions bot added the ci/operations Continuous Integration components label Jun 12, 2024
tlvu added a commit to bird-house/birdhouse-deploy-ouranos that referenced this pull request Jun 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci/deployment Related to deployment utilities and scripts ci/operations Continuous Integration components documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants