-
Notifications
You must be signed in to change notification settings - Fork 1
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
Feat: automate releases #74
base: main
Are you sure you want to change the base?
Conversation
Fix lint in release-source.yml Fix lint in release-source.yml Fix lint in release-source.yml Fix version validation error in release-source.yml Fix lint Fix lint in release-source.yml Remove empty line in release-source workflow Update release_version input to be required in release-source.yml Update release-source workflow Update release-source workflow inputs Update release-source workflow to use workflow_dispatch event Update release-source workflow Fix file_release_version extraction in release-source.yml Add version increment logic Fix version comparison bug Fix version validation in release-source.yml extend version comparison logic in release-source.yml Fix version comparison bug in release-source.yml Fix version comparison bug in release-source.yml Fix version comparison bug in release-source.yml Fix version comparison bug in release-source.yml Fix version comparison bug in release-source.yml Fix version comparison bug in release-source.yml Fix version comparison bug in release-source.yml Fix version comparison bug in release-source.yml
Fix typo fix helm dep fix helm dep fix helm build fix helm build add chartmuseum token add chartmuseum Add CHARTMUSEUM_TOKEN to environment variables chore: Update Chart.yaml with release version for marvinjs fix typo fix typo fix lint
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me, thank you for the implementation!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We are testing this action currently in reprovisyn and will post the errors one by one until it succeeds. 😉
.github/workflows/release-source.yml
Outdated
git commit -m "Update package.json version to $RELEASE_VERSION" | ||
git push origin release-"$RELEASE_VERSION" | ||
- name: Add Helm repository | ||
run: helm repo add datavisyn --username admin --password ${{ env.CHARTMUSEUM_TOKEN }} https://chartmuseum.app.datavisyn.io |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Testing this action in reprovisyn fails with the following error:
Error: "helm repo add" requires 2 arguments
Usage: helm repo add [NAME] [URL] [flags]
Error: Process completed with exit code 1.
https://github.com/datavisyn/reprovisyn/actions/runs/9765252351/job/26955506135
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fix committed
@dvviktordelev We tested the branch with marketplace360 and it worked very well 🎉 thank you! When you have time, please continue with automating the remaining steps:
Please let me know if you need any further details. Thank you! |
This is based on previous PR
it adds dynamic creation of release notes (= list of closed PRs since last release)
closes https://github.com/datavisyn/infrastructure-k8s/issues/241