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

Changes for 1.20.3 #549

Merged
merged 3 commits into from
Apr 11, 2024
Merged

Changes for 1.20.3 #549

merged 3 commits into from
Apr 11, 2024

Conversation

lengau
Copy link
Contributor

@lengau lengau commented Apr 11, 2024

  • Have you followed the guidelines for contributing?
  • Have you signed the CLA?
  • Have you successfully run tox?

This is ready so we can do a rebase.

@lengau lengau changed the base branch from main to hotfix/1.20 April 11, 2024 16:45
@lengau lengau requested review from syu-w and mr-cal April 11, 2024 16:45
@lengau lengau changed the title 1.20 fix cache directories Changes for 1.20.3 Apr 11, 2024
Copy link
Collaborator

@mr-cal mr-cal left a comment

Choose a reason for hiding this comment

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

What do you think about bumping the compatibility tag to v8?

v7 is used by 1.23, so theoretically someone could create an instance with 1.20.3, upgrade to 1.23.x and the same instance would be reused (which we don't want).

I'm happy to do a compatibility tag bump commit, it's more tedious than it should be and I've gotten pretty quick at it.

@mr-cal
Copy link
Collaborator

mr-cal commented Apr 11, 2024

I've gotten pretty quick at it.

Meaning I run git diff 69207f7~ 69207f7 --name-only | xargs sed -i 's/v6/v7/g'

@lengau lengau force-pushed the 1.20-fix-cache-directories branch from 54e05a6 to 21f58f8 Compare April 11, 2024 17:36
@lengau
Copy link
Contributor Author

lengau commented Apr 11, 2024

Thanks @mr-cal - I missed the v7 in 1.23. I've updated to v8 now

@lengau lengau requested a review from mr-cal April 11, 2024 17:37
Copy link
Collaborator

@mr-cal mr-cal left a comment

Choose a reason for hiding this comment

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

Thanks!

@mr-cal
Copy link
Collaborator

mr-cal commented Apr 11, 2024

Integration test failures are expected on this older branch. The tests are trying to launch 23.04 (lunar), which is no longer supported.

@mr-cal mr-cal merged commit 963408a into hotfix/1.20 Apr 11, 2024
9 of 13 checks passed
@mr-cal mr-cal deleted the 1.20-fix-cache-directories branch April 11, 2024 19:15
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.

3 participants