Skip to content

COPY DEFAULT - Copy default folder at root level of operations repo #18

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.

Sign up for GitHub

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

Jump to bottom
Closed
PhillypHenning opened this issue Nov 11, 2022 · 0 comments · Fixed by #17
Closed

COPY DEFAULT - Copy default folder at root level of operations repo #18

PhillypHenning opened this issue Nov 11, 2022 · 0 comments · Fixed by #17
Assignees
@PhillypHenning
Labels
bug Something isn't working

Comments

@PhillypHenning
Copy link
Contributor

PhillypHenning commented Nov 11, 2022 •
edited
Loading

The issue is well detailed in the following PR #17

To make it brief;
Currently the defaults are being copied from the operations repo environment folder. It should be copied from the root of the operations repo AND it should only match up by chart name.

Current behavior

ops_repo
  envA/
    _default/
    chartA/  <-- default folders would be merged into this chart folder
    chartB/  <-- default folders would be merged into this chart folder
      ...

Expected Behaviour

ops_repo/
  _default/
    helm/ 
      chartA/  <--- Copied into ANY environment depoying the `chartA` chart
  envA/ 
    helm/ 
      chartA/  <--- Matches a chart within the `_defaults` folder which will be merged
      chartB/  <--- Does not match a chart and so will not merge any default folders
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@PhillypHenning PhillypHenning

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix the _default copy behavior to use the correct path bitops-plugins/helm
1 participant
@PhillypHenning