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: upgradeability tests #1529

Merged

Conversation

wolf4ood
Copy link
Contributor

@wolf4ood wolf4ood commented Sep 3, 2024

WHAT

upgradeability tests

WHY

upgradeability tests are failing on `main

FURTHER NOTES

To fix that we have to change the V0_0_1__Init_Dataplaneinstance.sql to be equal to the one in 0.7.4. By doing so the migration from 0.8.0-rc2 to the next RC version 0.8.0-rc3 will fail instead.

Closes #1528

@wolf4ood wolf4ood force-pushed the fix/1528_upgradeability_tests branch from fe2a209 to 80a1dd3 Compare September 3, 2024 09:21
@wolf4ood wolf4ood force-pushed the fix/1528_upgradeability_tests branch from 80a1dd3 to 6d119fc Compare September 3, 2024 09:56
Copy link

sonarqubecloud bot commented Sep 3, 2024

@wolf4ood wolf4ood marked this pull request as ready for review September 3, 2024 10:27
@wolf4ood wolf4ood requested a review from ndr-brt September 3, 2024 10:27
Copy link
Contributor

@ndr-brt ndr-brt left a comment

Choose a reason for hiding this comment

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

I'm not sure what happened, so it won't be possible to update 0.7.4 to 0.8.0?

@wolf4ood
Copy link
Contributor Author

wolf4ood commented Sep 3, 2024

this is a fix for migration from 0.7.4 to future 0.8.0, we don't have yet a final 0.8.0 just RCs.

We backported the migration here for the data plane instance store but the init migration file differs from the one in main, hence the test upgradeability 0.7.4 -> main fails

@wolf4ood wolf4ood merged commit 273f3aa into eclipse-tractusx:main Sep 3, 2024
33 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Merged
Development

Successfully merging this pull request may close these issues.

Upgradeability tests broken on main
2 participants