Skip to content

Conversation

@epmartini
Copy link
Contributor

Relates SUITEDEV-39317

@epmartini epmartini force-pushed the SUITEDEV-39317-alloydb branch from c20abce to ce145a5 Compare November 27, 2025 07:33
Relates SUITEDEV-39317
@epmartini epmartini force-pushed the SUITEDEV-39317-alloydb branch from ce145a5 to 397f6f8 Compare December 1, 2025 08:00
@epmartini epmartini marked this pull request as ready for review December 1, 2025 09:37
@epmartini epmartini requested a review from edosrecki as a code owner December 1, 2025 09:37
Copy link
Owner

@edosrecki edosrecki left a comment

Choose a reason for hiding this comment

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

Since this is a breaking change to configuration it is going to cause the tool to fail for all existing users after they update.

Please add versioning to configuration.json file, and exit the app automatically if current configuration is not supported, and tell the user to run the migration.

@epmartini
Copy link
Contributor Author

epmartini commented Dec 2, 2025

Good point, now it is updating the old configuration to the new format

@epmartini epmartini requested a review from edosrecki December 2, 2025 10:18
@edosrecki edosrecki changed the base branch from master to alloydb January 10, 2026 08:22
@edosrecki edosrecki merged commit 71f29fe into edosrecki:alloydb Jan 10, 2026
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.

4 participants