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

fix(argo-workflows): revert BASE_HREF change #2770

Merged
merged 2 commits into from
Jun 17, 2024

Conversation

agilgur5
Copy link

@agilgur5 agilgur5 commented Jun 17, 2024

Reverts my own #2756
Fixes argoproj/argo-workflows#13199 (comment)

Sorry, I screwed this up and caused a regression. Didn't re-read my own release notes from argoproj/argo-workflows#12653 (I wrote them 4 months ago to be fair) where ARGO_BASE_HREF would not work previously as it was inconsistently ARGO_BASEHREF (missing an underscore as the flag was missing a dash --basehref).

So there is actually no forward compatible change possible, and as such, just revert #2756.
I probably realized this when I wrote argoproj/argo-workflows#12653 (comment) and didn't make a PR downstream here at the time (or earlier, when I first wrote the PR) and then forgot about that 4 months later (once I had approvals to merge).
That's my bad, sorry; I was trying to be proactive and instead caused a regression 😞

Note that we will have to reinstate this change when it comes time to make the 3.6 upgrade

Checklist:

  • I have bumped the chart version according to versioning
  • [n/a] I have updated the documentation according to documentation
  • I have updated the chart changelog with all the changes that come with this pull request according to changelog.
  • [n/a] Any new values are backwards compatible and/or have sensible default.
  • I have signed off all my commits as required by DCO.
  • My build is green (troubleshooting builds).

Anton Gilgur added 2 commits June 17, 2024 13:40
This reverts commit 134cecd.

Sorry, I screwed up this and caused a regression. Didn't read my own release notes (from 4 months ago to be fair) where `ARGO_BASE_HREF` would not work previously as it was inconsistently `ARGO_BASEHREF`. So there is no forward compatible change to be made here, so just revert this.

Will have to reinstate this change for the 3.6 upgrade

Signed-off-by: Anton Gilgur <agilgur5@gmail.com>
Signed-off-by: Anton Gilgur <agilgur5@gmail.com>
Copy link
Collaborator

@yu-croco yu-croco left a comment

Choose a reason for hiding this comment

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

Thank you!

@yu-croco yu-croco merged commit fab2809 into argoproj:main Jun 17, 2024
10 checks passed
@agilgur5 agilgur5 deleted the fix-workflows-revert-base-href-env branch June 17, 2024 20:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Regression: ARGO_BASE_HREF env var doesn't work
3 participants