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

multitenant: TestTenantUpgradeFailure fails when min version is bumped to 23.1 #112209

Closed
RaduBerinde opened this issue Oct 12, 2023 · 5 comments
Closed
Assignees
Labels
branch-master Failures and bugs on the master branch. C-test-failure Broken test (automatically or manually discovered). skipped-test T-multitenant Issues owned by the multi-tenant virtual team

Comments

@RaduBerinde
Copy link
Member

RaduBerinde commented Oct 12, 2023

TestTenantUpgradeFailure seems to rely on a release version existing strictly between the minimum supported version and the current version. This is no longer true once we bump the minimum supported version to 23.1 (for the 23.2 release).

Jira issue: CRDB-32295

@RaduBerinde RaduBerinde added C-test-failure Broken test (automatically or manually discovered). branch-master Failures and bugs on the master branch. release-blocker Indicates a release-blocker. Use with branch-release-2x.x label to denote which branch is blocked. skipped-test branch-release-23.2 Used to mark GA and release blockers, technical advisories, and bugs for 23.2 labels Oct 12, 2023
@yuzefovich yuzefovich self-assigned this Oct 16, 2023
@celiala celiala added the T-multitenant Issues owned by the multi-tenant virtual team label Oct 16, 2023
@yuzefovich
Copy link
Member

Indeed, based on the last time we adjusted this test, it assumes that there exists such a v1 that v0 < v1 < v2, and v1 and v2 only differ in Internal whereas v0 is the previous "major" release version. This state implies that we need to have V23_2 and V24_1Start defined, and then the test will work (which I verified).

I don't see an easy way to refactor this unit test to remove this assumption (but I'm not familiar with the upgrades). To me it seems like this test can be skipped until V23_2 = (23 : 2 : 0) and V24_1Start = (23 : 2 : 2) are defined. cc @ajstorm @stevendanna in case you have opinions.

@RaduBerinde
Copy link
Member Author

Cool, I will reenable on master when I bump the version.

@RaduBerinde RaduBerinde removed release-blocker Indicates a release-blocker. Use with branch-release-2x.x label to denote which branch is blocked. branch-release-23.2 Used to mark GA and release blockers, technical advisories, and bugs for 23.2 labels Oct 17, 2023
@yuzefovich
Copy link
Member

Thanks Radu, I'll assign the issue to you then.

@yuzefovich yuzefovich assigned RaduBerinde and unassigned yuzefovich Oct 18, 2023
@stevendanna
Copy link
Collaborator

@RaduBerinde It looks like maybe this was fixed here: f74fff67fdee#diff-47be6646fc129bea75b3b9999532a6dafb4f609cdbdb23234bd308f5fd2e4364L408 or am I misunderstanding?

@RaduBerinde
Copy link
Member Author

Right, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
branch-master Failures and bugs on the master branch. C-test-failure Broken test (automatically or manually discovered). skipped-test T-multitenant Issues owned by the multi-tenant virtual team
Projects
None yet
Development

No branches or pull requests

4 participants