Skip to content

CLOUDP-299786: Removed helm-charts as submodule. Added helm-charts as a simple folder #2209

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

Merged
merged 5 commits into from
Mar 21, 2025

Conversation

igor-karpukhin
Copy link
Collaborator

All Submissions:

Removed helm-charts as submodule. Added helm-charts as a simple folder.

  • Have you signed our CLA?
  • Put closes #XXXX in your comment to auto-close the issue that your PR fixes (if there is one).
  • Update docs/release-notes/release-notes-template.md if your changes should be included in the release notes for the next release.

@igor-karpukhin igor-karpukhin added the cloud-tests Run expensive Cloud Tests: Integration & E2E label Mar 18, 2025
Comment on lines +6 to +9
# TODO: remove after the job "Sync helm-charts" is applied to the main branch
# pull_request:
# types:
# - closed
Copy link
Collaborator

Choose a reason for hiding this comment

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

q: we can remove already right?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Not yet. I need to make sure the new workflow works as intended. Then we can remove the old workflow

@igor-karpukhin igor-karpukhin merged commit b500d06 into main Mar 21, 2025
60 checks passed
@igor-karpukhin igor-karpukhin deleted the CLOUDP-299786/move-helm-charts branch March 21, 2025 15:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cloud-tests Run expensive Cloud Tests: Integration & E2E
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants