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

chore(helm): Make release pipeline to update golden files #916

Merged
merged 1 commit into from
Mar 8, 2023

Conversation

Pothulapati
Copy link
Contributor

Golden files need to be updated as version is updated in the Chart.yaml file. This commit adds the step to update the golden files in the release pipeline.

To make that possible, The following additional changes are also required:

  • Go.Work: This adds support multi-module workspaces as we will have the tests along with the operator in sub-folders. This is required to run go cmds from the root directory. (like running tests in this case).
  • .helmignore: This updates the Helm chart to ignore the /ci folder during helm packaging as those are not required for users of the chart.

@Pothulapati Pothulapati force-pushed the tar/release-chart branch 2 times, most recently from 408556a to 30dc1b3 Compare March 6, 2023 17:04
@romange romange requested a review from ashotland March 6, 2023 17:40
@ashotland
Copy link
Contributor

Need to update the branch

Golden files need to be updated as version is updated in the
`Chart.yaml` file. This commit adds the step to update the golden
files in the release pipeline.

To make that possible, The following additional changes are also
required:
- `Go.Work`: This adds support multi-module projects as we will
   have the tests along with the operator in sub-folders. This
   is required to run go cmds from the root directory. (like
   running tests in this case).
- `.helmignore`: This updates the Helm chart to ignore the `/ci`
    folder during helm packaging as those are not required for
    users of the chart.
@Pothulapati
Copy link
Contributor Author

@ashotland Updated!

@Pothulapati Pothulapati merged commit 5c57e4e into main Mar 8, 2023
@romange romange deleted the tar/release-chart branch March 12, 2023 06:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants