-
Notifications
You must be signed in to change notification settings - Fork 106
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
Argo Workflows 3.6 release changes #1215
Labels
type:dependency-upgrade
A dependency upgrade
Comments
4 tasks
elliotgunton
added a commit
that referenced
this issue
Sep 30, 2024
**Pull Request Checklist** - [x] Fixes `make test` - [x] Tests added - [x] Documentation/examples added - [x] [Good commit messages](https://cbea.ms/git-commit/) and/or PR title **Description of PR** New examples were added for Argo Workflows 3.6, but we don't yet support the fields (until 3.6 official release, then #1215 will address this) - for now rename the `synchronization` examples (add `legacy` suffix). This PR also tidies up the `make` targets to avoid running unrelated code (`regenerate-test-data`) and separate out lint checks from the main `test` target for faster iteration speed (added `test-lint` to `ci`). --------- Signed-off-by: Elliot Gunton <[email protected]>
4 tasks
7 tasks
This was referenced Dec 12, 2024
Hey @elliotgunton , There's another change in 3.6 that I am looking forward to, but I don't see listed here: I hope this functionality will be included in Hera as well. Unfortunately I don't have time to contribute it myself. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
When 3.6 releases there will be new fields for some classes, and various new examples. This issue is to update the codebase once the official 3.6 release is made (currently rc1).
synchronization
field in the spec as we don't have a Hera wrapper for this)when
- extra field in CronWorkflowschedule
->schedules
- extra field in CronWorkflow, makeschedule: Optional
pod_gc
onWorkflowTemplate
is not rendered correctly) #1292The text was updated successfully, but these errors were encountered: