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

docs: update CHANGELOG.md for v3.4.17 #13043

Merged
merged 2 commits into from
May 13, 2024
Merged

Conversation

github-actions[bot]
Copy link
Contributor

Automated changes by create-pull-request GitHub action

terrytangyuan and others added 2 commits May 13, 2024 01:55
@terrytangyuan terrytangyuan enabled auto-merge (squash) May 13, 2024 02:27
@terrytangyuan terrytangyuan merged commit 026b14e into main May 13, 2024
16 checks passed
@terrytangyuan terrytangyuan deleted the create-pull-request/changelog branch May 13, 2024 02:32
@agilgur5 agilgur5 changed the title docs: updated CHANGELOG.md docs: updated CHANGELOG.md for v3.4.17 May 13, 2024
@agilgur5 agilgur5 added this to the v3.4.x patches milestone May 13, 2024
Copy link
Contributor

@agilgur5 agilgur5 left a comment

Choose a reason for hiding this comment

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

@terrytangyuan @isubasinghe It looks like some fixes to 3.5 features landed in 3.4.17, which aren't applicable to 3.4. Should we revert these all for 3.4.18?

See my in-line comments below for specific ones

* [4f8dd2ee7](https://github.com/argoproj/argo-workflows/commit/4f8dd2ee7d716ba2fc9e08edd013acb66bc9494c) fix: artifact subdir error when using volumeMount (#12638)
* [3cd016b00](https://github.com/argoproj/argo-workflows/commit/3cd016b004fbc57360b8b23989fc492ae7dd4313) fix: Allow valueFrom in dag arguments parameters. Fixes #11900 (#11902)
* [c15a75b00](https://github.com/argoproj/argo-workflows/commit/c15a75b0076a6a69be0d0f0efb4c6129d3732ec5) fix(resources): improve ressource accounting. Fixes #12468 (#12492)
* [83a49b4b9](https://github.com/argoproj/argo-workflows/commit/83a49b4b9638b160c9320cd0e808179c31482ee5) fix: upgrade expr-lang. Fixes #12037 (#12573)
Copy link
Contributor

Choose a reason for hiding this comment

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

This shouldn't have been backported -- #12573 was a fix for #12037 which was only an issue in Argo 3.5 which had expr 1.15.

Argo 3.4 was on expr 1.12 which did not have the bug. This upgraded it several minors to expr 1.16 unnecessarily. And 1.16 seems to have its own bugs #13061

CHANGELOG.md Show resolved Hide resolved
CHANGELOG.md Show resolved Hide resolved
CHANGELOG.md Show resolved Hide resolved
CHANGELOG.md Show resolved Hide resolved
CHANGELOG.md Show resolved Hide resolved
agilgur5 pushed a commit that referenced this pull request May 27, 2024
Signed-off-by: GitHub <[email protected]>
Signed-off-by: Yuan Tang <[email protected]>
Co-authored-by: terrytangyuan <[email protected]>
Co-authored-by: Yuan Tang <[email protected]>
(cherry picked from commit 026b14e)
@agilgur5 agilgur5 changed the title docs: updated CHANGELOG.md for v3.4.17 docs: update CHANGELOG.md for v3.4.17 May 27, 2024
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